@jskit-ai/users-web 0.1.166 → 0.1.170

package/src/client/composables/support/resourceLoadStateHelpers.js

@@ -77,12 +77,8 @@ function normalizeRequestRecoveryMethod(value = "") {
   return normalizeText(value).toUpperCase();
 }
 
-function isRequestRecoveryMetaDisabled(sourceJskitMeta = {}, sourceMeta = {}) {
-  return Boolean(
-    sourceJskitMeta.requestRecovery === false ||
-      sourceMeta.jskitRequestRecovery === false ||
-      sourceMeta.requestRecovery === false
-  );
+function isRequestRecoveryMetaDisabled(sourceJskitMeta = {}) {
+  return sourceJskitMeta.requestRecovery === false;
 }
 
 function hasUsableMetaValue(source = {}, key = "") {
@@ -161,37 +157,20 @@ function buildRequestRecoveryMeta(requestRecovery = null, defaults = {}) {
   return Object.keys(jskit).length > 0 ? { jskit } : {};
 }
 
-function hasRequestRecoveryMetaValue(sourceJskitMeta = {}, sourceMeta = {}, key = "") {
-  const suffix = String(key || "").trim();
-  if (!suffix) {
-    return false;
-  }
-
-  const jskitKey = `requestRecovery${suffix}`;
-  const legacyJskitKey = `jskitRequestRecovery${suffix}`;
-  const legacyKey = `requestRecovery${suffix}`;
-
-  return Boolean(
-    hasUsableMetaValue(sourceJskitMeta, jskitKey) ||
-      hasUsableMetaValue(sourceMeta, legacyJskitKey) ||
-      hasUsableMetaValue(sourceMeta, legacyKey)
-  );
-}
-
 function resolveRequestRecoveryDefaults(queryOptions = null, defaults = {}) {
   const sourceOptions = normalizePlainObject(queryOptions);
   const sourceMeta = normalizePlainObject(sourceOptions.meta);
   const sourceJskitMeta = normalizePlainObject(sourceMeta.jskit);
   const fallback = normalizePlainObject(defaults);
-  if (isRequestRecoveryMetaDisabled(sourceJskitMeta, sourceMeta)) {
+  if (isRequestRecoveryMetaDisabled(sourceJskitMeta)) {
     return {};
   }
 
-  const hasExplicitLabel = hasRequestRecoveryMetaValue(sourceJskitMeta, sourceMeta, "Label");
-  const hasExplicitSource = hasRequestRecoveryMetaValue(sourceJskitMeta, sourceMeta, "Source");
-  const hasExplicitDedupeKey = hasRequestRecoveryMetaValue(sourceJskitMeta, sourceMeta, "DedupeKey");
-  const hasExplicitDedupeWindowMs = hasRequestRecoveryMetaValue(sourceJskitMeta, sourceMeta, "DedupeWindowMs");
-  const hasExplicitMethod = hasRequestRecoveryMetaValue(sourceJskitMeta, sourceMeta, "Method");
+  const hasExplicitLabel = hasUsableMetaValue(sourceJskitMeta, "requestRecoveryLabel");
+  const hasExplicitSource = hasUsableMetaValue(sourceJskitMeta, "requestRecoverySource");
+  const hasExplicitDedupeKey = hasUsableMetaValue(sourceJskitMeta, "requestRecoveryDedupeKey");
+  const hasExplicitDedupeWindowMs = hasUsableMetaValue(sourceJskitMeta, "requestRecoveryDedupeWindowMs");
+  const hasExplicitMethod = hasUsableMetaValue(sourceJskitMeta, "requestRecoveryMethod");
 
   return {
     ...fallback,

package/src/client/composables/useScopeRuntime.js

@@ -4,8 +4,8 @@ import {
   ROUTE_VISIBILITY_WORKSPACE
 } from "@jskit-ai/kernel/shared/support/visibility";
 import { useAccess } from "./useAccess.js";
-import { useSurfaceRouteContext } from "./useSurfaceRouteContext.js";
-import { usePaths } from "./usePaths.js";
+import { usePaths } from "@jskit-ai/shell-web/client/navigation/usePaths";
+import { useSurfaceRouteContext } from "@jskit-ai/shell-web/client/navigation/useSurfaceRouteContext";
 import { resolveSurfaceDefinitionFromPlacementContext } from "@jskit-ai/shell-web/client/placement";
 import {
   asPlainObject,

package/src/client/index.js

@@ -3,6 +3,7 @@ import { UsersWebClientProvider } from "./providers/UsersWebClientProvider.js";
 export { UsersWebClientProvider } from "./providers/UsersWebClientProvider.js";
 export { default as AccountSettingsClientElement } from "./components/AccountSettingsClientElement.vue";
 export { default as CrudAddEditScreen } from "./components/CrudAddEditScreen.vue";
+export { default as CrudDeleteAction } from "./components/CrudDeleteAction.vue";
 export { default as CrudListBulkActionSurface } from "./components/CrudListBulkActionSurface.vue";
 export { default as CrudListFilterSurface } from "./components/CrudListFilterSurface.vue";
 export { default as CrudListScreen } from "./components/CrudListScreen.vue";

package/test/crudJsonApiTransportSupport.test.js

@@ -174,6 +174,6 @@ test("resolveCrudJsonApiTransport rejects explicit CRUD transport overrides", ()
           mode: "view"
         }
       ),
-    /no longer accept explicit transport/
+    /do not accept explicit transport/
   );
 });

package/test/crudListDateFilterSurface.browser.test.js

@@ -1,99 +1,22 @@
 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 } from "@playwright/test";
+import {
+  createChromiumLaunchOptions,
+  startViteFixture,
+  stopProcess
+} from "../../../tooling/testUtils/browserFixture.mjs";
 
 const TEST_DIRECTORY = path.dirname(fileURLToPath(import.meta.url));
-const REPOSITORY_ROOT = path.resolve(TEST_DIRECTORY, "../../..");
 const PACKAGE_ROOT = path.resolve(TEST_DIRECTORY, "..");
 const FIXTURE_ROOT = path.join(PACKAGE_ROOT, "fixtures", "crud-list-date-filters");
-const VITE_CLI = path.join(REPOSITORY_ROOT, "node_modules", "vite", "bin", "vite.js");
 const RUN_BROWSER_TEST = process.env.JSKIT_USERS_WEB_DATE_FILTER_VISUAL_INTEGRATION === "1";
 const VIEWPORT_WIDTHS = Object.freeze([375, 768, 1280, 1440]);
 const QUERY = "?status=active&submittedOn=2026-04-18&arrivalDate=2026-05-04..2026-05-10";
 
-function startCapturedProcess(command, args, { cwd } = {}) {
-  const child = spawn(command, args, {
-    cwd,
-    stdio: ["ignore", "pipe", "pipe"],
-    env: {
-      ...process.env,
-      NO_COLOR: "1",
-      FORCE_COLOR: "0"
-    }
-  });
-  let output = "";
-
-  child.stdout.on("data", (chunk) => {
-    output += chunk.toString("utf8");
-  });
-  child.stderr.on("data", (chunk) => {
-    output += chunk.toString("utf8");
-  });
-
-  function waitFor(pattern, timeout = 30_000) {
-    return new Promise((resolve, reject) => {
-      const startedAt = Date.now();
-      const timer = setInterval(() => {
-        if (pattern.test(output)) {
-          clearInterval(timer);
-          resolve();
-          return;
-        }
-        if (child.exitCode !== null) {
-          clearInterval(timer);
-          reject(new Error(`Vite exited before ${pattern}.\n${output}`));
-          return;
-        }
-        if (Date.now() - startedAt >= timeout) {
-          clearInterval(timer);
-          reject(new Error(`Timed out waiting for ${pattern}.\n${output}`));
-        }
-      }, 50);
-    });
-  }
-
-  return {
-    child,
-    readOutput: () => output,
-    waitFor
-  };
-}
-
-async function stopProcess(runtime) {
-  const child = runtime?.child;
-  if (!child || child.exitCode !== null || child.signalCode !== null) {
-    return;
-  }
-
-  await new Promise((resolve) => {
-    const forceTimer = setTimeout(() => child.kill("SIGKILL"), 5_000);
-    child.once("exit", () => {
-      clearTimeout(forceTimer);
-      resolve();
-    });
-    child.kill("SIGTERM");
-  });
-}
-
-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;
-}
-
 async function readContract(page) {
   return JSON.parse(await page.getByTestId("date-filter-contract").textContent());
 }
@@ -108,7 +31,7 @@ async function openCompactFilters(page) {
 
 async function assertResponsiveDateControls(page, width) {
   await page.setViewportSize({ width, height: 900 });
-  await page.goto(`http://127.0.0.1:${page.__jskitPort}/${QUERY}`, {
+  await page.goto(`/${QUERY}`, {
     waitUntil: "networkidle"
   });
   await openCompactFilters(page);
@@ -158,37 +81,23 @@ test("CRUD date filters render and operate without clipping across responsive la
     : "set JSKIT_USERS_WEB_DATE_FILTER_VISUAL_INTEGRATION=1 to run the browser visual regression",
   timeout: 180_000
 }, async () => {
-  const port = await reservePort();
-  const viteRuntime = startCapturedProcess(process.execPath, [
-    VITE_CLI,
-    "--config",
-    path.join(FIXTURE_ROOT, "vite.config.mjs"),
-    "--port",
-    String(port),
-    "--clearScreen",
-    "false"
-  ], { cwd: FIXTURE_ROOT });
+  const viteRuntime = await startViteFixture({ fixtureRoot: FIXTURE_ROOT });
   let browser = null;
 
   try {
-    await viteRuntime.waitFor(new RegExp(`http://127\\.0\\.0\\.1:${port}/`, "u"));
-    const chromiumExecutablePath = String(
-      process.env.PLAYWRIGHT_CHROMIUM_EXECUTABLE_PATH || ""
-    ).trim();
-    browser = await chromium.launch({
-      headless: true,
-      ...(chromiumExecutablePath ? { executablePath: chromiumExecutablePath } : {})
+    browser = await chromium.launch(createChromiumLaunchOptions());
+    const context = await browser.newContext({
+      baseURL: viteRuntime.baseURL,
+      locale: "en-US"
     });
-    const context = await browser.newContext({ locale: "en-US" });
     const page = await context.newPage();
-    page.__jskitPort = port;
 
     for (const width of VIEWPORT_WIDTHS) {
       await assertResponsiveDateControls(page, width);
     }
 
     await page.setViewportSize({ width: 1280, height: 900 });
-    await page.goto(`http://127.0.0.1:${port}/${QUERY}`, { waitUntil: "networkidle" });
+    await page.goto(`/${QUERY}`, { waitUntil: "networkidle" });
     assert.deepEqual(await readContract(page), {
       values: {
         submittedOn: "2026-04-18",

package/test/crudScreenComponents.test.js

@@ -19,6 +19,7 @@ test("CRUD screen components own generated list/view/form chrome centrally", asy
   const listSource = await readComponent("CrudListScreen.vue");
   const viewSource = await readComponent("CrudViewScreen.vue");
   const addEditSource = await readComponent("CrudAddEditScreen.vue");
+  const deleteActionSource = await readComponent("CrudDeleteAction.vue");
 
   assert.match(listSource, /CrudListBulkActionSurface/);
   assert.match(listSource, /CrudListFilterSurface/);
@@ -36,7 +37,10 @@ test("CRUD screen components own generated list/view/form chrome centrally", asy
   );
   assert.match(listSource, /button-label="More"/);
   assert.match(listSource, /selectableRows/);
-  assert.match(listSource, /min-height:\s*48px/);
+  assert.match(
+    listSource,
+    /\.ui-generator-list-element :deep\(\.v-btn\)\s*\{\s*min-height:\s*48px;/
+  );
   assert.match(listSource, /<slot[\s\S]*name="card-fields"/);
   assert.match(listSource, /<slot name="table-header"/);
   assert.match(listSource, /<slot[\s\S]*name="table-row"/);
@@ -55,6 +59,11 @@ test("CRUD screen components own generated list/view/form chrome centrally", asy
   assert.match(addEditSource, /addEdit\.canRetryLoad/);
   assert.match(addEditSource, /@click="addEdit\.refresh"/);
   assert.match(addEditSource, /<slot[\s\S]*name="fields"/);
+
+  assert.match(deleteActionSource, /@click="action\.request"/);
+  assert.match(deleteActionSource, /@click="action\.confirm"/);
+  assert.match(deleteActionSource, /@click="closeDialog"/);
+  assert.doesNotMatch(deleteActionSource, /\bactivator=/);
 });
 
 test("CRUD screen composables expose generated page extension inputs", async () => {

package/test/exportsContract.test.js

@@ -17,6 +17,7 @@ test("users-web exports are explicit and aligned with production/template usage"
       "./client",
       "./client/components/AccountSettingsClientElement",
       "./client/components/CrudAddEditScreen",
+      "./client/components/CrudDeleteAction",
       "./client/components/CrudListBulkActionSurface",
       "./client/components/CrudListFilterSurface",
       "./client/components/CrudListScreen",
@@ -29,7 +30,6 @@ test("users-web exports are explicit and aligned with production/template usage"
       "./client/composables/useCommand",
       "./client/composables/useEndpointResource",
       "./client/composables/useList",
-      "./client/composables/usePaths",
       "./client/composables/useView",
       "./client/composables/useCrudAddEdit",
       "./client/composables/useCrudAddEditScreen",

package/test/{packageDescriptorOwnership.test.js → packageMetadataOwnership.test.js}

@@ -1,6 +1,8 @@
 import assert from "node:assert/strict";
 import test from "node:test";
-import descriptor from "../package.descriptor.mjs";
+import packageJson from "../package.json" with { type: "json" };
+
+const packageMetadata = packageJson.jskit;
 
 test("users-web leaves account surface scaffolds app-owned", () => {
   const expectedIds = [
@@ -11,7 +13,7 @@ test("users-web leaves account surface scaffolds app-owned", () => {
   ];
 
   for (const id of expectedIds) {
-    const mutation = descriptor.mutations.files.find((entry) => entry.id === id);
+    const mutation = packageMetadata.mutations.files.find((entry) => entry.id === id);
     assert.ok(mutation, `Missing users-web scaffold mutation ${id}.`);
     assert.equal(mutation.ownership, "app", `${id} must remain app-owned across package updates.`);
   }

package/test/settingsPlacementContract.test.js

@@ -4,13 +4,15 @@ 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;
 
 const TEST_DIRECTORY = path.dirname(fileURLToPath(import.meta.url));
 const PACKAGE_DIR = path.resolve(TEST_DIRECTORY, "..");
 
 function readOutlets(host = "") {
-  const outlets = descriptor?.metadata?.ui?.placements?.outlets;
+  const outlets = packageMetadata?.metadata?.ui?.placements?.outlets;
   const normalizedTarget = String(host || "").trim();
   return Array.isArray(outlets)
     ? outlets.filter((entry) => String(entry?.target || "").trim() === normalizedTarget)
@@ -18,7 +20,7 @@ function readOutlets(host = "") {
 }
 
 function findTopology(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)
@@ -31,28 +33,28 @@ function findTopology(id, owner = "") {
 }
 
 function findContribution(id) {
-  const contributions = descriptor?.metadata?.ui?.placements?.contributions;
+  const contributions = packageMetadata?.metadata?.ui?.placements?.contributions;
   return Array.isArray(contributions)
     ? contributions.find((entry) => String(entry?.id || "").trim() === id) || null
     : null;
 }
 
 function findTextMutation(id) {
-  const textMutations = descriptor?.mutations?.text;
+  const textMutations = packageMetadata?.mutations?.text;
   return Array.isArray(textMutations)
     ? textMutations.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;
 }
 
 function findFileMutation(id) {
-  const fileMutations = descriptor?.mutations?.files;
+  const fileMutations = packageMetadata?.mutations?.files;
   return Array.isArray(fileMutations)
     ? fileMutations.find((entry) => String(entry?.id || "").trim() === id) || null
     : null;
@@ -197,7 +199,7 @@ test("users-web account settings section templates use direct settings panels",
   }
 });
 
-test("users-web descriptor metadata advertises home cog outlet and standard home settings placements", () => {
+test("users-web packageMetadata metadata advertises home cog outlet and standard home settings placements", () => {
   assert.deepEqual(
     readOutlets("home-cog:primary-menu"),
     [