@jskit-ai/users-web 0.1.72 → 0.1.74

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

@@ -1,6 +1,7 @@
 import { computed, proxyRefs, reactive, watch } from "vue";
 import { useRoute, useRouter } from "vue-router";
 import { asPlainObject } from "../support/scopeHelpers.js";
+import { resolveCrudJsonApiTransport } from "../crud/crudJsonApiTransportSupport.js";
 import { useAddEdit } from "./useAddEdit.js";
 import {
   resolveCrudBoundValues,
@@ -54,6 +55,15 @@ function useCrudAddEdit({
   const route = useRoute();
   const normalizedFields = normalizeCrudFormFields(formFields);
   const normalizedAddEditOptions = asPlainObject(addEditOptions);
+  const resolvedResource = normalizedAddEditOptions.resource || resource;
+  const resolvedTransport = resolveCrudJsonApiTransport(
+    normalizedAddEditOptions.transport,
+    resolvedResource,
+    {
+      mode: "add-edit",
+      operationName
+    }
+  );
   const saveSuccessOptions = normalizeSaveSuccessOptions(saveSuccess);
   const defaultFieldErrorKeys = normalizedFields.map((field) => field.key);
   const providedFieldErrorKeys = normalizeFieldErrorKeys(normalizedAddEditOptions.fieldErrorKeys);
@@ -101,7 +111,6 @@ function useCrudAddEdit({
     ? normalizedAddEditOptions.onSaveSuccess
     : null;
   const shouldApplyDefaultMapPayload = normalizedAddEditOptions.readEnabled !== false;
-  const resolvedResource = normalizedAddEditOptions.resource || resource;
   const resolvedInput = inputOverride || resolvedResource?.operations?.[operationName]?.body || null;
 
   function resolveBuildRawPayload(model = {}, context = {}) {
@@ -170,6 +179,7 @@ function useCrudAddEdit({
   const addEdit = useAddEdit({
     ...normalizedAddEditOptions,
     resource: resolvedResource,
+    transport: resolvedTransport,
     model: form,
     fieldErrorKeys,
     input: resolvedInput,

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/addEditUiRuntime.js

@@ -57,19 +57,20 @@ function createAddEditUiRuntime({
       ...currentRouteParams,
       ...asPlainObject(extraParams)
     };
+    const currentRouteRecordId = toResolvedRecordId({
+      routeParams: currentRouteParams,
+      recordIdParam: normalizedRecordIdParam,
+      routeRecordId
+    });
     const resolvedRecordId = toRouteParamValue(sourceParams[normalizedRecordIdParam]) ||
-      toResolvedRecordId({
-        routeParams: currentRouteParams,
-        recordIdParam: normalizedRecordIdParam,
-        routeRecordId
-      });
+      currentRouteRecordId;
     sourceParams[normalizedRecordIdParam] = resolvedRecordId;
     const currentPathname = resolveScopedRoutePathname({
       currentPathname: routePath,
       params: currentRouteParams,
       orderedParamNames: routeParamNames,
       anchorParamName: normalizedRecordIdParam,
-      anchorParamValue: resolvedRecordId,
+      anchorParamValue: currentRouteRecordId,
       anchorMode: "after"
     });
 

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/composables/useCrudListParentTitle.js

@@ -1,6 +1,7 @@
 import { computed, proxyRefs } from "vue";
 import { useRoute } from "vue-router";
 import { normalizeText } from "@jskit-ai/kernel/shared/support/normalize";
+import { inferCrudLookupJsonApiTransport } from "./crud/crudJsonApiTransportSupport.js";
 import {
   resolveRouteParamsSource,
   toRouteParamValue
@@ -59,6 +60,19 @@ function useCrudListParentTitle({
   });
 
   const initialParentDescriptor = parentDescriptor.value || {};
+  const parentTransport = (() => {
+    const baseTransport = inferCrudLookupJsonApiTransport({
+      namespace: initialParentDescriptor.relationNamespace
+    });
+    if (!baseTransport) {
+      return null;
+    }
+
+    return Object.freeze({
+      ...baseTransport,
+      responseKind: "record"
+    });
+  })();
   const normalizedQueryKeyPrefix = normalizeQueryKeyPrefix(queryKeyPrefix);
   const shouldLoadParentRecord = computed(() => {
     const descriptor = parentDescriptor.value;
@@ -70,12 +84,18 @@ function useCrudListParentTitle({
     }
 
     const items = Array.isArray(listRuntime?.items) ? listRuntime.items : [];
-    return items.length < 1;
+    if (items.length < 1) {
+      return true;
+    }
+
+    const listTitle = resolveCrudListParentTitleFromItems(items, descriptor);
+    return !listTitle;
   });
 
   const parentView = viewRuntimeFactory({
     adapter,
     apiUrlTemplate: normalizeText(initialParentDescriptor.apiUrlTemplate),
+    transport: parentTransport,
     readEnabled: shouldLoadParentRecord,
     recordIdParam: normalizeText(initialParentDescriptor.routeParamKey) || "recordId",
     includeRecordIdInQueryKey: true,

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/addEditUiRuntime.test.js

@@ -35,6 +35,23 @@ test("createAddEditUiRuntime resolves view urls for saved payload ids with neste
   assert.equal(runtime.resolveSavedViewUrl({ id: 99 }), "/contacts/7/addresses/99");
 });
 
+test("createAddEditUiRuntime keeps nested child routes stable when the saved child id matches a parent id", () => {
+  const runtime = createAddEditUiRuntime({
+    recordIdParam: "addressId",
+    routeParams: ref({
+      workspaceSlug: "tonymobily",
+      contactId: "1"
+    }),
+    routeParamNames: ref(["workspaceSlug", "contactId"]),
+    routePath: ref("/w/tonymobily/admin/contacts/1/addresses/new"),
+    viewUrlTemplate: "../:addressId",
+    listUrlTemplate: ".."
+  });
+
+  assert.equal(runtime.listUrl.value, "/w/tonymobily/admin/contacts/1/addresses");
+  assert.equal(runtime.resolveSavedViewUrl({ id: 1 }), "/w/tonymobily/admin/contacts/1/addresses/1");
+});
+
 test("createAddEditUiRuntime resolves edit-page relative list and cancel links", () => {
   const runtime = createAddEditUiRuntime({
     recordIdParam: "addressId",

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"
+      }
+    }
+  );
+});