Repository navigation
fix(server): cut background GitHub PR lookups (fork patch, board #15036) - #6
Open
MattRiddell wants to merge 1 commit into
Open
MattRiddell wants to merge 1 commit into
MattRiddell wants to merge 1 commit into
Conversation
…dotgg#15036) The 10-minute PR cache (393c055) did not cut t3-server's GraphQL reads: 2026-10-12 05-10Z still ran 106-147 calls an hour against a 283/day target. - The periodic thread PR pass and the settlement sweep run every 10 minutes, not every minute, so expired branch lookups fall due together and share GraphQL documents. Settlement lookups run 25 at a time to fill a document. - A finished run or captured checkpoint reads the cached branch answer; it no longer forces a fresh read or drops a cached "no pull request" answer. A pull request opened during a run shows within the cache's 10 minutes (Matt, q3440: PR badges can lag up to 10 min). User refreshes still bypass. - Provider-native subagents whose root thread is archived, settled or gone get no branch lookup (493 of 547 settlement candidates; 231 of 267 keys per pass). They still settle by age. - `git remote -v` lines of a partial clone end with `[blob:none]`; the remote regex rejected them, so the checkout resolved to provider "unknown" and its PR badges and merge settlement stopped. The three copies accept the suffix. Tests: GitManager, ThreadPullRequestService, ThreadSettlementService, RepositoryIdentityResolver, GitVcsDriver, GitVcsDriverCore: 362 passed (GIT_CONFIG_GLOBAL=/dev/null; eniac's global core.hooksPath fails 5 hook tests on the base too). New tests fail on the base: 3 of 3. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
github-actions
Bot
added
vouch:trusted
This branch has not been deployed
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
1 participant
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.
Fork patch for board pingdotgg#15036. It stacks on
build/2819-pr10m-fix16214, the live install's source: the 10-min PR cache (#4) plus the grok fix (#5).Why
The 10-min PR cache did not cut t3-server's GitHub GraphQL reads. On 2026-10-12, 05–10Z (Panama 00:00–05:00), it still ran 106–147 calls an hour. The target is ≤283 a day: −80% against the 2026-09-28 baseline of 1,414 a day.
What
refresh: true;refreshMissingPullRequestdropped the cached "no PR" answergit remote -vin a partial clone printsorigin <url> (fetch) [blob:none]; the regex rejected it[filter]suffix. Since 2026-10-12 ~10Z both product checkouts resolved to provider "unknown", so their PR badges and merge settlement stopped. Upstream main has the same bug.Tests
GIT_CONFIG_GLOBAL=/dev/null.core.hooksPathcauses it.tsc --noEmit: clean.vp linton changed files: exit 0, no new warnings.Install
Install is controller-only:
t3-patched-buildstages a new directory, then Matt restarts T3. The install packet is on board pingdotgg#15036.🤖 Generated with Claude Code