mirror of
https://github.com/obra/superpowers.git
synced 2026-07-10 03:59:04 +08:00
refactor(skills): fold Integration skill lists into points of use
The list-style Integration sections in subagent-driven-development and executing-plans duplicated references that already exist where the flow uses them (process digraph, When to Use, prompt templates, Step 3), so they added maintenance cost without carrying behavior. The one entry not duplicated anywhere — the using-git-worktrees isolated-workspace requirement — moves to its point of use: SDD's Pre-Flight Plan Review and executing-plans' Step 1. Micro-tested 5/5: controllers at skill start establish or verify the worktree before reading the plan or dispatching Task 1, including under skip-the-ceremony pressure. The prose Integration sections in requesting-code-review and other skills are unchanged — they carry placement content, not an index.
This commit is contained in:
@@ -16,10 +16,11 @@ Load plan, review critically, execute all tasks, report when complete.
|
|||||||
## The Process
|
## The Process
|
||||||
|
|
||||||
### Step 1: Load and Review Plan
|
### Step 1: Load and Review Plan
|
||||||
1. Read plan file
|
1. Ensure an isolated workspace: use superpowers:using-git-worktrees to create one or verify the existing one
|
||||||
2. Review critically - identify any questions or concerns about the plan
|
2. Read plan file
|
||||||
3. If concerns: Raise them with your human partner before starting
|
3. Review critically - identify any questions or concerns about the plan
|
||||||
4. If no concerns: Create todos for the plan items and proceed
|
4. If concerns: Raise them with your human partner before starting
|
||||||
|
5. If no concerns: Create todos for the plan items and proceed
|
||||||
|
|
||||||
### Step 2: Execute Tasks
|
### Step 2: Execute Tasks
|
||||||
|
|
||||||
@@ -61,10 +62,3 @@ After all tasks complete and verified:
|
|||||||
- Reference skills when plan says to
|
- Reference skills when plan says to
|
||||||
- Stop when blocked, don't guess
|
- Stop when blocked, don't guess
|
||||||
- Never start implementation on main/master branch without explicit user consent
|
- Never start implementation on main/master branch without explicit user consent
|
||||||
|
|
||||||
## Integration
|
|
||||||
|
|
||||||
**Required workflow skills:**
|
|
||||||
- **superpowers:using-git-worktrees** - Ensures isolated workspace (creates one or verifies existing)
|
|
||||||
- **superpowers:writing-plans** - Creates the plan this skill executes
|
|
||||||
- **superpowers:finishing-a-development-branch** - Complete development after all tasks
|
|
||||||
|
|||||||
@@ -84,6 +84,9 @@ digraph process {
|
|||||||
|
|
||||||
## Pre-Flight Plan Review
|
## Pre-Flight Plan Review
|
||||||
|
|
||||||
|
Ensure the work happens in an isolated workspace: use
|
||||||
|
superpowers:using-git-worktrees to create one or verify the existing one.
|
||||||
|
|
||||||
Before dispatching Task 1, scan the plan once for conflicts:
|
Before dispatching Task 1, scan the plan once for conflicts:
|
||||||
|
|
||||||
- tasks that contradict each other or the plan's Global Constraints
|
- tasks that contradict each other or the plan's Global Constraints
|
||||||
@@ -402,17 +405,3 @@ Done!
|
|||||||
**If subagent fails task:**
|
**If subagent fails task:**
|
||||||
- Dispatch fix subagent with specific instructions
|
- Dispatch fix subagent with specific instructions
|
||||||
- Don't try to fix manually (context pollution)
|
- Don't try to fix manually (context pollution)
|
||||||
|
|
||||||
## Integration
|
|
||||||
|
|
||||||
**Required workflow skills:**
|
|
||||||
- **superpowers:using-git-worktrees** - Ensures isolated workspace (creates one or verifies existing)
|
|
||||||
- **superpowers:writing-plans** - Creates the plan this skill executes
|
|
||||||
- **superpowers:requesting-code-review** - Code review template for the final whole-branch review
|
|
||||||
- **superpowers:finishing-a-development-branch** - Complete development after all tasks
|
|
||||||
|
|
||||||
**Subagents should use:**
|
|
||||||
- **superpowers:test-driven-development** - Subagents follow TDD for each task
|
|
||||||
|
|
||||||
**Alternative workflow:**
|
|
||||||
- **superpowers:executing-plans** - Use for parallel session instead of same-session execution
|
|
||||||
|
|||||||
Reference in New Issue
Block a user