npm - @brainpilot/web - Versions diffs - 0.0.4 → 0.0.6 - Mend

@brainpilot/web 0.0.4 → 0.0.6

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (114) hide show

package/dist/assets/index-Br55rkHb.css +1 -0
package/dist/assets/index-CeUzk-ej.js +445 -0
package/dist/index.html +2 -2
package/index.html +13 -0
package/package.json +12 -3
package/src/App.tsx +10 -0
package/src/__tests__/agentsReducer.test.ts +67 -0
package/src/__tests__/api.test.ts +221 -0
package/src/__tests__/chatScrollMemory.test.ts +49 -0
package/src/__tests__/demoConversation.test.ts +73 -0
package/src/__tests__/demoReset.test.ts +24 -0
package/src/__tests__/messageGroups.test.ts +80 -0
package/src/__tests__/newUiComponents.test.tsx +101 -0
package/src/__tests__/newUiEvents.test.ts +236 -0
package/src/__tests__/runningToast.test.ts +29 -0
package/src/__tests__/tokenUsage.test.ts +48 -0
package/src/__tests__/toolDisplay.test.ts +55 -0
package/src/__tests__/traceReducer.test.ts +62 -0
package/src/components/chat/AskUserCard.tsx +123 -0
package/src/components/chat/AutoRetryIndicator.tsx +71 -0
package/src/components/chat/ComposerInput.tsx +73 -0
package/src/components/chat/ComposerSendButton.tsx +26 -0
package/src/components/chat/MarkdownMessage.tsx +24 -0
package/src/components/chat/MessageStream.tsx +505 -0
package/src/components/chat/PromptComposer.tsx +489 -0
package/src/components/chat/SystemMessageBubble.tsx +46 -0
package/src/components/chat/chatScrollMemory.ts +49 -0
package/src/components/demo/DemoFileTree.tsx +146 -0
package/src/components/demo/DemoView.tsx +730 -0
package/src/components/demo/TraceNodeModal.tsx +80 -0
package/src/components/demo/demoBundle.ts +223 -0
package/src/components/demo/demoCache.ts +42 -0
package/src/components/demo/demoReset.ts +16 -0
package/src/components/files/FilePreviewView.tsx +153 -0
package/src/components/files/FileSidebar.tsx +664 -0
package/src/components/files/filePreview.ts +113 -0
package/src/components/primitives/CustomSelect.tsx +200 -0
package/src/components/primitives/IconButton.tsx +27 -0
package/src/components/quota/DiskQuotaCriticalDialog.tsx +56 -0
package/src/components/quota/DiskQuotaWarningDialog.tsx +65 -0
package/src/components/quota/QuotaFileManager.tsx +197 -0
package/src/components/search/SearchDialog.tsx +101 -0
package/src/components/session/AgentNetwork.tsx +1233 -0
package/src/components/session/AgentTraceViews.tsx +346 -0
package/src/components/session/AnalyticsTab.tsx +220 -0
package/src/components/session/GlobalOverview.tsx +108 -0
package/src/components/session/NodeTooltip.tsx +127 -0
package/src/components/session/TimelineTab.tsx +320 -0
package/src/components/session/TraceGraphView.tsx +307 -0
package/src/components/session/TraceNodeDetail.tsx +179 -0
package/src/components/session/agentAnalytics.ts +397 -0
package/src/components/session/agentNetworkShared.ts +339 -0
package/src/components/session/traceLayout.ts +182 -0
package/src/components/settings/SettingsDialog.tsx +737 -0
package/src/components/shell/DesktopShell.tsx +261 -0
package/src/components/shell/SandboxBuildingOverlay.tsx +73 -0
package/src/components/shell/SandboxStatus.tsx +287 -0
package/src/components/shell/TerminalDrawer.tsx +387 -0
package/src/components/sidebar/Sidebar.tsx +191 -0
package/src/config.ts +10 -0
package/src/contexts/AppProviders.tsx +20 -0
package/src/contexts/AuthContext.tsx +61 -0
package/src/contexts/PreferencesContext.tsx +125 -0
package/src/contexts/SSEContext.tsx +264 -0
package/src/contexts/SandboxContext.tsx +310 -0
package/src/contexts/SessionContext.tsx +919 -0
package/src/contexts/agentsReducer.ts +49 -0
package/src/contexts/draftStore.ts +103 -0
package/src/contexts/messageFilters.ts +29 -0
package/src/contexts/messageGroups.ts +77 -0
package/src/contexts/messageReducer.ts +401 -0
package/src/contexts/newUiEvents.ts +190 -0
package/src/contexts/runningToast.ts +33 -0
package/src/contexts/traceReducer.ts +62 -0
package/src/contexts/turnTimer.test.ts +97 -0
package/src/contexts/turnTimer.ts +108 -0
package/src/contexts/useTurnTimer.ts +104 -0
package/src/contracts/backend.ts +897 -0
package/src/contracts/demoBundle.ts +83 -0
package/src/i18n/messages/analytics.ts +106 -0
package/src/i18n/messages/chat.ts +130 -0
package/src/i18n/messages/contexts.ts +42 -0
package/src/i18n/messages/demo.ts +80 -0
package/src/i18n/messages/files.ts +82 -0
package/src/i18n/messages/network.ts +190 -0
package/src/i18n/messages/profile.ts +44 -0
package/src/i18n/messages/quota.ts +36 -0
package/src/i18n/messages/sandbox.ts +116 -0
package/src/i18n/messages/search.ts +16 -0
package/src/i18n/messages/settings.ts +188 -0
package/src/i18n/messages/shell.ts +38 -0
package/src/i18n/messages/sidebar.ts +52 -0
package/src/i18n/messages/terminal.ts +22 -0
package/src/i18n/messages/trace.ts +136 -0
package/src/i18n/messages.ts +32 -0
package/src/i18n/translate.ts +46 -0
package/src/i18n/types.ts +15 -0
package/src/i18n/useT.ts +15 -0
package/src/main.tsx +13 -0
package/src/mocks/backend.ts +729 -0
package/src/styles/global.css +7578 -0
package/src/styles/tokens.css +161 -0
package/src/utils/api.ts +724 -0
package/src/utils/download.ts +18 -0
package/src/utils/format.ts +7 -0
package/src/utils/toolDisplay.ts +74 -0
package/src/utils/zip.ts +119 -0
package/src/vite-env.d.ts +1 -0
package/tsconfig.app.json +22 -0
package/tsconfig.json +7 -0
package/tsconfig.node.json +13 -0
package/vite.config.ts +13 -0
package/dist/assets/index-Cd0Mi_WU.css +0 -1
package/dist/assets/index-FGg-DeYR.js +0 -448

package/dist/index.html CHANGED Viewed

@@ -5,8 +5,8 @@
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <meta name="color-scheme" content="light dark" />
     <title>BrainPilot</title>
-    <script type="module" crossorigin src="/assets/index-FGg-DeYR.js"></script>
-    <link rel="stylesheet" crossorigin href="/assets/index-Cd0Mi_WU.css">
+    <script type="module" crossorigin src="/assets/index-CeUzk-ej.js"></script>
+    <link rel="stylesheet" crossorigin href="/assets/index-Br55rkHb.css">
   </head>
   <body>
     <div id="root"></div>

package/index.html ADDED Viewed

@@ -0,0 +1,13 @@
+<!doctype html>
+<html lang="en">
+  <head>
+    <meta charset="UTF-8" />
+    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
+    <meta name="color-scheme" content="light dark" />
+    <title>BrainPilot</title>
+  </head>
+  <body>
+    <div id="root"></div>
+    <script type="module" src="/src/main.tsx"></script>
+  </body>
+</html>

package/package.json CHANGED Viewed

@@ -1,10 +1,19 @@
 {
   "name": "@brainpilot/web",
-  "version": "0.0.4",
+  "version": "0.0.6",
+  "engines": {
+    "node": ">=22"
+  },
   "license": "Apache-2.0",
   "type": "module",
   "files": [
-    "dist"
+    "dist",
+    "src",
+    "index.html",
+    "vite.config.ts",
+    "tsconfig.json",
+    "tsconfig.app.json",
+    "tsconfig.node.json"
   ],
   "repository": {
     "type": "git",
@@ -22,7 +31,7 @@
   },
   "dependencies": {},
   "devDependencies": {
-    "@brainpilot/protocol": "^0.0.4",
+    "@brainpilot/protocol": "^0.0.6",
     "@fontsource-variable/geist": "^5.2.9",
     "@fontsource-variable/geist-mono": "^5.2.8",
     "@types/react": "^18.3.12",

package/src/App.tsx ADDED Viewed

@@ -0,0 +1,10 @@
+import { DesktopShell } from "./components/shell/DesktopShell";
+import { AppProviders } from "./contexts/AppProviders";
+export function App() {
+  return (
+    <AppProviders>
+      <DesktopShell />
+    </AppProviders>
+  );
+}

package/src/__tests__/agentsReducer.test.ts ADDED Viewed

@@ -0,0 +1,67 @@
+import { describe, it, expect } from "vitest";
+import { reduceAgentsForEvent } from "../contexts/agentsReducer";
+import type { AgentStatus, WebSocketEvent } from "../contracts/backend";
+// #70: events arrive post-normalizeAgUiEvent (camelCase). agent_status_update's
+// authoritative agent key is the top-level `name` field; status is one of
+// idle|running|error|stopped (retrying normalized to running).
+const ev = (name: string, status: string): WebSocketEvent =>
+  ({ type: "agent_status_update", name, status } as WebSocketEvent);
+describe("reduceAgentsForEvent (#70)", () => {
+  it("appends an unknown agent with empty task", () => {
+    const out = reduceAgentsForEvent([], ev("librarian", "running"));
+    expect(out).toHaveLength(1);
+    expect(out[0]).toMatchObject({ name: "librarian", status: "running", task: "", alive: true });
+  });
+  it("updates a known agent's status but preserves its task", () => {
+    const prev: AgentStatus[] = [
+      { name: "librarian", status: "idle", task: "读文档", updatedAt: "t0", alive: true },
+    ];
+    const out = reduceAgentsForEvent(prev, ev("librarian", "running"));
+    expect(out[0]!.status).toBe("running");
+    expect(out[0]!.task).toBe("读文档"); // task preserved
+    expect(out).not.toBe(prev); // new reference on change
+  });
+  it("returns the same reference on a no-op status", () => {
+    const prev: AgentStatus[] = [
+      { name: "principal", status: "running", task: "", updatedAt: "t0", alive: true },
+    ];
+    expect(reduceAgentsForEvent(prev, ev("principal", "running"))).toBe(prev);
+  });
+  it("ignores non agent_status_update events (same reference)", () => {
+    const prev: AgentStatus[] = [
+      { name: "principal", status: "idle", task: "", updatedAt: "t0", alive: true },
+    ];
+    expect(reduceAgentsForEvent(prev, { type: "RUN_STARTED" } as WebSocketEvent)).toBe(prev);
+  });
+  it("ignores events with an empty name", () => {
+    const prev: AgentStatus[] = [];
+    expect(reduceAgentsForEvent(prev, ev("", "running"))).toBe(prev);
+  });
+  it("marks a stopped agent as not alive", () => {
+    const out = reduceAgentsForEvent([], ev("librarian", "stopped"));
+    expect(out[0]).toMatchObject({ status: "stopped", alive: false });
+  });
+  it("normalizes retrying to running for the panel", () => {
+    const out = reduceAgentsForEvent([], ev("principal", "retrying"));
+    expect(out[0]!.status).toBe("running");
+  });
+  it("merges multiple agents independently", () => {
+    let agents: AgentStatus[] = [];
+    agents = reduceAgentsForEvent(agents, ev("principal", "running"));
+    agents = reduceAgentsForEvent(agents, ev("librarian", "running"));
+    agents = reduceAgentsForEvent(agents, ev("principal", "idle"));
+    expect(agents).toHaveLength(2);
+    expect(agents.find((a) => a.name === "principal")!.status).toBe("idle");
+    expect(agents.find((a) => a.name === "librarian")!.status).toBe("running");
+  });
+});

package/src/__tests__/api.test.ts ADDED Viewed

@@ -0,0 +1,221 @@
+import { describe, it, expect, beforeEach, afterEach, vi } from "vitest";
+import { api } from "../utils/api";
+// These exercise the real-fetch path (runtimeConfig.useMockBackend is false in
+// tests — VITE_USE_MOCK_BACKEND is unset). We stub globalThis.fetch and a
+// minimal localStorage so authHeaders()/getStoredToken() don't blow up in the
+// node test environment.
+type FetchResponseInit = {
+  ok?: boolean;
+  status?: number;
+  contentType?: string;
+  json?: unknown;
+  /** When set, res.json() rejects with this (mirrors a non-JSON body). */
+  jsonThrows?: boolean;
+};
+function makeResponse(init: FetchResponseInit): Response {
+  const status = init.status ?? (init.ok === false ? 500 : 200);
+  const ok = init.ok ?? (status >= 200 && status < 300);
+  const headers = new Map<string, string>();
+  if (init.contentType) headers.set("content-type", init.contentType);
+  return {
+    ok,
+    status,
+    headers: { get: (k: string) => headers.get(k.toLowerCase()) ?? null },
+    json: async () => {
+      if (init.jsonThrows) throw new SyntaxError("Unexpected token '<'");
+      return init.json;
+    },
+    text: async () => (typeof init.json === "string" ? init.json : JSON.stringify(init.json ?? "")),
+  } as unknown as Response;
+}
+let fetchMock: ReturnType<typeof vi.fn>;
+beforeEach(() => {
+  fetchMock = vi.fn();
+  vi.stubGlobal("fetch", fetchMock);
+  vi.stubGlobal("localStorage", {
+    getItem: () => null,
+    setItem: () => undefined,
+    removeItem: () => undefined,
+  });
+});
+afterEach(() => {
+  vi.unstubAllGlobals();
+});
+describe("api.sessions.list — unwraps { sessions } and tolerates shape", () => {
+  it("unwraps the runtime's { sessions: [...] } envelope", async () => {
+    fetchMock.mockResolvedValueOnce(
+      makeResponse({ contentType: "application/json", json: { sessions: [{ id: "a" }, { id: "b" }] } }),
+    );
+    const out = await api.sessions.list();
+    expect(out).toHaveLength(2);
+    expect(out[0].id).toBe("a");
+  });
+  it("tolerates a bare array response (legacy / mock shape)", async () => {
+    fetchMock.mockResolvedValueOnce(
+      makeResponse({ contentType: "application/json", json: [{ id: "x" }] }),
+    );
+    const out = await api.sessions.list();
+    expect(out).toHaveLength(1);
+    expect(out[0].id).toBe("x");
+  });
+  it("returns [] (never throws .map) for an unexpected shape", async () => {
+    fetchMock.mockResolvedValueOnce(makeResponse({ contentType: "application/json", json: {} }));
+    await expect(api.sessions.list()).resolves.toEqual([]);
+  });
+  it("returns [] for a null body", async () => {
+    fetchMock.mockResolvedValueOnce(makeResponse({ contentType: "application/json", json: null }));
+    await expect(api.sessions.list()).resolves.toEqual([]);
+  });
+});
+describe("api.sessions.create — unwraps the { id, session } envelope (#96)", () => {
+  it("reads the real title from the runtime's { id, session } envelope", async () => {
+    // The runtime's POST /sessions returns `{ id, session }` (server.ts), unlike
+    // the GET routes which return the bare session. Before the fix the whole
+    // envelope was handed to normalizeSession, so `raw.title` was undefined and
+    // the sidebar/header fell back to `Session <id8>` until a reload.
+    fetchMock.mockResolvedValueOnce(
+      makeResponse({
+        contentType: "application/json",
+        json: { id: "f8f35032", session: { id: "f8f35032", title: "请用两句话介绍 BrainPilot" } },
+      }),
+    );
+    const out = await api.sessions.create("请用两句话介绍 BrainPilot");
+    expect(out.id).toBe("f8f35032");
+    expect(out.title).toBe("请用两句话介绍 BrainPilot");
+  });
+  it("tolerates a bare session object (no envelope)", async () => {
+    fetchMock.mockResolvedValueOnce(
+      makeResponse({
+        contentType: "application/json",
+        json: { id: "abc", title: "bare title" },
+      }),
+    );
+    const out = await api.sessions.create("bare title");
+    expect(out.id).toBe("abc");
+    expect(out.title).toBe("bare title");
+  });
+});
+describe("composer-driving requests carry an abort timeout (#106)", () => {
+  it("create passes an AbortSignal so a hung POST can't wedge the composer", async () => {
+    fetchMock.mockResolvedValueOnce(
+      makeResponse({ contentType: "application/json", json: { id: "x", session: { id: "x", title: "t" } } }),
+    );
+    await api.sessions.create("t");
+    const init = fetchMock.mock.calls[0][1] as RequestInit;
+    expect(init.signal).toBeInstanceOf(AbortSignal);
+  });
+  it("postMessage passes an AbortSignal", async () => {
+    fetchMock.mockResolvedValueOnce(
+      makeResponse({ contentType: "application/json", json: { status: "ok" } }),
+    );
+    await api.sessions.postMessage("s1", { content: "hi", uuid: "u", timestamp: "2026-06-18T00:00:00Z" });
+    const init = fetchMock.mock.calls[0][1] as RequestInit;
+    expect(init.signal).toBeInstanceOf(AbortSignal);
+  });
+  it("propagates the timeout rejection to the caller (releases isSending upstream)", async () => {
+    // Simulate AbortSignal.timeout firing: fetch rejects with a TimeoutError.
+    fetchMock.mockRejectedValueOnce(new DOMException("timed out", "TimeoutError"));
+    await expect(
+      api.sessions.postMessage("s1", { content: "hi", uuid: "u", timestamp: "2026-06-18T00:00:00Z" }),
+    ).rejects.toBeInstanceOf(DOMException);
+  });
+});
+describe("api.sessions.getEvents — tolerates SSE / non-JSON responses", () => {
+  it("returns the events array for a JSON { events } body", async () => {
+    fetchMock.mockResolvedValueOnce(
+      makeResponse({ contentType: "application/json", json: { events: [{ type: "RUN_STARTED" }] } }),
+    );
+    const out = await api.sessions.getEvents("s1");
+    expect(out).toHaveLength(1);
+  });
+  it("returns [] (does NOT throw) when the endpoint is an SSE stream", async () => {
+    // /sessions/:id/events is wired to sseHandler in backend-core → text/event-stream.
+    // Calling res.json() on it would reject; getEvents must short-circuit to [].
+    fetchMock.mockResolvedValueOnce(
+      makeResponse({ contentType: "text/event-stream", jsonThrows: true }),
+    );
+    await expect(api.sessions.getEvents("s1")).resolves.toEqual([]);
+  });
+  it("returns [] for a non-ok response", async () => {
+    fetchMock.mockResolvedValueOnce(makeResponse({ ok: false, status: 404, contentType: "application/json", json: { error: "not found" } }));
+    await expect(api.sessions.getEvents("s1")).resolves.toEqual([]);
+  });
+});
+describe("api.sessions.getHistory — persisted events.jsonl rehydration", () => {
+  it("returns the envelope shape from a JSON response", async () => {
+    fetchMock.mockResolvedValueOnce(
+      makeResponse({
+        contentType: "application/json",
+        json: { events: [{ type: "TEXT_MESSAGE_CHUNK", delta: "hi" }], total: 1, truncated: false },
+      }),
+    );
+    const out = await api.sessions.getHistory("s1");
+    expect(out.events).toHaveLength(1);
+    expect(out.total).toBe(1);
+    expect(out.truncated).toBe(false);
+  });
+  it("forwards the limit query string", async () => {
+    fetchMock.mockResolvedValueOnce(
+      makeResponse({ contentType: "application/json", json: { events: [], total: 0, truncated: false } }),
+    );
+    await api.sessions.getHistory("s1", { limit: 42 });
+    const url = String(fetchMock.mock.calls[0]![0]);
+    expect(url).toContain("/sessions/s1/history?limit=42");
+  });
+  it("returns the empty envelope on a non-ok response", async () => {
+    fetchMock.mockResolvedValueOnce(makeResponse({ ok: false, status: 404 }));
+    const out = await api.sessions.getHistory("s1");
+    expect(out).toEqual({ events: [], total: 0, truncated: false });
+  });
+  it("returns the empty envelope when the body is null", async () => {
+    fetchMock.mockResolvedValueOnce(makeResponse({ contentType: "application/json", json: null }));
+    const out = await api.sessions.getHistory("s1");
+    expect(out).toEqual({ events: [], total: 0, truncated: false });
+  });
+});
+describe("api.sessions.interrupt — hits the interrupt route, not /messages (#90)", () => {
+  it("POSTs to /sessions/:id/interrupt and returns { interrupted }", async () => {
+    fetchMock.mockResolvedValueOnce(
+      makeResponse({ contentType: "application/json", json: { interrupted: true } }),
+    );
+    const out = await api.sessions.interrupt("s1");
+    const [url, init] = fetchMock.mock.calls[0]!;
+    expect(String(url)).toMatch(/\/sessions\/s1\/interrupt$/);
+    expect(String(url)).not.toMatch(/\/messages$/);
+    expect((init as RequestInit).method).toBe("POST");
+    expect(out.interrupted).toBe(true);
+  });
+  it("never routes the Stop action through the messages endpoint", async () => {
+    fetchMock.mockResolvedValueOnce(
+      makeResponse({ contentType: "application/json", json: { interrupted: true } }),
+    );
+    await api.sessions.interrupt("abc");
+    const url = String(fetchMock.mock.calls[0]![0]);
+    expect(url.endsWith("/messages")).toBe(false);
+  });
+});

package/src/__tests__/chatScrollMemory.test.ts ADDED Viewed

@@ -0,0 +1,49 @@
+import { beforeEach, describe, expect, it } from "vitest";
+import {
+  getChatScroll,
+  setChatScroll,
+  resolveScrollTop,
+} from "../components/chat/chatScrollMemory";
+describe("chatScrollMemory (#89)", () => {
+  beforeEach(() => {
+    // Each test uses unique keys, but clear shared rows defensively.
+    setChatScroll("s1", { scrollTop: 0, pinned: true });
+    setChatScroll("s2", { scrollTop: 0, pinned: true });
+  });
+  it("stores and reads per-session state", () => {
+    setChatScroll("s1", { scrollTop: 120, pinned: false });
+    expect(getChatScroll("s1")).toEqual({ scrollTop: 120, pinned: false });
+  });
+  it("isolates sessions", () => {
+    setChatScroll("s1", { scrollTop: 50, pinned: false });
+    setChatScroll("s2", { scrollTop: 999, pinned: true });
+    expect(getChatScroll("s1")?.scrollTop).toBe(50);
+    expect(getChatScroll("s2")?.scrollTop).toBe(999);
+  });
+  it("ignores undefined keys", () => {
+    setChatScroll(undefined, { scrollTop: 5, pinned: false });
+    expect(getChatScroll(undefined)).toBeUndefined();
+  });
+  describe("resolveScrollTop", () => {
+    it("returns bottom when there is no memory (fresh conversation)", () => {
+      expect(resolveScrollTop(undefined, 4000)).toBe(4000);
+    });
+    it("returns bottom when the user was pinned (following live output)", () => {
+      expect(resolveScrollTop({ scrollTop: 100, pinned: true }, 4000)).toBe(4000);
+    });
+    it("restores the saved position when the user was reading history", () => {
+      expect(resolveScrollTop({ scrollTop: 800, pinned: false }, 4000)).toBe(800);
+    });
+    it("clamps a stale position to the current scrollHeight", () => {
+      expect(resolveScrollTop({ scrollTop: 9000, pinned: false }, 4000)).toBe(4000);
+    });
+  });
+});

package/src/__tests__/demoConversation.test.ts ADDED Viewed

@@ -0,0 +1,73 @@
+import { describe, expect, it } from "vitest";
+import type { ChatMessage } from "../contracts/backend";
+import { isDemoConversational } from "../components/demo/DemoView";
+function msg(overrides: Partial<ChatMessage>): ChatMessage {
+  return {
+    id: overrides.id ?? "m1",
+    role: overrides.role ?? "assistant",
+    content: overrides.content ?? "",
+    createdAt: "2026-06-18T00:00:00.000Z",
+    ...overrides,
+  };
+}
+describe("isDemoConversational (#98 multi-agent transcript)", () => {
+  it("keeps non-empty user prompts", () => {
+    expect(isDemoConversational(msg({ role: "user", content: "hello" }))).toBe(true);
+    expect(isDemoConversational(msg({ role: "user", content: "  " }))).toBe(false);
+  });
+  it("keeps the principal's substantive text replies", () => {
+    expect(
+      isDemoConversational(msg({ role: "assistant", agent: "principal", kind: "text", content: "done" })),
+    ).toBe(true);
+  });
+  it("keeps EXPERT agent text replies (the core #98 regression)", () => {
+    // librarian / engineer / experimentalist replies must survive — the live
+    // Chat shows them, so the demo must too.
+    expect(
+      isDemoConversational(msg({ role: "assistant", agent: "librarian", kind: "text", content: "searching…" })),
+    ).toBe(true);
+    expect(
+      isDemoConversational(msg({ role: "assistant", agent: "engineer", kind: undefined, content: "built it" })),
+    ).toBe(true);
+  });
+  it("treats a missing agent as conversational (older bundles)", () => {
+    expect(isDemoConversational(msg({ role: "assistant", kind: "text", content: "hi" }))).toBe(true);
+  });
+  it("keeps error bubbles with content", () => {
+    expect(isDemoConversational(msg({ role: "system", kind: "error", content: "librarian failed" }))).toBe(true);
+    expect(isDemoConversational(msg({ role: "system", kind: "error", content: "" }))).toBe(false);
+  });
+  it("keeps system_message bubbles that carry a payload", () => {
+    expect(
+      isDemoConversational(
+        msg({
+          role: "system",
+          kind: "system_message",
+          content: "",
+          systemMessage: { level: "error", message: "librarian error", recoverable: true },
+        }),
+      ),
+    ).toBe(true);
+    // No payload → nothing to render.
+    expect(isDemoConversational(msg({ role: "system", kind: "system_message", content: "" }))).toBe(false);
+  });
+  it("drops reasoning, tool calls/results, hooks and interactive cards", () => {
+    expect(isDemoConversational(msg({ kind: "thinking", content: "let me think" }))).toBe(false);
+    expect(isDemoConversational(msg({ kind: "tool", content: "Tool: read" }))).toBe(false);
+    expect(isDemoConversational(msg({ role: "system", kind: "hook", content: "reset" }))).toBe(false);
+    expect(isDemoConversational(msg({ kind: "ask_user", content: "pick one" }))).toBe(false);
+    expect(isDemoConversational(msg({ kind: "auto_retry", content: "retrying" }))).toBe(false);
+  });
+  it("drops empty text placeholders", () => {
+    expect(isDemoConversational(msg({ role: "assistant", kind: "text", content: "   " }))).toBe(false);
+  });
+});

package/src/__tests__/demoReset.test.ts ADDED Viewed

@@ -0,0 +1,24 @@
+import { describe, expect, it } from "vitest";
+import { shouldResetDemo } from "../components/demo/demoReset";
+describe("shouldResetDemo (#111 sidebar re-click returns to landing)", () => {
+  it("does not reset on the initial mount (same value)", () => {
+    // prev === next on first render — packing/importing a bundle must survive.
+    expect(shouldResetDemo(0, 0)).toBe(false);
+    expect(shouldResetDemo(3, 3)).toBe(false);
+  });
+  it("resets when the signal advances (sidebar clicked again)", () => {
+    expect(shouldResetDemo(0, 1)).toBe(true);
+    expect(shouldResetDemo(1, 2)).toBe(true);
+  });
+  it("treats an undefined signal (standalone mount) as no reset", () => {
+    // DemoView mounted without the prop: prev and next are both undefined.
+    expect(shouldResetDemo(undefined, undefined)).toBe(false);
+  });
+  it("resets when transitioning from undefined to a number", () => {
+    expect(shouldResetDemo(undefined, 1)).toBe(true);
+  });
+});

package/src/__tests__/messageGroups.test.ts ADDED Viewed

@@ -0,0 +1,80 @@
+import { describe, it, expect } from "vitest";
+import { buildRenderItems } from "../contexts/messageGroups";
+import type { ChatMessage } from "../contracts/backend";
+// Folded "activity" blocks (reasoning + tool steps) must stay "in progress"
+// while their owning agent's run is still active, even when no single step is
+// momentarily streaming. Without run-active awareness the per-message streaming
+// flags all clear between ReAct rounds and the block flashes "完成思考" in the
+// gap before the next round. See messageGroups.ts buildRenderItems.
+// A folded step (reasoning/tool) — these are the only kinds that group into an
+// activity block; assistant text / user / errors render standalone.
+function step(over: Partial<ChatMessage> = {}): ChatMessage {
+  return {
+    id: over.id ?? `s-${Math.random().toString(36).slice(2)}`,
+    role: "assistant",
+    content: "",
+    createdAt: new Date().toISOString(),
+    agent: "principal",
+    streaming: false,
+    kind: "thinking",
+    ...over,
+  };
+}
+describe("buildRenderItems — activity block run-active awareness", () => {
+  it("without runningAgents, block streaming derives only from step flags (legacy behavior)", () => {
+    const streamingBlock = buildRenderItems([step({ streaming: true })]);
+    expect(streamingBlock[0]).toMatchObject({ type: "activity", streaming: true });
+    const doneBlock = buildRenderItems([step({ streaming: false })]);
+    expect(doneBlock[0]).toMatchObject({ type: "activity", streaming: false });
+  });
+  it("keeps a block in progress when its agent's run is active but no step is streaming (the bug)", () => {
+    // Between ReAct rounds: every step's END already cleared streaming, but the
+    // principal run has NOT finished — the block must NOT show as done.
+    const items = buildRenderItems(
+      [step({ agent: "principal", streaming: false, kind: "tool" })],
+      new Set(["principal"]),
+    );
+    expect(items[0]).toMatchObject({ type: "activity", streaming: true });
+  });
+  it("marks a block done when its agent is idle and no step is streaming", () => {
+    const items = buildRenderItems(
+      [step({ agent: "principal", streaming: false })],
+      new Set<string>(), // principal not running anymore
+    );
+    expect(items[0]).toMatchObject({ type: "activity", streaming: false });
+  });
+  it("falls back to 'principal' for unattributed steps", () => {
+    const items = buildRenderItems([step({ agent: undefined, streaming: false })], new Set(["principal"]));
+    expect(items[0]).toMatchObject({ type: "activity", streaming: true });
+  });
+  it("multi-agent: an idle agent's block shows done even while another agent is still running", () => {
+    // worker has finished (not in the running set); expert is still running.
+    // Each block is scoped to its own agent's run state.
+    const workerStep = step({ id: "w1", agent: "worker", streaming: false, kind: "tool" });
+    const userBreak: ChatMessage = {
+      id: "u1",
+      role: "user",
+      content: "next",
+      createdAt: new Date().toISOString(),
+      agent: "user",
+      streaming: false,
+      kind: "text",
+    };
+    const expertStep = step({ id: "e1", agent: "expert", streaming: false, kind: "thinking" });
+    const items = buildRenderItems([workerStep, userBreak, expertStep], new Set(["expert"]));
+    const activities = items.filter((i) => i.type === "activity");
+    expect(activities).toHaveLength(2);
+    // worker block (agent idle) → done; expert block (agent running) → in progress
+    expect(activities[0]).toMatchObject({ streaming: false });
+    expect(activities[1]).toMatchObject({ streaming: true });
+  });
+});