@soleri/forge 9.2.0 → 9.3.1

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.
Files changed (46) hide show
  1. package/dist/scaffold-filetree.js +1 -1
  2. package/dist/skills/brain-debrief/SKILL.md +12 -12
  3. package/dist/skills/brainstorming/SKILL.md +7 -7
  4. package/dist/skills/code-patrol/SKILL.md +15 -15
  5. package/dist/skills/context-resume/SKILL.md +8 -8
  6. package/dist/skills/deep-review/SKILL.md +22 -11
  7. package/dist/skills/executing-plans/SKILL.md +10 -9
  8. package/dist/skills/fix-and-learn/SKILL.md +8 -8
  9. package/dist/skills/health-check/SKILL.md +11 -11
  10. package/dist/skills/knowledge-harvest/SKILL.md +17 -17
  11. package/dist/skills/onboard-me/SKILL.md +10 -10
  12. package/dist/skills/parallel-execute/SKILL.md +46 -32
  13. package/dist/skills/retrospective/SKILL.md +9 -9
  14. package/dist/skills/second-opinion/SKILL.md +11 -8
  15. package/dist/skills/systematic-debugging/SKILL.md +8 -8
  16. package/dist/skills/test-driven-development/SKILL.md +11 -11
  17. package/dist/skills/vault-capture/SKILL.md +15 -15
  18. package/dist/skills/vault-navigator/SKILL.md +10 -10
  19. package/dist/skills/vault-smells/SKILL.md +24 -16
  20. package/dist/skills/verification-before-completion/SKILL.md +18 -18
  21. package/dist/skills/writing-plans/SKILL.md +9 -9
  22. package/dist/templates/shared-rules.js +96 -9
  23. package/dist/templates/shared-rules.js.map +1 -1
  24. package/package.json +1 -1
  25. package/src/scaffold-filetree.ts +1 -1
  26. package/src/skills/brain-debrief/SKILL.md +12 -12
  27. package/src/skills/brainstorming/SKILL.md +7 -7
  28. package/src/skills/code-patrol/SKILL.md +15 -15
  29. package/src/skills/context-resume/SKILL.md +8 -8
  30. package/src/skills/deep-review/SKILL.md +22 -11
  31. package/src/skills/executing-plans/SKILL.md +10 -9
  32. package/src/skills/fix-and-learn/SKILL.md +8 -8
  33. package/src/skills/health-check/SKILL.md +11 -11
  34. package/src/skills/knowledge-harvest/SKILL.md +17 -17
  35. package/src/skills/onboard-me/SKILL.md +10 -10
  36. package/src/skills/parallel-execute/SKILL.md +46 -32
  37. package/src/skills/retrospective/SKILL.md +9 -9
  38. package/src/skills/second-opinion/SKILL.md +11 -8
  39. package/src/skills/systematic-debugging/SKILL.md +8 -8
  40. package/src/skills/test-driven-development/SKILL.md +11 -11
  41. package/src/skills/vault-capture/SKILL.md +15 -15
  42. package/src/skills/vault-navigator/SKILL.md +10 -10
  43. package/src/skills/vault-smells/SKILL.md +24 -16
  44. package/src/skills/verification-before-completion/SKILL.md +18 -18
  45. package/src/skills/writing-plans/SKILL.md +9 -9
  46. package/src/templates/shared-rules.ts +97 -9
@@ -73,7 +73,7 @@ When building a new feature, adding functionality, or creating components.
73
73
  tools: `tools:
74
74
  - soleri_vault op:search_intelligent
75
75
  - soleri_vault op:capture_knowledge
76
- - soleri_vault op:link_entries
76
+ - soleri_links op:link_entries
77
77
  - soleri_plan op:create_plan
78
78
  - soleri_plan op:approve_plan
79
79
  - soleri_brain op:recommend
@@ -56,15 +56,15 @@ Format as a report with: Strengths, Risks (recurring anti-patterns), Gaps, Stale
56
56
 
57
57
  ## Quick Reference
58
58
 
59
- | Op | When to Use |
60
- |----|-------------|
61
- | `brain_stats` | Aggregate metrics |
62
- | `brain_strengths` | Proven patterns ranked |
63
- | `brain_global_patterns` | Cross-project patterns |
64
- | `brain_recommend` | Project-similarity recommendations |
65
- | `brain_build_intelligence` | Rebuild intelligence pipeline |
66
- | `memory_topics` / `memory_stats` | Knowledge clusters and health |
67
- | `vault_age_report` | Stale entries |
68
- | `curator_health_audit` | Vault quality score |
69
- | `admin_vault_analytics` | Knowledge quality metrics |
70
- | `admin_search_insights` | Search miss analysis |
59
+ | Op | When to Use |
60
+ | -------------------------------- | ---------------------------------- |
61
+ | `brain_stats` | Aggregate metrics |
62
+ | `brain_strengths` | Proven patterns ranked |
63
+ | `brain_global_patterns` | Cross-project patterns |
64
+ | `brain_recommend` | Project-similarity recommendations |
65
+ | `brain_build_intelligence` | Rebuild intelligence pipeline |
66
+ | `memory_topics` / `memory_stats` | Knowledge clusters and health |
67
+ | `vault_age_report` | Stale entries |
68
+ | `curator_health_audit` | Vault quality score |
69
+ | `admin_vault_analytics` | Knowledge quality metrics |
70
+ | `admin_search_insights` | Search miss analysis |
@@ -79,11 +79,11 @@ Write validated design to `docs/plans/YYYY-MM-DD-<topic>-design.md` and commit.
79
79
 
80
80
  ## Quick Reference
81
81
 
82
- | Op | When to Use |
83
- |----|-------------|
84
- | `route_intent` | Classify work type |
85
- | `search_intelligent` | Search vault for prior art |
82
+ | Op | When to Use |
83
+ | ------------------------------ | -------------------------- |
84
+ | `route_intent` | Classify work type |
85
+ | `search_intelligent` | Search vault for prior art |
86
86
  | `vault_tags` / `vault_domains` | Browse knowledge landscape |
87
- | `brain_strengths` | Check proven patterns |
88
- | `memory_cross_project_search` | Check other projects |
89
- | `capture_knowledge` | Persist design decision |
87
+ | `brain_strengths` | Check proven patterns |
88
+ | `memory_cross_project_search` | Check other projects |
89
+ | `capture_knowledge` | Persist design decision |
@@ -32,13 +32,13 @@ YOUR_AGENT_core op:brain_strengths
32
32
 
33
33
  ### 3. Review the Code
34
34
 
35
- | Check | Source | Severity |
36
- |-------|--------|----------|
37
- | Violates critical rule | `search (severity: critical)` | Must fix |
38
- | Matches known anti-pattern | `search (type: anti-pattern)` | Must fix |
39
- | Doesn't follow proven pattern | `brain_strengths` | Should fix |
40
- | Breaks project conventions | `project_list_rules` | Should fix |
41
- | Misses pattern opportunity | `search_intelligent` | Could improve |
35
+ | Check | Source | Severity |
36
+ | ----------------------------- | ----------------------------- | ------------- |
37
+ | Violates critical rule | `search (severity: critical)` | Must fix |
38
+ | Matches known anti-pattern | `search (type: anti-pattern)` | Must fix |
39
+ | Doesn't follow proven pattern | `brain_strengths` | Should fix |
40
+ | Breaks project conventions | `project_list_rules` | Should fix |
41
+ | Misses pattern opportunity | `search_intelligent` | Could improve |
42
42
 
43
43
  ### 4. Present the Review
44
44
 
@@ -76,11 +76,11 @@ Re-run patrol after user applies fixes. Check `admin_health`.
76
76
 
77
77
  ## Quick Reference
78
78
 
79
- | Op | When to Use |
80
- |----|-------------|
81
- | `route_intent` | Classify code domain |
82
- | `search_intelligent` | Find relevant patterns |
83
- | `search` | Find anti-patterns and critical rules |
84
- | `project_list_rules` / `get_behavior_rules` | Project conventions |
85
- | `brain_strengths` | Proven patterns |
86
- | `capture_quick` / `capture_knowledge` | Capture new discoveries |
79
+ | Op | When to Use |
80
+ | ------------------------------------------- | ------------------------------------- |
81
+ | `route_intent` | Classify code domain |
82
+ | `search_intelligent` | Find relevant patterns |
83
+ | `search` | Find anti-patterns and critical rules |
84
+ | `project_list_rules` / `get_behavior_rules` | Project conventions |
85
+ | `brain_strengths` | Proven patterns |
86
+ | `capture_quick` / `capture_knowledge` | Capture new discoveries |
@@ -85,11 +85,11 @@ YOUR_AGENT_core op:admin_health
85
85
 
86
86
  ## Quick Reference
87
87
 
88
- | Op | When to Use |
89
- |----|-------------|
90
- | `plan_stats` / `get_plan` / `plan_list_tasks` | Active plans |
91
- | `memory_search` / `memory_list` | Session summaries |
92
- | `vault_recent` | Recently captured knowledge |
93
- | `loop_is_active` / `loop_status` | In-flight loops |
94
- | `brain_strengths` | Relevant proven patterns |
95
- | `admin_health` | System health check |
88
+ | Op | When to Use |
89
+ | --------------------------------------------- | --------------------------- |
90
+ | `plan_stats` / `get_plan` / `plan_list_tasks` | Active plans |
91
+ | `memory_search` / `memory_list` | Session summaries |
92
+ | `vault_recent` | Recently captured knowledge |
93
+ | `loop_is_active` / `loop_status` | In-flight loops |
94
+ | `brain_strengths` | Relevant proven patterns |
95
+ | `admin_health` | System health check |
@@ -20,12 +20,14 @@ The user provides a **target**: file, module, directory, PR diff, or function. I
20
20
  ### Pass 1: Structural Analysis & Code Smells
21
21
 
22
22
  **Metrics** (gather by reading the code):
23
+
23
24
  - File length and function count
24
25
  - Cyclomatic complexity (nesting depth, branch count)
25
26
  - Dependency count — imports from how many modules?
26
27
  - Export surface area — how much is public vs. should be internal?
27
28
 
28
29
  **Structural Smells:**
30
+
29
31
  - **God file/class** — too many responsibilities, >300 lines with mixed concerns
30
32
  - **Long parameter lists** — function takes 5+ params (should be an object/config)
31
33
  - **Deep nesting** — 4+ levels of if/for/try/catch
@@ -34,11 +36,13 @@ The user provides a **target**: file, module, directory, PR diff, or function. I
34
36
  - **Boolean blindness** — functions with multiple boolean params (`fn(true, false, true)`)
35
37
 
36
38
  **Duplication Smells:**
39
+
37
40
  - Copy-paste with slight variations
38
41
  - Repeated conditional logic — same if-chain in 3+ places
39
42
  - Parallel structures that always change together
40
43
 
41
44
  **Temporal Smells** (check git history):
45
+
42
46
  - Files that always change together but live in different modules → missing abstraction
43
47
  - Functions that get patched repeatedly → wrong abstraction
44
48
  - High churn files → instability signal
@@ -52,16 +56,19 @@ Present findings before moving to Pass 2.
52
56
  ### Pass 2: Architectural Fitness
53
57
 
54
58
  **Dependency Direction:**
59
+
55
60
  - Do dependencies flow in the right direction? (Outer layers depend on inner, not reverse)
56
61
  - Are there circular dependencies?
57
62
  - Does the code reach across module boundaries it shouldn't?
58
63
 
59
64
  **Abstraction Level:**
65
+
60
66
  - Is this the right level of abstraction for the problem?
61
67
  - Over-engineered? (abstraction for one use case, premature generalization)
62
68
  - Under-engineered? (inline logic that should be extracted)
63
69
 
64
70
  **Cohesion & Coupling:**
71
+
65
72
  - Does everything in this module belong together? (high cohesion)
66
73
  - Is the module tangled with others? (low coupling desired)
67
74
  - Feature envy — does a function touch another module's internals more than its own?
@@ -78,23 +85,27 @@ If vault has relevant patterns, check alignment. If not, skip — this pass work
78
85
  ### Pass 3: Solution Quality Assessment
79
86
 
80
87
  **Simplification:**
88
+
81
89
  - Is there a simpler way to achieve the same result?
82
90
  - Could any abstraction be removed without loss?
83
91
  - Are there standard library/framework features that replace custom code?
84
92
 
85
93
  **Edge Cases:**
94
+
86
95
  - What inputs would break this?
87
96
  - Are error paths handled or just the happy path?
88
97
  - What happens with empty/null/undefined inputs?
89
98
  - Concurrency: race conditions, shared mutable state?
90
99
 
91
100
  **Performance:**
101
+
92
102
  - Any O(n²) or worse hidden in loops?
93
103
  - Unnecessary allocations, copies, or serialization?
94
104
  - N+1 query patterns?
95
105
  - Unbounded growth (arrays/maps that grow without limit)?
96
106
 
97
107
  **Evolutionary Fitness:**
108
+
98
109
  - How does this code age? Easy to modify in 6 months?
99
110
  - Does it create "gravity" — attracting more complexity over time?
100
111
  - Are extension points in the right places?
@@ -145,12 +156,12 @@ If vault has relevant patterns, check alignment. If not, skip — this pass work
145
156
 
146
157
  ## Severity Scale
147
158
 
148
- | Level | Meaning |
149
- |-------|---------|
150
- | ✅ | Clean — no action needed |
151
- | 💡 | Info — worth knowing, low priority |
152
- | ⚠️ | Warning — should fix, causes friction |
153
- | 🔴 | Critical — fix before shipping, causes bugs or blocks scaling |
159
+ | Level | Meaning |
160
+ | ----- | ------------------------------------------------------------- |
161
+ | ✅ | Clean — no action needed |
162
+ | 💡 | Info — worth knowing, low priority |
163
+ | ⚠️ | Warning — should fix, causes friction |
164
+ | 🔴 | Critical — fix before shipping, causes bugs or blocks scaling |
154
165
 
155
166
  ## Capturing Learnings (Optional)
156
167
 
@@ -179,8 +190,8 @@ Only capture if genuinely reusable — not every review finding is vault-worthy.
179
190
 
180
191
  ## Quick Reference
181
192
 
182
- | Pass | Focus | Key Activities |
183
- |------|-------|----------------|
184
- | 1. Structural | Metrics + Smells | Read code, check complexity, find smells, check git history |
185
- | 2. Architecture | Fitness | Dependency direction, abstraction level, cohesion/coupling |
186
- | 3. Solution | Quality | Simplification, edge cases, performance, evolution |
193
+ | Pass | Focus | Key Activities |
194
+ | --------------- | ---------------- | ----------------------------------------------------------- |
195
+ | 1. Structural | Metrics + Smells | Read code, check complexity, find smells, check git history |
196
+ | 2. Architecture | Fitness | Dependency direction, abstraction level, cohesion/coupling |
197
+ | 3. Solution | Quality | Simplification, edge cases, performance, evolution |
@@ -36,6 +36,7 @@ YOUR_AGENT_core op:loop_start
36
36
  ### Step 3: Execute Batch (default: first 3 tasks)
37
37
 
38
38
  For each task:
39
+
39
40
  1. `op:update_task` — mark `in_progress`
40
41
  2. Follow each step exactly
41
42
  3. Run verifications as specified
@@ -78,14 +79,14 @@ Capture mid-execution learnings with `op:capture_quick` as they happen — don't
78
79
 
79
80
  ## Quick Reference
80
81
 
81
- | Op | When to Use |
82
- |----|-------------|
83
- | `get_plan` / `plan_list_tasks` / `plan_stats` | Load plan |
84
- | `update_task` | Mark task status |
85
- | `loop_start` / `loop_iterate` / `loop_complete` | Validation loop |
86
- | `plan_reconcile` | Post-execution drift report |
87
- | `plan_complete_lifecycle` | Extract knowledge, archive |
88
- | `session_capture` | Save session context |
89
- | `capture_quick` | Mid-execution learnings |
82
+ | Op | When to Use |
83
+ | ----------------------------------------------- | --------------------------- |
84
+ | `get_plan` / `plan_list_tasks` / `plan_stats` | Load plan |
85
+ | `update_task` | Mark task status |
86
+ | `loop_start` / `loop_iterate` / `loop_complete` | Validation loop |
87
+ | `plan_reconcile` | Post-execution drift report |
88
+ | `plan_complete_lifecycle` | Extract knowledge, archive |
89
+ | `session_capture` | Save session context |
90
+ | `capture_quick` | Mid-execution learnings |
90
91
 
91
92
  **Related skills:** writing-plans, verification-before-completion, test-driven-development
@@ -92,11 +92,11 @@ Bug resolved, tests pass, root cause captured in vault. A fix without a capture
92
92
 
93
93
  ## Quick Reference
94
94
 
95
- | Op | When to Use |
96
- |----|-------------|
97
- | `route_intent` | Classify as FIX intent |
98
- | `search_intelligent` | Check vault for known bugs |
99
- | `memory_search` | Search session memories |
100
- | `loop_start` / `loop_iterate` / `loop_complete` | Iterative fix cycle |
101
- | `capture_knowledge` / `capture_quick` | Persist anti-pattern |
102
- | `curator_detect_duplicates` | Prevent redundant entries |
95
+ | Op | When to Use |
96
+ | ----------------------------------------------- | -------------------------- |
97
+ | `route_intent` | Classify as FIX intent |
98
+ | `search_intelligent` | Check vault for known bugs |
99
+ | `memory_search` | Search session memories |
100
+ | `loop_start` / `loop_iterate` / `loop_complete` | Iterative fix cycle |
101
+ | `capture_knowledge` / `capture_quick` | Persist anti-pattern |
102
+ | `curator_detect_duplicates` | Prevent redundant entries |
@@ -122,14 +122,14 @@ YOUR_AGENT_core op:governance_expire
122
122
 
123
123
  ## Quick Reference
124
124
 
125
- | Op | When to Use |
126
- |----|-------------|
127
- | `admin_health` / `admin_diagnostic` | System health |
128
- | `admin_vault_analytics` / `admin_vault_size` | Vault metrics |
129
- | `curator_health_audit` | Quality score |
130
- | `curator_detect_duplicates` | Find duplicates |
131
- | `curator_contradictions` | Find conflicts |
132
- | `vault_age_report` | Stale entries |
133
- | `admin_search_insights` | Search miss analysis |
134
- | `curator_consolidate` | Full cleanup pipeline |
135
- | `brain_build_intelligence` | Rebuild after cleanup |
125
+ | Op | When to Use |
126
+ | -------------------------------------------- | --------------------- |
127
+ | `admin_health` / `admin_diagnostic` | System health |
128
+ | `admin_vault_analytics` / `admin_vault_size` | Vault metrics |
129
+ | `curator_health_audit` | Quality score |
130
+ | `curator_detect_duplicates` | Find duplicates |
131
+ | `curator_contradictions` | Find conflicts |
132
+ | `vault_age_report` | Stale entries |
133
+ | `admin_search_insights` | Search miss analysis |
134
+ | `curator_consolidate` | Full cleanup pipeline |
135
+ | `brain_build_intelligence` | Rebuild after cleanup |
@@ -30,13 +30,13 @@ Focus extraction on gaps — skip what vault already covers.
30
30
 
31
31
  ### 3. Extract and Classify
32
32
 
33
- | Type | What to Look For |
34
- |------|-----------------|
35
- | **pattern** | Repeatable approaches that work |
36
- | **anti-pattern** | Known mistakes to avoid |
37
- | **decision** | Architectural choices with rationale |
38
- | **principle** | Guiding rules or heuristics |
39
- | **workflow** | Step-by-step procedures |
33
+ | Type | What to Look For |
34
+ | ---------------- | ------------------------------------ |
35
+ | **pattern** | Repeatable approaches that work |
36
+ | **anti-pattern** | Known mistakes to avoid |
37
+ | **decision** | Architectural choices with rationale |
38
+ | **principle** | Guiding rules or heuristics |
39
+ | **workflow** | Step-by-step procedures |
40
40
 
41
41
  For each: determine category, severity, and tags.
42
42
 
@@ -85,13 +85,13 @@ Optionally promote universal patterns: `op:memory_promote_to_global`.
85
85
 
86
86
  ## Quick Reference
87
87
 
88
- | Op | When to Use |
89
- |----|-------------|
90
- | `search_intelligent` | Check existing knowledge |
91
- | `vault_tags` / `vault_domains` | See what's covered |
92
- | `capture_knowledge` | Capture each item |
93
- | `curator_detect_duplicates` | Post-harvest dedup |
94
- | `curator_groom_all` | Normalize entries |
95
- | `curator_contradictions` | Find conflicts |
96
- | `memory_promote_to_global` | Share cross-project |
97
- | `admin_vault_analytics` | Knowledge quality |
88
+ | Op | When to Use |
89
+ | ------------------------------ | ------------------------ |
90
+ | `search_intelligent` | Check existing knowledge |
91
+ | `vault_tags` / `vault_domains` | See what's covered |
92
+ | `capture_knowledge` | Capture each item |
93
+ | `curator_detect_duplicates` | Post-harvest dedup |
94
+ | `curator_groom_all` | Normalize entries |
95
+ | `curator_contradictions` | Find conflicts |
96
+ | `memory_promote_to_global` | Share cross-project |
97
+ | `admin_vault_analytics` | Knowledge quality |
@@ -109,13 +109,13 @@ YOUR_AGENT_core op:vault_age_report
109
109
 
110
110
  ## Quick Reference
111
111
 
112
- | Op | When to Use |
113
- |----|-------------|
114
- | `identity` / `project_get` | Project overview |
115
- | `project_list_rules` / `get_behavior_rules` | Conventions |
116
- | `vault_domains` / `vault_tags` | Knowledge landscape |
117
- | `search` (severity/type filters) | Critical rules, anti-patterns |
118
- | `search_intelligent` | Key decisions |
119
- | `brain_strengths` | Proven patterns |
120
- | `brain_global_patterns` | Cross-project patterns |
121
- | `admin_search_insights` | Knowledge gaps |
112
+ | Op | When to Use |
113
+ | ------------------------------------------- | ----------------------------- |
114
+ | `identity` / `project_get` | Project overview |
115
+ | `project_list_rules` / `get_behavior_rules` | Conventions |
116
+ | `vault_domains` / `vault_tags` | Knowledge landscape |
117
+ | `search` (severity/type filters) | Critical rules, anti-patterns |
118
+ | `search_intelligent` | Key decisions |
119
+ | `brain_strengths` | Proven patterns |
120
+ | `brain_global_patterns` | Cross-project patterns |
121
+ | `admin_search_insights` | Knowledge gaps |
@@ -24,6 +24,7 @@ You MUST have an approved, split plan before using this skill. If no plan exists
24
24
  - Tasks touch different files/modules (low merge conflict risk)
25
25
 
26
26
  **Do NOT use when:**
27
+
27
28
  - All tasks are sequential (each depends on the previous)
28
29
  - Tasks modify the same files (high conflict risk)
29
30
  - Plan has fewer than 3 tasks (use executing-plans instead)
@@ -60,17 +61,21 @@ Present the wave plan to the user before starting:
60
61
  For each task in the current wave:
61
62
 
62
63
  1. Check readiness:
64
+
63
65
  ```
64
66
  YOUR_AGENT_core op:plan_dispatch params:{ planId: "<id>", taskId: "<taskId>" }
65
67
  ```
68
+
66
69
  Only dispatch tasks where `ready: true`.
67
70
 
68
71
  2. Mark as in_progress:
72
+
69
73
  ```
70
74
  YOUR_AGENT_core op:update_task params:{ planId: "<id>", taskIndex: <n>, status: "in_progress" }
71
75
  ```
72
76
 
73
77
  3. Gather vault context for the task:
78
+
74
79
  ```
75
80
  YOUR_AGENT_core op:search params:{ query: "<task topic>", mode: "scan" }
76
81
  ```
@@ -114,12 +119,15 @@ Use the Agent tool with `isolation: "worktree"` when tasks touch nearby files to
114
119
  As subagents complete, collect their results. For each completed task:
115
120
 
116
121
  1. **Run spec review** — spawn a reviewer subagent:
122
+
117
123
  ```
118
124
  YOUR_AGENT_core op:plan_review_spec params:{ planId: "<id>", taskId: "<taskId>" }
119
125
  ```
126
+
120
127
  Use the returned prompt to launch a spec-review Agent that reads the ACTUAL code changes (not the implementer's self-report).
121
128
 
122
129
  2. **Record spec review outcome:**
130
+
123
131
  ```
124
132
  YOUR_AGENT_core op:plan_review_outcome params:{
125
133
  planId: "<id>", taskId: "<taskId>",
@@ -130,12 +138,15 @@ As subagents complete, collect their results. For each completed task:
130
138
  ```
131
139
 
132
140
  3. **If spec passes, run quality review:**
141
+
133
142
  ```
134
143
  YOUR_AGENT_core op:plan_review_quality params:{ planId: "<id>", taskId: "<taskId>" }
135
144
  ```
145
+
136
146
  Launch a quality-review Agent with the returned prompt.
137
147
 
138
148
  4. **Record quality review outcome:**
149
+
139
150
  ```
140
151
  YOUR_AGENT_core op:plan_review_outcome params:{
141
152
  planId: "<id>", taskId: "<taskId>",
@@ -147,12 +158,12 @@ As subagents complete, collect their results. For each completed task:
147
158
 
148
159
  5. **Handle outcomes:**
149
160
 
150
- | Spec | Quality | Action |
151
- |------|---------|--------|
152
- | Pass | Pass | Mark task completed |
153
- | Fail | — | Dispatch fix subagent with failure feedback (max 2 retries) |
154
- | Pass | Critical issues | Dispatch targeted fix subagent (max 2 retries) |
155
- | Pass | Minor issues only | Mark completed, note issues for later |
161
+ | Spec | Quality | Action |
162
+ | ---- | ----------------- | ----------------------------------------------------------- |
163
+ | Pass | Pass | Mark task completed |
164
+ | Fail | — | Dispatch fix subagent with failure feedback (max 2 retries) |
165
+ | Pass | Critical issues | Dispatch targeted fix subagent (max 2 retries) |
166
+ | Pass | Minor issues only | Mark completed, note issues for later |
156
167
 
157
168
  6. **Mark completed:**
158
169
  ```
@@ -164,6 +175,7 @@ As subagents complete, collect their results. For each completed task:
164
175
  After all tasks in a wave are complete (or failed after retries):
165
176
 
166
177
  1. Report wave results to the user:
178
+
167
179
  ```
168
180
  ## Wave N Complete
169
181
 
@@ -216,23 +228,23 @@ YOUR_AGENT_core op:session_capture params:{
216
228
 
217
229
  ## Subagent Isolation Rules
218
230
 
219
- | Situation | Isolation |
220
- |-----------|-----------|
221
- | Tasks touch completely different directories | No isolation needed |
222
- | Tasks touch files in the same package | Use `isolation: "worktree"` |
223
- | Tasks modify the same file | **Do NOT parallelize** — run sequentially |
231
+ | Situation | Isolation |
232
+ | -------------------------------------------- | ----------------------------------------- |
233
+ | Tasks touch completely different directories | No isolation needed |
234
+ | Tasks touch files in the same package | Use `isolation: "worktree"` |
235
+ | Tasks modify the same file | **Do NOT parallelize** — run sequentially |
224
236
 
225
237
  When using worktree isolation, the controller must merge worktree changes back after review passes.
226
238
 
227
239
  ## Failure Handling
228
240
 
229
- | Failure | Response |
230
- |---------|----------|
231
- | Subagent reports blocker | Pause that task, continue others in the wave |
232
- | Spec review fails | Dispatch fix subagent with feedback (retry 1/2) |
233
- | Second retry fails | Mark task as `failed`, escalate to user |
234
- | Merge conflict from worktree | Resolve manually, then re-run quality review |
235
- | All tasks in wave fail | Stop execution, report to user |
241
+ | Failure | Response |
242
+ | ---------------------------- | ----------------------------------------------- |
243
+ | Subagent reports blocker | Pause that task, continue others in the wave |
244
+ | Spec review fails | Dispatch fix subagent with feedback (retry 1/2) |
245
+ | Second retry fails | Mark task as `failed`, escalate to user |
246
+ | Merge conflict from worktree | Resolve manually, then re-run quality review |
247
+ | All tasks in wave fail | Stop execution, report to user |
236
248
 
237
249
  ## Capture Learnings
238
250
 
@@ -248,30 +260,32 @@ YOUR_AGENT_core op:capture_quick params:{
248
260
  ## When to Fall Back to Sequential
249
261
 
250
262
  **Switch to executing-plans skill mid-execution when:**
263
+
251
264
  - Subagents keep conflicting on shared files
252
265
  - Merge resolution is taking longer than the parallelization saves
253
266
  - User requests sequential execution
254
267
 
255
268
  ## Agent Tools Reference
256
269
 
257
- | Op | When to Use |
258
- |----|-------------|
259
- | `get_plan` | Load tracked plan |
260
- | `plan_list_tasks` | List all tasks with dependencies |
261
- | `plan_dispatch` | Check task readiness (dependencies met?) |
262
- | `update_task` | Mark tasks in_progress / completed / failed |
263
- | `plan_review_spec` | Generate spec compliance review prompt |
264
- | `plan_review_quality` | Generate code quality review prompt |
265
- | `plan_review_outcome` | Record review pass/fail result |
266
- | `plan_reconcile` | Post-execution drift analysis |
267
- | `plan_complete_lifecycle` | Extract knowledge, archive |
268
- | `session_capture` | Save session context |
269
- | `capture_quick` | Capture mid-execution learnings |
270
- | `search` | Vault lookup for task context |
270
+ | Op | When to Use |
271
+ | ------------------------- | ------------------------------------------- |
272
+ | `get_plan` | Load tracked plan |
273
+ | `plan_list_tasks` | List all tasks with dependencies |
274
+ | `plan_dispatch` | Check task readiness (dependencies met?) |
275
+ | `update_task` | Mark tasks in_progress / completed / failed |
276
+ | `plan_review_spec` | Generate spec compliance review prompt |
277
+ | `plan_review_quality` | Generate code quality review prompt |
278
+ | `plan_review_outcome` | Record review pass/fail result |
279
+ | `plan_reconcile` | Post-execution drift analysis |
280
+ | `plan_complete_lifecycle` | Extract knowledge, archive |
281
+ | `session_capture` | Save session context |
282
+ | `capture_quick` | Capture mid-execution learnings |
283
+ | `search` | Vault lookup for task context |
271
284
 
272
285
  ## Integration
273
286
 
274
287
  **Required skills:**
288
+
275
289
  - writing-plans — Creates the plan this skill executes
276
290
  - verification-before-completion — Verify work before claiming completion
277
291
  - executing-plans — Fallback for sequential execution
@@ -87,12 +87,12 @@ If quality issues found: `op:curator_consolidate` then `op:brain_build_intellige
87
87
 
88
88
  ## Quick Reference
89
89
 
90
- | Op | When to Use |
91
- |----|-------------|
92
- | `brain_stats` / `brain_strengths` | Metrics and patterns |
93
- | `vault_recent` | Recent captures |
94
- | `memory_topics` / `memory_stats` | Knowledge clusters |
95
- | `plan_stats` | Plan completion |
96
- | `admin_search_insights` | Search misses |
97
- | `curator_health_audit` | Vault quality |
98
- | `capture_knowledge` | Persist retrospective |
90
+ | Op | When to Use |
91
+ | --------------------------------- | --------------------- |
92
+ | `brain_stats` / `brain_strengths` | Metrics and patterns |
93
+ | `vault_recent` | Recent captures |
94
+ | `memory_topics` / `memory_stats` | Knowledge clusters |
95
+ | `plan_stats` | Plan completion |
96
+ | `admin_search_insights` | Search misses |
97
+ | `curator_health_audit` | Vault quality |
98
+ | `capture_knowledge` | Persist retrospective |
@@ -19,12 +19,14 @@ YOUR_AGENT_core op:route_intent
19
19
  ### 2. Search All Knowledge Sources
20
20
 
21
21
  **Vault** — previous decisions, patterns, anti-patterns:
22
+
22
23
  ```
23
24
  YOUR_AGENT_core op:search_intelligent
24
25
  params: { query: "<the decision or options>" }
25
26
  ```
26
27
 
27
28
  **Brain** — proven approaches:
29
+
28
30
  ```
29
31
  YOUR_AGENT_core op:brain_strengths
30
32
  YOUR_AGENT_core op:brain_recommend
@@ -32,6 +34,7 @@ YOUR_AGENT_core op:brain_recommend
32
34
  ```
33
35
 
34
36
  **Cross-project** — what other projects chose:
37
+
35
38
  ```
36
39
  YOUR_AGENT_core op:memory_cross_project_search
37
40
  params: { query: "<topic>", crossProject: true }
@@ -87,11 +90,11 @@ YOUR_AGENT_core op:capture_knowledge
87
90
 
88
91
  ## Quick Reference
89
92
 
90
- | Op | When to Use |
91
- |----|-------------|
92
- | `route_intent` | Classify decision type |
93
- | `search_intelligent` | Find previous decisions |
94
- | `brain_strengths` / `brain_recommend` | Proven approaches |
95
- | `memory_cross_project_search` | Other projects' choices |
96
- | `memory_search` | Session context |
97
- | `capture_knowledge` | Persist the decision |
93
+ | Op | When to Use |
94
+ | ------------------------------------- | ----------------------- |
95
+ | `route_intent` | Classify decision type |
96
+ | `search_intelligent` | Find previous decisions |
97
+ | `brain_strengths` / `brain_recommend` | Proven approaches |
98
+ | `memory_cross_project_search` | Other projects' choices |
99
+ | `memory_search` | Session context |
100
+ | `capture_knowledge` | Persist the decision |