feat(githubcopilot): add ability to fetch supported models#2717
feat(githubcopilot): add ability to fetch supported models#2717DOsinga merged 1 commit intoblock:mainfrom
Conversation
|
The list of providers displays correctly on cli. The UI doesn't really allow Copilot configuration so I've not made any changes there. I can filter the list further if necessary, as I believe we should only be showing models that can do tool calling. Please let me know.
|
gugahoi
force-pushed
the
list-copilot-models
branch
3 times, most recently
from
25b3f98 to
fd281e1
Compare
|
@taniashiba this would be a great addition for copilot (which seems to be a popular choice of provider) |
DOsinga
requested review from
DOsinga
and removed request for
baxen and
salman1993
|
hey @gugahoi - can you do the https://github.com/block/goose/pull/2717/checks?check_run_id=45941194620 so we can merge this? sorry about the slow reaction and overhead |
This commit introduces a new function `fetch_supported_models_async` to the `GithubCopilotProvider`. This function makes an API call to the `/models` endpoint of the GitHub Copilot service to retrieve a list of currently supported model identifiers. Signed-off-by: Gustavo Hoirisch <gugahoi@users.noreply.github.com>
gugahoi
force-pushed
the
list-copilot-models
branch
from
fd281e1 to
9c28a30
Compare
No worries, I've added the sign off to the commit. |
|
@DOsinga in case you didn't see my reply ^^ |
|
@DOsinga @taniashiba any changes needed to get this merged? |
* main: chore: small refactor on agent.rs (#3703) docs: Add GitMCP Tutorial to Extensions Library (#3716) chore: Speed up CI (#3711) Fix tool vector tests (#3709) docs: GitMCP Tutorial (#3708) Remove unused dependencies (#3626) feat: update Groq models for better tool calling support (#3676) chore: remove ffi libraries and related code (#3699) only run google analytics in prod (#3395) Fix typo in quickstart document (#3447) fix: pricing estimation for OpenRouter in goose-cli (#3675) fix: escape control characters in LLM tool call arguments JSON (#2893) feat(githubcopilot): add ability to fetch supported models (#2717) Create a message ID for tool response messages (#3591) fix: Fixed 404 broken link to extensions page in index.md (#3623)
Signed-off-by: Gustavo Hoirisch <gugahoi@users.noreply.github.com> Signed-off-by: Adam Tarantino <tarantino.adam@hey.com>
4 participants
This PR introduces a new function
fetch_supported_models_asyncto theGithubCopilotProvider. This function makes an API call to the/modelsendpoint of the GitHub Copilot service to retrieve a list of currently supported model identifiers.