| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
…ay to enable model discovery on a provider. Model discovery is opt-in by provider, to provide backwards compatibility
|
Hey! Your PR title Fixes #6231 - automatic discovery of models from providers doesn't follow conventional commit format. Please update it to start with one of:
Where scope is the package name (e.g., app, desktop, opencode). See CONTRIBUTING.md for details. |
Sorry, something went wrong.
|
The following comment was made by an LLM, it may be inaccurate: Based on my search results, I found several potentially related PRs that address similar model discovery functionality: Most Relevant:
Related Features:
These PRs are not exact duplicates of PR #39176, but they address overlapping concerns around automatic model discovery from various provider types. You may want to review these to ensure consistency in implementation approach and avoid duplicate logic. |
Sorry, something went wrong.
|
I implemented this same feature independently before I found your PR — yours is further along (docs, config schema, SDK types), so I'm standing down and would rather help this one land. Four things from digging through the earlier attempt in #8359 that I think are the actual blockers worth pre-empting:
Reusing buildConfigModel for discovered models is nicer than what I did (I hand-rolled a minimal Model) — this way anything models.dev already knows keeps its real metadata and variants. |
Sorry, something went wrong.
|
Automated PR Cleanup Thank you for contributing to opencode. Due to the high volume of PRs from users and AI agents, we periodically close older PRs using automated criteria so maintainers can focus review time on the most active and community-supported contributions. This PR was closed because it matched the following cleanup criteria:
PRs created within the last month are not affected by this cleanup. If you believe this PR was closed incorrectly, or if you are still actively working on it, please leave a comment explaining why it should be reopened. A maintainer can review and reopen it if appropriate. Thanks again for taking the time to contribute. |
Sorry, something went wrong.
|
it's still relevant because people have been asking for this for months or years |
Sorry, something went wrong.
|
have anyone checked if this is available/working in opencode2/beta ? I even cannot tell/find where is source code of the beta |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Issue for this PR
Closes #6231
Type of change
What does this PR do?
Fixes issue #6231 to provide a generic way that providers can pull /v1/models, and therefore automatically update their model list at any time.
Updates documentation with information how providers can use this functionality
How did you verify your code works?
Testing + Automated testing
Screenshots / recordings
none
Checklist