# **NOT APPROVED**

Drei **Important Findings** verhindern die Freigabe. Keine Critical Findings.

## Important Findings

1. **Vollständig fehlende Tage werden im Symptom-Dashboard-Chart nicht als `null` dargestellt**
   - `scripts/health/health_dashboard_v3.py:549-565`
   - Die X-Achse wird ausschließlich aus `sorted(quick_by_day)` gebildet. Ein vollständig undokumentierter Kalendertag fehlt daher komplett, statt als Datum mit `null` im Dataset zu erscheinen.
   - Synthetischer Gegenbeweis: Bei Quick-Logs am 01.01. und 03.01. lautet die Achse nur `["2026-01-01", "2026-01-03"]`; der 02.01. besitzt keinen Gap-Slot. Chart.js kann die Punkte dadurch optisch verbinden.
   - Der Ernährung/Symptom-Chart behandelt fehlende Symptome auf vorhandenen Ernährungstagen korrekt als `null` (`:388-405`), bildet aber ebenfalls keine vollständige Kalenderachse. Damit ist „Missing bleibt unknown in **beiden** Charts“ nicht vollständig erfüllt.

2. **Die Persistenzgrenze erzwingt die Labor-Prädiktor-Allowlist nicht**
   - Allowlist-Mapping beim Laden ist korrekt: `scripts/health/multimodal_correlations.py:43-52,235-243`.
   - `store_results()` persistiert den übergebenen `result.predictor` jedoch ungeprüft: `:506-524`.
   - Synthetischer Gegenbeweis: `predictor="lab:free text / Source Hospital"` wurde erfolgreich in `multimodal_correlation_results` gespeichert.
   - Damit hängt die Datenschutzgarantie nur vom aktuellen Aufrufer ab; Freitext bzw. Quellnamen werden an der Persistenzgrenze nicht strikt abgewiesen.

3. **Legacy-Duplikate machen Quick-Log trotz Runtime-Migration unbenutzbar**
   - Migration überspringt den Unique Index bei vorhandenen Duplikaten: `scripts/health/health_system_migrate.py:175-184`.
   - Quick-Log versucht denselben Index anschließend unbedingt anzulegen: `scripts/health/health_symptom_quick_add.py:43-54`.
   - Synthetischer Lauf mit zwei bestehenden identischen `daily_quick_score`-Zeilen endet mit:
     `IntegrityError: UNIQUE constraint failed`.
   - Die Migration selbst läuft zweimal und bleibt schema-idempotent, aber der anschließende Runtime-Pfad ist für eine explizit berücksichtigte Legacy-Situation nicht kompatibel.

## Erfolgreiche Prüfungen

- **Tests:** `65 passed` — finaler Lauf in 11,35 s.
- **Compile:** 34 Python-Dateien kompiliert, 0 Fehler, ohne Repository-Artefakte.
- **Schema:** In-Memory-Ausführung erfolgreich; 41 Tabellen.
- **Schema-Manifest:** Digest stimmt exakt:
  `eb3f9e8fc285ce61e28c08816af709e2132fe4eb1e04f796984308d3abc964d8`.
- **Migration:** Kanonisches und älteres Korrelationsschema jeweils zweimal erfolgreich; `PRAGMA integrity_check = ok`.
- **CLI:** Partielle Eingaben werden als `complete=false; partial_sum=…; unspecified=unknown` ausgegeben, nicht als Total.
- **Engine:** Fehlende/partielle Symptomtage erzeugen keinen Total; explizite Null bleibt beobachtet.
- **Legacy-Aufrufe:** Quick-Log, YAZIO, Pipeline und Dashboard referenzieren weder `nutrition_insights.py` noch die alten Safe-/Trigger-, Phasen- oder Empfehlungstabellen.
- **Claims:** Keine expliziten Kausalitäts-, Diagnose-, Therapie- oder Entwarnungsbehauptungen im Sprint-2-Pfad festgestellt.
- **Diff-Hygiene:** `git diff --check` bestanden; keine Secret-Treffer und keine medizinischen Rohartefakte im Diff/untracked Scope.

## Änderungen durch den Review

- Keine Dateien erstellt oder verändert.
- Keine produktiven Datenbanken, Reports, Exporte, Secrets oder medizinischen Rohdaten geöffnet.