mirror of
https://github.com/obra/superpowers.git
synced 2026-07-07 02:09:04 +08:00
refactor(skills): drop Advantages section from subagent-driven-development
Five blocks of benefits and cost/benefit selling aimed at a reader who has already invoked the skill; the vs-Executing-Plans comparison also duplicates the one under When to Use. Integration section untouched (PR #1932 owns it).
This commit is contained in:
@@ -332,38 +332,6 @@ Final reviewer: All requirements met, ready to merge
|
|||||||
Done!
|
Done!
|
||||||
```
|
```
|
||||||
|
|
||||||
## Advantages
|
|
||||||
|
|
||||||
**vs. Manual execution:**
|
|
||||||
- Subagents follow TDD naturally
|
|
||||||
- Fresh context per task (no confusion)
|
|
||||||
- Parallel-safe (subagents don't interfere)
|
|
||||||
- Subagent can ask questions (before AND during work)
|
|
||||||
|
|
||||||
**vs. Executing Plans:**
|
|
||||||
- Same session (no handoff)
|
|
||||||
- Continuous progress (no waiting)
|
|
||||||
- Review checkpoints automatic
|
|
||||||
|
|
||||||
**Efficiency gains:**
|
|
||||||
- Controller curates exactly what context is needed; bulk artifacts move
|
|
||||||
as files, not pasted text
|
|
||||||
- Subagent gets complete information upfront
|
|
||||||
- Questions surfaced before work begins (not after)
|
|
||||||
|
|
||||||
**Quality gates:**
|
|
||||||
- Self-review catches issues before handoff
|
|
||||||
- Task review carries two verdicts: spec compliance and code quality
|
|
||||||
- Review loops ensure fixes actually work
|
|
||||||
- Spec compliance prevents over/under-building
|
|
||||||
- Code quality ensures implementation is well-built
|
|
||||||
|
|
||||||
**Cost:**
|
|
||||||
- More subagent invocations (implementer + reviewer per task)
|
|
||||||
- Controller does more prep work (extracting all tasks upfront)
|
|
||||||
- Review loops add iterations
|
|
||||||
- But catches issues early (cheaper than debugging later)
|
|
||||||
|
|
||||||
## Red Flags
|
## Red Flags
|
||||||
|
|
||||||
**Never:**
|
**Never:**
|
||||||
|
|||||||
Reference in New Issue
Block a user