chore: Remove redundant roleId from user serializer

packages/backend/src/serializers/user.js

@@ -9,7 +9,6 @@ const userSerializer = (user) => {
     createdAt: user.createdAt,
     updatedAt: user.updatedAt,
     fullName: user.fullName,
-    roleId: user.roleId,
   };
 
   if (user.role) {

packages/backend/src/serializers/user.test.js

@@ -33,7 +33,6 @@ describe('userSerializer', () => {
       email: user.email,
       fullName: user.fullName,
       id: user.id,
-      roleId: user.roleId,
       updatedAt: user.updatedAt,
     };
 

packages/backend/test/mocks/rest/api/v1/users/get-current-user.js

@@ -15,7 +15,6 @@ const getCurrentUserMock = (currentUser, role) => {
         name: role.name,
         updatedAt: role.updatedAt.toISOString(),
       },
-      roleId: role.id,
       trialExpiryDate: currentUser.trialExpiryDate.toISOString(),
       updatedAt: currentUser.updatedAt.toISOString(),
     },

packages/backend/test/mocks/rest/api/v1/users/get-user.js

@@ -14,7 +14,6 @@ const getUserMock = (currentUser, role) => {
         name: role.name,
         updatedAt: role.updatedAt.toISOString(),
       },
-      roleId: role.id,
       trialExpiryDate: currentUser.trialExpiryDate.toISOString(),
       updatedAt: currentUser.updatedAt.toISOString(),
     },

packages/backend/test/mocks/rest/api/v1/users/get-users.js

@@ -1,6 +1,7 @@
 const getUsersMock = async (users, roles) => {
   const data = users.map((user) => {
     const role = roles.find((r) => r.id === user.roleId);
+
     return {
       createdAt: user.createdAt.toISOString(),
       email: user.email,
@@ -16,8 +17,7 @@ const getUsersMock = async (users, roles) => {
             name: role.name,
             updatedAt: role.updatedAt.toISOString(),
           }
-        : null, // Fallback to null if role not found
-      roleId: user.roleId,
+        : null,
       trialExpiryDate: user.trialExpiryDate.toISOString(),
       updatedAt: user.updatedAt.toISOString(),
     };