feat: Add SAP AI Core provider support by jerome-benoit · Pull Request #5023 · anomalyco/opencode

jerome-benoit · 2025-12-03T18:53:14Z

Add SAP AI Core provider integration using @mymediset/sap-ai-provider package. Enables access to 40+ models via SAP BTP. Configuration via SAP_AI_SERVICE_KEY environment variable.

packages/opencode/src/provider/provider.ts

packages/web/src/content/docs/providers.mdx

packages/opencode/src/provider/provider.ts

jerome-benoit

[Duplicate - please ignore, see next comment]

jerome-benoit

Thanks for the automated reviews! I've addressed all comments in commit 85dc856:

#2586265498 (Deployment ID): The ID d65d81e7c077e583 comes from official @mymediset/sap-ai-provider docs as a 'general-purpose deployment'. Updated docs to clarify these are configurable defaults.

#2586265951 (Documentation): ✅ Fixed - docs now accurately describe hardcoded defaults

#2586266385 (Unused parameter): ✅ Fixed - removed unused provider parameter

#2586266865 (Ternary): ✅ Fixed - simplified ternary expression

wsx999outlook · 2025-12-03T19:43:39Z

Good to go

packages/opencode/src/provider/provider.ts

Copilot

Pull request overview

This PR adds SAP AI Core as a new provider, enabling access to 40+ models from multiple vendors (OpenAI, Anthropic, Google, Amazon, Meta, Mistral, AI21) through SAP's BTP platform. The integration uses the @mymediset/sap-ai-provider package and supports configuration via environment variables or the /connect command.

Adds SAP AI Core provider implementation with support for service key authentication
Includes comprehensive documentation with setup instructions and environment variable configuration
Adds @mymediset/sap-ai-provider@1.3.1 package dependency

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

File	Description
packages/web/src/content/docs/providers.mdx	Adds user-facing documentation for SAP AI Core setup and configuration
packages/opencode/src/provider/provider.ts	Implements SAP AI Core provider loader with authentication and configuration handling
packages/opencode/package.json	Adds dependency on @mymediset/sap-ai-provider package
bun.lock	Updates lock file with new dependency and its transitive dependencies

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

packages/opencode/src/provider/provider.ts

jerome-benoit · 2025-12-03T20:40:44Z

Addressed in commit `25b413b`

#2586418762 - Added SAP AI to BUNDLED_PROVIDERS and updated type to support async providers

#2586418775 - Removed custom getModel that recreated the provider on every call. Now uses default sdk.languageModel(modelID) pattern with proper caching.

jerome-benoit · 2025-12-03T21:00:09Z

⚠️ Blocked by models.dev PR

This PR requires anomalyco/models.dev#466 to be merged first.

Why: The custom loader depends on database['sap-ai-core'] which doesn't exist yet. Without the models.dev entry, mergeProvider() returns early at line 355.

Merge order: models.dev anomalyco/models.dev#466 → this PR

packages/opencode/src/provider/provider.ts

rekram1-node · 2025-12-03T21:20:37Z

packages/opencode/src/provider/provider.ts

+    "sap-ai-core": async () => {
+      const auth = await Auth.get("sap-ai-core")
+      const serviceKey = process.env["SAP_AI_SERVICE_KEY"] || (auth?.type === "api" ? auth.key : undefined)
+      const deploymentId = process.env["SAP_AI_DEPLOYMENT_ID"] || "d65d81e7c077e583"
+      const resourceGroup = process.env["SAP_AI_RESOURCE_GROUP"] || "default"
+


does your provider not automatically respect these env vars?

Unfortunately, no, createSAPAIProvider needs the options to be passed explicitly and being compatible with documented env vars on the vercel sdk package is a good practice.

wsx999outlook · 2025-12-03T21:44:31Z

It’s good to go

wsx999outlook · 2025-12-03T23:31:08Z

Looks good

rekram1-node · 2025-12-04T03:50:56Z

Okay this looks good to me if we can resolve the conflicts

packages/opencode/src/provider/provider.ts

packages/opencode/package.json

Add integration for SAP AI Core using @mymediset/sap-ai-provider, enabling access to 40+ AI models (GPT-4o, Claude, Gemini, Amazon Nova, etc.) through SAP BTP. Features: - OAuth2 authentication with service key - Support for 40+ models via SAP AI Core Orchestration API - Configurable deployment ID and resource group - Full Vercel AI SDK compatibility (streaming, tools, multi-modal) - Environment variable configuration support Configuration: - SAP_AI_SERVICE_KEY: Service key from SAP BTP Cockpit - SAP_AI_DEPLOYMENT_ID: Optional custom deployment (default: d65d81e7c077e583) - SAP_AI_RESOURCE_GROUP: Optional resource group (default: default) Documentation added to providers.mdx with setup instructions.

- Remove unused provider parameter - Simplify ternary in options - Fix misleading documentation about default values

Add support for async create...Provider functions. The overhead is negligible sync the provider is created only once and cached. Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

* fix: ensure checkUpgrade sets `init:` (anomalyco#5040) * chore: format code * ci: use blacksmith runners in review workflow too (anomalyco#5042) * zen: make session provider sticky * ci: only maintainer can trigger * ci: cleaner * ignore: update download stats 2025-12-04 * fix(tui): cursor color * Preserve prompt input when creating new session (anomalyco#4993) * fix: model not being passed correctly to tool * tweak: bash tool messages regarding timeouts and truncation more clear for agent (anomalyco#5066) * tui: fix /new slash command being persisted in prompt input * add optional prompt Input to Github Action (anomalyco#4828) Co-authored-by: Github Action <action@github.com> Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com> Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com> * bump: builtin plugin versions * feat: Add SAP AI Core provider support (anomalyco#5023) Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org> * core: ensure model npm package falls back to dev models config when not explicitly defined * docs: add CodeCompanion.nvim integration instructions (anomalyco#5079) * tweak: bash tool description re commit stuff * core: add test for custom model npm package inheritance * feat(desktop): terminal pane (anomalyco#5081) Co-authored-by: Github Action <action@github.com> Co-authored-by: Dax Raad <d@ironbay.co> * Revert "feat(desktop): terminal pane (anomalyco#5081)" This reverts commit d763c11. * release: v1.0.133 * sync: record last synced tag v1.0.133 --------- Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org> Co-authored-by: YeonGyu-Kim <code.yeon.gyu@gmail.com> Co-authored-by: GitHub Action <action@github.com> Co-authored-by: wsx99outlook <247713593+wsx99outlook@users.noreply.github.com> Co-authored-by: Frank <frank@sst.dev> Co-authored-by: Aiden Cline <aidenpcline@gmail.com> Co-authored-by: Adam <2363879+adamdotdevin@users.noreply.github.com> Co-authored-by: Daniel Gray <DanielFGray@gmail.com> Co-authored-by: Shantur Rathore <i@shantur.com> Co-authored-by: Daniel Polito <danielbpolito@gmail.com> Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com> Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com> Co-authored-by: Jérôme Benoit <jerome.benoit@piment-noir.org> Co-authored-by: Dax Raad <d@ironbay.co> Co-authored-by: Cason Adams <casonadams@gmail.com> Co-authored-by: opencode <opencode@sst.dev>

* fix: ensure checkUpgrade sets `init:` (anomalyco#5040) * chore: format code * ci: use blacksmith runners in review workflow too (anomalyco#5042) * zen: make session provider sticky * ci: only maintainer can trigger * ci: cleaner * ignore: update download stats 2025-12-04 * fix(tui): cursor color * Preserve prompt input when creating new session (anomalyco#4993) * fix: model not being passed correctly to tool * tweak: bash tool messages regarding timeouts and truncation more clear for agent (anomalyco#5066) * tui: fix /new slash command being persisted in prompt input * add optional prompt Input to Github Action (anomalyco#4828) Co-authored-by: Github Action <action@github.com> Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com> Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com> * bump: builtin plugin versions * feat: Add SAP AI Core provider support (anomalyco#5023) Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org> * core: ensure model npm package falls back to dev models config when not explicitly defined * docs: add CodeCompanion.nvim integration instructions (anomalyco#5079) * tweak: bash tool description re commit stuff * core: add test for custom model npm package inheritance * feat(desktop): terminal pane (anomalyco#5081) Co-authored-by: Github Action <action@github.com> Co-authored-by: Dax Raad <d@ironbay.co> * Revert "feat(desktop): terminal pane (anomalyco#5081)" This reverts commit d763c11. * release: v1.0.133 * Update Nix flake.lock and hashes * Reapply "feat(desktop): terminal pane (anomalyco#5081)" This reverts commit f9dcd97. * chore: format code * Update Nix flake.lock and hashes * zen: fix byok * ignore: fix provider credentials query for BYOK Provider credentials field was being selected from ProviderTable even when the table wasn't joined (when byokProvider was undefined). Now the join is conditional - when byokProvider exists, we join and get the credentials; when it doesn't, the join condition is always false so provider remains null. * docs: llama.cpp docs: `limit` moved under model (anomalyco#5089) Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com> Co-authored-by: GitHub Action <action@github.com> * fix: add getModel to SAP AI Core provider for correct SDK initialization (anomalyco#5086) Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org> * fix pty builds * fix * increase default scroll speed * ci: review * fix: ensure that vcs is still set to git even if no commits in repo * ignore: rm slop commnand (only for opencode repo this isnt shipping) * chore: format code * ignore: cmd tweak * tui: wrap dialog option descriptions (anomalyco#5083) * ci: keybinds * fix: ensure projects that go from having no commits to having commits have sessions migrated (anomalyco#5105) Co-authored-by: GitHub Action <action@github.com> * ignore: tweak * ignore: update download stats 2025-12-05 * Zen: add codex max * do not use required pty for local dev * fix: anomalyco#5064 ProviderInitError github-copilot-enterprise (anomalyco#5123) * feat(theme): Vercel (anomalyco#5119) Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com> Co-authored-by: rekram1-node <rekram1-node@users.noreply.github.com> * fix(desktop): new session not selecting tab * fix(desktop): clone pty session on reconnect * add experimental.open_telemetry config option to enable OTEL spans (anomalyco#4978) Co-authored-by: noamzbr <noamzbr@users.noreply.github.com> Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com> Co-authored-by: rekram1-node <rekram1-node@users.noreply.github.com> * ignore: regen sdk * fix(desktop): prompting * fix(desktop): terminal cursor position * feat: add max steps for supervisor and sub-agents (anomalyco#4062) Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com> Co-authored-by: rekram1-node <rekram1-node@users.noreply.github.com> * tweak: bash tool improve output metadata for agent consumption, fix small timeout issue (anomalyco#5131) * OpenCode Desktop app (anomalyco#5044) Co-authored-by: Adam <2363879+adamdotdevin@users.noreply.github.com> * Update Nix flake.lock and hashes * release: v1.0.134 * sync: record last synced tag v1.0.134 * chore: format code --------- Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org> Co-authored-by: YeonGyu-Kim <code.yeon.gyu@gmail.com> Co-authored-by: GitHub Action <action@github.com> Co-authored-by: wsx99outlook <247713593+wsx99outlook@users.noreply.github.com> Co-authored-by: Frank <frank@sst.dev> Co-authored-by: Aiden Cline <aidenpcline@gmail.com> Co-authored-by: Adam <2363879+adamdotdevin@users.noreply.github.com> Co-authored-by: Daniel Gray <DanielFGray@gmail.com> Co-authored-by: Shantur Rathore <i@shantur.com> Co-authored-by: Daniel Polito <danielbpolito@gmail.com> Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com> Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com> Co-authored-by: Jérôme Benoit <jerome.benoit@piment-noir.org> Co-authored-by: Dax Raad <d@ironbay.co> Co-authored-by: Cason Adams <casonadams@gmail.com> Co-authored-by: opencode <opencode@sst.dev> Co-authored-by: opencode-agent[bot] <219766164+opencode-agent[bot]@users.noreply.github.com> Co-authored-by: ry2009 <134240944+ry2009@users.noreply.github.com> Co-authored-by: Julian Visser <12615757+justmejulian@users.noreply.github.com> Co-authored-by: Anthony Shew <anthonyshew@gmail.com> Co-authored-by: rekram1-node <rekram1-node@users.noreply.github.com> Co-authored-by: Noam Bressler <noamzbr@gmail.com> Co-authored-by: noamzbr <noamzbr@users.noreply.github.com> Co-authored-by: Nathan Thomas <nwthomas@me.com> Co-authored-by: Brendan Allan <brendonovich@outlook.com>

… on models.dev Co-authored-by: velzie <58010778+velzie@users.noreply.github.com>

github-actions bot reviewed Dec 3, 2025

View reviewed changes

packages/opencode/src/provider/provider.ts Show resolved Hide resolved

github-actions bot reviewed Dec 3, 2025

View reviewed changes

packages/web/src/content/docs/providers.mdx Outdated Show resolved Hide resolved

github-actions bot reviewed Dec 3, 2025

View reviewed changes

packages/opencode/src/provider/provider.ts Outdated Show resolved Hide resolved

github-actions bot reviewed Dec 3, 2025

View reviewed changes

packages/opencode/src/provider/provider.ts Outdated Show resolved Hide resolved

jerome-benoit commented Dec 3, 2025

View reviewed changes

jerome-benoit marked this pull request as ready for review December 3, 2025 19:48

Copilot AI review requested due to automatic review settings December 3, 2025 19:48

Copilot started reviewing on behalf of jerome-benoit December 3, 2025 19:49 View session

github-actions bot reviewed Dec 3, 2025

View reviewed changes

packages/opencode/src/provider/provider.ts Outdated Show resolved Hide resolved

Copilot finished reviewing on behalf of jerome-benoit December 3, 2025 19:51

Copilot AI reviewed Dec 3, 2025

View reviewed changes

packages/opencode/src/provider/provider.ts Outdated Show resolved Hide resolved

packages/opencode/src/provider/provider.ts Outdated Show resolved Hide resolved

jerome-benoit mentioned this pull request Dec 3, 2025

feat: Add SAP AI Core provider anomalyco/models.dev#466

Merged

rekram1-node reviewed Dec 3, 2025

View reviewed changes

packages/opencode/src/provider/provider.ts Outdated Show resolved Hide resolved

rekram1-node reviewed Dec 3, 2025

View reviewed changes

rekram1-node reviewed Dec 4, 2025

View reviewed changes

packages/opencode/src/provider/provider.ts Outdated Show resolved Hide resolved

rekram1-node reviewed Dec 4, 2025

View reviewed changes

packages/opencode/package.json Outdated Show resolved Hide resolved

jerome-benoit added 7 commits December 4, 2025 12:08

fix: address review comments for SAP AI Core provider

bc0d490

- Remove unused provider parameter - Simplify ternary in options - Fix misleading documentation about default values

fix: do not create SAP AI provider mutliple times

5752e9d

Add support for async create...Provider functions. The overhead is negligible sync the provider is created only once and cached. Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

fix: cache SAP AI provider and support async provider creation

0900876

refactor: remove SAP AI CORE from bundled providers

bceb238

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor: remove unneeded sap AI core vercel sdk dependency

08f18d9

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

refactor: use Env module in SAP AI Core provider implementation

6001775

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

jerome-benoit force-pushed the feat/add-sap-ai-core-provider branch from d15fb81 to 6001775 Compare December 4, 2025 11:15

jerome-benoit added 3 commits December 4, 2025 13:37

Merge branch 'dev' into feat/add-sap-ai-core-provider

30726d9

Merge branch 'dev' into feat/add-sap-ai-core-provider

9c16691

Merge branch 'dev' into feat/add-sap-ai-core-provider

5873c6b

rekram1-node merged commit b9c1f10 into anomalyco:dev Dec 4, 2025
3 checks passed

jerome-benoit deleted the feat/add-sap-ai-core-provider branch December 5, 2025 22:37

Copilot AI added a commit to velzie/opencode that referenced this pull request Feb 3, 2026

docs: Add real PR example (SAP AI Core anomalyco#5023) - provider NOT…

10cc1ab

… on models.dev Co-authored-by: velzie <58010778+velzie@users.noreply.github.com>

Copilot AI mentioned this pull request Feb 3, 2026

docs: Document provider addition process and npm package architecture velzie/opencode#2

Draft

Comments

Conversation

jerome-benoit commented Dec 3, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

jerome-benoit left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jerome-benoit left a comment

Choose a reason for hiding this comment

Uh oh!

wsx999outlook commented Dec 3, 2025

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

jerome-benoit commented Dec 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Addressed in commit 25b413b

Uh oh!

jerome-benoit commented Dec 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

⚠️ Blocked by models.dev PR

Uh oh!

Uh oh!

rekram1-node Dec 3, 2025

Choose a reason for hiding this comment

Uh oh!

jerome-benoit Dec 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

wsx999outlook commented Dec 3, 2025

Uh oh!

wsx999outlook commented Dec 3, 2025

Uh oh!

rekram1-node commented Dec 4, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

jerome-benoit left a comment •

edited

Loading

jerome-benoit commented Dec 3, 2025 •

edited

Loading

Addressed in commit `25b413b`

jerome-benoit commented Dec 3, 2025 •

edited

Loading

jerome-benoit Dec 3, 2025 •

edited

Loading