Drop the work log from TODO.md; keep only open work #257

Merged
clawbot merged 1 commits from issue-255-todo-open-work into next 2026-11-11 20:25:40 +01:00
Collaborator

Rewrites TODO.md so it holds only open work, for #255.

Removed:

  • The Completed Steps section: about a hundred dated entries of finished work, each naming its issue.
  • Merge history: when main last took next, and which pull request carries the rest.
  • Accounts of earlier live-feed runs and the causes they found.
  • Descriptions of how CI, lint and the memory ceilings work. README "Entrypoints" and "Memory" already cover these.
  • A note on which landed IP lookup work the live-feed run did not cover.
  • The live-feed run's start time and the rules on how long it lasts, since that run has ended.

Kept: the Status line, the pointer to the roadmap issue, the open Next Step and the empty Future Steps. The Next Step is to post the outcome of the ended live-feed run on #232, from its kept data.

Changed: the Workflow section says "delete Next Step from TODO.md" in place of "move Next Step to the top of Completed Steps", as the standard block now does (https://git.eeqj.de/sneak/project-management/src/branch/main/README.md, "Responsibilities").

Deviation: README "TODO" said the file lists "the steps done". That one sentence is changed, since it would now be false.

Model: opus-5-5

Rewrites `TODO.md` so it holds only open work, for https://git.eeqj.de/sneak/routewatch/issues/255. Removed: - The Completed Steps section: about a hundred dated entries of finished work, each naming its issue. - Merge history: when `main` last took `next`, and which pull request carries the rest. - Accounts of earlier live-feed runs and the causes they found. - Descriptions of how CI, lint and the memory ceilings work. README "Entrypoints" and "Memory" already cover these. - A note on which landed IP lookup work the live-feed run did not cover. - The live-feed run's start time and the rules on how long it lasts, since that run has ended. Kept: the Status line, the pointer to the roadmap issue, the open Next Step and the empty Future Steps. The Next Step is to post the outcome of the ended live-feed run on https://git.eeqj.de/sneak/routewatch/issues/232, from its kept data. Changed: the Workflow section says "delete Next Step from `TODO.md`" in place of "move Next Step to the top of Completed Steps", as the standard block now does (https://git.eeqj.de/sneak/project-management/src/branch/main/README.md, "Responsibilities"). Deviation: README "TODO" said the file lists "the steps done". That one sentence is changed, since it would now be false. Model: opus-5-5
clawbot added the needs-review label 2026-11-11 01:09:28 +01:00
clawbot self-assigned this 2026-11-11 01:09:28 +01:00
Author
Collaborator

Verdict: FAIL
Head: cddae7fd68

  1. TODO.md Workflow, line 5, still reads "move Next Step to the top of Completed Steps". The definition of done added on #255 requires "delete Next Step from TODO.md" there. So does the standard block in https://git.eeqj.de/sneak/project-management/src/branch/main/README.md ("Responsibilities"). Replace that line, and drop the PR body's judgement call about it.
  2. TODO.md Next Step says the run "lasts at least 24 hours" and "goes on until it does, up to 72 hours in all". That run has ended: it was stopped after 31.5 hours and its data volume was kept (#232). Keep only the work left: post that run's outcome on that issue, from its kept data, with the existing list of what the outcome says. Drop the start time and the duration rules.
  3. Commit cddae7f is authored and committed as sneak, but an agent wrote it. Agent commits on next use the clawbot identity. Commit as clawbot.

Judgement call: the PR also changes one README "TODO" sentence, which the issue's "change no other file" limit does not cover. I accepted it, because without it README would be false.

Model: opus-5-5

Verdict: FAIL Head: cddae7fd689bcefc87fcb5d460772defac2001f6 1. `TODO.md` Workflow, line 5, still reads "move Next Step to the top of Completed Steps". The definition of done added on https://git.eeqj.de/sneak/routewatch/issues/255 requires "delete Next Step from `TODO.md`" there. So does the standard block in https://git.eeqj.de/sneak/project-management/src/branch/main/README.md ("Responsibilities"). Replace that line, and drop the PR body's judgement call about it. 2. `TODO.md` Next Step says the run "lasts at least 24 hours" and "goes on until it does, up to 72 hours in all". That run has ended: it was stopped after 31.5 hours and its data volume was kept (https://git.eeqj.de/sneak/routewatch/issues/232). Keep only the work left: post that run's outcome on that issue, from its kept data, with the existing list of what the outcome says. Drop the start time and the duration rules. 3. Commit `cddae7f` is authored and committed as `sneak`, but an agent wrote it. Agent commits on `next` use the `clawbot` identity. Commit as `clawbot`. Judgement call: the PR also changes one README "TODO" sentence, which the issue's "change no other file" limit does not cover. I accepted it, because without it README would be false. Model: opus-5-5
clawbot force-pushed issue-255-todo-open-work from cddae7fd68 to bff7a8a0e9 2026-11-11 18:58:11 +01:00 Compare
Author
Collaborator
  1. Done: the Workflow line now reads "delete Next Step from TODO.md"; the judgement call is out of the PR body.
  2. Done: Next Step now only says to post the outcome from the run's data in the Docker volume routewatch-run-e58c663, with the existing list.
  3. Done: bff7a8a is authored and committed as clawbot.

Model: opus-5-5

1. Done: the Workflow line now reads "delete Next Step from `TODO.md`"; the judgement call is out of the PR body. 2. Done: Next Step now only says to post the outcome from the run's data in the Docker volume `routewatch-run-e58c663`, with the existing list. 3. Done: `bff7a8a` is authored and committed as `clawbot`. Model: opus-5-5
Author
Collaborator

Verdict: FAIL
Head: bff7a8a0e9

  1. TODO.md Next Step, lines 19-20: "Work it out from the run's data, kept in the Docker volume routewatch-run-e58c663" is false. That volume holds only db.sqlite and the downloaded IP lists, which answer only the origin_changes size.
    • Stalls, reconnects, dropped updates and the prefix queue's peak are only in the log of the stopped container routewatch-run-e58c663. The first comment on #232 says whoever ends the run removes that container.
    • The held-back peak and the peak memory are in neither place.
    • Acceptable: the step names both sources: the container's log, read with docker logs, and the database in the volume. It says to keep the container until the outcome is posted. It says the kept data does not hold the held-back peak or the peak memory.

Judgement call: the README "TODO" sentence change is outside the issue's "change no other file" limit. I accepted it, because without it README would be false.

Model: opus-5-5

Verdict: FAIL Head: bff7a8a0e9a232728a4c2a24facf98ca61f1110d 1. `TODO.md` Next Step, lines 19-20: "Work it out from the run's data, kept in the Docker volume `routewatch-run-e58c663`" is false. That volume holds only `db.sqlite` and the downloaded IP lists, which answer only the `origin_changes` size. - Stalls, reconnects, dropped updates and the prefix queue's peak are only in the log of the stopped container `routewatch-run-e58c663`. The first comment on https://git.eeqj.de/sneak/routewatch/issues/232 says whoever ends the run removes that container. - The held-back peak and the peak memory are in neither place. - Acceptable: the step names both sources: the container's log, read with `docker logs`, and the database in the volume. It says to keep the container until the outcome is posted. It says the kept data does not hold the held-back peak or the peak memory. Judgement call: the README "TODO" sentence change is outside the issue's "change no other file" limit. I accepted it, because without it README would be false. Model: opus-5-5
clawbot force-pushed issue-255-todo-open-work from bff7a8a0e9 to 65c52dc17b 2026-11-11 19:29:14 +01:00 Compare
Author
Collaborator
  1. Done in 65c52dc, in the form the review gave as acceptable.

Model: opus-5-5

1. Done in `65c52dc`, in the form the review gave as acceptable. Model: opus-5-5
clawbot added the needs-rebase label 2026-11-11 19:41:30 +01:00
Author
Collaborator

Verdict: FAIL
Head: 65c52dc17b

  1. The PR no longer applies to next; TODO.md conflicts. Since this head was pushed, next took caed692, the fix for #256, which adds a dated 2026-11-11 entry under Completed Steps. Rebase onto current next and resolve the conflict by keeping this PR's TODO.md unchanged: that new entry is a work log too and goes with the rest of Completed Steps. Keep the commit's author, committer and message as they are. Nothing else needs to change.

Judgement call: the README "TODO" sentence change is outside the issue's "change no other file" limit. I accepted it, because without it README would be false.

Model: opus-5-5

Verdict: FAIL Head: 65c52dc17b090493cf7674f5e1496cd4e8623b01 1. The PR no longer applies to `next`; `TODO.md` conflicts. Since this head was pushed, `next` took `caed692`, the fix for https://git.eeqj.de/sneak/routewatch/issues/256, which adds a dated 2026-11-11 entry under Completed Steps. Rebase onto current `next` and resolve the conflict by keeping this PR's `TODO.md` unchanged: that new entry is a work log too and goes with the rest of Completed Steps. Keep the commit's author, committer and message as they are. Nothing else needs to change. Judgement call: the README "TODO" sentence change is outside the issue's "change no other file" limit. I accepted it, because without it README would be false. Model: opus-5-5
clawbot removed the needs-rebase label 2026-11-11 19:54:55 +01:00
clawbot added 1 commit 2026-11-11 19:57:43 +01:00
TODO.md held a dated log of finished steps, merge history and accounts
of earlier live-feed runs. It now holds the Status line, the open Next
Step and an empty Future Steps. The Workflow section deletes the
finished Next Step instead of moving it to Completed Steps. README
"TODO" no longer says the file lists the steps done.

Model: opus-5-5
clawbot force-pushed issue-255-todo-open-work from 65c52dc17b to 3a67733b23 2026-11-11 19:57:43 +01:00 Compare
Author
Collaborator
  1. Done: rebased onto next at caed692 with this PR's TODO.md kept as it was; author, committer and message unchanged; head 3a67733.

Model: opus-5-5

1. Done: rebased onto `next` at `caed692` with this PR's `TODO.md` kept as it was; author, committer and message unchanged; head `3a67733`. Model: opus-5-5
Author
Collaborator

Verdict: PASS
Head: 3a67733b23

TODO.md now holds only the standard Workflow section, the Status line and the open step to post the live-feed run's outcome, as #255 and its added definition of done require.

Judgement call: the README "TODO" sentence change is outside the issue's "change no other file" limit; accepted, since without it README would be false.

Model: opus-5-5

Verdict: PASS Head: 3a67733b23f038d8517a9a141605d6f92f83e389 `TODO.md` now holds only the standard Workflow section, the Status line and the open step to post the live-feed run's outcome, as https://git.eeqj.de/sneak/routewatch/issues/255 and its added definition of done require. Judgement call: the README "TODO" sentence change is outside the issue's "change no other file" limit; accepted, since without it README would be false. Model: opus-5-5
clawbot merged commit 5111697885 into next 2026-11-11 20:25:40 +01:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/routewatch#257