v0.33.2.1 docs: fork-PR workflow for garrytan-agents#992
Merged
Conversation
Fork PRs from non-collaborator accounts don't receive base-repo secrets on pull_request events, so CI jobs needing ANTHROPIC_API_KEY / OPENAI_API_KEY fail with empty-env auth errors. Document the move-branch-to-base-repo workflow as the narrow-scope alternative to adding the account as a collaborator or flipping the repo-wide fork-secret toggle. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Per user direction: ship as v0.33.2.1 instead of v0.33.3.1. 0.33.2.x is unclaimed in the queue (PR #934 holds 0.33.3.0). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
brandonlipman
added a commit
to brandonlipman/gbrain
that referenced
this pull request
May 29, 2026
* upstream/master: v0.35.1.0: embedder shootout prereqs (pricing + gateway export + --resume-from) (garrytan#1055) v0.35.0.0 feat: ZeroEntropy zembed-1 + zerank-2 reranker (garrytan#1008) v0.34.4.0 fix(embed): cursor-paginated --stale hardening wave (D2/D3/D4/D6/D7/D8 + regression test) (garrytan#991) v0.34.3.0 fix: supervisor treats code=0 watchdog exits as crashes (garrytan#1003) v0.34.2.0 fix(import): path-based checkpoint resume — kills parallel-drop + failed-file-skip + sort-flip bugs (garrytan#988) v0.34.1.0 fix(mcp): MCP fix wave — source-isolation P0 + PKCE DCR + federated_read + 3 more (garrytan#996) v0.34.0.0 feat: Cathedral III — recursive code intelligence + Leiden clusters + eval gate (garrytan#994) v0.33.3.0 feat(v0.33.3): code intelligence MCP foundation (v0.34 W0a-c + W3) (garrytan#934) v0.33.2.1 docs: fork-PR workflow for garrytan-agents (garrytan#992) fix(sync): raise maxBuffer to 100 MiB to prevent silent ENOBUFS crash (garrytan#982) v0.33.2.0 feat(search-lite): token budget + semantic query cache + intent weighting (garrytan#897) v0.33.1.1 fix: Voyage output_dimension + flexible-dim guard + OOM-cap rethrow (garrytan#962)
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.
Summary
Doc-only ship. CLAUDE.md gains a
## Checking out PRs from garrytan-agentssection between "Community PR wave process" and "Skill routing" documenting the move-branch-to-base-repo workflow for fork PRs that need base-repo CI secrets.Why this matters.
garrytan-agentsis the AI-authored PR account and is NOT a collaborator. Its PRs live in a fork, and GitHub'spull_requestevent default doesn't ship base-repo secrets to forks — so CI jobs needingANTHROPIC_API_KEYorOPENAI_API_KEYquietly fail with empty-env auth errors. The doc names the narrow-scope fix (move the branch) as the alternative to adding the account as a collaborator or flipping the repo-wide fork-secret toggle (both broader).Four-step recipe in the new section:
gh pr checkout <N>→git push origin HEAD:<branch>→gh pr close <N> --comment ...→gh pr create --base master --head <branch>, preserving original title and body verbatim.llms-full.txtregenerated bybun run build:llmsso the committed doc bundle matches live CLAUDE.md. Required after every CLAUDE.md edit per the project's own docs; pinned bytest/build-llms.test.tson CI shard 1.Test Coverage
No new application code paths to audit — docs-only diff. The committed
llms-full.txtregeneration is gated bytest/build-llms.test.ts(7 pass, 64 expects), which runs in CI shard 1.Pre-Landing Review
No code paths in the diff. Reviewed the CLAUDE.md addition for accuracy: the four-step recipe matches the workflow that landed PR #962 successfully (
100yenadmin/electricsheephqfork → base-repo branch → secret access). No issues found.Plan Completion
No plan file detected — this was an ad-hoc fix off the back of recently shipping #962 / fork-PR friction.
TODOS
No TODO items completed in this PR. The TODOS.md
v0.33.xfollow-ups infunctional-area-resolverand embedding-provider sections are unrelated.Documentation
CLAUDE.md is the documentation in this PR. README.md / docs/ unchanged.
Test plan
bun test test/build-llms.test.ts— 7 pass, 0 fail (canary against committed-bundle drift)bun run typecheck— clean (no .ts source changed; runs as sanity)VERSION/package.json/ CHANGELOG header all show0.33.3.1🤖 Generated with Claude Code