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.