prizmkit 1.1.12 → 1.1.13
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/skills/_metadata.json +1 -1
- package/bundled/skills/feature-workflow/SKILL.md +61 -34
- package/bundled/skills/prizmkit-retrospective/references/structural-sync-steps.md +3 -7
- package/bundled/skills/refactor-workflow/SKILL.md +72 -66
- package/package.json +1 -1
package/bundled/VERSION.json
CHANGED
|
@@ -29,7 +29,7 @@ User says:
|
|
|
29
29
|
```
|
|
30
30
|
feature-workflow <idea / requirements>
|
|
31
31
|
│
|
|
32
|
-
├── Phase 1: Brainstorm →
|
|
32
|
+
├── Phase 1: Brainstorm → collect materials → parallel deep read → discuss requirements
|
|
33
33
|
│
|
|
34
34
|
├── Phase 2: Plan → feature-planner → .prizmkit/plans/feature-list.json
|
|
35
35
|
│
|
|
@@ -42,7 +42,7 @@ feature-workflow <idea / requirements>
|
|
|
42
42
|
|
|
43
43
|
| Phase | Action | Result |
|
|
44
44
|
|-------|--------|--------|
|
|
45
|
-
| 1 | **Brainstorm** —
|
|
45
|
+
| 1 | **Brainstorm** — collect reference materials, parallel deep read code & docs, discuss requirements grounded in real context | Fully clarified requirements document |
|
|
46
46
|
| 2 | Call `feature-planner` with clarified requirements | `.prizmkit/plans/feature-list.json` with N features |
|
|
47
47
|
| 3 | Call `feature-pipeline-launcher` | Pipeline started (execution mode chosen by user via launcher) |
|
|
48
48
|
| 4 | Monitor progress | Status updates, completion report |
|
|
@@ -100,7 +100,7 @@ When user says "add features to existing project" or the project already has fea
|
|
|
100
100
|
|
|
101
101
|
## Phase 1: Brainstorm — Deep Requirement Clarification
|
|
102
102
|
|
|
103
|
-
**Goal**: Through interactive Q&A, transform the user's rough idea into fully clarified, implementation-ready requirements. This phase is the foundation for high-quality code generation — vague requirements produce vague code.
|
|
103
|
+
**Goal**: Through interactive Q&A and deep context reading, transform the user's rough idea into fully clarified, implementation-ready requirements. This phase is the foundation for high-quality code generation — vague requirements produce vague code.
|
|
104
104
|
|
|
105
105
|
**CRITICAL RULE**: The number of questions is **unlimited**. Do NOT rush through this phase. Ask as many rounds as needed until every aspect is clear. The framework strives for perfect code generation, which requires perfect understanding of requirements.
|
|
106
106
|
|
|
@@ -111,25 +111,49 @@ Ask the user to describe what they want to build. Listen for:
|
|
|
111
111
|
- **Who** uses it (user roles, personas)
|
|
112
112
|
- **Why** it's needed (business value, problem being solved)
|
|
113
113
|
|
|
114
|
-
### Step 1.2:
|
|
114
|
+
### Step 1.2: Collect Reference Materials
|
|
115
115
|
|
|
116
|
-
|
|
117
|
-
- If `.prizm-docs/root.prizm` exists → read it to understand existing architecture, tech stack, patterns
|
|
118
|
-
- If `.prizmkit/config.json` exists → read tech stack preferences
|
|
119
|
-
- If existing source code exists → scan directory structure:
|
|
120
|
-
```bash
|
|
121
|
-
find . -maxdepth 2 -type d -not -path '*/node_modules/*' -not -path '*/.git/*' -not -path '*/dist/*' -not -path '*/build/*' -not -path '*/__pycache__/*' -not -path '*/vendor/*' | sed -e 's;[^/]*/;|____;g;s;____|; |;g'
|
|
122
|
-
```
|
|
123
|
-
- If database/schema files exist → scan them to understand existing data model:
|
|
124
|
-
```bash
|
|
125
|
-
find . -maxdepth 4 -type f \( -name "*.prisma" -o -name "*.sql" -o -path "*/migrations/*" -o -path "*/models/*" -o -name "schema.*" -o -name "*.entity.*" \) -not -path '*/node_modules/*' -not -path '*/.git/*' -not -path '*/dist/*' -not -path '*/__pycache__/*' | head -20
|
|
126
|
-
```
|
|
116
|
+
**Ask the user explicitly** what resources they have. Do NOT skip this step — user-provided materials are far more valuable than blind directory scanning.
|
|
127
117
|
|
|
128
|
-
|
|
118
|
+
Ask:
|
|
119
|
+
1. **Existing code** — "Is there existing code I should look at? Which files or directories are relevant?"
|
|
120
|
+
2. **Design documents** — "Do you have any design docs, wireframes, API specs, or PRDs I should read?"
|
|
121
|
+
3. **Knowledge docs** — "Are there related `.prizm-docs/`, README files, or internal wiki pages?"
|
|
122
|
+
4. **Reference projects** — "Any reference implementations or similar projects I should look at for inspiration?"
|
|
129
123
|
|
|
130
|
-
|
|
124
|
+
Record everything the user provides — these become inputs for Step 1.3.
|
|
131
125
|
|
|
132
|
-
|
|
126
|
+
### Step 1.3: Parallel Deep Reading
|
|
127
|
+
|
|
128
|
+
**Goal**: Build comprehensive understanding of the project context before discussing detailed requirements. Spawn multiple agents in parallel to read all relevant materials simultaneously.
|
|
129
|
+
|
|
130
|
+
**Parallel reading tasks** (launch concurrently):
|
|
131
|
+
|
|
132
|
+
| Agent | What to read | Purpose |
|
|
133
|
+
|-------|-------------|---------|
|
|
134
|
+
| Agent A | User-provided code paths — read existing source files | Understand current architecture, patterns, conventions |
|
|
135
|
+
| Agent B | User-provided documents — design docs, specs, PRDs | Understand intended requirements and constraints |
|
|
136
|
+
| Agent C | `.prizm-docs/` — root.prizm, L1/L2 docs, TRAPS, RULES | Understand existing architecture knowledge and known pitfalls |
|
|
137
|
+
| Agent D | Database/schema files + `.prizmkit/config.json` | Understand data model and tech stack preferences |
|
|
138
|
+
|
|
139
|
+
**Also gather** (can be included in any agent's task):
|
|
140
|
+
- Directory structure of the project
|
|
141
|
+
- Existing test patterns and conventions
|
|
142
|
+
- Dependency relationships between existing modules
|
|
143
|
+
|
|
144
|
+
**After all agents complete**: Synthesize findings into a coherent understanding before proceeding to discussion.
|
|
145
|
+
|
|
146
|
+
### Step 1.4: Discuss Requirements
|
|
147
|
+
|
|
148
|
+
**Now** — with deep knowledge of the actual codebase and documents — discuss the requirements with the user. This discussion is grounded in real context, not abstract questions.
|
|
149
|
+
|
|
150
|
+
Present what you learned from the parallel reading:
|
|
151
|
+
- Current project structure and patterns (with specific references)
|
|
152
|
+
- Existing data model and schema conventions
|
|
153
|
+
- Known TRAPS and pitfalls from `.prizm-docs/`
|
|
154
|
+
- Integration points with existing modules
|
|
155
|
+
|
|
156
|
+
Then ask targeted questions based on what you read. **Adapt question depth to the feature complexity** — a simple CRUD feature needs fewer questions than a real-time collaboration system.
|
|
133
157
|
|
|
134
158
|
**Functional Requirements:**
|
|
135
159
|
- What are the core user actions/workflows?
|
|
@@ -152,7 +176,7 @@ Based on the user's description, ask questions across these dimensions. **Adapt
|
|
|
152
176
|
- Are there any specific UI/UX requirements?
|
|
153
177
|
|
|
154
178
|
**Integration & Architecture:**
|
|
155
|
-
-
|
|
179
|
+
- "Based on the existing code, this feature would integrate with [modules]. Does that match your expectations?"
|
|
156
180
|
- Any external APIs or services involved?
|
|
157
181
|
- What authentication/authorization model applies?
|
|
158
182
|
- Any real-time requirements (WebSocket, SSE, polling)?
|
|
@@ -167,14 +191,14 @@ Based on the user's description, ask questions across these dimensions. **Adapt
|
|
|
167
191
|
- Scalability considerations?
|
|
168
192
|
- Security requirements? (encryption, audit logs, compliance)
|
|
169
193
|
|
|
170
|
-
### Step 1.
|
|
194
|
+
### Step 1.5: Confirm and Supplement
|
|
171
195
|
|
|
172
|
-
After the
|
|
196
|
+
After the discussion:
|
|
173
197
|
|
|
174
|
-
1. **Summarize**
|
|
175
|
-
2. **
|
|
176
|
-
3. **
|
|
177
|
-
4. **Repeat** until the user confirms: "
|
|
198
|
+
1. **Summarize** the requirements — present it back to the user
|
|
199
|
+
2. **Ask explicitly**: "Is there anything else you'd like to discuss or supplement before we proceed to formal planning?"
|
|
200
|
+
3. **Identify gaps** — if any areas are still unclear, list them explicitly and ask follow-up questions
|
|
201
|
+
4. **Repeat** until the user confirms: "That covers everything" or "Let's proceed"
|
|
178
202
|
|
|
179
203
|
**Signs that brainstorming is complete:**
|
|
180
204
|
- All functional requirements have concrete acceptance criteria
|
|
@@ -189,7 +213,7 @@ After the initial deep-dive:
|
|
|
189
213
|
- Data relationships are unclear ("somehow connected")
|
|
190
214
|
- User says "I'm not sure" — help them think through it with concrete options
|
|
191
215
|
|
|
192
|
-
### Step 1.
|
|
216
|
+
### Step 1.6: Requirements Summary
|
|
193
217
|
|
|
194
218
|
Once brainstorming is complete, produce a structured requirements summary:
|
|
195
219
|
|
|
@@ -220,6 +244,9 @@ Once brainstorming is complete, produce a structured requirements summary:
|
|
|
220
244
|
### Non-Functional Requirements
|
|
221
245
|
- [Requirement]
|
|
222
246
|
|
|
247
|
+
### Reference Materials Reviewed
|
|
248
|
+
- [List of code paths, documents, .prizm-docs/ files that were read]
|
|
249
|
+
|
|
223
250
|
### Confirmed by user: ✓
|
|
224
251
|
```
|
|
225
252
|
|
|
@@ -389,14 +416,14 @@ While the pipeline runs, the user can continue the conversation:
|
|
|
389
416
|
|
|
390
417
|
| Dimension | feature-workflow | bug-fix-workflow | refactor-workflow |
|
|
391
418
|
|-----------|-----------------|------------------|-------------------|
|
|
392
|
-
| **Purpose** | New features (batch) | Single bug fix (interactive) | Code restructuring |
|
|
393
|
-
| **Brainstorming** | Yes —
|
|
394
|
-
| **Planning Skill** | `feature-planner` | None (triage built-in) |
|
|
395
|
-
| **Branch** | Pipeline manages per-feature | `fix/<BUG_ID>-*` |
|
|
396
|
-
| **Execution** | Foreground or background daemon | In-session, interactive |
|
|
397
|
-
| **Input** | Rough idea or requirements | Bug report / stack trace |
|
|
398
|
-
| **Output** | Multiple `feat()` commits | Single `fix()` commit |
|
|
399
|
-
| **
|
|
419
|
+
| **Purpose** | New features (batch) | Single bug fix (interactive) | Code restructuring (batch) |
|
|
420
|
+
| **Brainstorming** | Yes — collect materials, parallel read, discuss | No (bug report is input) | Yes — clarify type, collect materials, parallel read, discuss |
|
|
421
|
+
| **Planning Skill** | `feature-planner` | None (triage built-in) | `refactor-planner` |
|
|
422
|
+
| **Branch** | Pipeline manages per-feature | `fix/<BUG_ID>-*` | Pipeline manages per-refactor |
|
|
423
|
+
| **Execution** | Foreground or background daemon | In-session, interactive | Foreground or background daemon |
|
|
424
|
+
| **Input** | Rough idea or requirements | Bug report / stack trace | Rough refactoring idea or target |
|
|
425
|
+
| **Output** | Multiple `feat()` commits | Single `fix()` commit | Multiple `refactor()` commits |
|
|
426
|
+
| **Behavior Change** | Expected (new functionality) | Fix behavior | Forbidden (structure only) |
|
|
400
427
|
| **Batch alternative** | (this is the batch flow) | `bug-planner` + `bugfix-pipeline-launcher` | (this is the batch flow) |
|
|
401
428
|
|
|
402
429
|
---
|
|
@@ -1,12 +1,8 @@
|
|
|
1
1
|
# Structural Sync — Detailed Steps
|
|
2
2
|
|
|
3
|
-
**1a.** Get changed files:
|
|
3
|
+
**1a.** Get changed files (staged + unstaged vs HEAD):
|
|
4
4
|
```bash
|
|
5
|
-
git diff
|
|
6
|
-
```
|
|
7
|
-
If nothing staged, fallback:
|
|
8
|
-
```bash
|
|
9
|
-
git diff --name-status
|
|
5
|
+
git diff HEAD --name-status
|
|
10
6
|
```
|
|
11
7
|
|
|
12
8
|
**1b.** Read `.prizm-docs/root.prizm` to get MODULE_INDEX (or MODULE_GROUPS). Map each changed file to its module.
|
|
@@ -19,7 +15,7 @@ git diff --name-status
|
|
|
19
15
|
|
|
20
16
|
**1d.** Update affected docs (bottom-up: L2 → L1 → L0):
|
|
21
17
|
|
|
22
|
-
- **L2**: If L2 exists → update KEY_FILES, INTERFACES
|
|
18
|
+
- **L2**: If L2 exists → update **only the sections affected by the diff files in this module**. For example, if only `api.js` changed: update its KEY_FILES entry, its INTERFACES (if exports changed), its DEPENDENCIES (if imports changed). Do NOT re-scan unchanged files in the module. If L2 does NOT exist AND the module has Added or Modified source files in the current diff with meaningful logic (not trivial config) → create L2 with these sections: MODULE, FILES, RESPONSIBILITY, INTERFACES, DATA_FLOW, KEY_FILES, DEPENDENCIES, RULES, TRAPS, DECISIONS, CHANGELOG. Populate **only from the diff files** (the Added/Modified files in this module from step 1a), not from the entire module directory.
|
|
23
19
|
- **L1**: Update FILES count, KEY_FILES (if major files added/removed), DEPENDENCIES (if module-level deps changed). **L1 does NOT contain INTERFACES, DATA_FLOW, TRAPS, or DECISIONS** — those belong in L2 only.
|
|
24
20
|
- **L0 root.prizm**: Update MODULE_INDEX file counts only if counts changed. Update CROSS_CUTTING if cross-module concerns changed. Update only if structural change (module added/removed). **Preserve** any `PROJECT_BRIEF:` line — it is managed by prizmkit-init.
|
|
25
21
|
|
|
@@ -29,7 +29,7 @@ User says:
|
|
|
29
29
|
```
|
|
30
30
|
refactor-workflow <target / goals>
|
|
31
31
|
│
|
|
32
|
-
├── Phase 1: Brainstorm →
|
|
32
|
+
├── Phase 1: Brainstorm → clarify type → collect materials → parallel deep read → discuss plan
|
|
33
33
|
│
|
|
34
34
|
├── Phase 2: Plan → refactor-planner → .prizmkit/plans/refactor-list.json
|
|
35
35
|
│
|
|
@@ -42,7 +42,7 @@ refactor-workflow <target / goals>
|
|
|
42
42
|
|
|
43
43
|
| Phase | Action | Result |
|
|
44
44
|
|-------|--------|--------|
|
|
45
|
-
| 1 | **Brainstorm** —
|
|
45
|
+
| 1 | **Brainstorm** — clarify type, collect reference materials, parallel deep read code & docs, discuss plan grounded in real code | Fully clarified refactoring goals |
|
|
46
46
|
| 2 | Call `refactor-planner` with clarified goals | `.prizmkit/plans/refactor-list.json` with N refactor items |
|
|
47
47
|
| 3 | Call `refactor-pipeline-launcher` | Pipeline started (execution mode chosen by user via launcher) |
|
|
48
48
|
| 4 | Monitor progress | Status updates, completion report |
|
|
@@ -100,57 +100,85 @@ When user says "add more refactors" or the project already has a .prizmkit/plans
|
|
|
100
100
|
|
|
101
101
|
## Phase 1: Brainstorm — Deep Refactoring Goal Clarification
|
|
102
102
|
|
|
103
|
-
**Goal**: Through interactive Q&A, transform the user's rough refactoring idea into fully clarified, implementation-ready refactoring goals. This phase is the foundation for safe, behavior-preserving code changes — vague goals produce risky refactors.
|
|
103
|
+
**Goal**: Through interactive Q&A and deep code reading, transform the user's rough refactoring idea into fully clarified, implementation-ready refactoring goals. This phase is the foundation for safe, behavior-preserving code changes — vague goals produce risky refactors.
|
|
104
104
|
|
|
105
105
|
**CRITICAL RULE**: The number of questions is **unlimited**. Do NOT rush through this phase. Ask as many rounds as needed until every aspect is clear. Refactoring is inherently risky — thorough understanding prevents broken behavior.
|
|
106
106
|
|
|
107
|
-
### Step 1.1:
|
|
107
|
+
### Step 1.1: Clarify Refactoring Type
|
|
108
108
|
|
|
109
|
-
|
|
109
|
+
**First question** — ask the user to classify the refactoring approach:
|
|
110
|
+
|
|
111
|
+
| Type | Description | Example |
|
|
112
|
+
|------|-------------|---------|
|
|
113
|
+
| **Incremental** | Piece-by-piece restructuring, each step independently safe | "Gradually extract shared utilities over several PRs" |
|
|
114
|
+
| **Comprehensive** | Full rewrite of a module/area in one pass | "Rewrite the auth module with new architecture" |
|
|
115
|
+
| **Targeted** | Specific, focused change to a particular part | "Extract the validation logic from the controller" |
|
|
116
|
+
|
|
117
|
+
Then ask:
|
|
110
118
|
- **What** code needs restructuring (modules, files, patterns)
|
|
111
119
|
- **Why** it needs refactoring (tech debt, coupling, complexity, readability, performance structure)
|
|
112
120
|
- **What outcome** they want (target architecture, desired structure, quality goals)
|
|
113
121
|
|
|
114
|
-
### Step 1.2:
|
|
122
|
+
### Step 1.2: Collect Reference Materials
|
|
123
|
+
|
|
124
|
+
**Ask the user explicitly** what resources they have. Do NOT skip this step — user-provided materials are far more valuable than blind directory scanning.
|
|
125
|
+
|
|
126
|
+
Ask:
|
|
127
|
+
1. **Code paths** — "Which files or directories are the main targets? Any specific files I should look at?"
|
|
128
|
+
2. **Design documents** — "Do you have any design docs, architecture diagrams, or refactoring proposals I should read?"
|
|
129
|
+
3. **Knowledge docs** — "Are there related `.prizm-docs/`, README files, or internal wiki pages for the target area?"
|
|
130
|
+
4. **Related issues/PRs** — "Any related issues, PRs, or previous refactoring attempts I should be aware of?"
|
|
131
|
+
|
|
132
|
+
Record everything the user provides — these become inputs for Step 1.3.
|
|
133
|
+
|
|
134
|
+
### Step 1.3: Parallel Deep Reading
|
|
135
|
+
|
|
136
|
+
**Goal**: Build comprehensive understanding of the target code and context before discussing plans. Spawn multiple agents in parallel to read all relevant materials simultaneously.
|
|
137
|
+
|
|
138
|
+
**Parallel reading tasks** (launch concurrently):
|
|
139
|
+
|
|
140
|
+
| Agent | What to read | Purpose |
|
|
141
|
+
|-------|-------------|---------|
|
|
142
|
+
| Agent A | User-provided code paths — read full source files | Understand current structure, interfaces, dependencies |
|
|
143
|
+
| Agent B | User-provided documents — design docs, proposals, wiki pages | Understand intended direction and constraints |
|
|
144
|
+
| Agent C | `.prizm-docs/` for affected modules — L1/L2 docs, TRAPS, RULES | Understand existing architecture knowledge and known pitfalls |
|
|
145
|
+
| Agent D | Test files for the target area — find and read existing tests | Understand current test coverage and behavior contracts |
|
|
146
|
+
|
|
147
|
+
**Also gather** (can be included in any agent's task):
|
|
148
|
+
- `.prizmkit/config.json` → tech stack preferences
|
|
149
|
+
- Directory structure of the target area
|
|
150
|
+
- Dependency relationships (imports/exports between target and other modules)
|
|
151
|
+
|
|
152
|
+
**After all agents complete**: Synthesize findings into a coherent understanding before proceeding to discussion.
|
|
115
153
|
|
|
116
|
-
|
|
117
|
-
- If `.prizm-docs/root.prizm` exists → read it to understand existing architecture, tech stack, patterns
|
|
118
|
-
- If `.prizmkit/config.json` exists → read tech stack preferences
|
|
119
|
-
- If existing source code exists → scan directory structure:
|
|
120
|
-
```bash
|
|
121
|
-
find . -maxdepth 2 -type d -not -path '*/node_modules/*' -not -path '*/.git/*' -not -path '*/dist/*' -not -path '*/build/*' -not -path '*/__pycache__/*' -not -path '*/vendor/*' | sed -e 's;[^/]*/;|____;g;s;____|; |;g'
|
|
122
|
-
```
|
|
123
|
-
- Check test suite existence and coverage for the target area:
|
|
124
|
-
```bash
|
|
125
|
-
find . -maxdepth 4 -type f \( -name "*.test.*" -o -name "*.spec.*" -o -name "test_*" -o -path "*/tests/*" -o -path "*/__tests__/*" \) -not -path '*/node_modules/*' -not -path '*/.git/*' | head -20
|
|
126
|
-
```
|
|
154
|
+
### Step 1.4: Discuss Refactoring Plan
|
|
127
155
|
|
|
128
|
-
|
|
156
|
+
**Now** — with deep knowledge of the actual code and documents — discuss the refactoring plan with the user. This discussion is grounded in real code, not abstract questions.
|
|
129
157
|
|
|
130
|
-
|
|
158
|
+
Present what you learned from the parallel reading:
|
|
159
|
+
- Current code structure and its problems (with specific file/function references)
|
|
160
|
+
- Existing test coverage status (which areas are safe, which are risky)
|
|
161
|
+
- Known TRAPS and pitfalls from `.prizm-docs/`
|
|
162
|
+
- Dependencies and potential impact on other modules
|
|
131
163
|
|
|
132
|
-
|
|
164
|
+
Then ask targeted questions based on what you read. **Adapt question depth to the refactoring complexity** — a simple extract-method refactor needs fewer questions than a full module decomposition.
|
|
133
165
|
|
|
134
166
|
**Code Structure:**
|
|
135
|
-
-
|
|
167
|
+
- "I see the current structure does X — is the target state Y, or something different?"
|
|
136
168
|
- What's the target state? What should the code look like after refactoring?
|
|
137
|
-
- Are there specific code smells? (duplication, deep nesting, god classes, tight coupling)
|
|
138
|
-
- What are the dependency relationships between the target and other modules?
|
|
169
|
+
- Are there specific code smells you've noticed? (duplication, deep nesting, god classes, tight coupling)
|
|
139
170
|
|
|
140
171
|
**Scope:**
|
|
141
|
-
-
|
|
142
|
-
-
|
|
143
|
-
- Should the refactoring be incremental (piece by piece) or comprehensive?
|
|
172
|
+
- Based on the code I read, these modules are affected: [list]. Anything else in/out of scope?
|
|
173
|
+
- For incremental refactoring: what's the order of priority?
|
|
144
174
|
|
|
145
175
|
**Behavior Preservation:**
|
|
146
|
-
-
|
|
147
|
-
-
|
|
148
|
-
- Are there integration tests or end-to-end tests that cover this area?
|
|
176
|
+
- "These public APIs/interfaces exist: [list]. Which must remain unchanged?"
|
|
177
|
+
- "I found these tests: [list]. Are they passing currently?"
|
|
149
178
|
- Any undocumented behavior that callers depend on?
|
|
150
179
|
|
|
151
180
|
**Risk Assessment:**
|
|
152
|
-
-
|
|
153
|
-
- Are there known gotchas or traps in this code? (check `.prizm-docs/` TRAPS)
|
|
181
|
+
- "I found these TRAPS in .prizm-docs/: [list]. Any other known gotchas?"
|
|
154
182
|
- Does this code have external consumers (other teams, published APIs)?
|
|
155
183
|
- Any concurrent development happening in the target area?
|
|
156
184
|
|
|
@@ -159,14 +187,14 @@ Based on the user's description, ask questions across these dimensions. **Adapt
|
|
|
159
187
|
- Team coordination needed? (other developers working in the same area)
|
|
160
188
|
- Deployment concerns? (feature flags, backward compatibility, migration)
|
|
161
189
|
|
|
162
|
-
### Step 1.
|
|
190
|
+
### Step 1.5: Confirm and Supplement
|
|
163
191
|
|
|
164
|
-
After the
|
|
192
|
+
After the discussion:
|
|
165
193
|
|
|
166
|
-
1. **Summarize**
|
|
167
|
-
2. **
|
|
168
|
-
3. **
|
|
169
|
-
4. **Repeat** until the user confirms: "
|
|
194
|
+
1. **Summarize** the refactoring plan — present it back to the user
|
|
195
|
+
2. **Ask explicitly**: "Is there anything else you'd like to discuss or supplement before we proceed to formal planning?"
|
|
196
|
+
3. **Identify gaps** — if any areas are still unclear, list them explicitly and ask follow-up questions
|
|
197
|
+
4. **Repeat** until the user confirms: "That covers everything" or "Let's proceed"
|
|
170
198
|
|
|
171
199
|
**Signs that brainstorming is complete:**
|
|
172
200
|
- All refactoring goals have concrete target state descriptions
|
|
@@ -182,7 +210,7 @@ After the initial deep-dive:
|
|
|
182
210
|
- Risk areas are handwaved ("it should be fine")
|
|
183
211
|
- User says "I'm not sure" — help them think through it with concrete options
|
|
184
212
|
|
|
185
|
-
### Step 1.
|
|
213
|
+
### Step 1.6: Requirements Summary
|
|
186
214
|
|
|
187
215
|
Once brainstorming is complete, produce a structured goals summary:
|
|
188
216
|
|
|
@@ -191,11 +219,13 @@ Once brainstorming is complete, produce a structured goals summary:
|
|
|
191
219
|
|
|
192
220
|
### Target: [Module/area name]
|
|
193
221
|
|
|
222
|
+
### Refactoring Type: [Incremental / Comprehensive / Targeted]
|
|
223
|
+
|
|
194
224
|
### Refactoring Objectives
|
|
195
225
|
- [Bullet list of what structural changes are needed and why]
|
|
196
226
|
|
|
197
227
|
### Current Problems
|
|
198
|
-
- [What's wrong with the current structure]
|
|
228
|
+
- [What's wrong with the current structure — with specific code references]
|
|
199
229
|
|
|
200
230
|
### Target State
|
|
201
231
|
- [What the code should look like after refactoring]
|
|
@@ -212,6 +242,9 @@ Once brainstorming is complete, produce a structured goals summary:
|
|
|
212
242
|
### Risk Assessment
|
|
213
243
|
- [Risk]: [Mitigation strategy]
|
|
214
244
|
|
|
245
|
+
### Reference Materials Reviewed
|
|
246
|
+
- [List of code paths, documents, .prizm-docs/ files that were read]
|
|
247
|
+
|
|
215
248
|
### Constraints
|
|
216
249
|
- [Timeline, coordination, deployment concerns]
|
|
217
250
|
|
|
@@ -368,34 +401,7 @@ While the pipeline runs, the user can continue the conversation:
|
|
|
368
401
|
| User wants to cancel mid-brainstorming | Save conversation context, offer to resume later |
|
|
369
402
|
| Behavior regression detected during pipeline | Pipeline handles per-item — failed items are retried or reported |
|
|
370
403
|
|
|
371
|
-
---
|
|
372
|
-
|
|
373
|
-
## Relationship to Other Skills
|
|
374
|
-
|
|
375
|
-
| Skill | Relationship |
|
|
376
|
-
|-------|-------------|
|
|
377
|
-
| `refactor-planner` | **Called by Phase 2** — generates .prizmkit/plans/refactor-list.json from clarified goals |
|
|
378
|
-
| `refactor-pipeline-launcher` | **Called by Phase 3** — starts pipeline (handles execution mode selection) |
|
|
379
|
-
| `feature-workflow` | **Alternative** — for new feature development |
|
|
380
|
-
| `bug-fix-workflow` | **Alternative** — for bug fix workflows |
|
|
381
404
|
|
|
382
|
-
---
|
|
383
|
-
|
|
384
|
-
## Comparison with Alternative Workflows
|
|
385
|
-
|
|
386
|
-
| Dimension | refactor-workflow | feature-workflow | bug-fix-workflow |
|
|
387
|
-
|-----------|-------------------|-----------------|------------------|
|
|
388
|
-
| **Purpose** | Code restructuring (batch) | New features (batch) | Single bug fix (interactive) |
|
|
389
|
-
| **Brainstorming** | Yes — deep interactive Q&A on refactoring goals | Yes — deep interactive Q&A on requirements | No (bug report is input) |
|
|
390
|
-
| **Planning Skill** | `refactor-planner` | `feature-planner` | None (triage built-in) |
|
|
391
|
-
| **Branch** | Pipeline manages per-refactor | Pipeline manages per-feature | `fix/<BUG_ID>-*` |
|
|
392
|
-
| **Execution** | Foreground or background daemon | Foreground or background daemon | In-session, interactive |
|
|
393
|
-
| **Input** | Rough refactoring idea or target | Rough idea or requirements | Bug report / stack trace |
|
|
394
|
-
| **Output** | Multiple `refactor()` commits | Multiple `feat()` commits | Single `fix()` commit |
|
|
395
|
-
| **Behavior Change** | Forbidden (structure only) | Expected (new functionality) | Fix behavior |
|
|
396
|
-
| **Batch alternative** | (this is the batch flow) | (this is the batch flow) | `bug-planner` + `bugfix-pipeline-launcher` |
|
|
397
|
-
|
|
398
|
-
---
|
|
399
405
|
|
|
400
406
|
## Output
|
|
401
407
|
|