fix(frontend): Manager — volles Sensor-Lagebild + aussagekräftiger Trend-Default

[patricznr1]

Problem

Beim Durchklicken: Die Maschinen-Detailseite zeigt als Manager im Sensortrend nur machine_running (bool) — die numerischen Sensoren (press_force, hydraulic_pressure …) sind disabled. Ursache: Der Manager landete in der reduzierten Sensor-Variante (sensorDetail: "reduced" → max 1 Sensor, Default = erster Datenpunkt = bool-Laufzustand). Laut Studie §4B ist "reduced" aber für mobile Werker (Handschuh, schnell) gedacht — nicht für den Manager am Desktop. Erster Eindruck dadurch nichtssagend.

Fix

• lib/machine/roles.ts: manager.sensorDetail reduced → full (max 4 statt 1 Sensor). aggregateOnly bleibt true — der Manager sieht das volle Bild, greift aber nicht ein.
• components/machine/machine-detail-view.tsx: analoge Messkurven als Default (stabiler Sort, kind === "analog" zuerst) statt des digitalen Laufzustands.

Verifiziert

vitest 60 passed (machine + roles), tsc clean, eslint clean.

Hinweis

Branch von main → enthält NICHT den Drill-Down (#40). Für einen Live-Frontend-Deploy mit allen Fixes erst #40 mergen, dann diesen auf main rebasen.

🤖 Generated with Claude Code

Summary by CodeRabbit

• Neue Funktionen
  • Für die Manager-Ansicht wird nun ein vollständigeres Sensorbild angezeigt, während Einzelaktionen weiterhin deaktiviert bleiben.
• Bug Fixes
  • Die standardmäßig ausgewählten Sensoren werden jetzt bevorzugt aus analogen Messwerten gewählt, bevor weitere Sensoren berücksichtigt werden.
• Tests
  • Die Prüffälle für die Manager-Rolle wurden an das neue Verhalten angepasst und verifizieren die aktualisierte Sensoranzeige.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 81591498-3872-40bd-b97a-45a77e59087a

📥 Commits

Reviewing files that changed from the base of the PR and between 291a9d9 and 9a25812.

📒 Files selected for processing (3)

• frontend/components/machine/machine-detail-view.tsx
• frontend/lib/machine/roles.test.ts
• frontend/lib/machine/roles.ts

---

📝 Walkthrough

Walkthrough

Die Manager-Rolle erhält in ROLE_VIEW nun sensorDetail: "full" statt "reduced". Parallel dazu sortiert MachineDetailView vor der Initialauswahl der Sensor-Datenpunkte analoge Einträge an den Anfang. Der zugehörige Testfall wird aktualisiert.

Änderungen

Sensor-Rollen und initiale Auswahl

Layer / Datei(en)	Zusammenfassung
Manager-Rolle: sensorDetail auf "full" gesetzt frontend/lib/machine/roles.ts, frontend/lib/machine/roles.test.ts	In der ROLE_VIEW-Map wird für manager sensorDetail von "reduced" auf "full" geändert; aggregateOnly: true bleibt unverändert. Der Kommentarblock und der Testfall werden entsprechend aktualisiert.
Initiale Sensorauswahl priorisiert analoge Datenpunkte frontend/components/machine/machine-detail-view.tsx	Vor dem Slicen für initialSelected wird eine sortierte Kopie von dataPoints erstellt, die analoge Einträge voranstellt; die Begrenzung auf 1 bzw. 2 Elemente je nach reduced bleibt unverändert.

Geschätzter Code-Review-Aufwand

🎯 2 (Einfach) | ⏱️ ~8 Minuten

Möglicherweise verwandte PRs

• patricznr1/foreman#26: Greift ebenfalls direkt in frontend/lib/machine/roles.ts und die initiale Sensorauswahl in machine-detail-view.tsx ein.

Gedicht

🐇 Ein Hase sortiert mit Bedacht,
analoge Kurven zuerst gebracht.
Der Manager schaut nun ins volle Bild,
kein Sensor bleibt mehr im Verborgnen still.
Mit "full" statt "reduced" — wie fein!
So soll die Maschine transparent sein. ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	Der Titel beschreibt die Hauptänderungen für den Manager-Frontend-Sensorblick und den neuen Trend-Default präzise und passend.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/manager-sensor-detail

---

_{Comment @coderabbitai help to get the list of available commands.}