flow-cc 0.5.6 → 0.5.8

package/CHANGELOG.md

@@ -5,6 +5,13 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.5.8] - 2026-02-14
+
+### Fixed
+- `/flow:go` Step 7 now enforces mandatory `/flow:done` suggestion in every phase completion output
+- Added STOP RULE to prevent autonomous post-phase work (PR creation, comment resolution, cleanup)
+- Separated "next flow commands" from project-specific actions so `/flow:done` never gets dropped
+
 ## [0.5.0] - 2026-02-13
 
 ### Added

package/VERSION

@@ -1 +1 @@
-0.5.6
+0.5.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "flow-cc",
-  "version": "0.5.6",
+  "version": "0.5.8",
   "description": "Structured workflow system for Claude Code — spec interviews, agent-team execution, session handoffs, compounding knowledge",
   "author": "Troy Hoffman",
   "license": "MIT",

package/skills/flow-done.md

@@ -10,6 +10,8 @@ You are executing the `/flow:done` skill. This finalizes the current session by
 
 **This is the most important skill for sustainability.** Without proper session-end docs, the next session starts blind.
 
+**Skill boundary:** You are inside the `/flow:*` workflow. NEVER invoke, suggest, or reference skills from other workflow systems (`/lisa:*`, `/gsd:*`, `/superpowers:*`, etc.). Only suggest `/flow:*` commands as next steps. Do NOT use the Skill tool to call any non-flow skill.
+
 ## Steps
 
 ### 1. Gather Context

package/skills/flow-go.md

@@ -12,26 +12,27 @@ You are executing the `/flow:go` skill. This reads the PRD, identifies the next
 
 **Plan mode warning:** Do NOT use this skill with plan mode enabled. `/flow:go` is execution — plan mode's read-only constraint prevents it from creating files, running agents, and committing work. The PRD is your plan; run `/flow:go` in normal mode.
 
+**Skill boundary:** You are inside the `/flow:*` workflow. NEVER invoke, suggest, or reference skills from other workflow systems (`/lisa:*`, `/gsd:*`, `/superpowers:*`, etc.). Only suggest `/flow:*` commands as next steps. Do NOT use the Skill tool to call any non-flow skill. If the user needs a different workflow, they will invoke it themselves.
+
 ## Step 1 — Orient
 
 Read these files (in parallel):
 - `.planning/STATE.md` — current position
 - `.planning/ROADMAP.md` — phase progress
-- The active PRD (see PRD Resolution below)
 - `tasks/lessons.md` — active lessons (max 10 one-liners)
 - `CLAUDE.md` — execution rules and verification commands
 
-### PRD Resolution
+### PRD Selection
+
+The user must always select which PRD to execute. No silent auto-resolution.
+
+1. **If the user passed an argument** (e.g., `/flow:go v3-payments`) — match it against files in `.planning/prds/` by slug or by the `**Milestone:**` header field. If an exact match is found, use it directly. If no match, show available PRDs and ask.
 
-Resolve the PRD file to use, in this order:
+2. **If no argument** — list all PRD files in `.planning/prds/`. For each PRD, read its `**Status:**` and `**Milestone:**` header fields. Use AskUserQuestion to let the user pick which PRD to execute. Pre-select the first PRD with status "Ready for execution" as the first option. Always show the picker, even if only one PRD exists — the user may want to confirm or run `/flow:spec` instead. Also check for legacy `PRD.md` at root (backward compat) and include it in the list if found.
 
-1. **User argument:** If the user passed an argument to `/flow:go` (e.g., `/flow:go v3-payments`), match it against files in `.planning/prds/` (by slug or milestone name from the `**Milestone:**` header field).
-2. **STATE.md Active PRD:** Read the "Active PRD" field from STATE.md's Current Position section. If it points to a valid file, use it.
-3. **Slug derivation:** Derive a slug from STATE.md's current milestone name — version-prefix + lowercase name, spaces/special chars → hyphens, collapse consecutive hyphens (e.g., "v2: Dashboard Analytics" → `v2-dashboard-analytics`). Check `.planning/prds/{slug}.md`.
-4. **Legacy fallback:** If `.planning/prds/` is empty or missing but `PRD.md` exists at the project root, use it. Print: "Using legacy PRD.md at project root. Future specs will use `.planning/prds/`."
-5. **Not found:** "No PRD found for [current milestone name]. Available PRDs: [list files in `.planning/prds/`]. Run `/flow:spec` first."
+3. **No PRDs found** — "No PRDs found in `.planning/prds/`. Run `/flow:spec` first." Stop here.
 
-**Milestone match check:** After resolving the PRD, read its `**Milestone:**` header field. If it doesn't match STATE.md's current milestone, warn: "PRD milestone ([PRD milestone]) doesn't match current milestone ([STATE milestone]). Continuing, but verify you're executing the right spec."
+**After selection:** Read the chosen PRD. If its `**Milestone:**` doesn't match STATE.md's current milestone, warn: "PRD milestone ([PRD milestone]) doesn't match current milestone ([STATE milestone]). Continuing, but verify you're executing the right spec."
 
 **Identify the next phase:** Find the first phase in ROADMAP.md with status "Pending" or the first unstarted phase in the PRD.
 
@@ -39,7 +40,7 @@ Resolve the PRD file to use, in this order:
 
 Run these checks before executing. If any fail, stop and tell the user what to do:
 
-1. **PRD resolved?** If PRD Resolution (above) reached step 5 (not found): stop with the "No PRD found" message and available PRD list.
+1. **PRD selected?** If PRD Selection (above) reached step 3 (no PRDs found): stop with the "No PRDs found" message.
 2. **Phase detailed enough?** The phase section in the PRD must have:
    - Wave structure with agent assignments
    - Explicit file lists per agent
@@ -162,18 +163,35 @@ Create an atomic commit for this phase:
 
 **ROADMAP.md:** Mark this phase as "Complete ([today's date])"
 
-## Step 7 — Route Next Action
+## Step 7 — Route Next Action (MANDATORY — FINAL STEP)
+
+**STOP RULE:** This is the LAST thing you do. After printing the output below, STOP IMMEDIATELY. Do NOT:
+- Review or resolve code review comments
+- Create or update pull requests
+- Run additional cleanup or refactoring
+- Do any work beyond printing this summary
+
+Any post-phase work belongs in a SEPARATE `/flow:go` invocation or `/flow:task`.
+
+Print this EXACT structure (fill in values):
 
-Print phase summary:
 ```
-Phase [N]: [Name] — Complete
+Phase [N]: [Name] — Complete ✓
 - [X] files created, [Y] modified
 - Commit: [SHA]
-- Verification: passed
+- Verification: [passed/failed]
+
+Next flow command:
+→ /flow:go — execute Phase [N+1]: [Next Phase Name]
+→ /flow:done — end session, update docs, generate handoff prompt
+```
+
+If this was the last phase, replace the flow commands block with:
+
+```
+All phases complete — milestone done!
 
-Next:
-- /flow:go for Phase [N+1]: [Next Phase Name]
-- /flow:done to end session
+→ /flow:done — finalize session (REQUIRED before ending)
 ```
 
-If this was the last phase: "All phases complete! Run `/flow:done` to finalize."
+**CRITICAL:** The `→ /flow:done` line MUST appear in EVERY phase completion output, whether or not more phases remain. This is non-negotiable. `/flow:done` is how session-end documentation happens.

package/skills/flow-spec.md

@@ -12,6 +12,8 @@ You are executing the `/flow:spec` skill. This is the KEYSTONE skill of the flow
 
 **Plan mode warning:** Do NOT use this skill with plan mode enabled. Plan mode's read-only constraint prevents the PRD from being written during the interview. `/flow:spec` IS the planning phase — plan mode on top of it is redundant and breaks the workflow.
 
+**Skill boundary:** You are inside the `/flow:*` workflow. NEVER invoke, suggest, or reference skills from other workflow systems (`/lisa:*`, `/gsd:*`, `/superpowers:*`, etc.). Only suggest `/flow:*` commands as next steps. Do NOT use the Skill tool to call any non-flow skill. If the user needs a different workflow, they will invoke it themselves.
+
 ## Phase 1 — Context Gathering
 
 1. Read `.planning/STATE.md` and `.planning/ROADMAP.md` — understand current milestone and what's done

package/skills/flow-status.md

@@ -8,6 +8,8 @@ user_invocable: true
 
 You are executing the `/flow:status` skill. This is a READ-ONLY operation. Do NOT modify any files.
 
+**Skill boundary:** You are inside the `/flow:*` workflow. NEVER invoke, suggest, or reference skills from other workflow systems (`/lisa:*`, `/gsd:*`, `/superpowers:*`, etc.). Only suggest `/flow:*` commands as next steps. Do NOT use the Skill tool to call any non-flow skill.
+
 ## Step 1 — Read Context Files
 
 Read ALL of the following in parallel:

package/skills/flow-task.md

@@ -8,6 +8,8 @@ user_invocable: true
 
 You are executing the `/flow:task` skill. This is for small, focused work — bug fixes, cleanup, one-off features — that doesn't warrant a full PRD. Understand the task, execute it, verify it works, commit it, and document it.
 
+**Skill boundary:** You are inside the `/flow:*` workflow. NEVER invoke, suggest, or reference skills from other workflow systems (`/lisa:*`, `/gsd:*`, `/superpowers:*`, etc.). Only suggest `/flow:*` commands as next steps. Do NOT use the Skill tool to call any non-flow skill.
+
 RULES:
 - NO AGENT TEAMS. NO PRD. Single execution context.
 - Exception: ONE Task agent for an isolated subtask to prevent context bloat.