[integrations] Support direct OpenAI API provider#289
Conversation
|
Hey @DavidSlayback — welcome to Open Brain Source! 👋 Thanks for submitting your first PR. The automated review will run shortly and check things like metadata, folder structure, and README completeness. If anything needs fixing, the review comment will tell you exactly what. Once the automated checks pass, a human admin will review for quality and clarity. Expect a response within a few days. If you have questions, check out CONTRIBUTING.md or open an issue. |
|
Thanks for the contribution, and welcome. The core feature — a backwards-compatible direct OpenAI provider path alongside OpenRouter — is a sensible addition, and keeping OpenRouter the default means existing deployments are unaffected. The main issue is scope. This PR also bundles a change to Recommend author refresh — splitting the workflow-file change into its own PR would keep #289 scoped to the provider feature plus its docs, make each piece easier to review, and avoid a collision with #308. The binary — Alan (community reviewer; non-binding) |
Contribution Type
/recipes)/schemas)/dashboards)/integrations)/skills)What does this do?
Adds a backwards-compatible direct OpenAI provider path for the base MCP server, the Open Brain REST gateway, and the Agent Memory API. Existing OpenRouter deployments keep working by default, while users can now set
OPENAI_API_KEYwithout routing through OpenRouter.The docs and credential tracker now explain OpenRouter default, OpenAI direct, and generic OpenAI-compatible endpoint configuration. The PR also includes a narrow workflow syntax fix so the OB1 PR Gate can start on forked PRs; it does not change
CONTRIBUTING.mdor loosen the gate scope rules.Requirements
OPENROUTER_API_KEY,OPENAI_API_KEY, orAI_API_KEYwithAI_API_BASE_URLVerification
npx --yes deno-bin check --config server/deno.json server/index.tsnpx --yes deno-bin check --config integrations/open-brain-rest/deno.json integrations/open-brain-rest/index.tsnpx --yes deno-bin check --config integrations/agent-memory-api/deno.json integrations/agent-memory-api/index.tsnpx --yes markdownlint-cli2 --config .github/.markdownlint.jsonc "**/*.md" "#node_modules"check-jsonschema --schemafile .github/metadata.schema.json dashboards/open-brain-dashboard-next/metadata.json integrations/open-brain-rest/metadata.json integrations/agent-memory-api/metadata.json skills/openclaw-agent-memory/metadata.jsongit diff --check origin/main...HEADdocs/open-brain-credential-tracker.xlsxI did not run a live Open Brain/Supabase smoke test because I do not have a project configured in this checkout.
CI Note
This PR intentionally touches the core MCP server (
server/index.ts) to add direct OpenAI support. The existing OB1 PR Gate scope rule is written for standalone community contributions and may still require maintainer review/override for core-maintenance changes.Checklist
metadata.jsonchanges are valid JSON and pass the repo schema