@codemcp/workflows-core 5.2.2 → 5.2.4
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/package.json +1 -1
- package/resources/workflows/adr.yaml +62 -48
- package/resources/workflows/big-bang-conversion.yaml +231 -172
- package/resources/workflows/boundary-testing.yaml +137 -95
- package/resources/workflows/bugfix.yaml +33 -38
- package/resources/workflows/business-analysis.yaml +130 -172
- package/resources/workflows/c4-analysis.yaml +120 -114
- package/resources/workflows/epcc.yaml +43 -10
- package/resources/workflows/game-beginner.yaml +113 -103
- package/resources/workflows/greenfield.yaml +61 -38
- package/resources/workflows/minor.yaml +51 -28
- package/resources/workflows/posts.yaml +41 -42
- package/resources/workflows/sdd-bugfix-crowd.yaml +181 -206
- package/resources/workflows/sdd-bugfix.yaml +168 -179
- package/resources/workflows/sdd-feature-crowd.yaml +357 -380
- package/resources/workflows/sdd-feature.yaml +162 -155
- package/resources/workflows/sdd-greenfield-crowd.yaml +82 -74
- package/resources/workflows/sdd-greenfield.yaml +153 -147
- package/resources/workflows/slides.yaml +29 -35
- package/resources/workflows/tdd.yaml +59 -48
- package/resources/workflows/waterfall.yaml +64 -44
|
@@ -31,8 +31,7 @@ states:
|
|
|
31
31
|
default_instructions: |
|
|
32
32
|
**Team Collaboration Mode**
|
|
33
33
|
|
|
34
|
-
You are
|
|
35
|
-
Current phase: REPRODUCE
|
|
34
|
+
You are `$VIBE_ROLE` working in a collaborative team on this bug fix.
|
|
36
35
|
|
|
37
36
|
Developer is responsible. Business-analyst and architect are consulted.
|
|
38
37
|
|
|
@@ -43,9 +42,9 @@ states:
|
|
|
43
42
|
role: business-analyst
|
|
44
43
|
transition_reason: 'Bug reproduced, preparing to specify correct behavior'
|
|
45
44
|
additional_instructions: |
|
|
46
|
-
**
|
|
45
|
+
**Consultant Role in Reproduce Phase**
|
|
47
46
|
|
|
48
|
-
|
|
47
|
+
The developer is investigating the bug. Your role is to provide consultative support.
|
|
49
48
|
|
|
50
49
|
**Your Responsibilities:**
|
|
51
50
|
- Monitor messages for questions about expected behavior
|
|
@@ -53,12 +52,12 @@ states:
|
|
|
53
52
|
- Explain the correct functionality from user perspective
|
|
54
53
|
|
|
55
54
|
**Important Constraints:**
|
|
56
|
-
- Do NOT edit the plan file (only developer can
|
|
55
|
+
- Do NOT edit the plan file (only developer can edit)
|
|
57
56
|
- Do NOT attempt to proceed to next phase
|
|
58
57
|
|
|
59
|
-
**
|
|
60
|
-
- When developer completes reproduce phase,
|
|
61
|
-
- You will
|
|
58
|
+
**Transition Notice:**
|
|
59
|
+
- When developer completes the reproduce phase, you become RESPONSIBLE
|
|
60
|
+
- You will lead the specify phase
|
|
62
61
|
- Prepare to document the correct behavior specification
|
|
63
62
|
|
|
64
63
|
# Architect is CONSULTED during reproduce phase
|
|
@@ -67,9 +66,9 @@ states:
|
|
|
67
66
|
role: architect
|
|
68
67
|
transition_reason: 'Bug reproduced, transitioning to specification'
|
|
69
68
|
additional_instructions: |
|
|
70
|
-
**
|
|
69
|
+
**Consultant Role in Reproduce Phase**
|
|
71
70
|
|
|
72
|
-
|
|
71
|
+
The developer is investigating the bug. Your role is to provide consultative support.
|
|
73
72
|
|
|
74
73
|
**Your Responsibilities:**
|
|
75
74
|
- Monitor messages for questions about system design
|
|
@@ -77,11 +76,11 @@ states:
|
|
|
77
76
|
- Explain expected component interactions
|
|
78
77
|
|
|
79
78
|
**Important Constraints:**
|
|
80
|
-
- Do NOT edit the plan file (only developer can)
|
|
79
|
+
- Do NOT edit the plan file (only developer can edit)
|
|
81
80
|
- Do NOT attempt to proceed to next phase
|
|
82
81
|
|
|
83
82
|
**How to Help:**
|
|
84
|
-
- Use get_my_messages to check for developer
|
|
83
|
+
- Use get_my_messages to check for developer questions
|
|
85
84
|
- Provide architectural context for the buggy component
|
|
86
85
|
|
|
87
86
|
# Developer is RESPONSIBLE for reproduce phase
|
|
@@ -90,43 +89,39 @@ states:
|
|
|
90
89
|
role: developer
|
|
91
90
|
transition_reason: 'Bug reproduced successfully, handing off to business-analyst'
|
|
92
91
|
additional_instructions: |
|
|
93
|
-
**
|
|
92
|
+
**Responsible for Reproduce Phase**
|
|
94
93
|
|
|
95
94
|
You have exclusive control during this phase:
|
|
96
95
|
- Only you can edit the plan file
|
|
97
96
|
- Only you can proceed to the next phase
|
|
98
97
|
|
|
99
|
-
**
|
|
98
|
+
**STEP 1: Gather Information**
|
|
99
|
+
- Collect exact OS, browser/runtime versions, hardware specs
|
|
100
|
+
- Document precise sequence of actions triggering the bug
|
|
101
|
+
- Obtain error messages, logs, stack traces
|
|
102
|
+
- Determine frequency (always or intermittent?)
|
|
103
|
+
- Assess business impact
|
|
100
104
|
|
|
101
|
-
|
|
102
|
-
|
|
103
|
-
|
|
104
|
-
|
|
105
|
-
|
|
106
|
-
- Business impact
|
|
105
|
+
**STEP 2: Create Reproduction**
|
|
106
|
+
- Develop minimal, reliable reproduction steps
|
|
107
|
+
- Document exact environment and conditions
|
|
108
|
+
- Create test data/scenarios that trigger the issue
|
|
109
|
+
- Verify bug occurs consistently
|
|
107
110
|
|
|
108
|
-
|
|
109
|
-
|
|
110
|
-
|
|
111
|
-
|
|
112
|
-
|
|
111
|
+
**STEP 3: Understand the Problem**
|
|
112
|
+
- Identify current (buggy) behavior
|
|
113
|
+
- Determine correct behavior
|
|
114
|
+
- Distinguish symptoms vs. root cause
|
|
115
|
+
- Identify related issues or edge cases
|
|
113
116
|
|
|
114
|
-
|
|
115
|
-
|
|
116
|
-
|
|
117
|
-
- Symptoms vs. root cause?
|
|
118
|
-
- Related issues or edge cases?
|
|
117
|
+
**STEP 4: Collaborate When Needed**
|
|
118
|
+
- Use send_message to business-analyst: "What is the expected behavior for [scenario]?"
|
|
119
|
+
- Use send_message to architect: "How should this component interact with [other component]?"
|
|
119
120
|
|
|
120
|
-
|
|
121
|
-
- Use send_message to business-analyst: "What is the expected behavior for [scenario]?"
|
|
122
|
-
- Use send_message to architect: "How should this component interact with [other component]?"
|
|
123
|
-
|
|
124
|
-
**Output**: Create `$VIBE_DIR/specs/$BRANCH_NAME/reproduction.md`
|
|
121
|
+
**Deliverable**: Create `$VIBE_DIR/specs/$BRANCH_NAME/reproduction.md`
|
|
125
122
|
|
|
126
123
|
**Handoff to Business-Analyst:**
|
|
127
|
-
|
|
128
|
-
Before proceeding to specify phase:
|
|
129
|
-
1. Use send_message to business-analyst: "Bug reproduced successfully. Please take the lead for specify phase to document correct behavior. See: $VIBE_DIR/specs/$BRANCH_NAME/reproduction.md"
|
|
124
|
+
1. Use send_message to business-analyst: "Bug reproduced successfully. Please take the lead for specify phase to document correct behavior. See: `$VIBE_DIR/specs/$BRANCH_NAME/reproduction.md`"
|
|
130
125
|
2. Use send_message to architect: "Bug reproduced, business-analyst will specify correct behavior"
|
|
131
126
|
3. Use send_message_to_operator: "Reproduction complete, handing off to business-analyst"
|
|
132
127
|
4. Call proceed_to_phase - you transition to CONSULTED in specify phase
|
|
@@ -145,8 +140,7 @@ states:
|
|
|
145
140
|
default_instructions: |
|
|
146
141
|
**Team Collaboration Mode**
|
|
147
142
|
|
|
148
|
-
You are
|
|
149
|
-
Current phase: SPECIFY
|
|
143
|
+
You are `$VIBE_ROLE` working in a collaborative team on this bug fix.
|
|
150
144
|
|
|
151
145
|
Business-analyst is responsible. Developer and architect are consulted.
|
|
152
146
|
|
|
@@ -157,35 +151,31 @@ states:
|
|
|
157
151
|
role: business-analyst
|
|
158
152
|
transition_reason: 'Bug specification completed, developer will create tests'
|
|
159
153
|
additional_instructions: |
|
|
160
|
-
**
|
|
154
|
+
**Responsible for Specify Phase**
|
|
161
155
|
|
|
162
156
|
You have exclusive control during this phase:
|
|
163
157
|
- Only you can edit the plan file
|
|
164
158
|
- Only you can proceed to the next phase
|
|
165
159
|
|
|
166
|
-
**
|
|
160
|
+
**STEP 1: Define Correct Behavior**
|
|
161
|
+
- Determine what should happen in the failing scenario
|
|
162
|
+
- Document expected outcomes for users
|
|
163
|
+
- Specify system behavior in edge cases
|
|
164
|
+
- Clarify business rules to enforce
|
|
167
165
|
|
|
168
|
-
|
|
169
|
-
|
|
170
|
-
|
|
171
|
-
|
|
172
|
-
- Business rules to enforce
|
|
166
|
+
**STEP 2: Handle Ambiguities**
|
|
167
|
+
- Make informed decisions about unclear requirements
|
|
168
|
+
- Use [NEEDS CLARIFICATION: question] only for critical ambiguities (max 3)
|
|
169
|
+
- Prioritize: correctness > user experience > technical details
|
|
173
170
|
|
|
174
|
-
|
|
175
|
-
|
|
176
|
-
|
|
177
|
-
- Prioritize: correctness > user experience > technical details
|
|
171
|
+
**STEP 3: Collaborate**
|
|
172
|
+
- Use send_message to developer: "Does this specification clearly define the fix?"
|
|
173
|
+
- Use send_message to architect: "Does this align with system design intent?"
|
|
178
174
|
|
|
179
|
-
|
|
180
|
-
- Use send_message to developer: "Does this specification clearly define the fix?"
|
|
181
|
-
- Use send_message to architect: "Does this align with system design intent?"
|
|
182
|
-
|
|
183
|
-
**Output**: Create `$VIBE_DIR/specs/$BRANCH_NAME/bug-spec.md`
|
|
175
|
+
**Deliverable**: Create `$VIBE_DIR/specs/$BRANCH_NAME/bug-spec.md`
|
|
184
176
|
|
|
185
177
|
**Handoff to Developer:**
|
|
186
|
-
|
|
187
|
-
Before proceeding to test phase:
|
|
188
|
-
1. Use send_message to developer: "Bug specification complete. Please take the lead for test phase to create failing tests. See: $VIBE_DIR/specs/$BRANCH_NAME/bug-spec.md"
|
|
178
|
+
1. Use send_message to developer: "Bug specification complete. Please take the lead for test phase to create failing tests. See: `$VIBE_DIR/specs/$BRANCH_NAME/bug-spec.md`"
|
|
189
179
|
2. Use send_message to architect: "Specification ready, developer will create tests and fix"
|
|
190
180
|
3. Use send_message_to_operator: "Bug spec complete, handing off to developer for testing"
|
|
191
181
|
4. Call proceed_to_phase - you transition to CONSULTED in test phase
|
|
@@ -196,9 +186,9 @@ states:
|
|
|
196
186
|
role: architect
|
|
197
187
|
transition_reason: 'Bug specification completed, transitioning to test phase'
|
|
198
188
|
additional_instructions: |
|
|
199
|
-
**
|
|
189
|
+
**Consultant Role in Specify Phase**
|
|
200
190
|
|
|
201
|
-
|
|
191
|
+
The business-analyst is defining correct behavior. Your role is to provide consultative support.
|
|
202
192
|
|
|
203
193
|
**Your Responsibilities:**
|
|
204
194
|
- Monitor messages for questions about system design intent
|
|
@@ -206,7 +196,7 @@ states:
|
|
|
206
196
|
- Validate specification against architectural principles
|
|
207
197
|
|
|
208
198
|
**Important Constraints:**
|
|
209
|
-
- Do NOT edit the plan file (only business-analyst can)
|
|
199
|
+
- Do NOT edit the plan file (only business-analyst can edit)
|
|
210
200
|
- Do NOT attempt to proceed to next phase
|
|
211
201
|
|
|
212
202
|
# Developer is CONSULTED during specify phase
|
|
@@ -215,9 +205,9 @@ states:
|
|
|
215
205
|
role: developer
|
|
216
206
|
transition_reason: 'Bug specification completed, preparing to create tests'
|
|
217
207
|
additional_instructions: |
|
|
218
|
-
**
|
|
208
|
+
**Consultant Role in Specify Phase**
|
|
219
209
|
|
|
220
|
-
|
|
210
|
+
The business-analyst is defining correct behavior. Your role is to provide consultative support.
|
|
221
211
|
|
|
222
212
|
**Your Responsibilities:**
|
|
223
213
|
- Monitor messages for questions about testability
|
|
@@ -225,12 +215,12 @@ states:
|
|
|
225
215
|
- Identify any technical constraints
|
|
226
216
|
|
|
227
217
|
**Important Constraints:**
|
|
228
|
-
- Do NOT edit the plan file (only business-analyst can)
|
|
218
|
+
- Do NOT edit the plan file (only business-analyst can edit)
|
|
229
219
|
- Do NOT attempt to proceed to next phase
|
|
230
220
|
|
|
231
|
-
**
|
|
232
|
-
- When business-analyst completes specify phase,
|
|
233
|
-
- You will
|
|
221
|
+
**Transition Notice:**
|
|
222
|
+
- When business-analyst completes the specify phase, you become RESPONSIBLE
|
|
223
|
+
- You will lead the test phase
|
|
234
224
|
- Prepare to create failing tests that capture the bug
|
|
235
225
|
|
|
236
226
|
test:
|
|
@@ -238,8 +228,7 @@ states:
|
|
|
238
228
|
default_instructions: |
|
|
239
229
|
**Team Collaboration Mode**
|
|
240
230
|
|
|
241
|
-
You are
|
|
242
|
-
Current phase: TEST
|
|
231
|
+
You are `$VIBE_ROLE` working in a collaborative team on this bug fix.
|
|
243
232
|
|
|
244
233
|
Developer is responsible for creating tests.
|
|
245
234
|
|
|
@@ -250,9 +239,9 @@ states:
|
|
|
250
239
|
role: business-analyst
|
|
251
240
|
transition_reason: 'Tests created, architect will plan the fix'
|
|
252
241
|
additional_instructions: |
|
|
253
|
-
**
|
|
242
|
+
**Consultant Role in Test Phase**
|
|
254
243
|
|
|
255
|
-
|
|
244
|
+
The developer is creating tests. Your role is to provide consultative support.
|
|
256
245
|
|
|
257
246
|
**Your Responsibilities:**
|
|
258
247
|
- Monitor messages for questions about acceptance criteria
|
|
@@ -260,7 +249,7 @@ states:
|
|
|
260
249
|
- Ensure tests verify correct business behavior
|
|
261
250
|
|
|
262
251
|
**Important Constraints:**
|
|
263
|
-
- Do NOT edit the plan file (only developer can)
|
|
252
|
+
- Do NOT edit the plan file (only developer can edit)
|
|
264
253
|
- Do NOT attempt to proceed to next phase
|
|
265
254
|
|
|
266
255
|
# Architect is CONSULTED during test phase
|
|
@@ -269,21 +258,21 @@ states:
|
|
|
269
258
|
role: architect
|
|
270
259
|
transition_reason: 'Tests created, preparing to plan fix approach'
|
|
271
260
|
additional_instructions: |
|
|
272
|
-
**
|
|
261
|
+
**Consultant Role in Test Phase**
|
|
273
262
|
|
|
274
|
-
|
|
263
|
+
The developer is creating tests. Your role is to provide consultative support.
|
|
275
264
|
|
|
276
265
|
**Your Responsibilities:**
|
|
277
266
|
- Monitor messages for questions about test architecture
|
|
278
267
|
- Provide guidance on integration testing approach
|
|
279
268
|
|
|
280
269
|
**Important Constraints:**
|
|
281
|
-
- Do NOT edit the plan file (only developer can)
|
|
270
|
+
- Do NOT edit the plan file (only developer can edit)
|
|
282
271
|
- Do NOT attempt to proceed to next phase
|
|
283
272
|
|
|
284
|
-
**
|
|
285
|
-
- When developer completes test phase,
|
|
286
|
-
- You will
|
|
273
|
+
**Transition Notice:**
|
|
274
|
+
- When developer completes the test phase, you become RESPONSIBLE
|
|
275
|
+
- You will lead the plan phase
|
|
287
276
|
- Prepare to design the fix approach
|
|
288
277
|
|
|
289
278
|
# Developer is RESPONSIBLE for test phase
|
|
@@ -292,37 +281,35 @@ states:
|
|
|
292
281
|
role: developer
|
|
293
282
|
transition_reason: 'Tests created, handing off to architect for fix planning'
|
|
294
283
|
additional_instructions: |
|
|
295
|
-
**
|
|
284
|
+
**Responsible for Test Phase**
|
|
296
285
|
|
|
297
286
|
You have exclusive control during this phase:
|
|
298
287
|
- Only you can edit the plan file
|
|
299
288
|
- Only you can proceed to the next phase
|
|
300
289
|
|
|
301
|
-
**
|
|
290
|
+
**STEP 1: Create Failing Tests for Current Bug**
|
|
291
|
+
- Create tests demonstrating current buggy behavior
|
|
292
|
+
- These should fail initially (Red phase of TDD)
|
|
293
|
+
- Cover main reproduction case and related scenarios
|
|
294
|
+
- Include edge cases and boundary conditions
|
|
302
295
|
|
|
303
|
-
|
|
304
|
-
|
|
305
|
-
|
|
306
|
-
|
|
307
|
-
|
|
296
|
+
**STEP 2: Create Tests for Expected Behavior**
|
|
297
|
+
- Create tests specifying correct behavior from bug-spec.md
|
|
298
|
+
- These should pass once bug is fixed
|
|
299
|
+
- Cover all functional requirements
|
|
300
|
+
- Include user scenarios and success criteria
|
|
308
301
|
|
|
309
|
-
|
|
310
|
-
|
|
311
|
-
|
|
312
|
-
- Cover all functional requirements
|
|
313
|
-
- Include user scenarios and success criteria
|
|
302
|
+
**STEP 3: Create Regression Tests**
|
|
303
|
+
- Tests for related functionality that shouldn't be affected
|
|
304
|
+
- Ensure fix doesn't break existing features
|
|
314
305
|
|
|
315
|
-
|
|
316
|
-
|
|
317
|
-
|
|
306
|
+
**STEP 4: Collaborate**
|
|
307
|
+
- Use send_message to business-analyst: "Do these tests cover all scenarios from the spec?"
|
|
308
|
+
- Use send_message to architect: "Is the test architecture appropriate?"
|
|
318
309
|
|
|
319
|
-
|
|
320
|
-
- Use send_message to business-analyst: "Do these tests cover all scenarios from the spec?"
|
|
321
|
-
- Use send_message to architect: "Is the test architecture appropriate?"
|
|
310
|
+
**Deliverable**: Create `$VIBE_DIR/specs/$BRANCH_NAME/tests.md`
|
|
322
311
|
|
|
323
312
|
**Handoff to Architect:**
|
|
324
|
-
|
|
325
|
-
Before proceeding to plan phase:
|
|
326
313
|
1. Use send_message to architect: "Tests created and failing as expected. Please take the lead for plan phase to design the fix approach."
|
|
327
314
|
2. Use send_message to business-analyst: "Tests ready, architect will plan the fix"
|
|
328
315
|
3. Use send_message_to_operator: "Tests complete, handing off to architect for fix planning"
|
|
@@ -333,8 +320,7 @@ states:
|
|
|
333
320
|
default_instructions: |
|
|
334
321
|
**Team Collaboration Mode**
|
|
335
322
|
|
|
336
|
-
You are
|
|
337
|
-
Current phase: PLAN
|
|
323
|
+
You are `$VIBE_ROLE` working in a collaborative team on this bug fix.
|
|
338
324
|
|
|
339
325
|
Architect is responsible. Business-analyst and developer are consulted.
|
|
340
326
|
|
|
@@ -345,9 +331,9 @@ states:
|
|
|
345
331
|
role: business-analyst
|
|
346
332
|
transition_reason: 'Fix plan complete, developer will implement'
|
|
347
333
|
additional_instructions: |
|
|
348
|
-
**
|
|
334
|
+
**Consultant Role in Plan Phase**
|
|
349
335
|
|
|
350
|
-
|
|
336
|
+
The architect is designing the fix approach. Your role is to provide consultative support.
|
|
351
337
|
|
|
352
338
|
**Your Responsibilities:**
|
|
353
339
|
- Monitor messages for questions about business rules
|
|
@@ -355,7 +341,7 @@ states:
|
|
|
355
341
|
- Ensure fix aligns with user expectations
|
|
356
342
|
|
|
357
343
|
**Important Constraints:**
|
|
358
|
-
- Do NOT edit the plan file (only architect can)
|
|
344
|
+
- Do NOT edit the plan file (only architect can edit)
|
|
359
345
|
- Do NOT attempt to proceed to next phase
|
|
360
346
|
|
|
361
347
|
# Architect is RESPONSIBLE for plan phase
|
|
@@ -364,42 +350,38 @@ states:
|
|
|
364
350
|
role: architect
|
|
365
351
|
transition_reason: 'Fix plan completed, handing off to developer'
|
|
366
352
|
additional_instructions: |
|
|
367
|
-
**
|
|
353
|
+
**Responsible for Plan Phase**
|
|
368
354
|
|
|
369
355
|
You have exclusive control during this phase:
|
|
370
356
|
- Only you can edit the plan file
|
|
371
357
|
- Only you can proceed to the next phase
|
|
372
358
|
|
|
373
|
-
**
|
|
374
|
-
|
|
375
|
-
|
|
376
|
-
|
|
377
|
-
|
|
378
|
-
- Understand test requirements
|
|
379
|
-
- Analyze existing codebase
|
|
359
|
+
**STEP 1: Load Context**
|
|
360
|
+
- Read bug spec: `$VIBE_DIR/specs/$BRANCH_NAME/bug-spec.md`
|
|
361
|
+
- Review reproduction: `$VIBE_DIR/specs/$BRANCH_NAME/reproduction.md`
|
|
362
|
+
- Understand test requirements
|
|
363
|
+
- Analyze existing codebase
|
|
380
364
|
|
|
381
|
-
2
|
|
382
|
-
|
|
383
|
-
|
|
384
|
-
|
|
365
|
+
**STEP 2: Perform Root Cause Analysis**
|
|
366
|
+
- Identify underlying cause of the bug
|
|
367
|
+
- Understand why current implementation fails
|
|
368
|
+
- Map problem to specific code areas or logic flaws
|
|
385
369
|
|
|
386
|
-
3
|
|
387
|
-
|
|
388
|
-
|
|
389
|
-
|
|
390
|
-
|
|
370
|
+
**STEP 3: Design Fix Strategy**
|
|
371
|
+
- Design minimal, targeted fix addressing root cause
|
|
372
|
+
- Ensure fix aligns with existing architecture patterns
|
|
373
|
+
- Consider impact on related functionality
|
|
374
|
+
- Plan for backward compatibility if needed
|
|
391
375
|
|
|
392
|
-
4
|
|
393
|
-
|
|
394
|
-
|
|
395
|
-
|
|
376
|
+
**STEP 4: Collaborate**
|
|
377
|
+
- Use send_message to business-analyst: "Does this fix approach meet business needs?"
|
|
378
|
+
- Use send_message to developer: "Is this fix approach implementable?"
|
|
379
|
+
- Incorporate their feedback
|
|
396
380
|
|
|
397
|
-
**
|
|
381
|
+
**Deliverable**: Create `$VIBE_DIR/specs/$BRANCH_NAME/fix-plan.md`
|
|
398
382
|
|
|
399
383
|
**Handoff to Developer:**
|
|
400
|
-
|
|
401
|
-
Before proceeding to fix phase:
|
|
402
|
-
1. Use send_message to developer: "Fix plan complete. Please take the lead for fix phase. See: $VIBE_DIR/specs/$BRANCH_NAME/fix-plan.md"
|
|
384
|
+
1. Use send_message to developer: "Fix plan complete. Please take the lead for fix phase. See: `$VIBE_DIR/specs/$BRANCH_NAME/fix-plan.md`"
|
|
403
385
|
2. Use send_message to business-analyst: "Plan ready, developer will implement fix"
|
|
404
386
|
3. Use send_message_to_operator: "Fix plan complete, handing off to developer"
|
|
405
387
|
4. Call proceed_to_phase - you transition to CONSULTED in fix phase
|
|
@@ -410,24 +392,23 @@ states:
|
|
|
410
392
|
role: developer
|
|
411
393
|
transition_reason: 'Fix plan completed, preparing to implement'
|
|
412
394
|
additional_instructions: |
|
|
413
|
-
**
|
|
395
|
+
**Consultant Role in Plan Phase**
|
|
414
396
|
|
|
415
|
-
|
|
397
|
+
The architect is designing fix approach. Your role is to provide consultative support.
|
|
416
398
|
|
|
417
399
|
**Your Responsibilities:**
|
|
418
400
|
- Monitor messages for questions about implementation
|
|
419
|
-
- Provide feedback on
|
|
420
|
-
|
|
421
|
-
|
|
422
|
-
* Testing approach
|
|
401
|
+
- Provide feedback on fix implementability and complexity
|
|
402
|
+
- Identify potential implementation challenges
|
|
403
|
+
- Discuss testing approach
|
|
423
404
|
|
|
424
405
|
**Important Constraints:**
|
|
425
|
-
- Do NOT edit the plan file (only architect can)
|
|
406
|
+
- Do NOT edit the plan file (only architect can edit)
|
|
426
407
|
- Do NOT attempt to proceed to next phase
|
|
427
408
|
|
|
428
|
-
**
|
|
429
|
-
- When architect completes plan phase,
|
|
430
|
-
- You will
|
|
409
|
+
**Transition Notice:**
|
|
410
|
+
- When architect completes the plan phase, you become RESPONSIBLE
|
|
411
|
+
- You will lead the fix phase
|
|
431
412
|
- Prepare to implement the minimal fix
|
|
432
413
|
|
|
433
414
|
fix:
|
|
@@ -435,8 +416,7 @@ states:
|
|
|
435
416
|
default_instructions: |
|
|
436
417
|
**Team Collaboration Mode**
|
|
437
418
|
|
|
438
|
-
You are
|
|
439
|
-
Current phase: FIX
|
|
419
|
+
You are `$VIBE_ROLE` working in a collaborative team on this bug fix.
|
|
440
420
|
|
|
441
421
|
Developer is responsible. Architect and business-analyst are consulted.
|
|
442
422
|
|
|
@@ -447,9 +427,9 @@ states:
|
|
|
447
427
|
role: business-analyst
|
|
448
428
|
transition_reason: 'Fix implemented, team will verify'
|
|
449
429
|
additional_instructions: |
|
|
450
|
-
**
|
|
430
|
+
**Consultant Role in Fix Phase**
|
|
451
431
|
|
|
452
|
-
|
|
432
|
+
The developer is implementing the fix. Your role is to provide consultative support.
|
|
453
433
|
|
|
454
434
|
**Your Responsibilities:**
|
|
455
435
|
- Monitor messages for requirements clarifications
|
|
@@ -457,7 +437,7 @@ states:
|
|
|
457
437
|
- Approve acceptance criteria when asked
|
|
458
438
|
|
|
459
439
|
**Important Constraints:**
|
|
460
|
-
- Do NOT edit the plan file (only developer can)
|
|
440
|
+
- Do NOT edit the plan file (only developer can edit)
|
|
461
441
|
- Do NOT attempt to proceed to next phase
|
|
462
442
|
|
|
463
443
|
# Architect is CONSULTED during fix phase
|
|
@@ -466,19 +446,18 @@ states:
|
|
|
466
446
|
role: architect
|
|
467
447
|
transition_reason: 'Fix implemented, team will verify'
|
|
468
448
|
additional_instructions: |
|
|
469
|
-
**
|
|
449
|
+
**Consultant Role in Fix Phase**
|
|
470
450
|
|
|
471
|
-
|
|
451
|
+
The developer is implementing the fix. Your role is to provide consultative support.
|
|
472
452
|
|
|
473
453
|
**Your Responsibilities:**
|
|
474
454
|
- Monitor messages for architecture questions
|
|
475
|
-
- Provide guidance on
|
|
476
|
-
|
|
477
|
-
|
|
478
|
-
* Technical trade-offs
|
|
455
|
+
- Provide guidance on design patterns and best practices
|
|
456
|
+
- Recommend integration approaches
|
|
457
|
+
- Discuss technical trade-offs
|
|
479
458
|
|
|
480
459
|
**Important Constraints:**
|
|
481
|
-
- Do NOT edit the plan file (only developer can)
|
|
460
|
+
- Do NOT edit the plan file (only developer can edit)
|
|
482
461
|
- Do NOT attempt to proceed to next phase
|
|
483
462
|
|
|
484
463
|
# Developer is RESPONSIBLE for fix phase
|
|
@@ -487,36 +466,35 @@ states:
|
|
|
487
466
|
role: developer
|
|
488
467
|
transition_reason: 'Fix implemented and tests passing, ready for verification'
|
|
489
468
|
additional_instructions: |
|
|
490
|
-
**
|
|
469
|
+
**Responsible for Fix Phase**
|
|
491
470
|
|
|
492
471
|
You have exclusive control during this phase:
|
|
493
472
|
- Only you can edit the plan file
|
|
494
473
|
- Only you can proceed to the next phase
|
|
495
474
|
|
|
496
|
-
**Fix
|
|
475
|
+
**STEP 1: Execute Fix Strategy**
|
|
476
|
+
- Execute fix strategy from `$VIBE_DIR/specs/$BRANCH_NAME/fix-plan.md`
|
|
477
|
+
- Make targeted changes to address root cause
|
|
478
|
+
- Avoid over-engineering
|
|
479
|
+
- Keep changes minimal and focused
|
|
497
480
|
|
|
498
|
-
|
|
499
|
-
|
|
500
|
-
|
|
501
|
-
|
|
481
|
+
**STEP 2: Implement with TDD**
|
|
482
|
+
- Run failing tests to confirm they capture the bug
|
|
483
|
+
- Implement changes to make tests pass (Green phase)
|
|
484
|
+
- Ensure all tests pass
|
|
502
485
|
|
|
503
|
-
|
|
504
|
-
|
|
505
|
-
|
|
506
|
-
- Ensure all tests pass
|
|
486
|
+
**STEP 3: Collaborate When Blocked**
|
|
487
|
+
- Use send_message to architect: "How should I handle [technical issue]?"
|
|
488
|
+
- Use send_message to business-analyst: "What is correct behavior for [edge case]?"
|
|
507
489
|
|
|
508
|
-
|
|
509
|
-
|
|
510
|
-
|
|
490
|
+
**STEP 4: Validate**
|
|
491
|
+
- Verify original reproduction case no longer occurs
|
|
492
|
+
- Run full test suite to check for regressions
|
|
493
|
+
- Test edge cases and boundary conditions
|
|
511
494
|
|
|
512
|
-
|
|
513
|
-
- Verify original reproduction case no longer occurs
|
|
514
|
-
- Run full test suite to check for regressions
|
|
515
|
-
- Test edge cases and boundary conditions
|
|
495
|
+
**Continue to Verify Phase**
|
|
516
496
|
|
|
517
|
-
|
|
518
|
-
|
|
519
|
-
**Before proceeding:**
|
|
497
|
+
You remain RESPONSIBLE in verify phase. Before proceeding:
|
|
520
498
|
1. Use send_message_to_operator: "Fix implemented and tests passing, moving to verification"
|
|
521
499
|
2. Only you can call proceed_to_phase
|
|
522
500
|
|
|
@@ -525,8 +503,7 @@ states:
|
|
|
525
503
|
default_instructions: |
|
|
526
504
|
**Team Collaboration Mode**
|
|
527
505
|
|
|
528
|
-
You are
|
|
529
|
-
Current phase: VERIFY
|
|
506
|
+
You are `$VIBE_ROLE` working in a collaborative team on this bug fix.
|
|
530
507
|
|
|
531
508
|
Developer is responsible. Team validates together.
|
|
532
509
|
|
|
@@ -537,16 +514,16 @@ states:
|
|
|
537
514
|
role: business-analyst
|
|
538
515
|
transition_reason: 'Verification complete, bug fixed'
|
|
539
516
|
additional_instructions: |
|
|
540
|
-
**
|
|
517
|
+
**Consultant Role in Verify Phase**
|
|
541
518
|
|
|
542
|
-
|
|
519
|
+
The developer is verifying the fix. Your role is to provide consultative support.
|
|
543
520
|
|
|
544
521
|
**Your Responsibilities:**
|
|
545
522
|
- Monitor messages for validation requests
|
|
546
523
|
- Perform user acceptance validation:
|
|
547
|
-
|
|
548
|
-
|
|
549
|
-
|
|
524
|
+
- Does fix address original user problem?
|
|
525
|
+
- Are all scenarios from spec working?
|
|
526
|
+
- Is user experience improved?
|
|
550
527
|
|
|
551
528
|
**How to Help:**
|
|
552
529
|
- Use get_my_messages to check for verification requests
|
|
@@ -559,9 +536,9 @@ states:
|
|
|
559
536
|
role: architect
|
|
560
537
|
transition_reason: 'Verification complete, bug fixed'
|
|
561
538
|
additional_instructions: |
|
|
562
|
-
**
|
|
539
|
+
**Consultant Role in Verify Phase**
|
|
563
540
|
|
|
564
|
-
|
|
541
|
+
The developer is verifying the fix. Your role is to provide consultative support.
|
|
565
542
|
|
|
566
543
|
**Your Responsibilities:**
|
|
567
544
|
- Monitor messages for architecture validation requests
|
|
@@ -579,37 +556,35 @@ states:
|
|
|
579
556
|
role: developer
|
|
580
557
|
transition_reason: 'Verification complete, bug fix ready for deployment'
|
|
581
558
|
additional_instructions: |
|
|
582
|
-
**
|
|
559
|
+
**Responsible for Verify Phase**
|
|
583
560
|
|
|
584
561
|
You have exclusive control during this phase:
|
|
585
562
|
- Only you can edit the plan file
|
|
586
563
|
- Only you can mark verification complete
|
|
587
564
|
|
|
588
|
-
**
|
|
589
|
-
|
|
590
|
-
|
|
591
|
-
|
|
592
|
-
|
|
593
|
-
|
|
594
|
-
|
|
595
|
-
|
|
596
|
-
|
|
597
|
-
|
|
598
|
-
|
|
599
|
-
|
|
600
|
-
|
|
601
|
-
|
|
602
|
-
|
|
603
|
-
|
|
604
|
-
|
|
605
|
-
|
|
606
|
-
|
|
607
|
-
|
|
608
|
-
|
|
609
|
-
|
|
610
|
-
|
|
611
|
-
|
|
612
|
-
This is a terminal state - bug fix is complete.
|
|
565
|
+
**STEP 1: Verify Bug Resolution**
|
|
566
|
+
- Confirm original reproduction case no longer occurs
|
|
567
|
+
- Verify all scenarios from bug-spec.md work correctly
|
|
568
|
+
- Test in original environment where bug was reported
|
|
569
|
+
- Validate success criteria are met
|
|
570
|
+
|
|
571
|
+
**STEP 2: Perform Regression Testing**
|
|
572
|
+
- Run full test suite
|
|
573
|
+
- Test related features and integration points
|
|
574
|
+
- Verify performance hasn't degraded
|
|
575
|
+
- Check for new edge case failures
|
|
576
|
+
|
|
577
|
+
**STEP 3: Get Team Validation**
|
|
578
|
+
- Use send_message to business-analyst: "Please perform user acceptance validation"
|
|
579
|
+
- Use send_message to architect: "Please review architectural integrity"
|
|
580
|
+
- Wait for their approval
|
|
581
|
+
|
|
582
|
+
**STEP 4: Complete and Document**
|
|
583
|
+
- Update relevant documentation if behavior changed
|
|
584
|
+
- Document fix approach for future reference
|
|
585
|
+
- Use send_message_to_operator: "Bug fix verified and approved by team, ready for deployment"
|
|
586
|
+
|
|
587
|
+
**This is the terminal state** - bug fix is complete.
|
|
613
588
|
|
|
614
589
|
# Regression detected - return to fix
|
|
615
590
|
- trigger: 'regression_detected'
|