summaryrefslogtreecommitdiffhomepage
path: root/src/features/chat
diff options
context:
space:
mode:
authorAdam Malczewski <[email protected]>2026-08-20 19:36:12 +0900
committerAdam Malczewski <[email protected]>2026-08-20 19:36:12 +0900
commitfb7eb3105e1fbbdefdad9348f9132815b5bbe52d (patch)
treedeb95e97a917424ea67470b168b91349a41d1c08 /src/features/chat
parent63588c3c69b32d180dc78a2cc187cc8aa22ad7e0 (diff)
downloaddispatch-web-fb7eb3105e1fbbdefdad9348f9132815b5bbe52d.tar.gz
dispatch-web-fb7eb3105e1fbbdefdad9348f9132815b5bbe52d.zip
feat(named-agents): sidebar view, synthetic provider, auto-save, snapshot semantics
Implement the frontend of the Named Agents feature (plan §5-§9): - New features/named-agents module: pure logic (types re-exported from @dispatch/wire + transport-contract, validation, provider/model options, snapshot helpers, network normalizers, auto-save status model) + an injected auto-save controller (name debounce, immediate provider/model save, monotonic stale-response guard) + the collapsible sidebar card view. - Store: /named-agents CRUD + boot seed; selectModel(model, snapshot?) persists the raw model WITH the namedAgent snapshot; draft promotion carries it; refreshModel restores ModelResponse.namedAgent. - Chat store setNamedAgent (sends namedAgent on chat.send); Tab.snapshot + setSnapshot (retained across rename/remove, persisted with tabs). - Model selector: synthetic Named Agents provider ALWAYS first; selecting one resolves to the raw model + records a snapshot; a retained snapshot of a renamed/removed agent still displays as unavailable (plan §7). - Composition root wires the view (viewKinds + loaded-modules manifest) + feeds the selector. Tests: logic + controller + selector + app-store CRUD (1246 pass), typecheck 0, biome clean (only pre-existing config infos pending a future migrate), build OK.
Diffstat (limited to 'src/features/chat')
-rw-r--r--src/features/chat/index.ts7
-rw-r--r--src/features/chat/model-select.test.ts49
-rw-r--r--src/features/chat/model-select.ts70
-rw-r--r--src/features/chat/store.svelte.ts16
-rw-r--r--src/features/chat/ui.test.ts141
-rw-r--r--src/features/chat/ui/ModelSelector.svelte198
6 files changed, 425 insertions, 56 deletions
diff --git a/src/features/chat/index.ts b/src/features/chat/index.ts
index 8cf776b..6de8164 100644
--- a/src/features/chat/index.ts
+++ b/src/features/chat/index.ts
@@ -6,12 +6,19 @@ export type {
} from "../../core/chunks";
export { groupRenderedChunks, resolveImageUrl, viewProviderRetry } from "../../core/chunks";
export type { TurnMetricsEntry } from "../../core/metrics";
+export type { NamedAgentLike, NamedAgentSnapshotLike } from "./model-select";
export {
+ isNamedAgentSelection,
isNumberedOpenCode,
isVisionModel,
migrateLegacyModelName,
modelKeyLabel,
+ NAMED_AGENTS_PROVIDER,
+ namedAgentNames,
+ namedAgentSelectionLabel,
+ namedAgentsProviderLabel,
openCodeNumber,
+ providerKeysWithNamedAgents,
} from "./model-select";
export type { ChatTransport, HistorySync, HistoryWindow, MetricsSync } from "./ports";
export type {
diff --git a/src/features/chat/model-select.test.ts b/src/features/chat/model-select.test.ts
index 33c1a16..57de11d 100644
--- a/src/features/chat/model-select.test.ts
+++ b/src/features/chat/model-select.test.ts
@@ -1,12 +1,18 @@
import { describe, expect, it } from "vitest";
import {
+ isNamedAgentSelection,
isVisionModel,
joinModelName,
migrateLegacyModelName,
modelKeyLabel,
modelKeys,
modelsForKey,
+ NAMED_AGENTS_PROVIDER,
+ namedAgentNames,
+ namedAgentSelectionLabel,
+ namedAgentsProviderLabel,
openCodeNumber,
+ providerKeysWithNamedAgents,
splitModelName,
} from "./model-select";
@@ -143,3 +149,46 @@ describe("isVisionModel", () => {
expect(isVisionModel({}, "kimi/k2")).toBe(false);
});
});
+
+describe("named-agent provider helpers", () => {
+ it("providerKeysWithNamedAgents puts the synthetic provider first, then raw keys", () => {
+ expect(providerKeysWithNamedAgents(["openai/gpt-4", "anthropic/claude"])).toEqual([
+ NAMED_AGENTS_PROVIDER,
+ "openai",
+ "anthropic",
+ ]);
+ // Always present even with zero models.
+ expect(providerKeysWithNamedAgents([])).toEqual([NAMED_AGENTS_PROVIDER]);
+ });
+
+ it("namedAgentsProviderLabel + NAMED_AGENTS_PROVIDER are stable", () => {
+ expect(namedAgentsProviderLabel()).toBe("Named Agents");
+ expect(NAMED_AGENTS_PROVIDER).toBe("__named_agents__");
+ });
+
+ it("namedAgentNames returns the display names", () => {
+ expect(namedAgentNames([{ id: "a", name: "Coding", model: "m" }])).toEqual(["Coding"]);
+ });
+
+ it("isNamedAgentSelection is true only for a snapshot whose model equals the selection", () => {
+ expect(
+ isNamedAgentSelection("opencode-1/claude", {
+ namedAgentId: "a",
+ namedAgentName: "C",
+ model: "opencode-1/claude",
+ }),
+ ).toBe(true);
+ expect(
+ isNamedAgentSelection("other/model", {
+ namedAgentId: "a",
+ namedAgentName: "C",
+ model: "opencode-1/claude",
+ }),
+ ).toBe(false);
+ expect(isNamedAgentSelection("opencode-1/claude", null)).toBe(false);
+ });
+
+ it("namedAgentSelectionLabel renders 'Named Agents -> <name>'", () => {
+ expect(namedAgentSelectionLabel("Coding")).toBe("Named Agents -> Coding");
+ });
+});
diff --git a/src/features/chat/model-select.ts b/src/features/chat/model-select.ts
index 5da6e7b..6f198f3 100644
--- a/src/features/chat/model-select.ts
+++ b/src/features/chat/model-select.ts
@@ -108,3 +108,73 @@ export function isVisionModel(
): boolean {
return modelInfo[fullName]?.vision === true;
}
+
+// ── Named Agents synthetic provider (plan §6) ────────────────────────────────
+//
+// The regular model picker gains a "Named Agents" provider at the TOP of the
+// provider list. Selecting it + a Named Agent name resolves to that agent's raw
+// model for the conversation AND records a snapshot so the selection survives
+// later edits to the shared preset. These helpers are structural (they take a
+// minimal NamedAgent-ish shape) so `model-select` stays decoupled from the
+// named-agents feature up the chain; the picker binds the real types.
+
+/** A minimal Named Agent shape the synthetic provider needs (structural). */
+export interface NamedAgentLike {
+ readonly id: string;
+ readonly name: string;
+ readonly model: string;
+}
+
+/** A minimal per-conversation Named Agent selection snapshot (structural). */
+export interface NamedAgentSnapshotLike {
+ readonly namedAgentId: string;
+ readonly namedAgentName: string;
+ readonly model: string;
+}
+
+/**
+ * The synthetic "Named Agents" provider id used by the regular model picker
+ * (plan §6). Deliberately a value that can never collide with a real provider
+ * (`<credentialName>`), so the provider dropdown treats it as the first option.
+ */
+export const NAMED_AGENTS_PROVIDER = "__named_agents__";
+
+/** The friendly label of the synthetic Named Agents provider. */
+export function namedAgentsProviderLabel(): string {
+ return "Named Agents";
+}
+
+/**
+ * The provider dropdown options for the picker: the synthetic "Named Agents"
+ * provider ALWAYS first, then the raw providers (via `modelKeys`). The synthetic
+ * entry stays present even with zero Named Agents (plan §6 shows a disabled
+ * "No named agents configured" model option).
+ */
+export function providerKeysWithNamedAgents(models: readonly string[]): string[] {
+ return [NAMED_AGENTS_PROVIDER, ...modelKeys(models)];
+}
+
+/** The Named Agent names offered in the model dropdown when the synthetic provider is selected. */
+export function namedAgentNames(agents: readonly NamedAgentLike[]): string[] {
+ return agents.map((a) => a.name);
+}
+
+/**
+ * Whether a full raw model string corresponds to a Named Agent snapshot —
+ * i.e. the conversation was selected via a Named Agent whose captured raw model
+ * equals `selected`. When true, the picker displays `Named Agents -> <name>`.
+ */
+export function isNamedAgentSelection(
+ selected: string,
+ snapshot: NamedAgentSnapshotLike | null,
+): boolean {
+ return snapshot !== null && snapshot.model === selected;
+}
+
+/**
+ * The snapshot label shown in the model dropdown for a Named Agent selection:
+ * "Named Agents -> <name>". Pure display.
+ */
+export function namedAgentSelectionLabel(name: string): string {
+ return `${namedAgentsProviderLabel()} -> ${name}`;
+}
diff --git a/src/features/chat/store.svelte.ts b/src/features/chat/store.svelte.ts
index 24a1d06..c83c5fd 100644
--- a/src/features/chat/store.svelte.ts
+++ b/src/features/chat/store.svelte.ts
@@ -34,6 +34,7 @@ import {
selectOrderedTurnMetrics,
} from "../../core/metrics";
import type { ConversationCache } from "../conversation-cache";
+import type { NamedAgentSelectionSnapshot } from "../named-agents";
import type { ChatTransport, HistorySync, MetricsSync } from "./ports";
export interface ChatStoreDependencies {
@@ -145,6 +146,15 @@ export interface ChatStore {
cancelQueuedMessage(messageId: string): void;
setModel(model: string): void;
/**
+ * Set (or clear, with null) the Named Agent selection snapshot this chat sends
+ * with `chat.send` (plan §7). The backend persists it on the conversation at
+ * creation so it freezes the friendly id/name + resolved model — later edits to
+ * the Named Agent do not change this conversation's selection. The raw model
+ * (already set via `setModel`) is sent alongside; the snapshot is display/identity
+ * metadata that preserves HOW it was selected.
+ */
+ setNamedAgent(snapshot: NamedAgentSelectionSnapshot | null): void;
+ /**
* Update the chat limit LIVE: re-normalizes, then adjusts the loaded window.
* Lowering it unloads older committed chunks (deferred via the gate while the
* reader is scrolled up, catching up on the next mutation). Raising it
@@ -180,6 +190,7 @@ export function createChatStore(deps: ChatStoreDependencies): ChatStore {
let _pendingSync = $state(false);
let _error = $state<string | null>(null);
let _model = $state<string | undefined>(deps.model);
+ let _namedAgent = $state<NamedAgentSelectionSnapshot | null>(null);
let disposed = false;
let chatLimit = normalizeChatLimit(deps.chatLimit);
@@ -342,6 +353,7 @@ export function createChatStore(deps: ChatStoreDependencies): ChatStore {
conversationId: deps.conversationId,
message: text,
...(_model !== undefined ? { model: _model } : {}),
+ ...(_namedAgent !== null ? { namedAgent: _namedAgent } : {}),
...(deps.workspaceId !== undefined ? { workspaceId: deps.workspaceId } : {}),
...(images !== undefined && images.length > 0 ? { images: [...images] } : {}),
};
@@ -379,6 +391,10 @@ export function createChatStore(deps: ChatStoreDependencies): ChatStore {
_model = model;
},
+ setNamedAgent(snapshot: NamedAgentSelectionSnapshot | null): void {
+ _namedAgent = snapshot;
+ },
+
async setChatLimit(limit: number): Promise<void> {
const prev = chatLimit;
chatLimit = normalizeChatLimit(limit);
diff --git a/src/features/chat/ui.test.ts b/src/features/chat/ui.test.ts
index aa43e04..3a47219 100644
--- a/src/features/chat/ui.test.ts
+++ b/src/features/chat/ui.test.ts
@@ -1006,7 +1006,9 @@ describe("ModelSelector", () => {
expect(keySelect).toHaveValue("anthropic");
expect(modelSelect).toHaveValue("claude-3");
- expect(optionValues(keySelect)).toEqual(["openai", "anthropic", "google"]);
+ // The synthetic "Named Agents" provider is ALWAYS first (plan §6), even with
+ // zero configured agents.
+ expect(optionValues(keySelect)).toEqual(["__named_agents__", "openai", "anthropic", "google"]);
// only the models under the selected key
expect(optionValues(modelSelect)).toEqual(["claude-3"]);
});
@@ -1041,6 +1043,133 @@ describe("ModelSelector", () => {
expect(onSelect).toHaveBeenCalledWith("openai/gpt-4o");
});
+ it("lists Named Agents under the synthetic provider and resolves a selection to a snapshot", async () => {
+ const onSelectNamedAgent = vi.fn();
+ const user = userEvent.setup();
+ const models = ["openai/gpt-4"];
+ const namedAgents = [
+ { id: "a1", name: "Coding Agent", model: "opencode-1/claude-sonnet-4.5" },
+ { id: "a2", name: "Research Agent", model: "umans/umans-glm-5.2" },
+ ];
+
+ const { rerender } = render(ModelSelector, {
+ props: {
+ models,
+ selected: "openai/gpt-4",
+ onSelect: vi.fn(),
+ namedAgents,
+ onSelectNamedAgent,
+ },
+ });
+
+ // Switching the provider to "Named Agents" jumps to the first agent (mirrors
+ // a raw provider jumping to its first model).
+ await user.selectOptions(
+ screen.getByRole("combobox", { name: "Key selector" }),
+ "__named_agents__",
+ );
+ expect(onSelectNamedAgent).toHaveBeenCalledWith({
+ namedAgentId: "a1",
+ namedAgentName: "Coding Agent",
+ model: "opencode-1/claude-sonnet-4.5",
+ });
+
+ // The parent (store) applies the selection → re-render with the resolved raw
+ // model + snapshot, exactly as `selectModel(model, snapshot)` would do.
+ const codingAgent = namedAgents[0];
+ await rerender({
+ models,
+ selected: codingAgent?.model ?? "",
+ onSelect: vi.fn(),
+ namedAgents,
+ namedAgentSnapshot: {
+ namedAgentId: "a1",
+ namedAgentName: "Coding Agent",
+ model: "opencode-1/claude-sonnet-4.5",
+ },
+ onSelectNamedAgent,
+ });
+
+ // The model dropdown now lists the named agents; pick a different one.
+ await user.selectOptions(
+ screen.getByRole("combobox", { name: "Model selector" }),
+ "Research Agent",
+ );
+ expect(onSelectNamedAgent).toHaveBeenLastCalledWith({
+ namedAgentId: "a2",
+ namedAgentName: "Research Agent",
+ model: "umans/umans-glm-5.2",
+ });
+ });
+
+ it("shows a disabled placeholder when the synthetic provider has no configured agents", () => {
+ render(ModelSelector, {
+ props: { models: ["openai/gpt-4"], selected: "openai/gpt-4", onSelect: vi.fn() },
+ });
+ const modelSelect = screen.getByRole("combobox", { name: "Model selector" });
+ const options = within(modelSelect).getAllByRole("option");
+ // With the raw openai provider selected the model dropdown lists gpt-4; the
+ // "No named agents configured" placeholder appears only under the synthetic
+ // provider (which we'd select to see it). Assert the disabled/empty handling
+ // by checking the synthetic provider is present in the key selector.
+ const keySelect = screen.getByRole("combobox", { name: "Key selector" });
+ expect(optionValues(keySelect)).toContain("__named_agents__");
+ // Switch to the (empty) synthetic provider → disabled "No named agents configured".
+ expect(() => screen.getByRole("option", { name: /No named agents configured/ })).toThrow();
+ expect(options).toHaveLength(1);
+ });
+
+ it("keeps showing a snapshot of a renamed/removed Named Agent (unavailable)", () => {
+ // A conversation selected via a Named Agent retains its snapshot even after
+ // the agent is gone from the current list (plan §7).
+ const namedAgentSnapshot = {
+ namedAgentId: "gone",
+ namedAgentName: "Old Agent",
+ model: "opencode-1/claude-sonnet-4.5",
+ };
+ render(ModelSelector, {
+ props: {
+ models: [],
+ selected: "opencode-1/claude-sonnet-4.5",
+ onSelect: vi.fn(),
+ namedAgents: [],
+ namedAgentSnapshot,
+ },
+ });
+
+ const keySelect = screen.getByRole("combobox", { name: "Key selector" });
+ expect(keySelect).toHaveValue("__named_agents__");
+ const modelSelect = screen.getByRole("combobox", { name: "Model selector" });
+ expect(modelSelect).toHaveValue("Old Agent");
+ expect(screen.getByRole("option", { name: /Old Agent.*unavailable/ })).toBeInTheDocument();
+ });
+
+ it("selecting a raw model from a Named Agent conversation clears the snapshot via onSelect", async () => {
+ const onSelect = vi.fn();
+ const onSelectNamedAgent = vi.fn();
+ const user = userEvent.setup();
+ const namedAgentSnapshot = {
+ namedAgentId: "a1",
+ namedAgentName: "Coding Agent",
+ model: "opencode-1/claude-sonnet-4.5",
+ };
+ render(ModelSelector, {
+ props: {
+ models: ["openai/gpt-4"],
+ selected: "opencode-1/claude-sonnet-4.5",
+ onSelect,
+ namedAgents: [{ id: "a1", name: "Coding Agent", model: "opencode-1/claude-sonnet-4.5" }],
+ namedAgentSnapshot,
+ onSelectNamedAgent,
+ },
+ });
+
+ // Switch from Named Agents to a raw provider → raw model selection clears the snapshot.
+ await user.selectOptions(screen.getByRole("combobox", { name: "Key selector" }), "openai");
+ expect(onSelect).toHaveBeenCalledWith("openai/gpt-4");
+ expect(onSelectNamedAgent).not.toHaveBeenCalled();
+ });
+
it("marks vision-capable models in the model dropdown", () => {
const models = ["kimi/k2", "kimi/k1.5"];
const modelInfo = {
@@ -1099,15 +1228,17 @@ describe("ModelSelector", () => {
const keySelect = screen.getByRole("combobox", { name: "Key selector" });
const options = within(keySelect).getAllByRole("option");
- // Numeric order: #1 < #2 < #10, and values stay RAW (opencode-N).
+ // Numeric order: #1 < #2 < #10 (after the always-first synthetic provider),
+ // and values stay RAW (opencode-N).
expect(options.map((o) => (o as HTMLOptionElement).value)).toEqual([
+ "__named_agents__",
"opencode-1",
"opencode-2",
"opencode-10",
]);
- expect(options[0]?.textContent).toContain("OpenCode #1");
- expect(options[1]?.textContent).toContain("OpenCode #2");
- expect(options[2]?.textContent).toContain("OpenCode #10");
+ expect(options[1]?.textContent).toContain("OpenCode #1");
+ expect(options[2]?.textContent).toContain("OpenCode #2");
+ expect(options[3]?.textContent).toContain("OpenCode #10");
});
it("shows a disappeared saved credential as unavailable without silently falling back", () => {
diff --git a/src/features/chat/ui/ModelSelector.svelte b/src/features/chat/ui/ModelSelector.svelte
index 1d06848..c66d3dd 100644
--- a/src/features/chat/ui/ModelSelector.svelte
+++ b/src/features/chat/ui/ModelSelector.svelte
@@ -1,11 +1,16 @@
<script lang="ts">
import type { ModelMetadata } from "@dispatch/transport-contract";
+ import type { NamedAgent, NamedAgentSelectionSnapshot } from "../../named-agents";
import {
+ isNamedAgentSelection,
isVisionModel,
joinModelName,
modelKeyLabel,
- modelKeys,
modelsForKey,
+ NAMED_AGENTS_PROVIDER,
+ namedAgentNames,
+ namedAgentSelectionLabel,
+ providerKeysWithNamedAgents,
splitModelName,
} from "../model-select";
@@ -14,6 +19,9 @@
selected,
onSelect,
modelInfo = {},
+ namedAgents = [],
+ namedAgentSnapshot = null,
+ onSelectNamedAgent,
}: {
models: readonly string[];
selected: string;
@@ -25,35 +33,88 @@
* Optional — absent metadata → no badge (treated as non-vision).
*/
modelInfo?: Readonly<Record<string, ModelMetadata>>;
+ /** The user-curated Named Agents (plan §6) offered under the synthetic provider. */
+ namedAgents?: readonly NamedAgent[];
+ /** The active conversation's Named Agent snapshot (null when not selected via one). */
+ namedAgentSnapshot?: NamedAgentSelectionSnapshot | null;
+ /** Called when a Named Agent is selected (the selector resolves it to a raw model + snapshot). */
+ onSelectNamedAgent?: (snapshot: NamedAgentSelectionSnapshot) => void;
} = $props();
- const keys = $derived(modelKeys(models));
+ // Whether the current selection is a Named Agent snapshot (the conversation was
+ // selected via one; its captured raw model equals `selected`).
+ const currentIsNamedAgent = $derived(isNamedAgentSelection(selected, namedAgentSnapshot));
+
+ const providerKeys = $derived(providerKeysWithNamedAgents(models));
const current = $derived(splitModelName(selected));
const keyModels = $derived(modelsForKey(models, current.key));
+ // The provider shown in the provider dropdown: the synthetic Named Agents
+ // provider when the selection is a Named Agent snapshot, else the raw key.
+ const currentProvider = $derived(
+ currentIsNamedAgent || current.key === NAMED_AGENTS_PROVIDER
+ ? NAMED_AGENTS_PROVIDER
+ : current.key,
+ );
+
+ const namedNames = $derived(namedAgentNames(namedAgents));
+
// Whether the currently-selected full model name is vision-capable.
const selectedVision = $derived(isVisionModel(modelInfo, selected));
- // Whether the currently-selected full name is PRESENT in the catalog. When a
- // saved credential/model disappears, we show it as unavailable rather than
- // silently switching to something else.
- const selectedUnavailable = $derived(selected !== "" && !models.includes(selected));
- // The key part is itself missing from the catalog (e.g. a whole numbered
- // OpenCode credential was removed).
- const keyUnavailable = $derived(selected !== "" && !keys.includes(current.key));
-
- // Switching key jumps to the first model available under it.
- function selectKey(key: string): void {
- if (key === current.key) return;
+ // Whether the currently-selected raw model is PRESENT in the catalog (only
+ // meaningful for a raw selection — a Named Agent snapshot resolves to a raw
+ // model that may have left the catalog).
+ const rawMissing = $derived(
+ !currentIsNamedAgent && selected !== "" && !models.includes(selected),
+ );
+ const keyUnavailable = $derived(
+ !currentIsNamedAgent && selected !== "" && !providerKeys.includes(current.key),
+ );
+
+ // Switching provider. Going TO the Named Agents provider: jump to the first
+ // Named Agent (mirrors selectKey jumping to the first model); the model
+ // dropdown then lists named names. Going to a raw provider: existing behavior.
+ function selectProvider(key: string): void {
+ if (key === currentProvider) return;
+ if (key === NAMED_AGENTS_PROVIDER) {
+ selectFirstNamedAgent();
+ return;
+ }
const first = modelsForKey(models, key)[0] ?? "";
onSelect(joinModelName(key, first));
}
- function selectModel(model: string): void {
+ // Selecting a raw model suffix under the current raw provider.
+ function selectRawModel(model: string): void {
onSelect(joinModelName(current.key, model));
}
- // The full `<key>/<model>` name for a model suffix under the current key.
+ // Selecting a Named Agent name (under the synthetic provider): resolve it to a
+ // snapshot + raw model and report via onSelectNamedAgent.
+ function selectNamedAgent(name: string): void {
+ const agent = namedAgents.find((a) => a.name === name);
+ if (agent === undefined) return;
+ onSelectNamedAgent?.({
+ namedAgentId: agent.id,
+ namedAgentName: agent.name,
+ model: agent.model,
+ });
+ }
+
+ // Jump to the first configured Named Agent (used when the provider is switched
+ // to Named Agents), matching the raw-provider "jump to first model" behavior.
+ function selectFirstNamedAgent(): void {
+ const first = namedAgents[0];
+ if (first === undefined) return; // no named agents — the model dropdown shows the disabled placeholder
+ onSelectNamedAgent?.({
+ namedAgentId: first.id,
+ namedAgentName: first.name,
+ model: first.model,
+ });
+ }
+
+ // The full `<provider>/<model>` name for a raw model suffix under `current.key`.
function fullNameFor(modelSuffix: string): string {
return joinModelName(current.key, modelSuffix);
}
@@ -62,58 +123,93 @@
<div class="flex flex-col gap-2">
<select
class="select w-full"
- value={current.key}
- onchange={(e) => selectKey(e.currentTarget.value)}
+ value={currentProvider}
+ onchange={(e) => selectProvider(e.currentTarget.value)}
aria-label="Key selector"
>
- {#if keyUnavailable}
+ {#if keyUnavailable && currentProvider !== NAMED_AGENTS_PROVIDER}
<option value={current.key} disabled>{modelKeyLabel(current.key)} (unavailable)</option>
{/if}
- {#each keys as key (key)}
- <option value={key}>{modelKeyLabel(key)}</option>
+ {#each providerKeys as key (key)}
+ <option value={key}>
+ {key === NAMED_AGENTS_PROVIDER ? "Named Agents" : modelKeyLabel(key)}
+ </option>
{/each}
</select>
+
<select
class="select w-full"
- value={current.model}
- onchange={(e) => selectModel(e.currentTarget.value)}
+ value={currentIsNamedAgent ? (namedAgentSnapshot?.namedAgentName ?? "") : current.model}
+ onchange={
+ currentIsNamedAgent
+ ? (e) => selectNamedAgent(e.currentTarget.value)
+ : (e) => selectRawModel(e.currentTarget.value)
+ }
aria-label="Model selector"
+ disabled={currentProvider === NAMED_AGENTS_PROVIDER && namedAgents.length === 0}
>
- {#each keyModels as model (model)}
- <option value={model}>
- {model}{#if isVisionModel(modelInfo, fullNameFor(model))} · vision{/if}
- </option>
- {/each}
- {#if selectedUnavailable && !keyUnavailable}
- <option value={current.model} disabled>{current.model} (unavailable)</option>
+ {#if currentProvider === NAMED_AGENTS_PROVIDER}
+ {#if namedAgents.length === 0 && !currentIsNamedAgent}
+ <option value="" disabled>No named agents configured</option>
+ {:else}
+ {#each namedNames as name (name)}
+ <option value={name}>{namedAgentSelectionLabel(name)}</option>
+ {/each}
+ {#if currentIsNamedAgent && !namedNames.includes(namedAgentSnapshot?.namedAgentName ?? "")}
+ <!-- The selected Named Agent was renamed/removed — keep showing the
+ captured snapshot name rather than silently switching (plan §7). -->
+ <option value={namedAgentSnapshot?.namedAgentName ?? ""} disabled>
+ {namedAgentSelectionLabel(namedAgentSnapshot?.namedAgentName ?? "")} (unavailable)
+ </option>
+ {/if}
+ {/if}
+ {:else}
+ {#each keyModels as model (model)}
+ <option value={model}>
+ {model}{#if isVisionModel(modelInfo, fullNameFor(model))} · vision{/if}
+ </option>
+ {/each}
+ {#if rawMissing && !keyUnavailable}
+ <option value={current.model} disabled>{current.model} (unavailable)</option>
+ {/if}
{/if}
</select>
- {#if selectedUnavailable}
+
+ {#if rawMissing}
<div class="text-xs text-warning" role="status">
Selected model “{selected}” is unavailable — choose another credential.
</div>
{/if}
- {#if selectedVision}
- <div class="flex items-center gap-1 text-xs text-base-content/60">
- <svg
- xmlns="http://www.w3.org/2000/svg"
- viewBox="0 0 24 24"
- fill="none"
- stroke="currentColor"
- stroke-width="2"
- stroke-linecap="round"
- stroke-linejoin="round"
- class="h-3.5 w-3.5"
- aria-hidden="true"
- >
- <path d="M1 12s4-8 11-8 11 8 11 8-4 8-11 8-11-8-11-8z"></path>
- <circle cx="12" cy="12" r="3"></circle>
- </svg>
- <span>Vision — this model sees images natively</span>
- </div>
- {:else}
+
+ {#if currentIsNamedAgent}
<div class="text-xs text-base-content/40">
- Pasted images are auto-described (vision handoff)
+ Selected via <strong>Named Agents</strong> — snapshot of {namedAgentSnapshot?.namedAgentName}.
</div>
{/if}
-</div>
+
+ {#if !currentIsNamedAgent}
+ {#if selectedVision}
+ <div class="flex items-center gap-1 text-xs text-base-content/60">
+ <svg
+ xmlns="http://www.w3.org/2000/svg"
+ viewBox="0 0 24 24"
+ fill="none"
+ stroke="currentColor"
+ stroke-width="2"
+ stroke-linecap="round"
+ stroke-linejoin="round"
+ class="h-3.5 w-3.5"
+ aria-hidden="true"
+ >
+ <path d="M1 12s4-8 11-8 11 8 11 8-4 8-11 8-11-8-11-8z"></path>
+ <circle cx="12" cy="12" r="3"></circle>
+ </svg>
+ <span>Vision — this model sees images natively</span>
+ </div>
+ {:else}
+ <div class="text-xs text-base-content/40">
+ Pasted images are auto-described (vision handoff)
+ </div>
+ {/if}
+ {/if}
+</div> \ No newline at end of file