{
"event": "PreToolUse",
"tool_name": "Edit",
"tool_input": {
"file_path": "\/var\/www\/dev.campus.systemische-tools.de\/docs\/codequality\/ist-soll-analyse.md",
"old_string": "# IST-SOLL Analyse: Qualitätskontroll-System\n\n**Datum:** 2025-12-28\n**Aktualisiert:** 2025-12-28 (Phase 1 + Phase 2 implementiert)\n**Basis:** `richtlinien.md` (15 Prinzipien) + Hook-System\n\n---\n\n## Executive Summary\n\nDas Hook-System deckt nach Phase 1 Implementation **ca. 66%** der definierten Prinzipien durch automatisierte Kontrollen ab. Die verbleibenden 34% erfordern entweder Phase 2 Erweiterungen, externe Tools oder sind prinzipiell nicht automatisierbar (Design-Review erforderlich).\n\n### Phase 1 Status: ABGESCHLOSSEN\n\n| Neue Regel | Typ | Prinzip | Status |\n|------------|-----|---------|--------|\n| P8.1-P8.5 | BLOCK | Deterministic (time, date, rand) | Aktiv |\n| P8.6-P8.9 | BLOCK | No Global State (global, GLOBALS, SESSION) | Aktiv |\n| W8.1-W8.5 | WARN | Fail Fast (empty catch, @-operator) | Aktiv |\n| W15.1-W15.6 | WARN | Least Surprise (getter sideeffects, naming) | Aktiv |\n\n**Neue Regeln Phase 1:** +11 Pre-Hook, +11 Post-Hook\n\n### Phase 2 Status: ABGESCHLOSSEN\n\n| Neue Regel | Typ | Prinzip | Status |\n|------------|-----|---------|--------|\n| P14.1-P14.3 | BLOCK | Test-DB-Isolation (ki_content, ki_dev) | Aktiv |\n| P14.4-P14.5 | BLOCK | Gefährliche Ops (TRUNCATE, DROP) | Aktiv |\n| P14.6 | BLOCK | Static State ohne tearDown | Aktiv |\n| W14.1-W14.7 | WARN | Test-Quality (setUp, sleep, assertions) | Aktiv |\n\n**Gesamt nach Phase 1+2:** **29 PHP-Regeln, 38 Warn-Regeln**",
"new_string": "# IST-SOLL Analyse: Qualitätskontroll-System\n\n**Datum:** 2025-12-28\n**Aktualisiert:** 2025-12-28 (Phase 1 + Phase 2 + Phase 3 implementiert)\n**Basis:** `richtlinien.md` (15 Prinzipien) + Hook-System\n\n---\n\n## Executive Summary\n\nDas Hook-System deckt nach Phase 3 Implementation **ca. 75%** der definierten Prinzipien durch automatisierte Kontrollen ab. Die verbleibenden 25% erfordern Design-Review oder sind prinzipiell nicht automatisierbar.\n\n### Phase 1 Status: ABGESCHLOSSEN\n\n| Neue Regel | Typ | Prinzip | Status |\n|------------|-----|---------|--------|\n| P8.1-P8.5 | BLOCK | Deterministic (time, date, rand) | Aktiv |\n| P8.6-P8.9 | BLOCK | No Global State (global, GLOBALS, SESSION) | Aktiv |\n| W8.1-W8.5 | WARN | Fail Fast (empty catch, @-operator) | Aktiv |\n| W15.1-W15.6 | WARN | Least Surprise (getter sideeffects, naming) | Aktiv |\n\n**Neue Regeln Phase 1:** +11 Pre-Hook, +11 Post-Hook\n\n### Phase 2 Status: ABGESCHLOSSEN\n\n| Neue Regel | Typ | Prinzip | Status |\n|------------|-----|---------|--------|\n| P14.1-P14.3 | BLOCK | Test-DB-Isolation (ki_content, ki_dev) | Aktiv |\n| P14.4-P14.5 | BLOCK | Gefährliche Ops (TRUNCATE, DROP) | Aktiv |\n| P14.6 | BLOCK | Static State ohne tearDown | Aktiv |\n| W14.1-W14.7 | WARN | Test-Quality (setUp, sleep, assertions) | Aktiv |\n\n### Phase 3 Status: ABGESCHLOSSEN\n\n| Neue Regel | Typ | Prinzip | Status |\n|------------|-----|---------|--------|\n| W9.1-W9.6 | WARN | Fail Safe (try-finally, cleanup) | Aktiv |\n| W10.1-W10.6 | WARN | Trade-off Docs (Factory, Cache, Deprecation) | Aktiv |\n| W13.1-W13.6 | WARN | Cohesion (LCOM, Coupling) via PHPMetrics | Aktiv |\n\n**PHPMetrics Integration:**\n- Installiert unter `\/opt\/php-tools\/vendor\/bin\/phpmetrics`\n- Cache: `\/tmp\/phpmetrics_cache.json` (1h TTL)\n- Cohesion-Script: `\/var\/www\/tools\/ki-protokoll\/claude-hook\/quality\/phpmetrics_check.py`\n\n**ADR-Template:** `\/var\/www\/dev.campus.systemische-tools.de\/docs\/templates\/adr-template.md`\n\n**Gesamt nach Phase 1+2+3:** **31 BLOCK-Regeln, 50 WARN-Regeln (81 total)**"
}
}
{
"tool_response": {
"filePath": "\/var\/www\/dev.campus.systemische-tools.de\/docs\/codequality\/ist-soll-analyse.md",
"oldString": "# IST-SOLL Analyse: Qualitätskontroll-System\n\n**Datum:** 2025-12-28\n**Aktualisiert:** 2025-12-28 (Phase 1 + Phase 2 implementiert)\n**Basis:** `richtlinien.md` (15 Prinzipien) + Hook-System\n\n---\n\n## Executive Summary\n\nDas Hook-System deckt nach Phase 1 Implementation **ca. 66%** der definierten Prinzipien durch automatisierte Kontrollen ab. Die verbleibenden 34% erfordern entweder Phase 2 Erweiterungen, externe Tools oder sind prinzipiell nicht automatisierbar (Design-Review erforderlich).\n\n### Phase 1 Status: ABGESCHLOSSEN\n\n| Neue Regel | Typ | Prinzip | Status |\n|------------|-----|---------|--------|\n| P8.1-P8.5 | BLOCK | Deterministic (time, date, rand) | Aktiv |\n| P8.6-P8.9 | BLOCK | No Global State (global, GLOBALS, SESSION) | Aktiv |\n| W8.1-W8.5 | WARN | Fail Fast (empty catch, @-operator) | Aktiv |\n| W15.1-W15.6 | WARN | Least Surprise (getter sideeffects, naming) | Aktiv |\n\n**Neue Regeln Phase 1:** +11 Pre-Hook, +11 Post-Hook\n\n### Phase 2 Status: ABGESCHLOSSEN\n\n| Neue Regel | Typ | Prinzip | Status |\n|------------|-----|---------|--------|\n| P14.1-P14.3 | BLOCK | Test-DB-Isolation (ki_content, ki_dev) | Aktiv |\n| P14.4-P14.5 | BLOCK | Gefährliche Ops (TRUNCATE, DROP) | Aktiv |\n| P14.6 | BLOCK | Static State ohne tearDown | Aktiv |\n| W14.1-W14.7 | WARN | Test-Quality (setUp, sleep, assertions) | Aktiv |\n\n**Gesamt nach Phase 1+2:** **29 PHP-Regeln, 38 Warn-Regeln**",
"newString": "# IST-SOLL Analyse: Qualitätskontroll-System\n\n**Datum:** 2025-12-28\n**Aktualisiert:** 2025-12-28 (Phase 1 + Phase 2 + Phase 3 implementiert)\n**Basis:** `richtlinien.md` (15 Prinzipien) + Hook-System\n\n---\n\n## Executive Summary\n\nDas Hook-System deckt nach Phase 3 Implementation **ca. 75%** der definierten Prinzipien durch automatisierte Kontrollen ab. Die verbleibenden 25% erfordern Design-Review oder sind prinzipiell nicht automatisierbar.\n\n### Phase 1 Status: ABGESCHLOSSEN\n\n| Neue Regel | Typ | Prinzip | Status |\n|------------|-----|---------|--------|\n| P8.1-P8.5 | BLOCK | Deterministic (time, date, rand) | Aktiv |\n| P8.6-P8.9 | BLOCK | No Global State (global, GLOBALS, SESSION) | Aktiv |\n| W8.1-W8.5 | WARN | Fail Fast (empty catch, @-operator) | Aktiv |\n| W15.1-W15.6 | WARN | Least Surprise (getter sideeffects, naming) | Aktiv |\n\n**Neue Regeln Phase 1:** +11 Pre-Hook, +11 Post-Hook\n\n### Phase 2 Status: ABGESCHLOSSEN\n\n| Neue Regel | Typ | Prinzip | Status |\n|------------|-----|---------|--------|\n| P14.1-P14.3 | BLOCK | Test-DB-Isolation (ki_content, ki_dev) | Aktiv |\n| P14.4-P14.5 | BLOCK | Gefährliche Ops (TRUNCATE, DROP) | Aktiv |\n| P14.6 | BLOCK | Static State ohne tearDown | Aktiv |\n| W14.1-W14.7 | WARN | Test-Quality (setUp, sleep, assertions) | Aktiv |\n\n### Phase 3 Status: ABGESCHLOSSEN\n\n| Neue Regel | Typ | Prinzip | Status |\n|------------|-----|---------|--------|\n| W9.1-W9.6 | WARN | Fail Safe (try-finally, cleanup) | Aktiv |\n| W10.1-W10.6 | WARN | Trade-off Docs (Factory, Cache, Deprecation) | Aktiv |\n| W13.1-W13.6 | WARN | Cohesion (LCOM, Coupling) via PHPMetrics | Aktiv |\n\n**PHPMetrics Integration:**\n- Installiert unter `\/opt\/php-tools\/vendor\/bin\/phpmetrics`\n- Cache: `\/tmp\/phpmetrics_cache.json` (1h TTL)\n- Cohesion-Script: `\/var\/www\/tools\/ki-protokoll\/claude-hook\/quality\/phpmetrics_check.py`\n\n**ADR-Template:** `\/var\/www\/dev.campus.systemische-tools.de\/docs\/templates\/adr-template.md`\n\n**Gesamt nach Phase 1+2+3:** **31 BLOCK-Regeln, 50 WARN-Regeln (81 total)**",
"originalFile": "# IST-SOLL Analyse: Qualitätskontroll-System\n\n**Datum:** 2025-12-28\n**Aktualisiert:** 2025-12-28 (Phase 1 + Phase 2 implementiert)\n**Basis:** `richtlinien.md` (15 Prinzipien) + Hook-System\n\n---\n\n## Executive Summary\n\nDas Hook-System deckt nach Phase 1 Implementation **ca. 66%** der definierten Prinzipien durch automatisierte Kontrollen ab. Die verbleibenden 34% erfordern entweder Phase 2 Erweiterungen, externe Tools oder sind prinzipiell nicht automatisierbar (Design-Review erforderlich).\n\n### Phase 1 Status: ABGESCHLOSSEN\n\n| Neue Regel | Typ | Prinzip | Status |\n|------------|-----|---------|--------|\n| P8.1-P8.5 | BLOCK | Deterministic (time, date, rand) | Aktiv |\n| P8.6-P8.9 | BLOCK | No Global State (global, GLOBALS, SESSION) | Aktiv |\n| W8.1-W8.5 | WARN | Fail Fast (empty catch, @-operator) | Aktiv |\n| W15.1-W15.6 | WARN | Least Surprise (getter sideeffects, naming) | Aktiv |\n\n**Neue Regeln Phase 1:** +11 Pre-Hook, +11 Post-Hook\n\n### Phase 2 Status: ABGESCHLOSSEN\n\n| Neue Regel | Typ | Prinzip | Status |\n|------------|-----|---------|--------|\n| P14.1-P14.3 | BLOCK | Test-DB-Isolation (ki_content, ki_dev) | Aktiv |\n| P14.4-P14.5 | BLOCK | Gefährliche Ops (TRUNCATE, DROP) | Aktiv |\n| P14.6 | BLOCK | Static State ohne tearDown | Aktiv |\n| W14.1-W14.7 | WARN | Test-Quality (setUp, sleep, assertions) | Aktiv |\n\n**Gesamt nach Phase 1+2:** **29 PHP-Regeln, 38 Warn-Regeln**\n\n---\n\n## 1. IST-Zustand: Bestehendes Hook-System\n\n### 1.1 Architektur\n\n```\n\/var\/www\/tools\/ki-protokoll\/claude-hook\/\n├── hook_dispatcher.py ← Einstiegspunkt (settings.json)\n├── architecture_guard.py ← Legacy H-Regeln\n├── block_direct_db.py ← MCP-DB Enforcement\n├── task_completion_guard.py ← Task-Workflow Enforcement\n├── file_backup_hook.py ← Automatische Backups\n├── log_to_db.py ← Session-Protokollierung\n└── quality\/\n ├── pre_rules.py ← BLOCK-Regeln (Master)\n ├── pre_rules_guard.py ← P1.x SRP\/KISS\n ├── pre_rules_mvc.py ← P2.x MVC\/CRUD\n ├── pre_rules_validation.py ← P3.x PSR\/Types\n ├── pre_rules_layers.py ← P6.x SOLID\/DIP + H2-H5\n ├── pre_rules_htmx.py ← HTMX-Contract\n ├── pre_rules_constants.py ← Magic Numbers\n ├── pre_rules_python.py ← Python-Regeln\n ├── pre_rules_deterministic.py ← P8.x Deterministic (Phase 1)\n ├── pre_rules_tests.py ← P14.x Test-Isolation (Phase 2)\n ├── post_rules.py ← WARN-Regeln (Master)\n ├── rules_quality.py ← W1.x Metriken\n ├── rules_security.py ← W2.x Security\n ├── rules_style.py ← W4.x Style\n ├── rules_constants.py ← W7.x Constants\n ├── rules_failfast.py ← W8.x Fail Fast (Phase 1)\n ├── rules_leastsurprise.py ← W15.x Least Surprise (Phase 1)\n ├── rules_testisolation.py ← W14.x Test-Isolation (Phase 2)\n ├── task_creator.py ← Violations → Tasks\n └── full_scan.py ← Codebase-Scan CLI\n```\n\n### 1.2 Aktive Regeln\n\n#### Pre-Hook (BLOCK) - 36 Regeln\n\n| ID | Regel | Scope | Prinzip |\n|----|-------|-------|---------|\n| H1 | strict_types erforderlich | *.php | Explicit |\n| H2 | Domain ohne Infrastructure | \/Domain\/ | Clean Architecture |\n| H3 | DatabaseFactory nur in Factory | !Factory\/ | Separation of Concerns |\n| H4 | Kein new Repository in Controller | \/Controller\/ | DIP |\n| H5 | Kein new Infrastructure in Controller | \/Controller\/ | DIP |\n| P1.1 | @responsibility Header | *.php | SRP |\n| P1.2 | Müllhalden-Namen blockiert | *.php | KISS |\n| P2.1 | Kein SQL in Controller | \/Controller\/ | Separation of Concerns |\n| P2.2 | Keine Transaktionen in Controller | \/Controller\/ | Separation of Concerns |\n| P2.3 | Kein echo\/print in Controller | \/Controller\/ | MVC |\n| P2.4 | Keine DB-Artefakte in UseCases | \/UseCases\/ | DIP |\n| P3.1 | strict_types in ersten Zeilen | *.php | Explicit |\n| P3.2 | Namespace = Pfad | *.php | Explicit |\n| P3.3 | Klassenname = Dateiname | *.php | Explicit |\n| P3.4 | Public Methods Return-Type | *.php | Explicit |\n| P4.1 | Keine public Properties in Domain | \/Domain\/ | OOP\/Encapsulation |\n| P6.1 | UseCases ohne Infrastructure-Import | \/UseCases\/ | DIP |\n| P6.2 | Domain ohne Application-Import | \/Domain\/ | Clean Architecture |\n| HTMX-C1 | hx-post mit CSRF | *.php | Security |\n| HTMX-C2 | hx-delete mit CSRF | *.php | Security |\n| HTMX-C3 | hx-patch mit CSRF | *.php | Security |\n| HTMX-C4 | hx-delete mit confirm | *.php | Fail Safe |\n| HTMX-C5 | hx-put mit CSRF | *.php | Security |\n| P7.x | Magic Numbers blockiert | *.php | DRY |\n| P8.1 | time() verboten | *.php | Deterministic |\n| P8.2 | date() verboten | *.php | Deterministic |\n| P8.3 | rand() verboten | *.php | Deterministic |\n| P8.4 | mt_rand() verboten | *.php | Deterministic |\n| P8.5 | random_int() verboten | *.php | Deterministic |\n| P8.6 | global $var verboten | *.php | Deterministic |\n| P8.7 | $GLOBALS verboten | *.php | Deterministic |\n| P8.8 | $_SESSION verboten | *.php (nicht Framework) | Deterministic |\n| P8.9 | $_COOKIE verboten | *.php (nicht Framework) | Deterministic |\n| P14.1 | ki_content in Tests | \/tests\/ | Test Isolation |\n| P14.2 | ki_dev in Tests | \/tests\/ | Test Isolation |\n| P14.3 | ki_protokoll in Tests | \/tests\/ | Test Isolation |\n| P14.4 | TRUNCATE in Tests | \/tests\/ | Test Isolation |\n| P14.5 | DROP in Tests | \/tests\/ | Test Isolation |\n| P14.6 | Static ohne tearDown | \/tests\/ | Test Isolation |\n\n#### Post-Hook (WARN) - 38 Regeln\n\n| ID | Regel | Schwelle | Prinzip |\n|----|-------|----------|---------|\n| W1.1 | Klassengröße (LOC) | > 300 | SRP |\n| W1.2 | Public Methods | > 10 | SRP |\n| W1.3 | Constructor Parameters | > 5 | Cohesion |\n| W1.4 | Dependencies (use) | > 10 | Stable Dependencies |\n| W1.5 | Verdächtige Namen | Manager, And | KISS |\n| W2.1 | Business-Keywords in Controller | - | Separation of Concerns |\n| W2.2 | Private Methods in Controller | > 5 | MVC |\n| W3.1 | Untypisierte Parameter | - | Explicit |\n| W3.3 | mixed Type | - | Explicit |\n| W4.1 | Anämisches Model | > 70% Accessors | OOP |\n| W4.2 | Klasse ohne Verhalten | 0 real methods | OOP |\n| W4.3 | Niedrige Kapselung | > 50% exposed | OOP |\n| W4.4 | Hohe Static-Ratio | > 50% | OOP |\n| W5.1 | Wiederholte Strings | > 20 chars, > 3x | DRY |\n| W5.2 | Magic Numbers (Warn) | nicht Common, > 2x | DRY |\n| W6.1 | Interface-Methoden | > 7 | Interface Segregation |\n| W8.1 | Leerer catch-Block | - | Fail Fast |\n| W8.2 | @-Error Suppression | - | Fail Fast |\n| W8.3 | Catch nur mit return | - | Fail Fast |\n| W8.4 | Generische Exception | - | Fail Fast |\n| W8.5 | throw in __destruct | - | Fail Fast |\n| W15.1 | Getter mit Seiteneffekten | - | Least Surprise |\n| W15.2 | Setter mit Return-Wert | - | Least Surprise |\n| W15.3 | Bool ohne is\/has Präfix | - | Least Surprise |\n| W15.4 | Constructor mit return | - | Least Surprise |\n| W15.5 | void Methode mit echo | - | Least Surprise |\n| W15.6 | __toString mit throw | - | Least Surprise |\n| W14.1 | setUp ohne tearDown | - | Test Isolation |\n| W14.2 | Filesystem ohne Cleanup | - | Test Isolation |\n| W14.3 | Global State Modifikation | - | Test Isolation |\n| W14.4 | Hardcoded Pfade | - | Test Isolation |\n| W14.5 | sleep() in Tests | - | Test Isolation |\n| W14.6 | Viele @depends | > 2 | Test Isolation |\n| W14.7 | Test ohne Assertions | - | Test Isolation |\n\n#### Externe Checks (Bash-Scripts)\n\n| Script | Prüft | Prinzip |\n|--------|-------|---------|\n| php-check.sh | PHPStan, CS-Fixer, Semgrep, Audit | Deterministic, Fail Fast |\n| contract-check.sh | View-Structure, Layers, Docs | Separation of Concerns |\n| di-check.sh | UseCase Dependencies | DIP |\n| interface-check.sh | Interface-Implementierung | Clean Architecture |\n\n---\n\n## 2. Prinzipien-Abdeckung\n\n### 2.1 Mapping: Richtlinien → Kontrollen\n\n| # | Prinzip | Abdeckung | Kontrolle | Status |\n|---|---------|-----------|-----------|--------|\n| 1 | **Explicit is Better than Implicit** | **90%** | H1, P3.1-P3.4, W3.1, W3.3 | ✅ Gut |\n| 2 | **Deterministic Behavior** | **70%** | P8.1-P8.9 (time, rand, global) | ✅ Gut |\n| 3 | **DRY** | **50%** | P7.x, W5.1, W5.2, Constants | ⚠️ Teilweise |\n| 4 | **Separation of Concerns** | **80%** | H2-H5, P2.x, P6.x, MVC-Checks | ✅ Gut |\n| 5 | **Clean Architecture** | **70%** | H2, P6.1, P6.2, Layer-Checks | ✅ Gut |\n| 6 | **Stable Dependencies** | **60%** | W1.4, P6.x, DI-Check | ⚠️ Teilweise |\n| 7 | **Fail Fast** | **80%** | W8.1-W8.5 (catch, @-operator) | ✅ Gut |\n| 8 | **Fail Safe** | **40%** | HTMX-C4, W8.5 (destructor) | ⚠️ Teilweise |\n| 9 | **KISS** | **70%** | P1.2, W1.x Metriken | ✅ Gut |\n| 10 | **Make Trade-offs Explicit** | **10%** | Nur via @responsibility | ❌ Lücke |\n| 11 | **SRP** | **80%** | P1.1, P1.2, W1.1-W1.5 | ✅ Gut |\n| 12 | **YAGNI** | **0%** | Nicht automatisierbar | ❌ Design-Review |\n| 13 | **Cohesion over Coupling** | **50%** | W1.3, W1.4, W6.2 | ⚠️ Teilweise |\n| 14 | **Test Isolation** | **80%** | P14.1-P14.6, W14.1-W14.7 | ✅ Gut |\n| 15 | **Least Surprise Principle** | **60%** | W15.1-W15.6 (getter, naming) | ⚠️ Teilweise |\n\n### 2.2 Zusammenfassung (nach Phase 1+2)\n\n| Kategorie | Anzahl | Anteil |\n|-----------|--------|--------|\n| ✅ Gut abgedeckt (≥70%) | 10 | 67% |\n| ⚠️ Teilweise abgedeckt (30-69%) | 2 | 13% |\n| ❌ Nicht abgedeckt (<30%) | 3 | 20% |\n\n**Verbesserung durch Phase 1+2:** +4 Prinzipien in \"Gut abgedeckt\" Kategorie\n- Phase 1: Deterministic, Fail Fast, Least Surprise\n- Phase 2: Test Isolation\n\n---\n\n## 3. SOLL-Zustand: Lücken schließen\n\n### 3.1 Automatisierbare Lücken\n\n#### Deterministic Behavior (Priorität 2)\n\n**Problem:** Keine Prüfung auf versteckten globalen Zustand, nicht-deterministische Funktionen.\n\n**Lösung:**\n\n```python\n# Pre-Hook P8.1: Verbiete problematische Funktionen\nFORBIDDEN_FUNCTIONS = [\n r\"\\btime\\s*\\(\", # Verwende Clock-Interface\n r\"\\bdate\\s*\\(\", # Verwende Clock-Interface\n r\"\\brand\\s*\\(\", # Verwende RandomGenerator-Interface\n r\"\\bmt_rand\\s*\\(\", # Verwende RandomGenerator-Interface\n r\"\\bglobal\\s+\\$\", # Kein global keyword\n r\"\\$GLOBALS\\b\", # Ke... [TRUNCATED-11f2c4413744b376]",
"structuredPatch": [
{
"oldStart": 1,
"oldLines": 14,
"newStart": 1,
"newLines": 14,
"lines": [
" # IST-SOLL Analyse: Qualitätskontroll-System",
" ",
" **Datum:** 2025-12-28",
"-**Aktualisiert:** 2025-12-28 (Phase 1 + Phase 2 implementiert)",
"+**Aktualisiert:** 2025-12-28 (Phase 1 + Phase 2 + Phase 3 implementiert)",
" **Basis:** `richtlinien.md` (15 Prinzipien) + Hook-System",
" ",
" ---",
" ",
" ## Executive Summary",
" ",
"-Das Hook-System deckt nach Phase 1 Implementation **ca. 66%** der definierten Prinzipien durch automatisierte Kontrollen ab. Die verbleibenden 34% erfordern entweder Phase 2 Erweiterungen, externe Tools oder sind prinzipiell nicht automatisierbar (Design-Review erforderlich).",
"+Das Hook-System deckt nach Phase 3 Implementation **ca. 75%** der definierten Prinzipien durch automatisierte Kontrollen ab. Die verbleibenden 25% erfordern Design-Review oder sind prinzipiell nicht automatisierbar.",
" ",
" ### Phase 1 Status: ABGESCHLOSSEN",
" "
]
},
{
"oldStart": 30,
"oldLines": 8,
"newStart": 30,
"newLines": 23,
"lines": [
" | P14.6 | BLOCK | Static State ohne tearDown | Aktiv |",
" | W14.1-W14.7 | WARN | Test-Quality (setUp, sleep, assertions) | Aktiv |",
" ",
"-**Gesamt nach Phase 1+2:** **29 PHP-Regeln, 38 Warn-Regeln**",
"+### Phase 3 Status: ABGESCHLOSSEN",
" ",
"+| Neue Regel | Typ | Prinzip | Status |",
"+|------------|-----|---------|--------|",
"+| W9.1-W9.6 | WARN | Fail Safe (try-finally, cleanup) | Aktiv |",
"+| W10.1-W10.6 | WARN | Trade-off Docs (Factory, Cache, Deprecation) | Aktiv |",
"+| W13.1-W13.6 | WARN | Cohesion (LCOM, Coupling) via PHPMetrics | Aktiv |",
"+",
"+**PHPMetrics Integration:**",
"+- Installiert unter `\/opt\/php-tools\/vendor\/bin\/phpmetrics`",
"+- Cache: `\/tmp\/phpmetrics_cache.json` (1h TTL)",
"+- Cohesion-Script: `\/var\/www\/tools\/ki-protokoll\/claude-hook\/quality\/phpmetrics_check.py`",
"+",
"+**ADR-Template:** `\/var\/www\/dev.campus.systemische-tools.de\/docs\/templates\/adr-template.md`",
"+",
"+**Gesamt nach Phase 1+2+3:** **31 BLOCK-Regeln, 50 WARN-Regeln (81 total)**",
"+",
" ---",
" ",
" ## 1. IST-Zustand: Bestehendes Hook-System"
]
}
],
"userModified": false,
"replaceAll": false
}
}