mirror of
https://github.com/catlog22/Claude-Code-Workflow.git
synced 2026-03-02 15:23:19 +08:00
fix(skill): constrain roadmap interactive feedback to plan-only modifications
Add explicit constraints preventing Phase 3 interactive rounds from modifying source code. Feedback handlers now clearly scope updates to roadmap.md and issues.jsonl only, with code changes deferred to handoff.
This commit is contained in:
@@ -687,7 +687,8 @@ ${selectedMode === 'progressive' ? `**Progressive Mode**:
|
|||||||
if (feedback.feedback === 'approve') {
|
if (feedback.feedback === 'approve') {
|
||||||
approved = true
|
approved = true
|
||||||
} else {
|
} else {
|
||||||
// Handle feedback type
|
// CONSTRAINT: All modifications below ONLY touch roadmap.md and issues.jsonl
|
||||||
|
// NEVER modify source code or project files during interactive rounds
|
||||||
switch (feedback.feedback) {
|
switch (feedback.feedback) {
|
||||||
case 'scope':
|
case 'scope':
|
||||||
// Collect scope adjustments
|
// Collect scope adjustments
|
||||||
@@ -695,8 +696,9 @@ ${selectedMode === 'progressive' ? `**Progressive Mode**:
|
|||||||
{ id: "adjustments", type: "text", prompt: "Describe scope adjustments needed:" }
|
{ id: "adjustments", type: "text", prompt: "Describe scope adjustments needed:" }
|
||||||
]) // BLOCKS
|
]) // BLOCKS
|
||||||
|
|
||||||
// Update roadmap.md and issues
|
// Update ONLY roadmap.md Roadmap table + Convergence sections
|
||||||
// ... implementation ...
|
Edit({ path: `${sessionFolder}/roadmap.md`, /* scope changes */ })
|
||||||
|
// Update ONLY issues.jsonl entries (scope/context fields)
|
||||||
|
|
||||||
break
|
break
|
||||||
|
|
||||||
@@ -706,8 +708,8 @@ ${selectedMode === 'progressive' ? `**Progressive Mode**:
|
|||||||
{ id: "refinements", type: "text", prompt: "Describe convergence refinements needed:" }
|
{ id: "refinements", type: "text", prompt: "Describe convergence refinements needed:" }
|
||||||
]) // BLOCKS
|
]) // BLOCKS
|
||||||
|
|
||||||
// Update roadmap.md
|
// Update ONLY roadmap.md Convergence Criteria section
|
||||||
// ... implementation ...
|
Edit({ path: `${sessionFolder}/roadmap.md`, /* convergence changes */ })
|
||||||
|
|
||||||
break
|
break
|
||||||
|
|
||||||
@@ -726,8 +728,7 @@ ${selectedMode === 'progressive' ? `**Progressive Mode**:
|
|||||||
]) // BLOCKS
|
]) // BLOCKS
|
||||||
|
|
||||||
selectedMode = newStrategy.strategy
|
selectedMode = newStrategy.strategy
|
||||||
// Re-execute Phase 2
|
// Re-execute Phase 2 (updates roadmap.md + issues.jsonl only)
|
||||||
// ... goto Phase 2 ...
|
|
||||||
break
|
break
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -871,6 +872,7 @@ To execute later:
|
|||||||
4. **Testable Convergence**: criteria = assertions, DoD = business language
|
4. **Testable Convergence**: criteria = assertions, DoD = business language
|
||||||
5. **Explicit Lifecycle**: Always close_agent after wait completes to free resources
|
5. **Explicit Lifecycle**: Always close_agent after wait completes to free resources
|
||||||
6. **DO NOT STOP**: Continuous workflow until handoff complete
|
6. **DO NOT STOP**: Continuous workflow until handoff complete
|
||||||
|
7. **Plan-Only Modifications**: Interactive feedback (Phase 3) MUST only update `roadmap.md` and `issues.jsonl`. NEVER modify source code, configuration files, or any project files during interactive rounds. Code changes happen only after handoff (Phase 4) via team-planex or other execution skills
|
||||||
|
|
||||||
---
|
---
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user