prizmkit 1.1.80 → 1.1.82
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/bundled/VERSION.json +3 -3
- package/bundled/dev-pipeline/lib/heartbeat.sh +1 -1
- package/bundled/dev-pipeline/run-feature.sh +3 -3
- package/bundled/dev-pipeline/templates/bootstrap-tier1.md +2 -5
- package/bundled/dev-pipeline/templates/bootstrap-tier2.md +2 -5
- package/bundled/dev-pipeline/templates/bootstrap-tier3.md +2 -5
- package/bundled/dev-pipeline/templates/sections/failure-capture.md +0 -5
- package/bundled/skills/_metadata.json +1 -1
- package/bundled/skills/app-planner/SKILL.md +2 -9
- package/bundled/skills/app-planner/references/rules/backend/derivation-rules.md +10 -0
- package/bundled/skills/app-planner/references/rules/database/derivation-rules.md +9 -0
- package/bundled/skills/app-planner/references/rules/frontend/derivation-rules.md +10 -0
- package/bundled/skills/app-planner/references/rules/frontend/question-bank.md +17 -0
- package/bundled/skills/app-planner/references/rules/frontend/template.md +19 -0
- package/bundled/skills/app-planner/references/rules/mobile/derivation-rules.md +10 -0
- package/bundled/skills/bug-fix-workflow/SKILL.md +17 -24
- package/bundled/skills/bug-fix-workflow/references/bug-diagnosis.md +4 -29
- package/bundled/skills/bug-planner/SKILL.md +20 -12
- package/bundled/skills/bugfix-pipeline-launcher/SKILL.md +5 -67
- package/bundled/skills/bugfix-pipeline-launcher/references/configuration.md +50 -1
- package/bundled/skills/feature-pipeline-launcher/SKILL.md +16 -56
- package/bundled/skills/feature-pipeline-launcher/references/configuration.md +26 -0
- package/bundled/skills/feature-planner/SKILL.md +5 -9
- package/bundled/skills/feature-workflow/SKILL.md +22 -63
- package/bundled/skills/prizm-kit/SKILL.md +3 -2
- package/bundled/skills/prizmkit-code-review/SKILL.md +1 -1
- package/bundled/skills/prizmkit-committer/SKILL.md +3 -1
- package/bundled/skills/prizmkit-deploy/SKILL.md +1 -1
- package/bundled/skills/prizmkit-implement/SKILL.md +1 -1
- package/bundled/skills/prizmkit-prizm-docs/SKILL.md +1 -0
- package/bundled/skills/prizmkit-retrospective/SKILL.md +1 -1
- package/bundled/skills/recovery-workflow/SKILL.md +23 -73
- package/bundled/skills/refactor-pipeline-launcher/SKILL.md +6 -51
- package/bundled/skills/refactor-pipeline-launcher/references/configuration.md +34 -0
- package/bundled/skills/refactor-planner/SKILL.md +6 -8
- package/bundled/skills/refactor-workflow/SKILL.md +14 -75
- package/bundled/skills-windows/app-planner/SKILL.md +2 -9
- package/bundled/skills-windows/app-planner/references/rules/backend/derivation-rules.md +10 -0
- package/bundled/skills-windows/app-planner/references/rules/database/derivation-rules.md +9 -0
- package/bundled/skills-windows/app-planner/references/rules/frontend/derivation-rules.md +10 -0
- package/bundled/skills-windows/app-planner/references/rules/frontend/question-bank.md +17 -0
- package/bundled/skills-windows/app-planner/references/rules/frontend/template.md +19 -0
- package/bundled/skills-windows/app-planner/references/rules/mobile/derivation-rules.md +10 -0
- package/bundled/skills-windows/bug-fix-workflow/SKILL.md +17 -24
- package/bundled/skills-windows/bug-fix-workflow/references/bug-diagnosis.md +4 -29
- package/bundled/skills-windows/bug-planner/SKILL.md +20 -12
- package/bundled/skills-windows/bugfix-pipeline-launcher/SKILL.md +5 -63
- package/bundled/skills-windows/bugfix-pipeline-launcher/references/configuration.md +46 -1
- package/bundled/skills-windows/feature-pipeline-launcher/SKILL.md +17 -53
- package/bundled/skills-windows/feature-pipeline-launcher/references/configuration.md +22 -0
- package/bundled/skills-windows/feature-planner/SKILL.md +5 -9
- package/bundled/skills-windows/feature-workflow/SKILL.md +22 -63
- package/bundled/skills-windows/prizm-kit/SKILL.md +3 -2
- package/bundled/skills-windows/prizmkit-code-review/SKILL.md +1 -1
- package/bundled/skills-windows/prizmkit-committer/SKILL.md +3 -1
- package/bundled/skills-windows/prizmkit-deploy/SKILL.md +1 -1
- package/bundled/skills-windows/prizmkit-implement/SKILL.md +1 -1
- package/bundled/skills-windows/prizmkit-prizm-docs/SKILL.md +1 -0
- package/bundled/skills-windows/prizmkit-retrospective/SKILL.md +1 -1
- package/bundled/skills-windows/recovery-workflow/SKILL.md +23 -73
- package/bundled/skills-windows/refactor-pipeline-launcher/SKILL.md +7 -46
- package/bundled/skills-windows/refactor-pipeline-launcher/references/configuration.md +28 -0
- package/bundled/skills-windows/refactor-planner/SKILL.md +6 -8
- package/bundled/skills-windows/refactor-workflow/SKILL.md +14 -75
- package/package.json +1 -1
|
@@ -55,30 +55,24 @@ Pipeline commands (for reference — Phase 1.3 will present these as a selectabl
|
|
|
55
55
|
|
|
56
56
|
## Overview
|
|
57
57
|
|
|
58
|
-
|
|
59
|
-
|
|
60
|
-
|
|
61
|
-
|
|
62
|
-
|
|
63
|
-
|
|
64
|
-
|
|
65
|
-
|
|
66
|
-
|
|
67
|
-
|
|
68
|
-
|
|
69
|
-
|
|
70
|
-
|
|
71
|
-
|
|
72
|
-
|
|
73
|
-
|
|
74
|
-
|
|
75
|
-
|
|
76
|
-
│
|
|
77
|
-
└── Phase 2: Execute remaining steps
|
|
78
|
-
├── Read target workflow's SKILL.md
|
|
79
|
-
├── Read existing artifacts to restore context
|
|
80
|
-
└── Execute from inferred phase through completion
|
|
81
|
-
```
|
|
58
|
+
1. **Phase 0: Auto-detect**
|
|
59
|
+
1. Read current branch name
|
|
60
|
+
2. Scan characteristic artifacts
|
|
61
|
+
3. Match workflow signature → determine workflow type
|
|
62
|
+
4. Based on artifact presence → infer current phase
|
|
63
|
+
5. No match → reject and guide user
|
|
64
|
+
2. **Phase 1: Diagnose + report + user choice**
|
|
65
|
+
1. Check branch and working tree state
|
|
66
|
+
2. Scan all pipelines for failed/in-progress tasks
|
|
67
|
+
3. Find residual dev branches from failed tasks
|
|
68
|
+
4. Display diagnosis + detection results
|
|
69
|
+
5. If code changes exist → run test suite
|
|
70
|
+
6. If multiple failed tasks → ask user which to recover
|
|
71
|
+
7. User chooses: run-recovery.ps1 (recommended) | interactive | start fresh
|
|
72
|
+
3. **Phase 2: Execute remaining steps**
|
|
73
|
+
1. Read target workflow's SKILL.md
|
|
74
|
+
2. Read existing artifacts to restore context
|
|
75
|
+
3. Execute from inferred phase through completion
|
|
82
76
|
|
|
83
77
|
---
|
|
84
78
|
|
|
@@ -290,53 +284,19 @@ This step replaces the context that was lost when the AI session was interrupted
|
|
|
290
284
|
|
|
291
285
|
---
|
|
292
286
|
|
|
293
|
-
### 2.1
|
|
294
|
-
|
|
295
|
-
Phase inference table:
|
|
287
|
+
### 2.1 Per-Workflow Phase Inference
|
|
296
288
|
|
|
297
|
-
|
|
298
|
-
|---------------|------------|---------|
|
|
299
|
-
| On `fix/<BUG_ID>-*` branch, no artifacts | Phase 1: Deep Bug Diagnosis | Read bug description from `.prizmkit/plans/bug-fix-list.json`. Start interactive diagnosis Q&A |
|
|
300
|
-
| `fix-plan.md` exists, no code changes | Phase 4: Fix | Read fix-plan.md. Implement the fix following the plan |
|
|
301
|
-
| `fix-plan.md` + code changes exist | Phase 5: Review | Invoke `/prizmkit-code-review` on all changes |
|
|
302
|
-
| All docs + review passed | Phase 6: User Verification | Ask user to verify the fix works |
|
|
303
|
-
| All docs + committed | Phase 7: Merge Decision | Ask merge vs keep branch |
|
|
289
|
+
Read `${SKILL_DIR}/references/detection.md` for the per-workflow phase-inference tables (bug-fix / feature / refactor) that map detected state → the phase to resume from and its actions.
|
|
304
290
|
|
|
305
|
-
**
|
|
306
|
-
|
|
307
|
-
**Execution for each remaining phase**: Follow the bug-fix-workflow SKILL.md instructions exactly. Call the same prizmkit sub-commands (`/prizmkit-code-review`, `/prizmkit-committer`) at the same points.
|
|
291
|
+
**Bug-fix execution**: Follow the bug-fix-workflow SKILL.md instructions exactly. Call the same prizmkit sub-commands (`/prizmkit-code-review`, `/prizmkit-committer`) at the same points.
|
|
308
292
|
|
|
309
293
|
**Special handling**:
|
|
310
294
|
- If resuming from Phase 5 (Review) and tests are failing, fix test failures first (max 3 attempts). If unfixable, ask user whether to continue or restart.
|
|
311
295
|
- If resuming from Phase 4 (Fix), read fix-plan.md to understand the planned approach before writing code.
|
|
312
296
|
|
|
313
|
-
|
|
297
|
+
**Feature recovery**: Phases 3-4 are pipeline-driven. The main recovery value is avoiding re-brainstorming (Phase 1) when `.prizmkit/plans/feature-list.json` already exists.
|
|
314
298
|
|
|
315
|
-
|
|
316
|
-
|
|
317
|
-
Phase inference table:
|
|
318
|
-
|
|
319
|
-
| Detected State | Resume From | Actions |
|
|
320
|
-
|---------------|------------|---------|
|
|
321
|
-
| No `.prizmkit/plans/feature-list.json` | Phase 1: Brainstorm | Cannot recover conversation context. Start requirement clarification, but leverage any workspace content (README, existing code) for context |
|
|
322
|
-
| `.prizmkit/plans/feature-list.json` exists, no pipeline state | Phase 3: Launch | Invoke `feature-pipeline-launcher` to start the pipeline |
|
|
323
|
-
| `.prizmkit/plans/feature-list.json` + pipeline state exists | Phase 4: Monitor | Check pipeline status via `feature-pipeline-launcher` (Intent B: Check Status) |
|
|
324
|
-
|
|
325
|
-
**Note**: Feature-workflow recovery is simpler because Phases 3-4 are pipeline-driven. The main recovery value is avoiding re-brainstorming (Phase 1) when `.prizmkit/plans/feature-list.json` already exists.
|
|
326
|
-
|
|
327
|
-
---
|
|
328
|
-
|
|
329
|
-
### 2.3 Refactor-Workflow Recovery
|
|
330
|
-
|
|
331
|
-
Phase inference table (mirrors feature-workflow):
|
|
332
|
-
|
|
333
|
-
| Detected State | Resume From | Actions |
|
|
334
|
-
|---------------|------------|---------|
|
|
335
|
-
| No `.prizmkit/plans/refactor-list.json` | Phase 1: Brainstorm | Start refactoring goal clarification |
|
|
336
|
-
| `.prizmkit/plans/refactor-list.json` exists, no pipeline state | Phase 3: Launch | Invoke `refactor-pipeline-launcher` to start the pipeline |
|
|
337
|
-
| `.prizmkit/plans/refactor-list.json` + pipeline state exists | Phase 4: Monitor | Check pipeline status |
|
|
338
|
-
|
|
339
|
-
---
|
|
299
|
+
**Refactor recovery**: Mirrors feature-workflow (brainstorm → launch → monitor).
|
|
340
300
|
|
|
341
301
|
### 2.4 Post-Recovery Report
|
|
342
302
|
|
|
@@ -388,16 +348,6 @@ Recovery complete.
|
|
|
388
348
|
|
|
389
349
|
---
|
|
390
350
|
|
|
391
|
-
## Comparison with Previous Version
|
|
392
|
-
|
|
393
|
-
| Dimension | Previous | Current |
|
|
394
|
-
|-----------|----------|---------|
|
|
395
|
-
| Scope | Feature pipeline only | All 3 interactive workflows |
|
|
396
|
-
| Input | Required `F-XXX` feature ID | No input — fully auto-detect |
|
|
397
|
-
| Detection | Feature-specific (spec/plan/git) | Signature-based (branch + artifacts) |
|
|
398
|
-
| Strategy | Multi-option (5 scenarios × 2-4 options each) | Single path: report → confirm → execute |
|
|
399
|
-
| Intrusion | None | None (zero changes to target workflows) |
|
|
400
|
-
|
|
401
351
|
## Output
|
|
402
352
|
|
|
403
353
|
- Resumed and completed workflow from the breakpoint
|
|
@@ -156,28 +156,13 @@ Detect user intent from their message, then follow the corresponding workflow:
|
|
|
156
156
|
|
|
157
157
|
Wait for user confirmation before proceeding.
|
|
158
158
|
|
|
159
|
-
5. **Ask execution mode** (first user decision
|
|
159
|
+
5. **Ask execution mode** (first user decision):
|
|
160
160
|
|
|
161
|
-
|
|
161
|
+
Present the three execution modes from the **Execution Mode** section above as a single standalone `AskUserQuestion` call (exactly 1 question, multiSelect: false). Wait for the user's response before continuing to step 6.
|
|
162
162
|
|
|
163
|
-
|
|
163
|
+
Each `AskUserQuestion` round is a genuine gate: callers tend to merge interactive rounds and pre-fill answers, which skips real user decisions and produces wrong configs. Therefore: ask execution mode (step 5) and configuration (step 6) as SEPARATE `AskUserQuestion` calls, in order, and do not assemble or show the final command (step 7) until the user has answered both. If you find yourself writing the final command before the user has answered, STOP — you are violating this rule.
|
|
164
164
|
|
|
165
|
-
|
|
166
|
-
- Foreground (Recommended) — pipeline runs in the current session via `run-refactor.ps1 run`. Visible output and direct error feedback.
|
|
167
|
-
- Background daemon — pipeline runs fully detached via `launch-refactor-daemon.ps1`. Survives AI CLI session closure.
|
|
168
|
-
- Manual — display the final assembled commands only. Do not execute anything. User runs them on their own.
|
|
169
|
-
|
|
170
|
-
⚠️ STOP HERE and wait for user response before continuing to step 6.
|
|
171
|
-
|
|
172
|
-
6. **Ask configuration options** ⚠️ MANDATORY INTERACTIVE STEP — applies to ALL execution modes (Foreground, Background, AND Manual). This is a SEPARATE `AskUserQuestion` call from step 5. You MUST ask the user to configure options and WAIT for their response BEFORE proceeding to step 7.
|
|
173
|
-
|
|
174
|
-
⛔ **HARD STOP**: You MUST call `AskUserQuestion` with the 3 questions below and WAIT for the user's response. You MUST NOT:
|
|
175
|
-
- Combine step 5 and step 6 into one `AskUserQuestion` call (this is the most common violation — execution mode MUST be asked separately in step 5)
|
|
176
|
-
- Skip this step and jump to step 7
|
|
177
|
-
- Merge step 6 and step 7 into one response
|
|
178
|
-
- Assume default values and show the command without asking
|
|
179
|
-
- Show the command as text and ask "ready?" without presenting the options
|
|
180
|
-
If you find yourself writing the final command before the user has answered these 3 questions, STOP — you are violating this rule.
|
|
165
|
+
6. **Ask configuration options** — a SEPARATE `AskUserQuestion` call from step 5, applies to ALL execution modes (Foreground, Background, AND Manual).
|
|
181
166
|
|
|
182
167
|
Use `AskUserQuestion` to present ALL 3 configuration choices (the full 3-question budget goes to config, NOT shared with execution mode):
|
|
183
168
|
|
|
@@ -196,35 +181,11 @@ Detect user intent from their message, then follow the corresponding workflow:
|
|
|
196
181
|
|
|
197
182
|
Note: Refactor filter defaults to all refactor items (by priority order). If the user selects "Other" on any option, handle their custom input.
|
|
198
183
|
|
|
199
|
-
**If user chose "Yes" to Advanced config**,
|
|
200
|
-
|
|
201
|
-
**Question 1 — Session timeout** (multiSelect: false):
|
|
202
|
-
- None (default) — No timeout
|
|
203
|
-
- 30 min — `SESSION_TIMEOUT=1800`
|
|
204
|
-
- 1 hour — `SESSION_TIMEOUT=3600`
|
|
205
|
-
- 2 hours — `SESSION_TIMEOUT=7200`
|
|
206
|
-
|
|
207
|
-
**Question 2 — Stop on failure** (multiSelect: false):
|
|
208
|
-
- Off (default) — Pipeline continues to next task after failure
|
|
209
|
-
- On — Pipeline halts immediately when a task exhausts all retries (`STOP_ON_FAILURE=1`)
|
|
210
|
-
|
|
211
|
-
**Question 3 — Critic review** (multiSelect: false):
|
|
212
|
-
- Off (default) — Skip adversarial review
|
|
213
|
-
- On — Enable adversarial critic review: an independent AI agent reviews the refactor plan for completeness and the implementation for regressions, missing edge cases, and behavior violations. Adds ~5-10 min per refactor task.
|
|
214
|
-
|
|
215
|
-
**Question 4 — Reasoning effort** (multiSelect: false):
|
|
216
|
-
- Default (none) — Use CLI default
|
|
217
|
-
- low — Minimize reasoning, fastest output (`PRIZMKIT_EFFORT=low`)
|
|
218
|
-
- medium — Moderate reasoning (`PRIZMKIT_EFFORT=medium`)
|
|
219
|
-
- high — Thorough reasoning for complex tasks (`PRIZMKIT_EFFORT=high`)
|
|
220
|
-
- xhigh — Extensive reasoning (`PRIZMKIT_EFFORT=xhigh`)
|
|
221
|
-
- max — Maximum reasoning, Claude Code only (`PRIZMKIT_EFFORT=max`)
|
|
222
|
-
|
|
223
|
-
Default Critic to Off unless refactor items have `priority: "critical"` (in which case default to On).
|
|
184
|
+
**If user chose "Yes" to Advanced config**, run the advanced configuration round (a second `AskUserQuestion` round). It applies to a minority of sessions, so the full question set lives in `${SKILL_DIR}/references/configuration.md` → **Advanced Configuration Round**. Default Critic to Off unless refactor items have `priority: "critical"` (in which case default to On).
|
|
224
185
|
|
|
225
186
|
**Environment variable mapping** and **advanced environment variables** tables — read `${SKILL_DIR}/references/configuration.md` for the full translation tables (10 config→env mappings + 8 advanced variables).
|
|
226
187
|
|
|
227
|
-
|
|
188
|
+
STOP HERE and wait for user response before continuing to step 7.
|
|
228
189
|
|
|
229
190
|
7. **Show final command**: After user confirms configuration in step 6, assemble the complete command from execution mode + user-confirmed configuration, and present it to the user.
|
|
230
191
|
|
|
@@ -373,7 +334,7 @@ Notes:
|
|
|
373
334
|
|
|
374
335
|
### Error Handling
|
|
375
336
|
|
|
376
|
-
Read `${SKILL_DIR}/references/configuration.md` for the full
|
|
337
|
+
Read the error-handling table in `${SKILL_DIR}/references/configuration.md` for the full list of errors and recovery actions.
|
|
377
338
|
|
|
378
339
|
### Integration Notes
|
|
379
340
|
|
|
@@ -52,3 +52,31 @@ Not exposed in interactive menu, pass via `--env`:
|
|
|
52
52
|
| `opencli` not installed | Browser verification skipped for opencli refactors (non-blocking). Install opencli for Chrome session-based browser verification |
|
|
53
53
|
| Deploy session failed | Pipeline completed but deploy session exited non-zero. Check `.prizmkit/state/refactor/deploy/<session_id>/logs/session.log`. Retry manually: `/prizmkit-deploy`. |
|
|
54
54
|
| PowerShell execution policy blocks script | Run `Set-ExecutionPolicy -Scope Process Bypass` for the current terminal |
|
|
55
|
+
|
|
56
|
+
## Advanced Configuration Round
|
|
57
|
+
|
|
58
|
+
Only run this when the user answered "Yes" to the **Advanced config?** question in step 6. It applies to a minority of sessions.
|
|
59
|
+
|
|
60
|
+
Ask a second round of `AskUserQuestion` with these 4 questions:
|
|
61
|
+
|
|
62
|
+
**Question 1 — Session timeout** (multiSelect: false):
|
|
63
|
+
- None (default) — No timeout
|
|
64
|
+
- 30 min — `SESSION_TIMEOUT=1800`
|
|
65
|
+
- 1 hour — `SESSION_TIMEOUT=3600`
|
|
66
|
+
- 2 hours — `SESSION_TIMEOUT=7200`
|
|
67
|
+
|
|
68
|
+
**Question 2 — Stop on failure** (multiSelect: false):
|
|
69
|
+
- Off (default) — Pipeline continues to next task after failure
|
|
70
|
+
- On — Pipeline halts immediately when a task exhausts all retries (`STOP_ON_FAILURE=1`)
|
|
71
|
+
|
|
72
|
+
**Question 3 — Critic review** (multiSelect: false):
|
|
73
|
+
- Off (default) — Skip adversarial review
|
|
74
|
+
- On — Enable adversarial critic review: an independent AI agent reviews the refactor plan for completeness and the implementation for regressions, missing edge cases, and behavior violations. Adds ~5-10 min per refactor task.
|
|
75
|
+
|
|
76
|
+
**Question 4 — Reasoning effort** (multiSelect: false):
|
|
77
|
+
- Default (none) — Use CLI default
|
|
78
|
+
- low — Minimize reasoning, fastest output (`PRIZMKIT_EFFORT=low`)
|
|
79
|
+
- medium — Moderate reasoning (`PRIZMKIT_EFFORT=medium`)
|
|
80
|
+
- high — Thorough reasoning for complex tasks (`PRIZMKIT_EFFORT=high`)
|
|
81
|
+
- xhigh — Extensive reasoning (`PRIZMKIT_EFFORT=xhigh`)
|
|
82
|
+
- max — Maximum reasoning, Claude Code only (`PRIZMKIT_EFFORT=max`)
|
|
@@ -78,7 +78,7 @@ Do NOT use this skill when the user wants to:
|
|
|
78
78
|
- Read `${SKILL_DIR}/references/behavior-preservation.md` for preservation strategy selection
|
|
79
79
|
|
|
80
80
|
4. **Load on-demand references when triggered**:
|
|
81
|
-
- Validation errors or interrupted session ->
|
|
81
|
+
- Validation errors or interrupted session -> see the "Error Recovery & Resume" section below
|
|
82
82
|
|
|
83
83
|
5. **Define the PowerShell Python helper before running validation scripts**:
|
|
84
84
|
```powershell
|
|
@@ -294,8 +294,6 @@ See `${SKILL_DIR}/references/planning-phases.md` for the full 5-step review chec
|
|
|
294
294
|
| **CP-RP-5** | Completeness OK | DAG valid, preservation strategies declared, no gaps | 6 |
|
|
295
295
|
| **CP-RP-6** | Output Valid | `.prizmkit/plans/refactor-list.json` passes validation script | 7 |
|
|
296
296
|
|
|
297
|
-
**Resume Detection**: If existing artifacts found (partial `.prizmkit/plans/refactor-list.json`, draft `refactor-list.draft.json` in `.prizmkit/plans/`), offer to resume from the appropriate checkpoint.
|
|
298
|
-
|
|
299
297
|
## Output Rules
|
|
300
298
|
|
|
301
299
|
`.prizmkit/plans/refactor-list.json` must satisfy:
|
|
@@ -363,6 +361,8 @@ Key behaviors:
|
|
|
363
361
|
|
|
364
362
|
### Resume Detection
|
|
365
363
|
|
|
364
|
+
If existing artifacts are found, offer to resume from the appropriate checkpoint/phase:
|
|
365
|
+
|
|
366
366
|
| Artifact Found | Resume From |
|
|
367
367
|
|---------------|------------|
|
|
368
368
|
| Nothing | Phase 1: Project Context |
|
|
@@ -370,18 +370,16 @@ Key behaviors:
|
|
|
370
370
|
| Partial `.prizmkit/plans/refactor-list.json` | Phase 6: Completeness Review |
|
|
371
371
|
| Valid `.prizmkit/plans/refactor-list.json` | Mode D: Summary |
|
|
372
372
|
|
|
373
|
-
|
|
373
|
+
### Session Exit Gate
|
|
374
374
|
|
|
375
375
|
Prevent accidental session exit without deliverable completion.
|
|
376
376
|
|
|
377
|
-
|
|
378
|
-
Activate exit gate when ALL are true:
|
|
377
|
+
**Trigger conditions** — activate the exit gate when ALL are true:
|
|
379
378
|
- User invoked `/refactor-planner` (not just mentioned refactoring)
|
|
380
379
|
- Current phase < Phase 7 (validation not yet passed)
|
|
381
380
|
- No valid `.prizmkit/plans/refactor-list.json` has been written in this session
|
|
382
381
|
|
|
383
|
-
|
|
384
|
-
When the session appears to be ending:
|
|
382
|
+
**Gate behavior** — when the session appears to be ending:
|
|
385
383
|
1. **Remind**: "You set out to produce `.prizmkit/plans/refactor-list.json` but we haven't completed it yet."
|
|
386
384
|
2. **Offer 3 options**:
|
|
387
385
|
- **(a) Continue to completion** — resume from current phase
|
|
@@ -26,39 +26,12 @@ User says:
|
|
|
26
26
|
|
|
27
27
|
## Overview
|
|
28
28
|
|
|
29
|
-
|
|
30
|
-
refactor-workflow <target / goals>
|
|
31
|
-
│
|
|
32
|
-
├── Phase 1: Brainstorm → clarify type → collect materials → parallel deep read → discuss plan
|
|
33
|
-
│
|
|
34
|
-
├── Phase 2: Plan → refactor-planner → .prizmkit/plans/refactor-list.json
|
|
35
|
-
│
|
|
36
|
-
├── Phase 3: Launch → refactor-pipeline-launcher → pipeline execution
|
|
37
|
-
│
|
|
38
|
-
└── Phase 4: Monitor → track progress → report results
|
|
39
|
-
```
|
|
40
|
-
|
|
41
|
-
### What This Skill Does
|
|
42
|
-
|
|
43
|
-
| Phase | Action | Result |
|
|
44
|
-
|-------|--------|--------|
|
|
45
|
-
| 1 | **Brainstorm** — clarify type, collect reference materials, parallel deep read code & docs, discuss plan grounded in real code | Fully clarified refactoring goals |
|
|
46
|
-
| 2 | Call `refactor-planner` with clarified goals | `.prizmkit/plans/refactor-list.json` with N refactor items |
|
|
47
|
-
| 3 | Call `refactor-pipeline-launcher` | Pipeline started (execution mode chosen by user via launcher) |
|
|
48
|
-
| 4 | Monitor progress | Status updates, completion report |
|
|
49
|
-
|
|
50
|
-
### Why This Skill Exists
|
|
51
|
-
|
|
52
|
-
Without this skill, users must:
|
|
53
|
-
1. Figure out all refactoring goals and scope themselves
|
|
54
|
-
2. Invoke `refactor-planner` → wait for .prizmkit/plans/refactor-list.json
|
|
55
|
-
3. Invoke `refactor-pipeline-launcher` → wait for pipeline start
|
|
56
|
-
4. Manually check progress
|
|
29
|
+
Four phases take a rough refactoring idea to verified, behavior-preserving code so the user doesn't have to manually chain planner → launcher → monitoring:
|
|
57
30
|
|
|
58
|
-
|
|
59
|
-
|
|
60
|
-
|
|
61
|
-
|
|
31
|
+
1. **Brainstorm** — clarify type, collect reference materials, parallel deep read code & docs, discuss plan grounded in real code → fully clarified refactoring goals
|
|
32
|
+
2. **Plan** — call `refactor-planner` → `.prizmkit/plans/refactor-list.json` with N refactor items
|
|
33
|
+
3. **Launch** — call `refactor-pipeline-launcher` → pipeline started (execution mode chosen by user via launcher)
|
|
34
|
+
4. **Monitor** — track progress → status updates, completion report
|
|
62
35
|
|
|
63
36
|
### Branch Management
|
|
64
37
|
|
|
@@ -80,21 +53,9 @@ Natural language description of the refactoring goals. Can be:
|
|
|
80
53
|
|
|
81
54
|
Flow: brainstorm → refactor-planner → refactor-pipeline-launcher → monitor
|
|
82
55
|
|
|
83
|
-
**Mode B: From existing
|
|
56
|
+
**Mode B: From existing `.prizmkit/plans/refactor-list.json`** — user says "run pipeline from existing file", or the file already exists. Skip ahead per the [Resume table](#resume--interruption-recovery).
|
|
84
57
|
|
|
85
|
-
|
|
86
|
-
- Skip brainstorm and `refactor-planner` (file already exists)
|
|
87
|
-
- Invoke `refactor-pipeline-launcher` directly
|
|
88
|
-
- Monitor and report progress
|
|
89
|
-
|
|
90
|
-
**Mode C: Incremental (add to existing refactor plan)**
|
|
91
|
-
|
|
92
|
-
When user says "add more refactors" or the project already has a .prizmkit/plans/refactor-list.json:
|
|
93
|
-
- Brainstorm new refactoring goals with the user
|
|
94
|
-
- Invoke `refactor-planner` in incremental mode (reads existing .prizmkit/plans/refactor-list.json)
|
|
95
|
-
- Append new refactor items to existing list
|
|
96
|
-
- Invoke `refactor-pipeline-launcher`
|
|
97
|
-
- Monitor and report progress
|
|
58
|
+
**Mode C: Incremental (add to existing refactor plan)** — user says "add more refactors". Brainstorm new goals, then invoke `refactor-planner` in incremental mode (appends to existing list) → launch → monitor.
|
|
98
59
|
|
|
99
60
|
---
|
|
100
61
|
|
|
@@ -176,28 +137,6 @@ After confirming refactoring goals, assess whether this refactor needs the full
|
|
|
176
137
|
- Existing tests fully cover the affected code paths
|
|
177
138
|
- No risk of behavior change
|
|
178
139
|
|
|
179
|
-
**User choice required (mandatory)** — Use `AskUserQuestion` to present interactive selectable options:
|
|
180
|
-
|
|
181
|
-
```
|
|
182
|
-
AskUserQuestion:
|
|
183
|
-
question: "This refactoring appears straightforward. How would you like to proceed?"
|
|
184
|
-
header: "Approach"
|
|
185
|
-
options:
|
|
186
|
-
- label: "Refactor now (fast path)"
|
|
187
|
-
description: "Plan and refactor directly in this session using /prizmkit-plan + /prizmkit-implement"
|
|
188
|
-
- label: "Add to refactor list (pipeline)"
|
|
189
|
-
description: "Generate .prizmkit/plans/refactor-list.json via refactor-planner, then launch pipeline for autonomous execution"
|
|
190
|
-
```
|
|
191
|
-
|
|
192
|
-
- **Refactor now** → Fast Path Workflow:
|
|
193
|
-
1. Invoke `/prizmkit-plan` with the refactoring goals → generates `spec.md` + `plan.md`
|
|
194
|
-
2. Invoke `/prizmkit-implement` to execute the plan (behavior preservation verified by tests)
|
|
195
|
-
3. After implementation, run `/prizmkit-code-review` for quality check
|
|
196
|
-
4. Commit via `/prizmkit-committer` with `refactor(<scope>):` prefix
|
|
197
|
-
5. Run `/prizmkit-retrospective` to sync `.prizmkit/prizm-docs/`
|
|
198
|
-
6. **End workflow** — skip Phase 2/3/4
|
|
199
|
-
- **Add to refactor list** → Continue to Phase 2 (Plan via pipeline)
|
|
200
|
-
|
|
201
140
|
**Complex refactor → Planning Path** (ANY is true):
|
|
202
141
|
- Cross-module impact (>2 modules affected)
|
|
203
142
|
- Public API or interface changes required
|
|
@@ -206,21 +145,21 @@ AskUserQuestion:
|
|
|
206
145
|
- Insufficient test coverage in target area (risk of hidden behavior changes)
|
|
207
146
|
- Requires coordination with other ongoing work
|
|
208
147
|
|
|
209
|
-
**User choice required (mandatory)** — Use `AskUserQuestion` to present interactive selectable options:
|
|
148
|
+
**User choice required (mandatory)** — Use `AskUserQuestion` to present interactive selectable options. Tailor the question text to the assessment ("This refactoring appears straightforward…" for simple, "This refactoring is complex and will benefit from structured planning…" for complex), but the two options are always the same:
|
|
210
149
|
|
|
211
150
|
```
|
|
212
151
|
AskUserQuestion:
|
|
213
|
-
question:
|
|
152
|
+
question: <tailored to simple vs complex assessment>
|
|
214
153
|
header: "Approach"
|
|
215
154
|
options:
|
|
216
|
-
- label: "
|
|
217
|
-
description: "
|
|
155
|
+
- label: "Refactor now (fast path)"
|
|
156
|
+
description: "Plan and refactor directly in this session using /prizmkit-plan + /prizmkit-implement"
|
|
218
157
|
- label: "Add to refactor list (pipeline)"
|
|
219
158
|
description: "Generate .prizmkit/plans/refactor-list.json via refactor-planner, then launch pipeline for autonomous execution"
|
|
220
159
|
```
|
|
221
160
|
|
|
222
|
-
- **
|
|
223
|
-
- **Add to refactor list** → Continue to Phase 2 (Plan via pipeline)
|
|
161
|
+
- **Refactor now** → Fast Path: `/prizmkit-plan` (goals → `spec.md` + `plan.md`) → `/prizmkit-implement` (behavior preservation verified by tests) → `/prizmkit-code-review` → `/prizmkit-committer` (`refactor(<scope>):` prefix) → `/prizmkit-retrospective` (sync `.prizmkit/prizm-docs/`). **End workflow** — skip Phase 2/3/4.
|
|
162
|
+
- **Add to refactor list** → Continue to Phase 2 (Plan via pipeline).
|
|
224
163
|
|
|
225
164
|
**NEVER proceed without explicit user confirmation via `AskUserQuestion`. Do NOT render options as plain text — the user must be able to click/select.**
|
|
226
165
|
|
|
@@ -251,7 +190,7 @@ AskUserQuestion:
|
|
|
251
190
|
|
|
252
191
|
**CHECKPOINT CP-RW-1**: `.prizmkit/plans/refactor-list.json` generated and validated.
|
|
253
192
|
|
|
254
|
-
**If user says `--from <file>`**:
|
|
193
|
+
**If `.prizmkit/plans/refactor-list.json` already exists or user says `--from <file>`**: skip ahead per the [Resume table](#resume--interruption-recovery).
|
|
255
194
|
|
|
256
195
|
---
|
|
257
196
|
|