Commit Graph

28 Commits

Author SHA1 Message Date
Lex Christopherson
200e004781 feat(execute-phase): add --gaps-only flag for gap closure execution
After verify-work diagnoses issues and creates fix plans, the next
command now explicitly signals intent with --gaps-only. This eliminates
redundant state discovery where execute-phase had to figure out why it
was being asked to run on an already-complete phase.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-17 02:03:29 -06:00
Lex Christopherson
8d199427a8 fix(offer-next): remove code blocks from output templates for proper markdown rendering
Commands with <offer_next> sections were outputting backticks literally
instead of rendering them as inline code. Removed code block wrappers
and added explicit 'Output this markdown directly' instruction.

Affected: plan-phase, execute-phase, audit-milestone
2026-01-17 00:01:30 -06:00
Lex Christopherson
60ebda93b1 fix(execute-phase): recommend discuss-phase before plan-phase
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-16 23:17:28 -06:00
Lex Christopherson
6fd95c3ce5 refactor: consolidate expertise into agents
Commands load agent expertise directly via Task tool spawning.
Thin orchestrator pattern — agents have methodology baked in.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-16 16:59:42 -06:00
Lex Christopherson
12373d2f2a fix(execute-phase): commit orchestrator corrections before verification
Orchestrator sometimes makes small fixes between executor completions
(missing imports, wiring fixes). These were left uncommitted. Now
checks for dirty state after all waves complete and commits if needed.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-16 16:33:10 -06:00
Lex Christopherson
daa54737cf feat: add GSD brand system for consistent UI
Add ui-brand.md reference with visual patterns:
- Stage banners with GSD ► prefix
- Checkpoint boxes (62-char width)
- Status symbols vocabulary
- Next Up routing blocks

Update orchestrators to use branded output.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-16 11:51:57 -06:00
Lex Christopherson
70cfd76c21 fix(execute-phase): include VERIFICATION.md in phase completion commit
Phase completion commit now stages VERIFICATION.md alongside ROADMAP.md,
STATE.md, and REQUIREMENTS.md so verification results are tracked.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-15 11:35:53 -06:00
Lex Christopherson
6b4f73efd8 fix(execute-phase): recommend audit-milestone when milestone completes
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-15 10:54:03 -06:00
Lex Christopherson
8e6ad96af1 feat(verification): add verify → plan → execute loop for gap closure
- Add verify_phase_goal step to execute-phase workflow
- Verifier outputs structured gaps: YAML for planner consumption
- Add --gaps flag to plan-phase for gap closure mode
- Route by verification status: passed, gaps_found, human_needed
- Gap closure creates sequential plans (04, 05...) from verification gaps
- User stays in control at each decision point

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-15 10:34:38 -06:00
Lex Christopherson
f34b5f8a03 chore: remove obsolete templates (logic now in subagents)
Deleted:
- subagent-task-prompt.md (baked into gsd-executor)
- subagent-verify-prompt.md (baked into gsd-verifier)
- continuation-prompt.md (baked into gsd-executor)
- checkpoint-return.md (baked into gsd-executor)

Co-Authored-By: Claude <noreply@anthropic.com>
2026-01-15 10:11:28 -06:00
Lex Christopherson
f3f6707cbe feat: add gsd-verifier subagent for phase goal verification
- gsd-verifier.md: dedicated verification subagent with all logic baked in
- Goal-backward verification: checks codebase, not SUMMARY claims
- Creates VERIFICATION.md, returns status to orchestrator
- execute-phase spawns gsd-verifier instead of general-purpose

Co-Authored-By: Claude <noreply@anthropic.com>
2026-01-15 09:51:14 -06:00
Lex Christopherson
82c522b8c3 fix: update remaining general-purpose refs to gsd-executor
- execute-phase.md: wave execution example now uses gsd-executor
- execute-plan.md workflow: subagent spawning uses gsd-executor
- continuation-prompt.md: continuation spawning uses gsd-executor
- Remove obsolete subagent-task-prompt.md reference

Co-Authored-By: Claude <noreply@anthropic.com>
2026-01-15 09:40:53 -06:00
Lex Christopherson
e95186b2a7 refactor: update execute commands to spawn gsd-executor subagent
- execute-plan.md: spawn gsd-executor instead of general-purpose with template
- execute-phase.md: spawn gsd-executor for wave-based parallel execution
- Remove template filling, subagent has all logic baked in

Co-Authored-By: Claude <noreply@anthropic.com>
2026-01-15 09:34:32 -06:00
Lex Christopherson
238332513b fix(execute-phase): bundle phase metadata into single commit
ROADMAP.md, STATE.md, and REQUIREMENTS.md updates now commit
together as phase completion instead of separate commits.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-15 02:38:25 -06:00
Lex Christopherson
365c04163c refactor: move requirements status update to commands (orchestrator)
Subagents can't determine if phase is complete. The orchestrator
(execute-plan and execute-phase commands) runs in main context,
can count plans vs summaries, and updates REQUIREMENTS.md when
phase completes.

- execute-plan: Step 2.5 updates requirements when summaries = plans
- execute-phase: Step 6 updates requirements (phase always complete)
- Removed duplicate from workflow (subagent can't run it anyway)
2026-01-15 00:20:00 -06:00
Lex Christopherson
194d1d88bb refactor: slim principles.md and load in core commands
Reduce principles.md from 158 to 73 lines:
- Remove duplicates (atomic_commits, tdd, deviation_rules)
- Remove version drag from claude_automates
- Keep core orientation: solo dev model, plans are prompts,
  scope control, ship fast, anti-enterprise

Add principles.md to 9 core commands so Claude always
understands what GSD is.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-14 23:48:44 -06:00
Lex Christopherson
aaf5bceebc refactor(execute-phase): slim command, delegate to workflow
Command now references workflow for checkpoint handling details
instead of duplicating logic. Keeps command thin as intended.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-14 16:30:23 -06:00
Lex Christopherson
e98bebfeeb fix(checkpoints): restore rich documentation and fix continuation pattern
- execute-plan.md: Replace broken Task(resume=...) with fresh continuation agent
- execute-phase.md: Expand checkpoint handling with full continuation flow
- checkpoints.md: Restore ~500 lines of detailed examples and anti-patterns

Checkpoint presentation formats now documented for all three types.
Templates referenced: checkpoint-return.md, continuation-prompt.md

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-14 16:26:08 -06:00
Lex Christopherson
e8199d9263 fix(execute): restore offer_next routing to orchestrator commands
The orchestrator pattern conversion (8ed6a8f) removed workflow context
loading for context efficiency but inadvertently removed the offer_next
step that presents copy/paste-ready next commands after plan/phase
completion.

Added explicit <offer_next> sections to both commands:
- execute-plan.md: routes to next plan, next phase, or milestone complete
- execute-phase.md: routes to next phase or milestone complete

Both include /clear hints and copy/paste-ready command paths.

Fixes #69

Co-Authored-By: Claude <noreply@anthropic.com>
2026-01-14 15:33:45 -06:00
Lex Christopherson
62f12794dd chore: remove dead ISSUES.md system
Remove the global ISSUES.md deferred enhancement tracking system.

- Delete /gsd:consider-issues command (never used)
- Delete issues.md template (never instantiated)
- Remove Rule 5 from deviation rules (never triggered)
- Remove all ISSUES.md, ISS-XXX, and "deferred issues" references
- Update STATE.md to track pending todos instead

The ISSUES.md system was designed to capture non-critical enhancements
during plan execution via "Rule 5", but it never fired in practice
across 100+ projects. The system added ~350 lines of dead code.

The /gsd:add-todo and /gsd:check-todos system serves the same purpose
and is actually used.

Note: UAT *-ISSUES.md files (per-plan, created by /gsd:verify-work)
are unaffected - those are a separate, active system.

Closes #56

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-14 12:33:18 -06:00
Lex Christopherson
d30893a834 feat: pre-compute wave numbers at plan time
Wave numbers now computed during plan-phase and stored in PLAN.md
frontmatter. Execute-phase reads wave directly instead of deriving
from depends_on at runtime.

- Add assign_waves step to plan-phase workflow
- Add wave field to frontmatter (plan-format, phase-prompt template)
- Simplify execute-phase: remove analyze_dependencies and group_into_waves
- Replace with group_by_wave that just reads frontmatter integers

Co-Authored-By: Claude <noreply@anthropic.com>
2026-01-13 18:10:20 -06:00
Lex Christopherson
0dd4cedf5a perf(execute-phase): remove subagent-only context from orchestrator
Orchestrator loads only what it needs:
- execute-phase.md workflow (orchestration)
- subagent-task-prompt.md (template to spawn agents)

Subagents load execution context themselves via template:
- execute-plan.md, summary.md, checkpoints.md, tdd.md

~80% context reduction for orchestrator (~2000 lines saved)

Co-Authored-By: Claude <noreply@anthropic.com>
2026-01-13 17:26:18 -06:00
Lex Christopherson
46cf4b11de feat(execute-phase): add deviation rules, commit rules, and workflow references
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-13 17:01:10 -06:00
Lex Christopherson
e30e387c00 refactor(execute-phase): wave-based blocking execution
Replace background agents + polling with blocking Task calls.
Orchestrator stays lean (~15% context), subagents load full workflow.

- Archive old execute-phase command and workflow
- Add subagent-task-prompt.md template (orchestrator→subagent contract)
- Rewrite execute-phase with wave-based parallelization
- Dependency analysis groups plans into execution waves
- Parallel Task calls within waves, sequential across waves
- No TaskOutput polling - Task blocks until completion

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-13 16:06:17 -06:00
Lex Christopherson
b810d1ddca fix(execute-phase): add one-subagent-per-plan constraint
Context isolation is the primary goal, not parallelization.
Even sequential plans spawn separate subagents for fresh 200k context.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-12 14:34:37 -06:00
Lex Christopherson
31f565064d fix(10-02): sync execute-phase command with execute-plan content
- Added SlashCommand to allowed-tools
- Added references/tdd.md to execution_context
- Added execution_strategies, deviation_rules, commit_rules sections
- Updated success_criteria to include per-task commits
- Now fully aligned with execute-plan.md command structure
2026-01-12 13:24:49 -06:00
Lex Christopherson
8e8fba2793 fix(10-02): load execute-plan.md context in execute-phase command
- Command now loads both workflows (execution + orchestration)
- Agents commit tasks normally (removed DON'T COMMIT instruction)
- Orchestrator only commits metadata after all agents complete
- No duplication - single source of truth for execution logic
2026-01-12 13:23:16 -06:00
Lex Christopherson
18a1fd17c6 feat(10-03): create execute-phase command
- New slash command for parallel phase execution
- References execute-phase.md workflow
- Requires phase number as argument
- Uses parallel agent spawning for concurrent plans
2026-01-12 13:14:53 -06:00