mirror of
https://github.com/catlog22/Claude-Code-Workflow.git
synced 2026-02-12 02:37:45 +08:00
refactor: Simplify and standardize TodoWrite structure in review-module-cycle and review-session-cycle documentation for clarity
This commit is contained in:
@@ -710,27 +710,13 @@ function getDimensionGuidance(dimension) {
|
|||||||
```javascript
|
```javascript
|
||||||
TodoWrite({
|
TodoWrite({
|
||||||
todos: [
|
todos: [
|
||||||
{
|
{ content: "Phase 1: Discovery & Initialization", status: "completed", activeForm: "Initializing" },
|
||||||
content: "Phase 1: Discovery & Initialization",
|
{ content: "Phase 2: Parallel Reviews (7 dimensions)", status: "in_progress", activeForm: "Reviewing" },
|
||||||
status: "completed",
|
{ content: " → Security review", status: "in_progress", activeForm: "Analyzing security" },
|
||||||
activeForm: "Completed discovery & initialization"
|
// ... other dimensions as sub-items
|
||||||
},
|
{ content: "Phase 3: Aggregation", status: "pending", activeForm: "Aggregating" },
|
||||||
{
|
{ content: "Phase 4: Deep-dive", status: "pending", activeForm: "Deep-diving" },
|
||||||
content: "Phase 2: Parallel Reviews (7 dimensions)",
|
{ content: "Phase 5: Completion", status: "pending", activeForm: "Completing" }
|
||||||
status: "in_progress",
|
|
||||||
activeForm: "Executing parallel reviews"
|
|
||||||
},
|
|
||||||
{
|
|
||||||
content: " → Security review (src/auth/**)",
|
|
||||||
status: "completed",
|
|
||||||
activeForm: "Analyzing security"
|
|
||||||
},
|
|
||||||
{
|
|
||||||
content: " → Architecture review (src/auth/**)",
|
|
||||||
status: "completed",
|
|
||||||
activeForm: "Analyzing architecture"
|
|
||||||
},
|
|
||||||
// ... (same pattern as review-session-cycle)
|
|
||||||
]
|
]
|
||||||
});
|
});
|
||||||
```
|
```
|
||||||
|
|||||||
@@ -715,76 +715,17 @@ function getDimensionGuidance(dimension) {
|
|||||||
```javascript
|
```javascript
|
||||||
TodoWrite({
|
TodoWrite({
|
||||||
todos: [
|
todos: [
|
||||||
{
|
{ content: "Phase 1: Discovery & Initialization", status: "completed", activeForm: "Initializing" },
|
||||||
content: "Phase 1: Discovery & Initialization",
|
{ content: "Phase 2: Parallel Reviews (7 dimensions)", status: "in_progress", activeForm: "Reviewing" },
|
||||||
status: "completed",
|
{ content: " → Security review", status: "in_progress", activeForm: "Analyzing security" },
|
||||||
activeForm: "Completed discovery & initialization"
|
// ... other dimensions as sub-items
|
||||||
},
|
{ content: "Phase 3: Aggregation", status: "pending", activeForm: "Aggregating" },
|
||||||
{
|
{ content: "Phase 4: Deep-dive", status: "pending", activeForm: "Deep-diving" },
|
||||||
content: "Phase 2: Parallel Reviews (7 dimensions)",
|
{ content: "Phase 5: Completion", status: "pending", activeForm: "Completing" }
|
||||||
status: "in_progress",
|
|
||||||
activeForm: "Executing parallel reviews"
|
|
||||||
},
|
|
||||||
{
|
|
||||||
content: " → Security review",
|
|
||||||
status: "completed",
|
|
||||||
activeForm: "Analyzing security"
|
|
||||||
},
|
|
||||||
{
|
|
||||||
content: " → Architecture review",
|
|
||||||
status: "completed",
|
|
||||||
activeForm: "Analyzing architecture"
|
|
||||||
},
|
|
||||||
{
|
|
||||||
content: " → Quality review",
|
|
||||||
status: "in_progress",
|
|
||||||
activeForm: "Analyzing quality"
|
|
||||||
},
|
|
||||||
{
|
|
||||||
content: " → Action-Items review",
|
|
||||||
status: "pending",
|
|
||||||
activeForm: "Analyzing action items"
|
|
||||||
},
|
|
||||||
{
|
|
||||||
content: " → Performance review",
|
|
||||||
status: "pending",
|
|
||||||
activeForm: "Analyzing performance"
|
|
||||||
},
|
|
||||||
{
|
|
||||||
content: " → Maintainability review",
|
|
||||||
status: "pending",
|
|
||||||
activeForm: "Analyzing maintainability"
|
|
||||||
},
|
|
||||||
{
|
|
||||||
content: " → Best-Practices review",
|
|
||||||
status: "pending",
|
|
||||||
activeForm: "Analyzing best practices"
|
|
||||||
},
|
|
||||||
{
|
|
||||||
content: "Phase 3: Aggregation",
|
|
||||||
status: "pending",
|
|
||||||
activeForm: "Aggregating findings"
|
|
||||||
},
|
|
||||||
{
|
|
||||||
content: "Phase 4: Iteration 1 - Deep-dive (5 findings)",
|
|
||||||
status: "pending",
|
|
||||||
activeForm: "Executing deep-dive iteration 1"
|
|
||||||
},
|
|
||||||
{
|
|
||||||
content: "Phase 5: Completion",
|
|
||||||
status: "pending",
|
|
||||||
activeForm: "Generating final report"
|
|
||||||
}
|
|
||||||
]
|
]
|
||||||
});
|
});
|
||||||
```
|
```
|
||||||
|
|
||||||
**Update Rules**:
|
|
||||||
- Add dimension items during parallel phase
|
|
||||||
- Add iteration items dynamically based on findings
|
|
||||||
- Mark completed immediately after each phase/agent
|
|
||||||
- Update parent task when phase complete
|
|
||||||
|
|
||||||
## Best Practices
|
## Best Practices
|
||||||
|
|
||||||
1. **Default Settings Work**: 7 dimensions + 3 iterations sufficient for most cases
|
1. **Default Settings Work**: 7 dimensions + 3 iterations sufficient for most cases
|
||||||
|
|||||||
Reference in New Issue
Block a user