Skip to content

prompt: retire two already-delivered PyAutoBrain bug prompts - #155

Merged
Jammy2211 merged 1 commit into
mainfrom
claude/automind-task-planning-ef00l2
Aug 8, 2026
Merged

prompt: retire two already-delivered PyAutoBrain bug prompts#155
Jammy2211 merged 1 commit into
mainfrom
claude/automind-task-planning-ef00l2

Conversation

@Jammy2211

Copy link
Copy Markdown
Collaborator

Both remaining prompts in draft/bug/pyautobrain/ were picked up for development and found already fixed on main. No code was written — this is a ledger backfill crediting the PRs that actually delivered them, and it clears two stale drafts that would otherwise send a future session to re-do finished work.

hygiene-gone-ref-countercomplete/2026/08/

Delivered by two PRs, neither of which the prompt knew about:

  • Defect 1 (the [gone] counter could never be non-zero) — PyAutoBrain#205, 7ad1e43, 2026-08-07. Switched to for-each-ref '%(upstream:track)' and added a regression test on a fixture with a genuinely gone upstream.
  • Defect 2 (the scan covered 9 repos, not the organism) — PyAutoBrain#200, 5cb1c73, 2026-08-05. Repo sets now derived from repos.yaml with a repos_sync.py coverage check. This is exactly the resolution the prompt asked someone to decide on.

#200 was already recorded as hygiene-coverage-drift; #205 had no record, so the ledger carried a gap.

version-drift-stamp-consistencycomplete/2026/07/

Delivered by PyAutoBrain#144, e26ab17, 2026-07-19 — option 2 of the three the prompt offered: flag stamps out of step with the sibling consensus rather than asserting equality to the release tag, plus both documentation updates the prompt required (version_drift.sh header, /wake_up step 5). No record existed.

The prompt's Status: still read "fix-implemented-on-branch — pending review/merge" three weeks after that branch merged. That stale status is what made it look open.

Verified, not assumed

The prompt for the [gone] counter specified its own control — "A green run on a tree with zero [gone] refs proves nothing — pick the tree first." That control was run: a throwaway repo with a real bare remote, a pushed-then-deleted branch and a fetch --prune, so the checkout genuinely carried a [gone] ref. On that same tree the old form counts 0 and the shipped form counts 1. Also confirmed no repo-name array literal survives in hygiene.sh, including in run_tidy and enumerate_condemn_candidates, the two the prompt named as sharing the limitation.

For version_drift: merge-base --is-ancestor against main, plus reading the shipped script and the /wake_up wording to confirm both doc requirements landed.

A trap worth keeping, recorded in the first record

Establishing "already fixed" required deepening the clone. Cloud-session checkouts are shallow, and a shallow PyAutoBrain makes git diff origin/main...<branch> fail with no merge base and makes git log origin/main..<branch> list commits that are on main. Both look like evidence of divergence and are artifacts of the graft boundary. This is the second time in this session that shallowness produced a confidently wrong reading of history.

lifecycle.py check: OK. lifecycle.py index --check: OK (937 records).

Note: the branch was reset from main and force-pushed with lease — it previously held only the two commits squash-merged as 8b33239 in #154.

🤖 Generated with Claude Code

https://claude.ai/code/session_016VnfCyAwgQdm1V3iJBtiga


Generated by Claude Code

Both prompts in draft/bug/pyautobrain/ were picked up for development and
found already fixed on main. No code was written; these are ledger backfills
crediting the PRs that actually delivered them.

hygiene-gone-ref-counter — PyAutoBrain#205 (7ad1e43) fixed the [gone] counter
2026-08-07; PyAutoBrain#200 (5cb1c73) derived the repo sets from the body map
2026-08-05. #200 was already recorded as hygiene-coverage-drift; #205 had no
record, so the ledger carried a gap.

version-drift-stamp-consistency — PyAutoBrain#144 (e26ab17) reframed the check
to stamp consistency 2026-07-19. No record existed. The prompt's Status still
read "pending review/merge" three weeks after that branch merged, which is what
made it look open.

Both verified before retiring rather than assumed: the [gone] control was run on
a purpose-built fixture with a genuinely gone upstream (old form 0, shipped form
1), and the version_drift fix was confirmed on main by merge-base ancestry plus
reading the shipped script and the /wake_up wording.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016VnfCyAwgQdm1V3iJBtiga
@Jammy2211
Jammy2211 merged commit b9023b0 into main Aug 8, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants