@happyvertical/smrt-svelte 0.53.1 → 0.53.3

package/dist/state/__tests__/app-state-ai-lifecycle.test.js

@@ -1,240 +0,0 @@
-/**
- * Regression tests for the browser-AI warm-cache lifecycle (T2 #1403):
- *
- * - R1 (listener leak): warm-cached adapters survive Provider remounts. Each
- *   manager that subscribes must capture + later remove its listeners on
- *   dispose(), and a shared adapter must never accumulate more than one live
- *   listener set across managers.
- * - R2 (state lies): dispose() must not leave a disposed adapter cached as
- *   'ready'; unloadLLM() must downgrade the warm-cache entry so the next init
- *   re-runs (reports progress) instead of cache-hitting a model-less adapter.
- * - R3 (scheduler thrash): setAIConfig() must no-op on a deep-equal config
- *   (inline `ai={{…}}` literal => new identity every render), and concurrent
- *   executePreload() passes must be guarded.
- * - R7 (nit): the empty-config early-return must reset loaded/failed too.
- */
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
-// --- Fakes -----------------------------------------------------------------
-/**
- * A fake STT/TTS adapter whose event-subscription methods return real
- * unsubscribe handles backed by Sets — so a test can read the live listener
- * count and prove subscriptions are torn down (R1).
- */
-function makeFakeSTT(type = 'whisper-cpp') {
-    const sets = {
-        result: new Set(),
-        start: new Set(),
-        end: new Set(),
-        error: new Set(),
-    };
-    const sub = (set, cb) => {
-        set.add(cb);
-        return () => {
-            set.delete(cb);
-        };
-    };
-    return {
-        type,
-        dispose: vi.fn(async () => { }),
-        ensureInitialized: vi.fn(async () => { }),
-        start: vi.fn(async () => { }),
-        stop: vi.fn(async () => { }),
-        abort: vi.fn(),
-        isListening: () => false,
-        onResult: (cb) => sub(sets.result, cb),
-        onStart: (cb) => sub(sets.start, cb),
-        onEnd: (cb) => sub(sets.end, cb),
-        onError: (cb) => sub(sets.error, cb),
-        /** total live listeners across all event sets */
-        listenerCount: () => sets.result.size + sets.start.size + sets.end.size + sets.error.size,
-    };
-}
-function makeFakeTTS(type = 'browser-synthesis') {
-    const sets = {
-        start: new Set(),
-        end: new Set(),
-        error: new Set(),
-    };
-    const sub = (set, cb) => {
-        set.add(cb);
-        return () => {
-            set.delete(cb);
-        };
-    };
-    return {
-        type,
-        dispose: vi.fn(async () => { }),
-        ensureInitialized: vi.fn(async () => { }),
-        speak: vi.fn(async () => { }),
-        stop: vi.fn(),
-        pause: vi.fn(),
-        resume: vi.fn(),
-        getVoices: () => [],
-        onStart: (cb) => sub(sets.start, cb),
-        onEnd: (cb) => sub(sets.end, cb),
-        onError: (cb) => sub(sets.error, cb),
-        listenerCount: () => sets.start.size + sets.end.size + sets.error.size,
-    };
-}
-function makeFakeLLM(type = 'webllm', model = 'tiny-model') {
-    return {
-        type,
-        currentModel: model,
-        dispose: vi.fn(async () => { }),
-        ensureInitialized: vi.fn(async () => { }),
-        unloadModel: vi.fn(async () => { }),
-        message: vi.fn(async () => 'ok'),
-    };
-}
-// --- Module mocks ----------------------------------------------------------
-// The manager imports the AI factories + capability helpers from the browser-ai
-// barrel. Mock only those; keep everything else (types) real via importOriginal.
-const fakeSTT = makeFakeSTT();
-const fakeTTS = makeFakeTTS();
-const fakeLLM = makeFakeLLM();
-vi.mock('../../browser-ai/index.js', async (importOriginal) => {
-    const actual = await importOriginal();
-    return {
-        ...actual,
-        detectCapabilities: () => ({}),
-        canEnableSmrtMode: () => false,
-        getSTT: vi.fn(async () => fakeSTT),
-        getTTS: vi.fn(async () => fakeTTS),
-        getLLM: vi.fn(async () => fakeLLM),
-    };
-});
-import { createAppState, } from '../app-state.svelte.js';
-import { clearAllCaches, getCachedLLM, getCachedSTT } from '../warm-clients.js';
-beforeEach(async () => {
-    await clearAllCaches();
-});
-afterEach(async () => {
-    await clearAllCaches();
-    vi.clearAllMocks();
-});
-describe('R1: warm-adapter listener lifecycle', () => {
-    it("removes a manager's listeners on dispose()", async () => {
-        const m = createAppState();
-        await m.initializeSTT({ type: 'whisper-cpp' });
-        expect(fakeSTT.listenerCount()).toBeGreaterThan(0);
-        await m.dispose();
-        expect(fakeSTT.listenerCount()).toBe(0);
-    });
-    it('does not accumulate listeners across remounts of the same cached adapter', async () => {
-        // Manager 1 subscribes then disposes.
-        const m1 = createAppState();
-        await m1.initializeSTT({ type: 'whisper-cpp' });
-        const afterFirst = fakeSTT.listenerCount();
-        await m1.dispose();
-        // Manager 2 cache-hits the SAME warm adapter, subscribes, disposes.
-        const m2 = createAppState();
-        await m2.initializeSTT({ type: 'whisper-cpp' });
-        // Exactly one manager's worth of listeners while live — not doubled.
-        expect(fakeSTT.listenerCount()).toBe(afterFirst);
-        await m2.dispose();
-        expect(fakeSTT.listenerCount()).toBe(0);
-    });
-    it('keeps at most one live listener set when two managers coexist', async () => {
-        const m1 = createAppState();
-        await m1.initializeSTT({ type: 'whisper-cpp' });
-        const single = fakeSTT.listenerCount();
-        // A second live manager taking over the shared adapter must evict the
-        // first's listeners rather than stack a second set.
-        const m2 = createAppState();
-        await m2.initializeSTT({ type: 'whisper-cpp' });
-        expect(fakeSTT.listenerCount()).toBe(single);
-        await m1.dispose();
-        await m2.dispose();
-        expect(fakeSTT.listenerCount()).toBe(0);
-    });
-    it('removes TTS listeners on dispose()', async () => {
-        const m = createAppState();
-        await m.initializeTTS({ type: 'browser-synthesis' });
-        expect(fakeTTS.listenerCount()).toBeGreaterThan(0);
-        await m.dispose();
-        expect(fakeTTS.listenerCount()).toBe(0);
-    });
-    it('subscribing the same adapter twice on one manager is idempotent', async () => {
-        const m = createAppState();
-        await m.initializeSTT({ type: 'whisper-cpp' });
-        const once = fakeSTT.listenerCount();
-        // Second init cache-hits and re-enters subscribeToSTTEvents.
-        await m.initializeSTT({ type: 'whisper-cpp' });
-        expect(fakeSTT.listenerCount()).toBe(once);
-        await m.dispose();
-    });
-});
-describe('R2: dispose()/unloadLLM() do not leave the cache lying', () => {
-    it('dispose() leaves the warm STT adapter intact (not disposed) and cached ready', async () => {
-        const m = createAppState();
-        await m.initializeSTT({ type: 'whisper-cpp' });
-        await m.dispose();
-        // Cached adapter survives navigation and was NOT disposed — so a later
-        // init won't restore a dead engine reported as 'ready'.
-        const cached = getCachedSTT('whisper-cpp');
-        expect(cached?.initState).toBe('ready');
-        expect(cached?.adapter).toBe(fakeSTT);
-        expect(fakeSTT.dispose).not.toHaveBeenCalled();
-    });
-    it('unloadLLM() downgrades the warm-cache entry to uninitialized', async () => {
-        const m = createAppState();
-        await m.initializeLLM('tiny-model', { type: 'webllm' });
-        expect(getCachedLLM('webllm', 'tiny-model')?.initState).toBe('ready');
-        await m.unloadLLM();
-        // Next initializeLLM must re-run (report progress) rather than cache-hit a
-        // model-less adapter reported as 'ready'.
-        expect(getCachedLLM('webllm', 'tiny-model')?.initState).toBe('uninitialized');
-        await m.dispose();
-    });
-});
-describe('R3: setAIConfig scheduler thrash + preload re-entrancy', () => {
-    it('no-ops on a deep-equal config of a different identity', async () => {
-        const m = createAppState();
-        const spy = vi.spyOn(m, 'schedulePreload');
-        m.setAIConfig({ preload: 'none', stt: { type: 'whisper-cpp' } });
-        const callsAfterFirst = spy.mock.calls.length;
-        expect(callsAfterFirst).toBeGreaterThan(0);
-        // Same shape, brand-new object literal (what an inline `ai={{…}}` produces).
-        m.setAIConfig({ preload: 'none', stt: { type: 'whisper-cpp' } });
-        expect(spy.mock.calls.length).toBe(callsAfterFirst); // no re-schedule
-    });
-    it('re-schedules when the config actually changes', async () => {
-        const m = createAppState();
-        const spy = vi.spyOn(m, 'schedulePreload');
-        m.setAIConfig({ preload: 'none', stt: { type: 'whisper-cpp' } });
-        const before = spy.mock.calls.length;
-        m.setAIConfig({ preload: 'none', stt: { type: 'whisper-wasm' } });
-        expect(spy.mock.calls.length).toBe(before + 1);
-    });
-    it('guards executePreload against concurrent passes', async () => {
-        const m = createAppState();
-        m.setAIConfig({
-            preload: 'none',
-            stt: { type: 'whisper-cpp' },
-        });
-        const exec = m.executePreload.bind(m);
-        // Two overlapping passes: the second must early-return (guarded), so the
-        // STT factory is invoked once, not twice.
-        const { getSTT } = (await import('../../browser-ai/index.js'));
-        getSTT.mockClear();
-        await Promise.all([exec(), exec()]);
-        expect(getSTT).toHaveBeenCalledTimes(1);
-        await m.dispose();
-    });
-});
-describe('R7: empty-config preload early-return resets loaded/failed', () => {
-    it('clears stale loaded/failed entries', async () => {
-        const m = createAppState();
-        // Seed stale entries through a normal pass, then re-run with empty config.
-        m.setAIConfig({ preload: 'none', stt: { type: 'whisper-cpp' } });
-        await m.executePreload();
-        expect(m.aiLoading.loaded.length).toBeGreaterThan(0);
-        // Now an empty config — the early-return path must reset loaded/failed.
-        m.setAIConfig({ preload: 'none' });
-        await m.executePreload();
-        expect(m.aiLoading.phase).toBe('idle');
-        expect(m.aiLoading.loaded).toEqual([]);
-        expect(m.aiLoading.failed).toEqual([]);
-        await m.dispose();
-    });
-});

package/dist/state/__tests__/warm-clients.test.js

@@ -1,40 +0,0 @@
-/**
- * Unit tests for the warm client cache (coverage uplift, S6 gate). Module-level
- * Map caches for STT/TTS/LLM adapters — get/set/update/remove + stats.
- */
-import { describe, expect, it, vi } from 'vitest';
-import { getCachedLLM, getCachedSTT, getCachedTTS, getCacheStats, removeCachedLLM, removeCachedSTT, removeCachedTTS, setCachedLLM, setCachedSTT, setCachedTTS, updateLLMCacheState, updateSTTCacheState, updateTTSCacheState, } from '../warm-clients';
-const fakeAdapter = () => ({ dispose: vi.fn() });
-describe('warm client cache', () => {
-    it('STT: set / get / update / remove', () => {
-        const adapter = fakeAdapter();
-        setCachedSTT('whisper-wasm', adapter);
-        expect(getCachedSTT('whisper-wasm')?.adapter).toBe(adapter);
-        updateSTTCacheState('whisper-wasm', { downloadProgress: 50 });
-        expect(getCachedSTT('whisper-wasm')?.downloadProgress).toBe(50);
-        removeCachedSTT('whisper-wasm');
-        expect(getCachedSTT('whisper-wasm')).toBeUndefined();
-    });
-    it('TTS: set / get / update / remove', () => {
-        const adapter = fakeAdapter();
-        setCachedTTS('browser-synthesis', adapter);
-        expect(getCachedTTS('browser-synthesis')?.adapter).toBe(adapter);
-        updateTTSCacheState('browser-synthesis', { initState: 'ready' });
-        removeCachedTTS('browser-synthesis');
-        expect(getCachedTTS('browser-synthesis')).toBeUndefined();
-    });
-    it('LLM: set / get / update / remove', () => {
-        const adapter = fakeAdapter();
-        setCachedLLM('webllm', adapter);
-        expect(getCachedLLM('webllm')?.adapter).toBe(adapter);
-        updateLLMCacheState('webllm', { downloadProgress: 10 });
-        removeCachedLLM('webllm');
-        expect(getCachedLLM('webllm')).toBeUndefined();
-    });
-    it('getCacheStats reports cache contents', () => {
-        setCachedSTT('whisper-cpp', fakeAdapter());
-        const stats = getCacheStats();
-        expect(stats).toBeDefined();
-        removeCachedSTT('whisper-cpp');
-    });
-});

package/dist/web/__tests__/activity-feed-harness.svelte

@@ -1,32 +0,0 @@
-<script lang="ts">
-/**
- * Integration harness — invokes activityFeed() inside a component-init scope so
- * the `$effect` it (and the underlying `liveCollection`) installs wires up to a
- * real component lifecycle, then hands the feed handle back to the test.
- */
-import type { SmrtWebCollection } from '@happyvertical/smrt-web';
-import type { ShellState } from '../../components/workspace/index.js';
-import {
-  type ActivityFeedHandle,
-  type ActivityFeedMap,
-  activityFeed,
-} from '../activity-feed.svelte.js';
-
-interface Props {
-  collection: SmrtWebCollection<Record<string, unknown>>;
-  shell: ShellState;
-  map: ActivityFeedMap<Record<string, unknown>>;
-  onReady: (handle: ActivityFeedHandle) => void;
-}
-
-const props: Props = $props();
-// This harness intentionally snapshots its props during setup.
-// svelte-ignore state_referenced_locally
-const handle = activityFeed({
-  collection: props.collection,
-  shell: props.shell,
-  map: props.map,
-});
-// svelte-ignore state_referenced_locally
-props.onReady(handle);
-</script>

package/dist/web/__tests__/activity-feed-harness.svelte.d.ts

@@ -1,18 +0,0 @@
-/**
- * Integration harness — invokes activityFeed() inside a component-init scope so
- * the `$effect` it (and the underlying `liveCollection`) installs wires up to a
- * real component lifecycle, then hands the feed handle back to the test.
- */
-import type { SmrtWebCollection } from '@happyvertical/smrt-web';
-import type { ShellState } from '../../components/workspace/index.js';
-import { type ActivityFeedHandle, type ActivityFeedMap } from '../activity-feed.svelte.js';
-interface Props {
-    collection: SmrtWebCollection<Record<string, unknown>>;
-    shell: ShellState;
-    map: ActivityFeedMap<Record<string, unknown>>;
-    onReady: (handle: ActivityFeedHandle) => void;
-}
-declare const ActivityFeedHarness: import("svelte").Component<Props, {}, "">;
-type ActivityFeedHarness = ReturnType<typeof ActivityFeedHarness>;
-export default ActivityFeedHarness;
-//# sourceMappingURL=activity-feed-harness.svelte.d.ts.map

package/dist/web/__tests__/activity-feed-harness.svelte.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"activity-feed-harness.svelte.d.ts","sourceRoot":"","sources":["../../../src/web/__tests__/activity-feed-harness.svelte.ts"],"names":[],"mappings":"AAGA;;;;GAIG;AACH,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,yBAAyB,CAAC;AACjE,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,qCAAqC,CAAC;AACtE,OAAO,EACL,KAAK,kBAAkB,EACvB,KAAK,eAAe,EAErB,MAAM,4BAA4B,CAAC;AAGpC,UAAU,KAAK;IACb,UAAU,EAAE,iBAAiB,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,CAAC;IACvD,KAAK,EAAE,UAAU,CAAC;IAClB,GAAG,EAAE,eAAe,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,CAAC;IAC9C,OAAO,EAAE,CAAC,MAAM,EAAE,kBAAkB,KAAK,IAAI,CAAC;CAC/C;AAsBD,QAAA,MAAM,mBAAmB,2CAAwC,CAAC;AAClE,KAAK,mBAAmB,GAAG,UAAU,CAAC,OAAO,mBAAmB,CAAC,CAAC;AAClE,eAAe,mBAAmB,CAAC"}

package/dist/web/__tests__/activity-feed.integration.svelte.test.js

@@ -1,190 +0,0 @@
-/**
- * Integration test for the AdminShell activity feed adapter (#1779) end to end.
- *
- * Unlike the reconciler unit tests (which feed plain arrays), this exercises the
- * FULL stack — `activityFeed` → `liveCollection` → the real `@happyvertical/smrt-web`
- * collection with its client-data engine — through a mounted harness, so the
- * `$effect` that drives reconciliation has a genuine component-init scope. Only
- * the network boundary is mocked (the generated REST fetchers passed to
- * `createSmrtCollection`); the engine, live view, runes reactivity, and the real
- * `ShellState` are all live. This proves the reactive wiring drives the shell:
- * a loaded row and an optimistic insert both surface as shell activities.
- */
-import { createSmrtCollection, } from '@happyvertical/smrt-web';
-import { flushSync, mount, unmount } from 'svelte';
-import { afterEach, describe, expect, it } from 'vitest';
-import { createShellState } from '../../components/workspace/admin-shell/state.svelte.js';
-import Harness from './activity-feed-harness.svelte';
-function jobDefinition(name) {
-    return {
-        name,
-        objectRef: '@test/smrt-svelte:RenderJob',
-        className: 'RenderJob',
-        endpoint: `/${name}`,
-        idField: 'id',
-        actions: ['create', 'get', 'list'],
-        fields: {
-            title: { type: 'text', required: true },
-            state: { type: 'text' },
-        },
-    };
-}
-/** Scripted stand-in for the generated REST fetchers (see live-collection test). */
-function makeScriptedFetchers(initialRows) {
-    const serverRows = [...initialRows];
-    const fetchers = {
-        list: async () => serverRows.map((row) => ({ ...row })),
-        create: async (data) => {
-            const created = { ...data, id: `server-${serverRows.length + 1}` };
-            serverRows.push(created);
-            return { ...created };
-        },
-    };
-    return { fetchers };
-}
-const mapJob = (row) => ({
-    kind: 'render-job',
-    scope: 'focus',
-    label: row.title,
-    status: row.state ?? 'running',
-});
-function mountFeed(collection, shell) {
-    const target = document.createElement('div');
-    document.body.appendChild(target);
-    let exposed = undefined;
-    const component = mount(Harness, {
-        target,
-        props: {
-            // The harness is typed against the base object shape; JobData is a
-            // structural subtype, so this cross-cast is purely nominal.
-            collection: collection,
-            shell,
-            map: mapJob,
-            onReady: (handle) => {
-                exposed = handle;
-            },
-        },
-    });
-    return {
-        handle: exposed,
-        teardown: () => {
-            unmount(component);
-            target.remove();
-        },
-    };
-}
-describe('activityFeed (integration)', () => {
-    const cleanup = [];
-    afterEach(() => {
-        for (const fn of cleanup.splice(0))
-            fn();
-    });
-    it('reconciles loaded collection rows into shell activities', async () => {
-        const scripted = makeScriptedFetchers([
-            { id: 'job-1', title: 'Render intro', state: 'running' },
-            { id: 'job-2', title: 'Render outro', state: 'queued' },
-        ]);
-        const collection = createSmrtCollection(jobDefinition('feed-load'), {
-            fetchers: scripted.fetchers,
-            staleTimeMs: 60_000,
-        });
-        const shell = createShellState();
-        const mounted = mountFeed(collection, shell);
-        cleanup.push(mounted.teardown);
-        await collection.preload();
-        flushSync();
-        const activities = shell.listActivities();
-        expect(activities.map((a) => a.id).sort()).toEqual(['job-1', 'job-2']);
-        expect(activities.every((a) => a.kind === 'render-job')).toBe(true);
-        // Homed to the focus edge (right) per the map's scope.
-        expect(shell.listActivities({ edge: 'right' })).toHaveLength(2);
-    });
-    it('surfaces an optimistic insert as a shell activity reactively', async () => {
-        const scripted = makeScriptedFetchers([
-            { id: 'job-1', title: 'Render intro', state: 'running' },
-        ]);
-        const collection = createSmrtCollection(jobDefinition('feed-insert'), {
-            fetchers: scripted.fetchers,
-            staleTimeMs: 60_000,
-        });
-        const shell = createShellState();
-        const mounted = mountFeed(collection, shell);
-        cleanup.push(mounted.teardown);
-        await collection.preload();
-        flushSync();
-        expect(shell.listActivities()).toHaveLength(1);
-        // An optimistic insert appears in the live view synchronously → the feed
-        // reconciles it into the shell on the reactive tick.
-        const tx = collection.insert({
-            id: 'local-1',
-            title: 'Render credits',
-            state: 'running',
-        });
-        flushSync();
-        expect(shell
-            .listActivities()
-            .map((a) => a.label)
-            .sort()).toEqual(['Render credits', 'Render intro']);
-        // Once persisted, the server-assigned row replaces the optimistic one; the
-        // activity for the temp id is removed and the server row's activity remains.
-        await tx.isPersisted.promise;
-        flushSync();
-        const labels = shell
-            .listActivities()
-            .map((a) => a.label)
-            .sort();
-        expect(labels).toEqual(['Render credits', 'Render intro']);
-        // The optimistic-id activity is gone; the server id is present.
-        const ids = shell.listActivities().map((a) => a.id);
-        expect(ids).toContain('server-2');
-        expect(ids).not.toContain('local-1');
-    });
-    it('dispose() retracts the feed activities from the shell', async () => {
-        const scripted = makeScriptedFetchers([
-            { id: 'job-1', title: 'Render intro', state: 'running' },
-        ]);
-        const collection = createSmrtCollection(jobDefinition('feed-dispose'), {
-            fetchers: scripted.fetchers,
-            staleTimeMs: 60_000,
-        });
-        const shell = createShellState();
-        const mounted = mountFeed(collection, shell);
-        cleanup.push(mounted.teardown);
-        await collection.preload();
-        flushSync();
-        expect(shell.listActivities()).toHaveLength(1);
-        mounted.handle.dispose();
-        expect(shell.listActivities()).toHaveLength(0);
-        expect(mounted.handle.isDisposed).toBe(true);
-    });
-    it('auto-retracts the feed activities when the host component unmounts', async () => {
-        const scripted = makeScriptedFetchers([
-            { id: 'job-1', title: 'Render intro', state: 'running' },
-            { id: 'job-2', title: 'Render outro', state: 'queued' },
-        ]);
-        const collection = createSmrtCollection(jobDefinition('feed-unmount'), {
-            fetchers: scripted.fetchers,
-            staleTimeMs: 60_000,
-        });
-        const shell = createShellState();
-        // An activity from another source must survive the feed's unmount.
-        shell.upsertActivity({
-            id: 'external-1',
-            label: 'External',
-            kind: 'other',
-            scope: 'app',
-            status: 'running',
-        });
-        const mounted = mountFeed(collection, shell);
-        await collection.preload();
-        flushSync();
-        expect(shell.listActivities()).toHaveLength(3);
-        // Unmount WITHOUT calling dispose() manually: the adapter's $effect teardown
-        // must retract exactly its own activities, leaving the external one intact.
-        mounted.teardown();
-        flushSync();
-        const remaining = shell.listActivities();
-        expect(remaining).toHaveLength(1);
-        expect(remaining[0].id).toBe('external-1');
-    });
-});