sh3-core 0.7.1 → 0.7.5

package/dist/overlays/FloatFrame.svelte

@@ -30,6 +30,7 @@
 
   function onHeaderPointerDown(e: PointerEvent): void {
     if (e.button !== 0) return;
+    if ((e.target as HTMLElement).closest('.sh3-float-close')) return;
     const target = e.currentTarget as HTMLElement;
     target.setPointerCapture(e.pointerId);
     dragging = true;

package/dist/overlays/float.d.ts

@@ -7,6 +7,8 @@ export interface FloatOptions {
         y: number;
     };
     size?: Size;
+    /** Instance data threaded to the view factory via `MountContext.meta`. */
+    meta?: Record<string, unknown>;
 }
 export interface FloatManager {
     open(viewId: string, options?: FloatOptions): string;

package/dist/overlays/float.js

@@ -74,9 +74,12 @@ function openFloat(viewId, options = {}) {
     // float body; the frame header still moves the float as a whole.
     const slotId = mintFloatSlotId(viewId);
     const label = (_a = options.title) !== null && _a !== void 0 ? _a : viewId;
+    const tab = { slotId, viewId, label };
+    if (options.meta)
+        tab.meta = options.meta;
     const content = {
         type: 'tabs',
-        tabs: [{ slotId, viewId, label }],
+        tabs: [tab],
         activeTab: 0,
     };
     const computedMin = computeMinSize(content);

package/dist/overlays/float.test.js

@@ -1,5 +1,6 @@
 import { describe, it, expect, beforeEach } from 'vitest';
-import { floatManager, __resetFloatManagerForTest } from './float';
+import { floatManager, __resetFloatManagerForTest, bindFloatStore } from './float';
+import { layoutStore } from '../layout/store.svelte';
 describe('floatManager', () => {
     beforeEach(() => {
         __resetFloatManagerForTest();
@@ -34,4 +35,104 @@ describe('floatManager', () => {
         expect(f.position).toEqual({ x: 100, y: 200 });
         expect(f.size).toEqual({ w: 800, h: 500 });
     });
+    it('open() threads meta into the content TabEntry', () => {
+        const meta = { viewConfigId: 'vc-42' };
+        const id = floatManager.open('test:view', { meta });
+        const f = floatManager.list().find((e) => e.id === id);
+        const tabs = f.content;
+        expect(tabs.type).toBe('tabs');
+        if (tabs.type === 'tabs') {
+            expect(tabs.tabs[0].meta).toEqual({ viewConfigId: 'vc-42' });
+        }
+    });
+    it('open() without meta leaves TabEntry.meta undefined', () => {
+        const id = floatManager.open('test:view');
+        const f = floatManager.list().find((e) => e.id === id);
+        const tabs = f.content;
+        expect(tabs.type).toBe('tabs');
+        if (tabs.type === 'tabs') {
+            expect(tabs.tabs[0].meta).toBeUndefined();
+        }
+    });
+});
+// ---------------------------------------------------------------------------
+// DOM tests — floatManager + FloatLayer.svelte in happy-dom
+// ---------------------------------------------------------------------------
+import { renderWithShell } from '../__test__/render';
+import FloatLayer from './FloatLayer.svelte';
+import { tick } from 'svelte';
+import { resetFramework } from '../__test__/reset';
+/**
+ * Wire the floatManager to the same FloatEntry[] that FloatLayer reads
+ * (layoutStore.floats → HOME_TREE.floats). Without this binding, the
+ * manager writes to its internal fallback array, which the component
+ * never observes.
+ */
+function bindManagerToStore() {
+    bindFloatStore(layoutStore.floats, () => ({ w: 1024, h: 768 }));
+}
+// ---------------------------------------------------------------------------
+// F.1 — open mounts a frame into the DOM; close removes it
+// ---------------------------------------------------------------------------
+describe('floats — F.1 open/close mounts DOM', () => {
+    beforeEach(() => {
+        resetFramework();
+        bindManagerToStore();
+    });
+    it('mounts a FloatFrame on open() and removes it on close()', async () => {
+        const { container } = renderWithShell(FloatLayer, {});
+        const id = floatManager.open('test:view', { title: 'Test Float' });
+        await tick();
+        // FloatFrame renders a div[role="dialog"] with the title as aria-label.
+        const frame = container.querySelector('[role="dialog"][aria-label="Test Float"]');
+        expect(frame).toBeTruthy();
+        floatManager.close(id);
+        await tick();
+        expect(container.querySelector('[role="dialog"][aria-label="Test Float"]')).toBeNull();
+    });
+});
+// ---------------------------------------------------------------------------
+// F.2 — focus stack: last opened is top, previous is restored after close
+// ---------------------------------------------------------------------------
+describe('floats — F.2 focus stack', () => {
+    beforeEach(() => {
+        resetFramework();
+        bindManagerToStore();
+    });
+    it('raises the newer float above the prior and restores previous on close', async () => {
+        var _a, _b;
+        renderWithShell(FloatLayer, {});
+        const id1 = floatManager.open('test:view', { title: 'First' });
+        const id2 = floatManager.open('test:view', { title: 'Second' });
+        await tick();
+        // The most recently focused float sits at the end of list() (top z-order).
+        expect((_a = floatManager.list().at(-1)) === null || _a === void 0 ? void 0 : _a.id).toBe(id2);
+        floatManager.close(id2);
+        await tick();
+        // After closing the top, id1 becomes the sole entry (top of z-order).
+        expect((_b = floatManager.list().at(-1)) === null || _b === void 0 ? void 0 : _b.id).toBe(id1);
+    });
+});
+// ---------------------------------------------------------------------------
+// F.3 — close button in FloatFrame calls floatManager.close()
+// ---------------------------------------------------------------------------
+describe('floats — F.3 close button removes float', () => {
+    beforeEach(() => {
+        resetFramework();
+        bindManagerToStore();
+    });
+    it('clicking the close button removes the float from list()', async () => {
+        const { container } = renderWithShell(FloatLayer, {});
+        const id = floatManager.open('test:view', { title: 'Closeable' });
+        await tick();
+        // FloatFrame renders a button[aria-label="Close float"] inside the frame.
+        const closeBtn = container.querySelector('[role="dialog"] button[aria-label="Close float"]');
+        expect(closeBtn).toBeTruthy();
+        closeBtn.click();
+        await tick();
+        // Float must be gone from the manager's list.
+        expect(floatManager.list().some((f) => f.id === id)).toBe(false);
+        // And its frame must be removed from the DOM.
+        expect(container.querySelector('[role="dialog"][aria-label="Closeable"]')).toBeNull();
+    });
 });

package/dist/primitives/ResizableSplitter.svelte

@@ -221,10 +221,12 @@
         class="splitter-handle"
         class:dragging={drag?.handleIndex === i}
         class:disabled={isCollapsed(i) || isCollapsed(i + 1)}
+        data-testid="splitter-handle-{i}"
         onpointerdown={(e) => beginDrag(e, i)}
         onpointermove={moveDrag}
         onpointerup={endDrag}
         onpointercancel={endDrag}
+        ondblclick={() => onCollapseToggle?.(i, !isCollapsed(i))}
         role="separator"
         aria-orientation={direction === 'horizontal' ? 'vertical' : 'horizontal'}
       ></div>

package/dist/primitives/TabbedPanel.svelte

@@ -76,6 +76,7 @@
     closable,
     dirty,
     onClose,
+    tabIds,
   }: {
     labels: string[];
     icons?: (string | undefined)[];
@@ -100,6 +101,8 @@
     dirty?: (boolean | undefined)[];
     /** Called when the user clicks a tab's close button. */
     onClose?: (index: number) => void;
+    /** Optional stable ids for each tab (e.g. slotId). Used as data-testid suffixes on close buttons. */
+    tabIds?: (string | undefined)[];
   } = $props();
 
   function select(i: number) {
@@ -172,6 +175,7 @@
             role="button"
             tabindex="-1"
             title="Close"
+            data-testid={tabIds?.[i] ? `tab-close-${tabIds[i]}` : undefined}
             onclick={(e) => handleClose(i, e)}
             onkeydown={(e) => { if (e.key === 'Enter' || e.key === ' ') handleClose(i, e); }}
           >&#x2715;</span>

package/dist/primitives/TabbedPanel.svelte.d.ts

@@ -44,6 +44,8 @@ type $$ComponentProps = {
     dirty?: (boolean | undefined)[];
     /** Called when the user clicks a tab's close button. */
     onClose?: (index: number) => void;
+    /** Optional stable ids for each tab (e.g. slotId). Used as data-testid suffixes on close buttons. */
+    tabIds?: (string | undefined)[];
 };
 declare const TabbedPanel: import("svelte").Component<$$ComponentProps, {}, "">;
 type TabbedPanel = ReturnType<typeof TabbedPanel>;

package/dist/shards/activate.svelte.d.ts

@@ -50,3 +50,9 @@ export declare function isActive(id: string): boolean;
  * Used by lifecycle.ts to pass context to `shard.resume()`.
  */
 export declare function getShardContext(id: string): ShardContext | undefined;
+/**
+ * Test-only reset. Tears down any active shard entries (without running
+ * deactivate hooks — tests should run deactivate explicitly if they care)
+ * and clears both registered and active maps.
+ */
+export declare function __resetShardRegistryForTest(): void;

package/dist/shards/activate.svelte.js

@@ -191,3 +191,13 @@ export function getShardContext(id) {
     var _a;
     return (_a = active.get(id)) === null || _a === void 0 ? void 0 : _a.ctx;
 }
+/**
+ * Test-only reset. Tears down any active shard entries (without running
+ * deactivate hooks — tests should run deactivate explicitly if they care)
+ * and clears both registered and active maps.
+ */
+export function __resetShardRegistryForTest() {
+    active.clear();
+    activeShards.clear();
+    registeredShards.clear();
+}

package/dist/shards/registry.d.ts

@@ -1,4 +1,6 @@
 import type { ViewFactory } from './types';
+export declare function __addViewRegistrationListener(fn: (viewId: string, factory: ViewFactory) => void): void;
+export declare function __removeViewRegistrationListener(fn: (viewId: string, factory: ViewFactory) => void): void;
 export declare function registerView(viewId: string, factory: ViewFactory): void;
 export declare function getView(viewId: string): ViewFactory | undefined;
 export declare function unregisterView(viewId: string): void;
@@ -7,3 +9,5 @@ export declare function registerVerb(name: string, verb: Verb): void;
 export declare function getVerb(name: string): Verb | undefined;
 export declare function unregisterVerb(name: string): void;
 export declare function listVerbs(): Verb[];
+/** Test-only reset: clear the view and verb registries. */
+export declare function __resetViewRegistryForTest(): void;

package/dist/shards/registry.js

@@ -14,11 +14,22 @@
  *     hotkeys get their own sibling maps when those kinds land.
  */
 const views = new Map();
+/** Listeners called after a new view factory is registered. */
+const viewRegistrationListeners = new Set();
+export function __addViewRegistrationListener(fn) {
+    viewRegistrationListeners.add(fn);
+}
+export function __removeViewRegistrationListener(fn) {
+    viewRegistrationListeners.delete(fn);
+}
 export function registerView(viewId, factory) {
     if (views.has(viewId)) {
         throw new Error(`View "${viewId}" is already registered`);
     }
     views.set(viewId, factory);
+    for (const listener of viewRegistrationListeners) {
+        listener(viewId, factory);
+    }
 }
 export function getView(viewId) {
     return views.get(viewId);
@@ -42,3 +53,10 @@ export function unregisterVerb(name) {
 export function listVerbs() {
     return Array.from(verbs.values()).sort((a, b) => a.name.localeCompare(b.name));
 }
+/** Test-only reset: clear the view and verb registries. */
+export function __resetViewRegistryForTest() {
+    views.clear();
+    verbs.clear();
+    // Do NOT clear viewRegistrationListeners — they are module-level subscriptions
+    // (e.g. slotHostPool's late-factory listener) that must survive registry resets.
+}

package/dist/shards/types.d.ts

@@ -43,6 +43,12 @@ export interface MountContext {
     viewId: string;
     /** Initial label for the tab; may be updated by the view via future API. */
     label: string;
+    /**
+     * Caller-supplied instance data. Present when the mount was triggered with
+     * metadata (e.g. `shell.float.open(viewId, { meta: { ... } })`).
+     * Not persisted with the layout — ephemeral per mount.
+     */
+    meta?: Record<string, unknown>;
     /**
      * Push dirty-state to the tab strip. The framework renders a dirty
      * indicator (filled dot) on the tab when true, clears it when false.

package/dist/version.d.ts

@@ -1,2 +1,2 @@
 /** Auto-generated from package.json — do not edit manually. */
-export declare const VERSION = "0.7.1";
+export declare const VERSION = "0.7.5";

package/dist/version.js

@@ -1,2 +1,2 @@
 /** Auto-generated from package.json — do not edit manually. */
-export const VERSION = '0.7.1';
+export const VERSION = '0.7.5';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sh3-core",
-  "version": "0.7.1",
+  "version": "0.7.5",
   "type": "module",
   "files": [
     "dist"
@@ -33,6 +33,9 @@
     "check": "svelte-check --tsconfig ./tsconfig.json",
     "pack": "npm run build && npm pack",
     "test": "vitest run --passWithNoTests",
+    "test:node": "vitest run --project node --passWithNoTests",
+    "test:dom": "vitest run --project dom --passWithNoTests",
+    "test:browser": "vitest run --project browser --passWithNoTests",
     "test:watch": "vitest"
   },
   "dependencies": {
@@ -50,7 +53,12 @@
   "devDependencies": {
     "@sveltejs/package": "^2.3.0",
     "@sveltejs/vite-plugin-svelte": "^4.0.0",
+    "@testing-library/jest-dom": "^6.9.1",
+    "@testing-library/svelte": "^5.3.1",
     "@tsconfig/svelte": "^5.0.4",
+    "@vitest/browser": "^2.1.9",
+    "happy-dom": "^15.11.7",
+    "playwright": "^1.59.1",
     "svelte": "^5.0.0",
     "svelte-check": "^4.0.0",
     "tsx": "^4.21.0",