@jskit-ai/shell-web 0.1.152 → 0.1.154

package/test/adaptiveShell.browser.test.js

@@ -1,110 +1,23 @@
 import assert from "node:assert/strict";
-import { spawn } from "node:child_process";
-import net from "node:net";
 import path from "node:path";
 import process from "node:process";
 import test from "node:test";
 import { fileURLToPath } from "node:url";
 import { chromium, expect } from "@playwright/test";
+import {
+  createChromiumLaunchOptions,
+  startViteFixture,
+  stopProcess
+} from "../../../tooling/testUtils/browserFixture.mjs";
 import {
   DEFAULT_VIEWPORTS,
   runAdaptiveShellSmokeCase
 } from "../src/test/adaptiveShellSmoke.js";
 
 const TEST_DIRECTORY = path.dirname(fileURLToPath(import.meta.url));
-const REPOSITORY_ROOT = path.resolve(TEST_DIRECTORY, "../../..");
 const FIXTURE_ROOT = path.resolve(TEST_DIRECTORY, "../fixtures/adaptive-shell");
-const VITE_CLI = path.join(REPOSITORY_ROOT, "node_modules", "vite", "bin", "vite.js");
 const RUN_BROWSER_TEST = process.env.JSKIT_SHELL_WEB_BROWSER_INTEGRATION === "1";
 
-function startVite(port) {
-  const child = spawn(process.execPath, [
-    VITE_CLI,
-    "--config",
-    path.join(FIXTURE_ROOT, "vite.config.mjs"),
-    "--port",
-    String(port),
-    "--clearScreen",
-    "false"
-  ], {
-    cwd: FIXTURE_ROOT,
-    stdio: ["ignore", "pipe", "pipe"],
-    env: {
-      ...process.env,
-      NO_COLOR: "1",
-      FORCE_COLOR: "0"
-    }
-  });
-  let output = "";
-  child.stdout.on("data", collectOutput);
-  child.stderr.on("data", collectOutput);
-
-  function collectOutput(chunk) {
-    output += chunk.toString("utf8");
-  }
-
-  async function waitUntilReady() {
-    await new Promise((resolve, reject) => {
-      const startedAt = Date.now();
-      const timer = setInterval(checkReadiness, 50);
-
-      function checkReadiness() {
-        if (new RegExp(`http://127\\.0\\.0\\.1:${port}/`, "u").test(output)) {
-          clearInterval(timer);
-          resolve();
-          return;
-        }
-        if (child.exitCode !== null) {
-          clearInterval(timer);
-          reject(new Error(`Vite exited before becoming ready.\n${output}`));
-          return;
-        }
-        if (Date.now() - startedAt >= 30_000) {
-          clearInterval(timer);
-          reject(new Error(`Timed out waiting for Vite.\n${output}`));
-        }
-      }
-    });
-  }
-
-  return Object.freeze({ child, waitUntilReady });
-}
-
-async function stopVite(runtime) {
-  const child = runtime?.child;
-  if (!child || child.exitCode !== null || child.signalCode !== null) {
-    return;
-  }
-  await new Promise((resolve) => {
-    const forceTimer = setTimeout(forceStop, 5_000);
-    child.once("exit", finishStop);
-    child.kill("SIGTERM");
-
-    function forceStop() {
-      child.kill("SIGKILL");
-    }
-
-    function finishStop() {
-      clearTimeout(forceTimer);
-      resolve();
-    }
-  });
-}
-
-async function reservePort() {
-  const server = net.createServer();
-  await new Promise((resolve, reject) => {
-    server.once("error", reject);
-    server.listen(0, "127.0.0.1", resolve);
-  });
-  const address = server.address();
-  assert.ok(address && typeof address === "object");
-  await new Promise((resolve, reject) => {
-    server.close((error) => error ? reject(error) : resolve());
-  });
-  return address.port;
-}
-
 function parseRgb(color) {
   const channels = String(color || "").match(/[\d.]+/gu)?.map(Number) || [];
   return {
@@ -260,25 +173,22 @@ test("shell-web adaptive navigation passes package-owned browser contracts", {
     : "set JSKIT_SHELL_WEB_BROWSER_INTEGRATION=1 to run shell-web browser integration",
   timeout: 180_000
 }, async () => {
-  const port = await reservePort();
-  const vite = startVite(port);
+  const vite = await startViteFixture({ fixtureRoot: FIXTURE_ROOT });
   let browser = null;
 
   try {
-    await vite.waitUntilReady();
-    const executablePath = String(process.env.PLAYWRIGHT_CHROMIUM_EXECUTABLE_PATH || "").trim();
-    browser = await chromium.launch({
-      headless: true,
-      ...(executablePath ? { executablePath } : {})
-    });
+    browser = await chromium.launch(createChromiumLaunchOptions());
     const context = await browser.newContext({
-      baseURL: `http://127.0.0.1:${port}`,
+      baseURL: vite.baseURL,
       locale: "en-US"
     });
 
     for (const viewport of DEFAULT_VIEWPORTS) {
       const page = await context.newPage();
       await runAdaptiveShellSmokeCase({ page, expect, viewport });
+      if (viewport.name !== "compact") {
+        await expect(page).toHaveURL(/\/home\/settings\/general$/u);
+      }
       await page.close();
     }
 
@@ -324,6 +234,6 @@ test("shell-web adaptive navigation passes package-owned browser contracts", {
     await context.close();
   } finally {
     await browser?.close();
-    await stopVite(vite);
+    await stopProcess(vite);
   }
 });

package/test/bootstrapClaimContract.test.js

@@ -3,14 +3,16 @@ import path from "node:path";
 import test from "node:test";
 import { readFile } from "node:fs/promises";
 import { fileURLToPath } from "node:url";
-import descriptor from "../package.descriptor.mjs";
+import packageJson from "../package.json" with { type: "json" };
+
+const packageMetadata = packageJson.jskit;
 
 const TEST_DIRECTORY = path.dirname(fileURLToPath(import.meta.url));
 const PACKAGE_DIR = path.resolve(TEST_DIRECTORY, "..");
 const CREATE_APP_TEMPLATE_DIR = path.resolve(PACKAGE_DIR, "..", "..", "tooling", "create-app", "templates", "minimal-shell");
 
 function findFileMutation(id) {
-  const files = descriptor?.mutations?.files;
+  const files = packageMetadata?.mutations?.files;
   return Array.isArray(files)
     ? files.find((entry) => String(entry?.id || "").trim() === id) || null
     : null;

package/test/linkItemScaffoldContract.test.js

@@ -3,7 +3,9 @@ import path from "node:path";
 import test from "node:test";
 import { readFile } from "node:fs/promises";
 import { fileURLToPath } from "node:url";
-import descriptor from "../package.descriptor.mjs";
+import packageJson from "../package.json" with { type: "json" };
+
+const packageMetadata = packageJson.jskit;
 import {
   LOCAL_LINK_ITEM_COMPONENT_DEFINITIONS,
   findLocalLinkItemDefinition,
@@ -14,14 +16,14 @@ const TEST_DIRECTORY = path.dirname(fileURLToPath(import.meta.url));
 const PACKAGE_DIR = path.resolve(TEST_DIRECTORY, "..");
 
 function findFileMutation(id) {
-  const files = descriptor?.mutations?.files;
+  const files = packageMetadata?.mutations?.files;
   return Array.isArray(files)
     ? files.find((entry) => String(entry?.id || "").trim() === id) || null
     : null;
 }
 
 function findSourceMutation(id) {
-  const sourceMutations = descriptor?.mutations?.source;
+  const sourceMutations = packageMetadata?.mutations?.source;
   return Array.isArray(sourceMutations)
     ? sourceMutations.find((entry) => String(entry?.id || "").trim() === id) || null
     : null;

package/test/playwrightContract.test.js

@@ -3,16 +3,19 @@ import { readFile } from "node:fs/promises";
 import path from "node:path";
 import test from "node:test";
 import { fileURLToPath } from "node:url";
+import {
+  isRequestedOrDescendantLocation
+} from "../src/test/adaptiveShellSmoke.js";
 
 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 descriptor = (await import(`${path.join(PACKAGE_ROOT, "package.descriptor.mjs")}?test=${Date.now()}`)).default;
+  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(descriptor.mutations.dependencies.dev["@playwright/test"], "1.61.1");
+  assert.equal(packageJson.jskit.mutations.dependencies.dev["@playwright/test"], "1.61.1");
   assert.equal(
-    descriptor.mutations.dependencies.dev["@playwright/test"],
+    packageJson.jskit.mutations.dependencies.dev["@playwright/test"],
     rootPackage.devDependencies["@playwright/test"]
   );
 });
@@ -25,6 +28,39 @@ test("adaptive shell smoke navigates through Playwright baseURL", async () => {
   assert.doesNotMatch(source, /http:\/\/127\.0\.0\.1/u);
 });
 
+test("adaptive shell navigation accepts canonical descendants without accepting prefix siblings", () => {
+  const requested = new URL("https://example.test/home/settings?tab=profile");
+
+  assert.equal(
+    isRequestedOrDescendantLocation(
+      new URL("https://example.test/home/settings?tab=profile"),
+      requested
+    ),
+    true
+  );
+  assert.equal(
+    isRequestedOrDescendantLocation(
+      new URL("https://example.test/home/settings/general?tab=profile"),
+      requested
+    ),
+    true
+  );
+  assert.equal(
+    isRequestedOrDescendantLocation(
+      new URL("https://example.test/home/settings-other?tab=profile"),
+      requested
+    ),
+    false
+  );
+  assert.equal(
+    isRequestedOrDescendantLocation(
+      new URL("https://example.test/home/settings/general?tab=security"),
+      requested
+    ),
+    false
+  );
+});
+
 test("adaptive shell smoke follows rendered layout state and waits for drawer transitions", async () => {
   const source = await readFile(path.join(PACKAGE_ROOT, "src/test/adaptiveShellSmoke.js"), "utf8");
 

package/test/settingsPlacementContract.test.js

@@ -4,14 +4,16 @@ import test from "node:test";
 import { readFile } from "node:fs/promises";
 import { fileURLToPath } from "node:url";
 import { assertGeneratedUiSourceContract } from "@jskit-ai/kernel/shared/support/generatedUiContract";
-import descriptor from "../package.descriptor.mjs";
+import packageJson from "../package.json" with { type: "json" };
+
+const packageMetadata = packageJson.jskit;
 import { resolveShellRouteTransitionKey } from "../src/client/support/routeTransitionKey.js";
 
 const TEST_DIRECTORY = path.dirname(fileURLToPath(import.meta.url));
 const PACKAGE_DIR = path.resolve(TEST_DIRECTORY, "..");
 
 function readOutlets(target = "") {
-  const outlets = descriptor?.metadata?.ui?.placements?.outlets;
+  const outlets = packageMetadata?.metadata?.ui?.placements?.outlets;
   const normalizedTarget = String(target || "").trim();
   return Array.isArray(outlets)
     ? outlets.filter((entry) => String(entry?.target || "").trim() === normalizedTarget)
@@ -19,7 +21,7 @@ function readOutlets(target = "") {
 }
 
 function readContributions(target = "") {
-  const contributions = descriptor?.metadata?.ui?.placements?.contributions;
+  const contributions = packageMetadata?.metadata?.ui?.placements?.contributions;
   const normalizedTarget = String(target || "").trim();
   return Array.isArray(contributions)
     ? contributions.filter((entry) => String(entry?.target || "").trim() === normalizedTarget)
@@ -27,7 +29,7 @@ function readContributions(target = "") {
 }
 
 function readTopology(id = "", owner = "") {
-  const placements = descriptor?.metadata?.ui?.placements?.topology?.placements;
+  const placements = packageMetadata?.metadata?.ui?.placements?.topology?.placements;
   const normalizedId = String(id || "").trim();
   const normalizedOwner = String(owner || "").trim();
   return Array.isArray(placements)
@@ -46,12 +48,12 @@ function readPackageImportSpecifiers(source = "") {
 }
 
 function readClientContainerTokens() {
-  const tokens = descriptor?.metadata?.apiSummary?.containerTokens?.client;
+  const tokens = packageMetadata?.metadata?.apiSummary?.containerTokens?.client;
   return Array.isArray(tokens) ? tokens : [];
 }
 
 function findFileMutation(id) {
-  const files = descriptor?.mutations?.files;
+  const files = packageMetadata?.mutations?.files;
   return Array.isArray(files)
     ? files.find((entry) => String(entry?.id || "").trim() === id) || null
     : null;
@@ -351,7 +353,7 @@ test("shell-web placement topology seeds global actions as a semantic shell plac
   assert.match(source, /outlet: "shell-layout:supporting-side-panel"/);
 });
 
-test("shell-web descriptor pre-optimizes package subpaths reached only through dynamic base-shell modules", async () => {
+test("shell-web packageMetadata pre-optimizes package subpaths reached only through dynamic base-shell modules", async () => {
   const [providerSource, placementSource, placementTopologySource, errorSource] = await Promise.all([
     readFile(path.join(PACKAGE_DIR, "src", "client", "providers", "ShellWebClientProvider.js"), "utf8"),
     readFile(path.join(PACKAGE_DIR, "templates", "src", "placement.js"), "utf8"),
@@ -378,16 +380,16 @@ test("shell-web descriptor pre-optimizes package subpaths reached only through d
     "@jskit-ai/shell-web/client/error"
   ]);
 
-  assert.deepEqual(descriptor?.metadata?.client?.optimizeDeps?.include, [
+  assert.deepEqual(packageMetadata?.metadata?.client?.optimizeDeps?.include, [
     "@jskit-ai/shell-web/client/placement",
     "@jskit-ai/shell-web/client/error"
   ]);
-  assert.deepEqual(descriptor?.metadata?.client?.optimizeDeps?.exclude, [
+  assert.deepEqual(packageMetadata?.metadata?.client?.optimizeDeps?.exclude, [
     "@jskit-ai/shell-web/client"
   ]);
 });
 
-test("shell-web descriptor metadata advertises adaptive shell outlets, default links, and installs the scaffold page", () => {
+test("shell-web packageMetadata metadata advertises adaptive shell outlets, default links, and installs the scaffold page", () => {
   const homePageMutationIds = [
     "shell-web-page-home-wrapper",
     "shell-web-page-home",