[UX] Session model switching should not overwrite the deployment default #3398
Replies: 1 comment
|
I verified this boundary against rc.7 (
The current For rc.7 operators, the safest sequence is:
That last point matters for cost investigations because the visible parent picker is not sufficient evidence for a child or future Agent's actual route. I turned the source path, four state lifetimes, and acceptance gates into a visual runbook here: https://sandbaseai.github.io/deepseek-harness-handbook/session-model-default.html Independent community handbook, not an official DeepSeek AI project. |
Uh oh!
There was an error while loading. Please reload this page.
Summary
In
0.1.0-rc.7, choosing a model in one Web session also overwrites the deployment-wideagent-default-modelsetting. These appear to be different user intents:Please consider making
session.selectModelsession-local and moving default changes to an explicit settings action (or an explicit “Set as default” action).Current behavior
agent-default-modelvalue insettings.yaml.agent-default-model, so subsequently created sessions inherit it.The Host path currently updates the session selection and then awaits the deployment-default write:
session.selectModel → dsh-host-apiproxy → saveDefaultModelSelection → ctx.agentDefaultModel.saveSelection → settings.yamlRelevant official sources:
session.selectModelimplementationctx.agentDefaultModel.saveSelectionf2d1a296Why this is surprising
A user may keep a stable default for new work while selecting specialized models for individual conversations. With the current behavior, visiting an old session and changing its model silently changes future-session behavior.
The Settings write is also on the interactive response path. In one local reproduction,
session.modelscompleted in about 1.7 ms, while a no-opsession.selectModeltook about 2.14 s because a stale external Settings writer lock made the default save wait for its timeout. The stale lock was not created by DSH, but a session-local model switch should not depend on deployment-settings persistence in the first place.Expected behavior
session.selectModelchanges only the addressed session.agent-default-modelchanges only through an explicit deployment-default configuration action.An optional combined “Switch and set as default” action could be added later, but it should be explicit.
Related discussions
I did not find a Discussion dedicated to this picker/default coupling. It is mentioned as a side effect in #1581, and the current behavior is used as a workaround in #455. Those reports focus on subagent inheritance from a stale creation-time route, so this request is related but distinct.
Environment
@deepseek-ai/dsh@0.1.0-rc.7All reactions