Fix manual-review findings: dynamic age buckets, AD mail match, stale-timestamp gap, naming, perf cleanups

[zerox80]

Summary

Eine vollständige manuelle Code-Review (Logic Error / Performance) über das gesamte Repo (Rust-Backend, JS-Frontend, PowerShell-Agent) ergab 8 konkrete Befunde — keine kritischen Bugs, aber mehrere echte Logikfehler und Performance-/Wartbarkeits-Nits. Dieser PR behebt alle.

• Alters-Histogramm ignorierte konfigurierbaren max_age_years-Schwellwert (overview.rs, mock.js): Bucket-Grenzen waren fix auf 2/4/5 Jahre verdrahtet, obwohl old5/old_age_label im selben Overview-Objekt bereits korrekt den Schwellwert nutzten. Grenzen jetzt proportional zum Schwellwert abgeleitet — beim Default (5,0 Jahre) identisch zur bisherigen Aufteilung, skaliert aber korrekt bei individuellen Schwellwerten. Labels nutzen jetzt durchgängig die 1-Nachkommastellen-Formatierung (fmt_de), konsistent mit dem Rest der App.
• AD-Suchtreffer per E-Mail wurden clientseitig wieder rausgefiltert (commands.rs, Get-AdUsers.ps1): Das LDAP-Skript sucht u. a. über mail, der nachgelagerte Rust-Filter prüfte das Feld aber nicht und verwarf dadurch gültige LDAP-Treffer. mail ergänzt (Rust + JS-Mock).
• Fehlendes/unparsbares collectedAtUtc verhinderte "stale" dauerhaft (upgrade.rs): Ein Gerät mit kaputtem Zeitstempel galt für immer als "frisch". Neue dritte Bedingung + eigenes Label ("Unplausibel · Zeitstempel fehlt"), abgesichert durch einen neuen geteilten Golden-Vector-Fall (upgrade-cases.json).
• DeptStat.upgrade irreführend benannt (model.rs): zählte tatsächlich auch "missing"-Geräte mit ("needs action", nicht nur "needs upgrade"). Umbenannt zu needs_action/needsAction, passend zum bereits bestehenden internen Closure-Namen in overview.rs.
• Doppelte clone()-Aufrufe in build_one (merge.rs): network, bios und win11 wurden je zweimal geklont (einmal pro abgeleitetem Feld). Jetzt einmal gebunden.
• build_overview-Performance: vier separate Status-count()-Scans in die ohnehin vorhandene Abteilungs-Schleife verschoben statt eines riskanten Vollrewrites (bei realistischer Flottengröße ohnehin kein messbares Problem).
• KPI-Rundung für Ø Alter inkonsistent formatiert (app.js): nutzte eigene Ad-hoc-Formatierung statt der 1-Nachkommastellen-Konvention; jetzt über neuen gemeinsamen fmtDe-Helper in view-model.js.
• Fragile Label-Regex für Chart-Farben (app-panels.js): ersetzt durch robuste Positionslogik (letzter Age-Bucket / erster RAM-Bucket sind per Konstruktion immer die hervorzuhebenden).
• Windows-Quoting-Randfall (Install-InventoryTask.ps1): ein OutputDir/ScriptPath, der mit einer ungeraden Anzahl Backslashes vor dem schließenden " endet, konnte über den klassischen CommandLineToArgvW-Escaping-Effekt das Argument falsch terminieren. Trailing-Backslash-Verdopplung ergänzt.

Bewusste kosmetische Änderung

Die Alters-Bucket-Labels gewinnen beim Default-Schwellwert eine Nachkommastelle ("< 2 Jahre" → "< 2,0 Jahre"), da die Formatierung jetzt durchgängig über fmt_de/fmtDe läuft (konsistent mit age_text einzelner Geräte). Kein Bug, nur Konsistenz.

Test plan

• cargo fmt --all -- --check
• cargo clippy --all-targets -- -D warnings
• cargo test --all (24/24 grün, inkl. 2 neuer Tests)
• npm run check
• npm test (5/5 grün, inkl. neuem Golden-Vector-Fall für fehlenden Zeitstempel)
• PowerShell ParseFile-Syntaxcheck auf alle Agent-Skripte
• Manueller Preview-Smoke-Test: Dashboard-Altersverteilung/RAM-Verteilung (Farben + Werte korrekt), Abteilungs-Ansicht, AD-Suche nach @example.com (bestätigt: findet jetzt alle 21 Mock-User über mail, vorher 0 Treffer)
• Alle geänderten Dateien unter dem 300-Zeilen-CI-Limit (Test-Datei store/tests.rs wurde dafür aufgeteilt, neue store/overview_tests.rs analog zum bestehenden io_tests.rs-Muster)

🤖 Generated with Claude Code