@hegemonart/get-design-done 1.28.0 → 1.28.6
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/.claude-plugin/marketplace.json +2 -2
- package/.claude-plugin/plugin.json +1 -1
- package/CHANGELOG.md +134 -0
- package/SKILL.md +1 -1
- package/hooks/gdd-decision-injector.js +149 -3
- package/package.json +1 -1
- package/reference/adr-format.md +96 -0
- package/reference/architecture-vocabulary.md +102 -0
- package/reference/context-md-format.md +106 -0
- package/reference/heuristics.md +84 -0
- package/reference/registry.json +29 -1
- package/reference/registry.schema.json +1 -1
- package/reference/shared-preamble.md +78 -6
- package/reference/skill-authoring-contract.md +159 -0
- package/scripts/validate-skill-length.cjs +283 -0
- package/skills/add-backlog/SKILL.md +1 -0
- package/skills/analyze-dependencies/SKILL.md +33 -122
- package/skills/apply-reflections/SKILL.md +1 -40
- package/skills/apply-reflections/apply-reflections-procedure.md +68 -0
- package/skills/audit/SKILL.md +3 -1
- package/skills/bandit-status/SKILL.md +31 -66
- package/skills/benchmark/SKILL.md +15 -55
- package/skills/brief/SKILL.md +12 -1
- package/skills/cache-manager/SKILL.md +3 -57
- package/skills/cache-manager/cache-policy.md +126 -0
- package/skills/check-update/SKILL.md +38 -75
- package/skills/compare/SKILL.md +29 -269
- package/skills/compare/compare-rubric.md +171 -0
- package/skills/complete-cycle/SKILL.md +1 -1
- package/skills/connections/SKILL.md +21 -427
- package/skills/connections/connections-onboarding.md +417 -0
- package/skills/continue/SKILL.md +1 -0
- package/skills/darkmode/SKILL.md +32 -287
- package/skills/darkmode/darkmode-audit-procedure.md +258 -0
- package/skills/debug/SKILL.md +11 -8
- package/skills/debug/debug-feedback-loops.md +119 -0
- package/skills/design/SKILL.md +27 -245
- package/skills/design/design-procedure.md +304 -0
- package/skills/discover/SKILL.md +26 -133
- package/skills/discover/discover-procedure.md +204 -0
- package/skills/discuss/SKILL.md +18 -2
- package/skills/explore/SKILL.md +40 -205
- package/skills/explore/explore-procedure.md +267 -0
- package/skills/fast/SKILL.md +1 -0
- package/skills/figma-write/SKILL.md +2 -2
- package/skills/health/SKILL.md +11 -33
- package/skills/health/health-mcp-detection.md +44 -0
- package/skills/health/health-skill-length-report.md +69 -0
- package/skills/help/SKILL.md +1 -0
- package/skills/list-assumptions/SKILL.md +1 -0
- package/skills/map/SKILL.md +8 -31
- package/skills/new-cycle/SKILL.md +3 -1
- package/skills/new-cycle/milestone-completeness-rubric.md +87 -0
- package/skills/next/SKILL.md +1 -0
- package/skills/note/SKILL.md +1 -0
- package/skills/optimize/SKILL.md +21 -44
- package/skills/pause/SKILL.md +1 -0
- package/skills/peer-cli-add/SKILL.md +26 -108
- package/skills/peer-cli-add/peer-cli-protocol.md +161 -0
- package/skills/peer-cli-customize/SKILL.md +22 -42
- package/skills/peers/SKILL.md +33 -57
- package/skills/plan/SKILL.md +33 -220
- package/skills/plan/plan-procedure.md +278 -0
- package/skills/plant-seed/SKILL.md +1 -0
- package/skills/pr-branch/SKILL.md +1 -0
- package/skills/progress/SKILL.md +1 -7
- package/skills/quality-gate/SKILL.md +34 -166
- package/skills/quality-gate/threat-modeling.md +101 -0
- package/skills/quick/SKILL.md +1 -0
- package/skills/reapply-patches/SKILL.md +1 -0
- package/skills/recall/SKILL.md +1 -0
- package/skills/resume/SKILL.md +1 -0
- package/skills/review-backlog/SKILL.md +1 -0
- package/skills/router/SKILL.md +3 -59
- package/skills/router/router-rules.md +84 -0
- package/skills/scan/SKILL.md +36 -675
- package/skills/scan/scan-procedure.md +731 -0
- package/skills/settings/SKILL.md +1 -0
- package/skills/ship/SKILL.md +1 -0
- package/skills/sketch/SKILL.md +1 -1
- package/skills/sketch-wrap-up/SKILL.md +13 -54
- package/skills/spike/SKILL.md +1 -1
- package/skills/spike-wrap-up/SKILL.md +12 -46
- package/skills/start/SKILL.md +13 -112
- package/skills/start/start-procedure.md +115 -0
- package/skills/stats/SKILL.md +1 -0
- package/skills/style/SKILL.md +18 -140
- package/skills/style/style-doc-procedure.md +150 -0
- package/skills/synthesize/SKILL.md +1 -0
- package/skills/timeline/SKILL.md +1 -0
- package/skills/todo/SKILL.md +1 -0
- package/skills/turn-closeout/SKILL.md +36 -56
- package/skills/undo/SKILL.md +1 -0
- package/skills/update/SKILL.md +1 -0
- package/skills/verify/SKILL.md +42 -457
- package/skills/verify/verify-procedure.md +512 -0
- package/skills/warm-cache/SKILL.md +3 -35
- package/skills/zoom-out/SKILL.md +26 -0
|
@@ -0,0 +1,304 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: design-procedure
|
|
3
|
+
type: meta-rules
|
|
4
|
+
version: 1.0.0
|
|
5
|
+
phase: 28.5
|
|
6
|
+
tags: [design, procedure, extracted, pipeline-stage, execute, wave-coordination]
|
|
7
|
+
last_updated: 2026-05-18
|
|
8
|
+
---
|
|
9
|
+
|
|
10
|
+
Source: extracted from `skills/design/SKILL.md` (Phase 28.5 rework — D-10 extract-then-link).
|
|
11
|
+
The skill's load-bearing workflow stays in `../skills/design/SKILL.md`; this file holds the
|
|
12
|
+
detail the agent reaches for when executing a specific step (agent spawn prompts, wave
|
|
13
|
+
coordination, executor STATE.md protocol, figma-write dispatch).
|
|
14
|
+
|
|
15
|
+
# Design Procedure
|
|
16
|
+
|
|
17
|
+
Detailed procedure for the get-design-done `design` Stage 4 orchestrator. Companion to
|
|
18
|
+
`../skills/design/SKILL.md`. Read this file when executing a specific design step; the
|
|
19
|
+
SKILL.md keeps the load-bearing wave-iteration workflow + decision tree, this file holds
|
|
20
|
+
the full executor prompts and parallelism semantics.
|
|
21
|
+
|
|
22
|
+
---
|
|
23
|
+
|
|
24
|
+
## Stage entry
|
|
25
|
+
|
|
26
|
+
1. Call `mcp__gdd_state__transition_stage` with `to: "design"`.
|
|
27
|
+
- Gate failure surfaces `error.context.blockers` to the user; do not advance.
|
|
28
|
+
- If the transition succeeds and the prior stage was already `design` with `status: in_progress`, this is a RESUME — use `task_progress` numerator as source of truth and skip tasks that already have a corresponding `.design/tasks/task-NN.md` file.
|
|
29
|
+
2. Call `mcp__gdd_state__get` -> snapshot `state`; read `state.position.wave` to decide execution plan.
|
|
30
|
+
|
|
31
|
+
Abort only if `.design/DESIGN-PLAN.md` is missing:
|
|
32
|
+
> "No plan found. Run `/get-design-done:plan` first."
|
|
33
|
+
|
|
34
|
+
---
|
|
35
|
+
|
|
36
|
+
## Flag Parsing
|
|
37
|
+
|
|
38
|
+
- `--auto` -> `auto_mode=true` (no mid-stage prompts; architectural deviations still stop the individual task but continue with remaining tasks)
|
|
39
|
+
- `--parallel` -> `parallel_mode=true` (use worktree isolation for `Parallel: true` tasks)
|
|
40
|
+
|
|
41
|
+
---
|
|
42
|
+
|
|
43
|
+
## Pre-execution — Directionally-open check
|
|
44
|
+
|
|
45
|
+
Scan DESIGN-PLAN.md for tasks marked as "directionally open" (exploration-appropriate — e.g., tasks whose acceptance criteria read "explore N directions" or "pick a visual approach"). If any are found, print:
|
|
46
|
+
|
|
47
|
+
> "Tasks [IDs] appear directionally open — consider running `/gdd:sketch` first to explore variants before implementation."
|
|
48
|
+
|
|
49
|
+
Skip if `auto_mode=true`.
|
|
50
|
+
|
|
51
|
+
## Pre-execution — Project-local conventions
|
|
52
|
+
|
|
53
|
+
When spawning the executor, include any `./.claude/skills/design-*-conventions.md` files in `<required_reading>` so the executor sees project-local design conventions (typography, color, layout, motion, component, interaction decisions codified from prior sketch wrap-ups). Also include any `~/.claude/gdd/global-skills/*.md` files if the directory exists — global skills are cross-project conventions that inform but do not override project-local D-XX decisions.
|
|
54
|
+
|
|
55
|
+
---
|
|
56
|
+
|
|
57
|
+
## .stories.tsx Stub (when storybook project detected)
|
|
58
|
+
|
|
59
|
+
After every new component file is created by the design-executor:
|
|
60
|
+
|
|
61
|
+
Step 1 — Check project detection (does not require server running):
|
|
62
|
+
Bash: ls .storybook/ 2>/dev/null || grep '"storybook"' package.json 2>/dev/null
|
|
63
|
+
-> Found -> storybook_project: true
|
|
64
|
+
-> Not found -> skip .stories.tsx emission
|
|
65
|
+
|
|
66
|
+
Step 2 — When storybook_project: true, emit a CSF stub alongside the component:
|
|
67
|
+
File: `<same directory as component>/<ComponentName>.stories.tsx`
|
|
68
|
+
Content follows CSF format (see `connections/storybook.md` for full template):
|
|
69
|
+
- Import `Meta` and `StoryObj` from `@storybook/react`
|
|
70
|
+
- Import the new component
|
|
71
|
+
- `meta: Meta<typeof ComponentName>` with `title` and `parameters.a11y.test = 'error'`
|
|
72
|
+
- Export `Default`, `Primary`, `Disabled` story variants
|
|
73
|
+
Adjust `title` to match directory structure (e.g., `'Components/Button'` or `'Features/Auth/LoginForm'`)
|
|
74
|
+
|
|
75
|
+
Note: the `.stories.tsx` stub is emitted whenever `storybook_project: true` regardless of whether
|
|
76
|
+
the dev server is running. New components need stories even in offline/CI contexts.
|
|
77
|
+
|
|
78
|
+
---
|
|
79
|
+
|
|
80
|
+
## Step 1 — Parse DESIGN-PLAN.md
|
|
81
|
+
|
|
82
|
+
Read `.design/DESIGN-PLAN.md`. Partition tasks by `## Wave N` heading. Within each wave, partition by `Parallel: true` vs `Parallel: false`. Compute `total_tasks` for `task_progress` denominator.
|
|
83
|
+
|
|
84
|
+
If resuming: skip tasks where `.design/tasks/task-NN.md` already exists.
|
|
85
|
+
|
|
86
|
+
---
|
|
87
|
+
|
|
88
|
+
## Parallelism Decision (per wave, before spawning)
|
|
89
|
+
|
|
90
|
+
For each wave:
|
|
91
|
+
1. Read `.design/config.json` `parallelism` (or defaults from `reference/config-schema.md`).
|
|
92
|
+
2. Collect candidates in the wave; check `Touches:`, `writes:`, `parallel-safe`, and `typical-duration-seconds` fields.
|
|
93
|
+
3. Apply rules in order from `reference/parallelism-rules.md` (hard -> soft). Overlapping Touches split into sequential sub-waves.
|
|
94
|
+
4. Record the parallelism decision for this wave via `mcp__gdd_state__update_progress` with `task_progress: "<completed>/<total>"` and `status: "design_wave_<N>_parallelism: <parallel|serial>, reason=<short-reason>"` — the status string is the canonical carrier (mirrors the plan-stage convention from Plan 20-09; a dedicated tool may be added in a follow-on plan).
|
|
95
|
+
5. If `parallel`: spawn all candidates via concurrent `Task()` calls in one response. If `serial`: spawn sequentially.
|
|
96
|
+
|
|
97
|
+
### Executor prompt template (applies to every spawned design-executor)
|
|
98
|
+
|
|
99
|
+
Every spawned executor receives the following STATE.md contract in its prompt:
|
|
100
|
+
|
|
101
|
+
> **STATE.md mutation protocol** — When you complete a task in your assigned batch, update STATE.md ONLY via the `gdd-state` MCP tools. Specifically:
|
|
102
|
+
> - Report task progress: `mcp__gdd_state__update_progress` with your new `task_progress` fraction.
|
|
103
|
+
> - Add blockers: `mcp__gdd_state__add_blocker` with `{ stage: "design", date: <today>, text: "..." }`.
|
|
104
|
+
> - Resolve your own blockers on fix: `mcp__gdd_state__resolve_blocker` with the blocker id.
|
|
105
|
+
>
|
|
106
|
+
> Do NOT `Read` + `Write` `.design/STATE.md` directly — the MCP tools enforce the lockfile and emit mutation events. Direct writes corrupt parallel state.
|
|
107
|
+
|
|
108
|
+
Inline this protocol block verbatim inside every design-executor prompt in both the parallel-batch and sequential-tail spawns below. Concurrent executors (Phase 10.1 parallel mode) each emit `update_progress` calls; the lockfile (Plan 20-01) and event stream (Plan 20-06) serialize them safely.
|
|
109
|
+
|
|
110
|
+
## Step 2 — Wave-by-Wave Execution
|
|
111
|
+
|
|
112
|
+
For each Wave in order (Wave 1, Wave 2, ...):
|
|
113
|
+
|
|
114
|
+
### Parallel batch (if `parallel_mode=true` AND any `Parallel: true` tasks in wave)
|
|
115
|
+
|
|
116
|
+
Report the partition before spawning:
|
|
117
|
+
|
|
118
|
+
```
|
|
119
|
+
=== Wave [N] — parallel mode ===
|
|
120
|
+
Parallel batch ([N] tasks — spawning concurrently):
|
|
121
|
+
[01] [type]: [scope] — touches: [files]
|
|
122
|
+
[02] [type]: [scope] — touches: [files]
|
|
123
|
+
|
|
124
|
+
Sequential tail ([N] tasks):
|
|
125
|
+
[03] [type]: [scope] — touches: [files]
|
|
126
|
+
|
|
127
|
+
Spawning agents now...
|
|
128
|
+
================================
|
|
129
|
+
```
|
|
130
|
+
|
|
131
|
+
Spawn ALL `Parallel: true` tasks in this wave as concurrent `Task()` calls in ONE response. Each call uses `isolation: "worktree"`:
|
|
132
|
+
|
|
133
|
+
```
|
|
134
|
+
Task("design-executor", """
|
|
135
|
+
<required_reading>
|
|
136
|
+
@.design/STATE.md
|
|
137
|
+
@.design/DESIGN-PLAN.md
|
|
138
|
+
@.design/DESIGN-CONTEXT.md
|
|
139
|
+
@reference/[type-relevant].md
|
|
140
|
+
</required_reading>
|
|
141
|
+
|
|
142
|
+
You are the design-executor agent. Execute Task NN from DESIGN-PLAN.md.
|
|
143
|
+
|
|
144
|
+
Prompt context:
|
|
145
|
+
task_id: NN
|
|
146
|
+
task_type: [type]
|
|
147
|
+
task_scope: [scope]
|
|
148
|
+
task_acceptance_criteria:
|
|
149
|
+
- [criterion 1]
|
|
150
|
+
- [criterion 2]
|
|
151
|
+
wave: N
|
|
152
|
+
is_parallel: true
|
|
153
|
+
auto_mode: [true|false]
|
|
154
|
+
|
|
155
|
+
Write .design/tasks/task-NN.md and make an atomic commit `feat(design-NN): [type] — [scope]`.
|
|
156
|
+
|
|
157
|
+
Emit `## EXECUTION COMPLETE` when done.
|
|
158
|
+
""", subagent_type="design-executor", isolation="worktree")
|
|
159
|
+
```
|
|
160
|
+
|
|
161
|
+
Wait for all parallel tasks to emit `## EXECUTION COMPLETE`.
|
|
162
|
+
|
|
163
|
+
**Merge worktrees** (preserved from v2.1.0 — do not redesign):
|
|
164
|
+
|
|
165
|
+
```
|
|
166
|
+
=== Parallel batch complete ===
|
|
167
|
+
[check/warn/cross] Task 01 — [type]
|
|
168
|
+
[check/warn/cross] Task 02 — [type]
|
|
169
|
+
|
|
170
|
+
Merging worktrees...
|
|
171
|
+
================================
|
|
172
|
+
```
|
|
173
|
+
|
|
174
|
+
Merge each worktree branch back into the working directory. Each agent touched non-overlapping files (guaranteed by the conflict check on `Touches:` fields). If an unexpected merge conflict appears, flag it and ask the user to resolve before continuing.
|
|
175
|
+
|
|
176
|
+
After merge, roll up the batch's progress:
|
|
177
|
+
|
|
178
|
+
- Call `mcp__gdd_state__update_progress` with `task_progress: "<completed>/<total>"` and `status: "design_wave_<N>_parallel_batch_complete"`.
|
|
179
|
+
- Call `mcp__gdd_state__checkpoint` — records the wave boundary in `<timestamps>` and bumps `last_checkpoint`.
|
|
180
|
+
|
|
181
|
+
### Sequential tail (Parallel: false tasks, or all tasks if `parallel_mode=false`)
|
|
182
|
+
|
|
183
|
+
Announce each wave before starting:
|
|
184
|
+
|
|
185
|
+
```
|
|
186
|
+
=== Wave [N] — [N tasks] — sequential ===
|
|
187
|
+
Tasks:
|
|
188
|
+
[01] [type]: [scope]
|
|
189
|
+
[02] [type]: [scope]
|
|
190
|
+
==========================================
|
|
191
|
+
```
|
|
192
|
+
|
|
193
|
+
Run one at a time. Same `Task("design-executor", ...)` pattern with `is_parallel: false` (no worktree isolation):
|
|
194
|
+
|
|
195
|
+
```
|
|
196
|
+
Task("design-executor", """
|
|
197
|
+
<required_reading>
|
|
198
|
+
@.design/STATE.md
|
|
199
|
+
@.design/DESIGN-PLAN.md
|
|
200
|
+
@.design/DESIGN-CONTEXT.md
|
|
201
|
+
@reference/[type-relevant].md
|
|
202
|
+
</required_reading>
|
|
203
|
+
|
|
204
|
+
You are the design-executor agent. Execute Task NN from DESIGN-PLAN.md.
|
|
205
|
+
|
|
206
|
+
Prompt context:
|
|
207
|
+
task_id: NN
|
|
208
|
+
task_type: [type]
|
|
209
|
+
task_scope: [scope]
|
|
210
|
+
task_acceptance_criteria:
|
|
211
|
+
- [criterion 1]
|
|
212
|
+
- [criterion 2]
|
|
213
|
+
wave: N
|
|
214
|
+
is_parallel: false
|
|
215
|
+
auto_mode: [true|false]
|
|
216
|
+
|
|
217
|
+
Write .design/tasks/task-NN.md and make an atomic commit `feat(design-NN): [type] — [scope]`.
|
|
218
|
+
|
|
219
|
+
Emit `## EXECUTION COMPLETE` when done.
|
|
220
|
+
""", subagent_type="design-executor")
|
|
221
|
+
```
|
|
222
|
+
|
|
223
|
+
After each task completes, call `mcp__gdd_state__update_progress` with the new `task_progress: "<completed>/<total>"` and `status: "design_wave_<N>_task_<NN>_complete"`.
|
|
224
|
+
|
|
225
|
+
After the final sequential task of the wave, call `mcp__gdd_state__checkpoint` — records the wave boundary in `<timestamps>` and bumps `last_checkpoint`.
|
|
226
|
+
|
|
227
|
+
---
|
|
228
|
+
|
|
229
|
+
## Step 3 — Wave Checkpoint
|
|
230
|
+
|
|
231
|
+
After each wave (unless `--auto` flag was passed):
|
|
232
|
+
|
|
233
|
+
```
|
|
234
|
+
=== Wave [N] complete ===
|
|
235
|
+
check [N] tasks complete
|
|
236
|
+
warn [N] deviations (see .design/tasks/ files)
|
|
237
|
+
|
|
238
|
+
Ready for Wave [N+1]? (yes / review first)
|
|
239
|
+
=========================
|
|
240
|
+
```
|
|
241
|
+
|
|
242
|
+
Skip checkpoint if `auto_mode=true`.
|
|
243
|
+
|
|
244
|
+
---
|
|
245
|
+
|
|
246
|
+
## Step 4 — Handle Deviations
|
|
247
|
+
|
|
248
|
+
After each wave, check task-NN.md files for `status: deviation`. If any found:
|
|
249
|
+
|
|
250
|
+
- Call `mcp__gdd_state__get` -> read `state.blockers`; present affected task IDs and their blocker descriptions from the returned snapshot.
|
|
251
|
+
- Offer: (a) stop stage, (b) continue remaining tasks
|
|
252
|
+
- In `auto_mode`: continue automatically, log all deviations
|
|
253
|
+
- When a blocker is addressed (fix committed by a follow-up task), call `mcp__gdd_state__resolve_blocker` with the blocker id to clear it from the live state.
|
|
254
|
+
|
|
255
|
+
---
|
|
256
|
+
|
|
257
|
+
## State Update (exit)
|
|
258
|
+
|
|
259
|
+
1. Call `mcp__gdd_state__set_status` with `status: "design_complete"` — marks the stage completed without transitioning; verify calls `transition_stage` on its entry, keeping the transition atomic with the owning stage.
|
|
260
|
+
2. Call `mcp__gdd_state__checkpoint` — stamps `last_checkpoint` and appends a `design_completed_at` entry to `<timestamps>`.
|
|
261
|
+
|
|
262
|
+
---
|
|
263
|
+
|
|
264
|
+
## After Completion
|
|
265
|
+
|
|
266
|
+
Print summary:
|
|
267
|
+
|
|
268
|
+
```
|
|
269
|
+
=== Design stage complete ===
|
|
270
|
+
Tasks: [N] complete / [M] total
|
|
271
|
+
Deviations: [N]
|
|
272
|
+
Commits: [git log --oneline since stage start]
|
|
273
|
+
|
|
274
|
+
Next: /get-design-done:verify
|
|
275
|
+
-> Scores the result against baseline, checks must-haves,
|
|
276
|
+
runs NNG heuristic evaluation, and identifies gaps.
|
|
277
|
+
==============================
|
|
278
|
+
```
|
|
279
|
+
|
|
280
|
+
---
|
|
281
|
+
|
|
282
|
+
## Figma Write Dispatch (after design-executor completes)
|
|
283
|
+
|
|
284
|
+
After design-executor has finished and DESIGN-PLAN.md tasks are complete:
|
|
285
|
+
|
|
286
|
+
1. Read `figma:` status from `.design/STATE.md` `<connections>` (the unified remote MCP covers both reads and writes as of v1.0.7.1):
|
|
287
|
+
- If `figma: not_configured` or `figma: unavailable` or absent -> skip this block entirely (no prompt, no output)
|
|
288
|
+
- If `figma: available` -> proceed to step 2
|
|
289
|
+
|
|
290
|
+
2. Offer the user a prompt:
|
|
291
|
+
```
|
|
292
|
+
figma write-back is available — propagate design decisions back to Figma?
|
|
293
|
+
Modes: annotate (layer comments) | tokenize (variable bindings) | mappings (Code Connect)
|
|
294
|
+
Run figma-write? (y/N):
|
|
295
|
+
```
|
|
296
|
+
|
|
297
|
+
3. If user answers "y" or "yes":
|
|
298
|
+
- Ask which mode: annotate / tokenize / mappings (or all)
|
|
299
|
+
- Spawn `design-figma-writer` agent with the selected mode
|
|
300
|
+
- Pass `--dry-run` flag if user requests preview first
|
|
301
|
+
|
|
302
|
+
4. If user answers "n", "N", or no response: skip silently.
|
|
303
|
+
|
|
304
|
+
Note: This dispatch is always opt-in. The design stage never auto-runs figma-writer without user confirmation.
|
package/skills/discover/SKILL.md
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: discover
|
|
3
|
-
description: "Stage 1.5 of 4
|
|
3
|
+
description: "Stage 1.5 of 4 orchestrator that probes Figma / Refero / Pinterest connections, spawns design-context-builder (auto-detect + interview) and (via lazy gate) design-context-checker (6-dimension validator), producing .design/DESIGN-CONTEXT.md. Use after /gdd:scan when a fast-path context build is wanted instead of the full /gdd:explore."
|
|
4
4
|
argument-hint: "[--auto]"
|
|
5
5
|
user-invocable: true
|
|
6
6
|
---
|
|
@@ -9,171 +9,64 @@ user-invocable: true
|
|
|
9
9
|
|
|
10
10
|
**Stage 1.5 of 4.** Produces `.design/DESIGN-CONTEXT.md`.
|
|
11
11
|
|
|
12
|
+
Full procedure detail: `./discover-procedure.md`.
|
|
13
|
+
|
|
12
14
|
---
|
|
13
15
|
|
|
14
16
|
## State Integration
|
|
15
17
|
|
|
16
18
|
1. Read `.design/STATE.md`.
|
|
17
|
-
-
|
|
18
|
-
-
|
|
19
|
-
- Otherwise
|
|
20
|
-
2.
|
|
21
|
-
|
|
22
|
-
**A — Figma probe (variant-agnostic):**
|
|
23
|
-
|
|
24
|
-
```
|
|
25
|
-
A1. ToolSearch({ query: "figma get_metadata use_figma", max_results: 10 })
|
|
26
|
-
A2. Parse tool names matching /^mcp__([^_]*figma[^_]*)__(get_metadata|use_figma)$/i
|
|
27
|
-
into read-capable and write-capable prefix sets.
|
|
28
|
-
A3. Empty read set → figma: not_configured (skip all Figma paths)
|
|
29
|
-
One or more matches → pick prefix via tiebreaker:
|
|
30
|
-
(1) both-sets > reads-only,
|
|
31
|
-
(2) `figma` > others,
|
|
32
|
-
(3) non-`figma-desktop` > desktop,
|
|
33
|
-
(4) alphabetical.
|
|
34
|
-
A4. Call {prefix}get_metadata:
|
|
35
|
-
Success → figma: available (prefix=mcp__<prefix>__, writes=<true|false>)
|
|
36
|
-
Error → figma: unavailable
|
|
37
|
-
```
|
|
38
|
-
|
|
39
|
-
**B — Refero probe (ToolSearch presence is sufficient — no tool call needed):**
|
|
40
|
-
|
|
41
|
-
```
|
|
42
|
-
B1. ToolSearch({ query: "refero", max_results: 5 })
|
|
43
|
-
B2. Empty result → refero: not_configured
|
|
44
|
-
Non-empty → refero: available
|
|
45
|
-
```
|
|
46
|
-
|
|
47
|
-
**C — Pinterest probe (ToolSearch-only, same pattern as Refero):**
|
|
48
|
-
|
|
49
|
-
```
|
|
50
|
-
C1. ToolSearch({ query: "mcp-pinterest", max_results: 5 })
|
|
51
|
-
C2. Empty result → pinterest: not_configured
|
|
52
|
-
Non-empty → pinterest: available
|
|
53
|
-
```
|
|
54
|
-
|
|
55
|
-
No live `pinterest_search` call at probe time — ToolSearch presence is sufficient. The synthesizer makes the actual search calls.
|
|
56
|
-
|
|
57
|
-
After all probes (A, B, C), update `.design/STATE.md` `<connections>` with the results and continue. Downstream stages (design-context-builder) read `<connections>` from STATE.md rather than re-probing.
|
|
58
|
-
3. Update last_checkpoint. Write STATE.md.
|
|
59
|
-
|
|
60
|
-
## Auto Mode
|
|
61
|
-
|
|
62
|
-
Auto Mode CSS detection (when `auto_mode: true` is passed to the builder):
|
|
63
|
-
1. If tailwind.config.{js,cjs,mjs,ts} exists → Tailwind-only project
|
|
64
|
-
- Skip CSS file grep
|
|
65
|
-
- Parse tailwind.config for color palette, spacing scale, font families
|
|
66
|
-
- Use tailwind.config values as the baseline style signal
|
|
67
|
-
2. Else → fall through to existing CSS file grep logic
|
|
68
|
-
|
|
69
|
-
## Step 1 — Spawn design-context-builder
|
|
19
|
+
- **Missing** -> create minimal skeleton from `reference/STATE-TEMPLATE.md` (stage=discover, status=in_progress, task_progress=0/1) and log warning "STATE.md not found — created fresh. If this is a resumed session, run /get-design-done:scan first."
|
|
20
|
+
- **Present + stage==discover + status==in_progress** -> RESUME (continue interview; do not reset).
|
|
21
|
+
- **Otherwise** -> normal transition: set frontmatter stage=discover, `<position>` stage=discover, status=in_progress, task_progress=0/1.
|
|
22
|
+
2. Probe connection availability. ToolSearch runs FIRST (MCP tools may be in the deferred tool set). Run three probes — A (Figma, variant-agnostic with prefix tiebreaker), B (Refero, ToolSearch-only), C (Pinterest, ToolSearch-only). After all probes, write `<connections>` to STATE.md so the builder doesn't re-probe. Full probe specs: `./discover-procedure.md` §Connection Probes.
|
|
23
|
+
3. Update `last_checkpoint`. Write STATE.md.
|
|
70
24
|
|
|
71
|
-
|
|
72
|
-
<required_reading>
|
|
73
|
-
@.design/STATE.md
|
|
74
|
-
@reference/audit-scoring.md
|
|
75
|
-
@reference/anti-patterns.md
|
|
76
|
-
</required_reading>
|
|
25
|
+
---
|
|
77
26
|
|
|
78
|
-
|
|
79
|
-
state via grep/glob before asking questions. Interview the user ONLY for areas
|
|
80
|
-
where auto-detect returned no confident answer. Write .design/DESIGN-CONTEXT.md.
|
|
27
|
+
## Auto Mode
|
|
81
28
|
|
|
82
|
-
|
|
83
|
-
1. If src/ exists → use src/
|
|
84
|
-
2. Elif app/ exists → use app/ (Next.js App Router)
|
|
85
|
-
3. Elif pages/ exists → use pages/ (Next.js Pages Router)
|
|
86
|
-
4. Elif lib/ exists → use lib/ (library-only projects)
|
|
87
|
-
5. Else → flag "layout unknown", skip baseline, note in DESIGN-CONTEXT.md
|
|
29
|
+
When `--auto` is passed to the builder: if `tailwind.config.{js,cjs,mjs,ts}` exists -> Tailwind-only project (skip CSS file grep, parse tailwind.config for palette/spacing/font, use those as the baseline style signal). Else fall through to the existing CSS file grep logic. Detail: `./discover-procedure.md` §Auto Mode.
|
|
88
30
|
|
|
89
|
-
|
|
90
|
-
1. font-change risk — switching type families when existing UI has body copy in a specific family. Ask: "Is the current body font intentional or inherited? OK to change?"
|
|
91
|
-
2. token-layer introduction risk — adding CSS custom properties to a codebase that uses direct values. Ask: "Do you want design tokens (--primary, --surface) or inline values (hex, rgb)?"
|
|
92
|
-
3. Component rebuild vs restyle — when to keep existing component, when to rebuild from scratch. Ask: "For <component>, restyle in place or rebuild?"
|
|
31
|
+
---
|
|
93
32
|
|
|
94
|
-
|
|
95
|
-
auto_mode: <true|false>
|
|
33
|
+
## Step 1 — Spawn design-context-builder
|
|
96
34
|
|
|
97
|
-
|
|
98
|
-
Emit `## CONTEXT COMPLETE` when done.
|
|
99
|
-
""")
|
|
35
|
+
Spawn `design-context-builder` -> `.design/DESIGN-CONTEXT.md`. The agent auto-detects via grep/glob first and interviews only for areas where auto-detect returned no confident answer. Baseline audit directory chain: `src/` -> `app/` -> `pages/` -> `lib/` -> flag "layout unknown". Common gray areas to probe (Area 7): font-change risk, token-layer introduction risk, component rebuild-vs-restyle. Wait for `## CONTEXT COMPLETE`, then update STATE.md `task_progress = 0.5`. Full prompt: `./discover-procedure.md` §Step 1.
|
|
100
36
|
|
|
101
|
-
|
|
37
|
+
---
|
|
102
38
|
|
|
103
39
|
## Step 1.75 — Lazy gate: should design-context-checker run? (Plan 10.1-04, D-21)
|
|
104
40
|
|
|
105
|
-
Spawn the cheap Haiku gate before the full
|
|
106
|
-
|
|
107
|
-
Task("design-context-checker-gate", """
|
|
108
|
-
<required_reading>
|
|
109
|
-
@.design/STATE.md
|
|
110
|
-
</required_reading>
|
|
41
|
+
Spawn the cheap Haiku gate `design-context-checker-gate` before the full checker. It applies the single-file heuristic (is `DESIGN-CONTEXT.md` in `git diff --name-only HEAD~1..HEAD`?) and emits JSON + `## GATE COMPLETE`. On `spawn: false`: append `lazy_skipped: true` telemetry row, skip Step 2, set STATE.md `<position>` as if checker passed. On `spawn: true`: proceed to Step 2. On first-run discover the gate always returns `spawn: true` (builder just wrote the file); the gate meaningfully short-circuits only on re-runs where the builder made no changes. Full prompt: `./discover-procedure.md` §Step 1.75.
|
|
111
42
|
|
|
112
|
-
|
|
43
|
+
**Parallel synthesizer note:** discover does not spawn parallel researchers in v1, so `skills/synthesize/` is not wired here. If future variants spawn N parallel interviewers, wire synthesize between dispatch and collate as in `skills/map/` Step 3.5.
|
|
113
44
|
|
|
114
|
-
|
|
115
|
-
diff_files: <git diff --name-only HEAD~1..HEAD>
|
|
116
|
-
diff_body: (not needed — single-file heuristic)
|
|
117
|
-
baseline_sha: <HEAD~1>
|
|
118
|
-
|
|
119
|
-
Apply the heuristic (DESIGN-CONTEXT.md in diff_files?). Emit JSON + `## GATE COMPLETE`.
|
|
120
|
-
""")
|
|
121
|
-
|
|
122
|
-
Wait for `## GATE COMPLETE`. Parse JSON:
|
|
123
|
-
|
|
124
|
-
- `spawn: false` → append `lazy_skipped: true` telemetry row for `design-context-checker`, skip Step 2, set STATE.md `<position>` as if checker passed (rationale: DESIGN-CONTEXT.md didn't change, last validation still holds), emit `design-context-checker skipped — <rationale>`.
|
|
125
|
-
- `spawn: true` → proceed to Step 2 as currently written.
|
|
126
|
-
|
|
127
|
-
**Note:** On first-run discover, the builder just wrote DESIGN-CONTEXT.md so the gate returns `spawn: true`. The gate meaningfully short-circuits only on re-runs where the builder made no changes.
|
|
128
|
-
|
|
129
|
-
**Parallel synthesizer note:** discover does not spawn parallel researchers in v1, so `skills/synthesize/` is not wired here. If future variants spawn N parallel interviewers/auto-detectors, wire synthesize between dispatch and collate as in `skills/map/` Step 3.5.
|
|
45
|
+
---
|
|
130
46
|
|
|
131
47
|
## Step 2 — Spawn design-context-checker
|
|
132
48
|
|
|
133
|
-
|
|
134
|
-
<required_reading>
|
|
135
|
-
@.design/STATE.md
|
|
136
|
-
@.design/DESIGN-CONTEXT.md
|
|
137
|
-
</required_reading>
|
|
49
|
+
Spawn `design-context-checker` with `<required_reading>` on STATE.md + DESIGN-CONTEXT.md. The agent validates DESIGN-CONTEXT.md across 6 dimensions and returns APPROVED or BLOCKED with per-dimension verdicts. Wait for `## CONTEXT CHECK COMPLETE`. Full prompt: `./discover-procedure.md` §Step 2.
|
|
138
50
|
|
|
139
|
-
|
|
140
|
-
6 dimensions. Return APPROVED or BLOCKED with per-dimension verdicts.
|
|
141
|
-
|
|
142
|
-
Emit `## CONTEXT CHECK COMPLETE` when done.
|
|
143
|
-
""")
|
|
144
|
-
|
|
145
|
-
Wait for `## CONTEXT CHECK COMPLETE`.
|
|
51
|
+
---
|
|
146
52
|
|
|
147
53
|
## Step 3 — Handle checker verdict
|
|
148
54
|
|
|
149
|
-
|
|
150
|
-
|
|
151
|
-
(re-spawn builder with specific fix instructions). Do not proceed to planning.
|
|
55
|
+
- **APPROVED** -> proceed to state update.
|
|
56
|
+
- **BLOCKED** -> present blocked dimensions to user, offer fix-and-retry loop (re-spawn builder with specific fix instructions). Do not proceed to planning.
|
|
152
57
|
|
|
153
58
|
---
|
|
154
59
|
|
|
155
60
|
## State Update (exit)
|
|
156
61
|
|
|
157
|
-
1. Set
|
|
158
|
-
2. Set
|
|
159
|
-
3. Update last_checkpoint
|
|
62
|
+
1. Set `<position>` `status=completed`, `task_progress=1/1`.
|
|
63
|
+
2. Set `<timestamps>` `discover_completed_at=<ISO 8601 now>`.
|
|
64
|
+
3. Update `last_checkpoint`. Write STATE.md.
|
|
160
65
|
|
|
161
66
|
---
|
|
162
67
|
|
|
163
68
|
## After Writing
|
|
164
69
|
|
|
165
|
-
|
|
166
|
-
━━━ Discovery complete ━━━
|
|
167
|
-
Saved: .design/DESIGN-CONTEXT.md
|
|
168
|
-
|
|
169
|
-
Baseline score: [N]/100 ([grade])
|
|
170
|
-
Key issues: [top issue 1], [top issue 2], [top issue 3]
|
|
171
|
-
|
|
172
|
-
Next: /get-design-done:plan
|
|
173
|
-
→ Decomposes your context into executable design tasks.
|
|
174
|
-
━━━━━━━━━━━━━━━━━━━━━━━━━
|
|
175
|
-
```
|
|
176
|
-
|
|
177
|
-
Do not proceed to planning automatically unless `--auto` was passed.
|
|
70
|
+
Print the "=== Discovery complete ===" block with saved path, baseline score, top key issues, and next step (`/get-design-done:plan`). Do not proceed to planning automatically unless `--auto` was passed. Template: `./discover-procedure.md` §After Writing.
|
|
178
71
|
|
|
179
72
|
## DISCOVER COMPLETE
|