@happyvertical/smrt-svelte 0.38.5 → 0.38.6

package/dist/components/workspace/__tests__/admin-shell-mutation-effect-harness.svelte

@@ -0,0 +1,52 @@
+<script lang="ts">
+/**
+ * Runs a caller-supplied ShellState mutation inside a consumer $effect. Tests
+ * use this to prove ShellState's internal reads do not become dependencies of
+ * the consumer effect.
+ */
+import type { ShellState } from '../admin-shell/state.svelte.js';
+import type { ShellActivity } from '../admin-shell/types.js';
+
+type EffectCleanup = () => void;
+
+interface MutationInputs {
+  activity: ShellActivity;
+}
+
+interface Props {
+  shell: ShellState;
+  run: (shell: ShellState, inputs: MutationInputs) => EffectCleanup | undefined;
+  onReady: (controls: {
+    getRuns: () => number;
+    setActivityProgress: (progress: number) => void;
+  }) => void;
+}
+
+const props: Props = $props();
+const activity = $state<ShellActivity>({
+  id: 'reactive-sync',
+  label: 'Reactive sync',
+  kind: 'sync',
+  scope: 'system',
+  status: 'running',
+  progress: 0,
+});
+
+// Plain let on purpose: making this $state would add tracking noise to the
+// effect-leak signal this harness exists to measure.
+let runs = 0;
+
+$effect(() => {
+  runs += 1;
+  return props.run(props.shell, { activity });
+});
+
+// This harness intentionally snapshots the callback during setup.
+// svelte-ignore state_referenced_locally
+props.onReady({
+  getRuns: () => runs,
+  setActivityProgress: (progress: number) => {
+    activity.progress = progress;
+  },
+});
+</script>

package/dist/components/workspace/__tests__/admin-shell-mutation-effect-harness.svelte.d.ts

@@ -0,0 +1,23 @@
+/**
+ * Runs a caller-supplied ShellState mutation inside a consumer $effect. Tests
+ * use this to prove ShellState's internal reads do not become dependencies of
+ * the consumer effect.
+ */
+import type { ShellState } from '../admin-shell/state.svelte.js';
+import type { ShellActivity } from '../admin-shell/types.js';
+type EffectCleanup = () => void;
+interface MutationInputs {
+    activity: ShellActivity;
+}
+interface Props {
+    shell: ShellState;
+    run: (shell: ShellState, inputs: MutationInputs) => EffectCleanup | undefined;
+    onReady: (controls: {
+        getRuns: () => number;
+        setActivityProgress: (progress: number) => void;
+    }) => void;
+}
+declare const AdminShellMutationEffectHarness: import("svelte").Component<Props, {}, "">;
+type AdminShellMutationEffectHarness = ReturnType<typeof AdminShellMutationEffectHarness>;
+export default AdminShellMutationEffectHarness;
+//# sourceMappingURL=admin-shell-mutation-effect-harness.svelte.d.ts.map

package/dist/components/workspace/__tests__/admin-shell-mutation-effect-harness.svelte.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"admin-shell-mutation-effect-harness.svelte.d.ts","sourceRoot":"","sources":["../../../../src/components/workspace/__tests__/admin-shell-mutation-effect-harness.svelte.ts"],"names":[],"mappings":"AAGA;;;;GAIG;AACH,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,gCAAgC,CAAC;AACjE,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,yBAAyB,CAAC;AAG7D,KAAK,aAAa,GAAG,MAAM,IAAI,CAAC;AAEhC,UAAU,cAAc;IACtB,QAAQ,EAAE,aAAa,CAAC;CACzB;AAED,UAAU,KAAK;IACb,KAAK,EAAE,UAAU,CAAC;IAClB,GAAG,EAAE,CAAC,KAAK,EAAE,UAAU,EAAE,MAAM,EAAE,cAAc,KAAK,aAAa,GAAG,SAAS,CAAC;IAC9E,OAAO,EAAE,CAAC,QAAQ,EAAE;QAClB,OAAO,EAAE,MAAM,MAAM,CAAC;QACtB,mBAAmB,EAAE,CAAC,QAAQ,EAAE,MAAM,KAAK,IAAI,CAAC;KACjD,KAAK,IAAI,CAAC;CACZ;AAwCD,QAAA,MAAM,+BAA+B,2CAAwC,CAAC;AAC9E,KAAK,+BAA+B,GAAG,UAAU,CAAC,OAAO,+BAA+B,CAAC,CAAC;AAC1F,eAAe,+BAA+B,CAAC"}

package/dist/components/workspace/__tests__/admin-shell-state.test.js

@@ -1,5 +1,32 @@
+import { flushSync, mount, unmount } from 'svelte';
 import { describe, expect, it, vi } from 'vitest';
 import { createShellState } from '../admin-shell/state.svelte.js';
+import MutationEffectHarness from './admin-shell-mutation-effect-harness.svelte';
+function mountMutationEffect(shell, run) {
+    const target = document.createElement('div');
+    document.body.appendChild(target);
+    let getRuns = () => 0;
+    let setActivityProgress = (_progress) => { };
+    const component = mount(MutationEffectHarness, {
+        target,
+        props: {
+            shell,
+            run,
+            onReady: (controls) => {
+                getRuns = controls.getRuns;
+                setActivityProgress = controls.setActivityProgress;
+            },
+        },
+    });
+    return {
+        getRuns,
+        setActivityProgress,
+        teardown: () => {
+            unmount(component);
+            target.remove();
+        },
+    };
+}
 describe('ShellState', () => {
     it('allows panels to expand independently by default', () => {
         const shell = createShellState();
@@ -107,4 +134,107 @@ describe('ShellState', () => {
         // upsert carrying `previous`, which the toaster filters out.
         expect(events).toEqual(['upsert:new', 'upsert:has-prev']);
     });
+    it('does not leak focus-tool state reads into consumer effects', () => {
+        const shell = createShellState();
+        const harness = mountMutationEffect(shell, (state) => {
+            const unregister = state.registerFocusTool({
+                id: 'activity',
+                label: 'Activity',
+            });
+            state.openFocusTool('activity');
+            return unregister;
+        });
+        try {
+            flushSync();
+            expect(harness.getRuns()).toBe(1);
+            shell.collapsePanel('right');
+            flushSync();
+            shell.openFocusTool('activity');
+            flushSync();
+            expect(harness.getRuns()).toBe(1);
+            expect(shell.focusTools.map((tool) => tool.id)).toEqual(['activity']);
+        }
+        finally {
+            harness.teardown();
+        }
+    });
+    it('does not leak panel state reads into consumer effects', () => {
+        const shell = createShellState();
+        const harness = mountMutationEffect(shell, (state) => {
+            state.setPanel('right', 'expanded');
+            return undefined;
+        });
+        try {
+            flushSync();
+            expect(harness.getRuns()).toBe(1);
+            shell.togglePanel('right');
+            flushSync();
+            expect(harness.getRuns()).toBe(1);
+            expect(shell.panels.right).toBe('collapsed');
+        }
+        finally {
+            harness.teardown();
+        }
+    });
+    it('does not leak settings state reads into consumer effects', () => {
+        const shell = createShellState();
+        const harness = mountMutationEffect(shell, (state) => {
+            state.applySettings({ panels: { right: 'expanded' } });
+            return undefined;
+        });
+        try {
+            flushSync();
+            expect(harness.getRuns()).toBe(1);
+            shell.setHotkey('right', 'KeyR');
+            flushSync();
+            expect(harness.getRuns()).toBe(1);
+            expect(shell.settings.keymap?.right).toEqual({ code: 'KeyR' });
+        }
+        finally {
+            harness.teardown();
+        }
+    });
+    it('does not leak activity state reads into consumer effects', () => {
+        const shell = createShellState();
+        const harness = mountMutationEffect(shell, (state) => {
+            state.upsertActivity({
+                id: 'sync-1',
+                label: 'Sync',
+                kind: 'sync',
+                scope: 'system',
+                status: 'running',
+            });
+            return undefined;
+        });
+        try {
+            flushSync();
+            expect(harness.getRuns()).toBe(1);
+            shell.updateActivity('sync-1', { progress: 50 });
+            flushSync();
+            expect(harness.getRuns()).toBe(1);
+            expect(shell.listActivities()[0]?.progress).toBe(50);
+        }
+        finally {
+            harness.teardown();
+        }
+    });
+    it('keeps caller-supplied reactive activity inputs tracked', () => {
+        const shell = createShellState();
+        const harness = mountMutationEffect(shell, (state, inputs) => {
+            state.upsertActivity(inputs.activity);
+            return undefined;
+        });
+        try {
+            flushSync();
+            expect(harness.getRuns()).toBe(1);
+            expect(shell.listActivities()[0]?.progress).toBe(0);
+            harness.setActivityProgress(75);
+            flushSync();
+            expect(harness.getRuns()).toBe(2);
+            expect(shell.listActivities()[0]?.progress).toBe(75);
+        }
+        finally {
+            harness.teardown();
+        }
+    });
 });

package/dist/components/workspace/admin-shell/state.svelte.d.ts

@@ -1,5 +1,6 @@
 import type { PanelEdge, PanelState, ResolvedShellConfig, ShellActivity, ShellActivityBadge, ShellActivityEvent, ShellActivityFilter, ShellFocusTool, ShellPanelDefaults, ShellScope, ShellSettingsAdapter, ShellSettingsDelta, ShellStateSnapshot, VisiblePanelState } from './types.js';
 type ActivityListener = (event: ShellActivityEvent) => void;
+type ShellActivityPatch = Partial<Omit<ShellActivity, 'id'>>;
 export interface ShellStateOptions {
     config?: ShellPanelDefaults;
     settings?: ShellSettingsDelta;
@@ -32,7 +33,7 @@ export declare class ShellState {
     unregisterFocusTool(id: string): void;
     openFocusTool(id: string): void;
     upsertActivity(activity: ShellActivity): void;
-    updateActivity(id: string, patch: Partial<Omit<ShellActivity, 'id'>>): void;
+    updateActivity(id: string, patch: ShellActivityPatch): void;
     removeActivity(id: string): void;
     watchActivities(listener: ActivityListener): () => void;
     listActivities(filter?: ShellActivityFilter): ShellActivity[];

package/dist/components/workspace/admin-shell/state.svelte.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"state.svelte.d.ts","sourceRoot":"","sources":["../../../../src/components/workspace/admin-shell/state.svelte.ts"],"names":[],"mappings":"AAMA,OAAO,KAAK,EAEV,SAAS,EACT,UAAU,EACV,mBAAmB,EACnB,aAAa,EACb,kBAAkB,EAClB,kBAAkB,EAClB,mBAAmB,EACnB,cAAc,EACd,kBAAkB,EAClB,UAAU,EACV,oBAAoB,EACpB,kBAAkB,EAClB,kBAAkB,EAClB,iBAAiB,EAClB,MAAM,YAAY,CAAC;AAGpB,KAAK,gBAAgB,GAAG,CAAC,KAAK,EAAE,kBAAkB,KAAK,IAAI,CAAC;AAE5D,MAAM,WAAW,iBAAiB;IAChC,MAAM,CAAC,EAAE,kBAAkB,CAAC;IAC5B,QAAQ,CAAC,EAAE,kBAAkB,CAAC;IAC9B,eAAe,CAAC,EAAE,oBAAoB,CAAC;IACvC,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAED,qBAAa,UAAU;IACrB,QAAQ,CAAC,MAAM,EAAE,mBAAmB,CAAC;IACrC,QAAQ,CAAC,OAAO,EAAE,oBAAoB,GAAG,IAAI,CAAC;IAE9C,QAAQ,qBAAkC;IAC1C,MAAM,gCAKH;IACH,UAAU,mBAAgC;IAC1C,iBAAiB,gBAA+B;IAChD,UAAU,kBAA+B;IAEzC,OAAO,CAAC,iBAAiB,CAA+B;gBAE5C,OAAO,GAAE,iBAAsB;IAUrC,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC;IAM9B,QAAQ,IAAI,kBAAkB;IAY9B,aAAa,CACX,KAAK,EAAE,kBAAkB,EACzB,OAAO,GAAE;QAAE,OAAO,CAAC,EAAE,OAAO,CAAA;KAAO,GAClC,IAAI;IAcP,iBAAiB,CAAC,OAAO,EAAE,OAAO,GAAG,IAAI;IAIzC,SAAS,CAAC,IAAI,EAAE,SAAS,EAAE,IAAI,EAAE,MAAM,GAAG,IAAI,GAAG,IAAI;IAQrD,QAAQ,CAAC,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,iBAAiB,GAAG,IAAI;IAazD,WAAW,CAAC,IAAI,EAAE,SAAS,GAAG,IAAI;IAQlC,WAAW,CAAC,IAAI,EAAE,SAAS,GAAG,IAAI;IAIlC,aAAa,CAAC,IAAI,EAAE,SAAS,GAAG,IAAI;IAIpC,oBAAoB,IAAI,OAAO;IAU/B,iBAAiB,CAAC,IAAI,EAAE,cAAc,GAAG,MAAM,IAAI;IASnD,mBAAmB,CAAC,EAAE,EAAE,MAAM,GAAG,IAAI;IAOrC,aAAa,CAAC,EAAE,EAAE,MAAM,GAAG,IAAI;IAU/B,cAAc,CAAC,QAAQ,EAAE,aAAa,GAAG,IAAI;IAuB7C,cAAc,CAAC,EAAE,EAAE,MAAM,EAAE,KAAK,EAAE,OAAO,CAAC,IAAI,CAAC,aAAa,EAAE,IAAI,CAAC,CAAC,GAAG,IAAI;IAW3E,cAAc,CAAC,EAAE,EAAE,MAAM,GAAG,IAAI;IAOhC,eAAe,CAAC,QAAQ,EAAE,gBAAgB,GAAG,MAAM,IAAI;IAKvD,cAAc,CAAC,MAAM,GAAE,mBAAwB,GAAG,aAAa,EAAE;IASjE,aAAa,CAAC,IAAI,EAAE,SAAS,GAAG,kBAAkB;IAsBlD,gBAAgB,CAAC,KAAK,EAAE,UAAU,GAAG,SAAS;IAW9C,OAAO,CAAC,mBAAmB;YAmBb,eAAe;IAI7B,OAAO,CAAC,YAAY;CAGrB;AAED,wBAAgB,gBAAgB,CAAC,OAAO,GAAE,iBAAsB,GAAG,UAAU,CAE5E"}
+{"version":3,"file":"state.svelte.d.ts","sourceRoot":"","sources":["../../../../src/components/workspace/admin-shell/state.svelte.ts"],"names":[],"mappings":"AAOA,OAAO,KAAK,EAEV,SAAS,EACT,UAAU,EACV,mBAAmB,EACnB,aAAa,EACb,kBAAkB,EAClB,kBAAkB,EAClB,mBAAmB,EACnB,cAAc,EACd,kBAAkB,EAClB,UAAU,EACV,oBAAoB,EACpB,kBAAkB,EAClB,kBAAkB,EAClB,iBAAiB,EAClB,MAAM,YAAY,CAAC;AAGpB,KAAK,gBAAgB,GAAG,CAAC,KAAK,EAAE,kBAAkB,KAAK,IAAI,CAAC;AAC5D,KAAK,kBAAkB,GAAG,OAAO,CAAC,IAAI,CAAC,aAAa,EAAE,IAAI,CAAC,CAAC,CAAC;AAE7D,MAAM,WAAW,iBAAiB;IAChC,MAAM,CAAC,EAAE,kBAAkB,CAAC;IAC5B,QAAQ,CAAC,EAAE,kBAAkB,CAAC;IAC9B,eAAe,CAAC,EAAE,oBAAoB,CAAC;IACvC,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAED,qBAAa,UAAU;IACrB,QAAQ,CAAC,MAAM,EAAE,mBAAmB,CAAC;IACrC,QAAQ,CAAC,OAAO,EAAE,oBAAoB,GAAG,IAAI,CAAC;IAE9C,QAAQ,qBAAkC;IAC1C,MAAM,gCAKH;IACH,UAAU,mBAAgC;IAC1C,iBAAiB,gBAA+B;IAChD,UAAU,kBAA+B;IAEzC,OAAO,CAAC,iBAAiB,CAA+B;gBAE5C,OAAO,GAAE,iBAAsB;IAUrC,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC;IAM9B,QAAQ,IAAI,kBAAkB;IAY9B,aAAa,CACX,KAAK,EAAE,kBAAkB,EACzB,OAAO,GAAE;QAAE,OAAO,CAAC,EAAE,OAAO,CAAA;KAAO,GAClC,IAAI;IAkBP,iBAAiB,CAAC,OAAO,EAAE,OAAO,GAAG,IAAI;IAIzC,SAAS,CAAC,IAAI,EAAE,SAAS,EAAE,IAAI,EAAE,MAAM,GAAG,IAAI,GAAG,IAAI;IAUrD,QAAQ,CAAC,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,iBAAiB,GAAG,IAAI;IAezD,WAAW,CAAC,IAAI,EAAE,SAAS,GAAG,IAAI;IAUlC,WAAW,CAAC,IAAI,EAAE,SAAS,GAAG,IAAI;IAIlC,aAAa,CAAC,IAAI,EAAE,SAAS,GAAG,IAAI;IAIpC,oBAAoB,IAAI,OAAO;IAY/B,iBAAiB,CAAC,IAAI,EAAE,cAAc,GAAG,MAAM,IAAI;IAYnD,mBAAmB,CAAC,EAAE,EAAE,MAAM,GAAG,IAAI;IASrC,aAAa,CAAC,EAAE,EAAE,MAAM,GAAG,IAAI;IAY/B,cAAc,CAAC,QAAQ,EAAE,aAAa,GAAG,IAAI;IA4B7C,cAAc,CAAC,EAAE,EAAE,MAAM,EAAE,KAAK,EAAE,kBAAkB,GAAG,IAAI;IAc3D,cAAc,CAAC,EAAE,EAAE,MAAM,GAAG,IAAI;IAWhC,eAAe,CAAC,QAAQ,EAAE,gBAAgB,GAAG,MAAM,IAAI;IAKvD,cAAc,CAAC,MAAM,GAAE,mBAAwB,GAAG,aAAa,EAAE;IASjE,aAAa,CAAC,IAAI,EAAE,SAAS,GAAG,kBAAkB;IAsBlD,gBAAgB,CAAC,KAAK,EAAE,UAAU,GAAG,SAAS;IAW9C,OAAO,CAAC,mBAAmB;YAmBb,eAAe;IAI7B,OAAO,CAAC,YAAY;CAGrB;AAED,wBAAgB,gBAAgB,CAAC,OAAO,GAAE,iBAAsB,GAAG,UAAU,CAE5E"}

package/dist/components/workspace/admin-shell/state.svelte.js

@@ -1,3 +1,4 @@
+import { untrack } from 'svelte';
 import { LocalStorageShellSettingsAdapter, mergeShellSettingsDelta, resolveInitialPanelState, resolveShellConfig, } from './settings.js';
 import { PANEL_EDGES, SCOPE_EDGES } from './types.js';
 export class ShellState {
@@ -42,121 +43,148 @@ export class ShellState {
         };
     }
     applySettings(delta, options = {}) {
-        this.settings = mergeShellSettingsDelta(this.settings, delta);
-        for (const edge of PANEL_EDGES) {
-            this.panels[edge] = resolveInitialPanelState(edge, this.config.panels[edge], this.settings);
-        }
-        this.activeFocusToolId =
-            this.settings.activeFocusToolId ?? this.activeFocusToolId;
-        if (options.persist !== false)
-            void this.persistSettings();
+        const nextDelta = snapshotShellSettingsDelta(delta);
+        const persist = options.persist;
+        untrack(() => {
+            this.settings = mergeShellSettingsDelta(this.settings, nextDelta);
+            for (const edge of PANEL_EDGES) {
+                this.panels[edge] = resolveInitialPanelState(edge, this.config.panels[edge], this.settings);
+            }
+            this.activeFocusToolId =
+                this.settings.activeFocusToolId ?? this.activeFocusToolId;
+            if (persist !== false)
+                void this.persistSettings();
+        });
     }
     setHotkeysEnabled(enabled) {
-        this.applySettings({ hotkeysEnabled: enabled });
+        untrack(() => this.applySettings({ hotkeysEnabled: enabled }));
     }
     setHotkey(edge, code) {
-        this.applySettings({
-            keymap: {
-                [edge]: code ? { code } : null,
-            },
+        untrack(() => {
+            this.applySettings({
+                keymap: {
+                    [edge]: code ? { code } : null,
+                },
+            });
         });
     }
     setPanel(edge, state) {
-        if (this.config.panels[edge].initial === 'hidden') {
-            this.panels[edge] = 'hidden';
-            return;
-        }
-        if (state === 'expanded')
-            this.closeExclusivePeers(edge);
-        this.panels[edge] = state;
-        this.settings = mergeShellSettingsDelta(this.settings, {
-            panels: { [edge]: state },
+        untrack(() => {
+            if (this.config.panels[edge].initial === 'hidden') {
+                this.panels[edge] = 'hidden';
+                return;
+            }
+            if (state === 'expanded')
+                this.closeExclusivePeers(edge);
+            this.panels[edge] = state;
+            this.settings = mergeShellSettingsDelta(this.settings, {
+                panels: { [edge]: state },
+            });
+            void this.persistSettings();
         });
-        void this.persistSettings();
     }
     togglePanel(edge) {
-        if (this.panels[edge] === 'hidden')
-            return;
-        this.setPanel(edge, this.panels[edge] === 'expanded' ? 'collapsed' : 'expanded');
+        untrack(() => {
+            if (this.panels[edge] === 'hidden')
+                return;
+            this.setPanel(edge, this.panels[edge] === 'expanded' ? 'collapsed' : 'expanded');
+        });
     }
     expandPanel(edge) {
-        this.setPanel(edge, 'expanded');
+        untrack(() => this.setPanel(edge, 'expanded'));
     }
     collapsePanel(edge) {
-        this.setPanel(edge, 'collapsed');
+        untrack(() => this.setPanel(edge, 'collapsed'));
     }
     closeTopmostExpanded() {
-        for (const edge of [...PANEL_EDGES].reverse()) {
-            if (this.panels[edge] === 'expanded') {
-                this.collapsePanel(edge);
-                return true;
+        return untrack(() => {
+            for (const edge of [...PANEL_EDGES].reverse()) {
+                if (this.panels[edge] === 'expanded') {
+                    this.collapsePanel(edge);
+                    return true;
+                }
             }
-        }
-        return false;
+            return false;
+        });
     }
     registerFocusTool(tool) {
-        this.focusTools = [
-            ...this.focusTools.filter((existing) => existing.id !== tool.id),
-            tool,
-        ].sort((a, b) => (a.order ?? 0) - (b.order ?? 0));
-        if (!this.activeFocusToolId)
-            this.activeFocusToolId = tool.id;
-        return () => this.unregisterFocusTool(tool.id);
+        const nextTool = snapshotFocusTool(tool);
+        return untrack(() => {
+            this.focusTools = [
+                ...this.focusTools.filter((existing) => existing.id !== nextTool.id),
+                nextTool,
+            ].sort((a, b) => (a.order ?? 0) - (b.order ?? 0));
+            if (!this.activeFocusToolId)
+                this.activeFocusToolId = nextTool.id;
+            return () => this.unregisterFocusTool(nextTool.id);
+        });
     }
     unregisterFocusTool(id) {
-        this.focusTools = this.focusTools.filter((tool) => tool.id !== id);
-        if (this.activeFocusToolId === id) {
-            this.activeFocusToolId = this.focusTools[0]?.id ?? null;
-        }
+        untrack(() => {
+            this.focusTools = this.focusTools.filter((tool) => tool.id !== id);
+            if (this.activeFocusToolId === id) {
+                this.activeFocusToolId = this.focusTools[0]?.id ?? null;
+            }
+        });
     }
     openFocusTool(id) {
-        if (!this.focusTools.some((tool) => tool.id === id))
-            return;
-        this.activeFocusToolId = id;
-        this.settings = mergeShellSettingsDelta(this.settings, {
-            activeFocusToolId: id,
+        untrack(() => {
+            if (!this.focusTools.some((tool) => tool.id === id))
+                return;
+            this.activeFocusToolId = id;
+            this.settings = mergeShellSettingsDelta(this.settings, {
+                activeFocusToolId: id,
+            });
+            this.expandPanel('right');
+            void this.persistSettings();
         });
-        this.expandPanel('right');
-        void this.persistSettings();
     }
     upsertActivity(activity) {
-        const now = new Date().toISOString();
-        const normalized = {
-            ...activity,
-            edge: activity.edge ?? this.homeEdgeForScope(activity.scope),
-            createdAt: activity.createdAt ?? now,
-            updatedAt: activity.updatedAt ?? now,
-        };
-        const previous = this.activities.find((item) => item.id === activity.id);
-        this.activities = [
-            ...this.activities.filter((item) => item.id !== activity.id),
-            normalized,
-        ];
-        this.emitActivity({
-            type: previous && previous.status !== normalized.status
-                ? 'transition'
-                : 'upsert',
-            activity: normalized,
-            previous,
+        const nextActivity = snapshotActivity(activity);
+        untrack(() => {
+            const now = new Date().toISOString();
+            const normalized = {
+                ...nextActivity,
+                edge: nextActivity.edge ?? this.homeEdgeForScope(nextActivity.scope),
+                createdAt: nextActivity.createdAt ?? now,
+                updatedAt: nextActivity.updatedAt ?? now,
+            };
+            const previous = this.activities.find((item) => item.id === nextActivity.id);
+            this.activities = [
+                ...this.activities.filter((item) => item.id !== nextActivity.id),
+                normalized,
+            ];
+            this.emitActivity({
+                type: previous && previous.status !== normalized.status
+                    ? 'transition'
+                    : 'upsert',
+                activity: normalized,
+                previous,
+            });
         });
     }
     updateActivity(id, patch) {
-        const current = this.activities.find((activity) => activity.id === id);
-        if (!current)
-            return;
-        this.upsertActivity({
-            ...current,
-            ...patch,
-            id,
-            updatedAt: new Date().toISOString(),
+        const nextPatch = snapshotActivityPatch(patch);
+        untrack(() => {
+            const current = this.activities.find((activity) => activity.id === id);
+            if (!current)
+                return;
+            this.upsertActivity({
+                ...current,
+                ...nextPatch,
+                id,
+                updatedAt: new Date().toISOString(),
+            });
         });
     }
     removeActivity(id) {
-        const current = this.activities.find((activity) => activity.id === id);
-        if (!current)
-            return;
-        this.activities = this.activities.filter((activity) => activity.id !== id);
-        this.emitActivity({ type: 'remove', activity: current });
+        untrack(() => {
+            const current = this.activities.find((activity) => activity.id === id);
+            if (!current)
+                return;
+            this.activities = this.activities.filter((activity) => activity.id !== id);
+            this.emitActivity({ type: 'remove', activity: current });
+        });
     }
     watchActivities(listener) {
         this.activityListeners.add(listener);
@@ -250,3 +278,53 @@ function matchesOne(value, expected) {
 function isActiveStatus(status) {
     return status === 'queued' || status === 'running';
 }
+function snapshotShellSettingsDelta(delta) {
+    const snapshot = { ...delta };
+    if ('keymap' in delta) {
+        snapshot.keymap = snapshotShellKeymap(delta.keymap);
+    }
+    if ('panels' in delta) {
+        snapshot.panels = delta.panels ? { ...delta.panels } : delta.panels;
+    }
+    return snapshot;
+}
+function snapshotShellKeymap(keymap) {
+    if (!keymap)
+        return keymap;
+    const snapshot = {};
+    for (const edge of PANEL_EDGES) {
+        if (edge in keymap) {
+            const binding = keymap[edge];
+            snapshot[edge] = binding ? { ...binding } : binding;
+        }
+    }
+    return snapshot;
+}
+function snapshotFocusTool(tool) {
+    const snapshot = { ...tool };
+    if ('subject' in tool) {
+        snapshot.subject = tool.subject ? { ...tool.subject } : tool.subject;
+    }
+    if ('activityKinds' in tool) {
+        snapshot.activityKinds = tool.activityKinds
+            ? [...tool.activityKinds]
+            : tool.activityKinds;
+    }
+    return snapshot;
+}
+function snapshotActivity(activity) {
+    const snapshot = { ...activity };
+    if ('subject' in activity) {
+        snapshot.subject = activity.subject
+            ? { ...activity.subject }
+            : activity.subject;
+    }
+    return snapshot;
+}
+function snapshotActivityPatch(patch) {
+    const snapshot = { ...patch };
+    if ('subject' in patch) {
+        snapshot.subject = patch.subject ? { ...patch.subject } : patch.subject;
+    }
+    return snapshot;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@happyvertical/smrt-svelte",
-  "version": "0.38.5",
+  "version": "0.38.6",
   "description": "Svelte 5 components for SMRT user management - auth, users, tenants, roles, permissions, groups",
   "type": "module",
   "smrtRawPrimitives": "strict",
@@ -92,10 +92,10 @@
     "@tanstack/db": "^0.6.14",
     "@tanstack/svelte-db": "^0.1.91",
     "esm-env": "^1.2.2",
-    "@happyvertical/smrt-languages": "0.38.5",
-    "@happyvertical/smrt-types": "0.38.5",
-    "@happyvertical/smrt-ui": "0.38.5",
-    "@happyvertical/smrt-web": "0.38.5"
+    "@happyvertical/smrt-types": "0.38.6",
+    "@happyvertical/smrt-languages": "0.38.6",
+    "@happyvertical/smrt-ui": "0.38.6",
+    "@happyvertical/smrt-web": "0.38.6"
   },
   "peerDependencies": {
     "@huggingface/transformers": ">=3.8.1",