Implementation Plan · solcom

feat: SOLCOM Reviewer-Agents — Diff-time Symmetrie zur Skill-Library

Status-Hinweis Superseded (2026-05-02): CE-Plugin-Integration zurückgenommen. Das CE-Plugin wurde global deaktiviert; die /ce-code-review-Orchestrator-Augmentation ist nicht mehr aktiv. Die vier Reviewer-Agents bleiben unter .claude/agents/, aber ohne CE-Run-Artifact-Pfad. Sie sind jetzt manuell via Task-Tool aufrufbar. Inhaltliche Regeln decken weiterhin die zwölf Skills unter .claude/skills/ ab. Dieses Dokument bleibt als Entscheidungs-Historie.
Status
Superseded
Reviewer-Agents
4
Stack-Schichten
Brand / Twig / PHP / Config
Origin
docs/brainstorms/…agents-requirements
01

Milestones

Sieben Units: 4 Agent-Definitions (parallel möglich), CLAUDE.md-Augmentation, Inventar-Doc, Live-Verifikation. CE-Integration-Mechanik ist superseded — Agents bleiben als manuelle Task-Tool-Einheiten.

U1–U4

Vier Reviewer-Agent-Definitionen

solcom-brand-reviewer (Brand-Tokens, Anti-Hex, Logo), drupal-twig-sdc-reviewer (13 AGENTS.md-Regeln, with only, SDC-Schema), drupal-php-reviewer (DI, Hooks, SQL-Injection, Attributes), drupal-config-architecture-reviewer (Dev-Split, ADR-Pflicht, customer-lens). Frontmatter: model: inherit, Tools: Read/Grep/Glob/Bash.

solcom-brand-reviewerdrupal-twig-sdc-reviewerdrupal-php-reviewerdrupal-config-architecture-reviewer
U5

CLAUDE.md-Augmentation (superseded)

Neue H3-Sektion mit Reviewer→File-Glob-Mapping und Dispatch-Verhalten für den /ce-code-review-Orchestrator. Status: CE-Plugin deaktiviert, Augmentation inaktiv. Reviewer bleiben im Repo.

CLAUDE.mdsuperseded
U6–U7

Inventar-Doc + Live-Verifikation

docs/agents/code-review.md als Pendant zu docs/agents/skills.md. Live-Verifikation U7: synthetischer Test-PR mit 4 bewussten Violations — empirischer Beweis, ob Augmentation wirkt. Bei CE-Negativbefund: Fallback auf Wrapper-Skill /solcom-code-review.

docs/agents/code-review.mdlive-verifikation
02

Reviewer-Inventar

Vier Reviewer je einer Stack-Schicht zugeordnet. Skill-Referenz per Inline-Pfad: Skill-Update wirkt beim nächsten Lauf ohne Reviewer-Edit.

Brand + Twig/SDC — Frontend-Stack
  • solcom-brand-reviewer — Triggers: web/themes/custom/**/*.css, *.scss, *.twig, *.svg. Prüft: hardcoded Hex statt Token, oklch-Literale, Lime-vs-Accent-Verwechslung, Logo-Misuse. Skill: solcom-styleguide/SKILL.md
  • drupal-twig-sdc-reviewer — Triggers: web/themes/custom/**/*.twig, *.component.yml. Prüft: fehlendes with only, dynamische Tag-Namen, conditional required props. Quellen: Skill + byte_theme/AGENTS.md
PHP/Backend + Config/Architektur
  • drupal-php-reviewer — Triggers: web/modules/custom/**/*.php, *.module, *.install, *.routing.yml, *.services.yml. Prüft: fehlender hook_update_N, statisches \Drupal::, SQL-Injection, fehlende t(). Skill: drupal-module-development/SKILL.md
  • drupal-config-architecture-reviewer — Triggers: config/sync/**, config/splits/**, composer.json, .ddev/**, settings*.php. Prüft: Dev-Split-Verletzungen, ADR-Pflicht, settings.local.php im Diff. Skills: config-workflow + adr-workflow
03

Key Code

Agent-Frontmatter-Schema und Output-JSON-Skelett (CE-findings-schema). Output-Schema verbindlich für Stage-5-Merge-Kompatibilität.

.claude/agents/solcom-brand-reviewer.md — Frontmatter
---
name: solcom-brand-reviewer
description: >
  Conditional code-review persona, selected
  when diff touches brand-relevante Files
  (CSS-Tokens, Twig-Klassen, Logo-Assets).
  Reviews SOLCOM Corporate Design adherence
  (siehe .claude/skills/solcom-styleguide/
  SKILL.md). Triggers on diffs touching
  web/themes/custom/**/*.css, *.twig, *.svg.
tools: [Read, Grep, Glob, Bash]
model: inherit
---
Output-JSON-Skelett (CE findings-schema)
{
  "reviewer": "solcom-brand",
  "findings": [{
    "severity": "P1",
    "confidence": 75,
    "title": "Hardcoded Brand-Hex statt Token",
    "file": "button.twig",
    "line": 42,
    "autofix_class": "manual"
  }],
  "residual_risks": [],
  "testing_gaps": []
}
// severity ∈ {P0, P1, P2, P3}
// confidence ∈ {0, 25, 50, 75, 100}
// autofix_class ∈ {safe_auto, gated_auto,
//                  manual, advisory}
04

Risks & Mitigations

Risk
Sev
Mitigation
CLAUDE.md-Augmentation wird vom CE-Orchestrator-LLM nicht zuverlässig befolgt (Stage-3 liest statischen Persona-Katalog, nicht CLAUDE.md).
HIGH
U7 ist der Lackmustest. Bei Fehlschlag: Fallback auf Wrapper-Skill /solcom-code-review. Brittleness-Hint in CLAUDE.md dokumentiert das Risiko. CE-Plugin aktuell deaktiviert — Risk nicht schlagend.
findings-schema.json-Drift bei CE-Plugin-Update bricht Reviewer-Outputs.
MED
Output-Schema im Reviewer-Prompt namentlich referenziert. Erster Live-Lauf zeigt Schema-Mismatch.
Skill-File-Verschiebung oder Umbenennung bricht Inline-Pfade im Reviewer-Prompt.
MED
docs/agents/code-review.md „Wartung & Drift"-Abschnitt dokumentiert Update-Pflicht bei Skill-Umzug.
File-Glob-Heuristik in natural-language Description führt zu Falsch-Aktivierung oder Falsch-Negativ.
LOW
Glob-Strings in description sind detailliert formuliert. U7-Verifikation ist der empirische Beweis.
05

Open Questions (Deferred)

Stage 3 von /ce-code-review liest CLAUDE.md gar nicht für Reviewer-Selection
Konkrete Mechanik-Lücke: Stage-3-Reviewer-Selector enumeriert preskriptiv „from each cross-cutting and stack-specific conditional persona in the persona catalog included below" (statisch via @./references/persona-catalog.md). CLAUDE.md wird nur in Stage 3b an den project-standards-Subagent weitergereicht. Risk: Augmentations-Sektion ist ggf. nicht im Selection-Context. Falsifizierungs-Test U7 deckt das post-hoc auf. CE-Plugin aktuell deaktiviert — kein unmittelbarer Handlungsdruck.
Deferred · empirisch via U7 · marc
U7-Smoke-Test vor Vollausbau: Single-Reviewer-First-Sequenz
Vorgeschlagene Restrukturierung: Smoke-Test mit nur einem Reviewer (solcom-brand-reviewer) + minimaler CLAUDE.md-Augmentation + synthetischem Test-PR vor U2/U3/U4/U6. Reduziert Half-State-Risiko von 4-Files-verwaist auf 1-File-verwaist. Deferred, weil CE-Plugin gerade deaktiviert ist.
Deferred · vor CE-Plugin-Reaktivierung · marc