Brainstorm · solcom · 2026-05-02

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

Kontext 12 Skills unter .claude/skills/ definieren Authoring-Konventionen für Twig/SDC, Brand-Tokens, Drupal-PHP und Config. Im Diff-time-Review sind diese Konventionen nicht abgedeckt — generische CE-Reviewer kennen weder Drupal noch SOLCOM-Brand. Drei Topologie-Optionen für Stack-spezifische Diff-time Reviewer-Agents.
Status (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; der Body beschreibt den Zustand vor der Rücknahme.

A4 spezialisierte Agents (gewählt)

Vier projektlokale Diff-time Reviewer — je einer pro Stack-Schicht (Brand, Twig/SDC, Drupal-PHP, Config/Architektur) — mit File-Glob-Triggern und Skill-Referenz als Source of Truth. 

# .claude/agents/solcom-brand-reviewer.md
---
triggers:
  file_globs:
    - "**/*.css"
    - "**/*.twig"
    - "**/*.svg"
---

# .claude/agents/drupal-twig-sdc-reviewer.md
---
triggers:
  file_globs:
    - "**/*.twig"
    - "**/*.component.yml"
---

# .claude/agents/drupal-php-reviewer.md
---
triggers:
  file_globs:
    - "**/*.php"
    - "**/*.module"
    - "**/*.install"
---

# .claude/agents/drupal-config-architecture-reviewer.md
---
triggers:
  file_globs:
    - "config/sync/**"
    - "config/splits/**"
---
Pro
Con
Granularität: jeder Reviewer kennt seine Schicht präzise
4 Agent-Files statt 1 — mehr Wartungsobjekte
Parallele Ausführung wenn Diff mehrere Schichten betrifft
Overlap möglich (Brand + Twig melden denselben Token-Fehler)
Skill-Referenz = DRY, kein Regelduplicat
CLAUDE.md-Konsolidierungsregel erfordert Klarstellung
Passiv wenn kein Glob matcht — kein Rauschen
Finding-Aggregation bei Reviewer-Overlap komplex
Agents: 4 Trigger: File-Glob Parallel: ja Skill-Referenz: DRY

B1 konsolidierter Solcom-Reviewer

Ein einzelner solcom-code-reviewer-Agent deckt alle vier Stack-Schichten ab — triggert auf alle relevanten File-Globs, prüft alle Lenses sequenziell. 

# .claude/agents/solcom-code-reviewer.md
---
triggers:
  file_globs:
    - "**/*.css"
    - "**/*.twig"
    - "**/*.component.yml"
    - "**/*.php"
    - "**/*.module"
    - "**/*.install"
    - "config/sync/**"
    - "config/splits/**"
---
# Prüft alle 4 Lenses sequenziell:
# 1. Brand-Compliance
# 2. Twig/SDC-Konventionen
# 3. Drupal-PHP-Patterns
# 4. Config-Architektur

# Refs alle 4 Skill-Files
Pro
Con
1 File statt 4 — einfachere Wartung
Kein paralleles Reviewer-Running möglich
Kein Finding-Overlap zwischen Reviewern
Großes Prompt: alle 4 Skill-Referenzen gleichzeitig aktiv
Konsolidierungsregel aus CLAUDE.md klar erfüllt
Triggert auf jeden PR mit irgendeinem Glob-Match
Einfache Onboarding-Erklärung
Lens-Mischung kann Finding-Qualität verschlechtern
Agents: 1 Trigger: alle Globs Parallel: nein Kontext-Last: hoch

CSkills um ## Review-Abschnitt erweitern

Kein separater Reviewer-Agent. Stattdessen bekommt jeder Skill einen ## Review-Abschnitt — die Review-Logik lebt im Skill selbst. Kein eigener Agent-Trigger nötig. 

# .claude/skills/drupal-twig-sdc-authoring/SKILL.md

## Authoring
# … bestehende Regeln …

## Review
Diff-time-Prüfliste:

1. Alle Twig-Includes mit `only`
   (Context-Isolation, Regel 13)
2. Kein direkter Zugriff auf `node`
   in SDC (SDC-Boundary)
3. CVA via `html_cva()`, nicht
   Inline-Concat
4. Slot-Schema deklariert für
   alle verwendeten Slots

# Brand-Skill analog:
## Review
1. Keine hardcoded Hex-Werte
2. Brand-Tokens aus theme.brand.css
3. …
Pro
Con
Authoring + Review koexistieren in einer Datei
Kein automatischer Trigger — manueller Aufruf nötig
DRY: Regel steht genau einmal
Größere Architektur-Änderung an bestehenden Skills
Kein neues Agent-Konzept einführen
Skill-Trigger greift bei Authoring, nicht bei Diff-Review
Wartung liegt beim Skill-Owner
Diff-Review-Context fehlt Skills grundsätzlich
Agents: 0 (Skill-Erweiterung) Trigger: manuell Architektur-Change: hoch Diff-Context: fehlt