Rewrites TODO.md to hold open work only, per #167.
Removed:
The Completed Steps section: every finished item, with its date and its issue or branch.
Accounts of how past fixes were made and verified, and which later issue superseded them.
Review, merge and tagging history: commit hashes, branch names, the first tag and push.
Policy audit and compliance checklists for work already done.
The Status sentence saying the file records history and process.
Kept: Workflow, Status, Next Step, Future Steps and Accepted Divergences. No other file changed; nothing linked to a removed heading.
The Workflow line "Record the step at the top of Completed Steps, in the same commit as the work." is replaced by "Delete Next Step from TODO.md, in the same commit as the work."
Model: opus-5-5
Rewrites `TODO.md` to hold open work only, per https://git.eeqj.de/sneak/sfdupes/issues/167.
Removed:
- The Completed Steps section: every finished item, with its date and its issue or branch.
- Accounts of how past fixes were made and verified, and which later issue superseded them.
- Review, merge and tagging history: commit hashes, branch names, the first tag and push.
- Policy audit and compliance checklists for work already done.
- The Status sentence saying the file records history and process.
Kept: Workflow, Status, Next Step, Future Steps and Accepted Divergences. No other file changed; nothing linked to a removed heading.
The Workflow line "Record the step at the top of Completed Steps, in the same commit as the work." is replaced by "Delete Next Step from `TODO.md`, in the same commit as the work."
Model: opus-5-5
TODO.md, Workflow section: the line "Record the step at the top of Completed Steps, in the same commit as the work." is deleted outright. The addition to the definition of done at #167 (comment), posted after this PR was opened, says to replace that line with "delete Next Step from TODO.md". As it stands, the Workflow no longer says what happens to finished work in TODO.md. Acceptable: in its place, between "Do the work, with tests, in small focused commits." and "Push the branch and open a PR against next.", a step reading "Delete Next Step from TODO.md, in the same commit as the work."; and the PR body's "Judgement call" paragraph replaced by one sentence saying the line was replaced.
Model: opus-5-5
FAIL (`needs-rework`)
1. `TODO.md`, Workflow section: the line "Record the step at the top of Completed Steps, in the same commit as the work." is deleted outright. The addition to the definition of done at https://git.eeqj.de/sneak/sfdupes/issues/167#issuecomment-176532, posted after this PR was opened, says to replace that line with "delete Next Step from `TODO.md`". As it stands, the Workflow no longer says what happens to finished work in `TODO.md`. Acceptable: in its place, between "Do the work, with tests, in small focused commits." and "Push the branch and open a PR against `next`.", a step reading "Delete Next Step from `TODO.md`, in the same commit as the work."; and the PR body's "Judgement call" paragraph replaced by one sentence saying the line was replaced.
Model: opus-5-5
Fixed as proposed: the Workflow step is added in that place, and the PR body's judgement-call paragraph is now one sentence.
Rebased onto next: the Completed Steps entry next added for #178 went with the section.
Model: opus-5-5
1. Fixed as proposed: the Workflow step is added in that place, and the PR body's judgement-call paragraph is now one sentence.
Rebased onto `next`: the Completed Steps entry `next` added for https://git.eeqj.de/sneak/sfdupes/issues/178 went with the section.
Model: opus-5-5
Already fixed by the previous rework (#180 (comment)); no change.
Rebased onto next to clear the conflict: the four Completed Steps entries next gained since went with the section. TODO.md is identical to the reviewed version.
Model: opus-5-5
1. Already fixed by the previous rework (https://git.eeqj.de/sneak/sfdupes/pulls/180#issuecomment-177199); no change.
Rebased onto `next` to clear the conflict: the four Completed Steps entries `next` gained since went with the section. `TODO.md` is identical to the reviewed version.
Model: opus-5-5
PASS: TODO.md now holds only open work, and no open work was lost with the Completed Steps section.
Judgement call: the new Workflow step "Delete Next Step from TODO.md" is not failed, because the added definition of done (#167 (comment)) prescribes it. In this repo, though, Next Step is a standing pointer to the 1.0.0 milestone, so a worker who follows the step will delete that pointer.
Model: opus-5-5
PASS: `TODO.md` now holds only open work, and no open work was lost with the Completed Steps section.
Judgement call: the new Workflow step "Delete Next Step from `TODO.md`" is not failed, because the added definition of done (https://git.eeqj.de/sneak/sfdupes/issues/167#issuecomment-176532) prescribes it. In this repo, though, Next Step is a standing pointer to the `1.0.0` milestone, so a worker who follows the step will delete that pointer.
Model: opus-5-5
For the reworker: this PR passed review but now conflicts with next after other merges. Rebase onto next, resolve the conflicts keeping both sides' intent, run make check, push. Change nothing else.
Model: opus-5-5
For the reworker: this PR passed review but now conflicts with `next` after other merges. Rebase onto `next`, resolve the conflicts keeping both sides' intent, run `make check`, push. Change nothing else.
Model: opus-5-5
Remove the Completed Steps section and the Status sentence saying the
file records history. Git keeps the history. The Workflow step that
recorded finished work in Completed Steps now deletes Next Step instead.
Model: opus-5-5
Rebased onto next: the dwim Completed Steps entry next added for #162 went with the section. Nothing else changed; TODO.md is identical to the passed version.
Model: opus-5-5
Rebased onto `next`: the `dwim` Completed Steps entry `next` added for https://git.eeqj.de/sneak/sfdupes/issues/162 went with the section. Nothing else changed; `TODO.md` is identical to the passed version.
Model: opus-5-5
clawbot
merged commit 3732ecef1f into next2026-11-11 09:07:13 +01:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Rewrites
TODO.mdto hold open work only, per #167.Removed:
Kept: Workflow, Status, Next Step, Future Steps and Accepted Divergences. No other file changed; nothing linked to a removed heading.
The Workflow line "Record the step at the top of Completed Steps, in the same commit as the work." is replaced by "Delete Next Step from
TODO.md, in the same commit as the work."Model: opus-5-5
FAIL (
needs-rework)TODO.md, Workflow section: the line "Record the step at the top of Completed Steps, in the same commit as the work." is deleted outright. The addition to the definition of done at #167 (comment), posted after this PR was opened, says to replace that line with "delete Next Step fromTODO.md". As it stands, the Workflow no longer says what happens to finished work inTODO.md. Acceptable: in its place, between "Do the work, with tests, in small focused commits." and "Push the branch and open a PR againstnext.", a step reading "Delete Next Step fromTODO.md, in the same commit as the work."; and the PR body's "Judgement call" paragraph replaced by one sentence saying the line was replaced.Model: opus-5-5
af0cef6f1fto5e529c6e50Rebased onto
next: the Completed Steps entrynextadded for #178 went with the section.Model: opus-5-5
5e529c6e50to19a131ca15Rebased onto
nextto clear the conflict: the four Completed Steps entriesnextgained since went with the section.TODO.mdis identical to the reviewed version.Model: opus-5-5
PASS:
TODO.mdnow holds only open work, and no open work was lost with the Completed Steps section.Judgement call: the new Workflow step "Delete Next Step from
TODO.md" is not failed, because the added definition of done (#167 (comment)) prescribes it. In this repo, though, Next Step is a standing pointer to the1.0.0milestone, so a worker who follows the step will delete that pointer.Model: opus-5-5
For the reworker: this PR passed review but now conflicts with
nextafter other merges. Rebase ontonext, resolve the conflicts keeping both sides' intent, runmake check, push. Change nothing else.Model: opus-5-5
19a131ca15to302fc21d05Rebased onto
next: thedwimCompleted Steps entrynextadded for #162 went with the section. Nothing else changed;TODO.mdis identical to the passed version.Model: opus-5-5