Skip to content

Commit 498251f

Browse files
fix(tasks,implement): address review — remove duplicate format note and fix context-loss gap
Remove governed-operation hint from tasks-template.md Format Legend; the generation rule in tasks.md is the single authoritative source, so the legend entry was redundant duplication. Add Task-level re-read instruction to implement.md step 6: before executing each task the agent re-reads that task's entry from tasks.md on disk. This closes the gap the reviewer identified — the initial tasks.md load can be compacted out of context during a long session, but a per-task re-read ensures embedded governed commands are always in active context at execution time.
1 parent b957ccc commit 498251f

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

templates/commands/implement.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -142,10 +142,11 @@ You **MUST** consider the user input before proceeding (if not empty).
142142
143143
6. Execute implementation following the task plan:
144144
- **Phase-by-phase execution**: Complete each phase before moving to the next
145-
- **Respect dependencies**: Run sequential tasks in order, parallel tasks [P] can run together
145+
- **Respect dependencies**: Run sequential tasks in order, parallel tasks [P] can run together
146146
- **Follow TDD approach**: Execute test tasks before their corresponding implementation tasks
147147
- **File-based coordination**: Tasks affecting the same files must run sequentially
148148
- **Validation checkpoints**: Verify each phase completion before proceeding
149+
- **Task-level re-read**: Before executing each individual task, re-read that task's entry from tasks.md to get its complete body. This ensures any embedded commands (e.g., governed operations with exact flags and env vars) are in active context even if the initial tasks.md load was compressed out during a long session.
149150
150151
7. Implementation execution rules:
151152
- **Setup first**: Initialize project structure, dependencies, configuration

templates/tasks-template.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ description: "Task list template for feature implementation"
1717
- **[P]**: Can run in parallel (different files, no dependencies)
1818
- **[Story]**: Which user story this task belongs to (e.g., US1, US2, US3)
1919
- Include exact file paths in descriptions
20-
- For **governed operations** (commands the constitution defines with specific syntax): append the exact command — e.g., `Push packages: \`dotnet nuget push "*.nupkg" --api-key $KEY --source $URL\``
2120

2221
## Path Conventions
2322

0 commit comments

Comments
 (0)