Restructure the product task workflow docs and skills - #3657
Conversation
Spec is now write-once; the breakdown moves out of grilling into a new to-tickets skill; land prunes working files to just spec.md. Rename subtask → ticket across docs and skills. Split the workflow doc into a README spine plus concepts.md and structure.md. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
tom2drum
left a comment
tom2drum
left a comment
There was a problem hiding this comment.
| Id | Sev | Axis | Loc |
|---|---|---|---|
| F1 | blocker | correctness | implement-ticket:25 |
| F2 | blocker | correctness | tasks/README.md:35 |
| F3 | major | correctness | create-pr:22 |
| F4 | major | correctness | to-spec:40 |
| F5 | major | correctness | concepts.md:95 |
| F6 | major | correctness | implement-ticket:37 |
| F7 | major | correctness | tasks/README.md:33 |
| F8 | major | standards | gh-commands.md:18 |
| F9 | nit | standards | tasks/README.md:32 |
| F10 | nit | standards | create-pr:118 |
No spec axis — this PR has no spec.md (work outside the task-folder workflow).
Not anchorable
F11 · major · correctness — .claude/agents/code-reviewer.md:3 still says “Dispatched by implement-task”, “the only file you write is the review record”, Outcome clear|blocked|needs-human. implement-task and review-template.md are gone; review-changes posts inline comments and writes no record. Point this agent at review-changes or delete it. (File is not in the PR diff.)
— Reviewed by Cursor Grok 4.6
|
F11 (from the review body's Not anchorable section) — fix, in 5ea1b18. |
tom2drum
left a comment
tom2drum
left a comment
There was a problem hiding this comment.
Arbitration round.
| Id | Ruling |
|---|---|
| F1 | verified |
| F2 | verified |
| F3 | verified |
| F4 | verified |
| F5 | verified |
| F6 | not fixed — still open |
| F7 | verified |
| F8 | verified |
| F9 | verified |
| F10 | verified |
| F11 | verified (was not-anchorable; file is now in the diff) |
Not anchorable
None this round.
— Reviewed by Cursor Grok 4.6
F6: make implement-ticket resumption-safe — start at the first unchecked leaf, stop at the first unchecked [human] leaf, and continue in-session after a human touchpoint rather than re-invoking the skill. F12: code-reviewer is invoked manually, not auto-launched at land. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
tom2drum
left a comment
tom2drum
left a comment
There was a problem hiding this comment.
Review clear
| Severity | Open |
|---|---|
| blocker | 0 |
| major | 0 |
| nit | 0 |
| Axis | Open |
|---|---|
| spec | n/a |
| standards | 0 |
| correctness | 0 |
— Reviewed by Cursor Grok 4.6
1 participant
Description
Restructures the product-task workflow — the docs under
.agents/tasks/and every skill that runs it. No product code.tasks/README.mdis the process spine; newconcepts.md(vocabulary, the ticket model, the write-once / freeze-on-land / prune-on-land rules) andstructure.md(task-folder layout, file-ownership table).to-ticketsskill owns the breakdown, ticket authoring, andprogress.md(moved out ofgrill-the-task). Newfinalize-taskprunes the working files at land (onlyspec.mdsurvives) and finalizes the PR.implement-task→implement-ticket: one ticket per run, commits when the ticket has no human touchpoint; the reviewer /--automachinery is gone.to-specis create-once (spec.md + questions.md + draft PR).review-changes/resolve-reviewbecome the land-only whole-task pass — the spec's Functional Requirements are the review contract, posted as inline PR comments, with no per-ticket review file.Blocked bylist withT<NN>(ticket) andQ<NN>(question) prefixes.Environment variables
None.
Minimum API version
None.
Breaking or incompatible changes
None. Existing task specs under
.agents/tasks/*/keep theirsubtasks/layout as frozen records.Additional information
Instruction-surface only;
pnpm lint:doc-linkspasses.