@happyvertical/smrt-chat 0.53.1 → 0.53.2

package/dist/manifest.json

@@ -2,7 +2,7 @@
   "version": "1.0.0",
   "timestamp": 0,
   "packageName": "@happyvertical/smrt-chat",
-  "packageVersion": "0.53.1",
+  "packageVersion": "0.53.2",
   "objects": {
     "@happyvertical/smrt-chat:AgentSessionCollection": {
       "name": "agentsessioncollection",

package/dist/smrt-knowledge.json

@@ -3,12 +3,12 @@
   "sensitiveFieldsExcluded": true,
   "generatedAt": "1970-01-01T00:00:00.000Z",
   "packageName": "@happyvertical/smrt-chat",
-  "packageVersion": "0.53.1",
+  "packageVersion": "0.53.2",
   "sourceManifestPath": "dist/manifest.json",
   "agentDocPath": "AGENTS.md",
   "sourceHashes": {
-    "manifest": "672e6d290de99f91c82eaf8fc60944373cdd16eaf1e0b007e6b3ae10ab6f2f6b",
-    "packageJson": "92fa75ad6dc0c51c4d2e98e5d8ca8855ddc80dc99c8acdbbd7275b4ef00bf0b9",
+    "manifest": "3865dce9ea07670b543e39c56ddee4c2244cdb8f6bf3450340c880d3d43d87ac",
+    "packageJson": "f6fc0cd19c3d9d6cfeb520da745cc9715e14f39f5fc0e2b8b45b5e05216433fc",
     "agents": "c50538672f9ba892ac0998b58c3a2b497e876df27eff7693c66f3103e730df3c",
     "moduleDoc:agents/dev-workbench.md": "69b6f827d25c8aeb0e7516c90303af23948803213b5d34bc334c1e3376c83a16",
     "agentSurface:src/routes/chat-dev.intents.ts": "ff8f7c19c7eb6e841a08f03c97773c53092ed208004666bfed372bd1ad456b35"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@happyvertical/smrt-chat",
-  "version": "0.53.1",
+  "version": "0.53.2",
   "smrtJsdoc": "strict",
   "description": "Chat rooms, DMs, threads, and agent conversations for the SMRT framework",
   "type": "module",
@@ -75,14 +75,14 @@
   },
   "dependencies": {
     "@happyvertical/ai": "^0.98.0",
-    "@happyvertical/smrt-agents": "0.53.1",
-    "@happyvertical/smrt-core": "0.53.1",
-    "@happyvertical/smrt-personas": "0.53.1",
-    "@happyvertical/smrt-profiles": "0.53.1",
-    "@happyvertical/smrt-tenancy": "0.53.1",
-    "@happyvertical/smrt-types": "0.53.1",
-    "@happyvertical/smrt-ui": "0.53.1",
-    "@happyvertical/smrt-users": "0.53.1",
+    "@happyvertical/smrt-agents": "0.53.2",
+    "@happyvertical/smrt-core": "0.53.2",
+    "@happyvertical/smrt-personas": "0.53.2",
+    "@happyvertical/smrt-profiles": "0.53.2",
+    "@happyvertical/smrt-tenancy": "0.53.2",
+    "@happyvertical/smrt-types": "0.53.2",
+    "@happyvertical/smrt-ui": "0.53.2",
+    "@happyvertical/smrt-users": "0.53.2",
     "@happyvertical/sql": "^0.98.0"
   },
   "peerDependencies": {
@@ -94,9 +94,9 @@
     }
   },
   "devDependencies": {
-    "@happyvertical/smrt-playground": "0.53.1",
-    "@happyvertical/smrt-vitest": "0.53.1",
-    "@happyvertical/smrt-web": "0.53.1",
+    "@happyvertical/smrt-playground": "0.53.2",
+    "@happyvertical/smrt-vitest": "0.53.2",
+    "@happyvertical/smrt-web": "0.53.2",
     "@mlc-ai/web-llm": "^0.2.85",
     "bitgpu": "0.19.1",
     "@sveltejs/kit": "2.69.1",
@@ -111,7 +111,7 @@
     "vitest": "4.1.10"
   },
   "scripts": {
-    "build": "SMRT_PACKAGE_BUILD=1 vite build --mode library && SMRT_PACKAGE_BUILD=1 svelte-package -i src/svelte -o dist/svelte --tsconfig tsconfig.svelte.json",
+    "build": "SMRT_PACKAGE_BUILD=1 vite build --mode library && SMRT_PACKAGE_BUILD=1 svelte-package -i src/svelte -o dist/svelte --tsconfig tsconfig.svelte.json && node ../../scripts/prune-svelte-package-artifacts.js dist",
     "build:watch": "vite build --mode library --watch",
     "check": "svelte-kit sync && pnpm exec svelte-check --tsconfig ./tsconfig.kit.json",
     "clean": "rm -rf dist",

package/dist/__tests__/chat-test-db.d.ts

@@ -1,8 +0,0 @@
-import type { DatabaseInterface } from '@happyvertical/sql';
-import '../index.js';
-export declare function createChatTestDatabase(url: string): Promise<DatabaseInterface>;
-export declare function seedChatRoom(db: DatabaseInterface, id?: string): Promise<void>;
-export declare function seedChatMessage(db: DatabaseInterface, id?: string, roomId?: string): Promise<void>;
-export declare function seedChatThread(db: DatabaseInterface, id: string, rootMessageId?: string): Promise<void>;
-export declare function seedAgentSession(db: DatabaseInterface, id: string): Promise<void>;
-//# sourceMappingURL=chat-test-db.d.ts.map

package/dist/__tests__/chat-test-db.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"chat-test-db.d.ts","sourceRoot":"","sources":["../../src/__tests__/chat-test-db.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,oBAAoB,CAAC;AAC5D,OAAO,aAAa,CAAC;AAgBrB,wBAAsB,sBAAsB,CAC1C,GAAG,EAAE,MAAM,GACV,OAAO,CAAC,iBAAiB,CAAC,CAE5B;AAED,wBAAsB,YAAY,CAChC,EAAE,EAAE,iBAAiB,EACrB,EAAE,SAAW,GACZ,OAAO,CAAC,IAAI,CAAC,CASf;AAED,wBAAsB,eAAe,CACnC,EAAE,EAAE,iBAAiB,EACrB,EAAE,SAAU,EACZ,MAAM,SAAW,GAChB,OAAO,CAAC,IAAI,CAAC,CASf;AAED,wBAAsB,cAAc,CAClC,EAAE,EAAE,iBAAiB,EACrB,EAAE,EAAE,MAAM,EACV,aAAa,SAA2B,GACvC,OAAO,CAAC,IAAI,CAAC,CAQf;AAED,wBAAsB,gBAAgB,CACpC,EAAE,EAAE,iBAAiB,EACrB,EAAE,EAAE,MAAM,GACT,OAAO,CAAC,IAAI,CAAC,CASf"}

package/dist/__tests__/raw-collections.d.ts

@@ -1,35 +0,0 @@
-/**
- * Test-only access to the raw chat collections (S5 #1392).
- *
- * The `ChatService` collections are `#private` and the package index no longer
- * exports the collection classes, so production consumers can only mutate chat
- * state through the authorized facade. Security regression tests, however, must
- * sometimes forge a row that "slipped past" the facade or read a row directly to
- * assert tenant binding. They obtain the collections from the `ObjectRegistry`
- * — the same registry `ChatService.create()` registers into — rather than
- * reaching into the service's private fields. This keeps the production surface
- * closed while letting tests exercise the collection layer.
- */
-import { type SmrtObjectOptions } from '@happyvertical/smrt-core';
-import type { AgentSessionCollection } from '../collections/AgentSessionCollection.js';
-import type { ChatMessageCollection } from '../collections/ChatMessageCollection.js';
-import type { ChatParticipantCollection } from '../collections/ChatParticipantCollection.js';
-import type { ChatReactionCollection } from '../collections/ChatReactionCollection.js';
-import type { ChatRoomCollection } from '../collections/ChatRoomCollection.js';
-import type { ChatThreadCollection } from '../collections/ChatThreadCollection.js';
-import type { VoiceSessionCollection } from '../collections/VoiceSessionCollection.js';
-export interface RawChatCollections {
-    rooms: ChatRoomCollection;
-    messages: ChatMessageCollection;
-    participants: ChatParticipantCollection;
-    threads: ChatThreadCollection;
-    agentSessions: AgentSessionCollection;
-    reactions: ChatReactionCollection;
-    voiceSessions: VoiceSessionCollection;
-}
-/**
- * Resolve the raw chat collections from the registry for the given DB options.
- * Must be called after `ChatService.create(options)` has registered them.
- */
-export declare function getRawChatCollections(options: SmrtObjectOptions): Promise<RawChatCollections>;
-//# sourceMappingURL=raw-collections.d.ts.map

package/dist/__tests__/raw-collections.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"raw-collections.d.ts","sourceRoot":"","sources":["../../src/__tests__/raw-collections.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAEH,OAAO,EAEL,KAAK,iBAAiB,EACvB,MAAM,0BAA0B,CAAC;AAClC,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,0CAA0C,CAAC;AACvF,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,yCAAyC,CAAC;AACrF,OAAO,KAAK,EAAE,yBAAyB,EAAE,MAAM,6CAA6C,CAAC;AAC7F,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,0CAA0C,CAAC;AACvF,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,sCAAsC,CAAC;AAC/E,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,wCAAwC,CAAC;AACnF,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,0CAA0C,CAAC;AAEvF,MAAM,WAAW,kBAAkB;IACjC,KAAK,EAAE,kBAAkB,CAAC;IAC1B,QAAQ,EAAE,qBAAqB,CAAC;IAChC,YAAY,EAAE,yBAAyB,CAAC;IACxC,OAAO,EAAE,oBAAoB,CAAC;IAC9B,aAAa,EAAE,sBAAsB,CAAC;IACtC,SAAS,EAAE,sBAAsB,CAAC;IAClC,aAAa,EAAE,sBAAsB,CAAC;CACvC;AAED;;;GAGG;AACH,wBAAsB,qBAAqB,CACzC,OAAO,EAAE,iBAAiB,GACzB,OAAO,CAAC,kBAAkB,CAAC,CAsC7B"}

package/dist/svelte/components/agent/__tests__/ToolCallDisplay.test.js

@@ -1,135 +0,0 @@
-import { expectNoA11yViolations, render, screen, userEvent, } from '@happyvertical/smrt-vitest/svelte';
-import { describe, expect, it, vi } from 'vitest';
-import ToolCallDisplay from '../ToolCallDisplay.svelte';
-function makeToolCall(overrides = {}) {
-    return {
-        toolName: 'search',
-        toolCallId: 'tc-1',
-        arguments: { q: 'hello' },
-        status: 'success',
-        result: ['a', 'b'],
-        duration: 1200,
-        ...overrides,
-    };
-}
-describe('ToolCallDisplay', () => {
-    it('renders the tool name and a human status label', () => {
-        render(ToolCallDisplay, { props: { toolCall: makeToolCall() } });
-        expect(screen.getByText('search')).toBeInTheDocument();
-        expect(screen.getByText('Completed')).toBeInTheDocument();
-    });
-    // #2904 review, Copilot PR #2919 jAwua/jAwtb: AssistantDock/the demo route
-    // map an in-flight preview/apply action to this 'running' toolCall status
-    // (see action-status.test.ts) — assert it actually renders "Running", not
-    // "Completed".
-    it('renders "Running" for a running tool call', () => {
-        render(ToolCallDisplay, {
-            props: {
-                toolCall: makeToolCall({ status: 'running', result: undefined }),
-            },
-        });
-        expect(screen.getByText('Running')).toBeInTheDocument();
-        expect(screen.queryByText('Completed')).not.toBeInTheDocument();
-    });
-    it('expands to reveal arguments and result on click', async () => {
-        render(ToolCallDisplay, { props: { toolCall: makeToolCall() } });
-        const header = screen.getByRole('button');
-        expect(header).toHaveAttribute('aria-expanded', 'false');
-        await userEvent.click(header);
-        expect(header).toHaveAttribute('aria-expanded', 'true');
-        expect(screen.getByText('Arguments')).toBeInTheDocument();
-        expect(screen.getByText('Result')).toBeInTheDocument();
-    });
-    it('is axe-clean', async () => {
-        const { container } = render(ToolCallDisplay, {
-            props: { toolCall: makeToolCall({ status: 'error', error: 'boom' }) },
-        });
-        await expectNoA11yViolations(container);
-    });
-    // #2904: additive DataSurfaceActionResult rendering, used by AssistantDock.
-    function makeActionResult(overrides = {}) {
-        return {
-            version: 1,
-            requestId: 'req-1',
-            identity: {
-                surfaceId: 'orders',
-                kind: 'table',
-                subject: { type: 'tenant', id: 'tenant-a' },
-            },
-            actionId: 'archive',
-            phase: 'preview',
-            ok: true,
-            ...overrides,
-        };
-    }
-    it('a pending preview reads "Awaiting confirmation", starts expanded, and exposes Confirm/Reject', async () => {
-        const onconfirmaction = vi.fn();
-        const onrejectaction = vi.fn();
-        render(ToolCallDisplay, {
-            props: {
-                toolCall: makeToolCall(),
-                actionResult: makeActionResult(),
-                onconfirmaction,
-                onrejectaction,
-            },
-        });
-        // #2904 review fix: a preview must never read "Completed" and must be
-        // expanded by default so Confirm/Reject are visible without a click.
-        expect(screen.queryByText('Completed')).not.toBeInTheDocument();
-        expect(screen.getByText('Awaiting confirmation')).toBeInTheDocument();
-        const header = screen.getByRole('button', { name: /search/i });
-        expect(header).toHaveAttribute('aria-expanded', 'true');
-        expect(screen.getByText('Proposed change')).toBeInTheDocument();
-        await userEvent.click(screen.getByRole('button', { name: 'Confirm' }));
-        expect(onconfirmaction).toHaveBeenCalledOnce();
-        await userEvent.click(screen.getByRole('button', { name: 'Reject' }));
-        expect(onrejectaction).toHaveBeenCalledOnce();
-    });
-    it('the header toggle still collapses an awaiting-confirmation card', async () => {
-        render(ToolCallDisplay, {
-            props: { toolCall: makeToolCall(), actionResult: makeActionResult() },
-        });
-        const header = screen.getByRole('button', { name: /search/i });
-        expect(header).toHaveAttribute('aria-expanded', 'true');
-        await userEvent.click(header);
-        expect(header).toHaveAttribute('aria-expanded', 'false');
-    });
-    it('renders "Applied" with a result body and no Confirm/Reject once applied', async () => {
-        render(ToolCallDisplay, {
-            props: {
-                toolCall: makeToolCall(),
-                actionResult: makeActionResult({
-                    phase: 'apply',
-                    details: { rowId: 'order-2', status: 'shipped' },
-                }),
-            },
-        });
-        expect(screen.getByText('Applied')).toBeInTheDocument();
-        const header = screen.getByRole('button', { name: /search/i });
-        expect(header).toHaveAttribute('aria-expanded', 'true');
-        expect(screen.getByText('Applied change')).toBeInTheDocument();
-        expect(screen.getByText(/order-2/)).toBeInTheDocument();
-        expect(screen.queryByRole('button', { name: 'Confirm' })).not.toBeInTheDocument();
-    });
-    it('falls back to a plain confirmation line when an applied result has no details', () => {
-        render(ToolCallDisplay, {
-            props: {
-                toolCall: makeToolCall(),
-                actionResult: makeActionResult({ phase: 'apply' }),
-            },
-        });
-        expect(screen.getByText('Change applied successfully.')).toBeInTheDocument();
-    });
-    it('renders "Failed" with the reason when the action result is not ok', async () => {
-        render(ToolCallDisplay, {
-            props: {
-                toolCall: makeToolCall(),
-                actionResult: makeActionResult({ ok: false, reason: 'stale_revision' }),
-            },
-        });
-        expect(screen.getByText('Failed')).toBeInTheDocument();
-        const header = screen.getByRole('button', { name: /search/i });
-        expect(header).toHaveAttribute('aria-expanded', 'true');
-        expect(screen.getByText('stale_revision')).toBeInTheDocument();
-    });
-});

package/dist/svelte/components/agent/message-blocks.test.js

@@ -1,31 +0,0 @@
-import { describe, expect, it } from 'vitest';
-import { parseAgentMessageBlocks } from './message-blocks';
-describe('parseAgentMessageBlocks', () => {
-    it('preserves fenced markdown blocks as readable content', () => {
-        expect(parseAgentMessageBlocks('Here is a draft:\n```markdown\n# Heading\n\nBody copy\n```\nThanks.')).toEqual([
-            { type: 'text', content: 'Here is a draft:\n' },
-            { type: 'markdown', content: '# Heading\n\nBody copy' },
-            { type: 'text', content: '\nThanks.' },
-        ]);
-    });
-    it('extracts field updates from json blocks', () => {
-        expect(parseAgentMessageBlocks('```json\n{"fields":{"title":"New title","body":"Updated body"}}\n```')).toEqual([
-            {
-                type: 'fields',
-                fields: { title: 'New title', body: 'Updated body' },
-            },
-        ]);
-    });
-    it('falls back to plain text for invalid json blocks', () => {
-        expect(parseAgentMessageBlocks('```json\n{not valid json}\n```')).toEqual([
-            { type: 'text', content: '```json\n{not valid json}\n```' },
-        ]);
-    });
-    it('supports windows newlines and whitespace after the language tag', () => {
-        expect(parseAgentMessageBlocks('Intro\r\n```markdown   \r\n# Heading\r\n\r\nBody copy\r\n```\r\nDone')).toEqual([
-            { type: 'text', content: 'Intro\r\n' },
-            { type: 'markdown', content: '# Heading\r\n\r\nBody copy' },
-            { type: 'text', content: '\r\nDone' },
-        ]);
-    });
-});

package/dist/svelte/components/assistant/__tests__/AssistantComposer.test.js

@@ -1,75 +0,0 @@
-// @vitest-environment jsdom
-/**
- * Component-level coverage for AssistantComposer (#2904 review finding 4).
- *
- * Before this fix, `handleSend` cleared `content`/`stagedAttachments`
- * synchronously before `onsend`'s promise settled, so a transport failure
- * discarded the user's typed message with no visible error.
- */
-import { expectNoA11yViolations, render, screen, userEvent, } from '@happyvertical/smrt-vitest/svelte';
-import { describe, expect, it, vi } from 'vitest';
-import AssistantComposer from '../AssistantComposer.svelte';
-function fileInput(container) {
-    const input = container.querySelector('input[type="file"]');
-    if (!input)
-        throw new Error('file input not found');
-    return input;
-}
-const png = (name = 'photo.png') => new File(['data'], name, { type: 'image/png' });
-describe('AssistantComposer', () => {
-    it('keeps the draft and shows an inline error when onsend rejects', async () => {
-        const onsend = vi.fn().mockRejectedValue(new Error('network down'));
-        const onupload = vi.fn();
-        render(AssistantComposer, { props: { onsend, onupload } });
-        const textarea = screen.getByLabelText('Message');
-        await userEvent.type(textarea, 'hello there');
-        await userEvent.click(screen.getByRole('button', { name: 'Send' }));
-        expect(onsend).toHaveBeenCalledWith('hello there', []);
-        // The draft text must still be in the textarea — not cleared.
-        expect(textarea).toHaveValue('hello there');
-        expect(await screen.findByText(/Could not send: network down/i)).toBeInTheDocument();
-    });
-    it('clears the draft only after onsend resolves successfully', async () => {
-        const onsend = vi.fn().mockResolvedValue(undefined);
-        const onupload = vi.fn();
-        render(AssistantComposer, { props: { onsend, onupload } });
-        const textarea = screen.getByLabelText('Message');
-        await userEvent.type(textarea, 'hi');
-        await userEvent.click(screen.getByRole('button', { name: 'Send' }));
-        expect(onsend).toHaveBeenCalledWith('hi', []);
-        expect(textarea).toHaveValue('');
-        expect(screen.queryByText(/Could not send/i)).not.toBeInTheDocument();
-    });
-    // Cycle-2 third final: a rejecting onupload previously had no catch
-    // anywhere in handleFileChange, so the chip row never updated and no
-    // error appeared — the rejection escaped as an unhandled promise
-    // rejection from the DOM change event instead.
-    it('shows an inline error and keeps existing chips when onupload rejects', async () => {
-        const onsend = vi.fn();
-        const onupload = vi
-            .fn()
-            .mockResolvedValueOnce([{ id: 'att-1', name: 'existing.png' }])
-            .mockRejectedValueOnce(new Error('no writeEndpoint configured'));
-        const { container } = render(AssistantComposer, {
-            props: { onsend, onupload },
-        });
-        // First upload succeeds and stages a chip.
-        await userEvent.upload(fileInput(container), png('existing.png'));
-        expect(await screen.findByText('existing.png')).toBeInTheDocument();
-        // Second upload rejects: the existing chip must remain, and the
-        // rejection must show as an inline error.
-        await userEvent.upload(fileInput(container), png('second.png'));
-        expect(await screen.findByText(/Could not attach file: no writeEndpoint configured/i)).toBeInTheDocument();
-        expect(screen.getByText('existing.png')).toBeInTheDocument();
-        expect(screen.queryByText('second.png')).not.toBeInTheDocument();
-    });
-    // Cycle-3 second final finding 2: the hidden file input had no accessible
-    // name (the visually-hidden `clip: rect(0 0 0 0)` styling keeps it in the
-    // a11y tree, unlike ../shared/FileUpload.svelte's wrapping visible label).
-    it('is axe-clean', async () => {
-        const { container } = render(AssistantComposer, {
-            props: { onsend: vi.fn(), onupload: vi.fn() },
-        });
-        await expectNoA11yViolations(container);
-    });
-});