From 4f4ed8f47dcf2afa37b2169d25d056750af23eb2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Tr=C3=B6ger?= Date: Sun, 19 Apr 2026 16:48:09 +0200 Subject: [PATCH] =?UTF-8?q?refactor:=20Pr=C3=A4zisiere=20plan-review-Ausga?= =?UTF-8?q?ben=20f=C3=BCr=20bessere=20Klarheit?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Verbessere die Begriffe in plan-review-Ausgabeformat: - Risiko: verdeutliche dass dies kein Blocker ist, nur Abwägung - Fehlt → Blocker: fokussiere auf echte Showstopper, nicht spekulative Tests Dies macht Pläne klarer und reduziert Falsch-Positive in der Review. Co-Authored-By: Claude Sonnet 4.6 --- skills/plan-review/SKILL.md | 6 +++--- skills/story/references/phase-3-plan-review.md | 11 ++++++----- 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/skills/plan-review/SKILL.md b/skills/plan-review/SKILL.md index 23c0e70..0a965bd 100644 --- a/skills/plan-review/SKILL.md +++ b/skills/plan-review/SKILL.md @@ -26,10 +26,10 @@ Ausgabe-Format: - [was in Ordnung ist] ### ⚠️ Risiko -- [potenzielle Probleme mit Empfehlung] +- [potenzielle Probleme mit Empfehlung — kein Blocker, aber bewusst abwägen] -### ❌ Fehlt -- [was ergänzt werden muss, bevor implementiert werden darf] +### ❌ Blocker +- [nur echte Showstopper: fehlende Migration, fehlendes Auth-Konzept, fehlende Akzeptanzkriterien — NICHT "könnte man noch testen"] ### Empfehlung [Implementierung freigeben / Plan zuerst anpassen] diff --git a/skills/story/references/phase-3-plan-review.md b/skills/story/references/phase-3-plan-review.md index fc304f8..d96e789 100644 --- a/skills/story/references/phase-3-plan-review.md +++ b/skills/story/references/phase-3-plan-review.md @@ -2,9 +2,10 @@ Rufe `/plan-review` auf — ein unabhängiger Opus-Agent prüft den Plan. -Warte auf das Ergebnis: -- Bei **❌ Fehlt**: Plan anpassen, dann erneut `/plan-review` aufrufen -- Bei **⚠️ Risiko**: Abwägen — entweder Plan anpassen oder Risiko bewusst akzeptieren und dokumentieren -- Bei **✅ Freigabe**: Weiter mit Phase 4 +Warte auf das Ergebnis und präsentiere es dem User. Dann fragen: -Erst nach Freigabe mit der Implementierung beginnen. +- Bei **❌ Blocker**: User fragen ob Plan angepasst und nochmal reviewt werden soll — kein automatischer Folge-Durchlauf +- Bei **⚠️ Risiko**: User entscheiden lassen — Risiko akzeptieren oder Plan anpassen +- Bei **✅ OK / keine Blocker**: Weiter mit Phase 4 + +Der User entscheidet immer — kein automatisches Re-Review. -- 2.54.0