Protokoll #6556

{
    "event": "PreToolUse",
    "tool_name": "Read",
    "tool_input": {
        "file_path": "\/var\/www\/dev.campus.systemische-tools.de\/docs\/plan-architecture-gate-enforcement.md"
    }
}

{
    "tool_response": {
        "type": "text",
        "file": {
            "filePath": "\/var\/www\/dev.campus.systemische-tools.de\/docs\/plan-architecture-gate-enforcement.md",
            "content": "# Plan: Architecture Gate Contract Enforcement\n\n## Status Quo\n\n| Komponente | Existiert | Enforced |\n|------------|-----------|----------|\n| Contract YAML | Ja (ID 11) | Nein |\n| Pre-Hook | Nein | - |\n| Post-Hook | Nein | - |\n| Validierung | Nein | - |\n\n**Problem:** Contract ist nur Dokumentation. Keine technische Durchsetzung.\n\n---\n\n## Ziel\n\nJede neue PHP-Datei muss den `architecture-gate-contract` passieren:\n- **Pre-Creation:** Blockierung bei Violation (A1-A9)\n- **Post-Creation:** Warnung oder Rollback bei Violation (B1-B14)\n\n---\n\n## Implementierungsplan\n\n### Phase 1: Pre-Hook (Blockierend)\n\n**Datei:** `\/opt\/scripts\/hooks\/architecture_guard.py`\n\n**Trigger:** `PreToolUse` für `Write` auf `*.php`\n\n**Prüfungen (hartcodiert, deterministisch):**\n\n| Regel | Pattern | Aktion |\n|-------|---------|--------|\n| B4\/B12 | `new \\w+Repository\\(` in Controller | BLOCK |\n| B4\/B12 | `new \\w+Service\\(` in Controller | BLOCK |\n| B4 | `DatabaseFactory::` außerhalb Factory | BLOCK |\n| B4 | `::createConnection()` in Service | BLOCK |\n| B5 | `use Infrastructure\\` in Domain | BLOCK |\n| B10 | Fehlendes `declare(strict_types=1)` | BLOCK |\n| B11 | Namespace stimmt nicht mit Pfad | BLOCK |\n\n**Output bei Violation:**\n```json\n{\n  \"allowed\": false,\n  \"message\": \"ARCHITECTURE VIOLATION [B4]: new Repository in Controller not allowed. Use DI via constructor.\"\n}\n```\n\n### Phase 2: Post-Hook (Warnend)\n\n**Datei:** `\/opt\/scripts\/hooks\/architecture_validator.py`\n\n**Trigger:** `PostToolUse` für `Write|Edit` auf `*.php`\n\n**Prüfungen:**\n\n| Regel | Check | Aktion |\n|-------|-------|--------|\n| B1 | Klasse > 300 Zeilen | WARN |\n| B2 | Duplizierte Methoden-Signaturen | WARN |\n| B3 | Mehr als 5 Constructor-Parameter | WARN |\n| B7 | Business Logic Pattern in Controller | WARN |\n| B14 | Keine PHPDoc an public methods | WARN |\n\n**Output:**\n```\n[ARCHITECTURE] Warnings for NewFile.php:\n  - B1: Class has 450 lines (max 300)\n  - B3: Constructor has 7 parameters (max 5)\n```\n\n### Phase 3: Integration mit Contract-System\n\n**Langfristig:** Hook liest Regeln dynamisch aus Contract YAML statt Hardcoding.\n\n```python\ncontract = load_contract('architecture-gate-contract')\nfor rule in contract['phase_2_post_creation_validation']:\n    check_rule(file_content, rule)\n```\n\n---\n\n## Hook-Konfiguration\n\n```json\n{\n  \"hooks\": {\n    \"PreToolUse\": [\n      {\n        \"matcher\": \"Write\",\n        \"hooks\": [\n          {\n            \"type\": \"command\",\n            \"command\": \"\/opt\/scripts\/hooks\/architecture_guard.py\",\n            \"timeout\": 5\n          }\n        ]\n      }\n    ],\n    \"PostToolUse\": [\n      {\n        \"matcher\": \"Write|Edit\",\n        \"hooks\": [\n          {\n            \"type\": \"command\",\n            \"command\": \"\/opt\/scripts\/hooks\/architecture_validator.py\",\n            \"timeout\": 10\n          }\n        ]\n      }\n    ]\n  }\n}\n```\n\n---\n\n## Offene Fragen an Supervision\n\n1. **Blockieren vs. Warnen:** Welche Regeln sollen hart blockieren, welche nur warnen?\n\n2. **Rollback:** Soll bei Post-Hook-Violation automatisch zurückgerollt werden?\n\n3. **Ausnahmen:** Gibt es legitime Ausnahmen (z.B. Factory darf `new` verwenden)?\n\n4. **Granularität:** Sollen alle 23 Regeln implementiert werden oder Fokus auf Top 5?\n\n5. **Priorität:** In welcher Reihenfolge implementieren?\n   - Option A: Erst alle Pre-Hooks, dann Post-Hooks\n   - Option B: Erst kritischste Regeln (B4, B5, B10), dann Rest\n\n6. **Dynamisch vs. Statisch:**\n   - Statisch: Regeln in Python hardcoded (schneller, einfacher)\n   - Dynamisch: Regeln aus Contract YAML laden (flexibler, komplexer)\n\n---\n\n## Nächste Schritte\n\n1. [ ] Supervision reviewt diesen Plan\n2. [ ] Entscheidung: Welche Regeln zuerst?\n3. [ ] Implementierung `architecture_guard.py`\n4. [ ] Test mit bewusster Violation\n5. [ ] Integration in `\/root\/.claude\/settings.json`\n6. [ ] Dokumentation in `\/docs\/prozesse\/`\n",
            "numLines": 150,
            "startLine": 1,
            "totalLines": 150
        }
    }
}