@jskit-ai/users-web 0.1.46 → 0.1.48

package/package.descriptor.mjs

@@ -1,7 +1,7 @@
 export default Object.freeze({
   packageVersion: 1,
   packageId: "@jskit-ai/users-web",
-  version: "0.1.46",
+  version: "0.1.48",
   kind: "runtime",
   description: "Users web module: account/profile UI plus shared shell link components.",
   dependsOn: [
@@ -153,12 +153,12 @@ export default Object.freeze({
       runtime: {
         "@tanstack/vue-query": "5.92.12",
         "@mdi/js": "^7.4.47",
-        "@jskit-ai/http-runtime": "0.1.30",
-        "@jskit-ai/realtime": "0.1.30",
-        "@jskit-ai/kernel": "0.1.31",
-        "@jskit-ai/shell-web": "0.1.30",
-        "@jskit-ai/uploads-image-web": "0.1.9",
-        "@jskit-ai/users-core": "0.1.41",
+        "@jskit-ai/http-runtime": "0.1.32",
+        "@jskit-ai/realtime": "0.1.32",
+        "@jskit-ai/kernel": "0.1.33",
+        "@jskit-ai/shell-web": "0.1.32",
+        "@jskit-ai/uploads-image-web": "0.1.11",
+        "@jskit-ai/users-core": "0.1.43",
         vuetify: "^4.0.0"
       },
       dev: {}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jskit-ai/users-web",
-  "version": "0.1.46",
+  "version": "0.1.48",
   "type": "module",
   "scripts": {
     "test": "node --test"
@@ -28,12 +28,12 @@
   "dependencies": {
     "@tanstack/vue-query": "5.92.12",
     "@mdi/js": "^7.4.47",
-    "@jskit-ai/http-runtime": "0.1.30",
-    "@jskit-ai/kernel": "0.1.31",
-    "@jskit-ai/realtime": "0.1.30",
-    "@jskit-ai/shell-web": "0.1.30",
-    "@jskit-ai/uploads-image-web": "0.1.9",
-    "@jskit-ai/users-core": "0.1.41",
+    "@jskit-ai/http-runtime": "0.1.32",
+    "@jskit-ai/kernel": "0.1.33",
+    "@jskit-ai/realtime": "0.1.32",
+    "@jskit-ai/shell-web": "0.1.32",
+    "@jskit-ai/uploads-image-web": "0.1.11",
+    "@jskit-ai/users-core": "0.1.43",
     "vuetify": "^4.0.0"
   }
 }

package/src/client/components/MembersAdminClientElement.vue

@@ -168,6 +168,7 @@
 <script setup>
 import { computed, toRefs, unref } from "vue";
 import { formatDateTime as formatKernelDateTime } from "@jskit-ai/kernel/shared/support";
+import { normalizeRecordId } from "@jskit-ai/kernel/shared/support/normalize";
 import { requireBoolean, requireFunction, requireRecord } from "../support/contractGuards.js";
 
 const props = defineProps({
@@ -188,11 +189,11 @@ const props = defineProps({
     required: true
   },
   revokeInviteId: {
-    type: Number,
+    type: String,
     required: true
   },
   removeMemberUserId: {
-    type: Number,
+    type: String,
     required: true
   },
   status: {
@@ -352,11 +353,11 @@ function isMemberRemoveLocked(member) {
 }
 
 function isRevokeInviteLoading(inviteId) {
-  return isRevokingInvite.value && revokeInviteId.value === Number(inviteId || 0);
+  return isRevokingInvite.value && revokeInviteId.value === normalizeRecordId(inviteId, { fallback: "" });
 }
 
 function isRemoveMemberLoading(memberUserId) {
-  return isRemovingMember.value && removeMemberUserId.value === Number(memberUserId || 0);
+  return isRemovingMember.value && removeMemberUserId.value === normalizeRecordId(memberUserId, { fallback: "" });
 }
 
 async function onSubmitInvite() {

package/src/client/components/WorkspaceMembersClientElement.vue

@@ -21,6 +21,7 @@
 <script setup>
 import { computed, reactive, ref, watch } from "vue";
 import { formatDateTime } from "@jskit-ai/kernel/shared/support";
+import { normalizeRecordId } from "@jskit-ai/kernel/shared/support/normalize";
 import MembersAdminClientElement from "./MembersAdminClientElement.vue";
 import { useCommand } from "../composables/useCommand.js";
 import { useList } from "../composables/records/useList.js";
@@ -61,8 +62,8 @@ const collections = reactive({
 const inviteFeedback = useUiFeedback();
 const membersFeedback = useUiFeedback();
 const teamFeedback = useUiFeedback();
-const revokeInviteId = ref(0);
-const removeMemberUserId = ref(0);
+const revokeInviteId = ref("");
+const removeMemberUserId = ref("");
 
 const { route, currentSurfaceId, workspaceSlugFromRoute, mergePlacementContext } =
   useWorkspaceRouteContext();
@@ -83,7 +84,8 @@ const workspaceInvitesApiPath = computed(() =>
 );
 
 function workspaceMembersPath(memberId) {
-  return `${workspaceMembersApiPath.value}/${Number(memberId || 0)}`;
+  const normalizedMemberId = encodeURIComponent(String(memberId || "").trim());
+  return `${workspaceMembersApiPath.value}/${normalizedMemberId}`;
 }
 
 function workspaceInvitePath(inviteId) {
@@ -145,8 +147,8 @@ function resetViewState() {
   collections.members = [];
   collections.invites = [];
   clearRoleOptions();
-  revokeInviteId.value = 0;
-  removeMemberUserId.value = 0;
+  revokeInviteId.value = "";
+  removeMemberUserId.value = "";
 }
 
 function toRoleTitle(roleSid) {
@@ -223,7 +225,7 @@ function normalizeMembers(entries) {
   return source.map((entry) => {
     const value = entry && typeof entry === "object" ? entry : {};
     return {
-      userId: Number(value.userId || 0),
+      userId: normalizeRecordId(value.userId, { fallback: "" }),
       roleSid: String(value.roleSid || "").trim().toLowerCase(),
       status: String(value.status || "").trim().toLowerCase(),
       displayName: String(value.displayName || "").trim(),
@@ -238,12 +240,12 @@ function normalizeInvites(entries) {
   return source.map((entry) => {
     const value = entry && typeof entry === "object" ? entry : {};
     return {
-      id: Number(value.id || 0),
+      id: normalizeRecordId(value.id, { fallback: "" }),
       email: String(value.email || "").trim().toLowerCase(),
       roleSid: String(value.roleSid || "").trim().toLowerCase(),
       status: String(value.status || "").trim().toLowerCase(),
       expiresAt: value.expiresAt || "",
-      invitedByUserId: value.invitedByUserId == null ? null : Number(value.invitedByUserId)
+      invitedByUserId: normalizeRecordId(value.invitedByUserId, { fallback: null })
     };
   });
 }
@@ -576,7 +578,7 @@ async function submitRevokeInvite(inviteId) {
     return;
   }
 
-  revokeInviteId.value = Number(inviteId || 0);
+  revokeInviteId.value = normalizeRecordId(inviteId, { fallback: "" });
   teamFeedback.clear();
 
   try {
@@ -591,7 +593,7 @@ async function submitRevokeInvite(inviteId) {
   } catch (error) {
     teamFeedback.error(error, "Unable to revoke invite.");
   } finally {
-    revokeInviteId.value = 0;
+    revokeInviteId.value = "";
   }
 }
 
@@ -603,8 +605,8 @@ async function submitMemberRoleUpdate(member, roleSid) {
   membersFeedback.clear();
 
   try {
-    const memberUserId = Number(member?.userId || 0);
-    if (!Number.isInteger(memberUserId) || memberUserId < 1) {
+    const memberUserId = normalizeRecordId(member?.userId, { fallback: null });
+    if (!memberUserId) {
       throw new Error("Member user id is invalid.");
     }
 
@@ -630,12 +632,12 @@ async function submitRemoveMember(member) {
   membersFeedback.clear();
 
   try {
-    const memberUserId = Number(member?.userId || 0);
-    if (!Number.isInteger(memberUserId) || memberUserId < 1) {
+    const memberUserId = normalizeRecordId(member?.userId, { fallback: null });
+    if (!memberUserId) {
       throw new Error("Member user id is invalid.");
     }
 
-    removeMemberUserId.value = memberUserId;
+    removeMemberUserId.value = normalizeRecordId(memberUserId, { fallback: "" });
     await memberRemoveCommand.run({
       memberUserId
     });
@@ -647,7 +649,7 @@ async function submitRemoveMember(member) {
   } catch (error) {
     membersFeedback.error(error, "Unable to remove member.");
   } finally {
-    removeMemberUserId.value = 0;
+    removeMemberUserId.value = "";
   }
 }
 </script>

package/src/client/composables/account-settings/accountSettingsRuntimeHelpers.js

@@ -3,6 +3,7 @@ import {
   normalizeReturnToPath as normalizeSharedReturnToPath,
   resolveAllowedOriginsFromPlacementContext
 } from "@jskit-ai/kernel/shared/support";
+import { normalizeRecordId } from "@jskit-ai/kernel/shared/support/normalize";
 import { normalizeRecord } from "../../support/runtimeNormalization.js";
 
 function normalizeReturnToPath(value, { fallback = "/", accountSettingsPath = "/account", allowedOrigins = [] } = {}) {
@@ -27,9 +28,9 @@ function normalizePendingInvite(entry) {
     return null;
   }
 
-  const id = Number(entry.id);
-  const workspaceId = Number(entry.workspaceId);
-  if (!Number.isInteger(id) || id < 1 || !Number.isInteger(workspaceId) || workspaceId < 1) {
+  const id = normalizeRecordId(entry.id, { fallback: null });
+  const workspaceId = normalizeRecordId(entry.workspaceId, { fallback: null });
+  if (!id || !workspaceId) {
     return null;
   }
 

package/src/client/lib/bootstrap.js

@@ -1,3 +1,5 @@
+import { normalizeRecordId } from "@jskit-ai/kernel/shared/support/normalize";
+
 function buildBootstrapApiPath(workspaceSlug = "") {
   const normalizedWorkspaceSlug = String(workspaceSlug || "").trim();
   if (!normalizedWorkspaceSlug) {
@@ -15,9 +17,9 @@ function normalizeWorkspaceEntry(entry) {
     return null;
   }
 
-  const id = Number(entry.id);
+  const id = normalizeRecordId(entry.id, { fallback: null });
   const slug = String(entry.slug || "").trim();
-  if (!Number.isInteger(id) || id < 1 || !slug) {
+  if (!id || !slug) {
     return null;
   }
 
@@ -69,8 +71,8 @@ function resolvePlacementUserFromBootstrapPayload(payload = {}, currentUser = nu
   const fallbackUser = currentUser && typeof currentUser === "object" ? currentUser : {};
   const nextUser = {};
 
-  const userId = Number(session.userId || fallbackUser.id || 0);
-  if (Number.isInteger(userId) && userId > 0) {
+  const userId = normalizeRecordId(session.userId || fallbackUser.id, { fallback: null });
+  if (userId) {
     nextUser.id = userId;
   }
 

package/test/bootstrap.test.js

@@ -17,7 +17,7 @@ test("resolvePlacementUserFromBootstrapPayload maps profile fields used by place
   const user = resolvePlacementUserFromBootstrapPayload({
     session: {
       authenticated: true,
-      userId: 42
+      userId: "42"
     },
     profile: {
       displayName: "Ada Lovelace",
@@ -29,7 +29,7 @@ test("resolvePlacementUserFromBootstrapPayload maps profile fields used by place
   });
 
   assert.deepEqual(user, {
-    id: 42,
+    id: "42",
     displayName: "Ada Lovelace",
     name: "Ada Lovelace",
     email: "ada@example.com",

package/test/bootstrapPlacementRuntime.test.js

@@ -245,7 +245,7 @@ test("bootstrap placement runtime writes user/workspace/permissions into placeme
       return {
         session: {
           authenticated: true,
-          userId: 7
+          userId: "7"
         },
         profile: {
           displayName: "Ada Lovelace",
@@ -260,10 +260,10 @@ test("bootstrap placement runtime writes user/workspace/permissions into placeme
           }
         },
         pendingInvites: [
-          { id: 1, workspaceId: 1, token: "a" },
-          { id: 2, workspaceId: 2, token: "b" }
+          { id: "1", workspaceId: "1", token: "a" },
+          { id: "2", workspaceId: "2", token: "b" }
         ],
-        workspaces: [{ id: 1, slug: "acme", name: "Acme Workspace" }],
+        workspaces: [{ id: "1", slug: "acme", name: "Acme Workspace" }],
         permissions: ["workspace.settings.view"]
       };
     }
@@ -280,7 +280,7 @@ test("bootstrap placement runtime writes user/workspace/permissions into placeme
   assert.equal(runtime.getWorkspaceBootstrapStatus("acme"), WORKSPACE_BOOTSTRAP_STATUS_RESOLVED);
   assert.equal(context.workspaceBootstrapStatuses?.acme, WORKSPACE_BOOTSTRAP_STATUS_RESOLVED);
   assert.deepEqual(context.user, {
-    id: 7,
+    id: "7",
     displayName: "Ada Lovelace",
     name: "Ada Lovelace",
     email: "ada@example.com",
@@ -305,7 +305,7 @@ test("bootstrap placement runtime resolves workspace slug from pathname when sur
       return {
         session: {
           authenticated: true,
-          userId: 1
+          userId: "1"
         },
         profile: {
           displayName: "User",
@@ -314,7 +314,7 @@ test("bootstrap placement runtime resolves workspace slug from pathname when sur
             effectiveUrl: ""
           }
         },
-        workspaces: [{ id: 1, slug: "acme", name: "Acme Workspace" }],
+        workspaces: [{ id: "1", slug: "acme", name: "Acme Workspace" }],
         permissions: ["workspace.settings.view"]
       };
     }
@@ -348,7 +348,7 @@ test("bootstrap placement runtime does not mutate placement auth context", async
       return {
         session: {
           authenticated: true,
-          userId: 9
+          userId: "9"
         },
         profile: {
           displayName: "User",
@@ -357,7 +357,7 @@ test("bootstrap placement runtime does not mutate placement auth context", async
             effectiveUrl: ""
           }
         },
-        workspaces: [{ id: 1, slug: "acme", name: "Workspace" }],
+        workspaces: [{ id: "1", slug: "acme", name: "Workspace" }],
         permissions: []
       };
     }
@@ -387,7 +387,7 @@ test("bootstrap placement runtime refetches on route changes and users.bootstrap
       return {
         session: {
           authenticated: true,
-          userId: 1
+          userId: "1"
         },
         profile: {
           displayName: "User",
@@ -396,7 +396,7 @@ test("bootstrap placement runtime refetches on route changes and users.bootstrap
             effectiveUrl: ""
           }
         },
-        workspaces: [{ id: 1, slug: workspaceSlug || "acme", name: "Workspace" }],
+        workspaces: [{ id: "1", slug: workspaceSlug || "acme", name: "Workspace" }],
         permissions: []
       };
     }
@@ -436,7 +436,7 @@ test("bootstrap placement runtime refetches when auth context changes", async ()
       return {
         session: {
           authenticated: true,
-          userId: 1
+          userId: "1"
         },
         profile: {
           displayName: "User",
@@ -445,7 +445,7 @@ test("bootstrap placement runtime refetches when auth context changes", async ()
             effectiveUrl: ""
           }
         },
-        workspaces: [{ id: 1, slug: workspaceSlug || "acme", name: "Workspace" }],
+        workspaces: [{ id: "1", slug: workspaceSlug || "acme", name: "Workspace" }],
         permissions: []
       };
     }
@@ -535,7 +535,7 @@ test("bootstrap placement runtime reapplies theme when bootstrap payload changes
       return {
         session: {
           authenticated: true,
-          userId: 1
+          userId: "1"
         },
         profile: {
           displayName: "User",
@@ -547,7 +547,7 @@ test("bootstrap placement runtime reapplies theme when bootstrap payload changes
         userSettings: {
           theme: fetchCount === 1 ? "dark" : "light"
         },
-        workspaces: [{ id: 1, slug: workspaceSlug || "acme", name: "Workspace" }],
+        workspaces: [{ id: "1", slug: workspaceSlug || "acme", name: "Workspace" }],
         permissions: []
       };
     }
@@ -575,7 +575,7 @@ test("bootstrap placement runtime applies workspace palette via Vuetify workspac
       return {
         session: {
           authenticated: true,
-          userId: 1
+          userId: "1"
         },
         workspaceSettings: {
           lightPrimaryColor: "#CC3344",
@@ -589,7 +589,7 @@ test("bootstrap placement runtime applies workspace palette via Vuetify workspac
         },
         workspaces: [
           {
-            id: 1,
+            id: "1",
             slug: "acme",
             name: "Acme Workspace"
           }
@@ -630,8 +630,8 @@ test("bootstrap placement runtime marks workspace slug as not_found and clears w
   const placementRuntime = createPlacementRuntimeStub();
   placementRuntime.setContext(
     {
-      workspace: { id: 1, slug: "acme", name: "Acme Workspace" },
-      workspaces: [{ id: 1, slug: "acme", name: "Acme Workspace" }],
+      workspace: { id: "1", slug: "acme", name: "Acme Workspace" },
+      workspaces: [{ id: "1", slug: "acme", name: "Acme Workspace" }],
       permissions: ["workspace.settings.view"]
     },
     { source: "test.seed" }
@@ -680,7 +680,7 @@ test("bootstrap placement runtime updates status per workspace slug across route
       return {
         session: {
           authenticated: true,
-          userId: 1
+          userId: "1"
         },
         profile: {
           displayName: "User",
@@ -689,7 +689,7 @@ test("bootstrap placement runtime updates status per workspace slug across route
             effectiveUrl: ""
           }
         },
-        workspaces: [{ id: 1, slug: workspaceSlug || "acme", name: "Workspace" }],
+        workspaces: [{ id: "1", slug: workspaceSlug || "acme", name: "Workspace" }],
         permissions: []
       };
     }
@@ -721,7 +721,7 @@ test("bootstrap placement runtime uses requestedWorkspace status and keeps globa
       return {
         session: {
           authenticated: true,
-          userId: 4
+          userId: "4"
         },
         profile: {
           displayName: "Chiara",
@@ -730,7 +730,7 @@ test("bootstrap placement runtime uses requestedWorkspace status and keeps globa
             effectiveUrl: ""
           }
         },
-        workspaces: [{ id: 3, slug: "chiaramobily", name: "Chiara Workspace" }],
+        workspaces: [{ id: "3", slug: "chiaramobily", name: "Chiara Workspace" }],
         requestedWorkspace: {
           slug: "tonymobily",
           status: "forbidden"
@@ -763,7 +763,7 @@ test("bootstrap placement runtime uses requestedWorkspace=not_found without forc
       return {
         session: {
           authenticated: true,
-          userId: 1
+          userId: "1"
         },
         profile: {
           displayName: "User",
@@ -772,7 +772,7 @@ test("bootstrap placement runtime uses requestedWorkspace=not_found without forc
             effectiveUrl: ""
           }
         },
-        workspaces: [{ id: 1, slug: "acme", name: "Acme Workspace" }],
+        workspaces: [{ id: "1", slug: "acme", name: "Acme Workspace" }],
         requestedWorkspace: {
           slug: "missing",
           status: "not_found"
@@ -811,9 +811,9 @@ test("bootstrap placement runtime guard wrapper preserves delegated deny outcome
       return {
         session: {
           authenticated: true,
-          userId: 1
+          userId: "1"
         },
-        workspaces: [{ id: 1, slug: "acme", name: "Acme" }],
+        workspaces: [{ id: "1", slug: "acme", name: "Acme" }],
         permissions: []
       };
     }
@@ -854,7 +854,7 @@ test("bootstrap placement runtime guard wrapper blocks forbidden workspace route
       return {
         session: {
           authenticated: true,
-          userId: 1
+          userId: "1"
         },
         workspaces: [],
         permissions: []
@@ -1032,7 +1032,7 @@ test("bootstrap placement runtime enforces surface access policies after bootstr
       return {
         session: {
           authenticated: true,
-          userId: 1
+          userId: "1"
         },
         workspaces: [],
         permissions: [],
@@ -1059,9 +1059,9 @@ test("bootstrap placement runtime captures guard evaluator assignments after ini
       return {
         session: {
           authenticated: true,
-          userId: 1
+          userId: "1"
         },
-        workspaces: [{ id: 1, slug: "acme", name: "Acme" }],
+        workspaces: [{ id: "1", slug: "acme", name: "Acme" }],
         permissions: []
       };
     }