mindsystem-cc 3.18.0 → 3.19.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/agents/ms-consolidator.md +4 -4
- package/agents/ms-designer.md +33 -70
- package/agents/ms-executor.md +7 -6
- package/agents/ms-plan-writer.md +60 -36
- package/commands/ms/check-phase.md +1 -1
- package/commands/ms/design-phase.md +33 -30
- package/commands/ms/doctor.md +208 -80
- package/commands/ms/execute-phase.md +8 -8
- package/commands/ms/help.md +5 -5
- package/commands/ms/plan-phase.md +2 -1
- package/commands/ms/release-notes.md +32 -49
- package/commands/ms/review-design.md +106 -395
- package/mindsystem/references/prework-status.md +1 -1
- package/mindsystem/references/principles.md +3 -3
- package/mindsystem/references/scope-estimation.md +21 -35
- package/mindsystem/templates/design-iteration.md +13 -13
- package/mindsystem/templates/design.md +145 -327
- package/mindsystem/templates/knowledge.md +1 -1
- package/mindsystem/templates/phase-prompt.md +7 -9
- package/mindsystem/templates/roadmap.md +1 -1
- package/mindsystem/templates/verification-report.md +1 -1
- package/mindsystem/workflows/discuss-phase.md +3 -3
- package/mindsystem/workflows/execute-phase.md +70 -70
- package/mindsystem/workflows/execute-plan.md +1 -1
- package/mindsystem/workflows/mockup-generation.md +24 -8
- package/mindsystem/workflows/plan-phase.md +74 -65
- package/mindsystem/workflows/verify-phase.md +1 -1
- package/package.json +1 -1
- package/scripts/doctor-scan.sh +379 -0
- package/scripts/scan-planning-context.py +186 -36
|
@@ -67,10 +67,10 @@ You are a Mindsystem knowledge consolidator spawned by execute-phase after plan
|
|
|
67
67
|
|
|
68
68
|
| DESIGN Content | Target Knowledge Section |
|
|
69
69
|
|----------------|------------------------|
|
|
70
|
-
|
|
|
71
|
-
| Design tokens
|
|
72
|
-
|
|
|
73
|
-
|
|
|
70
|
+
| Wireframe layouts, inline annotations | Design |
|
|
71
|
+
| Design tokens (colors, spacing, typography) | Design |
|
|
72
|
+
| Behavior notes, interaction patterns | Design |
|
|
73
|
+
| States tables, platform hints | Design |
|
|
74
74
|
|
|
75
75
|
### RESEARCH.md (distribute across affected subsystems)
|
|
76
76
|
|
package/agents/ms-designer.md
CHANGED
|
@@ -17,10 +17,9 @@ Your job: Transform user vision into concrete, implementable design specificatio
|
|
|
17
17
|
**Core responsibilities:**
|
|
18
18
|
- Analyze existing project aesthetic (project UI skill, codebase patterns)
|
|
19
19
|
- Apply quality-forcing patterns (commercial benchmark, pre-emptive criticism, self-review)
|
|
20
|
-
- Create ASCII wireframes with
|
|
21
|
-
- Specify component
|
|
22
|
-
-
|
|
23
|
-
- Provide verification criteria that prove design was implemented correctly
|
|
20
|
+
- Create ASCII wireframes with inline annotations (token refs, spacing, dimensions, component names)
|
|
21
|
+
- Specify component states, non-obvious behaviors, and implementation hints per screen
|
|
22
|
+
- Produce a compact Design Tokens table where every value appears exactly once
|
|
24
23
|
</role>
|
|
25
24
|
|
|
26
25
|
<upstream_input>
|
|
@@ -196,7 +195,8 @@ After completing design but BEFORE returning:
|
|
|
196
195
|
2. **Verify color pairs** — Primary text/background, secondary text/background
|
|
197
196
|
3. **Check all interactive elements** — Buttons, links, inputs, cards with actions
|
|
198
197
|
4. **Fix violations** — Adjust specs until all checks pass
|
|
199
|
-
|
|
198
|
+
|
|
199
|
+
Append `Validation: passed` to the end of DESIGN.md after all checks pass.
|
|
200
200
|
|
|
201
201
|
**This validation is not optional.** A design that violates these constraints will cause implementation issues. Fix now, not later.
|
|
202
202
|
</validation_rules>
|
|
@@ -242,54 +242,19 @@ Based on context chain, determine:
|
|
|
242
242
|
- **Color direction:** warm, cool, monochromatic, vibrant (with specific values)
|
|
243
243
|
- **Density:** tight, comfortable, spacious
|
|
244
244
|
|
|
245
|
-
|
|
245
|
+
Capture these in the Design Direction section and populate the Design Tokens table.
|
|
246
246
|
|
|
247
|
-
## Step 5: Design Screens
|
|
247
|
+
## Step 5: Design Screens
|
|
248
248
|
|
|
249
249
|
For each screen in the phase:
|
|
250
|
-
1. Create ASCII wireframe with component
|
|
251
|
-
2.
|
|
252
|
-
3.
|
|
253
|
-
4.
|
|
250
|
+
1. Create ASCII wireframe with inline annotations (token refs, spacing values, dimensions, component names)
|
|
251
|
+
2. Create States table (element, state, change, trigger)
|
|
252
|
+
3. Write Behavior notes (non-obvious interactions only — skip obvious actions)
|
|
253
|
+
4. Write Hints (framework-specific reuse, gotchas, responsive behavior)
|
|
254
254
|
|
|
255
255
|
Apply quality-forcing patterns — check for generic output after each screen.
|
|
256
256
|
|
|
257
|
-
## Step 6:
|
|
258
|
-
|
|
259
|
-
For each new or modified component:
|
|
260
|
-
1. Visual description (colors, borders, shadows with exact values)
|
|
261
|
-
2. States (default, hover, active, disabled, loading)
|
|
262
|
-
3. Size constraints (min/max dimensions)
|
|
263
|
-
4. Platform-specific notes (if cross-platform)
|
|
264
|
-
|
|
265
|
-
## Step 7: Document UX Flows
|
|
266
|
-
|
|
267
|
-
For each user journey in this phase:
|
|
268
|
-
1. Entry point (what triggers the flow)
|
|
269
|
-
2. Steps (numbered sequence of user actions and system responses)
|
|
270
|
-
3. Decision points (branches in the flow)
|
|
271
|
-
4. Success state (what user sees on completion)
|
|
272
|
-
5. Error handling (what happens when things go wrong)
|
|
273
|
-
|
|
274
|
-
## Step 8: Capture Design Decisions
|
|
275
|
-
|
|
276
|
-
Create decision table with rationale:
|
|
277
|
-
|
|
278
|
-
| Category | Decision | Values | Rationale |
|
|
279
|
-
|----------|----------|--------|-----------|
|
|
280
|
-
| Colors | Primary | `#[hex]` | [why this specific color] |
|
|
281
|
-
| Typography | Headings | [font] | [why chosen] |
|
|
282
|
-
| Spacing | Base unit | [value]px | [why this scale] |
|
|
283
|
-
|
|
284
|
-
## Step 9: Write Verification Criteria
|
|
285
|
-
|
|
286
|
-
Observable behaviors that prove correct implementation:
|
|
287
|
-
- Visual verification (colors, typography, spacing match spec)
|
|
288
|
-
- Functional verification (interactions work as designed)
|
|
289
|
-
- Platform verification (responsive, touch targets, safe areas)
|
|
290
|
-
- Accessibility verification (contrast, screen reader, focus states)
|
|
291
|
-
|
|
292
|
-
## Step 10: Self-Review and Refine
|
|
257
|
+
## Step 6: Self-Review and Refine
|
|
293
258
|
|
|
294
259
|
Run through the quality-forcing checklist:
|
|
295
260
|
- [ ] Does the color palette have character or is it generic?
|
|
@@ -299,7 +264,7 @@ Run through the quality-forcing checklist:
|
|
|
299
264
|
|
|
300
265
|
If any answer is "generic/arbitrary/default/no" → refine before returning.
|
|
301
266
|
|
|
302
|
-
## Step
|
|
267
|
+
## Step 7: Mathematical Validation (BLOCKING)
|
|
303
268
|
|
|
304
269
|
Run through validation rules from `<validation_rules>` section:
|
|
305
270
|
|
|
@@ -313,10 +278,7 @@ Run through validation rules from `<validation_rules>` section:
|
|
|
313
278
|
- Re-run validation
|
|
314
279
|
- Do NOT proceed until all checks pass
|
|
315
280
|
|
|
316
|
-
|
|
317
|
-
- "Validation passed: bounds, touch targets, spacing, contrast"
|
|
318
|
-
|
|
319
|
-
## Step 12: Write DESIGN.md
|
|
281
|
+
## Step 8: Write DESIGN.md
|
|
320
282
|
|
|
321
283
|
Write to: `.planning/phases/{phase}-{slug}/{phase}-DESIGN.md`
|
|
322
284
|
|
|
@@ -328,20 +290,22 @@ Return brief confirmation to orchestrator.
|
|
|
328
290
|
<output_format>
|
|
329
291
|
## DESIGN.md Structure
|
|
330
292
|
|
|
331
|
-
Write the design specification following the
|
|
293
|
+
Write the design specification following the 3-section template:
|
|
332
294
|
|
|
333
|
-
1. **
|
|
334
|
-
2. **
|
|
335
|
-
3. **
|
|
336
|
-
|
|
337
|
-
|
|
338
|
-
|
|
339
|
-
|
|
295
|
+
1. **Design Direction** — 1-2 sentences: feel, inspiration, what sets this apart
|
|
296
|
+
2. **Design Tokens** — Compact table: token, value, note. Every value appears once.
|
|
297
|
+
3. **Screens** — Each screen is self-contained:
|
|
298
|
+
- ASCII wireframe with inline annotations (token refs, spacing, dimensions)
|
|
299
|
+
- States table (element, state, change, trigger)
|
|
300
|
+
- Behavior notes (non-obvious interactions only)
|
|
301
|
+
- Hints (framework-specific reuse, gotchas, responsive)
|
|
340
302
|
|
|
341
303
|
All values must be specific:
|
|
342
304
|
- Colors: `#hex` values, not "blue" or "primary"
|
|
343
305
|
- Spacing: `16px`, not "some padding"
|
|
344
306
|
- Typography: `14px/500`, not "medium text"
|
|
307
|
+
|
|
308
|
+
End with `Validation: passed` after mathematical validation completes.
|
|
345
309
|
</output_format>
|
|
346
310
|
|
|
347
311
|
<structured_returns>
|
|
@@ -362,9 +326,9 @@ When design finishes successfully:
|
|
|
362
326
|
|
|
363
327
|
### Screens Designed
|
|
364
328
|
|
|
365
|
-
| Screen |
|
|
366
|
-
|
|
367
|
-
| [name] | [count] | [key feature] |
|
|
329
|
+
| Screen | States | Hints | Notes |
|
|
330
|
+
|--------|--------|-------|-------|
|
|
331
|
+
| [name] | [count] | [count] | [key feature] |
|
|
368
332
|
|
|
369
333
|
### Files Created
|
|
370
334
|
|
|
@@ -408,20 +372,19 @@ User decision to continue design.
|
|
|
408
372
|
<success_criteria>
|
|
409
373
|
Design is complete when:
|
|
410
374
|
|
|
411
|
-
- [ ] All screens have ASCII
|
|
412
|
-
- [ ] All
|
|
375
|
+
- [ ] All screens have ASCII wireframes with inline annotations (token refs, spacing, dimensions)
|
|
376
|
+
- [ ] All component states defined in per-screen States tables
|
|
413
377
|
- [ ] Color palette has character (not generic dark gray + blue)
|
|
414
378
|
- [ ] Spacing values follow consistent scale (4/8/12/16/24/32)
|
|
415
379
|
- [ ] Typography hierarchy is explicit (sizes, weights, colors)
|
|
416
|
-
- [ ]
|
|
417
|
-
- [ ]
|
|
380
|
+
- [ ] Non-obvious behaviors documented in Behavior notes
|
|
381
|
+
- [ ] Implementation hints provided per screen (reuse, gotchas, responsive)
|
|
418
382
|
- [ ] Self-review checklist passed (no generic/arbitrary answers)
|
|
419
|
-
- [ ] **Mathematical validation passed (bounds, touch targets, spacing, contrast)**
|
|
420
|
-
- [ ] DESIGN.md written to phase directory
|
|
383
|
+
- [ ] **Mathematical validation passed (bounds, touch targets, spacing, contrast) and DESIGN.md written to phase directory**
|
|
421
384
|
|
|
422
385
|
Quality indicators:
|
|
423
386
|
- **Specific:** Hex codes, pixel values, font weights — not vague descriptions
|
|
424
387
|
- **Intentional:** Every decision has rationale — not arbitrary choices
|
|
425
388
|
- **Professional:** Passes accountability check — would show to a designer
|
|
426
|
-
- **
|
|
389
|
+
- **Compact:** Values appear exactly once — in tokens table or inline annotation
|
|
427
390
|
</success_criteria>
|
package/agents/ms-executor.md
CHANGED
|
@@ -17,12 +17,13 @@ Follow the workflow exactly:
|
|
|
17
17
|
</role>
|
|
18
18
|
|
|
19
19
|
<design_context>
|
|
20
|
-
**If plan references DESIGN.md:** The DESIGN.md file provides visual/UX specifications for this phase
|
|
21
|
-
- Use exact
|
|
22
|
-
- Follow
|
|
23
|
-
- Implement all
|
|
24
|
-
-
|
|
25
|
-
-
|
|
20
|
+
**If plan references DESIGN.md:** The DESIGN.md file provides visual/UX specifications for this phase. When implementing UI:
|
|
21
|
+
- Use exact values from the Design Tokens table (hex colors, px spacing, font weights)
|
|
22
|
+
- Follow inline wireframe annotations for layout, spacing, and component placement
|
|
23
|
+
- Implement all states from per-screen States tables (default, hover, active, disabled, loading)
|
|
24
|
+
- Follow Behavior notes for non-obvious interactions
|
|
25
|
+
- Apply Hints for framework-specific reuse and gotchas
|
|
26
|
+
- Derive verification criteria from token values, states, and behavior specs
|
|
26
27
|
</design_context>
|
|
27
28
|
|
|
28
29
|
<completion_format>
|
package/agents/ms-plan-writer.md
CHANGED
|
@@ -11,10 +11,11 @@ You are a Mindsystem plan writer. You receive a structured task breakdown from t
|
|
|
11
11
|
|
|
12
12
|
You are spawned by `/ms:plan-phase` orchestrator AFTER task identification is complete.
|
|
13
13
|
|
|
14
|
-
Your job: Transform task lists into
|
|
14
|
+
Your job: Transform task lists into PLAN.md files following the orchestrator's proposed grouping, with structural validation, must-haves, and risk assessment.
|
|
15
15
|
|
|
16
16
|
**What you receive:**
|
|
17
17
|
- Task list with needs/creates/tdd_candidate flags
|
|
18
|
+
- Proposed grouping from orchestrator (plan boundaries, wave assignments, budget estimates)
|
|
18
19
|
- Phase context (number, name, goal, directory, requirements)
|
|
19
20
|
- Project references (paths to STATE, ROADMAP, CONTEXT, prior summaries)
|
|
20
21
|
- Relevant learnings from past work (debug resolutions, adhoc insights, established patterns, prior decisions, curated cross-milestone learnings)
|
|
@@ -77,6 +78,17 @@ The orchestrator provides structured XML:
|
|
|
77
78
|
</prior_summaries>
|
|
78
79
|
</project_refs>
|
|
79
80
|
|
|
81
|
+
<proposed_grouping>
|
|
82
|
+
<plan id="01" title="Auth foundation" budget="30%" wave="1">
|
|
83
|
+
<tasks>1, 2, 3</tasks>
|
|
84
|
+
<rationale>Core models and config — no external dependencies</rationale>
|
|
85
|
+
</plan>
|
|
86
|
+
<plan id="02" title="Auth endpoints + UI" budget="40%" wave="2">
|
|
87
|
+
<tasks>4, 5, 6</tasks>
|
|
88
|
+
<rationale>Depends on models from Plan 01</rationale>
|
|
89
|
+
</plan>
|
|
90
|
+
</proposed_grouping>
|
|
91
|
+
|
|
80
92
|
<learnings>
|
|
81
93
|
<learning type="debug" source=".planning/debug/resolved/n-plus-one-queries.md">Missing eager loading on association chains — fix: Added includes() for all relationship traversals</learning>
|
|
82
94
|
<learning type="pattern" source=".planning/phases/02-foundation/02-01-SUMMARY.md">All API endpoints use Zod validation on input</learning>
|
|
@@ -150,27 +162,30 @@ Verify:
|
|
|
150
162
|
Wave assignments are written to EXECUTION-ORDER.md, not to individual plans.
|
|
151
163
|
</step>
|
|
152
164
|
|
|
153
|
-
<step name="
|
|
154
|
-
**
|
|
165
|
+
<step name="validate_and_apply_grouping">
|
|
166
|
+
**Apply the orchestrator's proposed grouping after structural validation.**
|
|
155
167
|
|
|
156
|
-
|
|
157
|
-
1. **Same-wave tasks with no file conflicts → parallel plans**
|
|
158
|
-
2. **Tasks with shared files → same plan**
|
|
159
|
-
3. **TDD candidates → dedicated plans (one feature per TDD plan)**
|
|
160
|
-
4. **2-3 tasks per plan, ~50% context target**
|
|
161
|
-
5. **Default to 3 tasks for simple-medium work, 2 for complex**
|
|
168
|
+
The orchestrator proposed plan boundaries collaboratively with the user. Start from these boundaries — do NOT re-derive grouping from budget math.
|
|
162
169
|
|
|
163
|
-
|
|
164
|
-
|
|
165
|
-
1.
|
|
166
|
-
|
|
167
|
-
|
|
168
|
-
|
|
169
|
-
|
|
170
|
-
|
|
170
|
+
**If `<proposed_grouping>` is absent:** Stop and report the missing grouping to the orchestrator. Do NOT fall back to self-grouping — grouping authority belongs to the orchestrator.
|
|
171
|
+
|
|
172
|
+
**1. Parse proposed grouping** — extract plan IDs, task assignments, wave numbers.
|
|
173
|
+
|
|
174
|
+
**2. Validate structurally** against the dependency graph built in the previous step:
|
|
175
|
+
- **File conflicts:** Two tasks in the same parallel wave that modify the same file
|
|
176
|
+
- **Circular dependencies:** Plan A needs Plan B and Plan B needs Plan A
|
|
177
|
+
- **Missing dependency chains:** A task needs an artifact from another plan but isn't sequenced after it
|
|
178
|
+
- **TDD isolation:** Tasks marked `tdd_candidate` must be in dedicated plans
|
|
179
|
+
|
|
180
|
+
**3. Apply grouping:**
|
|
181
|
+
- If validation passes → use proposed boundaries as-is
|
|
182
|
+
- If structural issue found → adjust minimally to resolve (move conflicting task, add wave dependency). Record the deviation and reason in grouping rationale.
|
|
183
|
+
|
|
184
|
+
**4. Classify task weights** (L/M/H) for the grouping rationale table. Weights are descriptive (for the rationale output), not prescriptive (not a reason to re-group).
|
|
171
185
|
|
|
172
|
-
**
|
|
173
|
-
|
|
186
|
+
**5. Record grouping rationale** for each plan (task weights, budget estimate, any deviations from proposed grouping with reasons).
|
|
187
|
+
|
|
188
|
+
**Do NOT override proposed grouping because weight math exceeds 45%.** The orchestrator already considered context budget. Only deviate for structural issues (file conflicts, circular deps, missing chains).
|
|
174
189
|
</step>
|
|
175
190
|
|
|
176
191
|
<step name="derive_must_haves">
|
|
@@ -194,19 +209,14 @@ The verifier derives artifacts and key_links from the plan's ## Changes section.
|
|
|
194
209
|
</step>
|
|
195
210
|
|
|
196
211
|
<step name="estimate_scope">
|
|
197
|
-
**Verify each plan
|
|
212
|
+
**Verify each plan's scope is reasonable.**
|
|
198
213
|
|
|
199
214
|
Per plan:
|
|
200
|
-
-
|
|
201
|
-
- Count files modified
|
|
202
|
-
-
|
|
203
|
-
|
|
204
|
-
If any plan exceeds:
|
|
205
|
-
- 4+ tasks: Split by feature
|
|
206
|
-
- 10+ files: Split by subsystem
|
|
207
|
-
- Complex domain (auth, payments): Consider extra split
|
|
215
|
+
- Sum weights for grouping rationale table
|
|
216
|
+
- Count files modified
|
|
217
|
+
- Flag plans modifying 10+ files in grouping rationale
|
|
208
218
|
|
|
209
|
-
|
|
219
|
+
This step is informational. Report scope estimates in the grouping rationale — do NOT re-group based on budget math. The orchestrator already made grouping decisions with user input.
|
|
210
220
|
</step>
|
|
211
221
|
|
|
212
222
|
<step name="write_plan_files">
|
|
@@ -334,11 +344,11 @@ Capture commit hash for return.
|
|
|
334
344
|
score = 0
|
|
335
345
|
factors = []
|
|
336
346
|
|
|
337
|
-
#
|
|
338
|
-
|
|
339
|
-
if
|
|
347
|
+
# Budget per plan (>45%)
|
|
348
|
+
max_budget = max(budget_sum for each plan)
|
|
349
|
+
if max_budget > 45:
|
|
340
350
|
score += 15
|
|
341
|
-
factors.append(f"Plan
|
|
351
|
+
factors.append(f"Plan exceeds 45% budget ({max_budget}%)")
|
|
342
352
|
|
|
343
353
|
# Plan count (5+ plans in phase)
|
|
344
354
|
if plan_count >= 5:
|
|
@@ -400,6 +410,18 @@ Return structured markdown to orchestrator:
|
|
|
400
410
|
| 1 | 01, 02 | None (parallel) |
|
|
401
411
|
| 2 | 03 | Waits for 01, 02 |
|
|
402
412
|
|
|
413
|
+
### Grouping Rationale
|
|
414
|
+
|
|
415
|
+
| Plan | Tasks | Est. Weight | Notes |
|
|
416
|
+
|------|-------|-------------|-------|
|
|
417
|
+
| 01 | 4 (L+L+L+M) | ~25% | Per orchestrator proposal |
|
|
418
|
+
| 02 | 3 (M+M+M) | ~30% | Per orchestrator proposal |
|
|
419
|
+
|
|
420
|
+
{If any deviations from proposed grouping:}
|
|
421
|
+
|
|
422
|
+
### Grouping Deviations
|
|
423
|
+
- **Plan 03 split from Plan 02:** File conflict — both tasks modify `src/config.ts`
|
|
424
|
+
|
|
403
425
|
### Risk Assessment
|
|
404
426
|
|
|
405
427
|
**Score:** {score}/100 ({tier})
|
|
@@ -416,6 +438,8 @@ Return structured markdown to orchestrator:
|
|
|
416
438
|
- ...
|
|
417
439
|
```
|
|
418
440
|
|
|
441
|
+
**Include Grouping Rationale** when any deviation from proposed grouping occurred. Omit when proposed grouping was applied as-is.
|
|
442
|
+
|
|
419
443
|
The orchestrator parses this to present risk via AskUserQuestion and offer next steps.
|
|
420
444
|
</output_format>
|
|
421
445
|
|
|
@@ -432,8 +456,8 @@ Plan 02 does not depend on Plan 01 just because 01 comes first. Check actual nee
|
|
|
432
456
|
Bad: Plan 01 = all models, Plan 02 = all APIs
|
|
433
457
|
Good: Plan 01 = User (model + API), Plan 02 = Product (model + API)
|
|
434
458
|
|
|
435
|
-
**DO NOT
|
|
436
|
-
|
|
459
|
+
**DO NOT override proposed grouping for budget reasons.**
|
|
460
|
+
The orchestrator's grouping reflects collaborative decisions with the user. Deviate only for structural issues (file conflicts, circular deps). Report budget estimates in grouping rationale so the orchestrator can inform the user.
|
|
437
461
|
|
|
438
462
|
**DO NOT write implementation-focused truths.**
|
|
439
463
|
Bad: "bcrypt library installed"
|
|
@@ -451,7 +475,7 @@ Plan writing complete when:
|
|
|
451
475
|
- [ ] References loaded (phase-prompt, plan-format, scope-estimation, + tdd if needed)
|
|
452
476
|
- [ ] Dependency graph built from needs/creates
|
|
453
477
|
- [ ] Waves assigned (all roots wave 1, dependents correct)
|
|
454
|
-
- [ ]
|
|
478
|
+
- [ ] Proposed grouping validated structurally and applied (deviations only for file conflicts/circular deps)
|
|
455
479
|
- [ ] Must-haves derived as markdown checklists
|
|
456
480
|
- [ ] PLAN.md files written with pure markdown format
|
|
457
481
|
- [ ] EXECUTION-ORDER.md generated with wave groups
|
|
@@ -21,7 +21,7 @@ This spawns ms-plan-checker to analyze your PLAN.md files against the phase goal
|
|
|
21
21
|
2. **Task Completeness** — Does every change have Files, implementation details, and verification?
|
|
22
22
|
3. **Dependency Correctness** — Are plan dependencies valid and acyclic?
|
|
23
23
|
4. **Key Links Planned** — Are artifacts wired together, not just created in isolation?
|
|
24
|
-
5. **Scope Sanity** —
|
|
24
|
+
5. **Scope Sanity** — Are plans reasonably sized for context budget (target 25-45%)?
|
|
25
25
|
6. **Verification Derivation** — Are Must-Haves user-observable, not implementation-focused?
|
|
26
26
|
7. **Context Compliance** — Do plans honor decisions from CONTEXT.md?
|
|
27
27
|
</what_it_checks>
|
|
@@ -72,11 +72,30 @@ Wait for response and act accordingly.
|
|
|
72
72
|
|
|
73
73
|
**If doesn't exist:** Continue to gather context.
|
|
74
74
|
|
|
75
|
-
## 3.
|
|
75
|
+
## 3. Discover Relevant Skills
|
|
76
|
+
|
|
77
|
+
Before gathering context, check for skills that provide design-relevant conventions.
|
|
78
|
+
|
|
79
|
+
**3a. Scan available skills:**
|
|
80
|
+
|
|
81
|
+
Scan skills in your system-reminder for matches. Look for skills related to:
|
|
82
|
+
- The platform or framework (Flutter, React, web, etc.)
|
|
83
|
+
- UI conventions, design systems, or component patterns
|
|
84
|
+
- The specific domain of this phase (from the ROADMAP.md phase description extracted in step 1)
|
|
85
|
+
|
|
86
|
+
**3b. Confirm with user:**
|
|
87
|
+
|
|
88
|
+
Use AskUserQuestion to present findings. Always include an escape hatch for the user to name a skill manually.
|
|
89
|
+
|
|
90
|
+
**3c. Load selected skills:**
|
|
91
|
+
|
|
92
|
+
Invoke each confirmed skill via the Skill tool. Extract aesthetic patterns (colors, components, spacing, typography) from loaded content for the `<existing_aesthetic>` block in step 6.
|
|
93
|
+
|
|
94
|
+
## 4. Gather Context Chain
|
|
76
95
|
|
|
77
96
|
Load context in order of priority:
|
|
78
97
|
|
|
79
|
-
**
|
|
98
|
+
**4a. Mandatory context:**
|
|
80
99
|
|
|
81
100
|
```bash
|
|
82
101
|
# Load PROJECT.md for product context
|
|
@@ -97,7 +116,7 @@ Extract from ROADMAP.md:
|
|
|
97
116
|
- Success criteria
|
|
98
117
|
- Requirements mapped
|
|
99
118
|
|
|
100
|
-
**
|
|
119
|
+
**4b. Optional context - CONTEXT.md (from discuss-phase):**
|
|
101
120
|
|
|
102
121
|
```bash
|
|
103
122
|
cat .planning/phases/${PHASE}-*/${PHASE}-CONTEXT.md 2>/dev/null
|
|
@@ -108,7 +127,7 @@ If exists, extract:
|
|
|
108
127
|
- What Must Be Nailed (essentials)
|
|
109
128
|
- Specific Ideas (references to products)
|
|
110
129
|
|
|
111
|
-
**
|
|
130
|
+
**4b2. Optional context — prior knowledge:**
|
|
112
131
|
|
|
113
132
|
Match subsystem(s) to this phase by comparing ROADMAP phase description against subsystem names in config.json. Load matching knowledge files:
|
|
114
133
|
|
|
@@ -125,7 +144,7 @@ If knowledge files exist, extract:
|
|
|
125
144
|
|
|
126
145
|
Pass the extracted knowledge to ms-designer in the design prompt (see step 6 `<prior_knowledge>` block).
|
|
127
146
|
|
|
128
|
-
**
|
|
147
|
+
**4c. Optional context - codebase analysis:**
|
|
129
148
|
|
|
130
149
|
```bash
|
|
131
150
|
# Platform detection
|
|
@@ -145,16 +164,6 @@ grep -r "colors\|theme\|spacing" src/ --include="*.ts" --include="*.dart" 2>/dev
|
|
|
145
164
|
|
|
146
165
|
Document discovered patterns for the designer.
|
|
147
166
|
|
|
148
|
-
## 4. Load Project Skills
|
|
149
|
-
|
|
150
|
-
Scan the skill list in your system message for skills matching this phase's technology or domain. Invoke each match via the Skill tool before proceeding — skills contain conventions and patterns that change what you design for.
|
|
151
|
-
|
|
152
|
-
- One clear match → invoke it directly
|
|
153
|
-
- Multiple candidates → use AskUserQuestion to let the user choose
|
|
154
|
-
- No match → proceed without
|
|
155
|
-
|
|
156
|
-
Extract aesthetic patterns (colors, components, spacing, typography) from loaded skill content for the `<existing_aesthetic>` block passed to ms-designer.
|
|
157
|
-
|
|
158
167
|
## 5. Adaptive Q&A (If Gaps Exist)
|
|
159
168
|
|
|
160
169
|
Assess context coverage:
|
|
@@ -334,13 +343,11 @@ Generate: DESIGN.md following template structure
|
|
|
334
343
|
Location: .planning/phases/{phase}-{slug}/{phase}-DESIGN.md
|
|
335
344
|
|
|
336
345
|
Required sections:
|
|
337
|
-
1.
|
|
338
|
-
2.
|
|
339
|
-
3.
|
|
340
|
-
|
|
341
|
-
|
|
342
|
-
6. Platform-Specific Notes (responsive, touch targets, accessibility)
|
|
343
|
-
7. Verification Criteria (observable behaviors proving correct implementation)
|
|
346
|
+
1. Design Direction (1-2 sentences: feel, inspiration)
|
|
347
|
+
2. Design Tokens (compact table: token, value, note)
|
|
348
|
+
3. Screens (per screen: wireframe with inline annotations, States table, Behavior notes, Hints)
|
|
349
|
+
|
|
350
|
+
Verification criteria are not a section — plan writer derives from specs.
|
|
344
351
|
</output_specification>
|
|
345
352
|
```
|
|
346
353
|
|
|
@@ -426,16 +433,12 @@ Update `.planning/STATE.md` Last Command field:
|
|
|
426
433
|
</process>
|
|
427
434
|
|
|
428
435
|
<success_criteria>
|
|
429
|
-
- [ ]
|
|
430
|
-
- [ ]
|
|
431
|
-
- [ ]
|
|
432
|
-
- [ ] Project UI skill discovered and loaded if exists
|
|
433
|
-
- [ ] Codebase analyzed for existing patterns
|
|
434
|
-
- [ ] Adaptive Q&A completed if gaps existed
|
|
436
|
+
- [ ] Available skills scanned, surfaced via AskUserQuestion, and loaded via Skill tool
|
|
437
|
+
- [ ] Codebase analyzed for existing patterns (step 4c)
|
|
438
|
+
- [ ] Adaptive Q&A completed if context gaps existed
|
|
435
439
|
- [ ] Mockup generation offered if phase has significant new UI
|
|
436
440
|
- [ ] Mockup direction extracted and passed to ms-designer (if generated)
|
|
437
441
|
- [ ] ms-designer spawned with quality-forcing patterns
|
|
438
|
-
- [ ] DESIGN.md created with
|
|
439
|
-
- [ ] DESIGN.md committed
|
|
442
|
+
- [ ] DESIGN.md created with Design Direction, Design Tokens, and Screens sections and committed
|
|
440
443
|
- [ ] User informed of refinement options and next steps
|
|
441
444
|
</success_criteria>
|