From 86878b97218787dd2cba85de7a1f66339bd56bac Mon Sep 17 00:00:00 2001
From: Lex Christopherson <lex@glittercowboy.com>
Date: Mon, 29 Dec 2025 13:39:00 -0600
Subject: [PATCH] fix: restore deterministic bash commands, remove redundant
 decision_gate
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Remove decision_gate step (redundant after confirm_breakdown approval)
- Restore explicit bash commands in load_codebase_context, identify_phase,
  read_project_history, and gather_phase_context for consistent behavior

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---
 get-shit-done/workflows/plan-phase.md | 54 ++++++++++++++++++++-------
 1 file changed, 41 insertions(+), 13 deletions(-)

diff --git a/get-shit-done/workflows/plan-phase.md b/get-shit-done/workflows/plan-phase.md
index 4b40c0ef2..9ef016d12 100644
--- a/get-shit-done/workflows/plan-phase.md
+++ b/get-shit-done/workflows/plan-phase.md
@@ -46,7 +46,13 @@ If STATE.md missing but .planning/ exists, offer to reconstruct or continue with
 </step>
 
 <step name="load_codebase_context">
-Check for `.planning/codebase/*.md` and load relevant documents based on phase type:
+Check for codebase map:
+
+```bash
+ls .planning/codebase/*.md 2>/dev/null
+```
+
+**If .planning/codebase/ exists:** Load relevant documents based on phase type:
 
 | Phase Keywords | Load These |
 |----------------|------------|
@@ -63,7 +69,14 @@ Track extracted constraints for PLAN.md context section.
 </step>
 
 <step name="identify_phase">
-Check roadmap and existing phases. If multiple phases available, ask which one to plan.
+Check roadmap and existing phases:
+
+```bash
+cat .planning/ROADMAP.md
+ls .planning/phases/
+```
+
+If multiple phases available, ask which one to plan. If obvious (first incomplete phase), proceed.
 
 **Phase number parsing:** Regex `^(\d+)(?:\.(\d+))?$` - Group 1: integer, Group 2: decimal (optional)
 
@@ -112,9 +125,21 @@ For niche domains (3D, games, audio, shaders, ML), suggest `/gsd:research-phase`
 <step name="read_project_history">
 **From STATE.md:** Decisions → constrain approach. Deferred issues → candidates. Blockers → may need to address.
 
-**From prior summaries:** Scan `.planning/phases/*/*-SUMMARY.md` for decisions constraining this phase, issues flagged for "later", warnings in "Next Phase Readiness", patterns to maintain.
+**From prior summaries:**
 
-**From ISSUES.md:** Assess each open issue - relevant to this phase? Waiting long enough? Natural to address now? Blocking something?
+```bash
+ls .planning/phases/*/*-SUMMARY.md 2>/dev/null | sort
+```
+
+Scan for decisions constraining this phase, issues flagged for "later", warnings in "Next Phase Readiness", patterns to maintain.
+
+**From ISSUES.md:**
+
+```bash
+cat .planning/ISSUES.md 2>/dev/null
+```
+
+Assess each open issue - relevant to this phase? Waiting long enough? Natural to address now? Blocking something?
 
 **Answer before proceeding:**
 - Q1: What decisions from previous phases constrain this phase?
@@ -134,6 +159,18 @@ Understand:
 - Any DISCOVERY.md (from mandatory discovery)
 - Any {phase}-CONTEXT.md (from /gsd:discuss-phase)
 
+```bash
+# If mid-project, understand current state
+ls -la src/ 2>/dev/null
+cat package.json 2>/dev/null | head -20
+
+# Check for ecosystem research (from /gsd:research-phase)
+cat .planning/phases/XX-name/${PHASE}-RESEARCH.md 2>/dev/null
+
+# Check for phase context (from /gsd:discuss-phase)
+cat .planning/phases/XX-name/${PHASE}-CONTEXT.md 2>/dev/null
+```
+
 **If RESEARCH.md exists:** Use standard_stack (these libraries), architecture_patterns (follow in task structure), dont_hand_roll (NEVER custom solutions for listed problems), common_pitfalls (inform verification), code_examples (reference in actions).
 
 **If CONTEXT.md exists:** Honor vision, prioritize essential, respect boundaries, incorporate specifics.
@@ -201,15 +238,6 @@ Wait for confirmation. If "adjust": revise. If "start over": return to gather_ph
 </if>
 </step>
 
-<step name="decision_gate">
-<if mode="yolo">Auto-approve and proceed.</if>
-<if mode="interactive">
-Ask: "Ready to create the phase prompt, or ask more questions?"
-Options: Create phase prompt / Ask more questions / Let me add context
-Loop until "Create phase prompt" selected.
-</if>
-</step>
-
 <step name="write_phase_prompt">
 Use template from `~/.claude/get-shit-done/templates/phase-prompt.md`.