diff --git a/templates/commands/plan.md b/templates/commands/plan.md index 4c5cbf3..18a0b5c 100644 --- a/templates/commands/plan.md +++ b/templates/commands/plan.md @@ -19,7 +19,7 @@ Given the implementation details provided as an argument, do this: 4. Execute the implementation plan template: - Load `/templates/plan-template.md` (already copied to IMPL_PLAN path) - Set Input path to FEATURE_SPEC - - Run the Execution Flow (main) function steps 1-10 + - Run the Execution Flow (main) function steps 1-9 - The template is self-contained and executable - Follow error handling and gate checks as specified - Let the template guide artifact generation in $SPECS_DIR: diff --git a/templates/plan-template.md b/templates/plan-template.md index e88aa1e..4f9e401 100644 --- a/templates/plan-template.md +++ b/templates/plan-template.md @@ -17,18 +17,19 @@ scripts: 2. Fill Technical Context (scan for NEEDS CLARIFICATION) → Detect Project Type from context (web=frontend+backend, mobile=app+api) → Set Structure Decision based on project type -3. Evaluate Constitution Check section below +3. Fill the Constitution Check section based on the content of the constitution document. +4. Evaluate Constitution Check section below → If violations exist: Document in Complexity Tracking → If no justification possible: ERROR "Simplify approach first" → Update Progress Tracking: Initial Constitution Check -4. Execute Phase 0 → research.md +5. Execute Phase 0 → research.md → If NEEDS CLARIFICATION remain: ERROR "Resolve unknowns" -5. Execute Phase 1 → contracts, data-model.md, quickstart.md, agent-specific template file (e.g., `CLAUDE.md` for Claude Code, `.github/copilot-instructions.md` for GitHub Copilot, or `GEMINI.md` for Gemini CLI). -6. Re-evaluate Constitution Check section +6. Execute Phase 1 → contracts, data-model.md, quickstart.md, agent-specific template file (e.g., `CLAUDE.md` for Claude Code, `.github/copilot-instructions.md` for GitHub Copilot, or `GEMINI.md` for Gemini CLI). +7. Re-evaluate Constitution Check section → If new violations: Refactor design, return to Phase 1 → Update Progress Tracking: Post-Design Constitution Check -7. Plan Phase 2 → Describe task generation approach (DO NOT create tasks.md) -8. STOP - Ready for /tasks command +8. Plan Phase 2 → Describe task generation approach (DO NOT create tasks.md) +9. STOP - Ready for /tasks command ``` **IMPORTANT**: The /plan command STOPS at step 7. Phases 2-4 are executed by other commands: @@ -52,35 +53,7 @@ scripts: ## Constitution Check *GATE: Must pass before Phase 0 research. Re-check after Phase 1 design.* -**Simplicity**: -- Projects: [#] (max 3 - e.g., api, cli, tests) -- Using framework directly? (no wrapper classes) -- Single data model? (no DTOs unless serialization differs) -- Avoiding patterns? (no Repository/UoW without proven need) - -**Architecture**: -- EVERY feature as library? (no direct app code) -- Libraries listed: [name + purpose for each] -- CLI per library: [commands with --help/--version/--format] -- Library docs: llms.txt format planned? - -**Testing (NON-NEGOTIABLE)**: -- RED-GREEN-Refactor cycle enforced? (test MUST fail first) -- Git commits show tests before implementation? -- Order: Contract→Integration→E2E→Unit strictly followed? -- Real dependencies used? (actual DBs, not mocks) -- Integration tests for: new libraries, contract changes, shared schemas? -- FORBIDDEN: Implementation before test, skipping RED phase - -**Observability**: -- Structured logging included? -- Frontend logs → backend? (unified stream) -- Error context sufficient? - -**Versioning**: -- Version number assigned? (MAJOR.MINOR.BUILD) -- BUILD increments on every change? -- Breaking changes handled? (parallel tests, migration plan) +[Gates determined based on constitution file] ## Project Structure