create-byan-agent 1.1.3 → 1.2.1
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/CHANGELOG.md +273 -202
- package/LICENSE +21 -21
- package/README.md +1251 -421
- package/bin/create-byan-agent-backup.js +220 -220
- package/bin/create-byan-agent-fixed.js +301 -301
- package/bin/create-byan-agent.js +155 -322
- package/lib/errors.js +61 -0
- package/lib/exit-codes.js +54 -0
- package/lib/platforms/claude-code.js +113 -0
- package/lib/platforms/codex.js +92 -0
- package/lib/platforms/copilot-cli.js +123 -0
- package/lib/platforms/index.js +14 -0
- package/lib/platforms/vscode.js +51 -0
- package/lib/utils/config-loader.js +79 -0
- package/lib/utils/file-utils.js +104 -0
- package/lib/utils/git-detector.js +35 -0
- package/lib/utils/logger.js +64 -0
- package/lib/utils/node-detector.js +58 -0
- package/lib/utils/os-detector.js +74 -0
- package/lib/utils/yaml-utils.js +87 -0
- package/lib/yanstaller/backuper.js +308 -0
- package/lib/yanstaller/detector.js +141 -0
- package/lib/yanstaller/index.js +93 -0
- package/lib/yanstaller/installer.js +225 -0
- package/lib/yanstaller/interviewer.js +250 -0
- package/lib/yanstaller/recommender.js +298 -0
- package/lib/yanstaller/troubleshooter.js +498 -0
- package/lib/yanstaller/validator.js +578 -0
- package/lib/yanstaller/wizard.js +211 -0
- package/package.json +61 -55
- package/templates/.github/agents/bmad-agent-bmad-master.md +15 -15
- package/templates/.github/agents/bmad-agent-bmb-agent-builder.md +15 -15
- package/templates/.github/agents/bmad-agent-bmb-module-builder.md +15 -15
- package/templates/.github/agents/bmad-agent-bmb-workflow-builder.md +15 -15
- package/templates/.github/agents/bmad-agent-bmm-analyst.md +15 -15
- package/templates/.github/agents/bmad-agent-bmm-architect.md +15 -15
- package/templates/.github/agents/bmad-agent-bmm-dev.md +15 -15
- package/templates/.github/agents/bmad-agent-bmm-pm.md +15 -15
- package/templates/.github/agents/bmad-agent-bmm-quick-flow-solo-dev.md +15 -15
- package/templates/.github/agents/bmad-agent-bmm-quinn.md +15 -15
- package/templates/.github/agents/bmad-agent-bmm-sm.md +15 -15
- package/templates/.github/agents/bmad-agent-bmm-tech-writer.md +15 -15
- package/templates/.github/agents/bmad-agent-bmm-ux-designer.md +15 -15
- package/templates/.github/agents/bmad-agent-byan-test.md +32 -0
- package/templates/.github/agents/bmad-agent-byan.md +224 -224
- package/templates/.github/agents/bmad-agent-carmack.md +18 -0
- package/templates/.github/agents/bmad-agent-cis-brainstorming-coach.md +15 -15
- package/templates/.github/agents/bmad-agent-cis-creative-problem-solver.md +15 -15
- package/templates/.github/agents/bmad-agent-cis-design-thinking-coach.md +15 -15
- package/templates/.github/agents/bmad-agent-cis-innovation-strategist.md +15 -15
- package/templates/.github/agents/bmad-agent-cis-presentation-master.md +15 -15
- package/templates/.github/agents/bmad-agent-cis-storyteller.md +15 -15
- package/templates/.github/agents/bmad-agent-marc.md +48 -48
- package/templates/.github/agents/bmad-agent-patnote.md +48 -0
- package/templates/.github/agents/bmad-agent-rachid.md +47 -47
- package/templates/.github/agents/bmad-agent-tea-tea.md +15 -15
- package/templates/.github/agents/bmad-agent-test-dynamic.md +21 -0
- package/templates/.github/agents/expert-merise-agile.md +1 -0
- package/templates/.github/agents/franck.md +379 -0
- package/templates/_bmad/bmb/agents/agent-builder.md +59 -59
- package/templates/_bmad/bmb/agents/byan-test.md +116 -116
- package/templates/_bmad/bmb/agents/byan.md +215 -215
- package/templates/_bmad/bmb/agents/marc.md +303 -303
- package/templates/_bmad/bmb/agents/module-builder.md +60 -60
- package/templates/_bmad/bmb/agents/patnote.md +495 -495
- package/templates/_bmad/bmb/agents/rachid.md +184 -184
- package/templates/_bmad/bmb/agents/workflow-builder.md +61 -61
- package/templates/_bmad/bmb/workflows/byan/data/mantras.yaml +272 -272
- package/templates/_bmad/bmb/workflows/byan/data/templates.yaml +59 -59
- package/templates/_bmad/bmb/workflows/byan/delete-agent-workflow.md +657 -657
- package/templates/_bmad/bmb/workflows/byan/edit-agent-workflow.md +688 -688
- package/templates/_bmad/bmb/workflows/byan/interview-workflow.md +753 -753
- package/templates/_bmad/bmb/workflows/byan/quick-create-workflow.md +450 -450
- package/templates/_bmad/bmb/workflows/byan/templates/base-agent-template.md +79 -79
- package/templates/_bmad/bmb/workflows/byan/validate-agent-workflow.md +676 -676
- package/templates/_bmad/core/agents/carmack.md +238 -238
|
@@ -1,688 +1,688 @@
|
|
|
1
|
-
# BYAN Edit Agent Workflow
|
|
2
|
-
|
|
3
|
-
**Workflow:** Edit Existing Agent with Consequences Evaluation
|
|
4
|
-
**Duration:** 10-20 minutes
|
|
5
|
-
**Methodology:** Merise Agile + TDD + Mantra #39 (Consequences)
|
|
6
|
-
|
|
7
|
-
---
|
|
8
|
-
|
|
9
|
-
## OVERVIEW
|
|
10
|
-
|
|
11
|
-
Edit Agent workflow allows modification of existing agents while evaluating consequences of changes.
|
|
12
|
-
|
|
13
|
-
**Mantras Applied:**
|
|
14
|
-
- **Mantra #39: Chaque Action a des Conséquences** (PRIMARY)
|
|
15
|
-
- Mantra IA-16: Challenge Before Confirm
|
|
16
|
-
- Mantra #37: Rasoir d'Ockham (don't over-complicate)
|
|
17
|
-
- Mantra IA-1: Trust But Verify
|
|
18
|
-
|
|
19
|
-
**Key Principle:**
|
|
20
|
-
> "Before changing ANYTHING, evaluate what breaks, what improves, and what remains uncertain."
|
|
21
|
-
|
|
22
|
-
---
|
|
23
|
-
|
|
24
|
-
## WORKFLOW STEPS
|
|
25
|
-
|
|
26
|
-
### Step 1: Select Agent to Edit
|
|
27
|
-
|
|
28
|
-
```
|
|
29
|
-
"Which agent do you want to edit?
|
|
30
|
-
|
|
31
|
-
Available agents:
|
|
32
|
-
{list_all_agents_with_status}
|
|
33
|
-
|
|
34
|
-
1. {agent_1} - {role} [{status}]
|
|
35
|
-
2. {agent_2} - {role} [{status}]
|
|
36
|
-
3. {agent_3} - {role} [{status}]
|
|
37
|
-
...
|
|
38
|
-
|
|
39
|
-
Agent number or name: ____
|
|
40
|
-
"
|
|
41
|
-
```
|
|
42
|
-
|
|
43
|
-
**Load agent:**
|
|
44
|
-
```python
|
|
45
|
-
agent_spec = AgentSpec.load(agent_id_or_name)
|
|
46
|
-
|
|
47
|
-
# Check if deployed
|
|
48
|
-
if agent_spec.status == "deployed":
|
|
49
|
-
print(f"⚠️ WARNING: This agent is DEPLOYED (actively used).")
|
|
50
|
-
print(f"Editing deployed agents can have immediate consequences.")
|
|
51
|
-
print(f"Proceed with caution? (yes/no): ")
|
|
52
|
-
```
|
|
53
|
-
|
|
54
|
-
**Display current agent:**
|
|
55
|
-
```
|
|
56
|
-
"CURRENT AGENT SPEC:
|
|
57
|
-
|
|
58
|
-
NAME: {agent_name}
|
|
59
|
-
ROLE: {role}
|
|
60
|
-
STATUS: {status}
|
|
61
|
-
CREATED: {created_at}
|
|
62
|
-
LAST MODIFIED: {updated_at}
|
|
63
|
-
|
|
64
|
-
CAPABILITIES ({n}):
|
|
65
|
-
1. {capability_1}
|
|
66
|
-
2. {capability_2}
|
|
67
|
-
...
|
|
68
|
-
|
|
69
|
-
MANTRAS ({n}):
|
|
70
|
-
- Mantra #{id}: {name}
|
|
71
|
-
...
|
|
72
|
-
|
|
73
|
-
KNOWLEDGE:
|
|
74
|
-
Business: {business_concepts}
|
|
75
|
-
Technical: {technical_stack}
|
|
76
|
-
|
|
77
|
-
COMMUNICATION STYLE: {style}
|
|
78
|
-
|
|
79
|
-
USE CASES ({n}):
|
|
80
|
-
1. {use_case_1}
|
|
81
|
-
...
|
|
82
|
-
|
|
83
|
-
What would you like to edit?"
|
|
84
|
-
```
|
|
85
|
-
|
|
86
|
-
---
|
|
87
|
-
|
|
88
|
-
### Step 2: Select What to Edit
|
|
89
|
-
|
|
90
|
-
```
|
|
91
|
-
"What needs editing?
|
|
92
|
-
|
|
93
|
-
1. Agent name (⚠️ BREAKING CHANGE)
|
|
94
|
-
2. Role/responsibilities
|
|
95
|
-
3. Add capability
|
|
96
|
-
4. Remove capability (⚠️ BREAKING CHANGE)
|
|
97
|
-
5. Modify capability
|
|
98
|
-
6. Add mantra
|
|
99
|
-
7. Remove mantra
|
|
100
|
-
8. Change communication style
|
|
101
|
-
9. Add use case
|
|
102
|
-
10. Update knowledge (business/technical)
|
|
103
|
-
11. Everything (guided full review)
|
|
104
|
-
12. Cancel
|
|
105
|
-
|
|
106
|
-
Choice: ____
|
|
107
|
-
"
|
|
108
|
-
```
|
|
109
|
-
|
|
110
|
-
**Breaking changes flagged:**
|
|
111
|
-
- Agent name change → All references break
|
|
112
|
-
- Capability removal → Users relying on it affected
|
|
113
|
-
- Major role change → Behavior changes significantly
|
|
114
|
-
|
|
115
|
-
---
|
|
116
|
-
|
|
117
|
-
### Step 3: Make Changes (with Consequences)
|
|
118
|
-
|
|
119
|
-
#### 3.1: EDIT AGENT NAME (⚠️ CRITICAL)
|
|
120
|
-
|
|
121
|
-
```
|
|
122
|
-
"⚠️ CHANGING AGENT NAME IS A BREAKING CHANGE!
|
|
123
|
-
|
|
124
|
-
CURRENT NAME: {current_name}
|
|
125
|
-
|
|
126
|
-
CONSEQUENCES OF RENAMING:
|
|
127
|
-
❌ All file references to this agent break
|
|
128
|
-
❌ User workflows mentioning agent by name break
|
|
129
|
-
❌ Documentation must be updated
|
|
130
|
-
❌ CLI commands using old name fail
|
|
131
|
-
❌ Integration tests mentioning agent fail
|
|
132
|
-
|
|
133
|
-
SCOPE: High
|
|
134
|
-
DATA: Medium (file renames required)
|
|
135
|
-
CODE: Medium (references update)
|
|
136
|
-
TEAM: High (everyone must be notified)
|
|
137
|
-
DEPENDENCIES: High (all agent consumers affected)
|
|
138
|
-
|
|
139
|
-
RISK LEVEL: 🔥 CRITICAL
|
|
140
|
-
|
|
141
|
-
Still want to rename? (yes/no): ____
|
|
142
|
-
"
|
|
143
|
-
```
|
|
144
|
-
|
|
145
|
-
**If yes:**
|
|
146
|
-
```
|
|
147
|
-
"New agent name (kebab-case): ____
|
|
148
|
-
|
|
149
|
-
MITIGATION PLAN:
|
|
150
|
-
1. Create backup of current agent
|
|
151
|
-
2. Rename agent spec
|
|
152
|
-
3. Update all file references
|
|
153
|
-
4. Create redirect/alias from old name (temporary)
|
|
154
|
-
5. Notify all team members
|
|
155
|
-
6. Update documentation
|
|
156
|
-
7. Schedule old name deprecation (30 days)
|
|
157
|
-
|
|
158
|
-
Confirm mitigation plan? (yes/no): ____
|
|
159
|
-
"
|
|
160
|
-
```
|
|
161
|
-
|
|
162
|
-
**Validate new name:**
|
|
163
|
-
- Must be kebab-case
|
|
164
|
-
- Must be unique (RG-AGT-001)
|
|
165
|
-
- Must not conflict with reserved names
|
|
166
|
-
|
|
167
|
-
#### 3.2: EDIT ROLE/RESPONSIBILITIES
|
|
168
|
-
|
|
169
|
-
```
|
|
170
|
-
"CURRENT ROLE: {current_role}
|
|
171
|
-
|
|
172
|
-
New role (or press Enter to keep): ____
|
|
173
|
-
"
|
|
174
|
-
```
|
|
175
|
-
|
|
176
|
-
**If changed:**
|
|
177
|
-
```
|
|
178
|
-
"Role changed from:
|
|
179
|
-
OLD: {old_role}
|
|
180
|
-
NEW: {new_role}
|
|
181
|
-
|
|
182
|
-
CONSEQUENCES:
|
|
183
|
-
📊 SCOPE: Medium (agent behavior may shift)
|
|
184
|
-
👥 TEAM: Medium (user expectations change)
|
|
185
|
-
📝 CODE: Low (mostly documentation)
|
|
186
|
-
|
|
187
|
-
⚠️ Users may expect different behavior from this agent.
|
|
188
|
-
|
|
189
|
-
Recommendation: Add changelog note explaining why role changed.
|
|
190
|
-
|
|
191
|
-
Proceed? (yes/no): ____
|
|
192
|
-
"
|
|
193
|
-
```
|
|
194
|
-
|
|
195
|
-
#### 3.3: ADD CAPABILITY
|
|
196
|
-
|
|
197
|
-
```
|
|
198
|
-
"Add new capability:
|
|
199
|
-
|
|
200
|
-
Capability description: ____
|
|
201
|
-
Category (create/analyze/review/optimize/teach): ____
|
|
202
|
-
"
|
|
203
|
-
```
|
|
204
|
-
|
|
205
|
-
**Consequences (usually LOW risk):**
|
|
206
|
-
```
|
|
207
|
-
"Adding capability: {new_capability}
|
|
208
|
-
|
|
209
|
-
CONSEQUENCES:
|
|
210
|
-
✅ POSITIVE: Agent becomes more useful
|
|
211
|
-
✅ SCOPE: Low (additive change, non-breaking)
|
|
212
|
-
⚠️ CODE: Low (tests for new capability needed)
|
|
213
|
-
⚠️ DOCS: Low (document new capability)
|
|
214
|
-
|
|
215
|
-
RISK LEVEL: 🟢 LOW
|
|
216
|
-
|
|
217
|
-
Current capability count: {current_count}
|
|
218
|
-
After adding: {current_count + 1}
|
|
219
|
-
|
|
220
|
-
Note: More capabilities = more complexity. Keep it focused.
|
|
221
|
-
Mantra #37 (Ockham's Razor): Is this truly needed?
|
|
222
|
-
|
|
223
|
-
Proceed? (yes/add-more/cancel): ____
|
|
224
|
-
"
|
|
225
|
-
```
|
|
226
|
-
|
|
227
|
-
#### 3.4: REMOVE CAPABILITY (⚠️ BREAKING)
|
|
228
|
-
|
|
229
|
-
```
|
|
230
|
-
"Which capability to remove?
|
|
231
|
-
|
|
232
|
-
CURRENT CAPABILITIES:
|
|
233
|
-
1. {capability_1}
|
|
234
|
-
2. {capability_2}
|
|
235
|
-
3. {capability_3}
|
|
236
|
-
...
|
|
237
|
-
|
|
238
|
-
Capability number: ____
|
|
239
|
-
"
|
|
240
|
-
```
|
|
241
|
-
|
|
242
|
-
**ALWAYS evaluate consequences:**
|
|
243
|
-
```
|
|
244
|
-
"⚠️ REMOVING CAPABILITY: {capability}
|
|
245
|
-
|
|
246
|
-
CONSEQUENCES EVALUATION:
|
|
247
|
-
|
|
248
|
-
❌ BREAKING CHANGE - Users relying on this capability will be affected
|
|
249
|
-
|
|
250
|
-
QUESTIONS TO ASK:
|
|
251
|
-
1. Is anyone currently using this capability?
|
|
252
|
-
2. Are there workflows that depend on it?
|
|
253
|
-
3. Are there tests that verify it?
|
|
254
|
-
4. Why are we removing it? (technical debt? never used? superseded?)
|
|
255
|
-
|
|
256
|
-
ANALYSIS:
|
|
257
|
-
📊 SCOPE: High (feature removal)
|
|
258
|
-
👥 TEAM: High (check with team first!)
|
|
259
|
-
🔧 CODE: Medium (remove implementation + tests)
|
|
260
|
-
📚 DOCS: Medium (update docs, add migration guide)
|
|
261
|
-
⏱️ TIME: Medium (communication + migration)
|
|
262
|
-
|
|
263
|
-
RISK LEVEL: 🔴 HIGH
|
|
264
|
-
|
|
265
|
-
MITIGATION PLAN:
|
|
266
|
-
1. Survey team: Is this capability used?
|
|
267
|
-
2. If used: Deprecation period (30 days) before removal
|
|
268
|
-
3. If unused: Safe to remove
|
|
269
|
-
4. Update tests
|
|
270
|
-
5. Update documentation
|
|
271
|
-
6. Add to changelog
|
|
272
|
-
|
|
273
|
-
Check if capability is used? (yes/no/cancel): ____
|
|
274
|
-
"
|
|
275
|
-
```
|
|
276
|
-
|
|
277
|
-
**If capability is used:**
|
|
278
|
-
```
|
|
279
|
-
"CAPABILITY IN USE!
|
|
280
|
-
|
|
281
|
-
Found references in:
|
|
282
|
-
- {file_1}: {line}
|
|
283
|
-
- {file_2}: {line}
|
|
284
|
-
...
|
|
285
|
-
|
|
286
|
-
This capability CANNOT be safely removed without breaking things.
|
|
287
|
-
|
|
288
|
-
OPTIONS:
|
|
289
|
-
1. Keep the capability
|
|
290
|
-
2. Create migration guide and deprecate (30 day notice)
|
|
291
|
-
3. Find alternative capability to replace it
|
|
292
|
-
4. Force remove (⚠️ DANGEROUS)
|
|
293
|
-
|
|
294
|
-
What should we do?: ____
|
|
295
|
-
"
|
|
296
|
-
```
|
|
297
|
-
|
|
298
|
-
#### 3.5: MODIFY CAPABILITY
|
|
299
|
-
|
|
300
|
-
```
|
|
301
|
-
"Which capability to modify?
|
|
302
|
-
|
|
303
|
-
CURRENT CAPABILITIES:
|
|
304
|
-
1. {capability_1}
|
|
305
|
-
2. {capability_2}
|
|
306
|
-
...
|
|
307
|
-
|
|
308
|
-
Capability number: ____
|
|
309
|
-
|
|
310
|
-
CURRENT: {capability_description}
|
|
311
|
-
NEW: ____ (or press Enter to keep)
|
|
312
|
-
"
|
|
313
|
-
```
|
|
314
|
-
|
|
315
|
-
**Evaluate change:**
|
|
316
|
-
```
|
|
317
|
-
"Capability modified:
|
|
318
|
-
OLD: {old_description}
|
|
319
|
-
NEW: {new_description}
|
|
320
|
-
|
|
321
|
-
CHANGE TYPE: {minor|moderate|major}
|
|
322
|
-
- Minor: Wording/clarity improvement only
|
|
323
|
-
- Moderate: Behavior slightly different
|
|
324
|
-
- Major: Completely different capability
|
|
325
|
-
|
|
326
|
-
CONSEQUENCES:
|
|
327
|
-
📊 SCOPE: {evaluated_scope}
|
|
328
|
-
👥 TEAM: {evaluated_impact}
|
|
329
|
-
⚠️ Existing users may see behavior change
|
|
330
|
-
|
|
331
|
-
RISK LEVEL: {risk_level}
|
|
332
|
-
|
|
333
|
-
Proceed? (yes/no): ____
|
|
334
|
-
"
|
|
335
|
-
```
|
|
336
|
-
|
|
337
|
-
#### 3.6: ADD/REMOVE MANTRAS
|
|
338
|
-
|
|
339
|
-
**Add Mantra:**
|
|
340
|
-
```
|
|
341
|
-
"Current mantras ({n}/64):
|
|
342
|
-
{list_current_mantras}
|
|
343
|
-
|
|
344
|
-
Which mantra to add? (number 1-64 or name): ____
|
|
345
|
-
|
|
346
|
-
Why add this mantra?: ____
|
|
347
|
-
"
|
|
348
|
-
```
|
|
349
|
-
|
|
350
|
-
**Remove Mantra:**
|
|
351
|
-
```
|
|
352
|
-
"⚠️ Removing a mantra means this agent will no longer prioritize that principle.
|
|
353
|
-
|
|
354
|
-
Which mantra to remove?
|
|
355
|
-
{list_current_mantras_numbered}
|
|
356
|
-
|
|
357
|
-
Mantra number: ____
|
|
358
|
-
|
|
359
|
-
Why remove this mantra?: ____
|
|
360
|
-
|
|
361
|
-
CONSEQUENCE:
|
|
362
|
-
- Agent behavior may shift
|
|
363
|
-
- Quality checks associated with mantra won't apply
|
|
364
|
-
- Risk: Lower quality if mantra was critical
|
|
365
|
-
|
|
366
|
-
RISK LEVEL: 🟡 MEDIUM
|
|
367
|
-
|
|
368
|
-
Confirm removal? (yes/no): ____
|
|
369
|
-
"
|
|
370
|
-
```
|
|
371
|
-
|
|
372
|
-
**Validate RG-AGT-003:** Must have >= 5 mantras after changes
|
|
373
|
-
|
|
374
|
-
#### 3.7: CHANGE COMMUNICATION STYLE
|
|
375
|
-
|
|
376
|
-
```
|
|
377
|
-
"CURRENT STYLE: {current_style}
|
|
378
|
-
|
|
379
|
-
New communication style:
|
|
380
|
-
1. CONCISE (direct, minimal words)
|
|
381
|
-
2. EDUCATIONAL (detailed, teaches)
|
|
382
|
-
3. BALANCED (mix)
|
|
383
|
-
4. CUSTOM (specify)
|
|
384
|
-
|
|
385
|
-
Choice: ____
|
|
386
|
-
"
|
|
387
|
-
```
|
|
388
|
-
|
|
389
|
-
**Consequences (usually LOW):**
|
|
390
|
-
```
|
|
391
|
-
"Communication style changed: {old} → {new}
|
|
392
|
-
|
|
393
|
-
CONSEQUENCES:
|
|
394
|
-
📝 USER EXPERIENCE: Users will notice different tone
|
|
395
|
-
🎯 SCOPE: Low (no functionality change)
|
|
396
|
-
📚 DOCS: Low (update examples if needed)
|
|
397
|
-
|
|
398
|
-
RISK LEVEL: 🟢 LOW
|
|
399
|
-
|
|
400
|
-
User feedback recommended after deployment.
|
|
401
|
-
|
|
402
|
-
Proceed? (yes/no): ____
|
|
403
|
-
"
|
|
404
|
-
```
|
|
405
|
-
|
|
406
|
-
#### 3.8: ADD USE CASE
|
|
407
|
-
|
|
408
|
-
```
|
|
409
|
-
"Add new use case:
|
|
410
|
-
|
|
411
|
-
Scenario: ____
|
|
412
|
-
Expected behavior: ____
|
|
413
|
-
"
|
|
414
|
-
```
|
|
415
|
-
|
|
416
|
-
Consequences: 🟢 LOW (additive, documentation only)
|
|
417
|
-
|
|
418
|
-
---
|
|
419
|
-
|
|
420
|
-
### Step 4: Comprehensive Consequences Summary
|
|
421
|
-
|
|
422
|
-
**After all changes, show complete summary:**
|
|
423
|
-
|
|
424
|
-
```
|
|
425
|
-
"CHANGE SUMMARY:
|
|
426
|
-
|
|
427
|
-
Changes made:
|
|
428
|
-
1. {change_1}
|
|
429
|
-
2. {change_2}
|
|
430
|
-
...
|
|
431
|
-
|
|
432
|
-
FULL CONSEQUENCES EVALUATION (10 Dimensions):
|
|
433
|
-
|
|
434
|
-
1. 📊 SCOPE:
|
|
435
|
-
- Components affected: {list}
|
|
436
|
-
- Breaking changes: {yes/no}
|
|
437
|
-
- Impact: {Low/Medium/High/Critical}
|
|
438
|
-
|
|
439
|
-
2. 💾 DATA:
|
|
440
|
-
- Database changes: {yes/no}
|
|
441
|
-
- Data migration needed: {yes/no}
|
|
442
|
-
- Impact: {Low/Medium/High/Critical}
|
|
443
|
-
|
|
444
|
-
3. 🔧 CODE:
|
|
445
|
-
- Files to modify: {count}
|
|
446
|
-
- Tests to update: {count}
|
|
447
|
-
- Impact: {Low/Medium/High/Critical}
|
|
448
|
-
|
|
449
|
-
4. 👥 TEAM:
|
|
450
|
-
- Team members affected: {all/some/none}
|
|
451
|
-
- Training needed: {yes/no}
|
|
452
|
-
- Impact: {Low/Medium/High/Critical}
|
|
453
|
-
|
|
454
|
-
5. 🎯 CLIENTS:
|
|
455
|
-
- User workflows broken: {yes/no}
|
|
456
|
-
- Migration guide needed: {yes/no}
|
|
457
|
-
- Impact: {Low/Medium/High/Critical}
|
|
458
|
-
|
|
459
|
-
6. ⚖️ LEGAL:
|
|
460
|
-
- Compliance affected: {yes/no}
|
|
461
|
-
- Impact: {Low/Medium/High/Critical}
|
|
462
|
-
|
|
463
|
-
7. 🚀 OPERATIONS:
|
|
464
|
-
- Deployment complexity: {simple/moderate/complex}
|
|
465
|
-
- Rollback plan: {needed/not needed}
|
|
466
|
-
- Impact: {Low/Medium/High/Critical}
|
|
467
|
-
|
|
468
|
-
8. 🔗 DEPENDENCIES:
|
|
469
|
-
- Dependent systems: {count}
|
|
470
|
-
- Breaking changes: {yes/no}
|
|
471
|
-
- Impact: {Low/Medium/High/Critical}
|
|
472
|
-
|
|
473
|
-
9. ⏱️ TIME:
|
|
474
|
-
- Implementation time: {estimate}
|
|
475
|
-
- Communication time: {estimate}
|
|
476
|
-
- Impact: {Low/Medium/High/Critical}
|
|
477
|
-
|
|
478
|
-
10. 🔄 ALTERNATIVES:
|
|
479
|
-
- Alternative approaches considered: {yes/no}
|
|
480
|
-
- Simpler alternatives exist: {yes/no}
|
|
481
|
-
|
|
482
|
-
OVERALL RISK LEVEL: {🟢 LOW | 🟡 MEDIUM | 🔴 HIGH | 🔥 CRITICAL}
|
|
483
|
-
|
|
484
|
-
REQUIRED ACTIONS BEFORE DEPLOYMENT:
|
|
485
|
-
{action_checklist}
|
|
486
|
-
|
|
487
|
-
MITIGATION PLAN:
|
|
488
|
-
{mitigation_steps}
|
|
489
|
-
|
|
490
|
-
Proceed with these changes? (yes/review/cancel): ____
|
|
491
|
-
"
|
|
492
|
-
```
|
|
493
|
-
|
|
494
|
-
---
|
|
495
|
-
|
|
496
|
-
### Step 5: Execute Changes
|
|
497
|
-
|
|
498
|
-
**If approved:**
|
|
499
|
-
|
|
500
|
-
```python
|
|
501
|
-
# Create backup
|
|
502
|
-
backup = agent_spec.create_backup()
|
|
503
|
-
print(f"Backup created: {backup.path}")
|
|
504
|
-
|
|
505
|
-
# Apply changes
|
|
506
|
-
agent_spec.apply_changes(changes)
|
|
507
|
-
|
|
508
|
-
# Update status
|
|
509
|
-
if agent_spec.status == "deployed":
|
|
510
|
-
agent_spec.status = "modified" # Needs revalidation
|
|
511
|
-
elif agent_spec.status == "validated":
|
|
512
|
-
agent_spec.status = "modified" # Needs revalidation
|
|
513
|
-
|
|
514
|
-
# Save
|
|
515
|
-
agent_spec.save()
|
|
516
|
-
|
|
517
|
-
# Create changelog entry
|
|
518
|
-
changelog.add_entry(
|
|
519
|
-
agent=agent_spec,
|
|
520
|
-
changes=changes,
|
|
521
|
-
consequences=consequences_summary,
|
|
522
|
-
timestamp=now()
|
|
523
|
-
)
|
|
524
|
-
```
|
|
525
|
-
|
|
526
|
-
**Confirmation:**
|
|
527
|
-
```
|
|
528
|
-
"Changes applied successfully!
|
|
529
|
-
|
|
530
|
-
BACKUP LOCATION: {backup_path}
|
|
531
|
-
CHANGELOG: {changelog_path}
|
|
532
|
-
|
|
533
|
-
AGENT STATUS: {new_status}
|
|
534
|
-
- Needs revalidation before deployment
|
|
535
|
-
|
|
536
|
-
NEXT STEPS:
|
|
537
|
-
1. [VA] Validate agent against 64 mantras
|
|
538
|
-
2. Test modified agent
|
|
539
|
-
3. Update documentation
|
|
540
|
-
4. {additional_steps_from_mitigation}
|
|
541
|
-
|
|
542
|
-
Would you like to validate now? (yes/later): ____
|
|
543
|
-
"
|
|
544
|
-
```
|
|
545
|
-
|
|
546
|
-
---
|
|
547
|
-
|
|
548
|
-
## ROLLBACK PROCEDURE
|
|
549
|
-
|
|
550
|
-
If something goes wrong:
|
|
551
|
-
|
|
552
|
-
```
|
|
553
|
-
"❌ ERROR during changes: {error}
|
|
554
|
-
|
|
555
|
-
ROLLBACK OPTIONS:
|
|
556
|
-
1. Restore from backup (undo ALL changes)
|
|
557
|
-
2. Restore specific changes only
|
|
558
|
-
3. Try to fix error and continue
|
|
559
|
-
4. Save current state and exit
|
|
560
|
-
|
|
561
|
-
What should we do?: ____
|
|
562
|
-
"
|
|
563
|
-
```
|
|
564
|
-
|
|
565
|
-
**Restore from backup:**
|
|
566
|
-
```python
|
|
567
|
-
def rollback(agent_spec, backup):
|
|
568
|
-
agent_spec.restore_from_backup(backup)
|
|
569
|
-
agent_spec.save()
|
|
570
|
-
print("✅ Rollback successful. Agent restored to previous state.")
|
|
571
|
-
```
|
|
572
|
-
|
|
573
|
-
---
|
|
574
|
-
|
|
575
|
-
## VALIDATION AFTER EDIT
|
|
576
|
-
|
|
577
|
-
**Always recommend validation:**
|
|
578
|
-
|
|
579
|
-
```
|
|
580
|
-
"Edits complete!
|
|
581
|
-
|
|
582
|
-
⚠️ Modified agents should be validated before deployment.
|
|
583
|
-
|
|
584
|
-
Run validation now?
|
|
585
|
-
- [VA] Full validation (checks all 64 mantras)
|
|
586
|
-
- [TEST] Quick test (basic checks only)
|
|
587
|
-
- [LATER] Skip for now
|
|
588
|
-
|
|
589
|
-
Choice: ____
|
|
590
|
-
"
|
|
591
|
-
```
|
|
592
|
-
|
|
593
|
-
---
|
|
594
|
-
|
|
595
|
-
## CHANGE CATEGORIES
|
|
596
|
-
|
|
597
|
-
**LOW RISK (🟢) - Safe to edit:**
|
|
598
|
-
- Add capability
|
|
599
|
-
- Add mantra
|
|
600
|
-
- Add use case
|
|
601
|
-
- Update documentation
|
|
602
|
-
- Improve wording/clarity
|
|
603
|
-
- Change communication style
|
|
604
|
-
|
|
605
|
-
**MEDIUM RISK (🟡) - Caution required:**
|
|
606
|
-
- Modify capability behavior
|
|
607
|
-
- Remove mantra
|
|
608
|
-
- Change role significantly
|
|
609
|
-
- Update knowledge base
|
|
610
|
-
|
|
611
|
-
**HIGH RISK (🔴) - Team approval needed:**
|
|
612
|
-
- Remove capability
|
|
613
|
-
- Change agent name
|
|
614
|
-
- Major behavior changes
|
|
615
|
-
- Breaking API changes
|
|
616
|
-
|
|
617
|
-
**CRITICAL RISK (🔥) - Requires planning:**
|
|
618
|
-
- Rename deployed agent
|
|
619
|
-
- Remove core capability from deployed agent
|
|
620
|
-
- Changes affecting multiple systems
|
|
621
|
-
|
|
622
|
-
---
|
|
623
|
-
|
|
624
|
-
## ANTI-PATTERNS
|
|
625
|
-
|
|
626
|
-
**NEVER do these:**
|
|
627
|
-
|
|
628
|
-
❌ Edit without evaluating consequences
|
|
629
|
-
❌ Remove capabilities without checking usage
|
|
630
|
-
❌ Rename agents without migration plan
|
|
631
|
-
❌ Skip backup before editing
|
|
632
|
-
❌ Edit deployed agents without team notice
|
|
633
|
-
❌ Make breaking changes without deprecation
|
|
634
|
-
❌ Ignore dependency impacts
|
|
635
|
-
❌ Edit without understanding WHY
|
|
636
|
-
|
|
637
|
-
**ALWAYS do these:**
|
|
638
|
-
|
|
639
|
-
✅ Create backup first
|
|
640
|
-
✅ Evaluate consequences (Mantra #39)
|
|
641
|
-
✅ Challenge changes (Mantra IA-16)
|
|
642
|
-
✅ Check for simpler alternatives (Mantra #37)
|
|
643
|
-
✅ Communicate breaking changes to team
|
|
644
|
-
✅ Update documentation
|
|
645
|
-
✅ Revalidate after changes
|
|
646
|
-
✅ Test before deploying
|
|
647
|
-
|
|
648
|
-
---
|
|
649
|
-
|
|
650
|
-
## SUCCESS CRITERIA
|
|
651
|
-
|
|
652
|
-
✅ Agent loaded successfully
|
|
653
|
-
✅ Changes clearly identified
|
|
654
|
-
✅ Consequences evaluated (10 dimensions)
|
|
655
|
-
✅ Risk level assessed
|
|
656
|
-
✅ Mitigation plan created
|
|
657
|
-
✅ Backup created before changes
|
|
658
|
-
✅ Changes applied successfully
|
|
659
|
-
✅ Validation rules still pass
|
|
660
|
-
✅ Changelog updated
|
|
661
|
-
✅ User aware of next steps
|
|
662
|
-
|
|
663
|
-
---
|
|
664
|
-
|
|
665
|
-
## COMPLETION
|
|
666
|
-
|
|
667
|
-
```
|
|
668
|
-
"Edit workflow complete!
|
|
669
|
-
|
|
670
|
-
SUMMARY:
|
|
671
|
-
- Agent: {agent_name}
|
|
672
|
-
- Changes: {n} modifications
|
|
673
|
-
- Risk Level: {risk_level}
|
|
674
|
-
- Status: {new_status}
|
|
675
|
-
|
|
676
|
-
FILES UPDATED:
|
|
677
|
-
- Agent spec: {spec_path}
|
|
678
|
-
- Backup: {backup_path}
|
|
679
|
-
- Changelog: {changelog_path}
|
|
680
|
-
|
|
681
|
-
RECOMMENDED NEXT STEPS:
|
|
682
|
-
1. {step_1}
|
|
683
|
-
2. {step_2}
|
|
684
|
-
...
|
|
685
|
-
|
|
686
|
-
Session saved to: {output_folder}/edit-agent-{agent_name}-{timestamp}.md
|
|
687
|
-
"
|
|
688
|
-
```
|
|
1
|
+
# BYAN Edit Agent Workflow
|
|
2
|
+
|
|
3
|
+
**Workflow:** Edit Existing Agent with Consequences Evaluation
|
|
4
|
+
**Duration:** 10-20 minutes
|
|
5
|
+
**Methodology:** Merise Agile + TDD + Mantra #39 (Consequences)
|
|
6
|
+
|
|
7
|
+
---
|
|
8
|
+
|
|
9
|
+
## OVERVIEW
|
|
10
|
+
|
|
11
|
+
Edit Agent workflow allows modification of existing agents while evaluating consequences of changes.
|
|
12
|
+
|
|
13
|
+
**Mantras Applied:**
|
|
14
|
+
- **Mantra #39: Chaque Action a des Conséquences** (PRIMARY)
|
|
15
|
+
- Mantra IA-16: Challenge Before Confirm
|
|
16
|
+
- Mantra #37: Rasoir d'Ockham (don't over-complicate)
|
|
17
|
+
- Mantra IA-1: Trust But Verify
|
|
18
|
+
|
|
19
|
+
**Key Principle:**
|
|
20
|
+
> "Before changing ANYTHING, evaluate what breaks, what improves, and what remains uncertain."
|
|
21
|
+
|
|
22
|
+
---
|
|
23
|
+
|
|
24
|
+
## WORKFLOW STEPS
|
|
25
|
+
|
|
26
|
+
### Step 1: Select Agent to Edit
|
|
27
|
+
|
|
28
|
+
```
|
|
29
|
+
"Which agent do you want to edit?
|
|
30
|
+
|
|
31
|
+
Available agents:
|
|
32
|
+
{list_all_agents_with_status}
|
|
33
|
+
|
|
34
|
+
1. {agent_1} - {role} [{status}]
|
|
35
|
+
2. {agent_2} - {role} [{status}]
|
|
36
|
+
3. {agent_3} - {role} [{status}]
|
|
37
|
+
...
|
|
38
|
+
|
|
39
|
+
Agent number or name: ____
|
|
40
|
+
"
|
|
41
|
+
```
|
|
42
|
+
|
|
43
|
+
**Load agent:**
|
|
44
|
+
```python
|
|
45
|
+
agent_spec = AgentSpec.load(agent_id_or_name)
|
|
46
|
+
|
|
47
|
+
# Check if deployed
|
|
48
|
+
if agent_spec.status == "deployed":
|
|
49
|
+
print(f"⚠️ WARNING: This agent is DEPLOYED (actively used).")
|
|
50
|
+
print(f"Editing deployed agents can have immediate consequences.")
|
|
51
|
+
print(f"Proceed with caution? (yes/no): ")
|
|
52
|
+
```
|
|
53
|
+
|
|
54
|
+
**Display current agent:**
|
|
55
|
+
```
|
|
56
|
+
"CURRENT AGENT SPEC:
|
|
57
|
+
|
|
58
|
+
NAME: {agent_name}
|
|
59
|
+
ROLE: {role}
|
|
60
|
+
STATUS: {status}
|
|
61
|
+
CREATED: {created_at}
|
|
62
|
+
LAST MODIFIED: {updated_at}
|
|
63
|
+
|
|
64
|
+
CAPABILITIES ({n}):
|
|
65
|
+
1. {capability_1}
|
|
66
|
+
2. {capability_2}
|
|
67
|
+
...
|
|
68
|
+
|
|
69
|
+
MANTRAS ({n}):
|
|
70
|
+
- Mantra #{id}: {name}
|
|
71
|
+
...
|
|
72
|
+
|
|
73
|
+
KNOWLEDGE:
|
|
74
|
+
Business: {business_concepts}
|
|
75
|
+
Technical: {technical_stack}
|
|
76
|
+
|
|
77
|
+
COMMUNICATION STYLE: {style}
|
|
78
|
+
|
|
79
|
+
USE CASES ({n}):
|
|
80
|
+
1. {use_case_1}
|
|
81
|
+
...
|
|
82
|
+
|
|
83
|
+
What would you like to edit?"
|
|
84
|
+
```
|
|
85
|
+
|
|
86
|
+
---
|
|
87
|
+
|
|
88
|
+
### Step 2: Select What to Edit
|
|
89
|
+
|
|
90
|
+
```
|
|
91
|
+
"What needs editing?
|
|
92
|
+
|
|
93
|
+
1. Agent name (⚠️ BREAKING CHANGE)
|
|
94
|
+
2. Role/responsibilities
|
|
95
|
+
3. Add capability
|
|
96
|
+
4. Remove capability (⚠️ BREAKING CHANGE)
|
|
97
|
+
5. Modify capability
|
|
98
|
+
6. Add mantra
|
|
99
|
+
7. Remove mantra
|
|
100
|
+
8. Change communication style
|
|
101
|
+
9. Add use case
|
|
102
|
+
10. Update knowledge (business/technical)
|
|
103
|
+
11. Everything (guided full review)
|
|
104
|
+
12. Cancel
|
|
105
|
+
|
|
106
|
+
Choice: ____
|
|
107
|
+
"
|
|
108
|
+
```
|
|
109
|
+
|
|
110
|
+
**Breaking changes flagged:**
|
|
111
|
+
- Agent name change → All references break
|
|
112
|
+
- Capability removal → Users relying on it affected
|
|
113
|
+
- Major role change → Behavior changes significantly
|
|
114
|
+
|
|
115
|
+
---
|
|
116
|
+
|
|
117
|
+
### Step 3: Make Changes (with Consequences)
|
|
118
|
+
|
|
119
|
+
#### 3.1: EDIT AGENT NAME (⚠️ CRITICAL)
|
|
120
|
+
|
|
121
|
+
```
|
|
122
|
+
"⚠️ CHANGING AGENT NAME IS A BREAKING CHANGE!
|
|
123
|
+
|
|
124
|
+
CURRENT NAME: {current_name}
|
|
125
|
+
|
|
126
|
+
CONSEQUENCES OF RENAMING:
|
|
127
|
+
❌ All file references to this agent break
|
|
128
|
+
❌ User workflows mentioning agent by name break
|
|
129
|
+
❌ Documentation must be updated
|
|
130
|
+
❌ CLI commands using old name fail
|
|
131
|
+
❌ Integration tests mentioning agent fail
|
|
132
|
+
|
|
133
|
+
SCOPE: High
|
|
134
|
+
DATA: Medium (file renames required)
|
|
135
|
+
CODE: Medium (references update)
|
|
136
|
+
TEAM: High (everyone must be notified)
|
|
137
|
+
DEPENDENCIES: High (all agent consumers affected)
|
|
138
|
+
|
|
139
|
+
RISK LEVEL: 🔥 CRITICAL
|
|
140
|
+
|
|
141
|
+
Still want to rename? (yes/no): ____
|
|
142
|
+
"
|
|
143
|
+
```
|
|
144
|
+
|
|
145
|
+
**If yes:**
|
|
146
|
+
```
|
|
147
|
+
"New agent name (kebab-case): ____
|
|
148
|
+
|
|
149
|
+
MITIGATION PLAN:
|
|
150
|
+
1. Create backup of current agent
|
|
151
|
+
2. Rename agent spec
|
|
152
|
+
3. Update all file references
|
|
153
|
+
4. Create redirect/alias from old name (temporary)
|
|
154
|
+
5. Notify all team members
|
|
155
|
+
6. Update documentation
|
|
156
|
+
7. Schedule old name deprecation (30 days)
|
|
157
|
+
|
|
158
|
+
Confirm mitigation plan? (yes/no): ____
|
|
159
|
+
"
|
|
160
|
+
```
|
|
161
|
+
|
|
162
|
+
**Validate new name:**
|
|
163
|
+
- Must be kebab-case
|
|
164
|
+
- Must be unique (RG-AGT-001)
|
|
165
|
+
- Must not conflict with reserved names
|
|
166
|
+
|
|
167
|
+
#### 3.2: EDIT ROLE/RESPONSIBILITIES
|
|
168
|
+
|
|
169
|
+
```
|
|
170
|
+
"CURRENT ROLE: {current_role}
|
|
171
|
+
|
|
172
|
+
New role (or press Enter to keep): ____
|
|
173
|
+
"
|
|
174
|
+
```
|
|
175
|
+
|
|
176
|
+
**If changed:**
|
|
177
|
+
```
|
|
178
|
+
"Role changed from:
|
|
179
|
+
OLD: {old_role}
|
|
180
|
+
NEW: {new_role}
|
|
181
|
+
|
|
182
|
+
CONSEQUENCES:
|
|
183
|
+
📊 SCOPE: Medium (agent behavior may shift)
|
|
184
|
+
👥 TEAM: Medium (user expectations change)
|
|
185
|
+
📝 CODE: Low (mostly documentation)
|
|
186
|
+
|
|
187
|
+
⚠️ Users may expect different behavior from this agent.
|
|
188
|
+
|
|
189
|
+
Recommendation: Add changelog note explaining why role changed.
|
|
190
|
+
|
|
191
|
+
Proceed? (yes/no): ____
|
|
192
|
+
"
|
|
193
|
+
```
|
|
194
|
+
|
|
195
|
+
#### 3.3: ADD CAPABILITY
|
|
196
|
+
|
|
197
|
+
```
|
|
198
|
+
"Add new capability:
|
|
199
|
+
|
|
200
|
+
Capability description: ____
|
|
201
|
+
Category (create/analyze/review/optimize/teach): ____
|
|
202
|
+
"
|
|
203
|
+
```
|
|
204
|
+
|
|
205
|
+
**Consequences (usually LOW risk):**
|
|
206
|
+
```
|
|
207
|
+
"Adding capability: {new_capability}
|
|
208
|
+
|
|
209
|
+
CONSEQUENCES:
|
|
210
|
+
✅ POSITIVE: Agent becomes more useful
|
|
211
|
+
✅ SCOPE: Low (additive change, non-breaking)
|
|
212
|
+
⚠️ CODE: Low (tests for new capability needed)
|
|
213
|
+
⚠️ DOCS: Low (document new capability)
|
|
214
|
+
|
|
215
|
+
RISK LEVEL: 🟢 LOW
|
|
216
|
+
|
|
217
|
+
Current capability count: {current_count}
|
|
218
|
+
After adding: {current_count + 1}
|
|
219
|
+
|
|
220
|
+
Note: More capabilities = more complexity. Keep it focused.
|
|
221
|
+
Mantra #37 (Ockham's Razor): Is this truly needed?
|
|
222
|
+
|
|
223
|
+
Proceed? (yes/add-more/cancel): ____
|
|
224
|
+
"
|
|
225
|
+
```
|
|
226
|
+
|
|
227
|
+
#### 3.4: REMOVE CAPABILITY (⚠️ BREAKING)
|
|
228
|
+
|
|
229
|
+
```
|
|
230
|
+
"Which capability to remove?
|
|
231
|
+
|
|
232
|
+
CURRENT CAPABILITIES:
|
|
233
|
+
1. {capability_1}
|
|
234
|
+
2. {capability_2}
|
|
235
|
+
3. {capability_3}
|
|
236
|
+
...
|
|
237
|
+
|
|
238
|
+
Capability number: ____
|
|
239
|
+
"
|
|
240
|
+
```
|
|
241
|
+
|
|
242
|
+
**ALWAYS evaluate consequences:**
|
|
243
|
+
```
|
|
244
|
+
"⚠️ REMOVING CAPABILITY: {capability}
|
|
245
|
+
|
|
246
|
+
CONSEQUENCES EVALUATION:
|
|
247
|
+
|
|
248
|
+
❌ BREAKING CHANGE - Users relying on this capability will be affected
|
|
249
|
+
|
|
250
|
+
QUESTIONS TO ASK:
|
|
251
|
+
1. Is anyone currently using this capability?
|
|
252
|
+
2. Are there workflows that depend on it?
|
|
253
|
+
3. Are there tests that verify it?
|
|
254
|
+
4. Why are we removing it? (technical debt? never used? superseded?)
|
|
255
|
+
|
|
256
|
+
ANALYSIS:
|
|
257
|
+
📊 SCOPE: High (feature removal)
|
|
258
|
+
👥 TEAM: High (check with team first!)
|
|
259
|
+
🔧 CODE: Medium (remove implementation + tests)
|
|
260
|
+
📚 DOCS: Medium (update docs, add migration guide)
|
|
261
|
+
⏱️ TIME: Medium (communication + migration)
|
|
262
|
+
|
|
263
|
+
RISK LEVEL: 🔴 HIGH
|
|
264
|
+
|
|
265
|
+
MITIGATION PLAN:
|
|
266
|
+
1. Survey team: Is this capability used?
|
|
267
|
+
2. If used: Deprecation period (30 days) before removal
|
|
268
|
+
3. If unused: Safe to remove
|
|
269
|
+
4. Update tests
|
|
270
|
+
5. Update documentation
|
|
271
|
+
6. Add to changelog
|
|
272
|
+
|
|
273
|
+
Check if capability is used? (yes/no/cancel): ____
|
|
274
|
+
"
|
|
275
|
+
```
|
|
276
|
+
|
|
277
|
+
**If capability is used:**
|
|
278
|
+
```
|
|
279
|
+
"CAPABILITY IN USE!
|
|
280
|
+
|
|
281
|
+
Found references in:
|
|
282
|
+
- {file_1}: {line}
|
|
283
|
+
- {file_2}: {line}
|
|
284
|
+
...
|
|
285
|
+
|
|
286
|
+
This capability CANNOT be safely removed without breaking things.
|
|
287
|
+
|
|
288
|
+
OPTIONS:
|
|
289
|
+
1. Keep the capability
|
|
290
|
+
2. Create migration guide and deprecate (30 day notice)
|
|
291
|
+
3. Find alternative capability to replace it
|
|
292
|
+
4. Force remove (⚠️ DANGEROUS)
|
|
293
|
+
|
|
294
|
+
What should we do?: ____
|
|
295
|
+
"
|
|
296
|
+
```
|
|
297
|
+
|
|
298
|
+
#### 3.5: MODIFY CAPABILITY
|
|
299
|
+
|
|
300
|
+
```
|
|
301
|
+
"Which capability to modify?
|
|
302
|
+
|
|
303
|
+
CURRENT CAPABILITIES:
|
|
304
|
+
1. {capability_1}
|
|
305
|
+
2. {capability_2}
|
|
306
|
+
...
|
|
307
|
+
|
|
308
|
+
Capability number: ____
|
|
309
|
+
|
|
310
|
+
CURRENT: {capability_description}
|
|
311
|
+
NEW: ____ (or press Enter to keep)
|
|
312
|
+
"
|
|
313
|
+
```
|
|
314
|
+
|
|
315
|
+
**Evaluate change:**
|
|
316
|
+
```
|
|
317
|
+
"Capability modified:
|
|
318
|
+
OLD: {old_description}
|
|
319
|
+
NEW: {new_description}
|
|
320
|
+
|
|
321
|
+
CHANGE TYPE: {minor|moderate|major}
|
|
322
|
+
- Minor: Wording/clarity improvement only
|
|
323
|
+
- Moderate: Behavior slightly different
|
|
324
|
+
- Major: Completely different capability
|
|
325
|
+
|
|
326
|
+
CONSEQUENCES:
|
|
327
|
+
📊 SCOPE: {evaluated_scope}
|
|
328
|
+
👥 TEAM: {evaluated_impact}
|
|
329
|
+
⚠️ Existing users may see behavior change
|
|
330
|
+
|
|
331
|
+
RISK LEVEL: {risk_level}
|
|
332
|
+
|
|
333
|
+
Proceed? (yes/no): ____
|
|
334
|
+
"
|
|
335
|
+
```
|
|
336
|
+
|
|
337
|
+
#### 3.6: ADD/REMOVE MANTRAS
|
|
338
|
+
|
|
339
|
+
**Add Mantra:**
|
|
340
|
+
```
|
|
341
|
+
"Current mantras ({n}/64):
|
|
342
|
+
{list_current_mantras}
|
|
343
|
+
|
|
344
|
+
Which mantra to add? (number 1-64 or name): ____
|
|
345
|
+
|
|
346
|
+
Why add this mantra?: ____
|
|
347
|
+
"
|
|
348
|
+
```
|
|
349
|
+
|
|
350
|
+
**Remove Mantra:**
|
|
351
|
+
```
|
|
352
|
+
"⚠️ Removing a mantra means this agent will no longer prioritize that principle.
|
|
353
|
+
|
|
354
|
+
Which mantra to remove?
|
|
355
|
+
{list_current_mantras_numbered}
|
|
356
|
+
|
|
357
|
+
Mantra number: ____
|
|
358
|
+
|
|
359
|
+
Why remove this mantra?: ____
|
|
360
|
+
|
|
361
|
+
CONSEQUENCE:
|
|
362
|
+
- Agent behavior may shift
|
|
363
|
+
- Quality checks associated with mantra won't apply
|
|
364
|
+
- Risk: Lower quality if mantra was critical
|
|
365
|
+
|
|
366
|
+
RISK LEVEL: 🟡 MEDIUM
|
|
367
|
+
|
|
368
|
+
Confirm removal? (yes/no): ____
|
|
369
|
+
"
|
|
370
|
+
```
|
|
371
|
+
|
|
372
|
+
**Validate RG-AGT-003:** Must have >= 5 mantras after changes
|
|
373
|
+
|
|
374
|
+
#### 3.7: CHANGE COMMUNICATION STYLE
|
|
375
|
+
|
|
376
|
+
```
|
|
377
|
+
"CURRENT STYLE: {current_style}
|
|
378
|
+
|
|
379
|
+
New communication style:
|
|
380
|
+
1. CONCISE (direct, minimal words)
|
|
381
|
+
2. EDUCATIONAL (detailed, teaches)
|
|
382
|
+
3. BALANCED (mix)
|
|
383
|
+
4. CUSTOM (specify)
|
|
384
|
+
|
|
385
|
+
Choice: ____
|
|
386
|
+
"
|
|
387
|
+
```
|
|
388
|
+
|
|
389
|
+
**Consequences (usually LOW):**
|
|
390
|
+
```
|
|
391
|
+
"Communication style changed: {old} → {new}
|
|
392
|
+
|
|
393
|
+
CONSEQUENCES:
|
|
394
|
+
📝 USER EXPERIENCE: Users will notice different tone
|
|
395
|
+
🎯 SCOPE: Low (no functionality change)
|
|
396
|
+
📚 DOCS: Low (update examples if needed)
|
|
397
|
+
|
|
398
|
+
RISK LEVEL: 🟢 LOW
|
|
399
|
+
|
|
400
|
+
User feedback recommended after deployment.
|
|
401
|
+
|
|
402
|
+
Proceed? (yes/no): ____
|
|
403
|
+
"
|
|
404
|
+
```
|
|
405
|
+
|
|
406
|
+
#### 3.8: ADD USE CASE
|
|
407
|
+
|
|
408
|
+
```
|
|
409
|
+
"Add new use case:
|
|
410
|
+
|
|
411
|
+
Scenario: ____
|
|
412
|
+
Expected behavior: ____
|
|
413
|
+
"
|
|
414
|
+
```
|
|
415
|
+
|
|
416
|
+
Consequences: 🟢 LOW (additive, documentation only)
|
|
417
|
+
|
|
418
|
+
---
|
|
419
|
+
|
|
420
|
+
### Step 4: Comprehensive Consequences Summary
|
|
421
|
+
|
|
422
|
+
**After all changes, show complete summary:**
|
|
423
|
+
|
|
424
|
+
```
|
|
425
|
+
"CHANGE SUMMARY:
|
|
426
|
+
|
|
427
|
+
Changes made:
|
|
428
|
+
1. {change_1}
|
|
429
|
+
2. {change_2}
|
|
430
|
+
...
|
|
431
|
+
|
|
432
|
+
FULL CONSEQUENCES EVALUATION (10 Dimensions):
|
|
433
|
+
|
|
434
|
+
1. 📊 SCOPE:
|
|
435
|
+
- Components affected: {list}
|
|
436
|
+
- Breaking changes: {yes/no}
|
|
437
|
+
- Impact: {Low/Medium/High/Critical}
|
|
438
|
+
|
|
439
|
+
2. 💾 DATA:
|
|
440
|
+
- Database changes: {yes/no}
|
|
441
|
+
- Data migration needed: {yes/no}
|
|
442
|
+
- Impact: {Low/Medium/High/Critical}
|
|
443
|
+
|
|
444
|
+
3. 🔧 CODE:
|
|
445
|
+
- Files to modify: {count}
|
|
446
|
+
- Tests to update: {count}
|
|
447
|
+
- Impact: {Low/Medium/High/Critical}
|
|
448
|
+
|
|
449
|
+
4. 👥 TEAM:
|
|
450
|
+
- Team members affected: {all/some/none}
|
|
451
|
+
- Training needed: {yes/no}
|
|
452
|
+
- Impact: {Low/Medium/High/Critical}
|
|
453
|
+
|
|
454
|
+
5. 🎯 CLIENTS:
|
|
455
|
+
- User workflows broken: {yes/no}
|
|
456
|
+
- Migration guide needed: {yes/no}
|
|
457
|
+
- Impact: {Low/Medium/High/Critical}
|
|
458
|
+
|
|
459
|
+
6. ⚖️ LEGAL:
|
|
460
|
+
- Compliance affected: {yes/no}
|
|
461
|
+
- Impact: {Low/Medium/High/Critical}
|
|
462
|
+
|
|
463
|
+
7. 🚀 OPERATIONS:
|
|
464
|
+
- Deployment complexity: {simple/moderate/complex}
|
|
465
|
+
- Rollback plan: {needed/not needed}
|
|
466
|
+
- Impact: {Low/Medium/High/Critical}
|
|
467
|
+
|
|
468
|
+
8. 🔗 DEPENDENCIES:
|
|
469
|
+
- Dependent systems: {count}
|
|
470
|
+
- Breaking changes: {yes/no}
|
|
471
|
+
- Impact: {Low/Medium/High/Critical}
|
|
472
|
+
|
|
473
|
+
9. ⏱️ TIME:
|
|
474
|
+
- Implementation time: {estimate}
|
|
475
|
+
- Communication time: {estimate}
|
|
476
|
+
- Impact: {Low/Medium/High/Critical}
|
|
477
|
+
|
|
478
|
+
10. 🔄 ALTERNATIVES:
|
|
479
|
+
- Alternative approaches considered: {yes/no}
|
|
480
|
+
- Simpler alternatives exist: {yes/no}
|
|
481
|
+
|
|
482
|
+
OVERALL RISK LEVEL: {🟢 LOW | 🟡 MEDIUM | 🔴 HIGH | 🔥 CRITICAL}
|
|
483
|
+
|
|
484
|
+
REQUIRED ACTIONS BEFORE DEPLOYMENT:
|
|
485
|
+
{action_checklist}
|
|
486
|
+
|
|
487
|
+
MITIGATION PLAN:
|
|
488
|
+
{mitigation_steps}
|
|
489
|
+
|
|
490
|
+
Proceed with these changes? (yes/review/cancel): ____
|
|
491
|
+
"
|
|
492
|
+
```
|
|
493
|
+
|
|
494
|
+
---
|
|
495
|
+
|
|
496
|
+
### Step 5: Execute Changes
|
|
497
|
+
|
|
498
|
+
**If approved:**
|
|
499
|
+
|
|
500
|
+
```python
|
|
501
|
+
# Create backup
|
|
502
|
+
backup = agent_spec.create_backup()
|
|
503
|
+
print(f"Backup created: {backup.path}")
|
|
504
|
+
|
|
505
|
+
# Apply changes
|
|
506
|
+
agent_spec.apply_changes(changes)
|
|
507
|
+
|
|
508
|
+
# Update status
|
|
509
|
+
if agent_spec.status == "deployed":
|
|
510
|
+
agent_spec.status = "modified" # Needs revalidation
|
|
511
|
+
elif agent_spec.status == "validated":
|
|
512
|
+
agent_spec.status = "modified" # Needs revalidation
|
|
513
|
+
|
|
514
|
+
# Save
|
|
515
|
+
agent_spec.save()
|
|
516
|
+
|
|
517
|
+
# Create changelog entry
|
|
518
|
+
changelog.add_entry(
|
|
519
|
+
agent=agent_spec,
|
|
520
|
+
changes=changes,
|
|
521
|
+
consequences=consequences_summary,
|
|
522
|
+
timestamp=now()
|
|
523
|
+
)
|
|
524
|
+
```
|
|
525
|
+
|
|
526
|
+
**Confirmation:**
|
|
527
|
+
```
|
|
528
|
+
"Changes applied successfully!
|
|
529
|
+
|
|
530
|
+
BACKUP LOCATION: {backup_path}
|
|
531
|
+
CHANGELOG: {changelog_path}
|
|
532
|
+
|
|
533
|
+
AGENT STATUS: {new_status}
|
|
534
|
+
- Needs revalidation before deployment
|
|
535
|
+
|
|
536
|
+
NEXT STEPS:
|
|
537
|
+
1. [VA] Validate agent against 64 mantras
|
|
538
|
+
2. Test modified agent
|
|
539
|
+
3. Update documentation
|
|
540
|
+
4. {additional_steps_from_mitigation}
|
|
541
|
+
|
|
542
|
+
Would you like to validate now? (yes/later): ____
|
|
543
|
+
"
|
|
544
|
+
```
|
|
545
|
+
|
|
546
|
+
---
|
|
547
|
+
|
|
548
|
+
## ROLLBACK PROCEDURE
|
|
549
|
+
|
|
550
|
+
If something goes wrong:
|
|
551
|
+
|
|
552
|
+
```
|
|
553
|
+
"❌ ERROR during changes: {error}
|
|
554
|
+
|
|
555
|
+
ROLLBACK OPTIONS:
|
|
556
|
+
1. Restore from backup (undo ALL changes)
|
|
557
|
+
2. Restore specific changes only
|
|
558
|
+
3. Try to fix error and continue
|
|
559
|
+
4. Save current state and exit
|
|
560
|
+
|
|
561
|
+
What should we do?: ____
|
|
562
|
+
"
|
|
563
|
+
```
|
|
564
|
+
|
|
565
|
+
**Restore from backup:**
|
|
566
|
+
```python
|
|
567
|
+
def rollback(agent_spec, backup):
|
|
568
|
+
agent_spec.restore_from_backup(backup)
|
|
569
|
+
agent_spec.save()
|
|
570
|
+
print("✅ Rollback successful. Agent restored to previous state.")
|
|
571
|
+
```
|
|
572
|
+
|
|
573
|
+
---
|
|
574
|
+
|
|
575
|
+
## VALIDATION AFTER EDIT
|
|
576
|
+
|
|
577
|
+
**Always recommend validation:**
|
|
578
|
+
|
|
579
|
+
```
|
|
580
|
+
"Edits complete!
|
|
581
|
+
|
|
582
|
+
⚠️ Modified agents should be validated before deployment.
|
|
583
|
+
|
|
584
|
+
Run validation now?
|
|
585
|
+
- [VA] Full validation (checks all 64 mantras)
|
|
586
|
+
- [TEST] Quick test (basic checks only)
|
|
587
|
+
- [LATER] Skip for now
|
|
588
|
+
|
|
589
|
+
Choice: ____
|
|
590
|
+
"
|
|
591
|
+
```
|
|
592
|
+
|
|
593
|
+
---
|
|
594
|
+
|
|
595
|
+
## CHANGE CATEGORIES
|
|
596
|
+
|
|
597
|
+
**LOW RISK (🟢) - Safe to edit:**
|
|
598
|
+
- Add capability
|
|
599
|
+
- Add mantra
|
|
600
|
+
- Add use case
|
|
601
|
+
- Update documentation
|
|
602
|
+
- Improve wording/clarity
|
|
603
|
+
- Change communication style
|
|
604
|
+
|
|
605
|
+
**MEDIUM RISK (🟡) - Caution required:**
|
|
606
|
+
- Modify capability behavior
|
|
607
|
+
- Remove mantra
|
|
608
|
+
- Change role significantly
|
|
609
|
+
- Update knowledge base
|
|
610
|
+
|
|
611
|
+
**HIGH RISK (🔴) - Team approval needed:**
|
|
612
|
+
- Remove capability
|
|
613
|
+
- Change agent name
|
|
614
|
+
- Major behavior changes
|
|
615
|
+
- Breaking API changes
|
|
616
|
+
|
|
617
|
+
**CRITICAL RISK (🔥) - Requires planning:**
|
|
618
|
+
- Rename deployed agent
|
|
619
|
+
- Remove core capability from deployed agent
|
|
620
|
+
- Changes affecting multiple systems
|
|
621
|
+
|
|
622
|
+
---
|
|
623
|
+
|
|
624
|
+
## ANTI-PATTERNS
|
|
625
|
+
|
|
626
|
+
**NEVER do these:**
|
|
627
|
+
|
|
628
|
+
❌ Edit without evaluating consequences
|
|
629
|
+
❌ Remove capabilities without checking usage
|
|
630
|
+
❌ Rename agents without migration plan
|
|
631
|
+
❌ Skip backup before editing
|
|
632
|
+
❌ Edit deployed agents without team notice
|
|
633
|
+
❌ Make breaking changes without deprecation
|
|
634
|
+
❌ Ignore dependency impacts
|
|
635
|
+
❌ Edit without understanding WHY
|
|
636
|
+
|
|
637
|
+
**ALWAYS do these:**
|
|
638
|
+
|
|
639
|
+
✅ Create backup first
|
|
640
|
+
✅ Evaluate consequences (Mantra #39)
|
|
641
|
+
✅ Challenge changes (Mantra IA-16)
|
|
642
|
+
✅ Check for simpler alternatives (Mantra #37)
|
|
643
|
+
✅ Communicate breaking changes to team
|
|
644
|
+
✅ Update documentation
|
|
645
|
+
✅ Revalidate after changes
|
|
646
|
+
✅ Test before deploying
|
|
647
|
+
|
|
648
|
+
---
|
|
649
|
+
|
|
650
|
+
## SUCCESS CRITERIA
|
|
651
|
+
|
|
652
|
+
✅ Agent loaded successfully
|
|
653
|
+
✅ Changes clearly identified
|
|
654
|
+
✅ Consequences evaluated (10 dimensions)
|
|
655
|
+
✅ Risk level assessed
|
|
656
|
+
✅ Mitigation plan created
|
|
657
|
+
✅ Backup created before changes
|
|
658
|
+
✅ Changes applied successfully
|
|
659
|
+
✅ Validation rules still pass
|
|
660
|
+
✅ Changelog updated
|
|
661
|
+
✅ User aware of next steps
|
|
662
|
+
|
|
663
|
+
---
|
|
664
|
+
|
|
665
|
+
## COMPLETION
|
|
666
|
+
|
|
667
|
+
```
|
|
668
|
+
"Edit workflow complete!
|
|
669
|
+
|
|
670
|
+
SUMMARY:
|
|
671
|
+
- Agent: {agent_name}
|
|
672
|
+
- Changes: {n} modifications
|
|
673
|
+
- Risk Level: {risk_level}
|
|
674
|
+
- Status: {new_status}
|
|
675
|
+
|
|
676
|
+
FILES UPDATED:
|
|
677
|
+
- Agent spec: {spec_path}
|
|
678
|
+
- Backup: {backup_path}
|
|
679
|
+
- Changelog: {changelog_path}
|
|
680
|
+
|
|
681
|
+
RECOMMENDED NEXT STEPS:
|
|
682
|
+
1. {step_1}
|
|
683
|
+
2. {step_2}
|
|
684
|
+
...
|
|
685
|
+
|
|
686
|
+
Session saved to: {output_folder}/edit-agent-{agent_name}-{timestamp}.md
|
|
687
|
+
"
|
|
688
|
+
```
|