@agent-native/toolkit 0.9.1 → 0.10.0

package/src/chat-history/ChatHistoryRail.tsx

@@ -1,11 +1,17 @@
 import { IconDots, IconPlus } from "@tabler/icons-react";
-import { useEffect, useMemo, useState } from "react";
+import type { ReactNode } from "react";
 
+import { ActionButton, IconButton } from "../design-system/components.js";
+import { DesignSystemErrorBoundary } from "../design-system/error-boundary.js";
 import {
   ChatHistoryList,
   type ChatHistoryItem,
   type ChatHistoryListProps,
 } from "./ChatHistoryList.js";
+import {
+  type ChatHistoryRailController,
+  useChatHistoryRailController,
+} from "./useChatHistoryRailController.js";
 
 export interface ChatHistoryRailLabels {
   newChat: string;
@@ -22,63 +28,71 @@ export interface ChatHistoryRailProps extends Omit<
   railLabels: ChatHistoryRailLabels;
   previewCount?: number;
   expandedCount?: number;
+  /**
+   * Product-level presentation slot for design systems that want to replace
+   * the rail wholesale. State and actions still come from the same controller
+   * as the default view.
+   */
+  renderRail?: (context: ChatHistoryRailRenderContext) => ReactNode;
 }
 
-/**
- * Compact recent-chat rail for app sidebars. Hosts own thread persistence,
- * sorting, routing, and mutations; the rail only owns progressive disclosure.
- */
-export function ChatHistoryRail({
-  items,
-  onNewChat,
-  railLabels,
-  previewCount = 5,
-  expandedCount = 15,
-  className,
-  emptyLabel,
-  ...listProps
-}: ChatHistoryRailProps) {
-  const [expanded, setExpanded] = useState(false);
-  const collapsedLimit = Math.max(1, previewCount);
-  const expandedLimit = Math.max(collapsedLimit, expandedCount);
-  const canExpand = items.length > collapsedLimit;
-  const visibleItems = useMemo(
-    () => items.slice(0, expanded ? expandedLimit : collapsedLimit),
-    [collapsedLimit, expanded, expandedLimit, items],
-  );
+export interface ChatHistoryRailRenderContext {
+  controller: ChatHistoryRailController<ChatHistoryItem>;
+  listProps: Omit<
+    ChatHistoryListProps,
+    "footer" | "items" | "sections" | "variant"
+  >;
+}
 
-  useEffect(() => {
-    if (!canExpand) setExpanded(false);
-  }, [canExpand]);
+export type DefaultChatHistoryRailViewProps = ChatHistoryRailRenderContext;
+
+export function DefaultChatHistoryRailView({
+  controller,
+  listProps,
+}: ChatHistoryRailRenderContext) {
+  const {
+    canExpand,
+    disclosureLabel,
+    expanded,
+    newChatLabel,
+    onNewChat,
+    toggleExpanded,
+    visibleItems,
+  } = controller;
+  const { className, emptyLabel, ...rest } = listProps;
 
   const footer = (
     <div className="an-chat-history-rail__footer">
-      <button
+      <ActionButton
         type="button"
         className="an-chat-history-rail__new-chat"
-        onClick={onNewChat}
+        emphasis="ghost"
+        size="compact"
+        leadingIcon={
+          <IconPlus size={13} strokeWidth={1.8} aria-hidden="true" />
+        }
+        onPress={onNewChat}
       >
-        <IconPlus size={13} strokeWidth={1.8} aria-hidden="true" />
-        <span>{railLabels.newChat}</span>
-      </button>
+        <span>{newChatLabel}</span>
+      </ActionButton>
       {canExpand && (
-        <button
+        <IconButton
           type="button"
           className="an-chat-history-rail__disclosure"
-          onClick={() => setExpanded((current) => !current)}
+          size="compact"
+          icon={<IconDots size={14} strokeWidth={1.8} aria-hidden="true" />}
+          onPress={toggleExpanded}
           aria-expanded={expanded}
-          aria-label={expanded ? railLabels.showLess : railLabels.showMore}
-          title={expanded ? railLabels.showLess : railLabels.showMore}
-        >
-          <IconDots size={14} strokeWidth={1.8} aria-hidden="true" />
-        </button>
+          label={disclosureLabel}
+          title={disclosureLabel}
+        />
       )}
     </div>
   );
 
   return (
     <ChatHistoryList
-      {...listProps}
+      {...rest}
       items={visibleItems}
       footer={footer}
       emptyLabel={emptyLabel ?? null}
@@ -87,3 +101,50 @@ export function ChatHistoryRail({
     />
   );
 }
+
+/**
+ * Compact recent-chat rail for app sidebars. Hosts own thread persistence,
+ * sorting, routing, and mutations; the rail only owns progressive disclosure.
+ */
+export function ChatHistoryRail({
+  items,
+  onNewChat,
+  railLabels,
+  previewCount = 5,
+  expandedCount = 15,
+  renderRail,
+  className,
+  emptyLabel,
+  ...listProps
+}: ChatHistoryRailProps) {
+  const controller = useChatHistoryRailController({
+    items,
+    onNewChat,
+    labels: railLabels,
+    previewCount,
+    expandedCount,
+  });
+  const context: ChatHistoryRailRenderContext = {
+    controller,
+    listProps: { ...listProps, className, emptyLabel },
+  };
+  const fallback = <DefaultChatHistoryRailView {...context} />;
+
+  return renderRail ? (
+    <DesignSystemErrorBoundary component="ChatHistoryRail" fallback={fallback}>
+      <ChatHistoryRailCustomView renderRail={renderRail} context={context} />
+    </DesignSystemErrorBoundary>
+  ) : (
+    fallback
+  );
+}
+
+function ChatHistoryRailCustomView({
+  renderRail,
+  context,
+}: {
+  renderRail: NonNullable<ChatHistoryRailProps["renderRail"]>;
+  context: ChatHistoryRailRenderContext;
+}) {
+  return renderRail(context);
+}

package/src/chat-history/index.ts

@@ -7,6 +7,15 @@ export {
 } from "./ChatHistoryList.js";
 export {
   ChatHistoryRail,
+  DefaultChatHistoryRailView,
+  type DefaultChatHistoryRailViewProps,
   type ChatHistoryRailLabels,
   type ChatHistoryRailProps,
+  type ChatHistoryRailRenderContext,
 } from "./ChatHistoryRail.js";
+export {
+  type ChatHistoryRailController,
+  type ChatHistoryRailControllerLabels,
+  type UseChatHistoryRailControllerOptions,
+  useChatHistoryRailController,
+} from "./useChatHistoryRailController.js";

package/src/chat-history/useChatHistoryRailController.spec.tsx

@@ -0,0 +1,131 @@
+// @vitest-environment happy-dom
+
+import { act } from "react";
+import { createRoot, type Root } from "react-dom/client";
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+
+import {
+  type ChatHistoryRailController,
+  type UseChatHistoryRailControllerOptions,
+  useChatHistoryRailController,
+} from "./useChatHistoryRailController.js";
+
+interface Item {
+  id: string;
+}
+
+const labels = {
+  newChat: "New chat",
+  showMore: "Show more chats",
+  showLess: "Show fewer chats",
+};
+
+function makeItems(count: number): Item[] {
+  return Array.from({ length: count }, (_, index) => ({
+    id: `thread-${index + 1}`,
+  }));
+}
+
+describe("useChatHistoryRailController", () => {
+  let container: HTMLDivElement;
+  let root: Root;
+  let controller: ChatHistoryRailController<Item>;
+
+  function Harness(props: UseChatHistoryRailControllerOptions<Item>) {
+    controller = useChatHistoryRailController(props);
+    return null;
+  }
+
+  function render(options: Partial<UseChatHistoryRailControllerOptions<Item>>) {
+    const props: UseChatHistoryRailControllerOptions<Item> = {
+      items: makeItems(20),
+      labels,
+      onNewChat: () => {},
+      ...options,
+    };
+    act(() => root.render(<Harness {...props} />));
+  }
+
+  beforeEach(() => {
+    vi.stubGlobal("IS_REACT_ACT_ENVIRONMENT", true);
+    container = document.createElement("div");
+    document.body.appendChild(container);
+    root = createRoot(container);
+  });
+
+  afterEach(() => {
+    act(() => root.unmount());
+    container.remove();
+    vi.restoreAllMocks();
+    vi.unstubAllGlobals();
+  });
+
+  it("progressively discloses the default five and fifteen item limits", () => {
+    const items = makeItems(20);
+    render({ items });
+
+    expect(controller.expanded).toBe(false);
+    expect(controller.collapsedLimit).toBe(5);
+    expect(controller.expandedLimit).toBe(15);
+    expect(controller.visibleItems).toEqual(items.slice(0, 5));
+    expect(controller.canExpand).toBe(true);
+    expect(controller.disclosureLabel).toBe("Show more chats");
+
+    act(() => controller.toggleExpanded());
+
+    expect(controller.expanded).toBe(true);
+    expect(controller.visibleItems).toEqual(items.slice(0, 15));
+    expect(controller.disclosureLabel).toBe("Show fewer chats");
+  });
+
+  it("clamps finite limits while preserving legacy disclosure behavior", () => {
+    render({ previewCount: -2.5, expandedCount: 1.9 });
+
+    expect(controller.collapsedLimit).toBe(1);
+    expect(controller.expandedLimit).toBe(1);
+    expect(controller.visibleItems).toHaveLength(1);
+    expect(controller.canExpand).toBe(true);
+
+    act(() => controller.toggleExpanded());
+    expect(controller.expanded).toBe(true);
+  });
+
+  it("uses defaults for non-finite limits and never puts expanded below preview", () => {
+    render({
+      previewCount: Number.NaN,
+      expandedCount: Number.POSITIVE_INFINITY,
+    });
+    expect(controller.collapsedLimit).toBe(5);
+    expect(controller.expandedLimit).toBe(15);
+
+    render({ previewCount: 8, expandedCount: 3 });
+    expect(controller.collapsedLimit).toBe(8);
+    expect(controller.expandedLimit).toBe(8);
+  });
+
+  it("resets disclosure explicitly and when the collection becomes too short", () => {
+    render({});
+    act(() => controller.toggleExpanded());
+    expect(controller.expanded).toBe(true);
+
+    act(() => controller.resetExpanded());
+    expect(controller.expanded).toBe(false);
+
+    act(() => controller.toggleExpanded());
+    expect(controller.expanded).toBe(true);
+
+    render({ items: makeItems(3) });
+    expect(controller.expanded).toBe(false);
+    expect(controller.canExpand).toBe(false);
+    expect(controller.visibleItems).toHaveLength(3);
+  });
+
+  it("exposes view labels and the host new-chat action without UI assumptions", () => {
+    const onNewChat = vi.fn();
+    render({ onNewChat });
+
+    expect(controller.newChatLabel).toBe("New chat");
+    act(() => controller.onNewChat());
+    expect(onNewChat).toHaveBeenCalledOnce();
+  });
+});

package/src/chat-history/useChatHistoryRailController.ts

@@ -0,0 +1,76 @@
+import { useCallback, useEffect, useMemo, useState } from "react";
+
+export interface ChatHistoryRailControllerLabels {
+  newChat: string;
+  showMore: string;
+  showLess: string;
+}
+
+export interface UseChatHistoryRailControllerOptions<Item> {
+  items: readonly Item[];
+  onNewChat: () => void;
+  labels: ChatHistoryRailControllerLabels;
+  previewCount?: number;
+  expandedCount?: number;
+}
+
+export interface ChatHistoryRailController<Item> {
+  expanded: boolean;
+  collapsedLimit: number;
+  expandedLimit: number;
+  visibleItems: Item[];
+  canExpand: boolean;
+  newChatLabel: string;
+  disclosureLabel: string;
+  onNewChat: () => void;
+  toggleExpanded: () => void;
+  resetExpanded: () => void;
+}
+
+function normalizeLimit(value: number | undefined, fallback: number): number {
+  if (value === undefined || !Number.isFinite(value)) return fallback;
+  return Math.max(1, Math.floor(value));
+}
+
+export function useChatHistoryRailController<Item>({
+  items,
+  onNewChat,
+  labels,
+  previewCount,
+  expandedCount,
+}: UseChatHistoryRailControllerOptions<Item>): ChatHistoryRailController<Item> {
+  const [expandedState, setExpandedState] = useState(false);
+  const collapsedLimit = normalizeLimit(previewCount, 5);
+  const expandedLimit = Math.max(
+    collapsedLimit,
+    normalizeLimit(expandedCount, 15),
+  );
+  const canExpand = items.length > collapsedLimit;
+  const expanded = canExpand && expandedState;
+  const visibleItems = useMemo(
+    () => items.slice(0, expanded ? expandedLimit : collapsedLimit),
+    [collapsedLimit, expanded, expandedLimit, items],
+  );
+
+  useEffect(() => {
+    if (!canExpand) setExpandedState(false);
+  }, [canExpand]);
+
+  const toggleExpanded = useCallback(() => {
+    if (canExpand) setExpandedState((current) => !current);
+  }, [canExpand]);
+  const resetExpanded = useCallback(() => setExpandedState(false), []);
+
+  return {
+    expanded,
+    collapsedLimit,
+    expandedLimit,
+    visibleItems,
+    canExpand,
+    newChatLabel: labels.newChat,
+    disclosureLabel: expanded ? labels.showLess : labels.showMore,
+    onNewChat,
+    toggleExpanded,
+    resetExpanded,
+  };
+}

package/src/chat-history.css

@@ -113,12 +113,14 @@
   justify-content: center;
   flex: 0 0 auto;
   width: 30px;
+  height: auto;
   min-height: 30px;
 }
 
 .an-chat-history-rail__new-chat {
   flex: 1 1 auto;
   gap: 8px;
+  height: auto;
   min-width: 0;
   min-height: 30px;
   padding: 6px 10px;

package/src/composer/ComposerPlusMenu.tsx

@@ -490,14 +490,17 @@ function ComposerPlusMenuFull({
       <Popover open={open} onOpenChange={setOpen}>
         <Tooltip>
           <TooltipTrigger asChild>
-            <PopoverTrigger asChild>
-              <button
-                type="button"
-                className="shrink-0 flex h-7 w-7 items-center justify-center rounded-md text-muted-foreground hover:text-foreground hover:bg-accent/50 disabled:opacity-30 disabled:cursor-not-allowed"
-              >
-                <IconPlus className="h-4 w-4" />
-              </button>
-            </PopoverTrigger>
+            <span className="inline-flex shrink-0">
+              <PopoverTrigger asChild>
+                <button
+                  type="button"
+                  aria-label="Add..."
+                  className="shrink-0 flex h-7 w-7 items-center justify-center rounded-md text-muted-foreground hover:text-foreground hover:bg-accent/50 disabled:opacity-30 disabled:cursor-not-allowed"
+                >
+                  <IconPlus className="h-4 w-4" />
+                </button>
+              </PopoverTrigger>
+            </span>
           </TooltipTrigger>
           <TooltipContent>Add...</TooltipContent>
         </Tooltip>

package/src/composer/PromptComposer.tsx

@@ -519,6 +519,7 @@ function PromptComposerInner({
     resolvedModelStatusChecksEnabled,
   );
   const missingApiKey = agentEngineConfigured.missing;
+  const agentEngineUnavailable = agentEngineConfigured.state === "unavailable";
   const [missingKeyBouncePulse, setMissingKeyBouncePulse] = useState(0);
   const bounceMissingKeySetup = useCallback(() => {
     setMissingKeyBouncePulse((pulse) => pulse + 1);
@@ -542,8 +543,13 @@ function PromptComposerInner({
               timeoutMs: SUBMIT_ENGINE_STATUS_TIMEOUT_MS,
             },
           );
-    if (state !== "missing") return true;
-    bounceMissingKeySetup();
+    if (state === "missing") {
+      bounceMissingKeySetup();
+      return false;
+    }
+    if (typeof window !== "undefined") {
+      window.dispatchEvent(new Event("agent-engine:configured-changed"));
+    }
     return false;
   }, [
     agentEngineConfigured.state,
@@ -551,6 +557,11 @@ function PromptComposerInner({
     modelsAdapter,
     resolvedModelStatusChecksEnabled,
   ]);
+  const retryAgentEngineStatus = useCallback(() => {
+    if (typeof window !== "undefined") {
+      window.dispatchEvent(new Event("agent-engine:configured-changed"));
+    }
+  }, []);
 
   useEffect(() => {
     if (!autoFocus) return;
@@ -613,21 +624,31 @@ function PromptComposerInner({
       <AgentComposerFrame
         className={cn(
           "text-start",
-          missingApiKey && "cursor-pointer",
+          (missingApiKey || agentEngineUnavailable) && "cursor-pointer",
           className,
         )}
         rootClassName={rootClassName}
         style={style}
         rootStyle={rootStyle}
         layoutVariant={layoutVariant}
-        onClick={missingApiKey ? bounceMissingKeySetup : undefined}
+        onClick={
+          missingApiKey
+            ? bounceMissingKeySetup
+            : agentEngineUnavailable
+              ? retryAgentEngineStatus
+              : undefined
+        }
       >
         <PromptAttachmentStrip />
         <TiptapComposer
           focusRef={handleRef}
-          disabled={disabled || missingApiKey}
+          disabled={disabled || missingApiKey || agentEngineUnavailable}
           placeholder={
-            missingApiKey ? "Connect AI above to continue..." : placeholder
+            missingApiKey
+              ? "Connect AI above to continue..."
+              : agentEngineUnavailable
+                ? "Unable to check AI connection. Click to retry."
+                : placeholder
           }
           initialText={initialText}
           initialTextKey={initialTextKey}

package/src/composer/useRealtimeVoiceMode.spec.ts

@@ -6,6 +6,7 @@ import {
   createRealtimeVoiceGreetingEvent,
   createRealtimeVoiceGreetingStarter,
   createRealtimeVoicePreferenceUpdate,
+  createRealtimeVoiceResponseCoordinator,
   createRealtimeVoiceSession,
   createRealtimeVoiceSessionWithCapability,
   createRealtimeVoiceTranscriptSequencer,
@@ -521,7 +522,7 @@ describe("Realtime voice dynamic tool manifests", () => {
 });
 
 describe("extractRealtimeVoiceFunctionCalls", () => {
-  it("uses the low-latency completed-arguments event", () => {
+  it("waits for the terminal response event before returning a function call", () => {
     expect(
       extractRealtimeVoiceFunctionCalls({
         type: "response.function_call_arguments.done",
@@ -529,13 +530,7 @@ describe("extractRealtimeVoiceFunctionCalls", () => {
         call_id: "call-1",
         arguments: '{"path":"/inbox"}',
       }),
-    ).toEqual([
-      {
-        name: "navigate",
-        callId: "call-1",
-        argumentsText: '{"path":"/inbox"}',
-      },
-    ]);
+    ).toEqual([]);
   });
 
   it("falls back to completed function items on response.done", () => {
@@ -543,6 +538,7 @@ describe("extractRealtimeVoiceFunctionCalls", () => {
       extractRealtimeVoiceFunctionCalls({
         type: "response.done",
         response: {
+          status: "completed",
           output: [
             { type: "message", role: "assistant" },
             {
@@ -562,6 +558,29 @@ describe("extractRealtimeVoiceFunctionCalls", () => {
       },
     ]);
   });
+
+  it("ignores function items from failed or cancelled responses", () => {
+    const output = [
+      {
+        type: "function_call",
+        name: "navigate",
+        call_id: "call-3",
+        arguments: "{}",
+      },
+    ];
+    expect(
+      extractRealtimeVoiceFunctionCalls({
+        type: "response.done",
+        response: { status: "failed", output },
+      }),
+    ).toEqual([]);
+    expect(
+      extractRealtimeVoiceFunctionCalls({
+        type: "response.done",
+        response: { status: "cancelled", output },
+      }),
+    ).toEqual([]);
+  });
 });
 
 describe("extractCompletedRealtimeVoiceTranscript", () => {
@@ -632,6 +651,61 @@ describe("Realtime voice startup and transcript ordering", () => {
     expect(send).toHaveBeenCalledTimes(2);
   });
 
+  it("queues response requests behind the active response and retries a rejected request", () => {
+    const sent: Record<string, unknown>[] = [];
+    const coordinator = createRealtimeVoiceResponseCoordinator((event) =>
+      sent.push(event),
+    );
+
+    coordinator.request();
+    coordinator.handleEvent({
+      type: "response.created",
+      response: { id: "resp-active" },
+    });
+    coordinator.request();
+    expect(sent).toHaveLength(1);
+
+    expect(
+      coordinator.handleError(
+        "Conversation already has an active response in progress: resp-active",
+      ),
+    ).toBe(true);
+    coordinator.handleEvent({
+      type: "response.done",
+      response: { id: "resp-active", status: "completed" },
+    });
+
+    expect(sent).toHaveLength(2);
+    expect(sent[1]).toEqual({ type: "response.create" });
+  });
+
+  it("drops queued work after a failed response so the next turn can start", () => {
+    const sent: Record<string, unknown>[] = [];
+    const coordinator = createRealtimeVoiceResponseCoordinator((event) =>
+      sent.push(event),
+    );
+
+    coordinator.request();
+    coordinator.handleEvent({ type: "response.created" });
+    coordinator.request();
+    coordinator.handleEvent({
+      type: "response.done",
+      response: { status: "failed" },
+    });
+
+    expect(sent).toHaveLength(1);
+    coordinator.request();
+    expect(sent).toHaveLength(2);
+  });
+
+  it("ignores unrelated realtime errors", () => {
+    const coordinator = createRealtimeVoiceResponseCoordinator(vi.fn());
+
+    expect(coordinator.handleError("The microphone was disconnected.")).toBe(
+      false,
+    );
+  });
+
   it("publishes in conversation order when user ASR finishes after the assistant", () => {
     const published: Array<{ role: string; text: string }> = [];
     const sequencer = createRealtimeVoiceTranscriptSequencer((transcript) => {