From c21c5e978eaa850e443fb0173a8f91f9fe8dce80 Mon Sep 17 00:00:00 2001 From: Lex Christopherson Date: Wed, 17 Dec 2025 08:35:14 -0600 Subject: [PATCH] fix: remove shell context to fix permission errors for non-DSP users MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Shell commands in sections using !` backticks trigger permission checks for users not in dangerously-skip-permissions mode. Replaced with @-file references where needed; process logic handles missing files via allowed tools (Read, Glob, Grep). Affected commands: new-project, progress, create-roadmap, research-project, pause-work, consider-issues 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 --- commands/gsd/consider-issues.md | 7 +++---- commands/gsd/create-roadmap.md | 5 ----- commands/gsd/new-project.md | 3 --- commands/gsd/pause-work.md | 1 - commands/gsd/progress.md | 9 --------- commands/gsd/research-project.md | 4 ---- 6 files changed, 3 insertions(+), 26 deletions(-) diff --git a/commands/gsd/consider-issues.md b/commands/gsd/consider-issues.md index eaca9bcb4..b0e7341b5 100644 --- a/commands/gsd/consider-issues.md +++ b/commands/gsd/consider-issues.md @@ -17,10 +17,9 @@ This prevents issue pile-up by providing a triage mechanism with codebase awaren -Issues file: !`cat .planning/ISSUES.md 2>/dev/null || echo "NO_ISSUES_FILE"` -Project state: !`cat .planning/STATE.md 2>/dev/null | head -50` -Current phase: !`grep -E "^Phase:" .planning/STATE.md 2>/dev/null || echo "UNKNOWN"` -Roadmap phases: !`grep -E "^### Phase [0-9]" .planning/ROADMAP.md 2>/dev/null || echo "NO_ROADMAP"` +@.planning/ISSUES.md +@.planning/STATE.md +@.planning/ROADMAP.md diff --git a/commands/gsd/create-roadmap.md b/commands/gsd/create-roadmap.md index f8bf590c4..5badc7ea8 100644 --- a/commands/gsd/create-roadmap.md +++ b/commands/gsd/create-roadmap.md @@ -24,11 +24,6 @@ Roadmaps define the phase breakdown - what work happens in what order. This comm @.planning/PROJECT.md - -**Check for research:** -!`ls .planning/research/*.md 2>/dev/null || echo "NO_RESEARCH"` - -**Load config:** @.planning/config.json diff --git a/commands/gsd/new-project.md b/commands/gsd/new-project.md index 12b5d441e..3650a7826 100644 --- a/commands/gsd/new-project.md +++ b/commands/gsd/new-project.md @@ -22,9 +22,6 @@ Creates `.planning/` with PROJECT.md and config.json. @~/.claude/get-shit-done/templates/config.json - -!`[ -d .planning ] && echo "PLANNING_EXISTS" || echo "NO_PLANNING"` - diff --git a/commands/gsd/pause-work.md b/commands/gsd/pause-work.md index dfba862ca..d8441905d 100644 --- a/commands/gsd/pause-work.md +++ b/commands/gsd/pause-work.md @@ -14,7 +14,6 @@ Enables seamless resumption in fresh session with full context restoration. @.planning/STATE.md -Current phase: !`ls -lt .planning/phases/*/*.md 2>/dev/null | head -1` diff --git a/commands/gsd/progress.md b/commands/gsd/progress.md index 3cae0e77e..055630c82 100644 --- a/commands/gsd/progress.md +++ b/commands/gsd/progress.md @@ -14,15 +14,6 @@ Check project progress, summarize recent work and what's ahead, then intelligent Provides situational awareness before continuing work. - -Planning structure: !`ls -la .planning/ 2>/dev/null || echo "NO_PLANNING_STRUCTURE"` -Phases: !`ls .planning/phases/ 2>/dev/null || echo "NO_PHASES"` -State exists: !`[ -f .planning/STATE.md ] && echo "EXISTS" || echo "MISSING"` -Roadmap exists: !`[ -f .planning/ROADMAP.md ] && echo "EXISTS" || echo "MISSING"` -Recent summaries: !`find .planning/phases -name "*-SUMMARY.md" -type f 2>/dev/null | sort | tail -3` -All plans: !`find .planning/phases -name "*-PLAN.md" -type f 2>/dev/null | sort` -All summaries: !`find .planning/phases -name "*-SUMMARY.md" -type f 2>/dev/null | sort` - diff --git a/commands/gsd/research-project.md b/commands/gsd/research-project.md index c04b78961..0236d2ad4 100644 --- a/commands/gsd/research-project.md +++ b/commands/gsd/research-project.md @@ -31,11 +31,7 @@ Each subagent writes directly to `.planning/research/` preserving main context. -**Load project vision:** @.planning/PROJECT.md - -**Check for existing research:** -!`ls .planning/research/ 2>/dev/null || echo "NO_RESEARCH_DIR"`