@jskit-ai/users-web 0.1.72 → 0.1.73

package/src/client/composables/records/useCrudList.js

@@ -1,5 +1,6 @@
 import { computed, unref } from "vue";
 import { useRoute } from "vue-router";
+import { resolveCrudJsonApiTransport } from "../crud/crudJsonApiTransportSupport.js";
 import { resolveLookupFieldDisplayValue } from "../crud/crudLookupFieldLabelSupport.js";
 import { resolveCrudBoundValues } from "../crud/crudBindingSupport.js";
 import { resolveCrudListParentDescriptor } from "../internal/crudListParentTitleSupport.js";
@@ -65,6 +66,9 @@ function useCrudList({
   });
   const records = useList({
     ...listOptions,
+    transport: resolveCrudJsonApiTransport(listOptions.transport, resource, {
+      mode: "list"
+    }),
     recordIdParam,
     requestQueryParams(context = {}) {
       const baseRequestQueryParams = resolveRequestQueryParamsInput(requestQueryParams, context);

package/src/client/composables/records/useCrudView.js

@@ -1,5 +1,8 @@
 import { computed } from "vue";
 import { useRoute } from "vue-router";
+import {
+  resolveCrudJsonApiTransport
+} from "../crud/crudJsonApiTransportSupport.js";
 import {
   resolveLookupFieldDisplayValue,
   resolveRecordTitle
@@ -9,6 +12,7 @@ import { asPlainObject } from "../support/scopeHelpers.js";
 import { useView } from "./useView.js";
 
 function useCrudView({
+  resource = null,
   paramBinding = null,
   route = null,
   ...viewOptions
@@ -25,6 +29,9 @@ function useCrudView({
   });
   const view = useView({
     ...viewOptions,
+    transport: resolveCrudJsonApiTransport(viewOptions.transport, resource, {
+      mode: "view"
+    }),
     routeParams: boundRouteParams
   });
   view.resolveFieldDisplay = resolveLookupFieldDisplayValue;

package/src/client/composables/runtime/useListCore.js

@@ -4,6 +4,8 @@ import { asPlainObject } from "../support/scopeHelpers.js";
 import { resolveEnabledRef, resolveTextRef } from "../support/refValueHelpers.js";
 import { usePagedCollection } from "../usePagedCollection.js";
 
+const DEFAULT_LIST_LIMIT = 20;
+
 function buildListRequestOptions({
   requestOptions = null,
   transport = null,
@@ -18,6 +20,12 @@ function buildListRequestOptions({
     resolvedOptions.query && typeof resolvedOptions.query === "object" && !Array.isArray(resolvedOptions.query)
       ? { ...resolvedOptions.query }
       : {};
+  if (
+    !Object.hasOwn(sourceQuery, "limit") &&
+    !Object.hasOwn(sourceQuery, "page[limit]")
+  ) {
+    sourceQuery.limit = DEFAULT_LIST_LIMIT;
+  }
   if (pageParam !== null && pageParam !== undefined && pageParam !== "") {
     sourceQuery.cursor = String(pageParam);
   }

package/src/client/composables/useCrudListFilterLookups.js

@@ -15,6 +15,7 @@ import {
   mergeSelectedLookupOptions,
   resolveLookupOptionLabel
 } from "./internal/crudListFilterLookupSupport.js";
+import { inferCrudLookupJsonApiTransport } from "./crud/crudJsonApiTransportSupport.js";
 
 function useCrudListFilterLookups(
   definitions = {},
@@ -46,10 +47,14 @@ function useCrudListFilterLookups(
     if (!filter.lookup?.apiSuffix) {
       continue;
     }
+    const transport = inferCrudLookupJsonApiTransport({
+      apiPath: filter.lookup.apiSuffix
+    });
 
     const runtime = useList({
       adapter: adapter || undefined,
       apiSuffix: filter.lookup.apiSuffix,
+      ...(transport ? { transport } : {}),
       queryKeyFactory: (surfaceId = "", scopeParamValue = "") => [
         ...normalizedQueryKeyPrefix,
         filter.key,

package/src/client/index.js

@@ -1,6 +1,7 @@
 import { UsersWebClientProvider } from "./providers/UsersWebClientProvider.js";
 
 export { UsersWebClientProvider } from "./providers/UsersWebClientProvider.js";
+export { default as AccountSettingsClientElement } from "./components/AccountSettingsClientElement.vue";
 
 const clientProviders = Object.freeze([UsersWebClientProvider]);
 

package/templates/src/components/account/settings/vibe-coding-todo.todo

@@ -0,0 +1,20 @@
+This note was split.
+
+Use these repo-level docs instead:
+
+- `docs/jskit-product-lanes-rulebook.md`
+- `docs/jskit-product-lanes-workboard.md`
+
+Why:
+
+- the previous file had become a giant mixed artifact
+- the durable product direction now lives in the rulebook
+- the prioritized execution plan now lives in the workboard
+- this template-path file is intentionally reduced to a pointer so the catch-all handoff does not keep living inside a shipped template location
+
+If a future agent needs to continue the work:
+
+1. read `AGENTS.md`
+2. read the JSKIT pattern/workflow files named in the rulebook
+3. use the rulebook for architecture and product direction
+4. use the workboard for prioritization and execution planning

package/templates/src/pages/account/index.vue

@@ -13,5 +13,5 @@
 </template>
 
 <script setup>
-import AccountSettingsClientElement from "../../components/account/settings/AccountSettingsClientElement.vue";
+import AccountSettingsClientElement from "@jskit-ai/users-web/client/components/AccountSettingsClientElement";
 </script>

package/test/accountSettingsSections.test.js

@@ -16,12 +16,12 @@ test("resolveAccountSettingsSections normalizes, deduplicates, and sorts placeme
 
   const resolved = resolveAccountSettingsSections([
     {
-      id: "users.notifications.duplicate",
+      id: "users.profile.duplicate",
       order: 999,
       component: SectionA,
       props: {
-        value: "notifications",
-        title: "Ignore duplicate"
+        value: "profile",
+        title: "Profile duplicate"
       }
     },
     {
@@ -37,10 +37,20 @@ test("resolveAccountSettingsSections normalizes, deduplicates, and sorts placeme
       id: "invalid.missing-component",
       order: 250,
       props: {
-        value: "profile",
+        value: "broken",
         title: "Broken missing component"
       }
     },
+    {
+      id: "users.profile",
+      order: 100,
+      component: SectionB,
+      props: {
+        value: "profile",
+        title: "Profile",
+        usesSharedRuntime: true
+      }
+    },
     {
       id: "security.section",
       order: 350,
@@ -70,8 +80,9 @@ test("resolveAccountSettingsSections normalizes, deduplicates, and sorts placeme
       usesSharedRuntime: entry.usesSharedRuntime
     })),
     [
+      { value: "invites", title: "Second duplicate", order: 100, usesSharedRuntime: false },
+      { value: "profile", title: "Profile", order: 100, usesSharedRuntime: true },
       { value: "security", title: "Security", order: 350, usesSharedRuntime: true },
-      { value: "invites", title: "Invites", order: 400, usesSharedRuntime: false }
     ]
   );
 });

package/test/crudJsonApiTransportSupport.test.js

@@ -0,0 +1,166 @@
+import assert from "node:assert/strict";
+import test from "node:test";
+import { createSchema } from "json-rest-schema";
+import {
+  inferCrudJsonApiTransport,
+  inferCrudLookupJsonApiTransport,
+  resolveCrudJsonApiTransport,
+  resolveLookupFieldMap
+} from "../src/client/composables/crud/crudJsonApiTransportSupport.js";
+
+const resource = {
+  namespace: "pets",
+  contract: {
+    lookup: {
+      containerKey: "lookups"
+    }
+  },
+  operations: {
+    view: {
+      output: {
+        schema: createSchema({
+          id: {
+            type: "string",
+            required: true
+          },
+          contactId: {
+            type: "string",
+            required: true,
+            belongsTo: "contacts",
+            as: "contact"
+          },
+          breedId: {
+            type: "string",
+            nullable: true,
+            belongsTo: "breeds",
+            as: "breed"
+          }
+        })
+      }
+    }
+  }
+};
+
+test("inferCrudJsonApiTransport infers collection transport for CRUD lists", () => {
+  assert.deepEqual(
+    inferCrudJsonApiTransport(resource, {
+      mode: "list"
+    }),
+    {
+      kind: "jsonapi-resource",
+      responseType: "pets",
+      responseKind: "collection"
+    }
+  );
+});
+
+test("inferCrudJsonApiTransport infers record request/response transport for CRUD add/edit", () => {
+  assert.deepEqual(
+    inferCrudJsonApiTransport(resource, {
+      mode: "add-edit",
+      operationName: "patch"
+    }),
+    {
+      kind: "jsonapi-resource",
+      requestType: "pets",
+      responseType: "pets",
+      responseKind: "record"
+    }
+  );
+});
+
+test("inferCrudLookupJsonApiTransport infers collection transport from lookup namespace", () => {
+  assert.deepEqual(
+    inferCrudLookupJsonApiTransport({
+      namespace: "services"
+    }),
+    {
+      kind: "jsonapi-resource",
+      responseType: "services",
+      responseKind: "collection"
+    }
+  );
+});
+
+test("inferCrudLookupJsonApiTransport infers collection transport from lookup apiPath", () => {
+  assert.deepEqual(
+    inferCrudLookupJsonApiTransport({
+      apiPath: "/contact-roles"
+    }),
+    {
+      kind: "jsonapi-resource",
+      responseType: "contact-roles",
+      responseKind: "collection"
+    }
+  );
+});
+
+test("inferCrudLookupJsonApiTransport returns null without a lookup namespace", () => {
+  assert.equal(
+    inferCrudLookupJsonApiTransport({}),
+    null
+  );
+});
+
+test("resolveLookupFieldMap derives lookup aliases from the shared CRUD resource", () => {
+  assert.deepEqual(resolveLookupFieldMap(resource), {
+    breed: "breedId",
+    contact: "contactId"
+  });
+});
+
+test("resolveCrudJsonApiTransport infers and enriches JSON:API transport from the resource", () => {
+  assert.deepEqual(
+    resolveCrudJsonApiTransport(undefined, resource, {
+      mode: "list"
+    }),
+    {
+      kind: "jsonapi-resource",
+      responseType: "pets",
+      responseKind: "collection",
+      lookupContainerKey: "lookups",
+      lookupFieldMap: {
+        breed: "breedId",
+        contact: "contactId"
+      }
+    }
+  );
+});
+
+test("resolveCrudJsonApiTransport infers record request/response transport for CRUD add/edit", () => {
+  assert.deepEqual(
+    resolveCrudJsonApiTransport(undefined, resource, {
+      mode: "add-edit",
+      operationName: "patch"
+    }),
+    {
+      kind: "jsonapi-resource",
+      requestType: "pets",
+      responseType: "pets",
+      responseKind: "record",
+      lookupContainerKey: "lookups",
+      lookupFieldMap: {
+        breed: "breedId",
+        contact: "contactId"
+      }
+    }
+  );
+});
+
+test("resolveCrudJsonApiTransport rejects explicit CRUD transport overrides", () => {
+  assert.throws(
+    () =>
+      resolveCrudJsonApiTransport(
+        {
+          kind: "jsonapi-resource",
+          responseType: "pets",
+          responseKind: "record"
+        },
+        resource,
+        {
+          mode: "view"
+        }
+      ),
+    /no longer accept explicit transport/
+  );
+});

package/test/crudLookupFieldRuntime.test.js

@@ -203,6 +203,31 @@ test("resolveLookupFieldDisplayValue supports custom lookup container key", () =
   );
 });
 
+test("resolveLookupFieldDisplayValue falls back to alias-keyed lookups for relationship-backed fields", () => {
+  assert.equal(
+    resolveLookupFieldDisplayValue(
+      {
+        vetId: "17",
+        lookups: {
+          vet: {
+            id: "17",
+            name: "Harbor Vet"
+          }
+        }
+      },
+      {
+        key: "vetId",
+        relation: {
+          kind: "lookup",
+          valueKey: "id",
+          labelKey: "name"
+        }
+      }
+    ),
+    "Harbor Vet"
+  );
+});
+
 test("resolveLookupFieldDisplayValue returns raw value for non-lookup fields", () => {
   assert.equal(
     resolveLookupFieldDisplayValue(

package/test/exportsContract.test.js

@@ -15,6 +15,7 @@ test("users-web exports are explicit and aligned with production/template usage"
     packageId: "@jskit-ai/users-web",
     requiredExports: [
       "./client",
+      "./client/components/AccountSettingsClientElement",
       "./client/account-settings/sections",
       "./client/composables/useAddEdit",
       "./client/composables/useCommand",

package/test/requestTransportOptions.test.js

@@ -95,6 +95,7 @@ test("list request options stay GET and carry transport when provided", () => {
         "x-demo": "1"
       },
       query: {
+        limit: 20,
         cursor: "cursor_2"
       },
       transport: {
@@ -105,3 +106,37 @@ test("list request options stay GET and carry transport when provided", () => {
     }
   );
 });
+
+test("list request options preserve explicit limit values", () => {
+  assert.deepEqual(
+    buildListRequestOptions({
+      requestOptions: {
+        query: {
+          limit: 50
+        }
+      }
+    }),
+    {
+      method: "GET",
+      query: {
+        limit: 50
+      }
+    }
+  );
+
+  assert.deepEqual(
+    buildListRequestOptions({
+      requestOptions: {
+        query: {
+          "page[limit]": "75"
+        }
+      }
+    }),
+    {
+      method: "GET",
+      query: {
+        "page[limit]": "75"
+      }
+    }
+  );
+});

package/test/settingsPlacementContract.test.js

@@ -80,6 +80,28 @@ test("users-web home tools widget exposes home-cog outlet", async () => {
   assert.match(source, /:default-link-component-token="HOME_COG_OUTLET\.defaultLinkComponentToken"/);
 });
 
+test("users-web account page template uses the package-owned account settings host", async () => {
+  const source = await readFile(path.join(PACKAGE_DIR, "templates", "src", "pages", "account", "index.vue"), "utf8");
+
+  assert.match(
+    source,
+    /import AccountSettingsClientElement from "@jskit-ai\/users-web\/client\/components\/AccountSettingsClientElement";/
+  );
+  assert.doesNotMatch(source, /components\/account\/settings\/AccountSettingsClientElement\.vue/);
+});
+
+test("users-web package-owned account settings host is fully placement-backed", async () => {
+  const source = await readFile(
+    path.join(PACKAGE_DIR, "src", "client", "components", "AccountSettingsClientElement.vue"),
+    "utf8"
+  );
+
+  assert.match(source, /useAccountSettingsSections/);
+  assert.doesNotMatch(source, /AccountSettingsProfileSection/);
+  assert.doesNotMatch(source, /AccountSettingsPreferencesSection/);
+  assert.doesNotMatch(source, /AccountSettingsNotificationsSection/);
+});
+
 test("users-web descriptor metadata advertises home cog outlet and standard home settings placements", () => {
   assert.deepEqual(
     readOutlets("home-cog:primary-menu"),
@@ -98,7 +120,7 @@ test("users-web descriptor metadata advertises home cog outlet and standard home
       {
         target: "account-settings:sections",
         surfaces: ["account"],
-        source: "templates/src/components/account/settings/AccountSettingsClientElement.vue"
+        source: "src/client/components/AccountSettingsClientElement.vue"
       }
     ]
   );
@@ -130,6 +152,27 @@ test("users-web descriptor metadata advertises home cog outlet and standard home
     source: "mutations.text#users-web-home-tools-placement"
   });
   assert.equal(findContribution("users.home.settings.general"), null);
+  expectContribution("users.account.settings.profile", {
+    target: "account-settings:sections",
+    surfaces: ["account"],
+    order: 100,
+    componentToken: "local.main.account-settings.section.profile",
+    source: "mutations.text#users-web-account-settings-sections-placement"
+  });
+  expectContribution("users.account.settings.preferences", {
+    target: "account-settings:sections",
+    surfaces: ["account"],
+    order: 200,
+    componentToken: "local.main.account-settings.section.preferences",
+    source: "mutations.text#users-web-account-settings-sections-placement"
+  });
+  expectContribution("users.account.settings.notifications", {
+    target: "account-settings:sections",
+    surfaces: ["account"],
+    order: 300,
+    componentToken: "local.main.account-settings.section.notifications",
+    source: "mutations.text#users-web-account-settings-sections-placement"
+  });
 
   expectTextMutation("users-web-home-tools-placement", {
     reason: "Append users-web home tools widget and settings menu placements into app-owned placement registry.",
@@ -145,6 +188,22 @@ test("users-web descriptor metadata advertises home cog outlet and standard home
       'unscopedSuffix: "/settings"'
     ]
   });
+  expectTextMutation("users-web-account-settings-sections-placement", {
+    reason: "Append users-web account settings section placements into the app-owned placement registry.",
+    category: "users-web",
+    skipIfContains: 'id: "users.account.settings.profile"',
+    snippets: [
+      'id: "users.account.settings.profile"',
+      'componentToken: "local.main.account-settings.section.profile"',
+      'value: "profile"',
+      'id: "users.account.settings.preferences"',
+      'componentToken: "local.main.account-settings.section.preferences"',
+      'value: "preferences"',
+      'id: "users.account.settings.notifications"',
+      'componentToken: "local.main.account-settings.section.notifications"',
+      'value: "notifications"'
+    ]
+  });
 
   expectTextMutation("users-web-profile-settings-placement", {
     reason: "Append users-web profile settings menu placement into app-owned placement registry.",
@@ -159,6 +218,99 @@ test("users-web descriptor metadata advertises home cog outlet and standard home
     ]
   });
 
+  assert.equal(findFileMutation("users-web-component-account-settings-root"), null);
   assert.equal(findFileMutation("users-web-component-account-settings-invites"), null);
+  assert.deepEqual(findFileMutation("users-web-component-account-settings-profile"), {
+    from: "templates/src/components/account/settings/AccountSettingsProfileSection.vue",
+    to: "src/components/account/settings/AccountSettingsProfileSection.vue",
+    reason: "Install app-owned account settings profile section scaffold.",
+    category: "users-web",
+    id: "users-web-component-account-settings-profile"
+  });
+  assert.deepEqual(findFileMutation("users-web-component-account-settings-preferences"), {
+    from: "templates/src/components/account/settings/AccountSettingsPreferencesSection.vue",
+    to: "src/components/account/settings/AccountSettingsPreferencesSection.vue",
+    reason: "Install app-owned account settings preferences section scaffold.",
+    category: "users-web",
+    id: "users-web-component-account-settings-preferences"
+  });
+  assert.deepEqual(findFileMutation("users-web-component-account-settings-notifications"), {
+    from: "templates/src/components/account/settings/AccountSettingsNotificationsSection.vue",
+    to: "src/components/account/settings/AccountSettingsNotificationsSection.vue",
+    reason: "Install app-owned account settings notifications section scaffold.",
+    category: "users-web",
+    id: "users-web-component-account-settings-notifications"
+  });
+  assert.deepEqual(findTextMutation("users-web-main-client-provider-account-settings-profile-import"), {
+    op: "append-text",
+    file: "packages/main/src/client/providers/MainClientProvider.js",
+    position: "top",
+    skipIfContains:
+      "import AccountSettingsProfileSection from \"/src/components/account/settings/AccountSettingsProfileSection.vue\";",
+    value: "import AccountSettingsProfileSection from \"/src/components/account/settings/AccountSettingsProfileSection.vue\";\n",
+    reason: "Bind the app-owned account profile settings section into local main client provider imports.",
+    category: "users-web",
+    id: "users-web-main-client-provider-account-settings-profile-import"
+  });
+  assert.deepEqual(findTextMutation("users-web-main-client-provider-account-settings-preferences-import"), {
+    op: "append-text",
+    file: "packages/main/src/client/providers/MainClientProvider.js",
+    position: "top",
+    skipIfContains:
+      "import AccountSettingsPreferencesSection from \"/src/components/account/settings/AccountSettingsPreferencesSection.vue\";",
+    value:
+      "import AccountSettingsPreferencesSection from \"/src/components/account/settings/AccountSettingsPreferencesSection.vue\";\n",
+    reason: "Bind the app-owned account preferences settings section into local main client provider imports.",
+    category: "users-web",
+    id: "users-web-main-client-provider-account-settings-preferences-import"
+  });
+  assert.deepEqual(findTextMutation("users-web-main-client-provider-account-settings-notifications-import"), {
+    op: "append-text",
+    file: "packages/main/src/client/providers/MainClientProvider.js",
+    position: "top",
+    skipIfContains:
+      "import AccountSettingsNotificationsSection from \"/src/components/account/settings/AccountSettingsNotificationsSection.vue\";",
+    value:
+      "import AccountSettingsNotificationsSection from \"/src/components/account/settings/AccountSettingsNotificationsSection.vue\";\n",
+    reason: "Bind the app-owned account notifications settings section into local main client provider imports.",
+    category: "users-web",
+    id: "users-web-main-client-provider-account-settings-notifications-import"
+  });
+  assert.deepEqual(findTextMutation("users-web-main-client-provider-account-settings-profile-register"), {
+    op: "append-text",
+    file: "packages/main/src/client/providers/MainClientProvider.js",
+    position: "bottom",
+    skipIfContains:
+      "registerMainClientComponent(\"local.main.account-settings.section.profile\", () => AccountSettingsProfileSection);",
+    value:
+      "\nregisterMainClientComponent(\"local.main.account-settings.section.profile\", () => AccountSettingsProfileSection);\n",
+    reason: "Bind the app-owned account profile settings section token into local main client provider registry.",
+    category: "users-web",
+    id: "users-web-main-client-provider-account-settings-profile-register"
+  });
+  assert.deepEqual(findTextMutation("users-web-main-client-provider-account-settings-preferences-register"), {
+    op: "append-text",
+    file: "packages/main/src/client/providers/MainClientProvider.js",
+    position: "bottom",
+    skipIfContains:
+      "registerMainClientComponent(\"local.main.account-settings.section.preferences\", () => AccountSettingsPreferencesSection);",
+    value:
+      "\nregisterMainClientComponent(\"local.main.account-settings.section.preferences\", () => AccountSettingsPreferencesSection);\n",
+    reason: "Bind the app-owned account preferences settings section token into local main client provider registry.",
+    category: "users-web",
+    id: "users-web-main-client-provider-account-settings-preferences-register"
+  });
+  assert.deepEqual(findTextMutation("users-web-main-client-provider-account-settings-notifications-register"), {
+    op: "append-text",
+    file: "packages/main/src/client/providers/MainClientProvider.js",
+    position: "bottom",
+    skipIfContains:
+      "registerMainClientComponent(\"local.main.account-settings.section.notifications\", () => AccountSettingsNotificationsSection);",
+    value:
+      "\nregisterMainClientComponent(\"local.main.account-settings.section.notifications\", () => AccountSettingsNotificationsSection);\n",
+    reason: "Bind the app-owned account notifications settings section token into local main client provider registry.",
+    category: "users-web",
+    id: "users-web-main-client-provider-account-settings-notifications-register"
+  });
 
 });