@happyvertical/smrt-chat 0.53.0 → 0.53.2

package/dist/svelte/components/layout/__tests__/ChatLayout.test.js

@@ -1,67 +0,0 @@
-// @vitest-environment jsdom
-/**
- * Component coverage for ChatLayout via the shared S11 harness (#1416).
- *
- * Regression for T2 #1391: ChatLayout rendered RoomList but never forwarded
- * `oncreateroom`, so the bundled layout's create-room affordance was dead.
- * ChatLayout must now accept and forward `oncreateroom` to RoomList.
- */
-import { expectNoA11yViolations, render, screen, userEvent, } from '@happyvertical/smrt-vitest/svelte';
-import { createRawSnippet } from 'svelte';
-import { describe, expect, it, vi } from 'vitest';
-import ChatLayout from '../ChatLayout.svelte';
-const ROOMS = [
-    {
-        id: 'room-1',
-        name: 'General',
-        description: '',
-        roomType: 'public',
-        topic: '',
-        participantCount: 1,
-        unreadCount: 0,
-    },
-];
-// Minimal main-content snippet for the required `children` prop.
-const content = createRawSnippet(() => ({
-    render: () => '<div data-testid="main">Main content</div>',
-}));
-describe('ChatLayout', () => {
-    it('forwards oncreateroom to RoomList so the create affordance is wired', async () => {
-        const oncreateroom = vi.fn();
-        render(ChatLayout, {
-            props: {
-                rooms: ROOMS,
-                currentProfileId: 'p1',
-                onselectroom: vi.fn(),
-                oncreateroom,
-                children: content,
-            },
-        });
-        const createBtn = screen.getByRole('button', { name: 'Create new room' });
-        await userEvent.click(createBtn);
-        expect(oncreateroom).toHaveBeenCalledTimes(1);
-    });
-    it('omits the create affordance when no oncreateroom is supplied', () => {
-        render(ChatLayout, {
-            props: {
-                rooms: ROOMS,
-                currentProfileId: 'p1',
-                onselectroom: vi.fn(),
-                children: content,
-            },
-        });
-        expect(screen.queryByRole('button', { name: 'Create new room' })).toBeNull();
-    });
-    it('is axe-clean', async () => {
-        const { container } = render(ChatLayout, {
-            props: {
-                rooms: ROOMS,
-                currentProfileId: 'p1',
-                onselectroom: vi.fn(),
-                oncreateroom: vi.fn(),
-                children: content,
-            },
-        });
-        await expectNoA11yViolations(container);
-    });
-});

package/dist/svelte/components/messages/__tests__/MessageInput.test.js

@@ -1,38 +0,0 @@
-// @vitest-environment jsdom
-/**
- * Component coverage for MessageInput via the shared S11 harness (#1416).
- */
-import { expectNoA11yViolations, render, screen, userEvent, } from '@happyvertical/smrt-vitest/svelte';
-import { describe, expect, it, vi } from 'vitest';
-import MessageInput from '../MessageInput.svelte';
-describe('MessageInput', () => {
-    it('sends the trimmed content on Enter', async () => {
-        const onsend = vi.fn();
-        render(MessageInput, { props: { onsend } });
-        await userEvent.type(screen.getByLabelText('Message input'), 'Hello{Enter}');
-        expect(onsend).toHaveBeenCalledWith('Hello');
-    });
-    it('does not send empty content', async () => {
-        const onsend = vi.fn();
-        render(MessageInput, { props: { onsend } });
-        await userEvent.type(screen.getByLabelText('Message input'), '{Enter}');
-        expect(onsend).not.toHaveBeenCalled();
-    });
-    it('shows a reply banner and cancels it', async () => {
-        const oncancelreply = vi.fn();
-        render(MessageInput, {
-            props: {
-                onsend: vi.fn(),
-                replyTo: { id: 'm1', senderName: 'Ada', content: 'earlier message' },
-                oncancelreply,
-            },
-        });
-        expect(screen.getByText('Ada')).toBeInTheDocument();
-        await userEvent.click(screen.getByRole('button', { name: 'Cancel reply' }));
-        expect(oncancelreply).toHaveBeenCalledTimes(1);
-    });
-    it('is axe-clean', async () => {
-        const { container } = render(MessageInput, { props: { onsend: vi.fn() } });
-        await expectNoA11yViolations(container);
-    });
-});

package/dist/svelte/components/messages/__tests__/MessageItem.test.js

@@ -1,108 +0,0 @@
-// @vitest-environment jsdom
-/**
- * Component coverage for MessageItem via the shared S11 harness (#1416).
- *
- * Regression for the T2 #1391 a11y blocker: the per-message actions
- * (reply/edit/delete/react) used to be revealed ONLY by mouseenter/mouseleave,
- * so keyboard- and touch-only users could never reach them. There must now be a
- * keyboard-reachable "more actions" button that toggles the action bar, and
- * focusing into the row must reveal the actions.
- */
-import { expectNoA11yViolations, render, screen, userEvent, } from '@happyvertical/smrt-vitest/svelte';
-import { describe, expect, it, vi } from 'vitest';
-import MessageItem from '../MessageItem.svelte';
-function makeMessage(overrides = {}) {
-    return {
-        id: 'm1',
-        roomId: 'room-1',
-        threadId: null,
-        senderProfileId: 'p1',
-        senderName: 'Ada',
-        senderAvatarUrl: undefined,
-        agentSessionId: null,
-        content: 'Hello there',
-        messageType: 'text',
-        role: 'user',
-        isEdited: false,
-        isDeleted: false,
-        replyTo: null,
-        reactions: [],
-        attachments: [],
-        toolCallData: null,
-        createdAt: new Date('2026-01-01T00:00:00Z'),
-        ...overrides,
-    };
-}
-describe('MessageItem', () => {
-    it('renders a keyboard-reachable "more actions" button when actions are available', () => {
-        render(MessageItem, {
-            props: {
-                message: makeMessage(),
-                isOwn: false,
-                onreply: vi.fn(),
-                onreact: vi.fn(),
-            },
-        });
-        const moreBtn = screen.getByRole('button', { name: 'Message actions' });
-        expect(moreBtn).toBeInTheDocument();
-        // A real <button> is inherently in the tab order (no tabindex="-1").
-        expect(moreBtn.getAttribute('tabindex')).not.toBe('-1');
-    });
-    it('reveals the action bar via the more-actions button without any mouse hover', async () => {
-        const onreply = vi.fn();
-        render(MessageItem, {
-            props: {
-                message: makeMessage(),
-                isOwn: false,
-                onreply,
-                onreact: vi.fn(),
-            },
-        });
-        // Actions are hidden until requested — no hover, no focus yet.
-        expect(screen.queryByRole('button', { name: 'Reply' })).toBeNull();
-        await userEvent.click(screen.getByRole('button', { name: 'Message actions' }));
-        // The action bar is now reachable; clicking Reply invokes the callback.
-        const replyBtn = await screen.findByRole('button', { name: 'Reply' });
-        await userEvent.click(replyBtn);
-        expect(onreply).toHaveBeenCalledWith('m1');
-    });
-    it("exposes edit/delete actions for the user's own message", async () => {
-        const onedit = vi.fn();
-        const ondelete = vi.fn();
-        render(MessageItem, {
-            props: {
-                message: makeMessage(),
-                isOwn: true,
-                onedit,
-                ondelete,
-            },
-        });
-        await userEvent.click(screen.getByRole('button', { name: 'Message actions' }));
-        await userEvent.click(await screen.findByRole('button', { name: 'Edit' }));
-        expect(onedit).toHaveBeenCalledWith('m1');
-        // The bar stays open while focus remains inside the row, so Delete is
-        // reachable in the same interaction.
-        await userEvent.click(screen.getByRole('button', { name: 'Delete' }));
-        expect(ondelete).toHaveBeenCalledWith('m1');
-    });
-    it('does not render the more-actions affordance when no action callbacks are provided', () => {
-        render(MessageItem, {
-            props: {
-                message: makeMessage(),
-                isOwn: false,
-            },
-        });
-        expect(screen.queryByRole('button', { name: 'Message actions' })).toBeNull();
-    });
-    it('is axe-clean with the action affordance present', async () => {
-        const { container } = render(MessageItem, {
-            props: {
-                message: makeMessage(),
-                isOwn: false,
-                onreply: vi.fn(),
-                onreact: vi.fn(),
-            },
-        });
-        await expectNoA11yViolations(container);
-    });
-});

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

@@ -1,20 +0,0 @@
-// @vitest-environment jsdom
-/**
- * First component test in smrt-chat via the shared S11 harness (#1416). Avatar is
- * the thin adapter over the library Avatar (maps avatarUrl→src, dnd→busy).
- */
-import { expectNoA11yViolations, render, screen, } from '@happyvertical/smrt-vitest/svelte';
-import { describe, expect, it } from 'vitest';
-import Avatar from '../Avatar.svelte';
-describe('Avatar', () => {
-    it('renders the initials derived from the name', () => {
-        render(Avatar, { props: { name: 'Ada Lovelace' } });
-        expect(screen.getByText('AL')).toBeInTheDocument();
-    });
-    it('is axe-clean', async () => {
-        const { container } = render(Avatar, {
-            props: { name: 'Ada Lovelace', onlineStatus: 'online' },
-        });
-        await expectNoA11yViolations(container);
-    });
-});

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

@@ -1,29 +0,0 @@
-// @vitest-environment jsdom
-/**
- * Component coverage for LinkPreview via the shared S11 harness (#1416).
- */
-import { expectNoA11yViolations, render, screen, } from '@happyvertical/smrt-vitest/svelte';
-import { describe, expect, it } from 'vitest';
-import LinkPreview from '../LinkPreview.svelte';
-describe('LinkPreview', () => {
-    it('renders a link with title, description, and hostname', () => {
-        render(LinkPreview, {
-            props: {
-                url: 'https://example.com/article',
-                title: 'An Article',
-                description: 'A short summary',
-            },
-        });
-        const link = screen.getByRole('link');
-        expect(link).toHaveAttribute('href', 'https://example.com/article');
-        expect(screen.getByText('An Article')).toBeInTheDocument();
-        expect(screen.getByText('A short summary')).toBeInTheDocument();
-        expect(screen.getByText('example.com')).toBeInTheDocument();
-    });
-    it('is axe-clean', async () => {
-        const { container } = render(LinkPreview, {
-            props: { url: 'https://example.com', title: 'Example' },
-        });
-        await expectNoA11yViolations(container);
-    });
-});

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

@@ -1,66 +0,0 @@
-// @vitest-environment jsdom
-/**
- * Component coverage for MentionAutocomplete via the shared S11 harness (#1416).
- *
- * Regression for T2 #1391: `case 'Escape': break;` was a no-op, so the
- * suggestion popup could not be dismissed from the keyboard. Escape must now
- * invoke the `oncancel` callback so a parent can close the popup.
- */
-import { expectNoA11yViolations, render, screen, userEvent, } from '@happyvertical/smrt-vitest/svelte';
-import { describe, expect, it, vi } from 'vitest';
-import MentionAutocomplete from '../MentionAutocomplete.svelte';
-const SUGGESTIONS = [
-    { id: 'p1', name: 'Ada Lovelace' },
-    { id: 'p2', name: 'Alan Turing' },
-];
-describe('MentionAutocomplete', () => {
-    it('invokes oncancel when Escape is pressed while visible', async () => {
-        const oncancel = vi.fn();
-        render(MentionAutocomplete, {
-            props: {
-                query: 'a',
-                suggestions: SUGGESTIONS,
-                onselect: vi.fn(),
-                isVisible: true,
-                oncancel,
-            },
-        });
-        await userEvent.keyboard('{Escape}');
-        expect(oncancel).toHaveBeenCalledTimes(1);
-    });
-    it('does not throw on Escape when no oncancel is provided', async () => {
-        render(MentionAutocomplete, {
-            props: {
-                query: 'a',
-                suggestions: SUGGESTIONS,
-                onselect: vi.fn(),
-                isVisible: true,
-            },
-        });
-        await expect(userEvent.keyboard('{Escape}')).resolves.not.toThrow();
-    });
-    it('selects the active suggestion on Enter', async () => {
-        const onselect = vi.fn();
-        render(MentionAutocomplete, {
-            props: {
-                query: 'a',
-                suggestions: SUGGESTIONS,
-                onselect,
-                isVisible: true,
-            },
-        });
-        await userEvent.keyboard('{Enter}');
-        expect(onselect).toHaveBeenCalledWith('p1');
-    });
-    it('is axe-clean', async () => {
-        const { container } = render(MentionAutocomplete, {
-            props: {
-                query: 'a',
-                suggestions: SUGGESTIONS,
-                onselect: vi.fn(),
-                isVisible: true,
-            },
-        });
-        await expectNoA11yViolations(container);
-    });
-});

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

@@ -1,36 +0,0 @@
-// @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/components/shared/__tests__/ReactionPicker.test.js

@@ -1,35 +0,0 @@
-// @vitest-environment jsdom
-/**
- * Component coverage for ReactionPicker via the shared S11 harness (#1416).
- */
-import { expectNoA11yViolations, render, screen, userEvent, within, } from '@happyvertical/smrt-vitest/svelte';
-import { describe, expect, it, vi } from 'vitest';
-import ReactionPicker from '../ReactionPicker.svelte';
-// S12 a11y remediation (#1417): the picker is now a labelled `role="group"` of
-// buttons (was an invalid `role="grid"` with button children, which tripped axe's
-// `aria-required-children`), so axe is asserted on the open state.
-describe('ReactionPicker', () => {
-    it('renders a labelled group of emoji buttons when open', () => {
-        render(ReactionPicker, { props: { onreact: vi.fn(), isOpen: true } });
-        const group = screen.getByRole('group', { name: 'Emoji reactions' });
-        expect(within(group).getAllByRole('button').length).toBeGreaterThan(0);
-    });
-    it('reports the chosen emoji through onreact', async () => {
-        const onreact = vi.fn();
-        render(ReactionPicker, { props: { onreact, isOpen: true } });
-        const [first] = within(screen.getByRole('group', { name: 'Emoji reactions' })).getAllByRole('button');
-        await userEvent.click(first);
-        expect(onreact).toHaveBeenCalledTimes(1);
-        expect(onreact.mock.calls[0][0]).toBeTruthy();
-    });
-    it('renders nothing when closed', () => {
-        render(ReactionPicker, { props: { onreact: vi.fn(), isOpen: false } });
-        expect(screen.queryByRole('group')).toBeNull();
-    });
-    it('is axe-clean', async () => {
-        const { container } = render(ReactionPicker, {
-            props: { onreact: vi.fn(), isOpen: true },
-        });
-        await expectNoA11yViolations(container);
-    });
-});

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

@@ -1,28 +0,0 @@
-// @vitest-environment jsdom
-/**
- * Component coverage for ReadReceipts via the shared S11 harness (#1416).
- */
-import { expectNoA11yViolations, render, screen, } from '@happyvertical/smrt-vitest/svelte';
-import { describe, expect, it } from 'vitest';
-import ReadReceipts from '../ReadReceipts.svelte';
-describe('ReadReceipts', () => {
-    it('summarises how many participants have read', () => {
-        render(ReadReceipts, {
-            props: {
-                readBy: [{ name: 'Ada' }, { name: 'Bob' }],
-                totalParticipants: 3,
-            },
-        });
-        expect(screen.getByLabelText('2 of 3 read')).toBeInTheDocument();
-    });
-    it('renders the sent (unread) state with no readers', () => {
-        render(ReadReceipts, { props: { readBy: [], totalParticipants: 3 } });
-        expect(screen.getByLabelText('0 of 3 read')).toBeInTheDocument();
-    });
-    it('is axe-clean', async () => {
-        const { container } = render(ReadReceipts, {
-            props: { readBy: [{ name: 'Ada' }], totalParticipants: 2 },
-        });
-        await expectNoA11yViolations(container);
-    });
-});

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

@@ -1,23 +0,0 @@
-// @vitest-environment jsdom
-/**
- * Component coverage for UserPresence via the shared S11 harness (#1416).
- */
-import { expectNoA11yViolations, render, screen, } from '@happyvertical/smrt-vitest/svelte';
-import { describe, expect, it } from 'vitest';
-import UserPresence from '../UserPresence.svelte';
-describe('UserPresence', () => {
-    it('exposes the status as an accessible label', () => {
-        render(UserPresence, { props: { status: 'online' } });
-        expect(screen.getByLabelText('Online')).toBeInTheDocument();
-    });
-    it('shows a visible label when requested', () => {
-        render(UserPresence, { props: { status: 'dnd', showLabel: true } });
-        expect(screen.getByText('Do not disturb')).toBeInTheDocument();
-    });
-    it('is axe-clean', async () => {
-        const { container } = render(UserPresence, {
-            props: { status: 'away', showLabel: true },
-        });
-        await expectNoA11yViolations(container);
-    });
-});