Repository navigation
chore: pin TauCetiReview at 7d59664, which makes the shared prompt cache hit - #263
Merged
Merged
Conversation
…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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#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 as7d59664, 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:Changes: every TauCetiReview pin,
ec743f1→7d59664, all at once as in #262:tauceti.toml([tools.review] ref), theuses:lines andreview_refin.github/workflows/auto-merge.ymlandmerge-sweep.yml, and.github/actions/checkout-merge-policy/action.yml. Between the two commits, onlyrunner/reviewers.pyand 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 withoutTAUCETI_CONFreadstauceti.tomlfrom the checkout it runs in, so that checkout must be up to date.Roadmap: none
🤖 Generated with Claude Code