Skip to content

fix ollam provider#3858

Merged
sandy081 merged 1 commit intomainfrom
sandy081/supporting-stingray
Feb 19, 2026
Merged

fix ollam provider#3858
sandy081 merged 1 commit intomainfrom
sandy081/supporting-stingray

Conversation

@sandy081
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings February 19, 2026 16:59
@sandy081 sandy081 enabled auto-merge February 19, 2026 16:59
@sandy081 sandy081 self-assigned this Feb 19, 2026
@vs-code-engineering vs-code-engineering bot added this to the February 2026 milestone Feb 19, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Ollama BYOK provider to construct the correct OpenAI-compatible chat completions endpoint URL for Ollama servers.

Changes:

  • Add OpenAIEndpoint import to enable Ollama-specific endpoint creation.
  • Override createOpenAIEndPoint in OllamaLMProvider to target /v1/chat/completions.

@sandy081 sandy081 added this pull request to the merge queue Feb 19, 2026
Merged via the queue into main with commit f9df7eb Feb 19, 2026
32 checks passed
@sandy081 sandy081 deleted the sandy081/supporting-stingray branch February 19, 2026 17:33
@grische
Copy link

grische commented Feb 19, 2026

@sandy081 As the commit is from #3684, imho it would have been fair to either cherry-pick or at least give the original author @lsby a "Co-Authored-By:" tag or mention in the commit message, as they are the original author of this fix.

@riverar
Copy link

riverar commented Feb 19, 2026

(And a friendly reminder that they retain copyright of their code submissions.)

@elemenopyunome
Copy link

the only options i can choose from are stuck behind pay walls. typical

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.

6 participants