Skip to content

Commit 370992c

Browse files
shinprclaude
andcommitted
feat: Apply implementation-approach.md references to English version subagents
## Changes (English agents) - technical-designer: Add implementation-approach.md to initial tasks, clarify Phase 1-4 execution - work-planner: Add to initial tasks, specify L1/L2/L3 verification level usage - task-executor: Improve progress update with concrete Edit tool instructions ## Improvements - Ensures proper consideration of vertical/horizontal slicing strategies - Prevents progress tracking omissions - Improves AI execution accuracy across language environments 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent a246cfe commit 370992c

File tree

3 files changed

+19
-5
lines changed

3 files changed

+19
-5
lines changed

.claude/agents-en/task-executor.md

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,10 @@ ls docs/plans/tasks/*.md | grep -E "task-[0-9]{2}\.md$" | head -1
6666
### 3. Implementation Execution
6767
- If all checkboxes are `[x]`, report "already completed" and end
6868
- Staged implementation with incremental verification
69-
- 3-location synchronized update upon each step completion
69+
- Upon each step completion【Required】Update checkboxes using Edit tool:
70+
1. Task file: `[ ]``[x]`
71+
2. Corresponding section in work plan: `[ ]``[x]`
72+
3. Progress section in overall design document (if exists)
7073
- Run only added tests and confirm they pass (overall tests not needed)
7174

7275
### 3.5 Operation Verification【Mandatory】
@@ -97,6 +100,11 @@ Report in the following JSON format upon task completion (**without executing qu
97100
"filesModified": ["file1.ts", "file2.ts"],
98101
"testsAdded": ["test1.test.ts"],
99102
"newTestsPassed": true,
103+
"progressUpdated": {
104+
"taskFile": "5/8 items completed",
105+
"workPlan": "Updated",
106+
"designDoc": "N/A"
107+
},
100108
"runnableCheck": {
101109
"level": "L1/L2/L3",
102110
"executed": true,
@@ -111,7 +119,7 @@ Report in the following JSON format upon task completion (**without executing qu
111119

112120
## Execution Principles
113121

114-
- **3-location synchronized update**: Always update upon each action completion
122+
- **Progress checkbox update**: Always update `[ ]``[x]` using Edit tool upon each action completion
115123
- **Overall design document confirmation**: Required before implementation
116124
- **Complete self-containment**: Execute to the end without asking questions
117125
- **Test-first**: Adhere to Red-Green-Refactor process (follow project test rules)

.claude/agents-en/technical-designer.md

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ Before starting work, be sure to read and follow these rule files:
1414
- @docs/rules/typescript.md - TypeScript development rules
1515
- @docs/rules/ai-development-guide.md - AI development guide
1616
- @docs/rules/project-context.md - Project context
17+
- @docs/rules/architecture/implementation-approach.md - Metacognitive strategy selection process (used for implementation approach decisions)
1718
- @docs/rules/architecture/ architecture rule files (if exist)
1819
- Read if project-specific architecture rules are defined
1920
- Apply rules according to adopted architecture patterns
@@ -80,14 +81,15 @@ Must be performed at the beginning of Design Doc creation:
8081
Must be performed when creating Design Doc:
8182

8283
1. **Approach Selection Criteria**
84+
- Execute Phase 1-4 of @docs/rules/architecture/implementation-approach.md to select strategy
8385
- **Vertical Slice**: Complete by feature unit, minimal external dependencies, early value delivery
8486
- **Horizontal Slice**: Implementation by layer, important common foundation, technical consistency priority
8587
- **Hybrid**: Composite, handles complex requirements
86-
- Document selection reason (consider feature complexity, dependencies, integration points)
88+
- Document selection reason (record results of metacognitive strategy selection process)
8789

8890
2. **Integration Point Definition**
8991
- Which task first makes the whole system operational
90-
- Verification level for each task (defined in @docs/rules/architecture/implementation-approach.md)
92+
- Verification level for each task (L1/L2/L3 defined in @docs/rules/architecture/implementation-approach.md)
9193

9294
### Change Impact Map【Required】
9395
Must be included when creating Design Doc.

.claude/agents-en/work-planner.md

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,10 @@ Before starting work, be sure to read and follow these rule files:
1515
- @docs/rules/typescript-testing.md - Testing rules
1616
- @docs/rules/project-context.md - Project context
1717
- @docs/rules/typescript.md - TypeScript development rules
18+
- @docs/rules/architecture/implementation-approach.md - Implementation strategy patterns and verification level definitions (used for task decomposition)
19+
- @docs/rules/architecture/ architecture rule files (if exist)
20+
- Read if project-specific architecture rules are defined
21+
- Apply rules according to adopted architecture patterns
1822

1923
## Main Responsibilities
2024

@@ -80,7 +84,7 @@ Include completion conditions in task names (e.g., "Service implementation and u
8084
## Task Decomposition Principles
8185

8286
### Implementation Approach Application
83-
Determine task sequence based on the implementation approach (vertical/horizontal/hybrid slice) decided in the Design Doc.
87+
Decompose tasks based on the implementation approach decided in the Design Doc, following the integration point definitions and L1/L2/L3 verification levels from @docs/rules/architecture/implementation-approach.md.
8488

8589
### Task Dependency Minimization Rules
8690
- Dependencies up to 2 levels maximum (A→B→C acceptable, A→B→C→D requires redesign)

0 commit comments

Comments
 (0)