@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.
- package/dist/scaffold-filetree.js +1 -1
- package/dist/skills/brain-debrief/SKILL.md +12 -12
- package/dist/skills/brainstorming/SKILL.md +7 -7
- package/dist/skills/code-patrol/SKILL.md +15 -15
- package/dist/skills/context-resume/SKILL.md +8 -8
- package/dist/skills/deep-review/SKILL.md +22 -11
- package/dist/skills/executing-plans/SKILL.md +10 -9
- package/dist/skills/fix-and-learn/SKILL.md +8 -8
- package/dist/skills/health-check/SKILL.md +11 -11
- package/dist/skills/knowledge-harvest/SKILL.md +17 -17
- package/dist/skills/onboard-me/SKILL.md +10 -10
- package/dist/skills/parallel-execute/SKILL.md +46 -32
- package/dist/skills/retrospective/SKILL.md +9 -9
- package/dist/skills/second-opinion/SKILL.md +11 -8
- package/dist/skills/systematic-debugging/SKILL.md +8 -8
- package/dist/skills/test-driven-development/SKILL.md +11 -11
- package/dist/skills/vault-capture/SKILL.md +15 -15
- package/dist/skills/vault-navigator/SKILL.md +10 -10
- package/dist/skills/vault-smells/SKILL.md +24 -16
- package/dist/skills/verification-before-completion/SKILL.md +18 -18
- package/dist/skills/writing-plans/SKILL.md +9 -9
- package/dist/templates/shared-rules.js +96 -9
- package/dist/templates/shared-rules.js.map +1 -1
- package/package.json +1 -1
- package/src/scaffold-filetree.ts +1 -1
- package/src/skills/brain-debrief/SKILL.md +12 -12
- package/src/skills/brainstorming/SKILL.md +7 -7
- package/src/skills/code-patrol/SKILL.md +15 -15
- package/src/skills/context-resume/SKILL.md +8 -8
- package/src/skills/deep-review/SKILL.md +22 -11
- package/src/skills/executing-plans/SKILL.md +10 -9
- package/src/skills/fix-and-learn/SKILL.md +8 -8
- package/src/skills/health-check/SKILL.md +11 -11
- package/src/skills/knowledge-harvest/SKILL.md +17 -17
- package/src/skills/onboard-me/SKILL.md +10 -10
- package/src/skills/parallel-execute/SKILL.md +46 -32
- package/src/skills/retrospective/SKILL.md +9 -9
- package/src/skills/second-opinion/SKILL.md +11 -8
- package/src/skills/systematic-debugging/SKILL.md +8 -8
- package/src/skills/test-driven-development/SKILL.md +11 -11
- package/src/skills/vault-capture/SKILL.md +15 -15
- package/src/skills/vault-navigator/SKILL.md +10 -10
- package/src/skills/vault-smells/SKILL.md +24 -16
- package/src/skills/verification-before-completion/SKILL.md +18 -18
- package/src/skills/writing-plans/SKILL.md +9 -9
- 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
|
-
-
|
|
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
|
|
60
|
-
|
|
61
|
-
| `brain_stats`
|
|
62
|
-
| `brain_strengths`
|
|
63
|
-
| `brain_global_patterns`
|
|
64
|
-
| `brain_recommend`
|
|
65
|
-
| `brain_build_intelligence`
|
|
66
|
-
| `memory_topics` / `memory_stats` | Knowledge clusters and health
|
|
67
|
-
| `vault_age_report`
|
|
68
|
-
| `curator_health_audit`
|
|
69
|
-
| `admin_vault_analytics`
|
|
70
|
-
| `admin_search_insights`
|
|
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
|
|
83
|
-
|
|
84
|
-
| `route_intent`
|
|
85
|
-
| `search_intelligent`
|
|
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`
|
|
88
|
-
| `memory_cross_project_search`
|
|
89
|
-
| `capture_knowledge`
|
|
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
|
|
36
|
-
|
|
37
|
-
| Violates critical rule
|
|
38
|
-
| Matches known anti-pattern
|
|
39
|
-
| Doesn't follow proven pattern | `brain_strengths`
|
|
40
|
-
| Breaks project conventions
|
|
41
|
-
| Misses pattern opportunity
|
|
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
|
|
80
|
-
|
|
81
|
-
| `route_intent`
|
|
82
|
-
| `search_intelligent`
|
|
83
|
-
| `search`
|
|
84
|
-
| `project_list_rules` / `get_behavior_rules` | Project conventions
|
|
85
|
-
| `brain_strengths`
|
|
86
|
-
| `capture_quick` / `capture_knowledge`
|
|
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
|
|
89
|
-
|
|
90
|
-
| `plan_stats` / `get_plan` / `plan_list_tasks` | Active plans
|
|
91
|
-
| `memory_search` / `memory_list`
|
|
92
|
-
| `vault_recent`
|
|
93
|
-
| `loop_is_active` / `loop_status`
|
|
94
|
-
| `brain_strengths`
|
|
95
|
-
| `admin_health`
|
|
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
|
-
| ✅
|
|
151
|
-
| 💡
|
|
152
|
-
| ⚠️
|
|
153
|
-
| 🔴
|
|
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
|
|
183
|
-
|
|
184
|
-
| 1. Structural
|
|
185
|
-
| 2. Architecture | Fitness
|
|
186
|
-
| 3. Solution
|
|
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
|
|
82
|
-
|
|
83
|
-
| `get_plan` / `plan_list_tasks` / `plan_stats`
|
|
84
|
-
| `update_task`
|
|
85
|
-
| `loop_start` / `loop_iterate` / `loop_complete` | Validation loop
|
|
86
|
-
| `plan_reconcile`
|
|
87
|
-
| `plan_complete_lifecycle`
|
|
88
|
-
| `session_capture`
|
|
89
|
-
| `capture_quick`
|
|
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
|
|
96
|
-
|
|
97
|
-
| `route_intent`
|
|
98
|
-
| `search_intelligent`
|
|
99
|
-
| `memory_search`
|
|
100
|
-
| `loop_start` / `loop_iterate` / `loop_complete` | Iterative fix cycle
|
|
101
|
-
| `capture_knowledge` / `capture_quick`
|
|
102
|
-
| `curator_detect_duplicates`
|
|
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
|
|
126
|
-
|
|
127
|
-
| `admin_health` / `admin_diagnostic`
|
|
128
|
-
| `admin_vault_analytics` / `admin_vault_size` | Vault metrics
|
|
129
|
-
| `curator_health_audit`
|
|
130
|
-
| `curator_detect_duplicates`
|
|
131
|
-
| `curator_contradictions`
|
|
132
|
-
| `vault_age_report`
|
|
133
|
-
| `admin_search_insights`
|
|
134
|
-
| `curator_consolidate`
|
|
135
|
-
| `brain_build_intelligence`
|
|
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
|
|
34
|
-
|
|
35
|
-
| **pattern**
|
|
36
|
-
| **anti-pattern** | Known mistakes to avoid
|
|
37
|
-
| **decision**
|
|
38
|
-
| **principle**
|
|
39
|
-
| **workflow**
|
|
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
|
|
89
|
-
|
|
90
|
-
| `search_intelligent`
|
|
91
|
-
| `vault_tags` / `vault_domains` | See what's covered
|
|
92
|
-
| `capture_knowledge`
|
|
93
|
-
| `curator_detect_duplicates`
|
|
94
|
-
| `curator_groom_all`
|
|
95
|
-
| `curator_contradictions`
|
|
96
|
-
| `memory_promote_to_global`
|
|
97
|
-
| `admin_vault_analytics`
|
|
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
|
|
113
|
-
|
|
114
|
-
| `identity` / `project_get`
|
|
115
|
-
| `project_list_rules` / `get_behavior_rules` | Conventions
|
|
116
|
-
| `vault_domains` / `vault_tags`
|
|
117
|
-
| `search` (severity/type filters)
|
|
118
|
-
| `search_intelligent`
|
|
119
|
-
| `brain_strengths`
|
|
120
|
-
| `brain_global_patterns`
|
|
121
|
-
| `admin_search_insights`
|
|
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
|
|
151
|
-
|
|
152
|
-
| Pass | Pass
|
|
153
|
-
| Fail | —
|
|
154
|
-
| Pass | Critical issues
|
|
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
|
|
220
|
-
|
|
221
|
-
| Tasks touch completely different directories | No isolation needed
|
|
222
|
-
| Tasks touch files in the same package
|
|
223
|
-
| Tasks modify the same file
|
|
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
|
|
230
|
-
|
|
231
|
-
| Subagent reports blocker
|
|
232
|
-
| Spec review fails
|
|
233
|
-
| Second retry fails
|
|
234
|
-
| Merge conflict from worktree | Resolve manually, then re-run quality review
|
|
235
|
-
| All tasks in wave fail
|
|
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
|
|
258
|
-
|
|
259
|
-
| `get_plan`
|
|
260
|
-
| `plan_list_tasks`
|
|
261
|
-
| `plan_dispatch`
|
|
262
|
-
| `update_task`
|
|
263
|
-
| `plan_review_spec`
|
|
264
|
-
| `plan_review_quality`
|
|
265
|
-
| `plan_review_outcome`
|
|
266
|
-
| `plan_reconcile`
|
|
267
|
-
| `plan_complete_lifecycle` | Extract knowledge, archive
|
|
268
|
-
| `session_capture`
|
|
269
|
-
| `capture_quick`
|
|
270
|
-
| `search`
|
|
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
|
|
91
|
-
|
|
92
|
-
| `brain_stats` / `brain_strengths` | Metrics and patterns
|
|
93
|
-
| `vault_recent`
|
|
94
|
-
| `memory_topics` / `memory_stats`
|
|
95
|
-
| `plan_stats`
|
|
96
|
-
| `admin_search_insights`
|
|
97
|
-
| `curator_health_audit`
|
|
98
|
-
| `capture_knowledge`
|
|
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
|
|
91
|
-
|
|
92
|
-
| `route_intent`
|
|
93
|
-
| `search_intelligent`
|
|
94
|
-
| `brain_strengths` / `brain_recommend` | Proven approaches
|
|
95
|
-
| `memory_cross_project_search`
|
|
96
|
-
| `memory_search`
|
|
97
|
-
| `capture_knowledge`
|
|
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 |
|