Service-Termine: CI-Icons statt Wortlaut, Anstehende-Termine neu gegliedert

Naechster-Service-Kachel (Uebersicht): das "bis zum Oelwechsel"/
"bis zur Inspektion" vor der Kennzahl entfaellt zugunsten der echten
Audi-CI-Icons oil-change/inspection (vom Nutzer als SVG geliefert,
verbatim uebernommen wie die uebrigen CI-Icons). Hauptuntersuchung hat
kein eigenes Icon (kein Restkilometer-Ziel) und zeigt stattdessen ihr
Datum in derselben 40px-Groesse wie die km-Zahlen, mit dem Wort
"Hauptuntersuchung" darunter statt "bis zur Hauptuntersuchung" - vorher
war das Datumsfeld kleiner (30px), um Umbruch zu vermeiden; am
laufenden Panel nachgemessen, dass 40px keinen Umbruch verursacht.
"vsl." wird ausgeschrieben zu "voraussichtlich am". bisText()/ART_BIS
(Panel) und bisText()/BIS_TEXT (companion-app) wurden durch die
Aenderung ueberfluessig und als Orphans entfernt.

Anstehende-Termine-Box (Service-Seite): aus der dt/dd-Liste werden drei
sichtbar getrennte Bloecke (Kopfzeile mit Icon + Name + optionalem
Werkstatt-Chip, optionale Herstellervorgabe-Zeile, grosse Kennzahl,
optionale Prognose-Fusszeile) - Apple HIG (Hierarchie ueber
Position/Groesse, Inhalt nicht mit Nebensaechlichem ueberladen) plus
Audi-CI (Flaeche mit Haarlinien statt Karten). Durchlief drei
Mockup-Runden als Artefakt vor der Umsetzung, zwei davon vom Nutzer
zurueckgewiesen ("Major Information is missing" bzw. Kommentare zu
Icon-Groesse/Herstellervorgabe/Ausrichtung).

Herstellervorgabe-Logik korrigiert: eine Fahrzeugmeldung stammt immer
aus dem werkseigenen Wartungsprogramm des Bordcomputers, unabhaengig
vom in "Einrichten" gewaehlten App-Intervall - zaehlt also immer als
Herstellervorgabe, nicht nur wenn der App-Modus zufaellig "hersteller"
ist. Die Prognose-Fusszeile zeigt die Restkilometerzahl nur noch beim
eigenen (kuerzeren) Intervall, bei Herstellervorgabe nur noch das
Datum. "Eigenes" in der Ölwechsel-Intervall-Auswahl umbenannt zu
"Individuell".

companion-app-Portierung ist eine echte Neustrukturierung: die
bisherige "Naechster Oelwechsel"-Kachel mischte Oelwechsel-,
Inspektions- und Hauptuntersuchungs-Zeilen in einer gemeinsamen
Werteliste, jetzt "Anstehende Termine" mit denselben drei Bloecken wie
im Panel. Neuer Export letzterInspektion() in service.ts. Zwei
dokumentierte, vorbestehende Luecken bleiben bestehen statt neu
kaschiert zu werden: kein Werkstatt-Chip (companion-app hat keine
"Termin vereinbaren"-Funktion), Herstellervorgabe kommt aus
fahrzeug.oel.modus statt einem intervalle()-Aequivalent (das es dort
nicht gibt).

npm run typecheck sauber, npm run test 99/99, npm run build erfolgreich.
Panel deployed als Version 1787013003, synchron in installationspaket/,
live im Docker-Testcontainer per DOM-Abfrage geprueft.

Co-Authored-By: Claude Sonnet 5 <[email protected]>

companion-app/src/screens/Service.tsx

@@ -13,9 +13,10 @@ import { useState } from "react"
 import { ActionButton, Feld, Tile } from "@audi-dash/ui"
 
 import { useDaten } from "../daten/DatenKontext"
-import { inspektionPrognose, oelwechselPrognose } from "../daten/service"
+import { inspektionPrognose, letzterInspektion, letzterOelwechsel, oelwechselPrognose } from "../daten/service"
 import { datum, de, isoTag } from "../format"
 import type { SeitenName } from "../navigation"
+import { SymbolInspektion, SymbolOelwechsel } from "../symbole"
 import { Leerzustand, NaviKachel, Wertzeile, Werteliste, bestaetigen } from "./bausteine"
 import { kalenderDateiLaden } from "./kalender"
 
@@ -44,53 +45,106 @@ export function Service({ geheZu }: { geheZu: (name: SeitenName, id?: string) =>
   const prognose = oelwechselPrognose(fahrzeug, buch)
   const inspektionsprognose = inspektionPrognose(fahrzeug, buch)
 
+  // Ölwechsel und Inspektion je einzeln, statt einer gemeinsamen Wertzeilen-
+  // Liste unter "Nächster Ölwechsel" (die Inspektion/Hauptuntersuchung dort
+  // fachfremd mitschleppte) - deckungsgleich mit vService() im Panel: drei
+  // sichtbar getrennte Blöcke, jeder mit eigenem Icon, optionaler
+  // Herstellervorgabe-Zeile und optionaler Prognose-Fußzeile.
+  const oelHatEintrag = letzterOelwechsel(buch) != null
+  const oelHatMeldung = fahrzeug.oelwechselFaelligTs != null || fahrzeug.oelwechselFaelligKm != null
+  const oelWert = oelHatMeldung
+    ? [
+        fahrzeug.oelwechselFaelligKm != null ? `${de(Math.abs(fahrzeug.oelwechselFaelligKm))} km` : null,
+        fahrzeug.oelwechselFaelligTs ? datum(fahrzeug.oelwechselFaelligTs) : null,
+      ]
+        .filter(Boolean)
+        .join(" / ")
+    : prognose
+      ? `${de(prognose.restKm)} km · ${datum(prognose.datum)}`
+      : null
+  // Bei Herstellervorgabe rechnet die eigene Prognose ohnehin schon mit dem
+  // Herstellerintervall - die Restkilometerzahl daneben wäre nur eine zweite,
+  // verwirrende km-Angabe neben dem Fahrzeug-Wert oben. Nur beim eigenen
+  // (kürzeren) Intervall ist sie eine echte Zusatzinformation.
+  const oelFussnote = oelHatMeldung && prognose
+    ? fahrzeug.oel.modus === "hersteller"
+      ? `voraussichtlich am ${datum(prognose.datum)}`
+      : `${de(prognose.restKm)} km, voraussichtlich am ${datum(prognose.datum)}`
+    : null
+  // Der Ölwechsel folgt in "Einrichten" wahlweise der Herstellervorgabe oder
+  // einem kürzeren eigenen Intervall (fahrzeug.oel.modus) - die Inspektion hat
+  // kein einstellbares Intervall, rechnet also immer mit der Herstellervorgabe
+  // (siehe inspektionPrognose()'s eigener Kommentar dazu). Eine Fahrzeugmeldung
+  // stammt aber immer aus dem werkseigenen Wartungsprogramm des Bordcomputers -
+  // der kennt das eigene App-Intervall gar nicht -, zählt also unabhängig vom
+  // gewählten Modus als Herstellervorgabe.
+  const oelHerstellervorgabe = oelHatMeldung || fahrzeug.oel.modus === "hersteller"
+
+  const inspHatEintrag = letzterInspektion(buch) != null
+  const inspHatMeldung = fahrzeug.inspektionFaelligTs != null || fahrzeug.inspektionFaelligKm != null
+  const inspWert = inspHatMeldung
+    ? [
+        fahrzeug.inspektionFaelligKm != null ? `${de(Math.abs(fahrzeug.inspektionFaelligKm))} km` : null,
+        fahrzeug.inspektionFaelligTs ? datum(fahrzeug.inspektionFaelligTs) : null,
+      ]
+        .filter(Boolean)
+        .join(" / ")
+    : inspektionsprognose
+      ? datum(inspektionsprognose.datum)
+      : null
+  const inspFussnote = inspektionsprognose ? `voraussichtlich am ${datum(inspektionsprognose.datum)}` : null
+
+  const huDatum = fahrzeug.hu ? new Date(fahrzeug.hu) : null
+  const huGueltig = !!huDatum && !Number.isNaN(huDatum.getTime())
+
   return (
     <>
       <Tile>
-        <span className="ads-eyebrow">Nächster Ölwechsel</span>
-        <Werteliste
-          kinder={
+        <span className="ads-eyebrow">Anstehende Termine</span>
+
+        <div className="dm-termingruppe">
+          <div className="dm-tgkopf">
+            <SymbolOelwechsel groesse={26} />
+            <span>Ölwechsel</span>
+          </div>
+          {oelHatEintrag ? (
             <>
-              <Wertzeile
-                label="Vom Fahrzeug gemeldet"
-                wert={fahrzeug.oelwechselFaelligTs ? datum(fahrzeug.oelwechselFaelligTs) : "—"}
-                zusatz={
-                  fahrzeug.oelwechselFaelligKm != null
-                    ? `in ${de(Math.abs(fahrzeug.oelwechselFaelligKm))} km`
-                    : undefined
-                }
-              />
-              <Wertzeile
-                label="Eigene Prognose"
-                wert={prognose ? datum(prognose.datum) : "—"}
-                zusatz={
-                  prognose
-                    ? `${de(prognose.restKm)} km offen · ${de(prognose.kmProTag, 1)} km/Tag`
-                    : "Noch kein Ölwechsel im Servicebuch"
-                }
-              />
-              <Wertzeile
-                label="Nächste Inspektion"
-                wert={fahrzeug.inspektionFaelligTs ? datum(fahrzeug.inspektionFaelligTs) : "—"}
-                zusatz={
-                  fahrzeug.inspektionFaelligKm != null
-                    ? `in ${de(Math.abs(fahrzeug.inspektionFaelligKm))} km`
-                    : undefined
-                }
-              />
-              {inspektionsprognose && (
-                <Wertzeile
-                  label="Eigene Prognose (Inspektion)"
-                  wert={datum(inspektionsprognose.datum)}
-                />
-              )}
-              <Wertzeile
-                label="Hauptuntersuchung"
-                wert={fahrzeug.hu ? datum(fahrzeug.hu) : "—"}
-              />
+              {oelHerstellervorgabe && <div className="dm-tgvorgabe">Herstellervorgabe</div>}
+              <div className="dm-tgwert">{oelWert}</div>
+              {oelFussnote && <div className="dm-tgprognose">{oelFussnote}</div>}
             </>
-          }
-        />
+          ) : (
+            <div className="dm-tgwert dm-tgwert--leer">kein Eintrag im Servicebuch</div>
+          )}
+        </div>
+
+        <div className="dm-termingruppe">
+          <div className="dm-tgkopf">
+            <SymbolInspektion groesse={20} />
+            <span>Inspektion</span>
+          </div>
+          {inspHatEintrag ? (
+            <>
+              <div className="dm-tgvorgabe">Herstellervorgabe</div>
+              <div className="dm-tgwert">{inspWert}</div>
+              {inspFussnote && <div className="dm-tgprognose">{inspFussnote}</div>}
+            </>
+          ) : (
+            <div className="dm-tgwert dm-tgwert--leer">kein Eintrag im Servicebuch</div>
+          )}
+        </div>
+
+        <div className="dm-termingruppe">
+          <div className="dm-tgkopf">
+            <span>Hauptuntersuchung</span>
+          </div>
+          {huGueltig ? (
+            <div className="dm-tgwert">{datum(fahrzeug.hu)}</div>
+          ) : (
+            <div className="dm-tgwert dm-tgwert--leer">kein Eintrag im Servicebuch</div>
+          )}
+        </div>
+
         {prognose && (
           <div style={{ marginTop: 14 }}>
             <ActionButton
@@ -103,9 +157,9 @@ export function Service({ geheZu }: { geheZu: (name: SeitenName, id?: string) =>
           </div>
         )}
         <p className="dm-fussnote">
-          Die eigene Prognose rechnet ab dem letzten Ölwechsel im Servicebuch neu und nutzt die
-          seither tatsächlich gefahrene Strecke. Sie ersetzt die Meldung des Fahrzeugs nicht,
-          sondern ergänzt sie.
+          Die eigene Prognose rechnet ab dem letzten Eintrag der jeweiligen Art im Servicebuch neu
+          und nutzt die seither tatsächlich gefahrene Strecke. Sie ersetzt die Meldung des
+          Fahrzeugs nicht, sondern ergänzt sie.
         </p>
       </Tile>
 

companion-app/src/screens/Uebersicht.tsx

@@ -11,14 +11,24 @@ import { useState } from "react"
 import { Fig, ProgressBar, StatusRow, Tile } from "@audi-dash/ui"
 
 import { useDaten } from "../daten/DatenKontext"
-import { bisText, naechsterService } from "../daten/service"
+import { naechsterService } from "../daten/service"
 import { datum, datumZeit, dauer, de, deOderStrich, eur } from "../format"
 import type { SeitenName } from "../navigation"
 import { Leerzustand, NaviKachel } from "./bausteine"
 import { Bild } from "./Bild"
 import type { ServicebuchEintrag } from "./Service"
+import { SymbolInspektion, SymbolOelwechsel } from "../symbole"
 import { Typenschild, schildErkennen } from "./Typenschild"
 
+// "bis zum Ölwechsel"/"bis zur Inspektion" entfaellt zugunsten des CI-Icons
+// links der Kennzahl - deckungsgleich mit ART_ICON im Panel (vHome()). Die
+// Hauptuntersuchung hat kein Restkilometer-Ziel und zeigt stattdessen den
+// Art-Namen selbst unter dem (gleich großen) Datum.
+const SERVICE_SYMBOL: Partial<Record<string, typeof SymbolOelwechsel>> = {
+  "Ölwechsel": SymbolOelwechsel,
+  "Inspektion": SymbolInspektion,
+}
+
 export function Uebersicht({ geheZu }: { geheZu: (name: SeitenName, id?: string) => void }) {
   const { fahrzeug, einstellungen, fahrten, tankvorgaenge, jetztAktualisieren, statusStand } =
     useDaten()
@@ -143,15 +153,26 @@ export function Uebersicht({ geheZu }: { geheZu: (name: SeitenName, id?: string)
           </span>
           {service ? (
             <>
-              <Fig
-                value={service.restKm !== null ? de(service.restKm) : datum(service.datum)}
-                {...(service.restKm !== null ? { unit: "km" } : {})}
-                size={service.restKm !== null ? 40 : 30}
-              />
+              {(() => {
+                const Symbol = SERVICE_SYMBOL[service.art]
+                const fig = (
+                  <Fig
+                    value={service.restKm !== null ? de(service.restKm) : datum(service.datum)}
+                    {...(service.restKm !== null ? { unit: "km" } : {})}
+                    size={40}
+                  />
+                )
+                return Symbol ? (
+                  <div className="dm-servicefig">
+                    <Symbol groesse={31} />
+                    {fig}
+                  </div>
+                ) : (
+                  fig
+                )
+              })()}
               <span className="dm-fussnote">
-                {service.restKm !== null
-                  ? `${bisText(service.art)} · vsl. ${datum(service.datum)}`
-                  : bisText(service.art)}
+                {service.restKm !== null ? `voraussichtlich am ${datum(service.datum)}` : service.art}
               </span>
             </>
           ) : (