Prepare gbrain keyless-mode fix for garrytan/gbrain#94 (--mode claude-code) - #9
Open
mnemonik-dev wants to merge 4 commits into
Open
Prepare gbrain keyless-mode fix for garrytan/gbrain#94 (--mode claude-code)#9mnemonik-dev wants to merge 4 commits into
mnemonik-dev wants to merge 4 commits into
Conversation
git am-ready patch adding 'gbrain init --mode claude-code': zero-API-key operation for Claude Code subscribers (PGLite + keyword/graph search, chat via the claude-cli OAuth recipe), plus two related upstream bug fixes. See work/gbrain-no-api-keys/README.md for scope and verification. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WQ7za53ZWxWqL2XAfSgTux
Includes the node_modules-symlink heads-up for the upstream maintainer. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WQ7za53ZWxWqL2XAfSgTux
The prep session's proxy blocks all GitHub API calls to garrytan/gbrain (cross-owner add_repo is a v1 limitation), so the final PR-open call is packaged as a curl script + JSON payload runnable with any token that can see the upstream repo. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WQ7za53ZWxWqL2XAfSgTux
Two one-shot sessions failed to post it; manual posting fallback. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WQ7za53ZWxWqL2XAfSgTux
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
2 participants
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.
What
Adds
work/gbrain-no-api-keys/— a prepared, verified fix for garrytan/gbrain#94 ("Plans for a Claude Code-native mode (no API keys, like gstack)?"):0001-claude-code-keyless-mode.patch—git am-ready patch (built against gbrainmaster@ddd66e1) addinggbrain init --mode claude-code: a fully working brain with zero provider API keys. PGLite engine, keyword + graph + title search (hybrid search's existing no-embedding-provider path), chat via the existingclaude-cliOAuth recipe. Includes two related upstream bug fixes (the sentinel-beats-explicit-flag upgrade-path bug, and the wrong subagent-key caveat forclaude-cli:*models), docs, and a 12-test suite.README.md— scope, before/after gap analysis, and full verification log (all touched suites green,tsc --noEmitclean, end-to-end keyless smoke: init → import → search → embed refusal → upgrade re-init).PR-DESCRIPTION.md— ready-to-paste PR body for the upstream submission.Why a patch instead of an upstream PR
This session's GitHub scope is limited to
mnemonik-dev/coding-fabric; the cross-owner add ofgarrytan/gbrainwas rejected (v1 limitation), so the upstream PR must be opened from a fork by an account with access. Everything needed is inPR-DESCRIPTION.md.Upstream inconvenience worth reporting
gbrain's
masterhas an accidentally committednode_modulessymlink pointing at/tmp/fleet/repo/node_modules(a path from someone's build machine). It's tracked despite.gitignore, so every fresh clone gets a dangling symlink andbun installmisbehaves untilrm node_modules && bun install. The upstream PR description includes this heads-up with the one-line fix (git rm --cached node_modules); it's deliberately kept out of the patch to stay scoped to #94.🤖 Generated with Claude Code
https://claude.ai/code/session_01WQ7za53ZWxWqL2XAfSgTux
Generated by Claude Code