Skip to content

feat(proxy): enhance VS Code model handling with gateway integration and fallback logic - #507

Open
crowar wants to merge 9 commits into
codemie-ai:mainfrom
crowar:bugfix/vscode-get-models
Open

feat(proxy): enhance VS Code model handling with gateway integration and fallback logic#507
crowar wants to merge 9 commits into
codemie-ai:mainfrom
crowar:bugfix/vscode-get-models

Conversation

@crowar

@crowar crowar commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

This pull request enhances how the VS Code connector discovers and configures available language models, making the process more robust and informative. The main improvement is that the connector now queries the local gateway for the actual set of available models, falls back gracefully to a curated catalog if discovery fails, and always returns the list of configured model IDs. Logging and error handling have also been improved to aid debugging and transparency.

@romius

romius commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

@crowar
Thanks for the work - dynamic model discovery is the right direction. However, I think we should hold this PR for now.
This implementation retrieves only model IDs and derives the remaining attributes by matching the existing local catalog. For new models, it falls back to generic chat-completions defaults, which can configure Messages or Responses models incorrectly.
Full model metadata required by VS Code-API type, capabilities, token limits, reasoning, vision, tool calling, etc. - is being implemented on the gateway backend already. Once that contract is available, VS Code should consume the backend model definitions directly and switch fully to backend-driven configuration.
The change should be ready soon, so I suggest revisiting this PR after the backend changes are available rather than merging the current ID-only mapping approach.

@codemie-ai

Copy link
Copy Markdown
Owner

[AUTO_CLOSE_WARNING] ⏰ This pull request is older than 7 days and will be automatically closed in 7 more days (when it reaches 14 days old)! To maintain this PR, either convert it to Draft or complete your changes and merge.

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.

3 participants