Skip to content

Comments

Chromie/fix deprecate model to provider map#3

Merged
chromiebot merged 2 commits intomainfrom
chromie/fix-deprecate-model-to-provider-map
Jan 27, 2026
Merged

Chromie/fix deprecate model to provider map#3
chromiebot merged 2 commits intomainfrom
chromie/fix-deprecate-model-to-provider-map

Conversation

@chromiebot
Copy link
Owner

why

what changed

test plan

@chromiebot chromiebot force-pushed the chromie/fix-deprecate-model-to-provider-map branch from 4dea856 to 79c5f1c Compare January 27, 2026 23:36
Chromie Bot and others added 2 commits January 27, 2026 23:51
Add unit tests to verify:
- UnsupportedModelError includes guidance about provider/model format
- LLMProvider logs deprecation warning for legacy model names
- Legacy model names still work (non-breaking)
- Provider/model format works without deprecation warning
- UnsupportedAISDKModelProviderError for invalid providers

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Updates to inform users about the new provider/model format:

1. UnsupportedModelError now includes a note encouraging users to use
   the provider/model format (e.g., "openai/gpt-4o") instead of the
   legacy format (e.g., "gpt-4o")

2. Fixed the default case in LLMProvider switch to throw
   UnsupportedModelProviderError (for internal consistency issues)
   instead of UnsupportedModelError (which is for user-facing errors)

This is a non-breaking change - legacy model names still work but
users are now informed about the preferred format.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@chromiebot chromiebot force-pushed the chromie/fix-deprecate-model-to-provider-map branch from 79c5f1c to ef92609 Compare January 27, 2026 23:51
@chromiebot chromiebot merged commit dc560cb into main Jan 27, 2026
20 of 26 checks passed
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.

1 participant