Drop TODO.md's stale docs-only cache caveat (closes #421) #424

Merged
clawbot merged 1 commits from issue-421-todo-cache-caveat into next 2026-10-02 10:14:41 +02:00
Collaborator

Removes the Status paragraph in TODO.md that said a docs-only commit replays from the layer cache, so that a green check on one shows a replay rather than a run.

That stopped being true with #410. The check workflow now writes the hash of the commit being checked into .ci-fingerprint, and the build context carries .git (.dockerignore leaves out only .git/config). Every commit, docs-only ones included, therefore invalidates the COPY . . layer of both check stages, and the checks run. The README's "CI gate honesty" section already says this, so TODO.md has nothing left to add about reading a green check. The paragraph is removed rather than restated.

Nothing else in TODO.md changes.

Deviation: this issue branch edits TODO.md, whose Workflow section reserves the file to the manager; #421 directs the edit.
Judgement call: removed rather than corrected, because a corrected paragraph would only repeat the README.

Model: opus-5-5

Removes the Status paragraph in `TODO.md` that said a docs-only commit replays from the layer cache, so that a green check on one shows a replay rather than a run. That stopped being true with https://git.eeqj.de/sneak/webhooker/pulls/410. The `check` workflow now writes the hash of the commit being checked into `.ci-fingerprint`, and the build context carries `.git` (`.dockerignore` leaves out only `.git/config`). Every commit, docs-only ones included, therefore invalidates the `COPY . .` layer of both check stages, and the checks run. The README's "CI gate honesty" section already says this, so `TODO.md` has nothing left to add about reading a green check. The paragraph is removed rather than restated. Nothing else in `TODO.md` changes. Deviation: this issue branch edits `TODO.md`, whose Workflow section reserves the file to the manager; https://git.eeqj.de/sneak/webhooker/issues/421 directs the edit. Judgement call: removed rather than corrected, because a corrected paragraph would only repeat the README. Model: opus-5-5
clawbot added the needs-review label 2026-10-02 09:42:46 +02:00
clawbot self-assigned this 2026-10-02 09:42:46 +02:00
clawbot added 1 commit 2026-10-02 09:42:46 +02:00
The Status paragraph said a docs-only commit replays from the layer
cache. Since the check workflow writes the commit hash into
.ci-fingerprint and the build context carries .git, every commit
invalidates the COPY layer and the checks run. README's "CI gate
honesty" section already says so, so the paragraph is removed rather
than restated.

Model: opus-5-5
Author
Collaborator

Review passed.

Judgement call: the dated 2026-08-12 entry under Completed Steps in TODO.md (for #119) still says a docs-only commit replays from cache. It records what that change did at the time, and #421 says nothing else in TODO.md changes, so it is not held against this PR.

Model: opus-5-5

Review passed. Judgement call: the dated 2026-08-12 entry under Completed Steps in `TODO.md` (for https://git.eeqj.de/sneak/webhooker/issues/119) still says a docs-only commit replays from cache. It records what that change did at the time, and https://git.eeqj.de/sneak/webhooker/issues/421 says nothing else in `TODO.md` changes, so it is not held against this PR. Model: opus-5-5
clawbot merged commit b78abdc9da into next 2026-10-02 10:14:41 +02:00
clawbot deleted branch issue-421-todo-cache-caveat 2026-10-02 10:14:41 +02:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/webhooker#424