Skip to content

Commit 6331aec

Browse files
committed
Stronger wording to force skill invocation
1 parent e3bf110 commit 6331aec

5 files changed

Lines changed: 32 additions & 37 deletions

File tree

skills/build-unplanned-feature/SKILL.md

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,9 @@ Store the raw description for clarification.
2121

2222
### Step 2: Clarify Requirements
2323

24-
Invoke: `Skill(skill="groundwork:understanding-feature-requests")`
24+
**You MUST call the Skill tool now:** `Skill(skill="groundwork:understanding-feature-requests")`
25+
26+
Do NOT attempt to gather requirements yourself. The skill handles this.
2527

2628
Follow the skill to gather:
2729
- Problem being solved
@@ -84,7 +86,10 @@ Then use `AskUserQuestion` to ask:
8486

8587
### Step 5: Execute Implementation
8688

87-
Invoke: `Skill(skill="groundwork:implement-feature")`
89+
**CRITICAL INSTRUCTION: You MUST call the Skill tool now:**
90+
`Skill(skill="groundwork:implement-feature")`
91+
92+
Do NOT write implementation plans, do NOT create files, do NOT explore code, do NOT start coding. You are NOT permitted to implement the feature yourself. The implement-feature skill handles ALL implementation including worktree creation, TDD, validation, and merge. Call it NOW.
8893

8994
Session context provides:
9095
- **identifier**: FEATURE-<slug>
@@ -93,13 +98,6 @@ Session context provides:
9398
- **action-items**: [from requirements]
9499
- **acceptance-criteria**: [from clarification]
95100

96-
The `implement-feature` skill handles:
97-
1. Worktree creation via `use-git-worktree`
98-
2. TDD workflow for each requirement
99-
3. Multi-agent validation
100-
4. Worktree finalization (prompts user for merge decision)
101-
5. Completion reporting
102-
103101
---
104102

105103
## Reference

skills/execute-task/SKILL.md

Lines changed: 11 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -141,20 +141,17 @@ Then use `AskUserQuestion` to ask:
141141

142142
1. **Update status** - Change task to `**Status:** In Progress`
143143

144-
2. **Invoke implementation** - `Skill(skill="groundwork:implement-feature")`
145-
146-
Session context provides:
147-
- **identifier**: TASK-NNN
148-
- **title**: [Task Title]
149-
- **merge-mode**: `env`
150-
- **action-items**: [from task file]
151-
- **acceptance-criteria**: [from task file]
152-
153-
The `implement-feature` skill handles:
154-
- Worktree creation via `use-git-worktree`
155-
- TDD implementation
156-
- Multi-agent validation
157-
- Worktree finalization (merge based on `GROUNDWORK_AUTO_MERGE` env var)
144+
2. **CRITICAL INSTRUCTION: You MUST call the Skill tool now:**
145+
`Skill(skill="groundwork:implement-feature")`
146+
147+
Do NOT start implementing the task yourself. Do NOT create files, write code, or begin TDD directly. The implement-feature skill handles ALL of this. Call it NOW.
148+
149+
Session context provides:
150+
- **identifier**: TASK-NNN
151+
- **title**: [Task Title]
152+
- **merge-mode**: `env`
153+
- **action-items**: [from task file]
154+
- **acceptance-criteria**: [from task file]
158155

159156
### Step 8: Complete Task
160157

skills/implement-feature/SKILL.md

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,9 @@ Check session context for worktree parameters:
2626

2727
If no identifier create a new identifier yourself in the format FEATURE-slug. Otherwise, use the identifier provided.
2828

29-
Invoke: `Skill(skill="groundwork:use-git-worktree", args="<identifier>")`
29+
**You MUST call the Skill tool now:** `Skill(skill="groundwork:use-git-worktree", args="<identifier>")`
30+
31+
Do NOT create branches or worktrees with git commands directly. The skill handles setup and baseline validation.
3032

3133
Wait for worktree creation to complete and baseline tests to pass.
3234

@@ -50,7 +52,9 @@ Store the gathered context for use in subsequent steps.
5052

5153
### Step 3: Execute TDD
5254

53-
Invoke: `Skill(skill="groundwork:test-driven-development")`
55+
**You MUST call the Skill tool now:** `Skill(skill="groundwork:test-driven-development")`
56+
57+
Do NOT write implementation code without first loading the TDD skill. It enforces red-green-refactor discipline.
5458

5559
For each action item:
5660
1. Write failing test
@@ -101,7 +105,9 @@ Output verification results:
101105

102106
### Step 6: Multi-Agent Verification
103107

104-
This is mandatory. Invoke: `Skill(skill="groundwork:validation-loop")`
108+
**You MUST call the Skill tool now:** `Skill(skill="groundwork:validation-loop")`
109+
110+
Do NOT declare implementation complete or skip validation. This step is non-negotiable.
105111

106112
This runs verification agents in parallel with autonomous fix-and-retry.
107113

skills/next-task/SKILL.md

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -60,10 +60,7 @@ Parse the tasks to find the next task to work on:
6060

6161
### Step 4: Delegate to Execute Task
6262

63-
Once a task is identified, invoke the `groundwork:execute-task` skill with the task identifier (e.g., `TASK-004`).
63+
Once a task is identified, **you MUST call the Skill tool:**
64+
`Skill(skill="groundwork:execute-task", args="TASK-NNN")`
6465

65-
The execute-task skill will:
66-
1. Load full project context (PRD, architecture, tasks)
67-
2. Present the task summary
68-
3. Execute the task
69-
4. Complete the task and offer to continue
66+
Do NOT load project context, present summaries, or begin task execution yourself. The execute-task skill handles the complete workflow. Call it NOW with the identified task identifier.

skills/tasks/SKILL.md

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -285,12 +285,9 @@ Iterate until user approves, then write to `specs/tasks.md`.
285285

286286
After user approves the task list and before writing final version:
287287

288-
Invoke: `Skill(skill="groundwork:task-validation-loop")`
288+
**You MUST call the Skill tool now:** `Skill(skill="groundwork:task-validation-loop")`
289289

290-
This runs verification agents to ensure:
291-
- All PRD requirements have corresponding tasks
292-
- Tasks are assigned to correct architecture components
293-
- UI tasks include design system and accessibility criteria
290+
Do NOT skip validation or declare the task list complete without running this. It ensures PRD coverage, architecture alignment, and design system compliance.
294291

295292
**Wait for validation loop to pass before proceeding.**
296293

0 commit comments

Comments
 (0)