{
"assistant_response": "Hier ist die **kritische faktische Prüfung** des Implementierungsplans:\n\n---\n\n## KRITISCHER REVIEW - PROBLEME IDENTIFIZIERT\n\n### DRY - VERLETZT\n\n| Problem | Fundstelle |\n|---------|------------|\n| **Doppelte Dropdowns** | `system_prompt_id`, `structure_id`, `author_profile_id` existieren im Plan sowohl im Config-Panel ALS AUCH in `.chat-settings--advanced` |\n| **Fix erforderlich** | Settings müssen entfernt werden wenn Config-Panel existiert |\n\n```\nConfig-Panel: [System Prompt ▼]\nSettings: [System Prompt ▼] ← DUPLIKAT!\n```\n\n---\n\n### SRP - VERLETZT\n\n| Problem | Contract-Referenz |\n|---------|-------------------|\n| ChatController bekommt 3 neue Methoden für Config-Management | Contract #5, #9 |\n\n**Aktuell:** `ChatController.php` = 384 Zeilen (12 Methoden)\n**Nach Plan:** +3 Methoden (`getConfig`, `saveConfig`, `getConfigHistory`)\n\n**Verletzt:** ChatController hat eine Responsibility (Chat-Session-Management). Config-Management ist eine ANDERE Responsibility.\n\n**Fix:** Eigener `Controller\\Api\\ConfigController` oder Erweiterung von `ConfigController`\n\n---\n\n### MVC - VERLETZT\n\n| Problem | Im Plan |\n|---------|---------|\n| Business-Logik im Controller | Version-Inkrementierung `1.0 → 1.1` |\n\n```php\n\/\/ FALSCH (im Controller):\n$versionParts = explode('.', $currentVersion);\n$minor = (int) ($versionParts[1] ?? 0) + 1;\n$newVersion = $versionParts[0] . '.' . $minor;\n```\n\n**Fix:** `ContentConfigRepository::incrementVersion()` oder UseCase\n\n---\n\n### Interface Segregation - VERLETZT\n\n| Interface | Problem |\n|-----------|---------|\n| `ContentConfigRepositoryInterface` | Hat KEINE `saveHistory()` \/ `getHistory()` Methoden |\n\n```php\n\/\/ Aktuelles Interface (52 Zeilen):\ninterface ContentConfigRepositoryInterface {\n public function getAuthorProfiles(): array;\n public function getSystemPrompts(): array;\n public function getStructures(): array;\n \/\/ FEHLT: saveHistory(), getHistory()\n}\n```\n\n**Fix:** Interface erweitern oder separates `VersionedConfigRepositoryInterface`\n\n---\n\n### Dependency Injection - FEHLT\n\n| Problem |\n|---------|\n| ChatController hat KEINE `ContentConfigRepository` Dependency |\n\n**Aktueller Constructor:**\n```php\npublic function __construct(\n private CreateChatSessionUseCaseInterface $createSessionUseCase,\n private GetChatSessionUseCaseInterface $getSessionUseCase,\n \/\/ ... KEINE configRepo!\n)\n```\n\n**Fix:** Neue Dependency hinzufügen ODER separater Controller\n\n---\n\n### WCAG\/A11y - VERLETZT\n\n| Element | Problem | WCAG-Regel |\n|---------|---------|------------|\n| Toggle-Button `▸` | Kein `aria-label`, kein `aria-expanded` | 4.1.2 Name, Role, Value |\n| `<textarea>` | Kein `<label>` verknüpft | 1.3.1 Info and Relationships |\n| Version-Dropdown | Kein beschreibender Label | 1.3.1 |\n| `.chat-config__toggle-icon` | Nur visueller Pfeil, nicht screenreader-zugänglich | 1.1.1 Non-text Content |\n\n**Fix erforderlich:**\n```html\n<button type=\"button\" class=\"chat-config__toggle\" \n aria-expanded=\"false\" \n aria-controls=\"systemPromptContent\"\n aria-label=\"System Prompt Inhalt anzeigen\">\n```\n\n---\n\n### Contract-Verstöße\n\n| Contract | Regel | Status |\n|----------|-------|--------|\n| #5 layered-architecture | no-pdo-in-controller | ✓ OK |\n| #5 layered-architecture | no-sql-in-controller | ✓ OK |\n| #9 code-quality | di-constructor-injection | ⚠️ Neue Dependency fehlt |\n| #4 js-browser | no-double-equals | ⚠️ `option.value === 'new'` korrekt, aber `!id` ist loose |\n| #2 css-standards | no-outline-none | ⚠️ Nicht geprüft für neue Styles |\n| #7 view-structure | breadcrumb-first | ✗ Chat-View hat kein Breadcrumb |\n\n---\n\n### YAGNI - POTENTIELL VERLETZT\n\n| Feature | Gefordert? | Im Plan? |\n|---------|------------|----------|\n| Toggle für Edit-Area | ✓ Ja | ✓ Ja |\n| Speichern = Neue Version | ✓ Ja | ✓ Ja |\n| Version-History-Dropdown | ✗ Nein | ✓ Ja (Overengineering?) |\n| History laden und auswählen | ✗ Nein | ✓ Ja |\n\n**User sagte:** \"wenn man etwas ändert und speichert, wird davon eine neue version erstellt\"\n**Plan macht:** Komplettes Version-History-Browsing mit Wiederherstellung\n\n---\n\n### KISS - VERLETZT\n\nJavaScript im Plan: ~100 Zeilen mit:\n- 4 Event-Listener-Gruppen\n- Async\/Await für History-Loading\n- JSON.parse\/stringify für Content-Formatting\n- Verschachtelte DOM-Manipulation\n\n**Einfachere Alternative:**\n- HTMX für Save-Operation\n- Inline-Template für Textarea-Content\n- Keine History-Auswahl (YAGNI)\n\n---\n\n## ZUSAMMENFASSUNG\n\n| Prinzip | Status | Schwere |\n|---------|--------|---------|\n| DRY | ✗ VERLETZT | Major |\n| SRP | ✗ VERLETZT | Critical |\n| MVC | ✗ VERLETZT | Major |\n| Interface Segregation | ✗ VERLETZT | Major |\n| Dependency Injection | ✗ FEHLT | Critical |\n| WCAG | ✗ VERLETZT | Major |\n| YAGNI | ⚠️ GRENZWERTIG | Minor |\n| KISS | ⚠️ GRENZWERTIG | Minor |\n| OOP | ✓ OK | - |\n| CRUD | ✓ OK | - |\n\n**Kritische Fixes vor Implementation:**\n1. Separater `ConfigApiController` statt ChatController erweitern\n2. `ContentConfigRepositoryInterface` um History-Methoden erweitern\n3. A11y-Attribute für Toggle\/Textarea hinzufügen\n4. Duplikate aus Settings entfernen\n5. Version-History-Dropdown entfernen (YAGNI)"
}