claude-termux 1.0.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CLAUDE.md +60 -0
- package/GEMINI.md +20 -0
- package/README.md +135 -0
- package/TERMUX.md +204 -0
- package/agents/accessibility-reviewer.md +96 -0
- package/agents/ai-prompt-optimizer.md +94 -0
- package/agents/api-tester.md +102 -0
- package/agents/code-generator.md +94 -0
- package/agents/code-reviewer.md +47 -0
- package/agents/component-generator.md +102 -0
- package/agents/doc-generator.md +91 -0
- package/agents/migration-generator.md +94 -0
- package/agents/performance-analyzer.md +90 -0
- package/agents/proactive-mode.md +91 -0
- package/agents/readme-generator.md +101 -0
- package/agents/security-auditor.md +86 -0
- package/agents/terraform-generator.md +94 -0
- package/agents/test-generator.md +76 -0
- package/commands/brainstorm.md +5 -0
- package/commands/execute-plan.md +5 -0
- package/commands/write-plan.md +5 -0
- package/hooks/auto-context.json +31 -0
- package/hooks/hooks.json +15 -0
- package/hooks/run-hook.cmd +19 -0
- package/hooks/session-start.sh +52 -0
- package/hooks/smart-session.sh +96 -0
- package/install.sh +210 -0
- package/lib/skills-core.js +208 -0
- package/mcp.json +34 -0
- package/package.json +49 -0
- package/plugins/README.md +47 -0
- package/plugins/installed_plugins.json +5 -0
- package/plugins/known_marketplaces.json +10 -0
- package/plugins/marketplace-info/marketplace.json +517 -0
- package/postinstall.js +238 -0
- package/settings.json +27 -0
- package/settings.local.json +25 -0
- package/skills/api-development/SKILL.md +11 -0
- package/skills/api-development/openapi/api-documentation.yaml +108 -0
- package/skills/brainstorming/SKILL.md +54 -0
- package/skills/code-quality/SKILL.md +196 -0
- package/skills/condition-based-waiting/SKILL.md +120 -0
- package/skills/condition-based-waiting/example.ts +158 -0
- package/skills/database-development/SKILL.md +11 -0
- package/skills/database-development/migrations/migration.template.sql +49 -0
- package/skills/defense-in-depth/SKILL.md +127 -0
- package/skills/deployment/SKILL.md +11 -0
- package/skills/deployment/ci-cd/github-actions.yml +95 -0
- package/skills/deployment/docker/Dockerfile.template +39 -0
- package/skills/dispatching-parallel-agents/SKILL.md +180 -0
- package/skills/documentation-generation/SKILL.md +8 -0
- package/skills/documentation-generation/templates/README.template.md +60 -0
- package/skills/error-handling/SKILL.md +267 -0
- package/skills/executing-plans/SKILL.md +76 -0
- package/skills/finishing-a-development-branch/SKILL.md +200 -0
- package/skills/frontend-design/frontend-design/SKILL.md +42 -0
- package/skills/integration-testing/SKILL.md +13 -0
- package/skills/integration-testing/examples/contract-test.py +317 -0
- package/skills/integration-testing/examples/e2e-test.js +147 -0
- package/skills/integration-testing/examples/test-isolation.md +94 -0
- package/skills/logging-monitoring/SKILL.md +66 -0
- package/skills/mobile-development/SKILL.md +11 -0
- package/skills/mobile-development/responsive/responsive.css +80 -0
- package/skills/performance-optimization/SKILL.md +9 -0
- package/skills/performance-optimization/profiling/profile.template.js +21 -0
- package/skills/receiving-code-review/SKILL.md +209 -0
- package/skills/refactoring/SKILL.md +11 -0
- package/skills/refactoring/code-smells/common-smells.md +115 -0
- package/skills/requesting-code-review/SKILL.md +105 -0
- package/skills/requesting-code-review/code-reviewer.md +146 -0
- package/skills/root-cause-tracing/SKILL.md +174 -0
- package/skills/root-cause-tracing/find-polluter.sh +63 -0
- package/skills/security-review/SKILL.md +11 -0
- package/skills/security-review/checklists/owasp-checklist.md +31 -0
- package/skills/sharing-skills/SKILL.md +194 -0
- package/skills/subagent-driven-development/SKILL.md +240 -0
- package/skills/subagent-driven-development/code-quality-reviewer-prompt.md +20 -0
- package/skills/subagent-driven-development/implementer-prompt.md +78 -0
- package/skills/subagent-driven-development/spec-reviewer-prompt.md +61 -0
- package/skills/systematic-debugging/CREATION-LOG.md +119 -0
- package/skills/systematic-debugging/SKILL.md +295 -0
- package/skills/systematic-debugging/test-academic.md +14 -0
- package/skills/systematic-debugging/test-pressure-1.md +58 -0
- package/skills/systematic-debugging/test-pressure-2.md +68 -0
- package/skills/systematic-debugging/test-pressure-3.md +69 -0
- package/skills/test-driven-development/SKILL.md +364 -0
- package/skills/testing-anti-patterns/SKILL.md +302 -0
- package/skills/testing-skills-with-subagents/SKILL.md +387 -0
- package/skills/testing-skills-with-subagents/examples/CLAUDE_MD_TESTING.md +189 -0
- package/skills/ui-ux-review/SKILL.md +13 -0
- package/skills/ui-ux-review/checklists/ux-heuristics.md +61 -0
- package/skills/using-git-worktrees/SKILL.md +213 -0
- package/skills/using-superpowers/SKILL.md +101 -0
- package/skills/verification-before-completion/SKILL.md +139 -0
- package/skills/writing-plans/SKILL.md +116 -0
- package/skills/writing-skills/SKILL.md +622 -0
- package/skills/writing-skills/anthropic-best-practices.md +1150 -0
- package/skills/writing-skills/graphviz-conventions.dot +172 -0
- package/skills/writing-skills/persuasion-principles.md +187 -0
|
@@ -0,0 +1,194 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: sharing-skills
|
|
3
|
+
description: Use when you've developed a broadly useful skill and want to contribute it upstream via pull request - guides process of branching, committing, pushing, and creating PR to contribute skills back to upstream repository
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Sharing Skills
|
|
7
|
+
|
|
8
|
+
## Overview
|
|
9
|
+
|
|
10
|
+
Contribute skills from your local branch back to the upstream repository.
|
|
11
|
+
|
|
12
|
+
**Workflow:** Branch → Edit/Create skill → Commit → Push → PR
|
|
13
|
+
|
|
14
|
+
## When to Share
|
|
15
|
+
|
|
16
|
+
**Share when:**
|
|
17
|
+
- Skill applies broadly (not project-specific)
|
|
18
|
+
- Pattern/technique others would benefit from
|
|
19
|
+
- Well-tested and documented
|
|
20
|
+
- Follows writing-skills guidelines
|
|
21
|
+
|
|
22
|
+
**Keep personal when:**
|
|
23
|
+
- Project-specific or organization-specific
|
|
24
|
+
- Experimental or unstable
|
|
25
|
+
- Contains sensitive information
|
|
26
|
+
- Too narrow/niche for general use
|
|
27
|
+
|
|
28
|
+
## Prerequisites
|
|
29
|
+
|
|
30
|
+
- `gh` CLI installed and authenticated
|
|
31
|
+
- Working directory is `~/.config/superpowers/skills/` (your local clone)
|
|
32
|
+
- **REQUIRED:** Skill has been tested using writing-skills TDD process
|
|
33
|
+
|
|
34
|
+
## Sharing Workflow
|
|
35
|
+
|
|
36
|
+
### 1. Ensure You're on Main and Synced
|
|
37
|
+
|
|
38
|
+
```bash
|
|
39
|
+
cd ~/.config/superpowers/skills/
|
|
40
|
+
git checkout main
|
|
41
|
+
git pull upstream main
|
|
42
|
+
git push origin main # Push to your fork
|
|
43
|
+
```
|
|
44
|
+
|
|
45
|
+
### 2. Create Feature Branch
|
|
46
|
+
|
|
47
|
+
```bash
|
|
48
|
+
# Branch name: add-skillname-skill
|
|
49
|
+
skill_name="your-skill-name"
|
|
50
|
+
git checkout -b "add-${skill_name}-skill"
|
|
51
|
+
```
|
|
52
|
+
|
|
53
|
+
### 3. Create or Edit Skill
|
|
54
|
+
|
|
55
|
+
```bash
|
|
56
|
+
# Work on your skill in skills/
|
|
57
|
+
# Create new skill or edit existing one
|
|
58
|
+
# Skill should be in skills/category/skill-name/SKILL.md
|
|
59
|
+
```
|
|
60
|
+
|
|
61
|
+
### 4. Commit Changes
|
|
62
|
+
|
|
63
|
+
```bash
|
|
64
|
+
# Add and commit
|
|
65
|
+
git add skills/your-skill-name/
|
|
66
|
+
git commit -m "Add ${skill_name} skill
|
|
67
|
+
|
|
68
|
+
$(cat <<'EOF'
|
|
69
|
+
Brief description of what this skill does and why it's useful.
|
|
70
|
+
|
|
71
|
+
Tested with: [describe testing approach]
|
|
72
|
+
EOF
|
|
73
|
+
)"
|
|
74
|
+
```
|
|
75
|
+
|
|
76
|
+
### 5. Push to Your Fork
|
|
77
|
+
|
|
78
|
+
```bash
|
|
79
|
+
git push -u origin "add-${skill_name}-skill"
|
|
80
|
+
```
|
|
81
|
+
|
|
82
|
+
### 6. Create Pull Request
|
|
83
|
+
|
|
84
|
+
```bash
|
|
85
|
+
# Create PR to upstream using gh CLI
|
|
86
|
+
gh pr create \
|
|
87
|
+
--repo upstream-org/upstream-repo \
|
|
88
|
+
--title "Add ${skill_name} skill" \
|
|
89
|
+
--body "$(cat <<'EOF'
|
|
90
|
+
## Summary
|
|
91
|
+
Brief description of the skill and what problem it solves.
|
|
92
|
+
|
|
93
|
+
## Testing
|
|
94
|
+
Describe how you tested this skill (pressure scenarios, baseline tests, etc.).
|
|
95
|
+
|
|
96
|
+
## Context
|
|
97
|
+
Any additional context about why this skill is needed and how it should be used.
|
|
98
|
+
EOF
|
|
99
|
+
)"
|
|
100
|
+
```
|
|
101
|
+
|
|
102
|
+
## Complete Example
|
|
103
|
+
|
|
104
|
+
Here's a complete example of sharing a skill called "async-patterns":
|
|
105
|
+
|
|
106
|
+
```bash
|
|
107
|
+
# 1. Sync with upstream
|
|
108
|
+
cd ~/.config/superpowers/skills/
|
|
109
|
+
git checkout main
|
|
110
|
+
git pull upstream main
|
|
111
|
+
git push origin main
|
|
112
|
+
|
|
113
|
+
# 2. Create branch
|
|
114
|
+
git checkout -b "add-async-patterns-skill"
|
|
115
|
+
|
|
116
|
+
# 3. Create/edit the skill
|
|
117
|
+
# (Work on skills/async-patterns/SKILL.md)
|
|
118
|
+
|
|
119
|
+
# 4. Commit
|
|
120
|
+
git add skills/async-patterns/
|
|
121
|
+
git commit -m "Add async-patterns skill
|
|
122
|
+
|
|
123
|
+
Patterns for handling asynchronous operations in tests and application code.
|
|
124
|
+
|
|
125
|
+
Tested with: Multiple pressure scenarios testing agent compliance."
|
|
126
|
+
|
|
127
|
+
# 5. Push
|
|
128
|
+
git push -u origin "add-async-patterns-skill"
|
|
129
|
+
|
|
130
|
+
# 6. Create PR
|
|
131
|
+
gh pr create \
|
|
132
|
+
--repo upstream-org/upstream-repo \
|
|
133
|
+
--title "Add async-patterns skill" \
|
|
134
|
+
--body "## Summary
|
|
135
|
+
Patterns for handling asynchronous operations correctly in tests and application code.
|
|
136
|
+
|
|
137
|
+
## Testing
|
|
138
|
+
Tested with multiple application scenarios. Agents successfully apply patterns to new code.
|
|
139
|
+
|
|
140
|
+
## Context
|
|
141
|
+
Addresses common async pitfalls like race conditions, improper error handling, and timing issues."
|
|
142
|
+
```
|
|
143
|
+
|
|
144
|
+
## After PR is Merged
|
|
145
|
+
|
|
146
|
+
Once your PR is merged:
|
|
147
|
+
|
|
148
|
+
1. Sync your local main branch:
|
|
149
|
+
```bash
|
|
150
|
+
cd ~/.config/superpowers/skills/
|
|
151
|
+
git checkout main
|
|
152
|
+
git pull upstream main
|
|
153
|
+
git push origin main
|
|
154
|
+
```
|
|
155
|
+
|
|
156
|
+
2. Delete the feature branch:
|
|
157
|
+
```bash
|
|
158
|
+
git branch -d "add-${skill_name}-skill"
|
|
159
|
+
git push origin --delete "add-${skill_name}-skill"
|
|
160
|
+
```
|
|
161
|
+
|
|
162
|
+
## Troubleshooting
|
|
163
|
+
|
|
164
|
+
**"gh: command not found"**
|
|
165
|
+
- Install GitHub CLI: https://cli.github.com/
|
|
166
|
+
- Authenticate: `gh auth login`
|
|
167
|
+
|
|
168
|
+
**"Permission denied (publickey)"**
|
|
169
|
+
- Check SSH keys: `gh auth status`
|
|
170
|
+
- Set up SSH: https://docs.github.com/en/authentication
|
|
171
|
+
|
|
172
|
+
**"Skill already exists"**
|
|
173
|
+
- You're creating a modified version
|
|
174
|
+
- Consider different skill name or coordinate with the skill's maintainer
|
|
175
|
+
|
|
176
|
+
**PR merge conflicts**
|
|
177
|
+
- Rebase on latest upstream: `git fetch upstream && git rebase upstream/main`
|
|
178
|
+
- Resolve conflicts
|
|
179
|
+
- Force push: `git push -f origin your-branch`
|
|
180
|
+
|
|
181
|
+
## Multi-Skill Contributions
|
|
182
|
+
|
|
183
|
+
**Do NOT batch multiple skills in one PR.**
|
|
184
|
+
|
|
185
|
+
Each skill should:
|
|
186
|
+
- Have its own feature branch
|
|
187
|
+
- Have its own PR
|
|
188
|
+
- Be independently reviewable
|
|
189
|
+
|
|
190
|
+
**Why?** Individual skills can be reviewed, iterated, and merged independently.
|
|
191
|
+
|
|
192
|
+
## Related Skills
|
|
193
|
+
|
|
194
|
+
- **writing-skills** - REQUIRED: How to create well-tested skills before sharing
|
|
@@ -0,0 +1,240 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: subagent-driven-development
|
|
3
|
+
description: Use when executing implementation plans with independent tasks in the current session
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Subagent-Driven Development
|
|
7
|
+
|
|
8
|
+
Execute plan by dispatching fresh subagent per task, with two-stage review after each: spec compliance review first, then code quality review.
|
|
9
|
+
|
|
10
|
+
**Core principle:** Fresh subagent per task + two-stage review (spec then quality) = high quality, fast iteration
|
|
11
|
+
|
|
12
|
+
## When to Use
|
|
13
|
+
|
|
14
|
+
```dot
|
|
15
|
+
digraph when_to_use {
|
|
16
|
+
"Have implementation plan?" [shape=diamond];
|
|
17
|
+
"Tasks mostly independent?" [shape=diamond];
|
|
18
|
+
"Stay in this session?" [shape=diamond];
|
|
19
|
+
"subagent-driven-development" [shape=box];
|
|
20
|
+
"executing-plans" [shape=box];
|
|
21
|
+
"Manual execution or brainstorm first" [shape=box];
|
|
22
|
+
|
|
23
|
+
"Have implementation plan?" -> "Tasks mostly independent?" [label="yes"];
|
|
24
|
+
"Have implementation plan?" -> "Manual execution or brainstorm first" [label="no"];
|
|
25
|
+
"Tasks mostly independent?" -> "Stay in this session?" [label="yes"];
|
|
26
|
+
"Tasks mostly independent?" -> "Manual execution or brainstorm first" [label="no - tightly coupled"];
|
|
27
|
+
"Stay in this session?" -> "subagent-driven-development" [label="yes"];
|
|
28
|
+
"Stay in this session?" -> "executing-plans" [label="no - parallel session"];
|
|
29
|
+
}
|
|
30
|
+
```
|
|
31
|
+
|
|
32
|
+
**vs. Executing Plans (parallel session):**
|
|
33
|
+
- Same session (no context switch)
|
|
34
|
+
- Fresh subagent per task (no context pollution)
|
|
35
|
+
- Two-stage review after each task: spec compliance first, then code quality
|
|
36
|
+
- Faster iteration (no human-in-loop between tasks)
|
|
37
|
+
|
|
38
|
+
## The Process
|
|
39
|
+
|
|
40
|
+
```dot
|
|
41
|
+
digraph process {
|
|
42
|
+
rankdir=TB;
|
|
43
|
+
|
|
44
|
+
subgraph cluster_per_task {
|
|
45
|
+
label="Per Task";
|
|
46
|
+
"Dispatch implementer subagent (./implementer-prompt.md)" [shape=box];
|
|
47
|
+
"Implementer subagent asks questions?" [shape=diamond];
|
|
48
|
+
"Answer questions, provide context" [shape=box];
|
|
49
|
+
"Implementer subagent implements, tests, commits, self-reviews" [shape=box];
|
|
50
|
+
"Dispatch spec reviewer subagent (./spec-reviewer-prompt.md)" [shape=box];
|
|
51
|
+
"Spec reviewer subagent confirms code matches spec?" [shape=diamond];
|
|
52
|
+
"Implementer subagent fixes spec gaps" [shape=box];
|
|
53
|
+
"Dispatch code quality reviewer subagent (./code-quality-reviewer-prompt.md)" [shape=box];
|
|
54
|
+
"Code quality reviewer subagent approves?" [shape=diamond];
|
|
55
|
+
"Implementer subagent fixes quality issues" [shape=box];
|
|
56
|
+
"Mark task complete in TodoWrite" [shape=box];
|
|
57
|
+
}
|
|
58
|
+
|
|
59
|
+
"Read plan, extract all tasks with full text, note context, create TodoWrite" [shape=box];
|
|
60
|
+
"More tasks remain?" [shape=diamond];
|
|
61
|
+
"Dispatch final code reviewer subagent for entire implementation" [shape=box];
|
|
62
|
+
"Use superpowers:finishing-a-development-branch" [shape=box style=filled fillcolor=lightgreen];
|
|
63
|
+
|
|
64
|
+
"Read plan, extract all tasks with full text, note context, create TodoWrite" -> "Dispatch implementer subagent (./implementer-prompt.md)";
|
|
65
|
+
"Dispatch implementer subagent (./implementer-prompt.md)" -> "Implementer subagent asks questions?";
|
|
66
|
+
"Implementer subagent asks questions?" -> "Answer questions, provide context" [label="yes"];
|
|
67
|
+
"Answer questions, provide context" -> "Dispatch implementer subagent (./implementer-prompt.md)";
|
|
68
|
+
"Implementer subagent asks questions?" -> "Implementer subagent implements, tests, commits, self-reviews" [label="no"];
|
|
69
|
+
"Implementer subagent implements, tests, commits, self-reviews" -> "Dispatch spec reviewer subagent (./spec-reviewer-prompt.md)";
|
|
70
|
+
"Dispatch spec reviewer subagent (./spec-reviewer-prompt.md)" -> "Spec reviewer subagent confirms code matches spec?";
|
|
71
|
+
"Spec reviewer subagent confirms code matches spec?" -> "Implementer subagent fixes spec gaps" [label="no"];
|
|
72
|
+
"Implementer subagent fixes spec gaps" -> "Dispatch spec reviewer subagent (./spec-reviewer-prompt.md)" [label="re-review"];
|
|
73
|
+
"Spec reviewer subagent confirms code matches spec?" -> "Dispatch code quality reviewer subagent (./code-quality-reviewer-prompt.md)" [label="yes"];
|
|
74
|
+
"Dispatch code quality reviewer subagent (./code-quality-reviewer-prompt.md)" -> "Code quality reviewer subagent approves?";
|
|
75
|
+
"Code quality reviewer subagent approves?" -> "Implementer subagent fixes quality issues" [label="no"];
|
|
76
|
+
"Implementer subagent fixes quality issues" -> "Dispatch code quality reviewer subagent (./code-quality-reviewer-prompt.md)" [label="re-review"];
|
|
77
|
+
"Code quality reviewer subagent approves?" -> "Mark task complete in TodoWrite" [label="yes"];
|
|
78
|
+
"Mark task complete in TodoWrite" -> "More tasks remain?";
|
|
79
|
+
"More tasks remain?" -> "Dispatch implementer subagent (./implementer-prompt.md)" [label="yes"];
|
|
80
|
+
"More tasks remain?" -> "Dispatch final code reviewer subagent for entire implementation" [label="no"];
|
|
81
|
+
"Dispatch final code reviewer subagent for entire implementation" -> "Use superpowers:finishing-a-development-branch";
|
|
82
|
+
}
|
|
83
|
+
```
|
|
84
|
+
|
|
85
|
+
## Prompt Templates
|
|
86
|
+
|
|
87
|
+
- `./implementer-prompt.md` - Dispatch implementer subagent
|
|
88
|
+
- `./spec-reviewer-prompt.md` - Dispatch spec compliance reviewer subagent
|
|
89
|
+
- `./code-quality-reviewer-prompt.md` - Dispatch code quality reviewer subagent
|
|
90
|
+
|
|
91
|
+
## Example Workflow
|
|
92
|
+
|
|
93
|
+
```
|
|
94
|
+
You: I'm using Subagent-Driven Development to execute this plan.
|
|
95
|
+
|
|
96
|
+
[Read plan file once: docs/plans/feature-plan.md]
|
|
97
|
+
[Extract all 5 tasks with full text and context]
|
|
98
|
+
[Create TodoWrite with all tasks]
|
|
99
|
+
|
|
100
|
+
Task 1: Hook installation script
|
|
101
|
+
|
|
102
|
+
[Get Task 1 text and context (already extracted)]
|
|
103
|
+
[Dispatch implementation subagent with full task text + context]
|
|
104
|
+
|
|
105
|
+
Implementer: "Before I begin - should the hook be installed at user or system level?"
|
|
106
|
+
|
|
107
|
+
You: "User level (~/.config/superpowers/hooks/)"
|
|
108
|
+
|
|
109
|
+
Implementer: "Got it. Implementing now..."
|
|
110
|
+
[Later] Implementer:
|
|
111
|
+
- Implemented install-hook command
|
|
112
|
+
- Added tests, 5/5 passing
|
|
113
|
+
- Self-review: Found I missed --force flag, added it
|
|
114
|
+
- Committed
|
|
115
|
+
|
|
116
|
+
[Dispatch spec compliance reviewer]
|
|
117
|
+
Spec reviewer: ✅ Spec compliant - all requirements met, nothing extra
|
|
118
|
+
|
|
119
|
+
[Get git SHAs, dispatch code quality reviewer]
|
|
120
|
+
Code reviewer: Strengths: Good test coverage, clean. Issues: None. Approved.
|
|
121
|
+
|
|
122
|
+
[Mark Task 1 complete]
|
|
123
|
+
|
|
124
|
+
Task 2: Recovery modes
|
|
125
|
+
|
|
126
|
+
[Get Task 2 text and context (already extracted)]
|
|
127
|
+
[Dispatch implementation subagent with full task text + context]
|
|
128
|
+
|
|
129
|
+
Implementer: [No questions, proceeds]
|
|
130
|
+
Implementer:
|
|
131
|
+
- Added verify/repair modes
|
|
132
|
+
- 8/8 tests passing
|
|
133
|
+
- Self-review: All good
|
|
134
|
+
- Committed
|
|
135
|
+
|
|
136
|
+
[Dispatch spec compliance reviewer]
|
|
137
|
+
Spec reviewer: ❌ Issues:
|
|
138
|
+
- Missing: Progress reporting (spec says "report every 100 items")
|
|
139
|
+
- Extra: Added --json flag (not requested)
|
|
140
|
+
|
|
141
|
+
[Implementer fixes issues]
|
|
142
|
+
Implementer: Removed --json flag, added progress reporting
|
|
143
|
+
|
|
144
|
+
[Spec reviewer reviews again]
|
|
145
|
+
Spec reviewer: ✅ Spec compliant now
|
|
146
|
+
|
|
147
|
+
[Dispatch code quality reviewer]
|
|
148
|
+
Code reviewer: Strengths: Solid. Issues (Important): Magic number (100)
|
|
149
|
+
|
|
150
|
+
[Implementer fixes]
|
|
151
|
+
Implementer: Extracted PROGRESS_INTERVAL constant
|
|
152
|
+
|
|
153
|
+
[Code reviewer reviews again]
|
|
154
|
+
Code reviewer: ✅ Approved
|
|
155
|
+
|
|
156
|
+
[Mark Task 2 complete]
|
|
157
|
+
|
|
158
|
+
...
|
|
159
|
+
|
|
160
|
+
[After all tasks]
|
|
161
|
+
[Dispatch final code-reviewer]
|
|
162
|
+
Final reviewer: All requirements met, ready to merge
|
|
163
|
+
|
|
164
|
+
Done!
|
|
165
|
+
```
|
|
166
|
+
|
|
167
|
+
## Advantages
|
|
168
|
+
|
|
169
|
+
**vs. Manual execution:**
|
|
170
|
+
- Subagents follow TDD naturally
|
|
171
|
+
- Fresh context per task (no confusion)
|
|
172
|
+
- Parallel-safe (subagents don't interfere)
|
|
173
|
+
- Subagent can ask questions (before AND during work)
|
|
174
|
+
|
|
175
|
+
**vs. Executing Plans:**
|
|
176
|
+
- Same session (no handoff)
|
|
177
|
+
- Continuous progress (no waiting)
|
|
178
|
+
- Review checkpoints automatic
|
|
179
|
+
|
|
180
|
+
**Efficiency gains:**
|
|
181
|
+
- No file reading overhead (controller provides full text)
|
|
182
|
+
- Controller curates exactly what context is needed
|
|
183
|
+
- Subagent gets complete information upfront
|
|
184
|
+
- Questions surfaced before work begins (not after)
|
|
185
|
+
|
|
186
|
+
**Quality gates:**
|
|
187
|
+
- Self-review catches issues before handoff
|
|
188
|
+
- Two-stage review: spec compliance, then code quality
|
|
189
|
+
- Review loops ensure fixes actually work
|
|
190
|
+
- Spec compliance prevents over/under-building
|
|
191
|
+
- Code quality ensures implementation is well-built
|
|
192
|
+
|
|
193
|
+
**Cost:**
|
|
194
|
+
- More subagent invocations (implementer + 2 reviewers per task)
|
|
195
|
+
- Controller does more prep work (extracting all tasks upfront)
|
|
196
|
+
- Review loops add iterations
|
|
197
|
+
- But catches issues early (cheaper than debugging later)
|
|
198
|
+
|
|
199
|
+
## Red Flags
|
|
200
|
+
|
|
201
|
+
**Never:**
|
|
202
|
+
- Skip reviews (spec compliance OR code quality)
|
|
203
|
+
- Proceed with unfixed issues
|
|
204
|
+
- Dispatch multiple implementation subagents in parallel (conflicts)
|
|
205
|
+
- Make subagent read plan file (provide full text instead)
|
|
206
|
+
- Skip scene-setting context (subagent needs to understand where task fits)
|
|
207
|
+
- Ignore subagent questions (answer before letting them proceed)
|
|
208
|
+
- Accept "close enough" on spec compliance (spec reviewer found issues = not done)
|
|
209
|
+
- Skip review loops (reviewer found issues = implementer fixes = review again)
|
|
210
|
+
- Let implementer self-review replace actual review (both are needed)
|
|
211
|
+
- **Start code quality review before spec compliance is ✅** (wrong order)
|
|
212
|
+
- Move to next task while either review has open issues
|
|
213
|
+
|
|
214
|
+
**If subagent asks questions:**
|
|
215
|
+
- Answer clearly and completely
|
|
216
|
+
- Provide additional context if needed
|
|
217
|
+
- Don't rush them into implementation
|
|
218
|
+
|
|
219
|
+
**If reviewer finds issues:**
|
|
220
|
+
- Implementer (same subagent) fixes them
|
|
221
|
+
- Reviewer reviews again
|
|
222
|
+
- Repeat until approved
|
|
223
|
+
- Don't skip the re-review
|
|
224
|
+
|
|
225
|
+
**If subagent fails task:**
|
|
226
|
+
- Dispatch fix subagent with specific instructions
|
|
227
|
+
- Don't try to fix manually (context pollution)
|
|
228
|
+
|
|
229
|
+
## Integration
|
|
230
|
+
|
|
231
|
+
**Required workflow skills:**
|
|
232
|
+
- **superpowers:writing-plans** - Creates the plan this skill executes
|
|
233
|
+
- **superpowers:requesting-code-review** - Code review template for reviewer subagents
|
|
234
|
+
- **superpowers:finishing-a-development-branch** - Complete development after all tasks
|
|
235
|
+
|
|
236
|
+
**Subagents should use:**
|
|
237
|
+
- **superpowers:test-driven-development** - Subagents follow TDD for each task
|
|
238
|
+
|
|
239
|
+
**Alternative workflow:**
|
|
240
|
+
- **superpowers:executing-plans** - Use for parallel session instead of same-session execution
|
|
@@ -0,0 +1,20 @@
|
|
|
1
|
+
# Code Quality Reviewer Prompt Template
|
|
2
|
+
|
|
3
|
+
Use this template when dispatching a code quality reviewer subagent.
|
|
4
|
+
|
|
5
|
+
**Purpose:** Verify implementation is well-built (clean, tested, maintainable)
|
|
6
|
+
|
|
7
|
+
**Only dispatch after spec compliance review passes.**
|
|
8
|
+
|
|
9
|
+
```
|
|
10
|
+
Task tool (superpowers:code-reviewer):
|
|
11
|
+
Use template at requesting-code-review/code-reviewer.md
|
|
12
|
+
|
|
13
|
+
WHAT_WAS_IMPLEMENTED: [from implementer's report]
|
|
14
|
+
PLAN_OR_REQUIREMENTS: Task N from [plan-file]
|
|
15
|
+
BASE_SHA: [commit before task]
|
|
16
|
+
HEAD_SHA: [current commit]
|
|
17
|
+
DESCRIPTION: [task summary]
|
|
18
|
+
```
|
|
19
|
+
|
|
20
|
+
**Code reviewer returns:** Strengths, Issues (Critical/Important/Minor), Assessment
|
|
@@ -0,0 +1,78 @@
|
|
|
1
|
+
# Implementer Subagent Prompt Template
|
|
2
|
+
|
|
3
|
+
Use this template when dispatching an implementer subagent.
|
|
4
|
+
|
|
5
|
+
```
|
|
6
|
+
Task tool (general-purpose):
|
|
7
|
+
description: "Implement Task N: [task name]"
|
|
8
|
+
prompt: |
|
|
9
|
+
You are implementing Task N: [task name]
|
|
10
|
+
|
|
11
|
+
## Task Description
|
|
12
|
+
|
|
13
|
+
[FULL TEXT of task from plan - paste it here, don't make subagent read file]
|
|
14
|
+
|
|
15
|
+
## Context
|
|
16
|
+
|
|
17
|
+
[Scene-setting: where this fits, dependencies, architectural context]
|
|
18
|
+
|
|
19
|
+
## Before You Begin
|
|
20
|
+
|
|
21
|
+
If you have questions about:
|
|
22
|
+
- The requirements or acceptance criteria
|
|
23
|
+
- The approach or implementation strategy
|
|
24
|
+
- Dependencies or assumptions
|
|
25
|
+
- Anything unclear in the task description
|
|
26
|
+
|
|
27
|
+
**Ask them now.** Raise any concerns before starting work.
|
|
28
|
+
|
|
29
|
+
## Your Job
|
|
30
|
+
|
|
31
|
+
Once you're clear on requirements:
|
|
32
|
+
1. Implement exactly what the task specifies
|
|
33
|
+
2. Write tests (following TDD if task says to)
|
|
34
|
+
3. Verify implementation works
|
|
35
|
+
4. Commit your work
|
|
36
|
+
5. Self-review (see below)
|
|
37
|
+
6. Report back
|
|
38
|
+
|
|
39
|
+
Work from: [directory]
|
|
40
|
+
|
|
41
|
+
**While you work:** If you encounter something unexpected or unclear, **ask questions**.
|
|
42
|
+
It's always OK to pause and clarify. Don't guess or make assumptions.
|
|
43
|
+
|
|
44
|
+
## Before Reporting Back: Self-Review
|
|
45
|
+
|
|
46
|
+
Review your work with fresh eyes. Ask yourself:
|
|
47
|
+
|
|
48
|
+
**Completeness:**
|
|
49
|
+
- Did I fully implement everything in the spec?
|
|
50
|
+
- Did I miss any requirements?
|
|
51
|
+
- Are there edge cases I didn't handle?
|
|
52
|
+
|
|
53
|
+
**Quality:**
|
|
54
|
+
- Is this my best work?
|
|
55
|
+
- Are names clear and accurate (match what things do, not how they work)?
|
|
56
|
+
- Is the code clean and maintainable?
|
|
57
|
+
|
|
58
|
+
**Discipline:**
|
|
59
|
+
- Did I avoid overbuilding (YAGNI)?
|
|
60
|
+
- Did I only build what was requested?
|
|
61
|
+
- Did I follow existing patterns in the codebase?
|
|
62
|
+
|
|
63
|
+
**Testing:**
|
|
64
|
+
- Do tests actually verify behavior (not just mock behavior)?
|
|
65
|
+
- Did I follow TDD if required?
|
|
66
|
+
- Are tests comprehensive?
|
|
67
|
+
|
|
68
|
+
If you find issues during self-review, fix them now before reporting.
|
|
69
|
+
|
|
70
|
+
## Report Format
|
|
71
|
+
|
|
72
|
+
When done, report:
|
|
73
|
+
- What you implemented
|
|
74
|
+
- What you tested and test results
|
|
75
|
+
- Files changed
|
|
76
|
+
- Self-review findings (if any)
|
|
77
|
+
- Any issues or concerns
|
|
78
|
+
```
|
|
@@ -0,0 +1,61 @@
|
|
|
1
|
+
# Spec Compliance Reviewer Prompt Template
|
|
2
|
+
|
|
3
|
+
Use this template when dispatching a spec compliance reviewer subagent.
|
|
4
|
+
|
|
5
|
+
**Purpose:** Verify implementer built what was requested (nothing more, nothing less)
|
|
6
|
+
|
|
7
|
+
```
|
|
8
|
+
Task tool (general-purpose):
|
|
9
|
+
description: "Review spec compliance for Task N"
|
|
10
|
+
prompt: |
|
|
11
|
+
You are reviewing whether an implementation matches its specification.
|
|
12
|
+
|
|
13
|
+
## What Was Requested
|
|
14
|
+
|
|
15
|
+
[FULL TEXT of task requirements]
|
|
16
|
+
|
|
17
|
+
## What Implementer Claims They Built
|
|
18
|
+
|
|
19
|
+
[From implementer's report]
|
|
20
|
+
|
|
21
|
+
## CRITICAL: Do Not Trust the Report
|
|
22
|
+
|
|
23
|
+
The implementer finished suspiciously quickly. Their report may be incomplete,
|
|
24
|
+
inaccurate, or optimistic. You MUST verify everything independently.
|
|
25
|
+
|
|
26
|
+
**DO NOT:**
|
|
27
|
+
- Take their word for what they implemented
|
|
28
|
+
- Trust their claims about completeness
|
|
29
|
+
- Accept their interpretation of requirements
|
|
30
|
+
|
|
31
|
+
**DO:**
|
|
32
|
+
- Read the actual code they wrote
|
|
33
|
+
- Compare actual implementation to requirements line by line
|
|
34
|
+
- Check for missing pieces they claimed to implement
|
|
35
|
+
- Look for extra features they didn't mention
|
|
36
|
+
|
|
37
|
+
## Your Job
|
|
38
|
+
|
|
39
|
+
Read the implementation code and verify:
|
|
40
|
+
|
|
41
|
+
**Missing requirements:**
|
|
42
|
+
- Did they implement everything that was requested?
|
|
43
|
+
- Are there requirements they skipped or missed?
|
|
44
|
+
- Did they claim something works but didn't actually implement it?
|
|
45
|
+
|
|
46
|
+
**Extra/unneeded work:**
|
|
47
|
+
- Did they build things that weren't requested?
|
|
48
|
+
- Did they over-engineer or add unnecessary features?
|
|
49
|
+
- Did they add "nice to haves" that weren't in spec?
|
|
50
|
+
|
|
51
|
+
**Misunderstandings:**
|
|
52
|
+
- Did they interpret requirements differently than intended?
|
|
53
|
+
- Did they solve the wrong problem?
|
|
54
|
+
- Did they implement the right feature but wrong way?
|
|
55
|
+
|
|
56
|
+
**Verify by reading code, not by trusting report.**
|
|
57
|
+
|
|
58
|
+
Report:
|
|
59
|
+
- ✅ Spec compliant (if everything matches after code inspection)
|
|
60
|
+
- ❌ Issues found: [list specifically what's missing or extra, with file:line references]
|
|
61
|
+
```
|