Repository navigation
perf(artifact-delivery): spend GitHub quota once per Skill and commit - #493
Merged
Merged
Conversation
A spent GitHub quota is the only run failure production has shown, and launch traffic repeats the same Skills. - Commit and tree reads by SHA never change, so KV keeps them. - A ready build whose policy packed the same bytes is checked again from R2. #481 changed only checks, yet its bump reloaded every Skill. - A first run waits for an earlier build of the same Repository, commit and folder, then reuses it. The wait goes back as a new queue message, so it spends no retry.
…eft out #487 leaves large media out of a folder. R2 holds only the packed files, so a stored build that left a file out loads from GitHub. Every folder 2026-10-07.1 accepted packs the same files, so that policy joins the byte-compatible set.
# Conflicts: # layers/artifact-delivery/server/utils/github-source.ts
# Conflicts: # layers/artifact-delivery/server/utils/github-source.ts # layers/artifact-delivery/server/utils/queue.ts
1 of 6 tasks
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.
🔗 Linked issue
Related to #481, #484
❓ Type of change
📚 Description
A spent GitHub quota is the only run failure production has shown: 15
RATE_LIMITEDResolutions in 14 days. Launch traffic will repeat the same Skills, so a run should spend GitHub quota only when nothing it needs was read before.BYTE_COMPATIBLE_POLICY_VERSIONSlists earlier policies that packed the same bytes. A ready build under one of them is read back from R2 and checked under the current checks. It must also pass every rule a GitHub load applies today. fix(artifact-delivery): run a named Skill by its registry identity #481 changed only checks, yet its bump reloaded every Skill from GitHub. The set holds2026-10-07.1and2026-08-20.1: fix(artifact-delivery): run Skills that ship media or doc keys #487 leaves large media out only of folders those policies refused. A stored build that left a file out loads from GitHub, because R2 holds only the files it packed. A check version change under the current policy takes the re-check path too.GitHub requests per run, measured on the same fake Repository with the real source client. Each fresh build includes one tarball request.