dcdpr / jp / 26627902919

66%

Build Type

push

github

Committed by web-flow

Commit Message

fix(cli): Use configured model id in live role-header (#686)

The role-header model id was being sourced from `ModelDetails.id`
returned by the provider. Some providers rewrite the id — for example,
Anthropic resolves an unversioned name to a date-suffixed canonical
form. This caused the live turn header and the replay header to show
different model strings for the same conversation.

On replay, `TurnRenderer::reconfigure` reads the per-turn
`PartialAppConfig` snapshot stored in the conversation, which always
holds the *configured* id. Pinning the live path to the same source
(`cfg.assistant.model.id.resolved()`) keeps both paths consistent and
stable across API version drift.

`TurnRenderer::reconfigure` is updated in the same direction: identity
fields (assistant name and model id) are now read from the partial
directly, before the full `AppConfig::from_partial_with_defaults`
rebuild, so the role header never depends on default-filling for these
two stable fields. A new `render_model_id` helper maps a fully-empty
partial id to `None`, suppressing the `(model)` suffix from the header
rather than showing an empty string.

---------

Signed-off-by: Jean Mertz <git@jeanmertz.com>

Coverage Stats

8 of 8 new or added lines in 2 files covered. (100.0%)

8 existing lines in 2 files now uncovered.

31993 of 48154 relevant lines covered (66.44%)

260.91 hits per line