Protokoll #6525

{
    "event": "PreToolUse",
    "tool_name": "mcp__mcp-tasks__tasks_create",
    "tool_input": {
        "title": "Architektur-Refactoring: DI-Vereinheitlichung und Klassen-Aufteilung",
        "description": "## Supervision-Feedback: Architektur-Muster\n\n### A) Dependency Injection uneinheitlich\n**Problem:** DI Container + ServiceLocator + statische Factorys + UseCases mit Default-Konstruktion\n**Evidenz:**\n- `src\/Infrastructure\/Container\/ServiceLocator.php`: globaler Zustand, statische Registry\n- `src\/UseCases\/Chat\/SendChatMessageUseCase.php`: optional dependencies mit Fallback `new Repository()`\n- `src\/services.php`: registriert UseCases ohne Abhängigkeiten trotz Constructor-Dependencies\n\n### B) Persistence-Verbindungslogik mehrfach verteilt\n**Problem:** PDO-Verbindungen per static DatabaseFactory, deprecated Connection, createConnection-Methoden\n**Evidenz:**\n- `DatabaseFactory.php`: statische Connection-Erzeugung\n- `Connection.php`: deprecated Singleton\n- `ChunkingService`, `ChunkAnalysisService`, `HybridSearchService`, `ChunkSyncService`: jeweils private `createConnection()`\n\n### C) Große Klassen mit zu vielen Verantwortungen\n**Problem:** SRP-Bruch, hohe Kopplung\n**Evidenz:**\n- `SemanticExplorerRepository.php`: 797 Zeilen (CRUD, Filter, Stats, Relations, Chunks, Classifications)\n- `ChunkSyncService.php`: 551 Zeilen (Orchestrierung, IO, DB, Pipeline)\n\n### D) Domain Layer anämisch, Arrays dominieren\n**Problem:** Controller\/UseCases arbeiten mit assoziativen Arrays statt Domain-Objekten\n**Evidenz:**\n- `ChatController.php`: Session\/Messages als Arrays\n- Repositories geben Arrays statt Entities zurück\n\n---\n\n## Top 3 Maßnahmen\n\n### 1. Ein DI-Weg\n- Container als einziges Wiring\n- ServiceLocator entfernen\n- Alle Fallback `new`-Konstruktionen in UseCases entfernen\n- Falsche Registrierungen werden sofort sichtbar\n\n### 2. DB-Connection zentralisieren\n- PDO-Injection in Repositories\/Services\n- Keine `createConnection()` in Fachservices\n- Keine statischen `DatabaseFactory`-Calls im Fachcode\n\n### 3. Große Klassen aufteilen\n- `SemanticExplorerRepository` → Entity, Relation, Chunk, Classification, Search\n- `ChunkSyncService` → kleinere Steps mit je einer Responsibility\n\n---\n\n## Nächster Schritt\nDeterministische Liste der Top 20 konkreten Fundstellen mit:\n- Datei\n- Methode\n- Responsibility\n- Betroffenes Prinzip\n- Minimale Änderungsrichtung",
        "type": "ai_task"
    }
}

{
    "tool_response": [
        {
            "type": "text",
            "text": "{\n  \"success\": true,\n  \"task\": {\n    \"id\": 238,\n    \"uuid\": \"ab7f6f24-0b6a-4582-a527-7a4ad53163e4\",\n    \"title\": \"Architektur-Refactoring: DI-Vereinheitlichung und Klassen-Aufteilung\",\n    \"description\": \"## Supervision-Feedback: Architektur-Muster\\n\\n### A) Dependency Injection uneinheitlich\\n**Problem:** DI Container + ServiceLocator + statische Factorys + UseCases mit Default-Konstruktion\\n**Evidenz:**\\n- `src\/Infrastructure\/Container\/ServiceLocator.php`: globaler Zustand, statische Registry\\n- `src\/UseCases\/Chat\/SendChatMessageUseCase.php`: optional dependencies mit Fallback `new Repository()`\\n- `src\/services.php`: registriert UseCases ohne Abhängigkeiten trotz Constructor-Dependencies\\n\\n### B) Persistence-Verbindungslogik mehrfach verteilt\\n**Problem:** PDO-Verbindungen per static DatabaseFactory, deprecated Connection, createConnection-Methoden\\n**Evidenz:**\\n- `DatabaseFactory.php`: statische Connection-Erzeugung\\n- `Connection.php`: deprecated Singleton\\n- `ChunkingService`, `ChunkAnalysisService`, `HybridSearchService`, `ChunkSyncService`: jeweils private `createConnection()`\\n\\n### C) Große Klassen mit zu vielen Verantwortungen\\n**Problem:** SRP-Bruch, hohe Kopplung\\n**Evidenz:**\\n- `SemanticExplorerRepository.php`: 797 Zeilen (CRUD, Filter, Stats, Relations, Chunks, Classifications)\\n- `ChunkSyncService.php`: 551 Zeilen (Orchestrierung, IO, DB, Pipeline)\\n\\n### D) Domain Layer anämisch, Arrays dominieren\\n**Problem:** Controller\/UseCases arbeiten mit assoziativen Arrays statt Domain-Objekten\\n**Evidenz:**\\n- `ChatController.php`: Session\/Messages als Arrays\\n- Repositories geben Arrays statt Entities zurück\\n\\n---\\n\\n## Top 3 Maßnahmen\\n\\n### 1. Ein DI-Weg\\n- Container als einziges Wiring\\n- ServiceLocator entfernen\\n- Alle Fallback `new`-Konstruktionen in UseCases entfernen\\n- Falsche Registrierungen werden sofort sichtbar\\n\\n### 2. DB-Connection zentralisieren\\n- PDO-Injection in Repositories\/Services\\n- Keine `createConnection()` in Fachservices\\n- Keine statischen `DatabaseFactory`-Calls im Fachcode\\n\\n### 3. Große Klassen aufteilen\\n- `SemanticExplorerRepository` → Entity, Relation, Chunk, Classification, Search\\n- `ChunkSyncService` → kleinere Steps mit je einer Responsibility\\n\\n---\\n\\n## Nächster Schritt\\nDeterministische Liste der Top 20 konkreten Fundstellen mit:\\n- Datei\\n- Methode\\n- Responsibility\\n- Betroffenes Prinzip\\n- Minimale Änderungsrichtung\",\n    \"type\": \"ai_task\",\n    \"status\": \"pending\",\n    \"created_by\": \"mcp-tasks\",\n    \"created_by_type\": \"ai\",\n    \"parent_task_id\": null,\n    \"due_date\": null,\n    \"created_at\": \"2025-12-23T06:06:18.788692\",\n    \"updated_at\": \"2025-12-23T06:06:18.788695\",\n    \"completed_at\": null,\n    \"metadata\": {}\n  },\n  \"message\": \"Task #238 created\"\n}"
        }
    ]
}