Conversation
angelahning
force-pushed
the
aning/ui-model
branch
from
c900fdd to
cfd9660
Compare
- Update sdk.gen.ts with getProviderModels endpoint - Update types.gen.ts with new API types - Generated from updated OpenAPI specification
|
Very nice! Probably worth a test too with OpenAI / Anthropic or some other providers we have |
angelahning
force-pushed
the
aning/ui-model
branch
from
56feb65 to
b5ca66a
Compare
I added some error tests here https://github.com/block/goose/pull/4147/files#diff-1fd59fa8152efe9c7efb97fbbd7a0288bd5fb41ce5f5b4bc97182690e15787b4R877-R912. But it's hard to test for the happy path since fetch_supported_model just returns error without a proper api key. I could use a mock provider, but at that point I am just testing we can call that method and return the result? i dunno. What other tests do you think is useful? @jamadeo |
Signed-off-by: Angela Ning <aning@squareup.com>
angelahning
force-pushed
the
aning/ui-model
branch
from
b5ca66a to
b257707
Compare
|
Having too much trouble fixing the DCO check. Opened a new PR instead #4270 |
3 participants
The UI queries the providers for the list of supported models, instead of using our hard-coded lists.
fixes #3645
Screen.Recording.2025-08-17.at.10.44.26.PM.mov