@makolabs/ripple 1.6.2 → 1.6.4

package/dist/funcs/user-management.remote.js

@@ -275,19 +275,23 @@ async function fetchUserPermissions(userId) {
         if (userData?.data?.data && Array.isArray(userData.data.data)) {
             // Deduplicate permissions by using a Set
             const allPermissions = userData.data.data.flatMap((key) => key.scopes || []);
-            return Array.from(new Set(allPermissions));
+            const dedupedPermissions = Array.from(new Set(allPermissions));
+            return dedupedPermissions;
         }
         else if (userData?.scopes) {
-            return Array.isArray(userData.scopes) ? userData.scopes : [userData.scopes];
+            const permissions = Array.isArray(userData.scopes) ? userData.scopes : [userData.scopes];
+            return permissions;
         }
         return [];
     }
-    catch {
+    catch (error) {
+        console.error('[fetchUserPermissions] Error fetching user permissions:', error);
         try {
             const allKeysData = await makeAdminRequest('/admin/keys');
             const userKey = allKeysData.data.data.find((key) => key.sub === userId && key.client_id === CLIENT_ID && key.status === 'active');
             if (userKey) {
-                return Array.isArray(userKey.scopes) ? userKey.scopes : [userKey.scopes];
+                const permissions = Array.isArray(userKey.scopes) ? userKey.scopes : [userKey.scopes];
+                return permissions;
             }
             return [];
         }
@@ -345,20 +349,20 @@ async function refreshTokenIfSelfUpdate(userId) {
 export const updateUserPermissions = command('unchecked', async (options) => {
     const { userId, permissions } = options;
     try {
-        // Fetch all keys to find user's active keys
-        const allKeysData = await makeAdminRequest('/admin/keys');
+        // Fetch all keys for this specific user (not paginated, gets all keys)
+        const allKeysData = await makeAdminRequest(`/admin/keys?client_id=${CLIENT_ID}&sub=${userId}`);
         // Find ALL active keys for this user (not just one)
-        const userKeys = allKeysData.data.data.filter((key) => key.sub === userId && key.client_id === CLIENT_ID && key.status === 'active');
-        // Delete all old active keys to ensure clean state
-        if (userKeys.length > 0) {
+        const userKeys = (allKeysData?.data?.data || []).filter((key) => key.status === 'active');
+        // Create new key first, then delete old ones (safer order)
+        const newKey = await createUserPermissions(userId, permissions);
+        // Only delete old keys after new key is successfully created
+        if (userKeys.length > 0 && newKey) {
             await Promise.all(userKeys.map((key) => makeAdminRequest(`/admin/keys/${key.id}`, {
                 method: 'DELETE'
             }).catch((err) => {
                 console.warn(`[updateUserPermissions] Failed to delete key ${key.id}:`, err);
             })));
         }
-        // Create one new key with updated permissions
-        await createUserPermissions(userId, permissions);
         await refreshTokenIfSelfUpdate(userId);
     }
     catch (error) {

package/dist/user-management/UserModal.svelte

@@ -164,10 +164,11 @@
 				role: formData.role,
 				permissions: formData.permissions
 			};
+
 			await onSave(userData, mode);
 			handleClose();
 		} catch (error) {
-			console.error('Error saving user:', error);
+			console.error('[UserModal] Error saving user:', error);
 			formErrors.submit = error instanceof Error ? error.message : 'Failed to save user';
 		} finally {
 			saving = false;
@@ -176,7 +177,8 @@
 
 	function handleRoleChange(roleValue: string) {
 		const role = roles?.find((r: Role) => r.value === roleValue);
-		// Reassign entire formData object to ensure reactivity
+
+		// Reassign entire formData object to ensure Svelte 5 reactivity
 		formData = {
 			...formData,
 			role: roleValue,

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@makolabs/ripple",
-	"version": "1.6.2",
+	"version": "1.6.4",
 	"description": "Simple Svelte 5 powered component library ✨",
 	"license": "SEE LICENSE IN LICENSE",
 	"repository": {