Agile Close Task
Table of Contents
1. When to use this skill
As soon as implementation is complete and locally verified (build + tests green) — before raising the PR and before requesting review. The task is done as far as the developer is concerned; review is a post-done validation activity, not a step that bookkeeping waits on. Never close bookkeeping after review approves, and never in a follow-up PR/commit once CI has already gone green — either wastes a full CI cycle for a change with no code content.
If a review round turns up a real functional gap that reopens the
work, that's an exception to "done" (the implementation genuinely
wasn't finished) — fix it as part of that round's own commit and
amend * Result=/=* Decisions in the same commit, rather than treating
routine review feedback as reopening the task by default.
2. How to use this skill
Run the component audit first, when the task touched a component. Clearing codegen drift, moving a part, adding a facet — none of it says the component has a legal shape, and
validate_docsis only the audit's automated half. Walk the component architecture audit over each component the task touched, and record what it found on the task. A component whose task closed without it has been declared done on evidence nobody gathered.projects/ores.codegen/validate_docs.sh
Flip the state — task DONE, story table row synced, journal stamped:
./projects/ores.compass/compass.sh task done <task-slug>
- Write the
* Resultby hand: what shipped, what acceptance was met. - Distill the
* Planinto the parent story's* Decisionsif it holds durable choices. - Cascade if last task: when no open tasks remain, set the story DONE in
story.organd its row insprint.org— both files in the same commit. - Commit per commit conventions:
[agile] Close <task>. This commit is part of whatcompass-pr-raisepushes when it opens the PR — not a follow-up. Once the PR is open, CI and review run against a branch that already reflects the task as done; when CI goes green post-approval, merge immediately withcompass-pr-merge, no further commits.
3. Recipes
- How do I work a task? — the state transitions and the story and sprint sync.
4. Reference
- Work a task through to merged PR — steps 8–9.
- Clean up the compass work lifecycle — why merge never touches task state.