Plan — solcom-review Figma-Achse

solcom-review — dritte Achse „Figma-Parity"

Skill solcom-review erhält neben den bestehenden Achsen Standards und Spec eine dritte parallele Sub-Agent-Achse Figma-Parity. Trigger: Diff matched web/themes/custom/byte_theme/components/** oder web/themes/custom/byte_theme/src/**. Anker: docs/design/MAPPING.html liefert Figma-Frame-Key pro betroffenem SDC. Auslöser für den Plan: PR-Review P2503-84 (Header / Main-Nav / Footer) deckte auf, dass es heute keine automatische Figma-Compliance-Prüfung im PR-Review-Loop gibt — ein FE-PR durchläuft Standards + Spec, aber visuelle Drift gegenüber dem Design-System wird nur manuell entdeckt.

Drei tracer-bullet Slices. Jede für sich grabbar, Tests/Smoke nach jeder Slice.

Frame-Lookup-Mechanik (MAPPING-Parser inline im Skill)

Touch: SKILL.md-Schritt 3.5 (neu) · AFK

Der Skill bekommt einen neuen Pre-Step zwischen 3 (Standards-Korpora) und 4 (Sub-Agent-Spawn): liest docs/design/MAPPING.html, extrahiert pro <article class="card"> das Tupel { sdc-name, frame-key, status, verified-date } aus den <code class="node">- und <div class="sdc">-Knoten. Filtert auf SDCs, die im Diff geändert wurden (git diff --name-only origin/develop...HEAD | grep components/). Skip-Pfad: keine Treffer → keine Figma-Achse (kein Spawn, Hinweis im Output).

Beispiel-Output für P2503-84-Diff:

[
  { "sdc": "byte_theme/navbar",  "frame": "61148:47024", "status": "ready-for-dev", "verified": "2026-05-13" },
  { "sdc": "byte_theme/footer",  "frame": "9167b5c4",    "status": "wip",           "verified": "2026-05-04" }
]

Akzeptanzkriterien

Parser handhabt node-id sowohl im Figma-URL-Format (61148:47024) als auch componentKey-Format.
Filter respektiert nur SDC-Files (*.twig, *.tailwind.css, *.component.yml) — nicht .theme-Hook-Edits.
Bei Status gap oder obsolete: SDC im Diff, aber kein lebendes Mapping — explizit als „no-anchor-skip" markieren (nicht als Fehler).
Bei mehreren Frames pro SDC (matched + ready-for-dev): den jeweils ready-for-dev/matched-Eintrag bevorzugen, der verified-Date höher rankt.
Falls MAPPING.html fehlt oder leer ist: gracefully skippen, keine Exception.

Skill-Erweiterung: dritter parallel-spawned Sub-Agent

Touch: SKILL.md-Schritte 4 und 5 · AFK

Wenn S1 ≥1 Tupel liefert, spawnt der Skill in der selben Nachricht (parallel zu Standards + Spec) einen dritten Agent-Tool-Call mit subagent_type: "solcom-figma-reviewer" (kein general-purpose — der Reviewer hat dedizierte Figma-MCP-Tools). Input: das Tupel-Array aus S1, dazu der git-diff-Befehl, dazu Glob-Pfade zu den Twig-/CSS-Files für direkten Read durch den Reviewer. Aggregations-Schritt 5 bekommt eine dritte Heading ## Figma neben ## Standards und ## Spec.

Prompt-Skelett für den Figma-Sub-Agent:

Du bist solcom-figma-reviewer. Vergleiche pro SDC den Diff gegen den Figma-Frame.
Inputs:
  - working-dir: 
  - sdc-frame-pairs:
      byte_theme/navbar → 61148:47024
      byte_theme/footer → 9167b5c4
  - diff-cmd: git diff origin/develop...HEAD 

Pro SDC:
  1. Figma-Frame via mcp__claude_ai_Figma__get_design_context laden.
  2. Twig + CSS + component.yml des SDC lesen.
  3. Auf Spacing/Farben/Typo/Variants/Inhalt vergleichen.
  4. Status setzen: matched | partial | drift.
  5. Detail-Findings (Soll vs. Ist, Quelle: Figma-Variable-Name oder
     Twig-File:Line).

Auth-Fail-Verhalten: ≤2 Retries; danach NEEDS_REAUTH-Note (ADR-0020),
kein Hard-Stop für den Gesamt-Run.

Akzeptanzkriterien

Drei Agent-Spawns in einer einzigen Nachricht (parallel-Pattern).
Wenn S1 leer → kein dritter Spawn, Aggregations-Heading ## Figma erscheint mit no SDC diff — Figma-Achse übersprungen.
Sub-Agent-Prompt nennt den Pfad zum byte_theme/AGENTS.md als Brand-Token-Referenz (damit Findings auf Tokens mappen statt Hex-Vergleich).
Sub-Agent-Modell explizit opus oder sonnet — Visual-Pattern-Matching ist nicht Haiku-tauglich (Memory feedback_visual_verify_side_by_side).

Report-Template + Persistierung um Figma-Sektion erweitern

Touch: 03-code-review-pr.html + SKILL.md-Schritt 6 · AFK

Im persistierten Report (docs/reports/<YYYY-MM-DD>-<branch>-review.html) kommt eine dritte Top-Level-Section  nach  mit dem gleichen axis-card-Layout (eine Card pro SDC, Status-Tag matched/partial/drift, Findings-Liste). Risk-Map-Chips bekommen Status-Klasse chip.attention bei drift, chip.medium bei partial, chip.safe bei matched. Story-Hub-Write-Back-Format (Schritt 6.5) wird um eine zweite Datenzeile ergänzt: standards: N hard / M judgement, spec: K, figma: matched=A, partial=B, drift=C.

Akzeptanzkriterien

Template 03-code-review-pr.html behält Style-Pattern (axis-card, risk-tag, finding-label) bei.
Frontmatter-Meta um solcom:figma-frames erweitert (Liste der genutzten Frame-Keys, für späteren Lint).
Bei MCP-Auth-Fail rendert die Figma-Section eine warning-card mit dem NEEDS_REAUTH-Hinweis statt der Findings — Report bleibt vollständig.
LOG.html-Eintrag „decide" sagt zusätzlich figma: matched=A, partial=B, drift=C im Title.
Smoke: /solcom-review auf einer Branch ohne SDC-Diff (z. B. reiner Config-Change) rendert die Section trotzdem mit einer placeholder-card „no SDC diff" — keine harten Crashes.

solcom-review — dritte Achse „Figma-Parity"

Kontext

Vertical Slices

Frame-Lookup-Mechanik (MAPPING-Parser inline im Skill)

Skill-Erweiterung: dritter parallel-spawned Sub-Agent

Report-Template + Persistierung um Figma-Sektion erweitern

Risiken

Offene Items / Pre-Implementation-Gates