@fro.bot/systematic 1.23.0 → 1.23.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/agents/research/best-practices-researcher.md +9 -3
- package/agents/research/framework-docs-researcher.md +2 -0
- package/agents/research/git-history-analyzer.md +9 -6
- package/agents/research/issue-intelligence-analyst.md +232 -0
- package/agents/research/repo-research-analyst.md +6 -10
- package/commands/.gitkeep +0 -0
- package/package.json +1 -1
- package/skills/agent-browser/SKILL.md +4 -3
- package/skills/ce-brainstorm/SKILL.md +242 -52
- package/skills/ce-compound/SKILL.md +60 -40
- package/skills/ce-compound-refresh/SKILL.md +528 -0
- package/skills/ce-ideate/SKILL.md +371 -0
- package/skills/ce-plan/SKILL.md +40 -39
- package/skills/ce-plan-beta/SKILL.md +572 -0
- package/skills/ce-review/SKILL.md +7 -6
- package/skills/ce-work/SKILL.md +85 -75
- package/skills/create-agent-skill/SKILL.md +1 -1
- package/skills/create-agent-skills/SKILL.md +6 -5
- package/skills/deepen-plan/SKILL.md +11 -11
- package/skills/deepen-plan-beta/SKILL.md +323 -0
- package/skills/document-review/SKILL.md +14 -8
- package/skills/generate_command/SKILL.md +3 -2
- package/skills/lfg/SKILL.md +10 -7
- package/skills/report-bug/SKILL.md +15 -14
- package/skills/resolve_parallel/SKILL.md +2 -1
- package/skills/resolve_todo_parallel/SKILL.md +1 -1
- package/skills/slfg/SKILL.md +7 -4
- package/skills/test-browser/SKILL.md +3 -3
- package/skills/test-xcode/SKILL.md +2 -2
- package/agents/workflow/every-style-editor.md +0 -66
- package/commands/agent-native-audit.md +0 -279
- package/commands/ce/brainstorm.md +0 -145
- package/commands/ce/compound.md +0 -240
- package/commands/ce/plan.md +0 -636
- package/commands/ce/review.md +0 -525
- package/commands/ce/work.md +0 -456
- package/commands/changelog.md +0 -139
- package/commands/create-agent-skill.md +0 -9
- package/commands/deepen-plan.md +0 -546
- package/commands/deploy-docs.md +0 -120
- package/commands/feature-video.md +0 -352
- package/commands/generate_command.md +0 -164
- package/commands/heal-skill.md +0 -147
- package/commands/lfg.md +0 -20
- package/commands/report-bug.md +0 -151
- package/commands/reproduce-bug.md +0 -100
- package/commands/resolve_parallel.md +0 -36
- package/commands/resolve_todo_parallel.md +0 -37
- package/commands/slfg.md +0 -32
- package/commands/test-browser.md +0 -340
- package/commands/test-xcode.md +0 -332
- package/commands/triage.md +0 -311
- package/commands/workflows/brainstorm.md +0 -145
- package/commands/workflows/compound.md +0 -10
- package/commands/workflows/plan.md +0 -10
- package/commands/workflows/review.md +0 -10
- package/commands/workflows/work.md +0 -10
- package/skills/brainstorming/SKILL.md +0 -190
- package/skills/skill-creator/SKILL.md +0 -210
- package/skills/skill-creator/scripts/init_skill.py +0 -303
- package/skills/skill-creator/scripts/package_skill.py +0 -110
- package/skills/skill-creator/scripts/quick_validate.py +0 -65
|
@@ -0,0 +1,323 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: deepen-plan-beta
|
|
3
|
+
description: '[BETA] Stress-test an existing implementation plan and selectively strengthen weak sections with targeted research. Use when a plan needs more confidence around decisions, sequencing, system-wide impact, risks, or verification. Best for Standard or Deep plans, or high-risk topics such as auth, payments, migrations, external APIs, and security. For structural or clarity improvements, prefer document-review instead.'
|
|
4
|
+
argument-hint: '[path to plan file]'
|
|
5
|
+
disable-model-invocation: true
|
|
6
|
+
---
|
|
7
|
+
|
|
8
|
+
# Deepen Plan
|
|
9
|
+
|
|
10
|
+
## Introduction
|
|
11
|
+
|
|
12
|
+
**Note: The current year is 2026.** Use this when searching for recent documentation and best practices.
|
|
13
|
+
|
|
14
|
+
`ce:plan-beta` does the first planning pass. `deepen-plan-beta` is a second-pass confidence check.
|
|
15
|
+
|
|
16
|
+
Use this skill when the plan already exists and the question is not "Is this document clear?" but rather "Is this plan grounded enough for the complexity and risk involved?"
|
|
17
|
+
|
|
18
|
+
This skill does **not** turn plans into implementation scripts. It identifies weak sections, runs targeted research only for those sections, and strengthens the plan in place.
|
|
19
|
+
|
|
20
|
+
`document-review` and `deepen-plan-beta` are different:
|
|
21
|
+
- Use the `document-review` skill when the document needs clarity, simplification, completeness, or scope control
|
|
22
|
+
- Use `deepen-plan-beta` when the document is structurally sound but still needs stronger rationale, sequencing, risk treatment, or system-wide thinking
|
|
23
|
+
|
|
24
|
+
## Interaction Method
|
|
25
|
+
|
|
26
|
+
Use the platform's question tool when available. When asking the user a question, prefer the platform's blocking question tool if one exists (`AskUserQuestion` in OpenCode, `request_user_input` in Codex, `ask_user` in Gemini). Otherwise, present numbered options in chat and wait for the user's reply before proceeding.
|
|
27
|
+
|
|
28
|
+
Ask one question at a time. Prefer a concise single-select choice when natural options exist.
|
|
29
|
+
|
|
30
|
+
## Plan File
|
|
31
|
+
|
|
32
|
+
<plan_path> #$ARGUMENTS </plan_path>
|
|
33
|
+
|
|
34
|
+
If the plan path above is empty:
|
|
35
|
+
1. Check `docs/plans/` for recent files
|
|
36
|
+
2. Ask the user which plan to deepen using the platform's blocking question tool when available (see Interaction Method). Otherwise, present numbered options in chat and wait for the user's reply before proceeding
|
|
37
|
+
|
|
38
|
+
Do not proceed until you have a valid plan file path.
|
|
39
|
+
|
|
40
|
+
## Core Principles
|
|
41
|
+
|
|
42
|
+
1. **Stress-test, do not inflate** - Deepening should increase justified confidence, not make the plan longer for its own sake.
|
|
43
|
+
2. **Selective depth only** - Focus on the weakest 2-5 sections rather than enriching everything.
|
|
44
|
+
3. **Preserve the planning boundary** - No implementation code, no git command choreography, no exact test command recipes.
|
|
45
|
+
4. **Use artifact-contained evidence** - Work from the written plan, its `Context & Research`, `Sources & References`, and its origin document when present.
|
|
46
|
+
5. **Respect product boundaries** - Do not invent new product requirements. If deepening reveals a product-level gap, surface it as an open question or route back to `ce:brainstorm`.
|
|
47
|
+
6. **Prioritize risk and cross-cutting impact** - The more dangerous or interconnected the work, the more valuable another planning pass becomes.
|
|
48
|
+
|
|
49
|
+
## Workflow
|
|
50
|
+
|
|
51
|
+
### Phase 0: Load the Plan and Decide Whether Deepening Is Warranted
|
|
52
|
+
|
|
53
|
+
#### 0.1 Read the Plan and Supporting Inputs
|
|
54
|
+
|
|
55
|
+
Read the plan file completely.
|
|
56
|
+
|
|
57
|
+
If the plan frontmatter includes an `origin:` path:
|
|
58
|
+
- Read the origin document too
|
|
59
|
+
- Use it to check whether the plan still reflects the product intent, scope boundaries, and success criteria
|
|
60
|
+
|
|
61
|
+
#### 0.2 Classify Plan Depth and Topic Risk
|
|
62
|
+
|
|
63
|
+
Determine the plan depth from the document:
|
|
64
|
+
- **Lightweight** - small, bounded, low ambiguity, usually 2-4 implementation units
|
|
65
|
+
- **Standard** - moderate complexity, some technical decisions, usually 3-6 units
|
|
66
|
+
- **Deep** - cross-cutting, high-risk, or strategically important work, usually 4-8 units or phased delivery
|
|
67
|
+
|
|
68
|
+
Also build a risk profile. Treat these as high-risk signals:
|
|
69
|
+
- Authentication, authorization, or security-sensitive behavior
|
|
70
|
+
- Payments, billing, or financial flows
|
|
71
|
+
- Data migrations, backfills, or persistent data changes
|
|
72
|
+
- External APIs or third-party integrations
|
|
73
|
+
- Privacy, compliance, or user data handling
|
|
74
|
+
- Cross-interface parity or multi-surface behavior
|
|
75
|
+
- Significant rollout, monitoring, or operational concerns
|
|
76
|
+
|
|
77
|
+
#### 0.3 Decide Whether to Deepen
|
|
78
|
+
|
|
79
|
+
Use this default:
|
|
80
|
+
- **Lightweight** plans usually do not need deepening unless they are high-risk or the user explicitly requests it
|
|
81
|
+
- **Standard** plans often benefit when one or more important sections still look thin
|
|
82
|
+
- **Deep** or high-risk plans often benefit from a targeted second pass
|
|
83
|
+
|
|
84
|
+
If the plan already appears sufficiently grounded:
|
|
85
|
+
- Say so briefly
|
|
86
|
+
- Recommend moving to `/systematic:ce-work` or the `document-review` skill
|
|
87
|
+
- If the user explicitly asked to deepen anyway, continue with a light pass and deepen at most 1-2 sections
|
|
88
|
+
|
|
89
|
+
### Phase 1: Parse the Current `ce:plan-beta` Structure
|
|
90
|
+
|
|
91
|
+
Map the plan into the current template. Look for these sections, or their nearest equivalents:
|
|
92
|
+
- `Overview`
|
|
93
|
+
- `Problem Frame`
|
|
94
|
+
- `Requirements Trace`
|
|
95
|
+
- `Scope Boundaries`
|
|
96
|
+
- `Context & Research`
|
|
97
|
+
- `Key Technical Decisions`
|
|
98
|
+
- `Open Questions`
|
|
99
|
+
- `Implementation Units`
|
|
100
|
+
- `System-Wide Impact`
|
|
101
|
+
- `Risks & Dependencies`
|
|
102
|
+
- `Documentation / Operational Notes`
|
|
103
|
+
- `Sources & References`
|
|
104
|
+
- Optional deep-plan sections such as `Alternative Approaches Considered`, `Success Metrics`, `Phased Delivery`, `Risk Analysis & Mitigation`, and `Operational / Rollout Notes`
|
|
105
|
+
|
|
106
|
+
If the plan was written manually or uses different headings:
|
|
107
|
+
- Map sections by intent rather than exact heading names
|
|
108
|
+
- If a section is structurally present but titled differently, treat it as the equivalent section
|
|
109
|
+
- If the plan truly lacks a section, decide whether that absence is intentional for the plan depth or a confidence gap worth scoring
|
|
110
|
+
|
|
111
|
+
Also collect:
|
|
112
|
+
- Frontmatter, including existing `deepened:` date if present
|
|
113
|
+
- Number of implementation units
|
|
114
|
+
- Which files and test files are named
|
|
115
|
+
- Which learnings, patterns, or external references are cited
|
|
116
|
+
- Which sections appear omitted because they were unnecessary versus omitted because they are missing
|
|
117
|
+
|
|
118
|
+
### Phase 2: Score Confidence Gaps
|
|
119
|
+
|
|
120
|
+
Use a checklist-first, risk-weighted scoring pass.
|
|
121
|
+
|
|
122
|
+
For each section, compute:
|
|
123
|
+
- **Trigger count** - number of checklist problems that apply
|
|
124
|
+
- **Risk bonus** - add 1 if the topic is high-risk and this section is materially relevant to that risk
|
|
125
|
+
- **Critical-section bonus** - add 1 for `Key Technical Decisions`, `Implementation Units`, `System-Wide Impact`, `Risks & Dependencies`, or `Open Questions` in `Standard` or `Deep` plans
|
|
126
|
+
|
|
127
|
+
Treat a section as a candidate if:
|
|
128
|
+
- it hits **2+ total points**, or
|
|
129
|
+
- it hits **1+ point** in a high-risk domain and the section is materially important
|
|
130
|
+
|
|
131
|
+
Choose only the top **2-5** sections by score. If the user explicitly asked to deepen a lightweight plan, cap at **1-2** sections unless the topic is high-risk.
|
|
132
|
+
|
|
133
|
+
Example:
|
|
134
|
+
- A `Key Technical Decisions` section with 1 checklist trigger and the critical-section bonus scores **2 points** and is a candidate
|
|
135
|
+
- A `Risks & Dependencies` section with 1 checklist trigger in a high-risk migration plan also becomes a candidate because the risk bonus applies
|
|
136
|
+
|
|
137
|
+
If the plan already has a `deepened:` date:
|
|
138
|
+
- Prefer sections that have not yet been substantially strengthened, if their scores are comparable
|
|
139
|
+
- Revisit an already-deepened section only when it still scores clearly higher than alternatives or the user explicitly asks for another pass on it
|
|
140
|
+
|
|
141
|
+
#### 2.1 Section Checklists
|
|
142
|
+
|
|
143
|
+
Use these triggers.
|
|
144
|
+
|
|
145
|
+
**Requirements Trace**
|
|
146
|
+
- Requirements are vague or disconnected from implementation units
|
|
147
|
+
- Success criteria are missing or not reflected downstream
|
|
148
|
+
- Units do not clearly advance the traced requirements
|
|
149
|
+
- Origin requirements are not clearly carried forward
|
|
150
|
+
|
|
151
|
+
**Context & Research / Sources & References**
|
|
152
|
+
- Relevant repo patterns are named but never used in decisions or implementation units
|
|
153
|
+
- Cited learnings or references do not materially shape the plan
|
|
154
|
+
- High-risk work lacks appropriate external or internal grounding
|
|
155
|
+
- Research is generic instead of tied to this repo or this plan
|
|
156
|
+
|
|
157
|
+
**Key Technical Decisions**
|
|
158
|
+
- A decision is stated without rationale
|
|
159
|
+
- Rationale does not explain tradeoffs or rejected alternatives
|
|
160
|
+
- The decision does not connect back to scope, requirements, or origin context
|
|
161
|
+
- An obvious design fork exists but the plan never addresses why one path won
|
|
162
|
+
|
|
163
|
+
**Open Questions**
|
|
164
|
+
- Product blockers are hidden as assumptions
|
|
165
|
+
- Planning-owned questions are incorrectly deferred to implementation
|
|
166
|
+
- Resolved questions have no clear basis in repo context, research, or origin decisions
|
|
167
|
+
- Deferred items are too vague to be useful later
|
|
168
|
+
|
|
169
|
+
**Implementation Units**
|
|
170
|
+
- Dependency order is unclear or likely wrong
|
|
171
|
+
- File paths or test file paths are missing where they should be explicit
|
|
172
|
+
- Units are too large, too vague, or broken into micro-steps
|
|
173
|
+
- Approach notes are thin or do not name the pattern to follow
|
|
174
|
+
- Test scenarios or verification outcomes are vague
|
|
175
|
+
|
|
176
|
+
**System-Wide Impact**
|
|
177
|
+
- Affected interfaces, callbacks, middleware, entry points, or parity surfaces are missing
|
|
178
|
+
- Failure propagation is underexplored
|
|
179
|
+
- State lifecycle, caching, or data integrity risks are absent where relevant
|
|
180
|
+
- Integration coverage is weak for cross-layer work
|
|
181
|
+
|
|
182
|
+
**Risks & Dependencies / Documentation / Operational Notes**
|
|
183
|
+
- Risks are listed without mitigation
|
|
184
|
+
- Rollout, monitoring, migration, or support implications are missing when warranted
|
|
185
|
+
- External dependency assumptions are weak or unstated
|
|
186
|
+
- Security, privacy, performance, or data risks are absent where they obviously apply
|
|
187
|
+
|
|
188
|
+
Use the plan's own `Context & Research` and `Sources & References` as evidence. If those sections cite a pattern, learning, or risk that never affects decisions, implementation units, or verification, treat that as a confidence gap.
|
|
189
|
+
|
|
190
|
+
### Phase 3: Select Targeted Research Agents
|
|
191
|
+
|
|
192
|
+
For each selected section, choose the smallest useful agent set. Do **not** run every agent. Use at most **1-3 agents per section** and usually no more than **8 agents total**.
|
|
193
|
+
|
|
194
|
+
Use fully-qualified agent names inside Task calls.
|
|
195
|
+
|
|
196
|
+
#### 3.1 Deterministic Section-to-Agent Mapping
|
|
197
|
+
|
|
198
|
+
**Requirements Trace / Open Questions classification**
|
|
199
|
+
- `systematic:workflow:spec-flow-analyzer` for missing user flows, edge cases, and handoff gaps
|
|
200
|
+
- `systematic:research:repo-research-analyst` for repo-grounded patterns, conventions, and implementation reality checks
|
|
201
|
+
|
|
202
|
+
**Context & Research / Sources & References gaps**
|
|
203
|
+
- `systematic:research:learnings-researcher` for institutional knowledge and past solved problems
|
|
204
|
+
- `systematic:research:framework-docs-researcher` for official framework or library behavior
|
|
205
|
+
- `systematic:research:best-practices-researcher` for current external patterns and industry guidance
|
|
206
|
+
- Add `systematic:research:git-history-analyzer` only when historical rationale or prior art is materially missing
|
|
207
|
+
|
|
208
|
+
**Key Technical Decisions**
|
|
209
|
+
- `systematic:review:architecture-strategist` for design integrity, boundaries, and architectural tradeoffs
|
|
210
|
+
- Add `systematic:research:framework-docs-researcher` or `systematic:research:best-practices-researcher` when the decision needs external grounding beyond repo evidence
|
|
211
|
+
|
|
212
|
+
**Implementation Units / Verification**
|
|
213
|
+
- `systematic:research:repo-research-analyst` for concrete file targets, patterns to follow, and repo-specific sequencing clues
|
|
214
|
+
- `systematic:review:pattern-recognition-specialist` for consistency, duplication risks, and alignment with existing patterns
|
|
215
|
+
- Add `systematic:workflow:spec-flow-analyzer` when sequencing depends on user flow or handoff completeness
|
|
216
|
+
|
|
217
|
+
**System-Wide Impact**
|
|
218
|
+
- `systematic:review:architecture-strategist` for cross-boundary effects, interface surfaces, and architectural knock-on impact
|
|
219
|
+
- Add the specific specialist that matches the risk:
|
|
220
|
+
- `systematic:review:performance-oracle` for scalability, latency, throughput, and resource-risk analysis
|
|
221
|
+
- `systematic:review:security-sentinel` for auth, validation, exploit surfaces, and security boundary review
|
|
222
|
+
- `systematic:review:data-integrity-guardian` for migrations, persistent state safety, consistency, and data lifecycle risks
|
|
223
|
+
|
|
224
|
+
**Risks & Dependencies / Operational Notes**
|
|
225
|
+
- Use the specialist that matches the actual risk:
|
|
226
|
+
- `systematic:review:security-sentinel` for security, auth, privacy, and exploit risk
|
|
227
|
+
- `systematic:review:data-integrity-guardian` for persistent data safety, constraints, and transaction boundaries
|
|
228
|
+
- `systematic:review:data-migration-expert` for migration realism, backfills, and production data transformation risk
|
|
229
|
+
- `systematic:review:deployment-verification-agent` for rollout checklists, rollback planning, and launch verification
|
|
230
|
+
- `systematic:review:performance-oracle` for capacity, latency, and scaling concerns
|
|
231
|
+
|
|
232
|
+
#### 3.2 Agent Prompt Shape
|
|
233
|
+
|
|
234
|
+
For each selected section, pass:
|
|
235
|
+
- A short plan summary
|
|
236
|
+
- The exact section text
|
|
237
|
+
- Why the section was selected, including which checklist triggers fired
|
|
238
|
+
- The plan depth and risk profile
|
|
239
|
+
- A specific question to answer
|
|
240
|
+
|
|
241
|
+
Instruct the agent to return:
|
|
242
|
+
- findings that change planning quality
|
|
243
|
+
- stronger rationale, sequencing, verification, risk treatment, or references
|
|
244
|
+
- no implementation code
|
|
245
|
+
- no shell commands
|
|
246
|
+
|
|
247
|
+
### Phase 4: Run Targeted Research and Review
|
|
248
|
+
|
|
249
|
+
Launch the selected agents in parallel.
|
|
250
|
+
|
|
251
|
+
Prefer local repo and institutional evidence first. Use external research only when the gap cannot be closed responsibly from repo context or already-cited sources.
|
|
252
|
+
|
|
253
|
+
If a selected section can be improved by reading the origin document more carefully, do that before dispatching external agents.
|
|
254
|
+
|
|
255
|
+
If agent outputs conflict:
|
|
256
|
+
- Prefer repo-grounded and origin-grounded evidence over generic advice
|
|
257
|
+
- Prefer official framework documentation over secondary best-practice summaries when the conflict is about library behavior
|
|
258
|
+
- If a real tradeoff remains, record it explicitly in the plan rather than pretending the conflict does not exist
|
|
259
|
+
|
|
260
|
+
### Phase 5: Synthesize and Rewrite the Plan
|
|
261
|
+
|
|
262
|
+
Strengthen only the selected sections. Keep the plan coherent and preserve its overall structure.
|
|
263
|
+
|
|
264
|
+
Allowed changes:
|
|
265
|
+
- Clarify or strengthen decision rationale
|
|
266
|
+
- Tighten requirements trace or origin fidelity
|
|
267
|
+
- Reorder or split implementation units when sequencing is weak
|
|
268
|
+
- Add missing pattern references, file/test paths, or verification outcomes
|
|
269
|
+
- Expand system-wide impact, risks, or rollout treatment where justified
|
|
270
|
+
- Reclassify open questions between `Resolved During Planning` and `Deferred to Implementation` when evidence supports the change
|
|
271
|
+
- Add an optional deep-plan section only when it materially improves execution quality
|
|
272
|
+
- Add or update `deepened: YYYY-MM-DD` in frontmatter when the plan was substantively improved
|
|
273
|
+
|
|
274
|
+
Do **not**:
|
|
275
|
+
- Add fenced implementation code blocks unless the plan itself is about code shape as a design artifact
|
|
276
|
+
- Add git commands, commit choreography, or exact test command recipes
|
|
277
|
+
- Add generic `Research Insights` subsections everywhere
|
|
278
|
+
- Rewrite the entire plan from scratch
|
|
279
|
+
- Invent new product requirements, scope changes, or success criteria without surfacing them explicitly
|
|
280
|
+
|
|
281
|
+
If research reveals a product-level ambiguity that should change behavior or scope:
|
|
282
|
+
- Do not silently decide it here
|
|
283
|
+
- Record it under `Open Questions`
|
|
284
|
+
- Recommend `ce:brainstorm` if the gap is truly product-defining
|
|
285
|
+
|
|
286
|
+
### Phase 6: Final Checks and Write the File
|
|
287
|
+
|
|
288
|
+
Before writing:
|
|
289
|
+
- Confirm the plan is stronger in specific ways, not merely longer
|
|
290
|
+
- Confirm the planning boundary is intact
|
|
291
|
+
- Confirm the selected sections were actually the weakest ones
|
|
292
|
+
- Confirm origin decisions were preserved when an origin document exists
|
|
293
|
+
- Confirm the final plan still feels right-sized for its depth
|
|
294
|
+
|
|
295
|
+
Update the plan file in place by default.
|
|
296
|
+
|
|
297
|
+
If the user explicitly requests a separate file, append `-deepened` before `.md`, for example:
|
|
298
|
+
- `docs/plans/2026-03-15-001-feat-example-plan-deepened.md`
|
|
299
|
+
|
|
300
|
+
## Post-Enhancement Options
|
|
301
|
+
|
|
302
|
+
If substantive changes were made, present next steps using the platform's blocking question tool when available (see Interaction Method). Otherwise, present numbered options in chat and wait for the user's reply before proceeding.
|
|
303
|
+
|
|
304
|
+
**Question:** "Plan deepened at `[plan_path]`. What would you like to do next?"
|
|
305
|
+
|
|
306
|
+
**Options:**
|
|
307
|
+
1. **View diff** - Show what changed
|
|
308
|
+
2. **Run `document-review` skill** - Improve the updated plan through structured document review
|
|
309
|
+
3. **Start `ce:work` skill** - Begin implementing the plan
|
|
310
|
+
4. **Deepen specific sections further** - Run another targeted deepening pass on named sections
|
|
311
|
+
|
|
312
|
+
Based on selection:
|
|
313
|
+
- **View diff** -> Show the important additions and changed sections
|
|
314
|
+
- **`document-review` skill** -> Load the `document-review` skill with the plan path
|
|
315
|
+
- **Start `ce:work` skill** -> Call the `ce:work` skill with the plan path
|
|
316
|
+
- **Deepen specific sections further** -> Ask which sections still feel weak and run another targeted pass only for those sections
|
|
317
|
+
|
|
318
|
+
If no substantive changes were warranted:
|
|
319
|
+
- Say that the plan already appears sufficiently grounded
|
|
320
|
+
- Offer the `document-review` skill or `/systematic:ce-work` as the next step instead
|
|
321
|
+
|
|
322
|
+
NEVER CODE! Research, challenge, and strengthen the plan.
|
|
323
|
+
|
|
@@ -1,17 +1,17 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: document-review
|
|
3
|
-
description: This skill should be used to refine
|
|
3
|
+
description: This skill should be used to refine requirements or plan documents before proceeding to the next workflow step. It applies when a requirements document or plan document exists and the user wants to improve it.
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
# Document Review
|
|
7
7
|
|
|
8
|
-
Improve
|
|
8
|
+
Improve requirements or plan documents through structured review.
|
|
9
9
|
|
|
10
10
|
## Step 1: Get the Document
|
|
11
11
|
|
|
12
12
|
**If a document path is provided:** Read it, then proceed to Step 2.
|
|
13
13
|
|
|
14
|
-
**If no document is specified:** Ask which document to review, or look for the most recent
|
|
14
|
+
**If no document is specified:** Ask which document to review, or look for the most recent requirements/plan in `docs/brainstorms/` or `docs/plans/`.
|
|
15
15
|
|
|
16
16
|
## Step 2: Assess
|
|
17
17
|
|
|
@@ -32,11 +32,12 @@ Score the document against these criteria:
|
|
|
32
32
|
| Criterion | What to Check |
|
|
33
33
|
|-----------|---------------|
|
|
34
34
|
| **Clarity** | Problem statement is clear, no vague language ("probably," "consider," "try to") |
|
|
35
|
-
| **Completeness** | Required sections present, constraints stated,
|
|
36
|
-
| **Specificity** | Concrete enough for next step (
|
|
37
|
-
| **
|
|
35
|
+
| **Completeness** | Required sections present, constraints stated, and outstanding questions clearly marked as blocking or deferred |
|
|
36
|
+
| **Specificity** | Concrete enough for next step (requirements → can plan, plan → can implement) |
|
|
37
|
+
| **Appropriate Level** | Requirements doc stays at behavior/scope level and does not drift into implementation unless the document is inherently technical |
|
|
38
|
+
| **YAGNI** | Avoid speculative complexity whose carrying cost outweighs its value; keep low-cost, meaningful polish when it is easy to maintain |
|
|
38
39
|
|
|
39
|
-
If invoked within a workflow (after `/
|
|
40
|
+
If invoked within a workflow (after `/ce:brainstorm` or `/ce:plan`), also check:
|
|
40
41
|
- **User intent fidelity** — Document reflects what was discussed, assumptions validated
|
|
41
42
|
|
|
42
43
|
## Step 4: Identify the Critical Improvement
|
|
@@ -56,7 +57,7 @@ Present your findings, then:
|
|
|
56
57
|
Simplification is purposeful removal of unnecessary complexity, not shortening for its own sake.
|
|
57
58
|
|
|
58
59
|
**Simplify when:**
|
|
59
|
-
- Content serves hypothetical future needs
|
|
60
|
+
- Content serves hypothetical future needs without enough current value to justify its carrying cost
|
|
60
61
|
- Sections repeat information already covered elsewhere
|
|
61
62
|
- Detail exceeds what's needed to take the next step
|
|
62
63
|
- Abstractions or structure add overhead without clarity
|
|
@@ -65,6 +66,10 @@ Simplification is purposeful removal of unnecessary complexity, not shortening f
|
|
|
65
66
|
- Constraints or edge cases that affect implementation
|
|
66
67
|
- Rationale that explains why alternatives were rejected
|
|
67
68
|
- Open questions that need resolution
|
|
69
|
+
- Deferred technical or research questions that are intentionally carried forward to the next stage
|
|
70
|
+
|
|
71
|
+
**Also remove when inappropriate:**
|
|
72
|
+
- Library choices, file structures, endpoints, schemas, or other implementation details that do not belong in a non-technical requirements document
|
|
68
73
|
|
|
69
74
|
## Step 6: Offer Next Action
|
|
70
75
|
|
|
@@ -85,3 +90,4 @@ Return control to the caller (workflow or user) after selection.
|
|
|
85
90
|
- Do not add new sections or requirements the user didn't discuss
|
|
86
91
|
- Do not over-engineer or add complexity
|
|
87
92
|
- Do not create separate review files or add metadata sections
|
|
93
|
+
|
|
@@ -22,7 +22,7 @@ Create a new skill in `.opencode/skills/` for the requested task.
|
|
|
22
22
|
|
|
23
23
|
**Development:**
|
|
24
24
|
- Bash - run commands (git, tests, linters)
|
|
25
|
-
-
|
|
25
|
+
- Task - launch specialized agents for complex tasks
|
|
26
26
|
- todowrite - track progress with todo lists
|
|
27
27
|
|
|
28
28
|
**Web & APIs:**
|
|
@@ -114,7 +114,7 @@ Implement #$ARGUMENTS following these steps:
|
|
|
114
114
|
3. Implement
|
|
115
115
|
- Follow existing code patterns (reference specific files)
|
|
116
116
|
- Write tests first if doing TDD
|
|
117
|
-
- Ensure code follows
|
|
117
|
+
- Ensure code follows CLAUDE.md conventions
|
|
118
118
|
|
|
119
119
|
4. Verify
|
|
120
120
|
- Run tests: `bin/rails test`
|
|
@@ -161,3 +161,4 @@ Details about what to do.
|
|
|
161
161
|
- [ ] Expected outcome 1
|
|
162
162
|
- [ ] Expected outcome 2
|
|
163
163
|
```
|
|
164
|
+
|
package/skills/lfg/SKILL.md
CHANGED
|
@@ -5,23 +5,25 @@ argument-hint: '[feature description]'
|
|
|
5
5
|
disable-model-invocation: true
|
|
6
6
|
---
|
|
7
7
|
|
|
8
|
-
CRITICAL: You MUST execute every step below IN ORDER. Do NOT skip any step. Do NOT jump ahead to coding or implementation. The plan phase (
|
|
8
|
+
CRITICAL: You MUST execute every step below IN ORDER. Do NOT skip any required step. Do NOT jump ahead to coding or implementation. The plan phase (step 2, and step 3 when warranted) MUST be completed and verified BEFORE any work begins. Violating this order produces bad output.
|
|
9
9
|
|
|
10
10
|
1. **Optional:** If the `ralph-wiggum` skill is available, run `/ralph-wiggum:ralph-loop "finish all slash commands" --completion-promise "DONE"`. If not available or it fails, skip and continue to step 2 immediately.
|
|
11
11
|
|
|
12
|
-
2. `/
|
|
12
|
+
2. `/systematic:ce-plan $ARGUMENTS`
|
|
13
13
|
|
|
14
|
-
GATE: STOP. Verify that
|
|
14
|
+
GATE: STOP. Verify that the `ce:plan` workflow produced a plan file in `docs/plans/`. If no plan file was created, run `/systematic:ce-plan $ARGUMENTS` again. Do NOT proceed to step 3 until a written plan exists.
|
|
15
15
|
|
|
16
|
-
3. `/systematic:deepen-plan`
|
|
16
|
+
3. **Conditionally** run `/systematic:deepen-plan`
|
|
17
17
|
|
|
18
|
-
|
|
18
|
+
Run the `deepen-plan` workflow only if the plan is `Standard` or `Deep`, touches a high-risk area (auth, security, payments, migrations, external APIs, significant rollout concerns), or still has obvious confidence gaps in decisions, sequencing, system-wide impact, risks, or verification.
|
|
19
19
|
|
|
20
|
-
|
|
20
|
+
GATE: STOP. If you ran the `deepen-plan` workflow, confirm the plan was deepened or explicitly judged sufficiently grounded. If you skipped it, briefly note why and proceed to step 4.
|
|
21
|
+
|
|
22
|
+
4. `/systematic:ce-work`
|
|
21
23
|
|
|
22
24
|
GATE: STOP. Verify that implementation work was performed - files were created or modified beyond the plan. Do NOT proceed to step 5 if no code changes were made.
|
|
23
25
|
|
|
24
|
-
5. `/
|
|
26
|
+
5. `/systematic:ce-review`
|
|
25
27
|
|
|
26
28
|
6. `/systematic:resolve_todo_parallel`
|
|
27
29
|
|
|
@@ -32,3 +34,4 @@ CRITICAL: You MUST execute every step below IN ORDER. Do NOT skip any step. Do N
|
|
|
32
34
|
9. Output `<promise>DONE</promise>` when video is in PR
|
|
33
35
|
|
|
34
36
|
Start with step 2 now (or step 1 if ralph-wiggum is available). Remember: plan FIRST, then work. Never skip the plan.
|
|
37
|
+
|
|
@@ -1,13 +1,13 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: report-bug
|
|
3
|
-
description: Report a bug in the
|
|
3
|
+
description: Report a bug in the compound-engineering plugin
|
|
4
4
|
argument-hint: '[optional: brief description of the bug]'
|
|
5
5
|
disable-model-invocation: true
|
|
6
6
|
---
|
|
7
7
|
|
|
8
|
-
# Report a
|
|
8
|
+
# Report a Compounding Engineering Plugin Bug
|
|
9
9
|
|
|
10
|
-
Report bugs encountered while using the
|
|
10
|
+
Report bugs encountered while using the compound-engineering plugin. This command gathers structured information and creates a GitHub issue for the maintainer.
|
|
11
11
|
|
|
12
12
|
## Step 1: Gather Bug Information
|
|
13
13
|
|
|
@@ -42,10 +42,10 @@ Use the question tool to collect the following information:
|
|
|
42
42
|
Automatically gather:
|
|
43
43
|
```bash
|
|
44
44
|
# Get plugin version
|
|
45
|
-
cat ~/.config/opencode/plugins/installed_plugins.json 2>/dev/null | grep -A5 "
|
|
45
|
+
cat ~/.config/opencode/plugins/installed_plugins.json 2>/dev/null | grep -A5 "compound-engineering" | head -10 || echo "Plugin info not found"
|
|
46
46
|
|
|
47
47
|
# Get OpenCode version
|
|
48
|
-
|
|
48
|
+
claude --version 2>/dev/null || echo "Claude CLI version unknown"
|
|
49
49
|
|
|
50
50
|
# Get OS info
|
|
51
51
|
uname -a
|
|
@@ -64,7 +64,7 @@ Create a well-structured bug report with:
|
|
|
64
64
|
## Environment
|
|
65
65
|
|
|
66
66
|
- **Plugin Version:** [from installed_plugins.json]
|
|
67
|
-
- **OpenCode Version:** [from
|
|
67
|
+
- **OpenCode Version:** [from claude --version]
|
|
68
68
|
- **OS:** [from uname]
|
|
69
69
|
|
|
70
70
|
## What Happened
|
|
@@ -101,17 +101,17 @@ Use the GitHub CLI to create the issue:
|
|
|
101
101
|
|
|
102
102
|
```bash
|
|
103
103
|
gh issue create \
|
|
104
|
-
--repo
|
|
105
|
-
--title "[
|
|
104
|
+
--repo EveryInc/compound-engineering-plugin \
|
|
105
|
+
--title "[compound-engineering] Bug: [Brief description]" \
|
|
106
106
|
--body "[Formatted bug report from Step 3]" \
|
|
107
|
-
--label "bug,
|
|
107
|
+
--label "bug,compound-engineering"
|
|
108
108
|
```
|
|
109
109
|
|
|
110
110
|
**Note:** If labels don't exist, create without labels:
|
|
111
111
|
```bash
|
|
112
112
|
gh issue create \
|
|
113
|
-
--repo
|
|
114
|
-
--title "[
|
|
113
|
+
--repo EveryInc/compound-engineering-plugin \
|
|
114
|
+
--title "[compound-engineering] Bug: [Brief description]" \
|
|
115
115
|
--body "[Formatted bug report]"
|
|
116
116
|
```
|
|
117
117
|
|
|
@@ -127,10 +127,10 @@ After the issue is created:
|
|
|
127
127
|
```
|
|
128
128
|
✅ Bug report submitted successfully!
|
|
129
129
|
|
|
130
|
-
Issue: https://github.com/
|
|
131
|
-
Title: [
|
|
130
|
+
Issue: https://github.com/EveryInc/compound-engineering-plugin/issues/[NUMBER]
|
|
131
|
+
Title: [compound-engineering] Bug: [description]
|
|
132
132
|
|
|
133
|
-
Thank you for helping improve the
|
|
133
|
+
Thank you for helping improve the compound-engineering plugin!
|
|
134
134
|
The maintainer will review your report and respond as soon as possible.
|
|
135
135
|
```
|
|
136
136
|
|
|
@@ -149,3 +149,4 @@ This command does NOT collect:
|
|
|
149
149
|
- File paths beyond basic OS info
|
|
150
150
|
|
|
151
151
|
Only technical information about the bug is included in the report.
|
|
152
|
+
|
|
@@ -27,9 +27,10 @@ So if there are 3 comments, it will spawn 3 pr-comment-resolver agents in parall
|
|
|
27
27
|
2. task pr-comment-resolver(comment2)
|
|
28
28
|
3. task pr-comment-resolver(comment3)
|
|
29
29
|
|
|
30
|
-
Always run all in parallel subagents/
|
|
30
|
+
Always run all in parallel subagents/Tasks for each Todo item.
|
|
31
31
|
|
|
32
32
|
### 4. Commit & Resolve
|
|
33
33
|
|
|
34
34
|
- Commit changes
|
|
35
35
|
- Push to remote
|
|
36
|
+
|
|
@@ -12,7 +12,7 @@ Resolve all TODO comments using parallel processing.
|
|
|
12
12
|
|
|
13
13
|
Get all unresolved TODOs from the /todos/\*.md directory
|
|
14
14
|
|
|
15
|
-
If any todo recommends deleting, removing, or gitignoring files in `docs/plans
|
|
15
|
+
If any todo recommends deleting, removing, or gitignoring files in `docs/brainstorms/`, `docs/plans/`, or `docs/solutions/`, skip it and mark it as `wont_fix`. These are compound-engineering pipeline artifacts that are intentional and permanent.
|
|
16
16
|
|
|
17
17
|
### 2. Plan
|
|
18
18
|
|
package/skills/slfg/SKILL.md
CHANGED
|
@@ -10,15 +10,18 @@ Swarm-enabled LFG. Run these steps in order, parallelizing where indicated. Do n
|
|
|
10
10
|
## Sequential Phase
|
|
11
11
|
|
|
12
12
|
1. **Optional:** If the `ralph-wiggum` skill is available, run `/ralph-wiggum:ralph-loop "finish all slash commands" --completion-promise "DONE"`. If not available or it fails, skip and continue to step 2 immediately.
|
|
13
|
-
2. `/ce
|
|
14
|
-
3. `/systematic:deepen-plan`
|
|
15
|
-
|
|
13
|
+
2. `/systematic:ce-plan $ARGUMENTS`
|
|
14
|
+
3. **Conditionally** run `/systematic:deepen-plan`
|
|
15
|
+
- Run the `deepen-plan` workflow only if the plan is `Standard` or `Deep`, touches a high-risk area (auth, security, payments, migrations, external APIs, significant rollout concerns), or still has obvious confidence gaps in decisions, sequencing, system-wide impact, risks, or verification
|
|
16
|
+
- If you run the `deepen-plan` workflow, confirm the plan was deepened or explicitly judged sufficiently grounded before moving on
|
|
17
|
+
- If you skip it, note why and continue to step 4
|
|
18
|
+
4. `/systematic:ce-work` — **Use swarm mode**: Make a Task list and launch an army of agent swarm subagents to build the plan
|
|
16
19
|
|
|
17
20
|
## Parallel Phase
|
|
18
21
|
|
|
19
22
|
After work completes, launch steps 5 and 6 as **parallel swarm agents** (both only need code to be written):
|
|
20
23
|
|
|
21
|
-
5. `/ce
|
|
24
|
+
5. `/systematic:ce-review` — spawn as background Task agent
|
|
22
25
|
6. `/systematic:test-browser` — spawn as background Task agent
|
|
23
26
|
|
|
24
27
|
Wait for both to complete before continuing.
|
|
@@ -133,8 +133,8 @@ If the user passed a port number (e.g., `/test-browser 5000` or `/test-browser -
|
|
|
133
133
|
|
|
134
134
|
**Priority 2: AGENTS.md / project instructions**
|
|
135
135
|
```bash
|
|
136
|
-
# Check
|
|
137
|
-
grep -Eio '(port\s*[:=]\s*|localhost:)([0-9]{4,5})'
|
|
136
|
+
# Check CLAUDE.md for port references
|
|
137
|
+
grep -Eio '(port\s*[:=]\s*|localhost:)([0-9]{4,5})' CLAUDE.md 2>/dev/null | grep -Eo '[0-9]{4,5}' | head -1
|
|
138
138
|
```
|
|
139
139
|
|
|
140
140
|
**Priority 3: package.json scripts**
|
|
@@ -158,7 +158,7 @@ Store the result in a `PORT` variable for use in all subsequent steps.
|
|
|
158
158
|
# Combined detection (run this)
|
|
159
159
|
PORT="${EXPLICIT_PORT:-}"
|
|
160
160
|
if [ -z "$PORT" ]; then
|
|
161
|
-
PORT=$(grep -Eio '(port\s*[:=]\s*|localhost:)([0-9]{4,5})'
|
|
161
|
+
PORT=$(grep -Eio '(port\s*[:=]\s*|localhost:)([0-9]{4,5})' CLAUDE.md 2>/dev/null | grep -Eo '[0-9]{4,5}' | head -1)
|
|
162
162
|
fi
|
|
163
163
|
if [ -z "$PORT" ]; then
|
|
164
164
|
PORT=$(grep -Eo '\-\-port[= ]+[0-9]{4,5}' package.json 2>/dev/null | grep -Eo '[0-9]{4,5}' | head -1)
|
|
@@ -323,9 +323,9 @@ mcp__xcodebuildmcp__shutdown_simulator({ simulator_id: "[uuid]" })
|
|
|
323
323
|
/xcode-test current
|
|
324
324
|
```
|
|
325
325
|
|
|
326
|
-
## Integration with /ce
|
|
326
|
+
## Integration with /systematic:ce-review
|
|
327
327
|
|
|
328
|
-
When reviewing PRs that touch iOS code, the `/ce
|
|
328
|
+
When reviewing PRs that touch iOS code, the `/systematic:ce-review` command can spawn this as a subagent:
|
|
329
329
|
|
|
330
330
|
```
|
|
331
331
|
Task general-purpose("Run /xcode-test for scheme [name]. Build, install on simulator, test key screens, check for crashes.")
|