Skip to content

fix(config): Improve ModelIdConfigError::StrParse error message#654

Merged
JeanMertz merged 1 commit into
mainfrom
prr222
May 18, 2026
Merged

fix(config): Improve ModelIdConfigError::StrParse error message#654
JeanMertz merged 1 commit into
mainfrom
prr222

Conversation

@JeanMertz
Copy link
Copy Markdown
Collaborator

@JeanMertz JeanMertz commented May 18, 2026

The previous error message only mentioned the <provider>/<model> full ID format, without acknowledging that model aliases are also valid. This was misleading when a user provided an invalid model ID — the error didn't reflect the full set of accepted inputs.

The message now reads: "model ID does not match a known alias nor matches the full ID format provider/model", which more accurately describes what was attempted and why it failed.

The previous error message only mentioned the `<provider>/<model>` full
ID format, without acknowledging that model aliases are also valid. This
was misleading when a user provided an invalid model ID — the error
didn't reflect the full set of accepted inputs.

The message now reads: "model ID does not match a known alias nor
matches the full ID format <provider>/<model>", which more accurately
describes what was attempted and why it failed.

Signed-off-by: Jean Mertz <git@jeanmertz.com>
@JeanMertz JeanMertz merged commit b709f79 into main May 18, 2026
14 checks passed
@JeanMertz JeanMertz deleted the prr222 branch May 18, 2026 08:27
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