-
Notifications
You must be signed in to change notification settings - Fork 2.4k
feat: fetch openrouter supported models in goose configure
#3347
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: fetch openrouter supported models in goose configure
#3347
Conversation
goose configure
The-Best-Codes
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ready for review!
michaelneale
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
very nice - I don't have access to manually test open router - any chance you can add a small recording showing it in action?
Also - where is fetch_supported_models_async called from? I must be missing some context
|
@michaelneale The goose/crates/goose/src/providers/base.rs Lines 251 to 253 in 67664a2
It gets called in goose/crates/goose-cli/src/commands/configure.rs Lines 330 to 338 in 67664a2
|
michaelneale
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice isolated change, will help with experience with open router
|
@The-Best-Codes merged - wonder if future enhancement is to have some UI that lets people find.. as there is a lot! (but no idea what that would look like) |
* 'main' of github.com:block/goose: feat: fetch openrouter supported models in `goose configure` (#3347) Add the ability to configure rustyline to use a different edit mode (e.g. vi) (#2769) docs: update CLI provider guide (#3397) Streamable HTTP CLI flag (#3394) docs: Show both remote options for extensions in CLI (#3392)
|
@michaelneale Awesome!
Do you mean in Goose desktop or a TUI? In the terminal, maybe type to search the list of models? |
* main: (51 commits) docs: reflecting benefits of CLI providers (block#3399) feat: fetch openrouter supported models in `goose configure` (block#3347) Add the ability to configure rustyline to use a different edit mode (e.g. vi) (block#2769) docs: update CLI provider guide (block#3397) Streamable HTTP CLI flag (block#3394) docs: Show both remote options for extensions in CLI (block#3392) docs: fix YouTube Transcript MCP package manager (block#3390) docs: simplify alby mcp (block#3379) docs: add max turns (block#3372) feat(cli): add cost estimation per provider for Goose CLI (block#3330) feat: Allow Ollama for non-tool models for chat only (block#3308) [cli] Add --provider and --model CLI options to run command (block#3295) Docs: Lead/worker model in Goose Desktop (block#3342) revert: refactor: abstract keyring logic to better enable DI (block#3358) Drop temporal-service binary (block#3340) docs: add fuzzy search (block#3357) Fix name of GPT-4.1 System Prompt (block#3348) (block#3351) docs: add goose-mobile (block#3315) refactor: abstract keyring logic to better enable DI (block#3262) fix: correct tool use for anthropic (block#3311) ...
) Signed-off-by: Adam Tarantino <[email protected]>
) Signed-off-by: Soroosh <[email protected]>
) Signed-off-by: Kyle Santiago <[email protected]>
Before:
You have to type out a model manually.
After:
Goose CLI will automatically fetch the list of models that support tool use from OpenRouter when you configure it as a provider: