@jskit-ai/shell-web 0.1.162 → 0.1.164

package/test/bootstrapRuntime.test.js

@@ -1,6 +1,6 @@
 import assert from "node:assert/strict";
 import test from "node:test";
-import { registerBootstrapPayloadHandler } from "../src/client/bootstrap/bootstrapPayloadHandlerRegistry.js";
+import { createBootstrapPayloadHandlerRegistry } from "../src/client/bootstrap/bootstrapPayloadHandlerRegistry.js";
 import { createShellBootstrapRuntime } from "../src/client/runtime/bootstrapRuntime.js";
 
 function createPlacementRuntime(initialContext = {}) {
@@ -32,52 +32,6 @@ function createPlacementRuntime(initialContext = {}) {
   };
 }
 
-function createAppDouble({ placementRuntime, realtimeSocket = null } = {}) {
-  const singletons = new Map();
-  const singletonInstances = new Map();
-  return {
-    singleton(token, factory) {
-      singletons.set(token, factory);
-    },
-    tag(token, tagName) {
-      const current = this._tags.get(tagName) || [];
-      current.push(token);
-      this._tags.set(tagName, current);
-    },
-    resolveTag(tagName) {
-      return (this._tags.get(tagName) || []).map((token) => this.make(token));
-    },
-    _tags: new Map(),
-    has(token) {
-      if (token === "runtime.web-placement.client") {
-        return true;
-      }
-      if (token === "runtime.realtime.client.socket") {
-        return Boolean(realtimeSocket);
-      }
-      return singletons.has(token) || singletonInstances.has(token);
-    },
-    make(token) {
-      if (token === "runtime.web-placement.client") {
-        return placementRuntime;
-      }
-      if (token === "runtime.realtime.client.socket") {
-        return realtimeSocket;
-      }
-      if (singletonInstances.has(token)) {
-        return singletonInstances.get(token);
-      }
-      const factory = singletons.get(token);
-      if (!factory) {
-        throw new Error(`Unknown token ${String(token)}`);
-      }
-      const instance = factory(this);
-      singletonInstances.set(token, instance);
-      return instance;
-    }
-  };
-}
-
 test("shell bootstrap runtime refreshes /api/bootstrap on init and applies registered handlers", async () => {
   const placementRuntime = createPlacementRuntime({
     auth: {}
@@ -97,9 +51,8 @@ test("shell bootstrap runtime refreshes /api/bootstrap on init and applies regis
   const calls = [];
   const observedRequests = [];
   const observedResolveMeta = [];
-  const app = createAppDouble({ placementRuntime });
-  registerBootstrapPayloadHandler(app, "test.bootstrap.request", () =>
-    Object.freeze({
+  const handlers = createBootstrapPayloadHandlerRegistry();
+  handlers.register(Object.freeze({
       handlerId: "test.bootstrap.request",
       order: -10,
       resolveBootstrapRequest() {
@@ -115,10 +68,8 @@ test("shell bootstrap runtime refreshes /api/bootstrap on init and applies regis
       applyBootstrapPayload({ request }) {
         observedRequests.push(request);
       }
-    })
-  );
-  registerBootstrapPayloadHandler(app, "test.bootstrap.request-meta", () =>
-    Object.freeze({
+    }));
+  handlers.register(Object.freeze({
       handlerId: "test.bootstrap.request-meta",
       order: -5,
       resolveBootstrapRequest({ request }) {
@@ -126,10 +77,8 @@ test("shell bootstrap runtime refreshes /api/bootstrap on init and applies regis
         return {};
       },
       applyBootstrapPayload() {}
-    })
-  );
-  registerBootstrapPayloadHandler(app, "test.bootstrap.surfaceAccess", () =>
-    Object.freeze({
+    }));
+  handlers.register(Object.freeze({
       handlerId: "test.bootstrap.surfaceAccess",
       order: 0,
       applyBootstrapPayload({ payload, placementRuntime: targetRuntime }) {
@@ -137,11 +86,11 @@ test("shell bootstrap runtime refreshes /api/bootstrap on init and applies regis
           surfaceAccess: payload.surfaceAccess || {}
         });
       }
-    })
-  );
+    }));
 
   const runtime = createShellBootstrapRuntime({
-    app,
+    handlers,
+    placementRuntime,
     fetchImplementation: async (url) => {
       calls.push(String(url || ""));
       const payload = payloads.shift() || {};

package/test/pageAndPlacementPattern.test.js

@@ -0,0 +1,38 @@
+import assert from "node:assert/strict";
+import { readFile } from "node:fs/promises";
+import test from "node:test";
+import { parse as parseSfc } from "@vue/compiler-sfc";
+
+const patternRoot = new URL("../patterns/page-and-placement/example/", import.meta.url);
+const vueFiles = [
+  "src/pages/home/reports.vue",
+  "src/pages/home/reports/index.vue",
+  "src/pages/home/reports/overview.vue",
+  "src/pages/home/reports/activity.vue",
+  "src/components/SyncStatusElement.vue"
+];
+const jsFiles = [
+  "src/placement.js",
+  "src/placementTopology.js",
+  "packages/main/src/client/providers/MainClientProvider.js"
+];
+
+test("page and placement pattern is executable, semantic application source", async () => {
+  const sources = [];
+  for (const relativePath of vueFiles) {
+    const source = await readFile(new URL(relativePath, patternRoot), "utf8");
+    sources.push(source);
+    const parsed = parseSfc(source, { filename: relativePath });
+    assert.equal(parsed.errors.length, 0, `${relativePath}: ${parsed.errors.join("\n")}`);
+  }
+  for (const relativePath of jsFiles) {
+    const source = await readFile(new URL(relativePath, patternRoot), "utf8");
+    sources.push(source);
+    assert.match(source, /(?:export|addPlacement|registerMainClientComponent)/u, relativePath);
+  }
+
+  const combined = sources.join("\n");
+  assert.match(combined, /createPlacementRegistry|addPlacementTopology|ShellOutlet/u);
+  assert.doesNotMatch(combined, /ui-generator|generated-ui|scaffold|receipt|provenance/u);
+  assert.doesNotMatch(combined, /replace this|ready for your/u);
+});

package/test/placementRuntime.test.js

@@ -3,22 +3,16 @@ import test from "node:test";
 import { definePlacement } from "../src/client/placement/validators.js";
 import { createWebPlacementRuntime } from "../src/client/placement/runtime.js";
 
-function createAppStub({ tokens = {}, contextContributors = [] } = {}) {
+function createComponentsStub({ tokens = {} } = {}) {
   return {
     has(token) {
       return Object.prototype.hasOwnProperty.call(tokens, token);
     },
-    make(token) {
+    get(token) {
       if (!this.has(token)) {
         throw new Error(`Unknown token: ${String(token)}`);
       }
       return tokens[token];
-    },
-    resolveTag(tagName) {
-      if (tagName === "web-placement.context.client") {
-        return contextContributors;
-      }
-      return [];
     }
   };
 }
@@ -60,14 +54,14 @@ function semanticTopologyEntry({
 }
 
 test("web placement runtime resolves semantic targets through topology variants", () => {
-  const app = createAppStub({
+  const components = createComponentsStub({
     tokens: {
       "component.bottom": () => null,
       "component.menu": () => null
     }
   });
 
-  const runtime = createWebPlacementRuntime({ app });
+  const runtime = createWebPlacementRuntime({ components });
   runtime.replacePlacementTopology([
     semanticTopologyEntry({
       id: "shell.primary-nav",
@@ -109,13 +103,13 @@ test("web placement runtime resolves semantic targets through topology variants"
 });
 
 test("web placement runtime accepts append-only topology objects", () => {
-  const app = createAppStub({
+  const components = createComponentsStub({
     tokens: {
       "component.menu": () => null
     }
   });
 
-  const runtime = createWebPlacementRuntime({ app });
+  const runtime = createWebPlacementRuntime({ components });
   runtime.replacePlacementTopology({
     placements: [
       semanticTopologyEntry({
@@ -146,7 +140,7 @@ test("web placement runtime accepts append-only topology objects", () => {
 });
 
 test("web placement runtime filters by surface/host/position, resolves component tokens, and sorts by order", () => {
-  const app = createAppStub({
+  const components = createComponentsStub({
     tokens: {
       "component.alerts": () => null,
       "component.profile": () => null,
@@ -154,7 +148,7 @@ test("web placement runtime filters by surface/host/position, resolves component
     }
   });
 
-  const runtime = createWebPlacementRuntime({ app });
+  const runtime = createWebPlacementRuntime({ components });
   runtime.replacePlacements([
     definePlacement({
       id: "test.menu",
@@ -195,14 +189,14 @@ test("web placement runtime filters by surface/host/position, resolves component
 });
 
 test("web placement runtime preserves source order when placements share the same order", () => {
-  const app = createAppStub({
+  const components = createComponentsStub({
     tokens: {
       "component.beta": () => null,
       "component.alpha": () => null
     }
   });
 
-  const runtime = createWebPlacementRuntime({ app });
+  const runtime = createWebPlacementRuntime({ components });
   runtime.replacePlacements([
     definePlacement({
       id: "test.beta",
@@ -228,19 +222,19 @@ test("web placement runtime preserves source order when placements share the sam
 });
 
 test("web placement runtime applies context contributors and placement when() predicates", () => {
-  const app = createAppStub({
+  const components = createComponentsStub({
     tokens: {
       "component.guest": () => null,
       "component.authenticated": () => null
-    },
+    }
+  });
+
+  const runtime = createWebPlacementRuntime({
+    components,
     contextContributors: [
-      () => ({
-        auth: { authenticated: true }
-      })
+      () => ({ auth: { authenticated: true } })
     ]
   });
-
-  const runtime = createWebPlacementRuntime({ app });
   runtime.replacePlacements([
     definePlacement({
       id: "guest.item",
@@ -265,20 +259,18 @@ test("web placement runtime applies context contributors and placement when() pr
 });
 
 test("web placement runtime uses runtime context and local context overrides contributor values", () => {
-  const app = createAppStub({
+  const components = createComponentsStub({
     tokens: {
       "component.allowed": () => null
-    },
+    }
+  });
+
+  const runtime = createWebPlacementRuntime({
+    components,
     contextContributors: [
-      () => ({
-        auth: {
-          authenticated: false
-        }
-      })
+      () => ({ auth: { authenticated: false } })
     ]
   });
-
-  const runtime = createWebPlacementRuntime({ app });
   runtime.replacePlacements([
     definePlacement({
       id: "allowed",
@@ -311,8 +303,8 @@ test("web placement runtime uses runtime context and local context overrides con
 });
 
 test("web placement runtime notifies subscribers on placement and context updates", () => {
-  const app = createAppStub();
-  const runtime = createWebPlacementRuntime({ app });
+  const components = createComponentsStub();
+  const runtime = createWebPlacementRuntime({ components });
   const events = [];
   const unsubscribe = runtime.subscribe((event) => {
     events.push(event.type);
@@ -336,8 +328,8 @@ test("web placement runtime notifies subscribers on placement and context update
 });
 
 test("web placement runtime rejects duplicate placement ids", () => {
-  const app = createAppStub();
-  const runtime = createWebPlacementRuntime({ app });
+  const components = createComponentsStub();
+  const runtime = createWebPlacementRuntime({ components });
 
   assert.throws(() => {
     runtime.replacePlacements([
@@ -360,11 +352,11 @@ test("web placement runtime rejects duplicate placement ids", () => {
 });
 
 test("web placement runtime skips throwing component tokens and logs resolution errors once", () => {
-  const app = {
+  const components = {
     has(token) {
       return token === "component.bad" || token === "component.good";
     },
-    make(token) {
+    get(token) {
       if (token === "component.bad") {
         throw new Error("bad component token");
       }
@@ -372,15 +364,12 @@ test("web placement runtime skips throwing component tokens and logs resolution
         return () => null;
       }
       throw new Error(`Unknown token: ${String(token)}`);
-    },
-    resolveTag() {
-      return [];
     }
   };
 
   const errors = [];
   const runtime = createWebPlacementRuntime({
-    app,
+    components,
     logger: {
       warn() {},
       error(payload, message) {
@@ -417,11 +406,11 @@ test("web placement runtime skips throwing component tokens and logs resolution
 
 test("web placement runtime clears failed token cache when placements are replaced", () => {
   let shouldThrow = true;
-  const app = {
+  const components = {
     has(token) {
       return token === "component.toggle";
     },
-    make(token) {
+    get(token) {
       if (token !== "component.toggle") {
         throw new Error(`Unknown token: ${String(token)}`);
       }
@@ -429,14 +418,11 @@ test("web placement runtime clears failed token cache when placements are replac
         throw new Error("toggle failure");
       }
       return () => null;
-    },
-    resolveTag() {
-      return [];
     }
   };
 
   const runtime = createWebPlacementRuntime({
-    app,
+    components,
     logger: {
       warn() {},
       error() {}
@@ -475,14 +461,14 @@ test("web placement runtime clears failed token cache when placements are replac
 });
 
 test("web placement runtime follows explicit surface targeting without role indirection", () => {
-  const app = createAppStub({
+  const components = createComponentsStub({
     tokens: {
       "component.global": () => null,
       "component.app": () => null,
       "component.admin": () => null
     }
   });
-  const runtime = createWebPlacementRuntime({ app });
+  const runtime = createWebPlacementRuntime({ components });
   runtime.replacePlacements([
     definePlacement({
       id: "global.banner",

package/test/playwrightContract.test.js

@@ -9,17 +9,6 @@ import {
 
 const PACKAGE_ROOT = path.resolve(path.dirname(fileURLToPath(import.meta.url)), "..");
 
-test("shell-web owns the exact Playwright dependency used by its generated smoke", async () => {
-  const packageJson = JSON.parse(await readFile(path.join(PACKAGE_ROOT, "package.json"), "utf8"));
-  const rootPackage = JSON.parse(await readFile(path.resolve(PACKAGE_ROOT, "../../package.json"), "utf8"));
-
-  assert.equal(packageJson.jskit.mutations.dependencies.dev["@playwright/test"], "1.61.1");
-  assert.equal(
-    packageJson.jskit.mutations.dependencies.dev["@playwright/test"],
-    rootPackage.devDependencies["@playwright/test"]
-  );
-});
-
 test("adaptive shell smoke navigates through Playwright baseURL", async () => {
   const source = await readFile(path.join(PACKAGE_ROOT, "src/test/adaptiveShellSmoke.js"), "utf8");
 
@@ -69,7 +58,8 @@ test("adaptive shell smoke follows rendered layout state and waits for drawer tr
   assert.match(source, /data-rail-width/u);
   assert.match(source, /expect\.poll/u);
   assert.match(source, /toBeFocused/u);
-  assert.match(source, /v-navigation-drawer__scrim/u);
+  assert.match(source, /getByRole\("button", \{ name: "Close navigation menu" \}\)/u);
+  assert.doesNotMatch(source, /v-navigation-drawer__scrim/u);
   assert.match(source, /page\.keyboard\.press\("Escape"\)[\s\S]*data-presentation", "drawer"/u);
   assert.match(source, /centers\.push\(box\.x \+ box\.width \/ 2\)/u);
   assert.match(source, /fit\.endGap - configuredSpacing/u);