Skip to content

fix: ignore placeholder API keys for optional tools#264

Open
Zavianx wants to merge 1 commit into
virattt:mainfrom
Zavianx:fix/ignore-placeholder-api-keys
Open

fix: ignore placeholder API keys for optional tools#264
Zavianx wants to merge 1 commit into
virattt:mainfrom
Zavianx:fix/ignore-placeholder-api-keys

Conversation

@Zavianx
Copy link
Copy Markdown

@Zavianx Zavianx commented May 12, 2026

Summary

  • Do not register web_search providers when their env vars still contain env.example-style your-* placeholder values
  • Do not register or invoke x_search with a placeholder X_BEARER_TOKEN
  • Avoid selecting OpenAI/Gemini memory embeddings from placeholder API keys
  • Add regression coverage for optional tool registration and memory embedding provider selection

Testing

  • bun test src/tools/registry.test.ts src/memory/embeddings.test.ts
  • bun run typecheck
  • bun test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant