Add optional Gemini embeddings support#58
Conversation
|
Thanks for this contribution — and apologies for the slow triage. We did a full pass over the entire PR backlog. gbrain has moved fast, and the maintainer's larger "cathedral" rewrites have superseded a big share of community PRs: the AI gateway + recipes + user_provided_models system replaced almost all individual provider PRs; #1805 fixed the whole Postgres module-singleton class; #1542 unified the type taxonomy; #1657 the retrieval path; #1802 the doctor; and so on. We're closing this one in that cleanup — either the fix already landed on master, it duplicates another PR or merged change, or it's outside the current merge bar. Where a closed PR carried a genuinely valuable idea, we've recorded it in docs/designs/COMMUNITY_IDEAS.md so nothing good is lost (a few may graduate into TODOs). Please don't read the close as a judgment of the work — thank you for contributing. If you believe the underlying issue is still live on the latest master, reopen with a quick note and we'll take another look. 🙏 |
Summary
What changed
embedding_provider,embedding_model, andgemini_api_keysupport in config loadingopenaiorgemini)batchEmbedContentswithoutputDimensionality: 1536gbrain init --provider openai|gemini --key <api_key>supportWhy this is safe
Verification
bun test test/embedding-provider.test.ts test/config.test.ts test/cli.test.tsembed --stalecompleted successfully on a local PGLite brainNotes