feat: orchestrator marks plan steps complete after LGTM

This commit is contained in:
Bryan Ramos 2026-04-01 17:19:38 -04:00
parent d3bc447563
commit f7d3e1bd73
2 changed files with 10 additions and 11 deletions

View file

@ -102,12 +102,10 @@ What could go wrong. Edge cases. Breaking changes.
[see Risk Tags section below]
## Implementation Plan
Ordered list of concrete steps. Each step must include:
- **What**: The specific change
- **Where**: File path(s)
- **How**: Implementation approach
Ordered list of concrete steps using checkbox format. Each step must include:
- [ ] **Step N: [short title]** — What, Where, How
Each step scoped to a single logical change.
Each step scoped to a single logical change. The orchestrator checks off steps as they are completed and approved — do not use any other format for steps.
## Acceptance Criteria
Numbered list of specific, testable criteria.
@ -157,11 +155,10 @@ What could go wrong. Edge cases. Breaking changes. Security implications.
[see Risk Tags section below]
## Implementation Plan
Ordered list of concrete steps. Each step must include:
- **What**: The specific change (function to add, interface to implement, config to update)
- **Where**: File path(s) and location within the file
- **How**: Implementation approach including function signatures and key logic
- **Why**: Brief rationale if the step is non-obvious
Ordered list of concrete steps using checkbox format. Each step must include:
- [ ] **Step N: [short title]** — What/Where/How. Add **Why** if non-obvious.
Each step scoped to a single logical change — one commit's worth of work. The orchestrator checks off steps as they are completed and approved — do not use any other format for steps.
Each step scoped to a single logical change — one commit's worth of work.

View file

@ -242,11 +242,13 @@ When multiple risk tags are present, take the union of all mandatory reviewers.
### Git flow
Workers signal `RFR` when done. You control commits:
- `LGTM` → worker commits
- `LGTM` → worker commits → **update the plan file: mark the completed step as `- [x]`**
- `REVISE` → worker fixes and resubmits with `RFR`
- Merge worktree branches after individual validation
- On Tier 2+: merge each worker's branch after validation, resolve conflicts if branches overlap
Only the orchestrator updates the plan file. Workers must not modify `.claude/plans/`.
### Review signals
| Signal | Direction | Meaning |