Skip to content

chore: pin TauCetiReview at ec743f1, which caches the PR across rubrics - #262

Merged
RemyDegenne merged 1 commit into
mainfrom
chore/pin-tauceti-review-ec743f1
Nov 10, 2026
Merged

RemyDegenne merged 1 commit into
mainfrom
chore/pin-tauceti-review-ec743f1

Conversation

@RemyDegenne

Copy link
Copy Markdown
Contributor

Reviews of LMLExplore PRs run TauCetiReview at the commit pinned in tauceti.toml, still 32ce206. TauCetiProject/TauCetiReview#224, merged as ec743f1, sends the shared protocol and the pull request first, as a separately cached block. The rubrics after the first in a round then read the PR from the prompt cache instead of each writing its own copy.

In the reviews one worker ran on LMLExplore from 30 October to 9 November (5,168 rubric runs on 228 PRs), cache writes were $1,331 of the $1,579 total, at API prices. The first request of every rubric read only Claude Code's 3,223-token system prompt from the cache.

Changes: every TauCetiReview pin, 32ce206 → ec743f1, all at once as .github/actions/checkout-merge-policy/action.yml asks:

  • tauceti.toml: [tools.review] ref, which the workers use to run reviews;
  • .github/workflows/auto-merge.yml and merge-sweep.yml: the uses: lines and review_ref;
  • .github/actions/checkout-merge-policy/action.yml: the merge-policy checkout.

TauCetiTools' project template fills these values from tauceti.toml, so a later re-render keeps them. The comments refer to scripts/test_workflow_pins.py to check that the pins agree, but that script isn't in the repository; I checked by hand that all six agree.

Between the two commits (40 commits):

  • Reusable workflows: merge-only.yml and merge-sweep.yml keep the same inputs and secrets. Their only change is runs-on, and only for TauCetiProject/TauCeti.
  • Rubric text: unchanged. Only rubrics/README.md differs, which changes the rubrics fingerprint. Approvals stay tied to the head commit, so open PRs are not re-reviewed. An approval is not carried over to a rebased head with an unchanged patch, so those rubrics run once more.

The new prompt layout needs Claude Code 2.1.242 or later on the machine running the reviews. With an older CLI, reviews keep the old layout and work as before.

Like #109, this PR changes a pin and .github/, so it may need an admin merge. After the merge, restart running workers so that they pick up the new pin.

Roadmap: none

🤖 Generated with Claude Code

TauCetiProject/TauCetiReview#224 sends the shared protocol and the pull
request as a separately cached prompt block, so the rubrics after the
first in a round read the PR from the prompt cache instead of writing it
again. Re-pin every TauCetiReview reference together: tauceti.toml, the
auto-merge and merge-sweep callers, and the merge-policy checkout.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@lmlexplore-automation lmlexplore-automation Bot added the awaiting-CI CI has not yet reported on the latest commit label Nov 10, 2026
@RemyDegenne
RemyDegenne merged commit 4d7bd80 into main Nov 10, 2026
10 of 12 checks passed
@lmlexplore-automation lmlexplore-automation Bot removed the awaiting-CI CI has not yet reported on the latest commit label Nov 10, 2026
RemyDegenne added a commit that referenced this pull request Nov 10, 2026
…che hit (#263)

TauCetiProject/TauCetiReview#227 gives all the Claude attempts of a
review one HOME, so Claude Code's system prompt no longer changes between
rubrics and the later rubrics of a round read the PR from the prompt
cache. Re-pin every TauCetiReview reference together, as in #262.

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
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.

1 participant