prizmkit 1.1.106 → 1.1.108
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/bin/create-prizmkit.js +4 -0
- package/bundled/VERSION.json +3 -3
- package/bundled/dev-pipeline/prizmkit_runtime/config.py +81 -14
- package/bundled/dev-pipeline/prizmkit_runtime/heartbeat.py +19 -8
- package/bundled/dev-pipeline/prizmkit_runtime/interoperability.py +0 -1
- package/bundled/dev-pipeline/prizmkit_runtime/launch_profiles.py +18 -0
- package/bundled/dev-pipeline/prizmkit_runtime/runner_bookkeeping.py +22 -2
- package/bundled/dev-pipeline/prizmkit_runtime/runner_classification.py +56 -5
- package/bundled/dev-pipeline/prizmkit_runtime/runner_models.py +0 -8
- package/bundled/dev-pipeline/prizmkit_runtime/runner_recovery.py +11 -1
- package/bundled/dev-pipeline/prizmkit_runtime/runners.py +298 -63
- package/bundled/dev-pipeline/prizmkit_runtime/sessions.py +243 -37
- package/bundled/dev-pipeline/scripts/continuation.py +0 -5
- package/bundled/dev-pipeline/scripts/generate-bootstrap-prompt.py +10 -82
- package/bundled/dev-pipeline/scripts/generate-bugfix-prompt.py +7 -64
- package/bundled/dev-pipeline/scripts/generate-refactor-prompt.py +40 -63
- package/bundled/dev-pipeline/scripts/utils.py +123 -0
- package/bundled/dev-pipeline/templates/agent-prompts/critic-plan-challenge.md +4 -3
- package/bundled/dev-pipeline/templates/agent-prompts/reviewer-review.md +5 -6
- package/bundled/dev-pipeline/templates/bootstrap-prompt.md +1 -1
- package/bundled/dev-pipeline/templates/bootstrap-tier1.md +36 -35
- package/bundled/dev-pipeline/templates/bootstrap-tier2.md +53 -80
- package/bundled/dev-pipeline/templates/bootstrap-tier3.md +43 -78
- package/bundled/dev-pipeline/templates/bugfix-bootstrap-prompt.md +5 -17
- package/bundled/dev-pipeline/templates/refactor-bootstrap-prompt.md +6 -18
- package/bundled/dev-pipeline/templates/sections/checkpoint-system.md +14 -48
- package/bundled/dev-pipeline/templates/sections/context-budget-rules.md +7 -14
- package/bundled/dev-pipeline/templates/sections/feature-context.md +0 -4
- package/bundled/dev-pipeline/templates/sections/phase-browser-verification-auto.md +0 -7
- package/bundled/dev-pipeline/templates/sections/phase-browser-verification-opencli.md +0 -7
- package/bundled/dev-pipeline/templates/sections/phase-browser-verification.md +0 -7
- package/bundled/dev-pipeline/templates/sections/phase-commit-full.md +0 -14
- package/bundled/dev-pipeline/templates/sections/phase-commit.md +0 -14
- package/bundled/dev-pipeline/templates/sections/phase-context-snapshot-agent-suffix.md +0 -7
- package/bundled/dev-pipeline/templates/sections/phase-context-snapshot-base.md +0 -2
- package/bundled/dev-pipeline/templates/sections/phase-context-snapshot-lite-suffix.md +0 -7
- package/bundled/dev-pipeline/templates/sections/phase-critic-plan-full.md +6 -8
- package/bundled/dev-pipeline/templates/sections/phase-critic-plan.md +3 -7
- package/bundled/dev-pipeline/templates/sections/phase-implement-agent.md +4 -16
- package/bundled/dev-pipeline/templates/sections/phase-implement-full.md +4 -17
- package/bundled/dev-pipeline/templates/sections/phase-implement-lite.md +0 -7
- package/bundled/dev-pipeline/templates/sections/phase-plan-agent.md +2 -9
- package/bundled/dev-pipeline/templates/sections/phase-plan-lite.md +1 -8
- package/bundled/dev-pipeline/templates/sections/phase-prizmkit-test.md +15 -16
- package/bundled/dev-pipeline/templates/sections/phase-review-agent.md +3 -10
- package/bundled/dev-pipeline/templates/sections/phase-review-full.md +3 -10
- package/bundled/dev-pipeline/templates/sections/phase-specify-plan-full.md +2 -9
- package/bundled/dev-pipeline/templates/sections/phase0-init.md +0 -7
- package/bundled/dev-pipeline/templates/sections/phase0-test-baseline.md +2 -15
- package/bundled/dev-pipeline/tests/conftest.py +0 -1
- package/bundled/dev-pipeline/tests/test_generate_bootstrap_prompt.py +395 -21
- package/bundled/dev-pipeline/tests/test_generate_bugfix_prompt.py +50 -14
- package/bundled/dev-pipeline/tests/test_generate_refactor_prompt.py +154 -2
- package/bundled/dev-pipeline/tests/test_python_runner_parity.py +431 -3
- package/bundled/dev-pipeline/tests/test_unified_cli.py +857 -5
- package/bundled/skills/_metadata.json +4 -4
- package/bundled/skills/app-planner/SKILL.md +1 -1
- package/bundled/skills/app-planner/references/architecture-decisions.md +1 -1
- package/bundled/skills/bug-fix-workflow/SKILL.md +174 -128
- package/bundled/skills/bug-planner/SKILL.md +101 -15
- package/bundled/skills/bug-planner/references/severity-rules.md +20 -8
- package/bundled/skills/feature-planner/SKILL.md +87 -8
- package/bundled/skills/feature-planner/assets/planning-guide.md +34 -10
- package/bundled/skills/feature-workflow/SKILL.md +100 -81
- package/bundled/skills/{prizm-kit → prizmkit}/SKILL.md +1 -1
- package/bundled/skills/prizmkit-code-review/SKILL.md +12 -13
- package/bundled/skills/prizmkit-code-review/references/reviewer-agent-prompt.md +10 -10
- package/bundled/skills/refactor-planner/SKILL.md +88 -7
- package/bundled/skills/refactor-planner/references/fast-path.md +2 -1
- package/bundled/skills/refactor-planner/references/planning-phases.md +29 -11
- package/bundled/skills/refactor-workflow/SKILL.md +116 -57
- package/bundled/templates/project-memory-template.md +1 -1
- package/package.json +1 -1
- package/src/ai-cli-launch.js +194 -0
- package/src/config.js +26 -14
- package/src/index.js +11 -44
- package/src/platforms.js +10 -3
- package/src/prompts.js +60 -2
- package/src/scaffold.js +26 -14
- package/bundled/dev-pipeline/scripts/prizmkit-test-gate.py +0 -446
- package/bundled/dev-pipeline/templates/agent-prompts/dev-fix.md +0 -7
- package/bundled/dev-pipeline/templates/agent-prompts/dev-resume.md +0 -5
- package/bundled/dev-pipeline/templates/sections/log-size-awareness.md +0 -20
- package/bundled/dev-pipeline/tests/test_prizmkit_test_gate.py +0 -107
- package/bundled/skills/prizmkit-code-review/references/dev-agent-prompt.md +0 -30
|
@@ -51,9 +51,22 @@ When the user provides detailed specifications, rules, or implementation require
|
|
|
51
51
|
1. **Verbatim preservation**: The user's exact wording MUST be preserved in `description` and `acceptance_criteria` fields. Do NOT paraphrase, summarize, abstract, or simplify.
|
|
52
52
|
2. **No autonomous simplification**: A 200-word user specification must NOT become a 30-word description. Match the detail level of the user's input.
|
|
53
53
|
3. **Clarify, don't assume**: If any user-provided rule is ambiguous or potentially conflicts with another, ASK the user to clarify. No limit on clarification rounds. Do NOT proceed with unresolved ambiguities.
|
|
54
|
-
4. **
|
|
55
|
-
-
|
|
56
|
-
-
|
|
54
|
+
4. **Task-scoped `user_context`**: In single-refactor planning, write the relevant user-provided materials into that refactor item's `user_context`. In multi-refactor planning, multi-refactor `user_context` must contain only the user text, file references, goals, and constraints relevant to that specific refactor item.
|
|
55
|
+
- Preserve the matching user wording verbatim, but scope it to the item it belongs to.
|
|
56
|
+
- Do NOT copy the full multi-refactor request into every item.
|
|
57
|
+
- Ensure there are no unrelated sibling refactor goals, target files, constraints, desired end states, or supplementary materials in a refactor item's `user_context`.
|
|
58
|
+
- Shared global context may be copied to every item only when it is explicitly applicable to all generated items.
|
|
59
|
+
- If a goal, constraint, target file, or architecture note applies to only one refactor item, attach it only to that matching item.
|
|
60
|
+
- If applicability is unclear, ask the user to map the material to the correct refactor item before generating the final list.
|
|
61
|
+
5. **`user_context` format**:
|
|
62
|
+
- Refactor-specific supplementary content, goals, or constraints → store as-is (verbatim text) on the matching refactor item only
|
|
63
|
+
- Refactor-specific file references → store as path string, e.g. `src/auth/login.ts:42-78` or `src/utils/validate.ts — focus on validateEmail function`
|
|
64
|
+
- Truly shared global constraints → repeat only when they apply to every generated refactor item
|
|
65
|
+
|
|
66
|
+
**Multi-refactor isolation example:**
|
|
67
|
+
- User request includes: `Refactor A: extract auth validation into middleware`, `Refactor B: split dashboard chart utilities`, and `Shared: preserve all public APIs`.
|
|
68
|
+
- The auth middleware refactor `user_context` includes `Refactor A: extract auth validation into middleware` and the shared API constraint; it MUST NOT include `Refactor B: split dashboard chart utilities`.
|
|
69
|
+
- The chart utilities refactor `user_context` includes `Refactor B: split dashboard chart utilities` and the shared API constraint; it MUST NOT include `Refactor A: extract auth validation into middleware`.
|
|
57
70
|
|
|
58
71
|
## When to Use
|
|
59
72
|
- "Plan refactoring", "Scope a restructuring"
|
|
@@ -252,10 +265,13 @@ See `${SKILL_DIR}/references/planning-phases.md` for the full 5-step review chec
|
|
|
252
265
|
```
|
|
253
266
|
The script fills in defaults (`$schema`, `created_at`, `created_by`), validates all fields, and writes the final file only if validation passes.
|
|
254
267
|
3. If validation fails -> fix the draft and retry (max 3 attempts)
|
|
255
|
-
4. If validation passes ->
|
|
268
|
+
4. If validation passes -> run the mandatory Post-Generation Plan Review Gate (see §Post-Generation Plan Review Gate below); apply accepted fixes to the draft/source plan, regenerate, and revalidate until pass
|
|
269
|
+
5. If review and validation pass -> present final summary
|
|
256
270
|
|
|
257
271
|
**CHECKPOINT CP-RP-6**: `.prizmkit/plans/refactor-list.json` generated and validated.
|
|
258
272
|
|
|
273
|
+
**CHECKPOINT CP-RP-7**: `/prizmkit-code-review` reviewed only newly added/changed refactor entries; accepted fixes were regenerated and revalidated.
|
|
274
|
+
|
|
259
275
|
## Checkpoints (Mandatory Gates)
|
|
260
276
|
|
|
261
277
|
| Checkpoint | Artifact/State | Criteria | Phase |
|
|
@@ -267,6 +283,32 @@ See `${SKILL_DIR}/references/planning-phases.md` for the full 5-step review chec
|
|
|
267
283
|
| **CP-RP-4** | Items Confirmed | User confirmed/modified/skipped each item | 5 |
|
|
268
284
|
| **CP-RP-5** | Completeness OK | DAG valid, preservation strategies declared, no gaps | 6 |
|
|
269
285
|
| **CP-RP-6** | Output Valid | `.prizmkit/plans/refactor-list.json` passes validation script | 7 |
|
|
286
|
+
| **CP-RP-7** | Plan Review Passed | `/prizmkit-code-review` reviewed only newly added/changed refactor entries; accepted fixes were regenerated and revalidated | 7 |
|
|
287
|
+
|
|
288
|
+
## Post-Generation Plan Review Gate
|
|
289
|
+
|
|
290
|
+
User requirement preserved verbatim: `每次完成任务规划的时候,应该调用/prizmkit-code-review 对新增的计划内容进行review,确保生成新增的计划的正确行和完整性。`
|
|
291
|
+
|
|
292
|
+
Run this gate **after** `.prizmkit/plans/refactor-list.json` passes the validation/generate script and **before** the final handoff summary recommends `refactor-pipeline-launcher`. This gate applies to full workflow, Mode B from analysis, and fast-path refactor planning.
|
|
293
|
+
|
|
294
|
+
1. **Identify review scope**: compare the final validated refactor list against the pre-session list when one existed. Review only newly added or changed refactor entries. Do not re-review unchanged historical entries except when needed to verify dependency references or behavior-preservation interactions.
|
|
295
|
+
2. **Invoke `/prizmkit-code-review` as a planning-quality gate** with an explicit prompt that this is a refactor-plan-entry review, not an implementation review. The review must check:
|
|
296
|
+
- schema correctness and compatibility with `dev-pipeline-refactor-list-v1`
|
|
297
|
+
- dependency/DAG soundness, including safe behavior-preserving order
|
|
298
|
+
- description completeness and headless execution readiness
|
|
299
|
+
- acceptance criteria specificity and measurable behavior-preservation verification
|
|
300
|
+
- `behavior_preservation` strategy quality and target scope consistency
|
|
301
|
+
- preservation of user-provided wording in `description`, `acceptance_criteria`, and `user_context`
|
|
302
|
+
- task-scoped `user_context` isolation: no unrelated sibling refactor goals, target files, constraints, or supplementary materials, and shared context appears on multiple items only when explicitly global
|
|
303
|
+
- priority/complexity calibration: every reviewed refactor has a short priority rationale and complexity rationale, medium/low assignments are allowed when criteria fit, and high/high is not used merely because the work is cleanup, framework-related, multi-file, or user-requested
|
|
304
|
+
3. **Keep the review planning-only**: do NOT start `refactor-pipeline-launcher`, do NOT run the refactor pipeline, do NOT run tests/builds/installs, and do NOT implement source-code refactors.
|
|
305
|
+
4. **Apply accepted review fixes through the draft/source plan**: when a finding is accepted, update `.prizmkit/plans/refactor-list.draft.json` or the in-memory draft representation first, then regenerate the final file with:
|
|
306
|
+
```bash
|
|
307
|
+
python3 ${SKILL_DIR}/scripts/validate-and-generate-refactor.py generate --input .prizmkit/plans/refactor-list.draft.json --output .prizmkit/plans/refactor-list.json
|
|
308
|
+
```
|
|
309
|
+
Do not hand-patch the final JSON as the source of truth; map final-output findings back to draft entries before regeneration.
|
|
310
|
+
5. **Revalidate after every accepted fix batch**: the generate script must pass again before the planner can proceed. If validation fails, fix the draft and rerun until pass.
|
|
311
|
+
6. **Report the gate outcome** in the final summary: include `/prizmkit-code-review` verdict, reviewed refactor IDs, accepted fixes (or "none"), and final validation result.
|
|
270
312
|
|
|
271
313
|
## Output Rules
|
|
272
314
|
|
|
@@ -307,6 +349,44 @@ Browser verification is a feature-pipeline capability only. Refactors use `behav
|
|
|
307
349
|
|
|
308
350
|
---
|
|
309
351
|
|
|
352
|
+
## Priority and Complexity Calibration (Mandatory)
|
|
353
|
+
|
|
354
|
+
Refactor planning must not collapse into uniform `Priority: high` and `Complexity: high`. Refactoring work can be low or medium priority even when it touches several files. Do not use high/high merely because code needs cleanup, the user cares about maintainability, the target is framework code, or multiple files are involved.
|
|
355
|
+
|
|
356
|
+
### Priority Rubric
|
|
357
|
+
|
|
358
|
+
| Priority | Use When | Do NOT Use Merely Because |
|
|
359
|
+
|----------|----------|---------------------------|
|
|
360
|
+
| `critical` | Behavior-preserving restructure is required to unblock an imminent release, security/compliance fix, or safe migration with no acceptable workaround. | The refactor is large or architectural in wording. |
|
|
361
|
+
| `high` | Current structure blocks core feature delivery, creates ongoing correctness/security/data-risk, prevents supported runtime upgrades, or many downstream tasks cannot proceed. | The code is messy, multi-file, or important to the user. |
|
|
362
|
+
| `medium` | Refactor improves maintainability, testability, or dependency boundaries for planned work but does not block current operation. | Medium is the normal default for useful refactors. |
|
|
363
|
+
| `low` | Local cleanup, rename, simplification, documentation-oriented organization, or isolated code-health improvement with low behavior risk. | Small scope alone does not make it low if it blocks a high-impact outcome. |
|
|
364
|
+
|
|
365
|
+
### Complexity Rationale Rules
|
|
366
|
+
|
|
367
|
+
Assign complexity from concrete refactor scope indicators and include a short rationale:
|
|
368
|
+
- file count and expected import/update sites
|
|
369
|
+
- cross-module scope and dependency depth
|
|
370
|
+
- public API/interface or import-path change
|
|
371
|
+
- existing test coverage and behavior-preservation strategy
|
|
372
|
+
- behavior-preservation risk and snapshot/manual verification needs
|
|
373
|
+
- migration impact, rollout phases, or compatibility shims
|
|
374
|
+
|
|
375
|
+
Use `low` for 1-3 files in one module with good tests and no public API/import-path change. Use `medium` for 4-8 files or two modules with bounded public impact and clear behavior-preservation tests. Use `high` only for 9+ files, 3+ modules, public API/import-path migration, weak test coverage, manual preservation risk, or cross-module migration impact.
|
|
376
|
+
|
|
377
|
+
### Required Planning Review Summary
|
|
378
|
+
|
|
379
|
+
Before writing `.prizmkit/plans/refactor-list.json`, show a calibration table with rationale columns:
|
|
380
|
+
|
|
381
|
+
```
|
|
382
|
+
ID | Title | Priority | Priority Rationale | Complexity | Complexity Rationale
|
|
383
|
+
R-001 | Break auth/user cycle | high | Blocks feature work; core risk | high | 3 modules + public imports + weak tests
|
|
384
|
+
R-002 | Extract chart format helpers | medium | Maintainability for planned UI | medium | 5 files; bounded helper API; tests exist
|
|
385
|
+
R-003 | Rename local temp variable | low | Local readability only | low | One file; no interface or behavior change
|
|
386
|
+
```
|
|
387
|
+
|
|
388
|
+
Anti-pattern rule: do not assign every generated item `Priority: high` and `Complexity: high` unless each item independently satisfies the documented high-priority and high-complexity criteria and the rationale names those criteria.
|
|
389
|
+
|
|
310
390
|
## Refactoring-Specific Features
|
|
311
391
|
|
|
312
392
|
### Behavior Preservation Check
|
|
@@ -319,7 +399,7 @@ Auto-detect inter-item dependencies and enforce safe ordering. Read `${SKILL_DIR
|
|
|
319
399
|
|
|
320
400
|
### Complexity Assessment
|
|
321
401
|
|
|
322
|
-
Assess each item's complexity
|
|
402
|
+
Assess each item's complexity from concrete indicators: file count, cross-module scope, public API/import-path changes, dependency depth, test coverage, behavior-preservation risk, and migration impact. Read `${SKILL_DIR}/references/planning-phases.md` for the full complexity assessment criteria and scoring rules. Every confirmation/review summary must include a short complexity rationale.
|
|
323
403
|
|
|
324
404
|
## Next-Step Execution Policy (after planning)
|
|
325
405
|
|
|
@@ -364,9 +444,10 @@ Prevent accidental session exit without deliverable completion.
|
|
|
364
444
|
|
|
365
445
|
## Handoff Message Template
|
|
366
446
|
|
|
367
|
-
After successful validation, report:
|
|
447
|
+
After successful validation and the Post-Generation Plan Review Gate, report:
|
|
368
448
|
1. Output file path
|
|
369
449
|
2. Total refactor items
|
|
370
450
|
3. Dependency ordering highlights (which items must run first)
|
|
371
451
|
4. Behavior preservation strategy distribution (N items with test-gate, M with snapshot, etc.)
|
|
372
|
-
5.
|
|
452
|
+
5. Review result: `/prizmkit-code-review` verdict, reviewed refactor IDs, accepted fixes (or "none"), and final validation result
|
|
453
|
+
6. Recommended next action: `refactor-pipeline-launcher`
|
|
@@ -36,7 +36,7 @@ For simple refactoring with minimal scope.
|
|
|
36
36
|
```bash
|
|
37
37
|
python3 ${SKILL_DIR}/scripts/validate-and-generate-refactor.py generate --input .prizmkit/plans/refactor-list.draft.json --output .prizmkit/plans/refactor-list.json
|
|
38
38
|
```
|
|
39
|
-
5. If valid ->
|
|
39
|
+
5. If valid -> run the mandatory Post-Generation Plan Review Gate from the main `refactor-planner` skill before summarizing or recommending `refactor-pipeline-launcher`.
|
|
40
40
|
6. If invalid -> apply fixes to the draft, re-run generate (max 2 attempts, then escalate to full workflow)
|
|
41
41
|
|
|
42
42
|
## When NOT to Use Fast Path
|
|
@@ -55,5 +55,6 @@ AI: [Uses AskUserQuestion with options: "Fast-path", "Full workflow", "Implement
|
|
|
55
55
|
User: [Selects "Fast-path"]
|
|
56
56
|
AI: "Drafting R-001..."
|
|
57
57
|
AI: [Validates immediately]
|
|
58
|
+
AI: [Runs the Post-Generation Plan Review Gate on R-001 only]
|
|
58
59
|
AI: "Ready to proceed to dev-pipeline."
|
|
59
60
|
```
|
|
@@ -11,8 +11,9 @@ Read `${SKILL_DIR}/assets/planning-guide.md` for decomposition patterns and depe
|
|
|
11
11
|
For each refactoring goal:
|
|
12
12
|
1. Identify atomic refactoring operations
|
|
13
13
|
2. Determine inter-item dependencies (safe renames first, structural changes later)
|
|
14
|
-
3. Assess complexity per item (file count, cross-module scope, test coverage)
|
|
15
|
-
4. Assign
|
|
14
|
+
3. Assess complexity per item from concrete indicators (file count, cross-module scope, public API/import-path impact, dependency depth, test coverage, behavior-preservation risk, migration impact)
|
|
15
|
+
4. Assign priority from concrete impact (blocked delivery, correctness/security/data risk, maintainability value, or local cleanup), not from size or user urgency alone
|
|
16
|
+
5. Assign behavior preservation strategy per item (read `${SKILL_DIR}/references/behavior-preservation.md`)
|
|
16
17
|
|
|
17
18
|
---
|
|
18
19
|
|
|
@@ -26,7 +27,9 @@ Refactor Item R-001:
|
|
|
26
27
|
Type: [extract/rename/restructure/simplify/decouple/migrate]
|
|
27
28
|
Scope: [files list]
|
|
28
29
|
Priority: [critical/high/medium/low]
|
|
30
|
+
Priority Rationale: [short concrete impact reason]
|
|
29
31
|
Complexity: [low/medium/high]
|
|
32
|
+
Complexity Rationale: [short scope/risk reason]
|
|
30
33
|
Behavior Preservation: [test-gate/snapshot/manual]
|
|
31
34
|
Acceptance Criteria:
|
|
32
35
|
- [criterion 1]
|
|
@@ -63,11 +66,11 @@ Continue until all items are confirmed or skipped.
|
|
|
63
66
|
Present review summary:
|
|
64
67
|
|
|
65
68
|
```
|
|
66
|
-
Item | Deps Valid | Preservation | Gaps | Status
|
|
67
|
-
R-001 | OK | test-gate | - | Ready
|
|
68
|
-
R-002 | OK | test-gate | - | Ready
|
|
69
|
-
R-003 | OK | manual | No test coverage| Needs attention
|
|
70
|
-
R-004 | OK | snapshot | - | Ready
|
|
69
|
+
Item | Priority | Priority Rationale | Complexity | Complexity Rationale | Deps Valid | Preservation | Gaps | Status
|
|
70
|
+
R-001 | high | Blocks core feature work | high | 3 modules + public imports | OK | test-gate | - | Ready
|
|
71
|
+
R-002 | medium | Planned maintainability | medium | 5 files + tests exist | OK | test-gate | - | Ready
|
|
72
|
+
R-003 | low | Local readability only | low | One file, no API change | OK | manual | No test coverage| Needs attention
|
|
73
|
+
R-004 | medium | Reduces bounded coupling | medium | 2 modules + snapshot check | OK | snapshot | - | Ready
|
|
71
74
|
```
|
|
72
75
|
|
|
73
76
|
If issues found, discuss with user and resolve before proceeding.
|
|
@@ -100,13 +103,28 @@ Auto-detect inter-item dependencies and enforce safe ordering:
|
|
|
100
103
|
|
|
101
104
|
## Complexity Assessment
|
|
102
105
|
|
|
106
|
+
Complexity is a behavior-preserving execution risk estimate, not a measure of how much the user wants the refactor. Do not default to high because a refactor spans multiple files, targets framework code, or improves maintainability.
|
|
107
|
+
|
|
103
108
|
Assess each item's complexity based on:
|
|
104
|
-
- **File count**: 1-
|
|
109
|
+
- **File count**: 1-3 files = low, 4-8 files = medium, 9+ files = high
|
|
105
110
|
- **Cross-module scope**: same module = low, 2 modules = medium, 3+ modules = high
|
|
106
|
-
- **
|
|
107
|
-
- **
|
|
111
|
+
- **Public API/import-path impact**: no public change = low, bounded internal interface = medium, public API/import migration = high
|
|
112
|
+
- **Dependency depth**: 0-2 import/update sites = low, 3-5 = medium, 6+ = high
|
|
113
|
+
- **Test coverage**: strong existing tests can keep complexity lower; weak/missing tests increase complexity
|
|
114
|
+
- **Behavior-preservation risk**: test-gate with clear assertions = lower, snapshot/manual verification = higher
|
|
115
|
+
- **Migration impact**: no rollout = low, compatibility shim = medium, cross-module migration or phased rollout = high
|
|
116
|
+
|
|
117
|
+
Take the highest well-supported individual assessment as the item's complexity, then record the rationale in the confirmation table.
|
|
118
|
+
|
|
119
|
+
### Mixed Calibration Examples
|
|
120
|
+
|
|
121
|
+
| Refactor | Priority | Complexity | Rationale |
|
|
122
|
+
|----------|----------|------------|-----------|
|
|
123
|
+
| Break circular dependency blocking auth feature work | high | high | Blocks core delivery; 3 modules, public imports, and weak tests. |
|
|
124
|
+
| Extract chart formatting helpers from dashboard files | medium | medium | Improves planned maintainability; 5 files with bounded helper API and existing tests. |
|
|
125
|
+
| Rename local variable in parser for readability | low | low | Local cleanup only; one file, no interface or behavior change. |
|
|
108
126
|
|
|
109
|
-
|
|
127
|
+
Anti-pattern: do not generate an all-high refactor list unless every item independently satisfies documented high-priority and high-complexity criteria.
|
|
110
128
|
|
|
111
129
|
---
|
|
112
130
|
|
|
@@ -1,61 +1,90 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: "refactor-workflow"
|
|
3
|
-
description: "
|
|
3
|
+
description: "Triage-based refactor workflow. Clarifies behavior-preserving refactoring goals, then lets the user choose Fast Path for simple current-session/current-workspace refactors or Pipeline Path for complex/batch refactors via refactor-planner → refactor-pipeline-launcher. Use when the user wants to restructure, clean up, or optimize code without changing behavior. Trigger on: 'refactor', 'clean up code', 'restructure', 'optimize code structure', 'extract module', 'code refactoring', 'batch refactor'."
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
# Refactor Workflow
|
|
7
7
|
|
|
8
|
-
|
|
8
|
+
Triage-based entry point for code refactoring. The workflow clarifies behavior-preserving goals, assesses risk and scope, asks the user to choose an approach, then follows one of two paths:
|
|
9
|
+
|
|
10
|
+
- **Fast Path** — a simple behavior-preserving refactor is implemented directly in the **current session** and **current workspace** with the normal PrizmKit lifecycle commands.
|
|
11
|
+
- **Pipeline Path** — complex or batch refactors are planned with `refactor-planner`, launched with `refactor-pipeline-launcher`, then monitored.
|
|
12
|
+
|
|
13
|
+
This skill keeps Fast Path and Pipeline Path separate. Pipeline monitoring only happens after the user selects Pipeline Path and the launcher starts execution.
|
|
9
14
|
|
|
10
15
|
## When to Use
|
|
11
16
|
|
|
12
17
|
User says:
|
|
13
18
|
- "Refactor the auth module", "Clean up this code", "Restructure the payment service"
|
|
14
19
|
- "Extract shared logic into a separate module", "Optimize code structure"
|
|
20
|
+
- "Do this small behavior-preserving cleanup now"
|
|
15
21
|
- "Batch refactor these modules", "Code refactoring across multiple files"
|
|
16
22
|
- Code has accumulated tech debt that needs structural improvement
|
|
17
23
|
- Module needs to be split, merged, or reorganized
|
|
18
24
|
|
|
19
25
|
**Do NOT use this skill when:**
|
|
20
|
-
- User only wants to plan refactoring
|
|
21
|
-
- User only wants to launch
|
|
22
|
-
- User wants to add features
|
|
23
|
-
- User wants to fix bugs
|
|
26
|
+
- User only wants to plan refactoring without execution → use `refactor-planner` directly
|
|
27
|
+
- User only wants to launch an existing `.prizmkit/plans/refactor-list.json` → use `refactor-pipeline-launcher`
|
|
28
|
+
- User wants to add features → use `feature-workflow`
|
|
29
|
+
- User wants to fix bugs → use `bug-fix-workflow` or `bug-planner` + `bugfix-pipeline-launcher`
|
|
24
30
|
|
|
25
31
|
---
|
|
26
32
|
|
|
27
33
|
## Overview
|
|
28
34
|
|
|
29
|
-
|
|
35
|
+
The shared contract is:
|
|
36
|
+
|
|
37
|
+
1. **Brainstorm** — clarify refactoring type, collect reference materials, read relevant code/docs/tests, and confirm behavior-preservation requirements.
|
|
38
|
+
2. **Assess difficulty and behavior risk** — decide whether the request is a simple behavior-preserving Fast Path candidate or a complex/batch Pipeline Path candidate.
|
|
39
|
+
3. **AskUserQuestion approach selection** — present selectable options; never silently choose the path.
|
|
40
|
+
4. **Execute selected path**:
|
|
41
|
+
- **Fast Path**: `/prizmkit-plan` → `/prizmkit-implement` with behavior-preservation tests/checks → `/prizmkit-code-review` → `/prizmkit-retrospective` → `/prizmkit-committer` in the current workspace.
|
|
42
|
+
- **Pipeline Path**: `refactor-planner` → `refactor-pipeline-launcher` → monitoring.
|
|
43
|
+
|
|
44
|
+
### Terminology
|
|
30
45
|
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
|
|
34
|
-
|
|
46
|
+
| Term | Meaning |
|
|
47
|
+
|------|---------|
|
|
48
|
+
| **Fast Path** | Current-session/current-workspace implementation for simple, behavior-preserving refactors. It does not create or consume `.prizmkit/plans/refactor-list.json` and does not start the refactor pipeline. |
|
|
49
|
+
| **Pipeline Path** | Complex or batch refactor execution. Planner handoff goes to `refactor-planner`; launcher handoff goes to `refactor-pipeline-launcher`; monitoring follows only after launch. |
|
|
50
|
+
| **Current session** | The interactive AI CLI session that is already discussing the refactor with the user. |
|
|
51
|
+
| **Current workspace** | The current git checkout and working tree, not a background pipeline worktree/session. |
|
|
52
|
+
| **Behavior preservation** | External behavior, public contracts, tests, and user-visible outcomes remain unchanged unless the user explicitly approves a behavior change as a feature. |
|
|
53
|
+
| **Monitoring** | Status/log follow-up for a launched refactor pipeline; it is not part of the Fast Path. |
|
|
35
54
|
|
|
36
|
-
### Branch
|
|
55
|
+
### Workspace and Branch Model
|
|
37
56
|
|
|
38
|
-
|
|
39
|
-
-
|
|
40
|
-
- Branches are created, committed, and managed by the pipeline session
|
|
41
|
-
- This workflow does NOT create a top-level branch — the pipeline manages granular per-refactor branches
|
|
57
|
+
- **Fast Path**: work happens in the current workspace and current interactive session. Use the current branch unless the project policy or user explicitly asks to create a branch.
|
|
58
|
+
- **Pipeline Path**: `refactor-pipeline-launcher` and the dev-pipeline manage per-refactor branches/worktrees. This workflow does not create a top-level branch before handing off to the launcher.
|
|
42
59
|
|
|
43
60
|
---
|
|
44
61
|
|
|
45
62
|
## Input Modes
|
|
46
63
|
|
|
47
|
-
**Mode A:
|
|
64
|
+
**Mode A: Natural language refactoring goal** (default)
|
|
48
65
|
|
|
49
66
|
Natural language description of the refactoring goals. Can be:
|
|
67
|
+
- A simple behavior-preserving refactor: "Extract this helper function and keep behavior unchanged"
|
|
50
68
|
- A module target: "Refactor the auth module — extract shared middleware, simplify the token flow"
|
|
51
69
|
- A batch of refactors: "Clean up the payment service, decouple the notification module, and extract common utilities"
|
|
52
70
|
- An incremental request: "Also refactor the logging layer to use structured logging"
|
|
53
71
|
|
|
54
|
-
Flow: brainstorm →
|
|
72
|
+
Flow: brainstorm → complexity/behavior-risk assessment → AskUserQuestion → Fast Path or Pipeline Path.
|
|
55
73
|
|
|
56
|
-
**Mode B:
|
|
74
|
+
**Mode B: Existing `.prizmkit/plans/refactor-list.json`**
|
|
57
75
|
|
|
58
|
-
|
|
76
|
+
When the user says "run pipeline from existing file" or `.prizmkit/plans/refactor-list.json` already exists and should be executed:
|
|
77
|
+
- Skip brainstorm and `refactor-planner`
|
|
78
|
+
- Invoke `refactor-pipeline-launcher` directly
|
|
79
|
+
- Monitor and report progress
|
|
80
|
+
|
|
81
|
+
**Mode C: Incremental refactor request**
|
|
82
|
+
|
|
83
|
+
When user says "add more refactors":
|
|
84
|
+
- Brainstorm only the new refactoring goals
|
|
85
|
+
- Assess whether the new scope is simple and behavior-preserving enough for Fast Path
|
|
86
|
+
- For simple work: use Fast Path in the current workspace
|
|
87
|
+
- For complex or batch work: invoke `refactor-planner` in incremental mode, then `refactor-pipeline-launcher`
|
|
59
88
|
|
|
60
89
|
---
|
|
61
90
|
|
|
@@ -79,6 +108,7 @@ Then ask:
|
|
|
79
108
|
- **What** code needs restructuring (modules, files, patterns)
|
|
80
109
|
- **Why** it needs refactoring (tech debt, coupling, complexity, readability, performance structure)
|
|
81
110
|
- **What outcome** they want (target architecture, desired structure, quality goals)
|
|
111
|
+
- **What behavior must remain unchanged** (public APIs, outputs, workflows, tests, performance-sensitive paths)
|
|
82
112
|
|
|
83
113
|
### Step 1.2: Collect Reference Materials
|
|
84
114
|
|
|
@@ -95,6 +125,7 @@ Ask:
|
|
|
95
125
|
2. **Design documents** — "Do you have any design docs, architecture diagrams, or refactoring proposals I should read?"
|
|
96
126
|
3. **Knowledge docs** — "Are there related `.prizmkit/prizm-docs/`, README files, or internal wiki pages for the target area?"
|
|
97
127
|
4. **Related issues/PRs** — "Any related issues, PRs, or previous refactoring attempts I should be aware of?"
|
|
128
|
+
5. **Behavior checks** — "Which tests, commands, or manual scenarios prove behavior must stay the same?"
|
|
98
129
|
|
|
99
130
|
Record everything the user provides — these become inputs for Step 1.3.
|
|
100
131
|
|
|
@@ -121,23 +152,26 @@ Read `${SKILL_DIR}/references/brainstorm-guide.md` §Completion Signs for the fu
|
|
|
121
152
|
|
|
122
153
|
Once brainstorming is complete, produce a structured goals summary. Read `${SKILL_DIR}/references/brainstorm-guide.md` §Refactoring Goals Summary Template for the structured output format. Present the summary to the user and get explicit confirmation before proceeding.
|
|
123
154
|
|
|
155
|
+
The summary must include behavior-preservation expectations: tests to run, public contracts to keep stable, and any explicitly approved behavior changes. If behavior changes are required, route them as feature work rather than refactoring unless the user explicitly scopes them here.
|
|
156
|
+
|
|
124
157
|
**CHECKPOINT CP-RW-0**: Refactoring goals fully clarified and confirmed by user.
|
|
125
158
|
|
|
126
159
|
---
|
|
127
160
|
|
|
128
161
|
### Step 1.7: Complexity Assessment & Approach Selection
|
|
129
162
|
|
|
130
|
-
After confirming refactoring goals, assess whether this refactor needs the
|
|
163
|
+
After confirming refactoring goals, assess whether this refactor needs the Pipeline Path or can be done directly in the current session/current workspace.
|
|
131
164
|
|
|
132
|
-
**Simple refactor → Fast Path candidate** (ALL must be true):
|
|
165
|
+
**Simple behavior-preserving refactor → Fast Path candidate** (ALL must be true):
|
|
133
166
|
- Single module, no cross-module impact
|
|
134
167
|
- ≤3 files affected
|
|
135
168
|
- No public API or interface changes
|
|
136
169
|
- Straightforward transformation (extract method, rename, move file, simplify logic)
|
|
137
|
-
- Existing tests
|
|
138
|
-
- No risk
|
|
170
|
+
- Existing tests or focused checks cover the affected behavior
|
|
171
|
+
- No behavior change risk beyond what tests/checks can verify
|
|
139
172
|
|
|
140
|
-
**Complex
|
|
173
|
+
**Complex or batch refactors → Pipeline Path candidate** (ANY is true):
|
|
174
|
+
- Multiple refactor items or batch refactoring request
|
|
141
175
|
- Cross-module impact (>2 modules affected)
|
|
142
176
|
- Public API or interface changes required
|
|
143
177
|
- Multiple interrelated refactoring steps with dependency ordering
|
|
@@ -145,48 +179,56 @@ After confirming refactoring goals, assess whether this refactor needs the full
|
|
|
145
179
|
- Insufficient test coverage in target area (risk of hidden behavior changes)
|
|
146
180
|
- Requires coordination with other ongoing work
|
|
147
181
|
|
|
148
|
-
**User choice required (mandatory)** — Use `AskUserQuestion` to present interactive selectable options. Tailor the question text to the assessment
|
|
182
|
+
**User choice required (mandatory)** — Use `AskUserQuestion` to present interactive selectable options. Tailor the question text to the assessment: for a simple refactor, "This refactor appears straightforward and behavior-preserving enough for the current-session Fast Path." For a complex or batch refactor, "This refactor is complex or batch-oriented and should use the Pipeline Path."
|
|
149
183
|
|
|
150
184
|
```
|
|
151
185
|
AskUserQuestion:
|
|
152
|
-
question:
|
|
186
|
+
question: "How would you like to proceed?"
|
|
153
187
|
header: "Approach"
|
|
154
188
|
options:
|
|
155
|
-
- label: "Refactor now (
|
|
156
|
-
description: "
|
|
157
|
-
- label: "
|
|
158
|
-
description: "Generate .prizmkit/plans/refactor-list.json
|
|
189
|
+
- label: "Refactor now (Fast Path — current session/current workspace)"
|
|
190
|
+
description: "Run /prizmkit-plan → /prizmkit-implement → /prizmkit-code-review → /prizmkit-retrospective → /prizmkit-committer directly in this workspace with behavior-preservation checks"
|
|
191
|
+
- label: "Use Pipeline Path (refactor-planner → refactor-pipeline-launcher)"
|
|
192
|
+
description: "Generate .prizmkit/plans/refactor-list.json with refactor-planner, then launch autonomous execution with refactor-pipeline-launcher"
|
|
159
193
|
```
|
|
160
194
|
|
|
161
|
-
- **Refactor now
|
|
162
|
-
-
|
|
195
|
+
- **Refactor now (Fast Path)**:
|
|
196
|
+
1. Invoke `/prizmkit-plan` with the goals summary → generates `spec.md` + `plan.md` including behavior-preservation strategy
|
|
197
|
+
2. Invoke `/prizmkit-implement` to execute the plan in the current workspace and verify behavior stays unchanged
|
|
198
|
+
3. Run `/prizmkit-code-review` for quality and behavior-preservation review
|
|
199
|
+
4. Run `/prizmkit-retrospective` to sync `.prizmkit/prizm-docs/`
|
|
200
|
+
5. Commit via `/prizmkit-committer` with `refactor(<scope>):` prefix
|
|
201
|
+
6. End workflow — do not call `refactor-planner`, do not call `refactor-pipeline-launcher`, and do not enter monitoring
|
|
202
|
+
- **Use Pipeline Path** → Continue to Phase 2 (Planner handoff)
|
|
163
203
|
|
|
164
204
|
**NEVER proceed without explicit user confirmation via `AskUserQuestion`. Do NOT render options as plain text — the user must be able to click/select.**
|
|
165
205
|
|
|
166
|
-
**CHECKPOINT CP-RW-0.5**: Approach selected by user (
|
|
206
|
+
**CHECKPOINT CP-RW-0.5**: Approach selected by user (Fast Path or Pipeline Path).
|
|
167
207
|
|
|
168
208
|
---
|
|
169
209
|
|
|
170
|
-
## Phase 2: Plan
|
|
210
|
+
## Phase 2: Plan — Pipeline Path Only
|
|
171
211
|
|
|
172
|
-
**Goal**: Generate structured
|
|
212
|
+
**Goal**: Generate structured `.prizmkit/plans/refactor-list.json` from the clarified refactoring goals.
|
|
213
|
+
|
|
214
|
+
Run this phase only when the user selected Pipeline Path or provided a batch/complex refactor request that they chose to send through the pipeline.
|
|
173
215
|
|
|
174
216
|
**STEPS**:
|
|
175
217
|
|
|
176
|
-
1. **
|
|
218
|
+
1. **Planner handoff — invoke `refactor-planner`** with the full goals summary from Phase 1:
|
|
177
219
|
- Pass the structured goals summary as input — NOT the raw user conversation
|
|
178
220
|
- For new refactoring: standard planning mode
|
|
179
221
|
- For existing projects with `--incremental`: incremental planning mode
|
|
180
222
|
- **Input**: Markdown goals summary (refactor targets, scope, behavior preservation strategy)
|
|
181
223
|
- **Output**: `.prizmkit/plans/refactor-list.json` (schema: `dev-pipeline-refactor-list-v1`) containing `project_name`, `refactors[]` with id (R-NNN), title, description, scope, type, priority, complexity, behavior_preservation, acceptance_criteria, dependencies, status
|
|
182
224
|
|
|
183
|
-
2. **Interactive planning** (if refactor-planner requires clarification):
|
|
184
|
-
- Because Phase 1 was thorough, refactor-planner should need minimal clarification
|
|
225
|
+
2. **Interactive planning** (if `refactor-planner` requires clarification):
|
|
226
|
+
- Because Phase 1 was thorough, `refactor-planner` should need minimal clarification
|
|
185
227
|
- If questions arise, answer from the Phase 1 context or pass through to user
|
|
186
228
|
|
|
187
229
|
3. **Validate output**:
|
|
188
230
|
- Confirm `.prizmkit/plans/refactor-list.json` exists
|
|
189
|
-
- Show summary: total refactor items, complexity distribution, dependencies
|
|
231
|
+
- Show summary: total refactor items, complexity distribution, dependencies, behavior-preservation coverage
|
|
190
232
|
|
|
191
233
|
**CHECKPOINT CP-RW-1**: `.prizmkit/plans/refactor-list.json` generated and validated.
|
|
192
234
|
|
|
@@ -194,15 +236,15 @@ AskUserQuestion:
|
|
|
194
236
|
|
|
195
237
|
---
|
|
196
238
|
|
|
197
|
-
## Phase 3: Launch
|
|
239
|
+
## Phase 3: Launch — Pipeline Path Only
|
|
198
240
|
|
|
199
|
-
**Goal**: Start the refactoring pipeline.
|
|
241
|
+
**Goal**: Start the refactoring pipeline after planner handoff has produced a validated refactor list.
|
|
200
242
|
|
|
201
243
|
**STEPS**:
|
|
202
244
|
|
|
203
245
|
1. **Show refactor summary** before launching:
|
|
204
246
|
```
|
|
205
|
-
Ready to launch
|
|
247
|
+
Ready to launch Pipeline Path with N refactor items:
|
|
206
248
|
R-001: Extract auth middleware (medium complexity)
|
|
207
249
|
R-002: Decouple notification service (high complexity)
|
|
208
250
|
R-003: Simplify token flow (low complexity)
|
|
@@ -210,10 +252,11 @@ AskUserQuestion:
|
|
|
210
252
|
Proceed? (Y/n)
|
|
211
253
|
```
|
|
212
254
|
|
|
213
|
-
2. **
|
|
255
|
+
2. **Launcher handoff — invoke `refactor-pipeline-launcher`**:
|
|
214
256
|
- **Input**: Path to validated `.prizmkit/plans/refactor-list.json`
|
|
215
257
|
- The launcher handles all prerequisites checks
|
|
216
258
|
- The launcher presents execution mode choices to the user (foreground/background/manual)
|
|
259
|
+
- The launcher preserves strict behavior-check defaults
|
|
217
260
|
- Do NOT duplicate execution mode selection here — let the launcher handle it
|
|
218
261
|
- **Output**: PID/status, log file path, execution mode selected
|
|
219
262
|
- Returns PID/status and log file location
|
|
@@ -226,9 +269,9 @@ AskUserQuestion:
|
|
|
226
269
|
|
|
227
270
|
---
|
|
228
271
|
|
|
229
|
-
## Phase 4: Monitor
|
|
272
|
+
## Phase 4: Monitor — Pipeline Path Only
|
|
230
273
|
|
|
231
|
-
**Goal**: Track pipeline progress and report to user.
|
|
274
|
+
**Goal**: Track pipeline progress and report to user. Monitoring applies only after `refactor-pipeline-launcher` has started a pipeline.
|
|
232
275
|
|
|
233
276
|
**STEPS**:
|
|
234
277
|
|
|
@@ -271,13 +314,14 @@ AskUserQuestion:
|
|
|
271
314
|
The workflow supports resuming by detecting existing state:
|
|
272
315
|
|
|
273
316
|
| State Found | Resume From |
|
|
274
|
-
|
|
275
|
-
|
|
|
317
|
+
|-------------|-------------|
|
|
318
|
+
| Fast Path `spec.md` + `plan.md` exist, code not committed | Resume `/prizmkit-implement` / review / retrospective / commit based on current state |
|
|
319
|
+
| No `.prizmkit/plans/refactor-list.json` and no Fast Path artifacts | Phase 1: Brainstorm |
|
|
276
320
|
| `.prizmkit/plans/refactor-list.json` exists, no pipeline state | Phase 3: Launch |
|
|
277
321
|
| `.prizmkit/plans/refactor-list.json` + pipeline state exists | Phase 4: Monitor (check status) |
|
|
278
322
|
| All refactors completed | Report completion, suggest next steps |
|
|
279
323
|
|
|
280
|
-
**Resume**: If `.prizmkit/plans/refactor-list.json` exists, ask user: "Existing refactor plan found with N items. Resume
|
|
324
|
+
**Resume**: If `.prizmkit/plans/refactor-list.json` exists, ask user: "Existing refactor plan found with N items. Resume Pipeline Path or re-plan?" If Fast Path artifacts exist, ask whether to continue the current-workspace refactor or discard/re-plan.
|
|
281
325
|
|
|
282
326
|
---
|
|
283
327
|
|
|
@@ -300,21 +344,36 @@ While the pipeline runs, the user can continue the conversation:
|
|
|
300
344
|
|-------|--------|
|
|
301
345
|
| User's refactoring goal is too vague | Ask for more context: "Can you describe what's wrong with the current code structure?" |
|
|
302
346
|
| Brainstorming stalls | Offer concrete options: "Would you prefer incremental or comprehensive refactoring?" |
|
|
303
|
-
| No tests exist for target module | WARN user, recommend writing tests
|
|
347
|
+
| No tests exist for target module | WARN user, recommend writing tests/checks before Fast Path; consider Pipeline Path for risky refactors |
|
|
348
|
+
| Fast Path starts risking behavior change | Pause and ask whether to switch to Pipeline Path (`refactor-planner` → `refactor-pipeline-launcher`) |
|
|
304
349
|
| `refactor-planner` cannot parse goals | Refine the goals summary and retry |
|
|
305
350
|
| `.prizmkit/plans/refactor-list.json` generation failed | Show error, retry with refined input |
|
|
306
351
|
| Pipeline launch failed | Show daemon log, suggest manual start with `python3 ./.prizmkit/dev-pipeline/cli.py refactor run .prizmkit/plans/refactor-list.json` or `python3 ./.prizmkit/dev-pipeline/cli.py daemon refactor start .prizmkit/plans/refactor-list.json` |
|
|
307
352
|
| All refactor items blocked/failed | Show status, suggest retrying specific items |
|
|
308
353
|
| User wants to cancel mid-brainstorming | Save conversation context, offer to resume later |
|
|
309
|
-
| Behavior regression detected during
|
|
354
|
+
| Behavior regression detected during Fast Path | Revert or fix immediately before commit; refactoring cannot ship with behavior regression |
|
|
355
|
+
| Behavior regression detected during pipeline | Pipeline handles per-item retries/failures; report failed items and behavior checks |
|
|
356
|
+
|
|
357
|
+
---
|
|
310
358
|
|
|
359
|
+
## HANDOFF
|
|
311
360
|
|
|
361
|
+
| From | To | Condition |
|
|
362
|
+
|------|----|-----------|
|
|
363
|
+
| **this skill** | `/prizmkit-plan` | User selects Fast Path for simple behavior-preserving refactor work in the current workspace |
|
|
364
|
+
| **this skill** | `/prizmkit-implement` | Fast Path plan is ready for current-session implementation with behavior checks |
|
|
365
|
+
| **this skill** | `/prizmkit-code-review` | Fast Path refactor is complete and needs quality/behavior-preservation review |
|
|
366
|
+
| **this skill** | `/prizmkit-retrospective` | Fast Path refactor changes need `.prizmkit/prizm-docs/` sync before commit |
|
|
367
|
+
| **this skill** | `/prizmkit-committer` | Fast Path refactor is reviewed, docs are synced, and behavior preservation is verified |
|
|
368
|
+
| **this skill** | `refactor-planner` | User selects Pipeline Path for complex or batch refactors |
|
|
369
|
+
| `refactor-planner` | `refactor-pipeline-launcher` | Planner handoff produced validated `.prizmkit/plans/refactor-list.json` |
|
|
370
|
+
| `refactor-pipeline-launcher` | **this skill** | Launcher handoff completed and monitoring/status follow-up is needed |
|
|
371
|
+
|
|
372
|
+
---
|
|
312
373
|
|
|
313
374
|
## Output
|
|
314
375
|
|
|
315
|
-
|
|
316
|
-
|
|
317
|
-
-
|
|
318
|
-
-
|
|
319
|
-
- Multiple git commits with `refactor(<scope>):` prefix
|
|
320
|
-
- Updated `.prizmkit/prizm-docs/` (via prizmkit-retrospective per refactor item)
|
|
376
|
+
Output depends on the selected path:
|
|
377
|
+
|
|
378
|
+
- **Fast Path**: `spec.md` + `plan.md`, behavior-preserving current-workspace code changes, behavior test/check evidence, `/prizmkit-code-review` result, updated `.prizmkit/prizm-docs/`, and one `refactor(<scope>):` commit from `/prizmkit-committer`.
|
|
379
|
+
- **Pipeline Path**: structured refactoring goals summary, `.prizmkit/plans/refactor-list.json`, pipeline execution via `refactor-pipeline-launcher`, monitoring updates, and per-refactor commits managed by the dev-pipeline.
|
|
@@ -30,7 +30,7 @@ This project uses PrizmKit with the Prizm documentation system for AI-optimized
|
|
|
30
30
|
3. Add a pointer in the parent L1 doc's SUBDIRS section
|
|
31
31
|
|
|
32
32
|
### Available Commands
|
|
33
|
-
Run `/
|
|
33
|
+
Run `/prizmkit` to see all available PrizmKit commands.
|
|
34
34
|
|
|
35
35
|
### Fast Path for Simple Changes
|
|
36
36
|
Not every change needs the full spec -> plan workflow. Use fast path for:
|