@jskit-ai/users-web 0.1.72 → 0.1.74

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

package/test/useCrudAddEdit.test.js

@@ -51,6 +51,22 @@ test("createCrudFormModel preserves explicit boolean defaults for nullable field
   });
 });
 
+test("createCrudFormModel initializes nullable lookup fields as null", () => {
+  const model = createCrudFormModel([
+    {
+      key: "serviceId",
+      type: "string",
+      nullable: true,
+      component: "lookup",
+      relation: { kind: "lookup", namespace: "services" }
+    }
+  ]);
+
+  assert.deepEqual(model, {
+    serviceId: null
+  });
+});
+
 test("buildCrudFormPayload normalizes booleans and numbers while skipping empty numeric values", () => {
   const payload = buildCrudFormPayload(
     [
@@ -114,6 +130,13 @@ test("buildCrudFormPayload serializes cleared nullable typed fields as null", ()
     [
       { key: "reviewed", type: "boolean", nullable: true },
       { key: "serviceId", type: "integer", nullable: true },
+      {
+        key: "selectedServiceId",
+        type: "string",
+        nullable: true,
+        component: "lookup",
+        relation: { kind: "lookup", namespace: "services" }
+      },
       { key: "fromDate", type: "string", format: "date", nullable: true },
       { key: "scheduledAt", type: "string", format: "date-time", nullable: true },
       { key: "fromTime", type: "string", format: "time", nullable: true }
@@ -121,6 +144,7 @@ test("buildCrudFormPayload serializes cleared nullable typed fields as null", ()
     {
       reviewed: null,
       serviceId: null,
+      selectedServiceId: "",
       fromDate: "",
       scheduledAt: "",
       fromTime: ""
@@ -130,6 +154,7 @@ test("buildCrudFormPayload serializes cleared nullable typed fields as null", ()
   assert.deepEqual(payload, {
     reviewed: null,
     serviceId: null,
+    selectedServiceId: null,
     fromDate: null,
     scheduledAt: null,
     fromTime: null
@@ -247,6 +272,31 @@ test("applyCrudPayloadToForm preserves nullable boolean payload values", () => {
   });
 });
 
+test("applyCrudPayloadToForm preserves null lookup payload values", () => {
+  const fields = [
+    {
+      key: "serviceId",
+      type: "string",
+      nullable: true,
+      component: "lookup",
+      relation: { kind: "lookup", namespace: "services" }
+    }
+  ];
+  const form = reactive({
+    serviceId: "existing"
+  });
+
+  applyCrudPayloadToForm(fields, form, {
+    serviceId: null
+  });
+  assert.equal(form.serviceId, null);
+
+  applyCrudPayloadToForm(fields, form, {
+    serviceId: 42
+  });
+  assert.equal(form.serviceId, "42");
+});
+
 test("resolveCrudRouteBoundFieldValues maps route params for route-bound form fields", () => {
   const values = resolveCrudRouteBoundFieldValues(
     [

package/test/useCrudListParentTitle.test.js

@@ -2,6 +2,7 @@ import assert from "node:assert/strict";
 import test from "node:test";
 import { createSchema } from "json-rest-schema";
 import { resolveCrudListParentDescriptor, resolveCrudListParentRecordTitle, resolveCrudListParentTitleFromItems } from "../src/client/composables/internal/crudListParentTitleSupport.js";
+import { useCrudListParentTitle } from "../src/client/composables/useCrudListParentTitle.js";
 
 const contactChildResource = Object.freeze({
   contract: {
@@ -106,6 +107,32 @@ test("resolveCrudListParentTitleFromItems uses the hydrated lookup label", () =>
   assert.equal(title, "Jessica Dickinson");
 });
 
+test("resolveCrudListParentTitleFromItems ignores raw lookup ids when no hydrated label is present", () => {
+  const descriptor = resolveCrudListParentDescriptor({
+    resource: contactChildResource,
+    route: {
+      matched: [{ path: "/w/:workspaceSlug/admin/contacts/:contactId/availabilities" }],
+      params: {
+        workspaceSlug: "dogandgroom",
+        contactId: "538779"
+      }
+    },
+    recordIdParam: "availabilityRuleId"
+  });
+
+  const title = resolveCrudListParentTitleFromItems(
+    [
+      {
+        id: 1,
+        contactId: 538779
+      }
+    ],
+    descriptor
+  );
+
+  assert.equal(title, "");
+});
+
 test("resolveCrudListParentRecordTitle falls back to entity label plus id", () => {
   const title = resolveCrudListParentRecordTitle(
     {
@@ -154,3 +181,82 @@ test("resolveCrudListParentDescriptor supports parentRouteParamKey aliases", ()
   assert.equal(descriptor?.fieldKey, "staffContactId");
   assert.equal(descriptor?.routeParamKey, "contactId");
 });
+
+test("useCrudListParentTitle loads the parent record when child rows only expose the raw parent id", () => {
+  const runtime = useCrudListParentTitle({
+    listRuntime: {
+      items: [
+        {
+          id: 1,
+          contactId: 538779
+        }
+      ],
+      isInitialLoading: false,
+      loadError: ""
+    },
+    resource: contactChildResource,
+    recordIdParam: "availabilityRuleId",
+    route: {
+      matched: [{ path: "/w/:workspaceSlug/admin/contacts/:contactId/availabilities" }],
+      params: {
+        workspaceSlug: "dogandgroom",
+        contactId: "538779"
+      }
+    },
+    viewRuntimeFactory: () => ({
+      record: {
+        id: 538779,
+        firstName: "Jessica",
+        lastName: "Dickinson"
+      },
+      isLoading: false,
+      loadError: ""
+    })
+  });
+
+  assert.equal(runtime.shouldLoadParentRecord, true);
+  assert.equal(runtime.title, "Jessica Dickinson");
+});
+
+test("useCrudListParentTitle requests the parent through JSON:API record transport", () => {
+  let capturedTransport = null;
+
+  useCrudListParentTitle({
+    listRuntime: {
+      items: [
+        {
+          id: 1,
+          contactId: 538779
+        }
+      ],
+      isInitialLoading: false,
+      loadError: ""
+    },
+    resource: contactChildResource,
+    recordIdParam: "availabilityRuleId",
+    route: {
+      matched: [{ path: "/w/:workspaceSlug/admin/contacts/:contactId/availabilities" }],
+      params: {
+        workspaceSlug: "dogandgroom",
+        contactId: "538779"
+      }
+    },
+    viewRuntimeFactory: (options = {}) => {
+      capturedTransport = options.transport;
+      return {
+        record: {
+          id: 538779,
+          fullName: "Jessica Dickinson"
+        },
+        isLoading: false,
+        loadError: ""
+      };
+    }
+  });
+
+  assert.deepEqual(capturedTransport, {
+    kind: "jsonapi-resource",
+    responseType: "contacts",
+    responseKind: "record"
+  });
+});