Skip to content

chore: pin TauCetiReview at 7d59664, which makes the shared prompt cache hit - #263

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

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

Conversation

@RemyDegenne

Copy link
Copy Markdown
Contributor

#262 pinned TauCetiReview at ec743f1, which sends the shared protocol and the pull request as a cached prompt block, so that rubrics after the first in a round read the PR from the cache. In that version, each rubric's Claude Code ran with a different HOME, which changed Claude Code's own system prompt between rubrics, so the cache was not reused. TauCetiProject/TauCetiReview#227, merged as 7d59664, gives all the Claude attempts of a review one HOME.

With 7d59664, the review runs archived in TauCetiData since the merge (4 rounds on TauCeti PRs, 37 Claude rubric runs) show the cache working:

  • First rubric of each round: it writes the shared part, 8–14k tokens.
  • Later rubrics: the 8 whose first request can be isolated all read the shared part from the cache and write only their own 1–2k tokens. Of the 33 later rubrics, 21 certainly hit the cache, 12 can't be told from the records, and none is seen missing.
  • Cost: the 4 rounds cost $4.20, against an estimated $7.55–8.65 with the old layout.

Changes: every TauCetiReview pin, ec743f1 → 7d59664, all at once as in #262: tauceti.toml ([tools.review] ref), the uses: lines and review_ref in .github/workflows/auto-merge.yml and merge-sweep.yml, and .github/actions/checkout-merge-policy/action.yml. Between the two commits, only runner/reviewers.py and a test change, so the reusable workflows and the rubrics (and their fingerprint) are the same.

Like #262, this PR changes .github/, so it may need an admin merge. After the merge, restart running workers so that they pick up the new pin. A worker started without TAUCETI_CONF reads tauceti.toml from the checkout it runs in, so that checkout must be up to date.

Roadmap: none

🤖 Generated with Claude Code

…che hit

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>
@lmlexplore-automation lmlexplore-automation Bot added awaiting-CI CI has not yet reported on the latest commit awaiting-review CI is green; waiting for review verdicts and removed awaiting-CI CI has not yet reported on the latest commit labels Nov 10, 2026
@RemyDegenne
RemyDegenne merged commit 22906e3 into main Nov 10, 2026
14 of 16 checks passed
@lmlexplore-automation lmlexplore-automation Bot removed the awaiting-review CI is green; waiting for review verdicts label Nov 10, 2026
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