Skip to content

[WIP] Fix Ollama requirement in openai integration#111

Closed
Copilot wants to merge 1 commit into
mainfrom
copilot/fix-ollama-requirement-issue
Closed

[WIP] Fix Ollama requirement in openai integration#111
Copilot wants to merge 1 commit into
mainfrom
copilot/fix-ollama-requirement-issue

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Oct 7, 2025

Thanks for assigning this issue to me. I'm starting to work on it and will keep this PR's description up to date as I form a plan and make progress.

Original prompt

This section details on the original issue you should resolve

<issue_title>Ollama is required?</issue_title>
<issue_description>Hello, i am trying to use openai but it seems its requiring Ollama?

plexsync: Sending request to LLM service
Initialized music search with providers: brave_search
Using Ollama model 'qwen3:latest' for result extraction and tool selection
Searching for track info: Sicko Mode by Travis Scott from Astroworld
Unified search failed: Client.chat() got an unexpected keyword argument 'tools'
Found track info: {'title': 'Sicko Mode by Travis Scott from Astroworld', 'album': None, 'artist': ''}
Searching for track info: Pop Out by Polo G feat. Lil Tjay from Die a Legend
Unified search failed: Client.chat() got an unexpected keyword argument 'tools'
Found track info: {'title': 'Pop Out by Polo G feat. Lil Tjay from Die a Legend', 'album': None, 'artist': ''}
Searching for track info: Life Is Good by Future feat. Drake from High Off Life
Unified search failed: Client.chat() got an unexpected keyword argument 'tools'
</issue_description>

<agent_instructions>The current llm search implementation relies on Ollama. Can we extend it allow any OpenAI compatible API. Re use the llm configuration for the search</agent_instructions>

Comments on the Issue (you are @copilot in this section)

Fixes #110

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

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.

Ollama is required?

2 participants