@jskit-ai/users-core 0.1.30 → 0.1.32

package/test/workspaceMembersService.test.js

@@ -52,7 +52,7 @@ function createFixture() {
         return [
           {
             userId: 11,
-            roleId: "member",
+            roleSid: "member",
             status: "active",
             displayName: "Alice",
             email: "alice@example.com"
@@ -65,7 +65,7 @@ function createFixture() {
         return {
           workspaceId: 7,
           userId: 11,
-          roleId: "member",
+          roleSid: "member",
           status: "active"
         };
       },
@@ -73,7 +73,7 @@ function createFixture() {
         assert.equal(Number(workspaceId), 7);
         assert.equal(Number(userId), 11);
         assert.deepEqual(patch, {
-          roleId: "admin",
+          roleSid: "admin",
           status: "active"
         });
       }
@@ -134,7 +134,7 @@ test("workspaceMembersService.createInvite uses configured inviteExpiresInMs", a
     { id: 11 },
     {
       email: "alice@example.com",
-      roleId: "member"
+      roleSid: "member"
     },
     authorizedOptions(["workspace.members.invite"])
   );
@@ -260,13 +260,13 @@ test("workspaceMembersService.updateMemberRole returns the refreshed member list
     workspace,
     {
       memberUserId: 11,
-      roleId: "admin"
+      roleSid: "admin"
     },
     authorizedOptions(["workspace.members.manage"])
   );
 
   assert.equal(response.members.length, 1);
-  assert.equal(response.members[0].roleId, "member");
+  assert.equal(response.members[0].roleSid, "member");
 });
 
 test("workspaceMembersService.removeMember marks membership revoked and returns refreshed members", async () => {
@@ -287,7 +287,7 @@ test("workspaceMembersService.removeMember marks membership revoked and returns
           : [
               {
                 userId: 11,
-                roleId: "member",
+                roleSid: "member",
                 status: "active",
                 displayName: "Alice",
                 email: "alice@example.com"
@@ -300,7 +300,7 @@ test("workspaceMembersService.removeMember marks membership revoked and returns
         return {
           workspaceId: 7,
           userId: 11,
-          roleId: "member",
+          roleSid: "member",
           status: "active"
         };
       },
@@ -308,7 +308,7 @@ test("workspaceMembersService.removeMember marks membership revoked and returns
         assert.equal(Number(workspaceId), 7);
         assert.equal(Number(userId), 11);
         assert.deepEqual(patch, {
-          roleId: "member",
+          roleSid: "member",
           status: "revoked"
         });
         removed = true;
@@ -359,7 +359,7 @@ test("workspaceMembersService.removeMember rejects removing the owner", async ()
         return {
           workspaceId: 7,
           userId: 9,
-          roleId: "owner",
+          roleSid: "owner",
           status: "active"
         };
       },

package/test/workspacePendingInvitationsResource.test.js

@@ -14,7 +14,7 @@ test("workspacePendingInvitationsResource output normalizer shapes raw invite ro
         workspaceSlug: "tonymobily3",
         workspaceName: "",
         workspaceAvatarUrl: "",
-        roleId: "Member",
+        roleSid: "Member",
         status: "Pending",
         expiresAt: "2030-01-01T00:00:00.000Z",
         tokenHash
@@ -29,7 +29,7 @@ test("workspacePendingInvitationsResource output normalizer shapes raw invite ro
       workspaceSlug: "tonymobily3",
       workspaceName: "tonymobily3",
       workspaceAvatarUrl: "",
-      roleId: "member",
+      roleSid: "member",
       status: "pending",
       expiresAt: "2030-01-01T00:00:00.000Z",
       token: encodeInviteTokenHash(tokenHash)

package/test/workspacePendingInvitationsService.test.js

@@ -64,7 +64,7 @@ test("listPendingInvitesForUser returns raw pending invite rows for the action l
         workspaceSlug: "tonymobily3",
         workspaceName: "TonyMobily3",
         workspaceAvatarUrl: "",
-        roleId: "member",
+        roleSid: "member",
         status: "pending",
         expiresAt: "2030-01-01T00:00:00.000Z",
         tokenHash
@@ -91,7 +91,7 @@ test("acceptInviteByToken accepts opaque invite token and resolves invite by dec
         id: 44,
         workspaceId: 1,
         email: "chiaramobily@gmail.com",
-        roleId: "member",
+        roleSid: "member",
         status: "pending",
         tokenHash,
         expiresAt: "2030-01-01T00:00:00.000Z"
@@ -125,7 +125,7 @@ test("refuseInviteByToken revokes the invite and returns refused", async () => {
         id: 45,
         workspaceId: 1,
         email: "chiaramobily@gmail.com",
-        roleId: "member",
+        roleSid: "member",
         status: "pending",
         tokenHash,
         expiresAt: "2030-01-01T00:00:00.000Z"

package/test/workspaceService.test.js

@@ -95,7 +95,7 @@ function createWorkspaceServiceFixture({
             slug: "tonymobily3",
             name: "TonyMobily3",
             avatarUrl: "",
-            roleId: "owner",
+            roleSid: "owner",
             membershipStatus: "active"
           },
           {
@@ -103,7 +103,7 @@ function createWorkspaceServiceFixture({
             slug: "pending-workspace",
             name: "Pending Workspace",
             avatarUrl: "",
-            roleId: "member",
+            roleSid: "member",
             membershipStatus: "pending"
           }
         ];
@@ -158,7 +158,7 @@ function createWorkspaceServiceFixture({
         return {
           workspaceId,
           userId,
-          roleId: "owner",
+          roleSid: "owner",
           status: "active"
         };
       }
@@ -190,7 +190,7 @@ test("workspaceService.listWorkspacesForUser returns only accessible workspaces"
 
   assert.equal(workspaces.length, 1);
   assert.equal(workspaces[0].slug, "tonymobily3");
-  assert.equal(workspaces[0].roleId, "owner");
+  assert.equal(workspaces[0].roleSid, "owner");
   assert.equal(calls.listForUserId, 1);
   assert.equal(calls.insert, 0);
 });
@@ -220,7 +220,7 @@ test("workspaceService.listWorkspacesForUser returns all active memberships in p
         slug: "chiaramobily",
         name: "Chiara Personal",
         avatarUrl: "",
-        roleId: "owner",
+        roleSid: "owner",
         membershipStatus: "active"
       },
       {
@@ -228,7 +228,7 @@ test("workspaceService.listWorkspacesForUser returns all active memberships in p
         slug: "tonymobily",
         name: "Tony Workspace",
         avatarUrl: "",
-        roleId: "member",
+        roleSid: "member",
         membershipStatus: "active"
       },
       {
@@ -236,7 +236,7 @@ test("workspaceService.listWorkspacesForUser returns all active memberships in p
         slug: "pending-workspace",
         name: "Pending Workspace",
         avatarUrl: "",
-        roleId: "member",
+        roleSid: "member",
         membershipStatus: "pending"
       }
     ]
@@ -393,7 +393,7 @@ test("workspaceService.resolveWorkspaceContextForUserBySlug allows personal tena
   );
 
   assert.equal(context.workspace.slug, "team-alpha");
-  assert.equal(context.membership.roleId, "owner");
+  assert.equal(context.membership.roleSid, "owner");
   assert.deepEqual(context.permissions, ["*"]);
 });
 
@@ -439,7 +439,7 @@ test("workspaceService.resolveWorkspaceContextForUserBySlug grants owner access
         membershipRecord = {
           workspaceId,
           userId,
-          roleId: "owner",
+          roleSid: "owner",
           status: "active"
         };
         return membershipRecord;
@@ -464,7 +464,7 @@ test("workspaceService.resolveWorkspaceContextForUserBySlug grants owner access
   );
 
   assert.equal(ensuredMembershipCount, 1);
-  assert.equal(context.membership.roleId, "owner");
+  assert.equal(context.membership.roleSid, "owner");
   assert.deepEqual(context.permissions, ["*"]);
 });
 

package/src/server/accountProfile/registerAvatarMultipartSupport.js

@@ -1,40 +0,0 @@
-import fastifyMultipart from "@fastify/multipart";
-
-async function registerAvatarMultipartSupport(app) {
-  if (!app || typeof app.has !== "function" || typeof app.make !== "function") {
-    throw new Error("registerAvatarMultipartSupport requires application has()/make().");
-  }
-
-  if (!app.has("jskit.fastify")) {
-    return;
-  }
-
-  const fastify = app.make("jskit.fastify");
-  if (!fastify || typeof fastify.register !== "function") {
-    throw new Error("registerAvatarMultipartSupport requires Fastify register().");
-  }
-
-  if (fastify["jskit.users-core.avatar.multipart.support"] === true) {
-    return;
-  }
-
-  if (typeof fastify.hasContentTypeParser === "function" && fastify.hasContentTypeParser("multipart")) {
-    Object.defineProperty(fastify, "jskit.users-core.avatar.multipart.support", {
-      value: true,
-      configurable: false,
-      enumerable: false,
-      writable: false
-    });
-    return;
-  }
-
-  await fastify.register(fastifyMultipart);
-  Object.defineProperty(fastify, "jskit.users-core.avatar.multipart.support", {
-    value: true,
-    configurable: false,
-    enumerable: false,
-    writable: false
-  });
-}
-
-export { registerAvatarMultipartSupport };

package/test/registerAvatarMultipartSupport.test.js

@@ -1,63 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-import { registerAvatarMultipartSupport } from "../src/server/accountProfile/registerAvatarMultipartSupport.js";
-
-function createAppStub({ hasFastify = true, fastify = null } = {}) {
-  const resolvedFastify =
-    fastify ||
-    {
-      register: async () => {},
-      hasContentTypeParser: () => false
-    };
-
-  return {
-    has(token) {
-      if (token === "jskit.fastify") {
-        return hasFastify;
-      }
-      return false;
-    },
-    make(token) {
-      if (token === "jskit.fastify") {
-        return resolvedFastify;
-      }
-      return null;
-    }
-  };
-}
-
-test("registerAvatarMultipartSupport returns early when Fastify is not available", async () => {
-  const app = createAppStub({ hasFastify: false });
-  await assert.doesNotReject(async () => registerAvatarMultipartSupport(app));
-});
-
-test("registerAvatarMultipartSupport registers multipart parser only once", async () => {
-  let registerCount = 0;
-  const fastify = {
-    register: async () => {
-      registerCount += 1;
-    },
-    hasContentTypeParser: () => false
-  };
-  const app = createAppStub({ fastify });
-
-  await registerAvatarMultipartSupport(app);
-  await registerAvatarMultipartSupport(app);
-
-  assert.equal(registerCount, 1);
-});
-
-test("registerAvatarMultipartSupport skips registration when parser already exists", async () => {
-  let registerCount = 0;
-  const fastify = {
-    register: async () => {
-      registerCount += 1;
-    },
-    hasContentTypeParser: (contentType) => String(contentType || "").trim().toLowerCase() === "multipart"
-  };
-  const app = createAppStub({ fastify });
-
-  await registerAvatarMultipartSupport(app);
-
-  assert.equal(registerCount, 0);
-});