Protokoll #6667

{
    "event": "PreToolUse",
    "tool_name": "Write",
    "tool_input": {
        "file_path": "\/var\/www\/dev.campus.systemische-tools.de\/docs\/plan-code-quality-gates.md",
        "content": "# Plan: Code Quality Gates\n\n## Prüfungsreihenfolge (nicht verhandelbar)\n\n| # | Prinzip | Prüfzeitpunkt |\n|---|---------|---------------|\n| 1 | SRP + KISS | Pre + Post |\n| 2 | MVC + CRUD | Pre + Post |\n| 3 | OOP | Post |\n| 4 | DRY | Post |\n| 5 | SOLID + DIP | Pre |\n| 6 | PSR + Types | Pre |\n\n---\n\n## Logik: Pre vs. Post\n\n**Pre-Hook (blockiert):**\n- Strukturelle Fehler, die sofort erkennbar sind\n- Verletzungen, die nie entstehen dürfen\n\n**Post-Hook (warnt):**\n- Qualitätsindikatoren, die Kontext brauchen\n- Metriken, die über Zeit gemessen werden\n\n---\n\n# 1. SRP + KISS\n\n## Definition\n\n**SRP (Single Responsibility Principle):**\nEine Klasse hat genau einen Grund sich zu ändern.\n\n**KISS (Keep It Simple Stupid):**\nKeine unnötige Komplexität. Keine vorzeitigen Abstraktionen.\n\n---\n\n## Pre-Hook Prüfungen (blockierend)\n\n### P1.1: Verantwortungs-Header erforderlich\n\n**Regel:** Jede neue PHP-Datei braucht einen `@responsibility` Header.\n\n**Format:**\n```php\n<?php\ndeclare(strict_types=1);\n\n\/\/ @responsibility: <ein-satz-beschreibung>\n```\n\n**Prüfung:**\n```python\npattern = r\"\/\/\\s*@responsibility:\\s*.+\"\nif not re.search(pattern, content):\n    block(\"Missing @responsibility header\")\n```\n\n**Warum Pre:** Ohne explizite Verantwortung entsteht Scope Creep.\n\n---\n\n### P1.2: Keine Multi-Responsibility Keywords\n\n**Regel:** Dateiname und Inhalt dürfen keine Multi-Responsibility signalisieren.\n\n**Verbotene Patterns:**\n```\n*AndService.php      → \"User AND Auth\" = 2 Verantwortungen\n*ManagerController   → \"Manager\" = unklar\n*HelperService       → \"Helper\" = Müllhalde\n*UtilsRepository     → \"Utils\" = Müllhalde\n```\n\n**Prüfung:**\n```python\nforbidden_names = [\"And\", \"Manager\", \"Helper\", \"Utils\", \"Misc\", \"Common\", \"Base\"]\nfor word in forbidden_names:\n    if word in filename:\n        block(f\"Forbidden name pattern: {word} suggests multiple responsibilities\")\n```\n\n**Warum Pre:** Diese Namen sind Symptome für SRP-Verletzung.\n\n---\n\n## Post-Hook Prüfungen (warnend)\n\n### W1.1: Klassengröße\n\n**Regel:** Klasse > 200 LOC = Warnung, > 300 LOC = starke Warnung.\n\n**Prüfung:**\n```python\nloc = count_non_empty_lines(content)\nif loc > 300:\n    warn(f\"Class has {loc} LOC (max 300). Consider splitting.\")\nelif loc > 200:\n    warn(f\"Class approaching limit: {loc}\/300 LOC\")\n```\n\n**Warum Post:** Größe allein ist kein harter Fehler, aber ein Indikator.\n\n---\n\n### W1.2: Methoden-Anzahl\n\n**Regel:** Klasse > 10 public methods = Warnung.\n\n**Prüfung:**\n```python\npublic_methods = re.findall(r\"public\\s+function\\s+\\w+\", content)\nif len(public_methods) > 10:\n    warn(f\"Class has {len(public_methods)} public methods. Consider splitting.\")\n```\n\n**Warum Post:** Viele Methoden = viele Gründe sich zu ändern.\n\n---\n\n### W1.3: Constructor-Parameter\n\n**Regel:** Constructor > 5 Parameter = Warnung.\n\n**Prüfung:**\n```python\nconstructor = re.search(r\"__construct\\s*\\([^)]*\\)\", content)\nif constructor:\n    params = constructor.group().count(\"$\")\n    if params > 5:\n        warn(f\"Constructor has {params} parameters (max 5). Consider refactoring.\")\n```\n\n**Warum Post:** Viele Dependencies = zu viel Verantwortung.\n\n---\n\n### W1.4: KISS - Verschachtelungstiefe\n\n**Regel:** > 3 Ebenen Verschachtelung = Warnung.\n\n**Prüfung:**\n```python\nmax_indent = 0\nfor line in content.split(\"\\n\"):\n    indent = len(line) - len(line.lstrip())\n    spaces = indent \/\/ 4  # oder tabs\n    max_indent = max(max_indent, spaces)\nif max_indent > 4:\n    warn(f\"Nesting depth {max_indent} (max 4). Simplify logic.\")\n```\n\n**Warum Post:** Tiefe Verschachtelung = komplexe Logik.\n\n---\n\n## Zusammenfassung Prüfung 1\n\n| ID | Typ | Prüfung | Aktion |\n|----|-----|---------|--------|\n| P1.1 | Pre | @responsibility Header fehlt | BLOCK |\n| P1.2 | Pre | Verbotene Namenspattern | BLOCK |\n| W1.1 | Post | > 300 LOC | WARN |\n| W1.2 | Post | > 10 public methods | WARN |\n| W1.3 | Post | > 5 constructor params | WARN |\n| W1.4 | Post | > 4 nesting levels | WARN |\n\n---\n\n## Nächste Schritte\n\n1. [ ] P1.1 + P1.2 in Pre-Hook implementieren\n2. [ ] W1.1 - W1.4 in Post-Hook implementieren\n3. [ ] Tests für alle Prüfungen\n4. [ ] Weiter mit Prüfung 2 (MVC + CRUD)\n\n---\n\n# 2. MVC + CRUD\n\n*Wird nach Abschluss von Prüfung 1 definiert.*\n\n---\n\n# 3. OOP\n\n*Wird nach Abschluss von Prüfung 2 definiert.*\n\n---\n\n# 4. DRY\n\n*Wird nach Abschluss von Prüfung 3 definiert.*\n\n---\n\n# 5. SOLID + DIP\n\n*Wird nach Abschluss von Prüfung 4 definiert.*\n\n---\n\n# 6. PSR + Types\n\n*Wird nach Abschluss von Prüfung 5 definiert.*\n"
    }
}

{
    "tool_response": {
        "type": "create",
        "filePath": "\/var\/www\/dev.campus.systemische-tools.de\/docs\/plan-code-quality-gates.md",
        "content": "# Plan: Code Quality Gates\n\n## Prüfungsreihenfolge (nicht verhandelbar)\n\n| # | Prinzip | Prüfzeitpunkt |\n|---|---------|---------------|\n| 1 | SRP + KISS | Pre + Post |\n| 2 | MVC + CRUD | Pre + Post |\n| 3 | OOP | Post |\n| 4 | DRY | Post |\n| 5 | SOLID + DIP | Pre |\n| 6 | PSR + Types | Pre |\n\n---\n\n## Logik: Pre vs. Post\n\n**Pre-Hook (blockiert):**\n- Strukturelle Fehler, die sofort erkennbar sind\n- Verletzungen, die nie entstehen dürfen\n\n**Post-Hook (warnt):**\n- Qualitätsindikatoren, die Kontext brauchen\n- Metriken, die über Zeit gemessen werden\n\n---\n\n# 1. SRP + KISS\n\n## Definition\n\n**SRP (Single Responsibility Principle):**\nEine Klasse hat genau einen Grund sich zu ändern.\n\n**KISS (Keep It Simple Stupid):**\nKeine unnötige Komplexität. Keine vorzeitigen Abstraktionen.\n\n---\n\n## Pre-Hook Prüfungen (blockierend)\n\n### P1.1: Verantwortungs-Header erforderlich\n\n**Regel:** Jede neue PHP-Datei braucht einen `@responsibility` Header.\n\n**Format:**\n```php\n<?php\ndeclare(strict_types=1);\n\n\/\/ @responsibility: <ein-satz-beschreibung>\n```\n\n**Prüfung:**\n```python\npattern = r\"\/\/\\s*@responsibility:\\s*.+\"\nif not re.search(pattern, content):\n    block(\"Missing @responsibility header\")\n```\n\n**Warum Pre:** Ohne explizite Verantwortung entsteht Scope Creep.\n\n---\n\n### P1.2: Keine Multi-Responsibility Keywords\n\n**Regel:** Dateiname und Inhalt dürfen keine Multi-Responsibility signalisieren.\n\n**Verbotene Patterns:**\n```\n*AndService.php      → \"User AND Auth\" = 2 Verantwortungen\n*ManagerController   → \"Manager\" = unklar\n*HelperService       → \"Helper\" = Müllhalde\n*UtilsRepository     → \"Utils\" = Müllhalde\n```\n\n**Prüfung:**\n```python\nforbidden_names = [\"And\", \"Manager\", \"Helper\", \"Utils\", \"Misc\", \"Common\", \"Base\"]\nfor word in forbidden_names:\n    if word in filename:\n        block(f\"Forbidden name pattern: {word} suggests multiple responsibilities\")\n```\n\n**Warum Pre:** Diese Namen sind Symptome für SRP-Verletzung.\n\n---\n\n## Post-Hook Prüfungen (warnend)\n\n### W1.1: Klassengröße\n\n**Regel:** Klasse > 200 LOC = Warnung, > 300 LOC = starke Warnung.\n\n**Prüfung:**\n```python\nloc = count_non_empty_lines(content)\nif loc > 300:\n    warn(f\"Class has {loc} LOC (max 300). Consider splitting.\")\nelif loc > 200:\n    warn(f\"Class approaching limit: {loc}\/300 LOC\")\n```\n\n**Warum Post:** Größe allein ist kein harter Fehler, aber ein Indikator.\n\n---\n\n### W1.2: Methoden-Anzahl\n\n**Regel:** Klasse > 10 public methods = Warnung.\n\n**Prüfung:**\n```python\npublic_methods = re.findall(r\"public\\s+function\\s+\\w+\", content)\nif len(public_methods) > 10:\n    warn(f\"Class has {len(public_methods)} public methods. Consider splitting.\")\n```\n\n**Warum Post:** Viele Methoden = viele Gründe sich zu ändern.\n\n---\n\n### W1.3: Constructor-Parameter\n\n**Regel:** Constructor > 5 Parameter = Warnung.\n\n**Prüfung:**\n```python\nconstructor = re.search(r\"__construct\\s*\\([^)]*\\)\", content)\nif constructor:\n    params = constructor.group().count(\"$\")\n    if params > 5:\n        warn(f\"Constructor has {params} parameters (max 5). Consider refactoring.\")\n```\n\n**Warum Post:** Viele Dependencies = zu viel Verantwortung.\n\n---\n\n### W1.4: KISS - Verschachtelungstiefe\n\n**Regel:** > 3 Ebenen Verschachtelung = Warnung.\n\n**Prüfung:**\n```python\nmax_indent = 0\nfor line in content.split(\"\\n\"):\n    indent = len(line) - len(line.lstrip())\n    spaces = indent \/\/ 4  # oder tabs\n    max_indent = max(max_indent, spaces)\nif max_indent > 4:\n    warn(f\"Nesting depth {max_indent} (max 4). Simplify logic.\")\n```\n\n**Warum Post:** Tiefe Verschachtelung = komplexe Logik.\n\n---\n\n## Zusammenfassung Prüfung 1\n\n| ID | Typ | Prüfung | Aktion |\n|----|-----|---------|--------|\n| P1.1 | Pre | @responsibility Header fehlt | BLOCK |\n| P1.2 | Pre | Verbotene Namenspattern | BLOCK |\n| W1.1 | Post | > 300 LOC | WARN |\n| W1.2 | Post | > 10 public methods | WARN |\n| W1.3 | Post | > 5 constructor params | WARN |\n| W1.4 | Post | > 4 nesting levels | WARN |\n\n---\n\n## Nächste Schritte\n\n1. [ ] P1.1 + P1.2 in Pre-Hook implementieren\n2. [ ] W1.1 - W1.4 in Post-Hook implementieren\n3. [ ] Tests für alle Prüfungen\n4. [ ] Weiter mit Prüfung 2 (MVC + CRUD)\n\n---\n\n# 2. MVC + CRUD\n\n*Wird nach Abschluss von Prüfung 1 definiert.*\n\n---\n\n# 3. OOP\n\n*Wird nach Abschluss von Prüfung 2 definiert.*\n\n---\n\n# 4. DRY\n\n*Wird nach Abschluss von Prüfung 3 definiert.*\n\n---\n\n# 5. SOLID + DIP\n\n*Wird nach Abschluss von Prüfung 4 definiert.*\n\n---\n\n# 6. PSR + Types\n\n*Wird nach Abschluss von Prüfung 5 definiert.*\n",
        "structuredPatch": [],
        "originalFile": null
    }
}