{
"event": "PreToolUse",
"tool_name": "Write",
"tool_input": {
"file_path": "\/var\/www\/dev.campus.systemische-tools.de\/docs\/codequality\/01-solid-violations.md",
"content": "# SOLID Violations Report\n\n**Projekt:** dev.campus.systemische-tools.de\n**Analysedatum:** 2025-12-22\n\n---\n\n## S - Single Responsibility Principle (SRP)\n\n### Kritische Violations\n\n#### 1. ChatController.php (978 LOC) - GOD CLASS\n**Pfad:** `src\/Controller\/ChatController.php`\n\n**Identifizierte Verantwortlichkeiten:**\n1. HTTP Request Handling\n2. Session CRUD (Create, Read, Update, Delete)\n3. Message CRUD\n4. Input Validation\n5. Chat Orchestration (KI-Kommunikation)\n6. HTML Rendering (HTMX Partials)\n7. Markdown zu HTML Konvertierung\n8. Kosten-Berechnung\n9. Logging\n10. UUID-Generierung\n11. Collection-Validierung\n\n**Empfohlene Aufteilung:**\n```\nChatController.php -> HTTP Routing nur\nChatSessionRepository.php -> Session CRUD\nChatMessageRepository.php -> Message CRUD\nChatResponseRenderer.php -> HTML\/HTMX Rendering\nMarkdownFormatter.php -> Markdown->HTML\nCostCalculator.php -> Token\/Kosten\nInputValidator.php -> Validierungen\n```\n\n#### 2. SemanticExplorerController.php (836 LOC)\n**Pfad:** `src\/Controller\/SemanticExplorerController.php`\n\n**Identifizierte Verantwortlichkeiten:**\n1. Dokumente CRUD\n2. Chunks Verwaltung\n3. Entities CRUD\n4. Relations CRUD\n5. Taxonomy CRUD\n6. Ontology CRUD\n7. Semantik-Ansicht\n8. Vector Search (Ollama + Qdrant)\n\n**Empfohlene Aufteilung:**\n```\nSemanticExplorerController.php -> Dashboard nur\nEntityController.php -> Entities CRUD\nRelationController.php -> Relations CRUD\nTaxonomyController.php -> Taxonomy CRUD\nOntologyController.php -> Ontology CRUD\nVectorSearchService.php -> Embedding + Qdrant\n```\n\n#### 3. SystemExplorerController.php (544 LOC)\n**Pfad:** `src\/Controller\/SystemExplorerController.php`\n\n**Problem:** Enthaelt direkte SQL-Queries statt Repository-Pattern\n- 15+ SQL Queries direkt im Controller\n- Mischung aus Read-Operationen und View-Rendering\n\n#### 4. ContentController.php (513 LOC)\n**Pfad:** `src\/Controller\/ContentController.php`\n\n**Probleme:**\n- Python-Script Aufrufe (`callPython()`)\n- HTML Rendering in Controller (`renderVersionPartial`, `renderCritiquePartial`)\n- Collection-Validierung\n\n---\n\n## O - Open\/Closed Principle (OCP)\n\n### Violations\n\n#### 1. ModelConfig - Hardcoded Models\n**Pfad:** `src\/Infrastructure\/AI\/ModelConfig.php`\n\n```php\npublic const MODELS = [\n 'claude-opus-4-5-20251101' => 'Claude Opus 4.5',\n 'claude-sonnet-4-20250514' => 'Claude Sonnet 4',\n \/\/ ... hardcoded\n];\n```\n**Problem:** Neue Modelle erfordern Code-Aenderungen statt Konfiguration\n\n#### 2. DatabaseFactory - Statische Factory\n**Pfad:** `src\/Infrastructure\/Config\/DatabaseFactory.php`\n\nNeue Datenbanken erfordern Code-Aenderung statt Konfiguration.\n\n---\n\n## L - Liskov Substitution Principle (LSP)\n\n**Status:** Keine kritischen Violations gefunden.\n- Controller erben korrekt von `Framework\\Controller`\n- Keine problematischen Overrides identifiziert\n\n---\n\n## I - Interface Segregation Principle (ISP)\n\n### Violations\n\n#### 1. Fehlende Interfaces\n**Problem:** Repository-Klassen haben keine Interfaces\n\n```\nContentRepository -> KEIN Interface\nCollectionRepository -> KEIN Interface\nSemanticExplorerRepository -> KEIN Interface\nDokumentationRepository -> KEIN Interface\n```\n\n**Auswirkung:**\n- Keine Austauschbarkeit (Mocking fuer Tests)\n- Keine klare API-Definition\n\n---\n\n## D - Dependency Inversion Principle (DIP)\n\n### Kritische Violations (30+ Stellen)\n\n#### 1. Direkte Repository-Instanziierung in Controllern\n\n| Datei | Zeile | Violation |\n|-------|-------|-----------|\n| ChatController.php | 26 | `new CollectionRepository()` |\n| ContentController.php | 21-23 | `new ContentRepository()`, `new CollectionRepository()`, `new CollectionValidator()` |\n| SemanticExplorerController.php | 20 | `new SemanticExplorerRepository()` |\n| SystemExplorerController.php | 21 | `DatabaseFactory::dev()` |\n| Api\/ExplorerController.php | 21 | `DatabaseFactory::dev()` |\n| ConfigController.php | 24 | `DatabaseFactory::content()` |\n| BackupRestoreController.php | 19 | `new FileBackupRepository()` |\n| DocsController.php | 19, 67, 76, 95, 113, 123-125 | 8x `new XRepository()` |\n| ContractController.php | 14 | `new ContractRepository()` |\n| Api\/ContentController.php | 14 | `new ContentRepository()` |\n\n#### 2. Direkte DatabaseFactory Aufrufe (17 Stellen)\n\n```php\n\/\/ Statt Dependency Injection:\n$this->db = DatabaseFactory::dev();\n\n\/\/ Sollte sein:\npublic function __construct(PDO $db) {\n $this->db = $db;\n}\n```\n\n**Betroffene Dateien:**\n- `src\/Controller\/Api\/ChatController.php:141`\n- `src\/Controller\/Api\/ExplorerController.php:21`\n- `src\/Controller\/ChatController.php:319, 750`\n- `src\/Controller\/ConfigController.php:24`\n- `src\/Controller\/SystemExplorerController.php:21`\n- `src\/Controller\/ProtokollController.php:20`\n- `src\/Infrastructure\/Persistence\/ContentRepository.php:13`\n- `src\/Infrastructure\/Persistence\/DokumentationRepository.php:13`\n- `src\/Infrastructure\/Persistence\/CollectionRepository.php:24, 212`\n- `src\/Infrastructure\/Persistence\/FileBackupRepository.php:28`\n- `src\/Infrastructure\/Docs\/ChunkingService.php:359`\n- `src\/Infrastructure\/Docs\/ChunkSyncService.php:538`\n- `src\/Infrastructure\/Docs\/ChunkAnalysisService.php:410`\n- `src\/Infrastructure\/Docs\/HybridSearchService.php:443`\n- `src\/Infrastructure\/SemanticExplorerRepository.php:19`\n\n#### 3. Positive Ausnahme: UseCases\n\nDie Task-UseCases erlauben optionale Injection:\n```php\npublic function __construct(?TaskRepository $taskRepository = null) {\n $this->taskRepository = $taskRepository ?? new TaskRepository();\n}\n```\nDies ist ein Kompromiss, aber besser als reine direkte Instanziierung.\n\n---\n\n## Zusammenfassung\n\n| Prinzip | Violations | Schweregrad |\n|---------|------------|-------------|\n| SRP | 4 God Classes | KRITISCH |\n| OCP | 2 Hardcoded Configs | MITTEL |\n| LSP | 0 | OK |\n| ISP | 4+ fehlende Interfaces | MITTEL |\n| DIP | 30+ direkte Instanziierungen | KRITISCH |\n\n**Hauptempfehlung:** DI-Container einfuehren (z.B. PHP-DI, Symfony DI) und alle `new Repository()` Aufrufe durch Container-Resolution ersetzen.\n"
}
}
{
"tool_response": {
"type": "create",
"filePath": "\/var\/www\/dev.campus.systemische-tools.de\/docs\/codequality\/01-solid-violations.md",
"content": "# SOLID Violations Report\n\n**Projekt:** dev.campus.systemische-tools.de\n**Analysedatum:** 2025-12-22\n\n---\n\n## S - Single Responsibility Principle (SRP)\n\n### Kritische Violations\n\n#### 1. ChatController.php (978 LOC) - GOD CLASS\n**Pfad:** `src\/Controller\/ChatController.php`\n\n**Identifizierte Verantwortlichkeiten:**\n1. HTTP Request Handling\n2. Session CRUD (Create, Read, Update, Delete)\n3. Message CRUD\n4. Input Validation\n5. Chat Orchestration (KI-Kommunikation)\n6. HTML Rendering (HTMX Partials)\n7. Markdown zu HTML Konvertierung\n8. Kosten-Berechnung\n9. Logging\n10. UUID-Generierung\n11. Collection-Validierung\n\n**Empfohlene Aufteilung:**\n```\nChatController.php -> HTTP Routing nur\nChatSessionRepository.php -> Session CRUD\nChatMessageRepository.php -> Message CRUD\nChatResponseRenderer.php -> HTML\/HTMX Rendering\nMarkdownFormatter.php -> Markdown->HTML\nCostCalculator.php -> Token\/Kosten\nInputValidator.php -> Validierungen\n```\n\n#### 2. SemanticExplorerController.php (836 LOC)\n**Pfad:** `src\/Controller\/SemanticExplorerController.php`\n\n**Identifizierte Verantwortlichkeiten:**\n1. Dokumente CRUD\n2. Chunks Verwaltung\n3. Entities CRUD\n4. Relations CRUD\n5. Taxonomy CRUD\n6. Ontology CRUD\n7. Semantik-Ansicht\n8. Vector Search (Ollama + Qdrant)\n\n**Empfohlene Aufteilung:**\n```\nSemanticExplorerController.php -> Dashboard nur\nEntityController.php -> Entities CRUD\nRelationController.php -> Relations CRUD\nTaxonomyController.php -> Taxonomy CRUD\nOntologyController.php -> Ontology CRUD\nVectorSearchService.php -> Embedding + Qdrant\n```\n\n#### 3. SystemExplorerController.php (544 LOC)\n**Pfad:** `src\/Controller\/SystemExplorerController.php`\n\n**Problem:** Enthaelt direkte SQL-Queries statt Repository-Pattern\n- 15+ SQL Queries direkt im Controller\n- Mischung aus Read-Operationen und View-Rendering\n\n#### 4. ContentController.php (513 LOC)\n**Pfad:** `src\/Controller\/ContentController.php`\n\n**Probleme:**\n- Python-Script Aufrufe (`callPython()`)\n- HTML Rendering in Controller (`renderVersionPartial`, `renderCritiquePartial`)\n- Collection-Validierung\n\n---\n\n## O - Open\/Closed Principle (OCP)\n\n### Violations\n\n#### 1. ModelConfig - Hardcoded Models\n**Pfad:** `src\/Infrastructure\/AI\/ModelConfig.php`\n\n```php\npublic const MODELS = [\n 'claude-opus-4-5-20251101' => 'Claude Opus 4.5',\n 'claude-sonnet-4-20250514' => 'Claude Sonnet 4',\n \/\/ ... hardcoded\n];\n```\n**Problem:** Neue Modelle erfordern Code-Aenderungen statt Konfiguration\n\n#### 2. DatabaseFactory - Statische Factory\n**Pfad:** `src\/Infrastructure\/Config\/DatabaseFactory.php`\n\nNeue Datenbanken erfordern Code-Aenderung statt Konfiguration.\n\n---\n\n## L - Liskov Substitution Principle (LSP)\n\n**Status:** Keine kritischen Violations gefunden.\n- Controller erben korrekt von `Framework\\Controller`\n- Keine problematischen Overrides identifiziert\n\n---\n\n## I - Interface Segregation Principle (ISP)\n\n### Violations\n\n#### 1. Fehlende Interfaces\n**Problem:** Repository-Klassen haben keine Interfaces\n\n```\nContentRepository -> KEIN Interface\nCollectionRepository -> KEIN Interface\nSemanticExplorerRepository -> KEIN Interface\nDokumentationRepository -> KEIN Interface\n```\n\n**Auswirkung:**\n- Keine Austauschbarkeit (Mocking fuer Tests)\n- Keine klare API-Definition\n\n---\n\n## D - Dependency Inversion Principle (DIP)\n\n### Kritische Violations (30+ Stellen)\n\n#### 1. Direkte Repository-Instanziierung in Controllern\n\n| Datei | Zeile | Violation |\n|-------|-------|-----------|\n| ChatController.php | 26 | `new CollectionRepository()` |\n| ContentController.php | 21-23 | `new ContentRepository()`, `new CollectionRepository()`, `new CollectionValidator()` |\n| SemanticExplorerController.php | 20 | `new SemanticExplorerRepository()` |\n| SystemExplorerController.php | 21 | `DatabaseFactory::dev()` |\n| Api\/ExplorerController.php | 21 | `DatabaseFactory::dev()` |\n| ConfigController.php | 24 | `DatabaseFactory::content()` |\n| BackupRestoreController.php | 19 | `new FileBackupRepository()` |\n| DocsController.php | 19, 67, 76, 95, 113, 123-125 | 8x `new XRepository()` |\n| ContractController.php | 14 | `new ContractRepository()` |\n| Api\/ContentController.php | 14 | `new ContentRepository()` |\n\n#### 2. Direkte DatabaseFactory Aufrufe (17 Stellen)\n\n```php\n\/\/ Statt Dependency Injection:\n$this->db = DatabaseFactory::dev();\n\n\/\/ Sollte sein:\npublic function __construct(PDO $db) {\n $this->db = $db;\n}\n```\n\n**Betroffene Dateien:**\n- `src\/Controller\/Api\/ChatController.php:141`\n- `src\/Controller\/Api\/ExplorerController.php:21`\n- `src\/Controller\/ChatController.php:319, 750`\n- `src\/Controller\/ConfigController.php:24`\n- `src\/Controller\/SystemExplorerController.php:21`\n- `src\/Controller\/ProtokollController.php:20`\n- `src\/Infrastructure\/Persistence\/ContentRepository.php:13`\n- `src\/Infrastructure\/Persistence\/DokumentationRepository.php:13`\n- `src\/Infrastructure\/Persistence\/CollectionRepository.php:24, 212`\n- `src\/Infrastructure\/Persistence\/FileBackupRepository.php:28`\n- `src\/Infrastructure\/Docs\/ChunkingService.php:359`\n- `src\/Infrastructure\/Docs\/ChunkSyncService.php:538`\n- `src\/Infrastructure\/Docs\/ChunkAnalysisService.php:410`\n- `src\/Infrastructure\/Docs\/HybridSearchService.php:443`\n- `src\/Infrastructure\/SemanticExplorerRepository.php:19`\n\n#### 3. Positive Ausnahme: UseCases\n\nDie Task-UseCases erlauben optionale Injection:\n```php\npublic function __construct(?TaskRepository $taskRepository = null) {\n $this->taskRepository = $taskRepository ?? new TaskRepository();\n}\n```\nDies ist ein Kompromiss, aber besser als reine direkte Instanziierung.\n\n---\n\n## Zusammenfassung\n\n| Prinzip | Violations | Schweregrad |\n|---------|------------|-------------|\n| SRP | 4 God Classes | KRITISCH |\n| OCP | 2 Hardcoded Configs | MITTEL |\n| LSP | 0 | OK |\n| ISP | 4+ fehlende Interfaces | MITTEL |\n| DIP | 30+ direkte Instanziierungen | KRITISCH |\n\n**Hauptempfehlung:** DI-Container einfuehren (z.B. PHP-DI, Symfony DI) und alle `new Repository()` Aufrufe durch Container-Resolution ersetzen.\n",
"structuredPatch": [],
"originalFile": null
}
}