diff options
| author | Danilo M. <danix@danix.xyz> | 2026-10-17 10:59:22 +0200 |
|---|---|---|
| committer | Danilo M. <danix@danix.xyz> | 2026-10-17 10:59:28 +0200 |
| commit | 8e9d8b9ac1b593032dea9eca312fa1495c0c8858 (patch) | |
| tree | 9329824bc6a1d8e4f44da39e3722b4e2ea36c9f6 /desktop/modules/assistant/AssistantModule.qml | |
| parent | ca3c66d882e60874b1cf4a1c5f9ea4232a593255 (diff) | |
| download | quickshell-8e9d8b9ac1b593032dea9eca312fa1495c0c8858.tar.gz quickshell-8e9d8b9ac1b593032dea9eca312fa1495c0c8858.zip | |
assistant: Load no longer changes the preferred model
Load set the preference as well as swapping the model in, so loading a
model from the settings page for any reason made it the assistant's
model too: after loading Qwen there, the canvas kept Qwen instead of
bringing the preferred Gemma back. Load and Prefer are now separate
buttons: Load swaps a model in now and leaves the preference alone, and
Prefer only records which model the canvas loads when it opens.
The canvas header also showed a stale model name after a swap; it now
takes the model from the daemon's ctx event, which carries it from
desktop-assistant f9d4644 on.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Diffstat (limited to 'desktop/modules/assistant/AssistantModule.qml')
| -rw-r--r-- | desktop/modules/assistant/AssistantModule.qml | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/desktop/modules/assistant/AssistantModule.qml b/desktop/modules/assistant/AssistantModule.qml index b53a33f..8253cc6 100644 --- a/desktop/modules/assistant/AssistantModule.qml +++ b/desktop/modules/assistant/AssistantModule.qml @@ -179,11 +179,11 @@ Module { } // The router keeps one model resident, so this swaps out whatever is - // loaded, for every client: it runs only on a click. The id becomes the - // preferred one, and the list is polled until the router reports it - // loaded, since the request may return before the load finishes. + // loaded, for every client. It does not touch the preferred model: a + // load from the page is for now, and the canvas brings the preferred one + // back when it opens. The list is polled until the router reports the + // model loaded, since the request may return before the load finishes. function loadModel(id) { - Prefs.setAssistantModel(id); am.modelError = ""; am.loading = id; am.run(["curl", "-s", "-m", "600", "-X", "POST", "-H", "Content-Type: application/json", |
