@happyvertical/smrt-chat 0.51.5 → 0.51.6

package/dist/svelte/components/assistant/create-assistant-dock-controller.svelte.d.ts

@@ -0,0 +1,172 @@
+/**
+ * Headless AssistantDock state (#2904).
+ *
+ * Composes: route-scoped `DataSurfaceDescriptor` discovery from a
+ * `DataSurfaceRegistry` (`@happyvertical/smrt-ui/data-surface`), an
+ * `AssistantTransport` (`./assistant-transport.js`) for thread/message I/O,
+ * and an `AssistantActionClient` for action preview/apply.
+ *
+ * Action preview/apply is NOT routed through `DataSurfaceCommandBridge`
+ * (`../../../data-surface-bridge.js`) — that bridge is the live-collaboration
+ * query/select command channel (see its own tests in
+ * `data-surface-conformance.integration.svelte.test.ts` around `ackBridge`).
+ * The actual action pathway is `DataSurfaceActionAdapter.preview()`/`.apply()`
+ * (`@happyvertical/smrt-agents/server`, exercised directly in
+ * `packages/smrt-svelte/src/web/__tests__/data-surface-conformance.integration.svelte.test.ts:1221-1441`).
+ * That adapter is server-only (it needs a `DataSurfaceExecutionContext` with a
+ * principal/tenant the browser cannot self-assert). `AssistantActionClient` is
+ * the client-side seam a host application implements to reach it — typically
+ * an authenticated HTTP call to a server route that wraps the adapter. Tests
+ * use an in-process implementation that calls the real adapter directly with
+ * a fixed context, the same shape the conformance exemplar uses.
+ *
+ * Polling and stale-send recovery follow the pattern in anytown's
+ * `PortalChatTool.svelte` (read in full for #2904 phase 2):
+ *  - `clientRequestId` is generated once per distinct (threadId, content) draft
+ *    and reused on retry until the send resolves or errors — mirrors
+ *    `clientRequestIdForDraft`/`clearPendingRequest`,
+ *    `PortalChatTool.svelte:304-322`.
+ *  - A still-processing send (`inProgress: true`, mirrors
+ *    `payload.inProgress`, `PortalChatTool.svelte:391`) schedules a poll of
+ *    `loadMessages` rather than blocking — mirrors `scheduleThreadPoll`,
+ *    `PortalChatTool.svelte:339-353` (there: fixed 8 attempts / 1500ms). This
+ *    controller generalizes that into a configurable interval plus an
+ *    absolute staleness timeout (default 90s) after which the pending send is
+ *    marked `'stale'` and the composer can offer "retry", reusing the same
+ *    `clientRequestId` so the transport dedups it (`assistant-transport.ts`
+ *    `createInMemoryAssistantTransport.sendMessage`).
+ */
+import { type DataSurfaceActionRequest, type DataSurfaceActionResult, type DataSurfaceIdentity, type DataSurfaceRegistry } from '@happyvertical/smrt-ui/data-surface';
+import type { AssistantAttachmentRef, AssistantMessage, AssistantThreadSummary, AssistantTransport, ModelOption } from './assistant-transport.js';
+export type AssistantPendingSendStatus = 'sending' | 'processing' | 'stale' | 'failed';
+export interface AssistantPendingSend {
+    clientRequestId: string;
+    threadId: string;
+    content: string;
+    status: AssistantPendingSendStatus;
+    sentAt: number;
+    attachments?: AssistantAttachmentRef[];
+}
+/** Client-side seam to a server-hosted `DataSurfaceActionAdapter`. */
+export interface AssistantActionClient {
+    preview(request: DataSurfaceActionRequest): Promise<DataSurfaceActionResult>;
+    /** `idempotencyKey` is distinct from the send transport's `clientRequestId`
+     * (binding decision #2904) and maps to
+     * `DataSurfaceActionWireRequest.idempotencyKey`
+     * (`packages/types/src/data-surface.ts:274`), which the host's actionClient
+     * implementation attaches when calling the server-side adapter. */
+    apply(request: DataSurfaceActionRequest, idempotencyKey: string): Promise<DataSurfaceActionResult>;
+}
+export interface AssistantActionState {
+    request: DataSurfaceActionRequest;
+    previewResult?: DataSurfaceActionResult;
+    applyResult?: DataSurfaceActionResult;
+    status: 'previewing' | 'previewed' | 'applying' | 'applied' | 'failed';
+    error?: string;
+    /** Minted once when the preview is created and reused for every apply
+     * attempt on this proposed action (including retries after a failure), so
+     * a retried Confirm click after a timeout where the server DID apply
+     * dedups against that earlier attempt instead of re-executing. */
+    idempotencyKey: string;
+    /** Cycle-4 final finding 2: true only when a genuine APPLY attempt itself
+     * failed (the apply-time mount re-check, `actionClient.apply` throwing, or
+     * resolving `{ ok: false }`) — never for a preview-phase failure. This is
+     * the ONLY thing `applyAction`'s guard consults to permit a retry on a
+     * `'failed'` entry; deriving retry eligibility from `status === 'failed'`
+     * plus a mutated `request.phase` (the earlier approach) let a refusal for
+     * an already-terminal state (e.g. `'applied'`) itself manufacture this
+     * condition by downgrading the entry to `'failed'`, permitting a SECOND
+     * `applyAction` call to replay an already-applied action. Cleared
+     * (`false`) whenever an apply attempt succeeds. */
+    retryable?: boolean;
+}
+export interface AssistantDockControllerOptions {
+    transport: AssistantTransport;
+    registry: DataSurfaceRegistry;
+    /** Explicit surface narrowing filter (design decision #2904: `AssistantDock`
+     * accepts `registry` plus an optional `surfaces` override rather than only
+     * ever trusting live discovery). When set, only identities present in
+     * BOTH this list and `registry.list()` are ever mounted (Copilot PR #2919
+     * jAwr0) — an override entry that isn't genuinely registered is never
+     * treated as mounted. */
+    surfaces?: DataSurfaceIdentity[];
+    actionClient?: AssistantActionClient;
+    now?: () => number;
+    createClientRequestId?: () => string;
+    /** Generator for the per-action idempotency key minted at preview time;
+     * defaults to `crypto.randomUUID()`. */
+    createIdempotencyKey?: () => string;
+    /** Poll cadence while a send is in flight. Default 3000ms. */
+    activePollIntervalMs?: number;
+    /** Poll cadence while idle (thread open, no send in flight). Default 15000ms. */
+    idlePollIntervalMs?: number;
+    /** Absolute age after which a pending send is marked stale. Default 90000ms. */
+    staleAfterMs?: number;
+    /** Whether the dock is currently visible; polling pauses when false. */
+    visible?: () => boolean;
+}
+export interface AssistantDockController {
+    readonly threads: AssistantThreadSummary[];
+    readonly activeThreadId: string | null;
+    readonly messages: AssistantMessage[];
+    readonly pendingSends: AssistantPendingSend[];
+    readonly surfaces: DataSurfaceIdentity[];
+    readonly actions: Map<string, AssistantActionState>;
+    readonly models: ModelOption[];
+    readonly selectedModel: string | undefined;
+    /** A background failure (e.g. a failed `loadThreads`/`loadModels` call the
+     * mount effect couldn't surface any other way) for the dock to render
+     * (#2904 review finding 4). `null` when nothing is wrong. Set via
+     * `setError`; a host or the dock's own `handleSend` can also report a
+     * failure here. */
+    readonly error: string | null;
+    /** Records (or clears, with `null`) a background failure for `error` to
+     * report. Distinct from a failed `AssistantPendingSend`/`AssistantActionState`,
+     * which already carry their own `error` field — this is for failures with
+     * no narrower place to live (a failed `loadThreads`/`loadModels`, or a
+     * `send()` rejection AssistantDock's `handleSend` wants surfaced at the
+     * dock level in addition to the pending send's own `'failed'` status). */
+    setError(message: string | null): void;
+    loadThreads(): Promise<void>;
+    loadModels(): Promise<void>;
+    setSelectedModel(modelId: string | undefined): void;
+    openThread(threadId: string): Promise<void>;
+    createThread(title: string): Promise<AssistantThreadSummary>;
+    send(content: string, attachments?: AssistantAttachmentRef[]): Promise<void>;
+    retry(clientRequestId: string): Promise<void>;
+    previewAction(request: DataSurfaceActionRequest): Promise<void>;
+    /** Applies the action using the idempotency key minted at preview time
+     * (`AssistantActionState.idempotencyKey`) — never a fresh key per call. */
+    applyAction(requestId: string): Promise<void>;
+    rejectAction(requestId: string): void;
+    startPolling(): void;
+    stopPolling(): void;
+    /** Re-checks whether the host has reassigned the `registry` prop to a
+     * different instance and, if so, re-subscribes, resyncs `surfaces`
+     * (#2904 review finding B), and clears/reloads conversation state
+     * (threads, activeThreadId, messages, pendingSends, actions, error,
+     * draftIds — Copilot PR #2919 jAwsd). A no-op when unchanged. Call from a
+     * `registry`-scoped effect, never from the mount effect (see
+     * AssistantDock.svelte — F1 requires that one to run exactly once). */
+    syncRegistry(): void;
+    /** Re-reads the `surfaces` override getter (and falls back to the live
+     * registry contents when unset) and invalidates outstanding previews for
+     * any surface that fell out of scope on a narrowing change (#2904 review,
+     * cycle-2 second final finding 1). A no-op when the effective set is
+     * unchanged. Call from a `surfaces`-scoped effect, never from the mount
+     * effect (see AssistantDock.svelte — F1 requires that one to run exactly
+     * once). */
+    syncSurfaces(): void;
+    /** Re-checks whether the host has reassigned the `transport` prop to a
+     * different instance and, if so, clears conversation state (threads,
+     * activeThreadId, messages, pendingSends, actions, error, draftIds) and
+     * reloads threads/models from the new transport (Copilot PR #2919 jAwsd —
+     * mirrors `syncRegistry()`'s reset for the identical class of stale-context
+     * bug). A no-op when unchanged. Call from a `transport`-scoped effect,
+     * never from the mount effect (see AssistantDock.svelte — F1 requires that
+     * one to run exactly once). */
+    syncTransport(): void;
+    dispose(): void;
+}
+export declare function createAssistantDockController(options: AssistantDockControllerOptions): AssistantDockController;
+//# sourceMappingURL=create-assistant-dock-controller.svelte.d.ts.map

package/dist/svelte/components/assistant/create-assistant-dock-controller.svelte.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"create-assistant-dock-controller.svelte.d.ts","sourceRoot":"","sources":["../../../../src/svelte/components/assistant/create-assistant-dock-controller.svelte.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAqCG;AAEH,OAAO,EACL,KAAK,wBAAwB,EAC7B,KAAK,uBAAuB,EAC5B,KAAK,mBAAmB,EACxB,KAAK,mBAAmB,EAEzB,MAAM,qCAAqC,CAAC;AAE7C,OAAO,KAAK,EACV,sBAAsB,EACtB,gBAAgB,EAChB,sBAAsB,EACtB,kBAAkB,EAClB,WAAW,EACZ,MAAM,0BAA0B,CAAC;AAElC,MAAM,MAAM,0BAA0B,GAClC,SAAS,GACT,YAAY,GACZ,OAAO,GACP,QAAQ,CAAC;AAEb,MAAM,WAAW,oBAAoB;IACnC,eAAe,EAAE,MAAM,CAAC;IACxB,QAAQ,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,0BAA0B,CAAC;IACnC,MAAM,EAAE,MAAM,CAAC;IACf,WAAW,CAAC,EAAE,sBAAsB,EAAE,CAAC;CACxC;AAED,sEAAsE;AACtE,MAAM,WAAW,qBAAqB;IACpC,OAAO,CAAC,OAAO,EAAE,wBAAwB,GAAG,OAAO,CAAC,uBAAuB,CAAC,CAAC;IAC7E;;;;uEAImE;IACnE,KAAK,CACH,OAAO,EAAE,wBAAwB,EACjC,cAAc,EAAE,MAAM,GACrB,OAAO,CAAC,uBAAuB,CAAC,CAAC;CACrC;AAED,MAAM,WAAW,oBAAoB;IACnC,OAAO,EAAE,wBAAwB,CAAC;IAClC,aAAa,CAAC,EAAE,uBAAuB,CAAC;IACxC,WAAW,CAAC,EAAE,uBAAuB,CAAC;IACtC,MAAM,EAAE,YAAY,GAAG,WAAW,GAAG,UAAU,GAAG,SAAS,GAAG,QAAQ,CAAC;IACvE,KAAK,CAAC,EAAE,MAAM,CAAC;IACf;;;sEAGkE;IAClE,cAAc,EAAE,MAAM,CAAC;IACvB;;;;;;;;;uDASmD;IACnD,SAAS,CAAC,EAAE,OAAO,CAAC;CACrB;AAED,MAAM,WAAW,8BAA8B;IAC7C,SAAS,EAAE,kBAAkB,CAAC;IAC9B,QAAQ,EAAE,mBAAmB,CAAC;IAC9B;;;;;6BAKyB;IACzB,QAAQ,CAAC,EAAE,mBAAmB,EAAE,CAAC;IACjC,YAAY,CAAC,EAAE,qBAAqB,CAAC;IACrC,GAAG,CAAC,EAAE,MAAM,MAAM,CAAC;IACnB,qBAAqB,CAAC,EAAE,MAAM,MAAM,CAAC;IACrC;4CACwC;IACxC,oBAAoB,CAAC,EAAE,MAAM,MAAM,CAAC;IACpC,8DAA8D;IAC9D,oBAAoB,CAAC,EAAE,MAAM,CAAC;IAC9B,iFAAiF;IACjF,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B,gFAAgF;IAChF,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,wEAAwE;IACxE,OAAO,CAAC,EAAE,MAAM,OAAO,CAAC;CACzB;AAED,MAAM,WAAW,uBAAuB;IACtC,QAAQ,CAAC,OAAO,EAAE,sBAAsB,EAAE,CAAC;IAC3C,QAAQ,CAAC,cAAc,EAAE,MAAM,GAAG,IAAI,CAAC;IACvC,QAAQ,CAAC,QAAQ,EAAE,gBAAgB,EAAE,CAAC;IACtC,QAAQ,CAAC,YAAY,EAAE,oBAAoB,EAAE,CAAC;IAC9C,QAAQ,CAAC,QAAQ,EAAE,mBAAmB,EAAE,CAAC;IACzC,QAAQ,CAAC,OAAO,EAAE,GAAG,CAAC,MAAM,EAAE,oBAAoB,CAAC,CAAC;IACpD,QAAQ,CAAC,MAAM,EAAE,WAAW,EAAE,CAAC;IAC/B,QAAQ,CAAC,aAAa,EAAE,MAAM,GAAG,SAAS,CAAC;IAC3C;;;;uBAImB;IACnB,QAAQ,CAAC,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IAC9B;;;;;8EAK0E;IAC1E,QAAQ,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,GAAG,IAAI,CAAC;IACvC,WAAW,IAAI,OAAO,CAAC,IAAI,CAAC,CAAC;IAC7B,UAAU,IAAI,OAAO,CAAC,IAAI,CAAC,CAAC;IAC5B,gBAAgB,CAAC,OAAO,EAAE,MAAM,GAAG,SAAS,GAAG,IAAI,CAAC;IACpD,UAAU,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IAC5C,YAAY,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC,sBAAsB,CAAC,CAAC;IAC7D,IAAI,CAAC,OAAO,EAAE,MAAM,EAAE,WAAW,CAAC,EAAE,sBAAsB,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IAC7E,KAAK,CAAC,eAAe,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IAC9C,aAAa,CAAC,OAAO,EAAE,wBAAwB,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IAChE;+EAC2E;IAC3E,WAAW,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IAC9C,YAAY,CAAC,SAAS,EAAE,MAAM,GAAG,IAAI,CAAC;IACtC,YAAY,IAAI,IAAI,CAAC;IACrB,WAAW,IAAI,IAAI,CAAC;IACpB;;;;;;2EAMuE;IACvE,YAAY,IAAI,IAAI,CAAC;IACrB;;;;;;gBAMY;IACZ,YAAY,IAAI,IAAI,CAAC;IACrB;;;;;;;mCAO+B;IAC/B,aAAa,IAAI,IAAI,CAAC;IACtB,OAAO,IAAI,IAAI,CAAC;CACjB;AA0BD,wBAAgB,6BAA6B,CAC3C,OAAO,EAAE,8BAA8B,GACtC,uBAAuB,CAyhCzB"}

package/dist/svelte/components/shared/ModelPicker.svelte

@@ -0,0 +1,84 @@
+<script lang="ts">
+/**
+ * ModelPicker - AI model selector
+ *
+ * Extracted from `@happyvertical/smrt-content`'s `ContentAgentChat.svelte`
+ * (its `AI_MODELS` constant + `availableAIModels`/`allowedModelIds` filtering,
+ * `packages/content/src/svelte/components/ContentAgentChat.svelte:28-121`) for
+ * reuse by AssistantDock (#2904). `ContentAgentChat` itself is NOT migrated to
+ * this component in this change — that is a separate, behavior-preserving
+ * follow-up (docs/assistant-dock.md "Gaps").
+ */
+import { Select } from '@happyvertical/smrt-ui/forms';
+import { useI18n } from '@happyvertical/smrt-ui/i18n';
+import { M } from '../../i18n.js';
+
+export interface ModelOption {
+  id: string;
+  label: string;
+}
+
+export const DEFAULT_MODEL_OPTIONS: ModelOption[] = [
+  { id: 'gemini-2.5-pro', label: 'Gemini 2.5 Pro' },
+  { id: 'gemini-2.5-flash', label: 'Gemini 2.5 Flash' },
+  { id: 'claude-3-5-sonnet-latest', label: 'Claude 3.5 Sonnet' },
+  { id: 'claude-3-5-haiku-latest', label: 'Claude 3.5 Haiku' },
+  { id: 'gpt-4o', label: 'GPT-4o' },
+  { id: 'gpt-4o-mini', label: 'GPT-4o Mini' },
+];
+
+export interface Props {
+  /** Full candidate model catalog; defaults to `DEFAULT_MODEL_OPTIONS`. */
+  models?: ModelOption[];
+  /** Allow-list of model ids; empty/omitted means all `models` are offered. */
+  allowedModelIds?: string[];
+  /** The selected model id; bindable. */
+  value: string;
+  /** Fired with the newly selected model id. */
+  onchange?: (modelId: string) => void;
+  /** Accessible name for the underlying `<select>` (#2904 review, cycle-3
+   * second final F2 — the control previously had no aria-label, id, or
+   * wrapping `FormGroup`, so axe/screen readers saw an unnamed combobox).
+   * Defaults to an i18n'd "Model" label; override when a host embeds this
+   * picker somewhere the generic default isn't descriptive enough. */
+  ariaLabel?: string;
+}
+
+let {
+  models = DEFAULT_MODEL_OPTIONS,
+  allowedModelIds = [],
+  value = $bindable(),
+  onchange,
+  ariaLabel,
+}: Props = $props();
+
+const { t } = useI18n();
+
+const availableModels = $derived(
+  allowedModelIds.length > 0
+    ? [
+        ...models.filter((model) => allowedModelIds.includes(model.id)),
+        ...allowedModelIds
+          .filter((id) => !models.some((model) => model.id === id))
+          .map((id) => ({ id, label: id })),
+      ]
+    : models,
+);
+
+function handleChange(event: Event) {
+  const next = (event.target as HTMLSelectElement).value;
+  value = next;
+  onchange?.(next);
+}
+</script>
+
+<Select
+  class="smrt-select model-picker-select"
+  bind:value
+  onchange={handleChange}
+  aria-label={ariaLabel ?? t(M['chat.model_picker.label'])}
+>
+  {#each availableModels as model (model.id)}
+    <option value={model.id}>{model.label}</option>
+  {/each}
+</Select>

package/dist/svelte/components/shared/ModelPicker.svelte.d.ts

@@ -0,0 +1,26 @@
+export interface ModelOption {
+    id: string;
+    label: string;
+}
+export interface Props {
+    /** Full candidate model catalog; defaults to `DEFAULT_MODEL_OPTIONS`. */
+    models?: ModelOption[];
+    /** Allow-list of model ids; empty/omitted means all `models` are offered. */
+    allowedModelIds?: string[];
+    /** The selected model id; bindable. */
+    value: string;
+    /** Fired with the newly selected model id. */
+    onchange?: (modelId: string) => void;
+    /** Accessible name for the underlying `<select>` (#2904 review, cycle-3
+     * second final F2 — the control previously had no aria-label, id, or
+     * wrapping `FormGroup`, so axe/screen readers saw an unnamed combobox).
+     * Defaults to an i18n'd "Model" label; override when a host embeds this
+     * picker somewhere the generic default isn't descriptive enough. */
+    ariaLabel?: string;
+}
+declare const ModelPicker: import("svelte").Component<Props, {
+    DEFAULT_MODEL_OPTIONS: ModelOption[];
+}, "value">;
+type ModelPicker = ReturnType<typeof ModelPicker>;
+export default ModelPicker;
+//# sourceMappingURL=ModelPicker.svelte.d.ts.map

package/dist/svelte/components/shared/ModelPicker.svelte.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"ModelPicker.svelte.d.ts","sourceRoot":"","sources":["../../../../src/svelte/components/shared/ModelPicker.svelte.ts"],"names":[],"mappings":"AAkBA,MAAM,WAAW,WAAW;IAC1B,EAAE,EAAE,MAAM,CAAC;IACX,KAAK,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,KAAK;IACpB,yEAAyE;IACzE,MAAM,CAAC,EAAE,WAAW,EAAE,CAAC;IACvB,6EAA6E;IAC7E,eAAe,CAAC,EAAE,MAAM,EAAE,CAAC;IAC3B,uCAAuC;IACvC,KAAK,EAAE,MAAM,CAAC;IACd,8CAA8C;IAC9C,QAAQ,CAAC,EAAE,CAAC,OAAO,EAAE,MAAM,KAAK,IAAI,CAAC;IACrC;;;;wEAIoE;IACpE,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAsDD,QAAA,MAAM,WAAW;2BAD8G,WAAW,EAAE;WACnF,CAAC;AAC1D,KAAK,WAAW,GAAG,UAAU,CAAC,OAAO,WAAW,CAAC,CAAC;AAClD,eAAe,WAAW,CAAC"}

package/dist/svelte/components/shared/__tests__/ModelPicker.test.js

@@ -0,0 +1,36 @@
+// @vitest-environment jsdom
+/**
+ * Component-level coverage for ModelPicker (#2904 review, cycle-3 second
+ * final F2 — the underlying `<select>` had no accessible name, no test in
+ * this suite ever supplied `models` to any component, and this file had no
+ * test at all).
+ */
+import { expectNoA11yViolations, render, screen, userEvent, } from '@happyvertical/smrt-vitest/svelte';
+import { describe, expect, it, vi } from 'vitest';
+import ModelPicker, {} from '../ModelPicker.svelte';
+const models = [
+    { id: 'model-a', label: 'Model A' },
+    { id: 'model-b', label: 'Model B' },
+];
+describe('ModelPicker', () => {
+    it('has an accessible name by default and lets the user pick a model', async () => {
+        const onchange = vi.fn();
+        render(ModelPicker, { props: { models, value: 'model-a', onchange } });
+        const select = screen.getByLabelText('Model');
+        expect(select).toBeInTheDocument();
+        await userEvent.selectOptions(select, 'model-b');
+        expect(onchange).toHaveBeenCalledWith('model-b');
+    });
+    it('accepts an override accessible name via ariaLabel', () => {
+        render(ModelPicker, {
+            props: { models, value: 'model-a', ariaLabel: 'Reply model' },
+        });
+        expect(screen.getByLabelText('Reply model')).toBeInTheDocument();
+    });
+    it('is axe-clean', async () => {
+        const { container } = render(ModelPicker, {
+            props: { models, value: 'model-a' },
+        });
+        await expectNoA11yViolations(container);
+    });
+});

package/dist/svelte/i18n.d.ts

@@ -18,6 +18,27 @@ export declare const M: {
     readonly 'chat.agent_session_panel.empty': "chat.agent_session_panel.empty";
     readonly 'chat.tool_call_display.tool_call': "chat.tool_call_display.tool_call";
     readonly 'chat.tool_call_display.running': "chat.tool_call_display.running";
+    readonly 'chat.tool_call_display.applied_successfully': "chat.tool_call_display.applied_successfully";
+    readonly 'chat.tool_call_display.confirm': "chat.tool_call_display.confirm";
+    readonly 'chat.tool_call_display.reject': "chat.tool_call_display.reject";
+    readonly 'chat.model_picker.label': "chat.model_picker.label";
+    readonly 'chat.assistant_composer.attach_files': "chat.assistant_composer.attach_files";
+    readonly 'chat.assistant_composer.message_label': "chat.assistant_composer.message_label";
+    readonly 'chat.assistant_composer.send': "chat.assistant_composer.send";
+    readonly 'chat.assistant_composer.remove_attachment': "chat.assistant_composer.remove_attachment";
+    readonly 'chat.assistant_composer.send_error': "chat.assistant_composer.send_error";
+    readonly 'chat.assistant_composer.send_failed': "chat.assistant_composer.send_failed";
+    readonly 'chat.assistant_composer.upload_error': "chat.assistant_composer.upload_error";
+    readonly 'chat.assistant_composer.upload_failed': "chat.assistant_composer.upload_failed";
+    readonly 'chat.assistant_thread_list.conversations_label': "chat.assistant_thread_list.conversations_label";
+    readonly 'chat.assistant_thread_list.new_conversation': "chat.assistant_thread_list.new_conversation";
+    readonly 'chat.assistant_thread_list.untitled': "chat.assistant_thread_list.untitled";
+    readonly 'chat.assistant_dock.no_surfaces': "chat.assistant_dock.no_surfaces";
+    readonly 'chat.assistant_dock.taking_longer': "chat.assistant_dock.taking_longer";
+    readonly 'chat.assistant_dock.retry': "chat.assistant_dock.retry";
+    readonly 'chat.assistant_dock.error': "chat.assistant_dock.error";
+    readonly 'chat.assistant_dock.send_failed': "chat.assistant_dock.send_failed";
+    readonly 'chat.assistant_dock.attachments': "chat.assistant_dock.attachments";
     readonly 'chat.room_create_dialog.close': "chat.room_create_dialog.close";
     readonly 'chat.room_create_dialog.title': "chat.room_create_dialog.title";
     readonly 'chat.room_create_dialog.room_name': "chat.room_create_dialog.room_name";

package/dist/svelte/i18n.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"i18n.d.ts","sourceRoot":"","sources":["../../src/svelte/i18n.ts"],"names":[],"mappings":"AAaA,eAAO,MAAM,CAAC;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CAqEZ,CAAC"}
+{"version":3,"file":"i18n.d.ts","sourceRoot":"","sources":["../../src/svelte/i18n.ts"],"names":[],"mappings":"AAaA,eAAO,MAAM,CAAC;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CAqGZ,CAAC"}

package/dist/svelte/i18n.js

@@ -34,6 +34,31 @@ export const M = defineMessages({
     // ToolCallDisplay
     'chat.tool_call_display.tool_call': 'Tool call: {toolName}',
     'chat.tool_call_display.running': 'Running',
+    'chat.tool_call_display.applied_successfully': 'Change applied successfully.',
+    'chat.tool_call_display.confirm': 'Confirm',
+    'chat.tool_call_display.reject': 'Reject',
+    // ModelPicker (#2904)
+    'chat.model_picker.label': 'Model',
+    // AssistantComposer (#2904)
+    'chat.assistant_composer.attach_files': 'Attach files',
+    'chat.assistant_composer.message_label': 'Message',
+    'chat.assistant_composer.send': 'Send',
+    'chat.assistant_composer.remove_attachment': 'Remove {name}',
+    'chat.assistant_composer.send_error': 'Could not send: {message}',
+    'chat.assistant_composer.send_failed': 'Send failed',
+    'chat.assistant_composer.upload_error': 'Could not attach file: {message}',
+    'chat.assistant_composer.upload_failed': 'Attachment failed',
+    // AssistantThreadList (#2904)
+    'chat.assistant_thread_list.conversations_label': 'Assistant conversations',
+    'chat.assistant_thread_list.new_conversation': '+ New conversation',
+    'chat.assistant_thread_list.untitled': 'Untitled',
+    // AssistantDock (#2904)
+    'chat.assistant_dock.no_surfaces': 'No data surfaces are mounted on this route — the assistant can chat but has no actions available here.',
+    'chat.assistant_dock.taking_longer': 'The assistant is taking longer than expected.',
+    'chat.assistant_dock.retry': 'Retry "{content}"',
+    'chat.assistant_dock.error': 'Something went wrong: {message}',
+    'chat.assistant_dock.send_failed': 'A message failed to send. You can retry it below.',
+    'chat.assistant_dock.attachments': 'Attachments',
     // RoomCreateDialog
     'chat.room_create_dialog.close': 'Close room creation dialog',
     'chat.room_create_dialog.title': 'Create a Room',

package/dist/svelte/index.d.ts

@@ -22,6 +22,11 @@ export { default as AgentChat } from './components/agent/AgentChat.svelte';
 export { default as AgentSelector } from './components/agent/AgentSelector.svelte';
 export { default as AgentSessionPanel } from './components/agent/AgentSessionPanel.svelte';
 export { default as ToolCallDisplay } from './components/agent/ToolCallDisplay.svelte';
+export { default as AssistantComposer } from './components/assistant/AssistantComposer.svelte';
+export { default as AssistantDock } from './components/assistant/AssistantDock.svelte';
+export { default as AssistantThreadList } from './components/assistant/AssistantThreadList.svelte';
+export { type AssistantAttachmentRef, type AssistantMessage, type AssistantSendMessageInput, type AssistantSendMessageResult, type AssistantThreadSummary, type AssistantTransport, createInMemoryAssistantTransport, createSmrtAssistantTransport, type InMemoryAssistantTransportOptions, type SmrtAssistantTransportOptions, } from './components/assistant/assistant-transport.js';
+export { type AssistantActionClient, type AssistantActionState, type AssistantDockController, type AssistantDockControllerOptions, type AssistantPendingSend, type AssistantPendingSendStatus, createAssistantDockController, } from './components/assistant/create-assistant-dock-controller.svelte.js';
 export { default as RoomCreateDialog } from './components/dialogs/RoomCreateDialog.svelte';
 export { default as SearchMessages } from './components/dialogs/SearchMessages.svelte';
 export { default as ChatLayout } from './components/layout/ChatLayout.svelte';
@@ -36,6 +41,8 @@ export { default as Avatar } from './components/shared/Avatar.svelte';
 export { default as FileUpload } from './components/shared/FileUpload.svelte';
 export { default as LinkPreview } from './components/shared/LinkPreview.svelte';
 export { default as MentionAutocomplete } from './components/shared/MentionAutocomplete.svelte';
+export type { ModelOption } from './components/shared/ModelPicker.svelte';
+export { default as ModelPicker } from './components/shared/ModelPicker.svelte';
 export { default as ReactionPicker } from './components/shared/ReactionPicker.svelte';
 export { default as ReadReceipts } from './components/shared/ReadReceipts.svelte';
 export { default as UserPresence } from './components/shared/UserPresence.svelte';

package/dist/svelte/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/svelte/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;GAkBG;AAwCH,OAAO,EAAE,aAAa,EAAE,eAAe,EAAE,MAAM,6BAA6B,CAAC;AAC7E,OAAO,EAAE,OAAO,IAAI,SAAS,EAAE,MAAM,qCAAqC,CAAC;AAC3E,OAAO,EAAE,OAAO,IAAI,aAAa,EAAE,MAAM,yCAAyC,CAAC;AACnF,OAAO,EAAE,OAAO,IAAI,iBAAiB,EAAE,MAAM,6CAA6C,CAAC;AAC3F,OAAO,EAAE,OAAO,IAAI,eAAe,EAAE,MAAM,2CAA2C,CAAC;AACvF,OAAO,EAAE,OAAO,IAAI,gBAAgB,EAAE,MAAM,8CAA8C,CAAC;AAC3F,OAAO,EAAE,OAAO,IAAI,cAAc,EAAE,MAAM,4CAA4C,CAAC;AAEvF,OAAO,EAAE,OAAO,IAAI,UAAU,EAAE,MAAM,uCAAuC,CAAC;AAC9E,OAAO,EAAE,OAAO,IAAI,UAAU,EAAE,MAAM,uCAAuC,CAAC;AAC9E,OAAO,EAAE,OAAO,IAAI,UAAU,EAAE,MAAM,uCAAuC,CAAC;AAC9E,OAAO,EAAE,OAAO,IAAI,QAAQ,EAAE,MAAM,qCAAqC,CAAC;AAC1E,OAAO,EAAE,OAAO,IAAI,YAAY,EAAE,MAAM,2CAA2C,CAAC;AACpF,OAAO,EAAE,OAAO,IAAI,WAAW,EAAE,MAAM,0CAA0C,CAAC;AAClF,OAAO,EAAE,OAAO,IAAI,WAAW,EAAE,MAAM,0CAA0C,CAAC;AAClF,OAAO,EAAE,OAAO,IAAI,WAAW,EAAE,MAAM,0CAA0C,CAAC;AAClF,OAAO,EAAE,OAAO,IAAI,MAAM,EAAE,MAAM,mCAAmC,CAAC;AACtE,OAAO,EAAE,OAAO,IAAI,UAAU,EAAE,MAAM,uCAAuC,CAAC;AAC9E,OAAO,EAAE,OAAO,IAAI,WAAW,EAAE,MAAM,wCAAwC,CAAC;AAChF,OAAO,EAAE,OAAO,IAAI,mBAAmB,EAAE,MAAM,gDAAgD,CAAC;AAChG,OAAO,EAAE,OAAO,IAAI,cAAc,EAAE,MAAM,2CAA2C,CAAC;AACtF,OAAO,EAAE,OAAO,IAAI,YAAY,EAAE,MAAM,yCAAyC,CAAC;AAClF,OAAO,EAAE,OAAO,IAAI,YAAY,EAAE,MAAM,yCAAyC,CAAC;AAClF,OAAO,EAAE,OAAO,IAAI,OAAO,EAAE,MAAM,kCAAkC,CAAC;AACtE,OAAO,EAAE,OAAO,IAAI,WAAW,EAAE,MAAM,sCAAsC,CAAC;AAC9E,OAAO,EAAE,OAAO,IAAI,QAAQ,EAAE,MAAM,mCAAmC,CAAC;AACxE,OAAO,EAAE,OAAO,IAAI,QAAQ,EAAE,MAAM,mCAAmC,CAAC;AAGxE,YAAY,EACV,eAAe,EACf,gBAAgB,EAChB,kBAAkB,EAClB,eAAe,EACf,mBAAmB,EACnB,YAAY,EACZ,YAAY,EACZ,cAAc,EACd,eAAe,EACf,mBAAmB,GACpB,MAAM,YAAY,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/svelte/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;GAkBG;AA8CH,OAAO,EAAE,aAAa,EAAE,eAAe,EAAE,MAAM,6BAA6B,CAAC;AAC7E,OAAO,EAAE,OAAO,IAAI,SAAS,EAAE,MAAM,qCAAqC,CAAC;AAC3E,OAAO,EAAE,OAAO,IAAI,aAAa,EAAE,MAAM,yCAAyC,CAAC;AACnF,OAAO,EAAE,OAAO,IAAI,iBAAiB,EAAE,MAAM,6CAA6C,CAAC;AAC3F,OAAO,EAAE,OAAO,IAAI,eAAe,EAAE,MAAM,2CAA2C,CAAC;AACvF,OAAO,EAAE,OAAO,IAAI,iBAAiB,EAAE,MAAM,iDAAiD,CAAC;AAC/F,OAAO,EAAE,OAAO,IAAI,aAAa,EAAE,MAAM,6CAA6C,CAAC;AACvF,OAAO,EAAE,OAAO,IAAI,mBAAmB,EAAE,MAAM,mDAAmD,CAAC;AACnG,OAAO,EACL,KAAK,sBAAsB,EAC3B,KAAK,gBAAgB,EACrB,KAAK,yBAAyB,EAC9B,KAAK,0BAA0B,EAC/B,KAAK,sBAAsB,EAC3B,KAAK,kBAAkB,EACvB,gCAAgC,EAChC,4BAA4B,EAC5B,KAAK,iCAAiC,EACtC,KAAK,6BAA6B,GACnC,MAAM,+CAA+C,CAAC;AACvD,OAAO,EACL,KAAK,qBAAqB,EAC1B,KAAK,oBAAoB,EACzB,KAAK,uBAAuB,EAC5B,KAAK,8BAA8B,EACnC,KAAK,oBAAoB,EACzB,KAAK,0BAA0B,EAC/B,6BAA6B,GAC9B,MAAM,mEAAmE,CAAC;AAC3E,OAAO,EAAE,OAAO,IAAI,gBAAgB,EAAE,MAAM,8CAA8C,CAAC;AAC3F,OAAO,EAAE,OAAO,IAAI,cAAc,EAAE,MAAM,4CAA4C,CAAC;AAEvF,OAAO,EAAE,OAAO,IAAI,UAAU,EAAE,MAAM,uCAAuC,CAAC;AAC9E,OAAO,EAAE,OAAO,IAAI,UAAU,EAAE,MAAM,uCAAuC,CAAC;AAC9E,OAAO,EAAE,OAAO,IAAI,UAAU,EAAE,MAAM,uCAAuC,CAAC;AAC9E,OAAO,EAAE,OAAO,IAAI,QAAQ,EAAE,MAAM,qCAAqC,CAAC;AAC1E,OAAO,EAAE,OAAO,IAAI,YAAY,EAAE,MAAM,2CAA2C,CAAC;AACpF,OAAO,EAAE,OAAO,IAAI,WAAW,EAAE,MAAM,0CAA0C,CAAC;AAClF,OAAO,EAAE,OAAO,IAAI,WAAW,EAAE,MAAM,0CAA0C,CAAC;AAClF,OAAO,EAAE,OAAO,IAAI,WAAW,EAAE,MAAM,0CAA0C,CAAC;AAClF,OAAO,EAAE,OAAO,IAAI,MAAM,EAAE,MAAM,mCAAmC,CAAC;AACtE,OAAO,EAAE,OAAO,IAAI,UAAU,EAAE,MAAM,uCAAuC,CAAC;AAC9E,OAAO,EAAE,OAAO,IAAI,WAAW,EAAE,MAAM,wCAAwC,CAAC;AAChF,OAAO,EAAE,OAAO,IAAI,mBAAmB,EAAE,MAAM,gDAAgD,CAAC;AAChG,YAAY,EAAE,WAAW,EAAE,MAAM,wCAAwC,CAAC;AAC1E,OAAO,EAAE,OAAO,IAAI,WAAW,EAAE,MAAM,wCAAwC,CAAC;AAChF,OAAO,EAAE,OAAO,IAAI,cAAc,EAAE,MAAM,2CAA2C,CAAC;AACtF,OAAO,EAAE,OAAO,IAAI,YAAY,EAAE,MAAM,yCAAyC,CAAC;AAClF,OAAO,EAAE,OAAO,IAAI,YAAY,EAAE,MAAM,yCAAyC,CAAC;AAClF,OAAO,EAAE,OAAO,IAAI,OAAO,EAAE,MAAM,kCAAkC,CAAC;AACtE,OAAO,EAAE,OAAO,IAAI,WAAW,EAAE,MAAM,sCAAsC,CAAC;AAC9E,OAAO,EAAE,OAAO,IAAI,QAAQ,EAAE,MAAM,mCAAmC,CAAC;AACxE,OAAO,EAAE,OAAO,IAAI,QAAQ,EAAE,MAAM,mCAAmC,CAAC;AAGxE,YAAY,EACV,eAAe,EACf,gBAAgB,EAChB,kBAAkB,EAClB,eAAe,EACf,mBAAmB,EACnB,YAAY,EACZ,YAAY,EACZ,cAAc,EACd,eAAe,EACf,mBAAmB,GACpB,MAAM,YAAY,CAAC"}

package/dist/svelte/index.js

@@ -25,6 +25,10 @@ import AgentChat from './components/agent/AgentChat.svelte';
 import AgentSelector from './components/agent/AgentSelector.svelte';
 import AgentSessionPanel from './components/agent/AgentSessionPanel.svelte';
 import ToolCallDisplay from './components/agent/ToolCallDisplay.svelte';
+// Assistant dock components (#2904)
+import AssistantComposer from './components/assistant/AssistantComposer.svelte';
+import AssistantDock from './components/assistant/AssistantDock.svelte';
+import AssistantThreadList from './components/assistant/AssistantThreadList.svelte';
 // Dialog components
 import RoomCreateDialog from './components/dialogs/RoomCreateDialog.svelte';
 import SearchMessages from './components/dialogs/SearchMessages.svelte';
@@ -43,6 +47,8 @@ import Avatar from './components/shared/Avatar.svelte';
 import FileUpload from './components/shared/FileUpload.svelte';
 import LinkPreview from './components/shared/LinkPreview.svelte';
 import MentionAutocomplete from './components/shared/MentionAutocomplete.svelte';
+// Shared model picker
+import ModelPicker from './components/shared/ModelPicker.svelte';
 import ReactionPicker from './components/shared/ReactionPicker.svelte';
 import ReadReceipts from './components/shared/ReadReceipts.svelte';
 import UserPresence from './components/shared/UserPresence.svelte';
@@ -59,6 +65,11 @@ export { default as AgentChat } from './components/agent/AgentChat.svelte';
 export { default as AgentSelector } from './components/agent/AgentSelector.svelte';
 export { default as AgentSessionPanel } from './components/agent/AgentSessionPanel.svelte';
 export { default as ToolCallDisplay } from './components/agent/ToolCallDisplay.svelte';
+export { default as AssistantComposer } from './components/assistant/AssistantComposer.svelte';
+export { default as AssistantDock } from './components/assistant/AssistantDock.svelte';
+export { default as AssistantThreadList } from './components/assistant/AssistantThreadList.svelte';
+export { createInMemoryAssistantTransport, createSmrtAssistantTransport, } from './components/assistant/assistant-transport.js';
+export { createAssistantDockController, } from './components/assistant/create-assistant-dock-controller.svelte.js';
 export { default as RoomCreateDialog } from './components/dialogs/RoomCreateDialog.svelte';
 export { default as SearchMessages } from './components/dialogs/SearchMessages.svelte';
 // Export components
@@ -74,6 +85,7 @@ export { default as Avatar } from './components/shared/Avatar.svelte';
 export { default as FileUpload } from './components/shared/FileUpload.svelte';
 export { default as LinkPreview } from './components/shared/LinkPreview.svelte';
 export { default as MentionAutocomplete } from './components/shared/MentionAutocomplete.svelte';
+export { default as ModelPicker } from './components/shared/ModelPicker.svelte';
 export { default as ReactionPicker } from './components/shared/ReactionPicker.svelte';
 export { default as ReadReceipts } from './components/shared/ReadReceipts.svelte';
 export { default as UserPresence } from './components/shared/UserPresence.svelte';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@happyvertical/smrt-chat",
-  "version": "0.51.5",
+  "version": "0.51.6",
   "smrtJsdoc": "strict",
   "description": "Chat rooms, DMs, threads, and agent conversations for the SMRT framework",
   "type": "module",
@@ -70,13 +70,13 @@
   },
   "dependencies": {
     "@happyvertical/ai": "^0.89.11",
-    "@happyvertical/smrt-agents": "0.51.5",
-    "@happyvertical/smrt-core": "0.51.5",
-    "@happyvertical/smrt-personas": "0.51.5",
-    "@happyvertical/smrt-tenancy": "0.51.5",
-    "@happyvertical/smrt-types": "0.51.5",
-    "@happyvertical/smrt-ui": "0.51.5",
-    "@happyvertical/smrt-users": "0.51.5",
+    "@happyvertical/smrt-agents": "0.51.6",
+    "@happyvertical/smrt-core": "0.51.6",
+    "@happyvertical/smrt-personas": "0.51.6",
+    "@happyvertical/smrt-tenancy": "0.51.6",
+    "@happyvertical/smrt-types": "0.51.6",
+    "@happyvertical/smrt-ui": "0.51.6",
+    "@happyvertical/smrt-users": "0.51.6",
     "@happyvertical/sql": "^0.89.11"
   },
   "peerDependencies": {
@@ -88,9 +88,9 @@
     }
   },
   "devDependencies": {
-    "@happyvertical/smrt-playground": "0.51.5",
-    "@happyvertical/smrt-profiles": "0.51.5",
-    "@happyvertical/smrt-vitest": "0.51.5",
+    "@happyvertical/smrt-playground": "0.51.6",
+    "@happyvertical/smrt-profiles": "0.51.6",
+    "@happyvertical/smrt-vitest": "0.51.6",
     "@sveltejs/kit": "^2.69.1",
     "@sveltejs/package": "^2.5.8",
     "@sveltejs/vite-plugin-svelte": "^7.1.2",