@jmruthers/pace-core 0.6.10 → 0.6.11

package/src/rbac/hooks/{__tests__/usePermissions.integration.test.ts → usePermissions.integration.test.ts}

@@ -14,17 +14,17 @@
 
 import { renderHook, waitFor } from '@testing-library/react';
 import { vi, describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { usePermissions } from '../usePermissions';
+import { usePermissions } from './usePermissions';
 
 // Mock the RBAC API
-vi.mock('../../api', () => ({
+vi.mock('../api', () => ({
   getPermissionMap: vi.fn(),
   getAccessLevel: vi.fn(),
   isPermitted: vi.fn(),
   isPermittedCached: vi.fn()
 }));
 
-import { getPermissionMap } from '../../api';
+import { getPermissionMap } from '../api';
 
 const mockUserId = 'user-123';
 const mockOrgId = 'org-123';
@@ -43,7 +43,7 @@ describe('usePermissions Integration Tests', () => {
 
   beforeEach(() => {
     vi.clearAllMocks();
-    mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+    mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
   });
 
   afterEach(() => {
@@ -141,7 +141,7 @@ describe('usePermissions Integration Tests', () => {
 
       // Update mock to simulate external permission change
       const updatedPermissions = { ...mockPermissionMap, 'read:users': false };
-      mockGetPermissionMap.mockResolvedValue(updatedPermissions);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: updatedPermissions });
 
       // Call refetch
       result.current.refetch();
@@ -161,7 +161,7 @@ describe('usePermissions Integration Tests', () => {
       });
 
       // Second refetch succeeds
-      mockGetPermissionMap.mockResolvedValueOnce(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValueOnce({ ok: true, data: mockPermissionMap });
       result.current.refetch();
 
       await waitFor(() => {
@@ -309,7 +309,7 @@ describe('usePermissions Integration Tests', () => {
         largePermissionMap[`permission:${i}`] = i % 2 === 0;
       }
 
-      mockGetPermissionMap.mockResolvedValue(largePermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: largePermissionMap });
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
       await waitFor(() => {
@@ -326,7 +326,7 @@ describe('usePermissions Integration Tests', () => {
     it('recovers from error on refetch', async () => {
       mockGetPermissionMap
         .mockRejectedValueOnce(new Error('Network error'))
-        .mockResolvedValueOnce(mockPermissionMap);
+        .mockResolvedValueOnce({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -345,7 +345,7 @@ describe('usePermissions Integration Tests', () => {
     it('returns to normal state after error recovery', async () => {
       mockGetPermissionMap
         .mockRejectedValueOnce(new Error('Initial error'))
-        .mockResolvedValueOnce(mockPermissionMap);
+        .mockResolvedValueOnce({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 

package/src/{__tests__/hooks/usePermissions.test.ts → rbac/hooks/usePermissions.stability.test.ts}

@@ -12,16 +12,16 @@
 
 import { renderHook, act } from '@testing-library/react';
 import { vi } from 'vitest';
-import { usePermissions, useCan } from '../../rbac/hooks/usePermissions';
+import { usePermissions, useCan } from './usePermissions';
 
 // Mock the API functions
-vi.mock('../../rbac/api', () => ({
+vi.mock('../api', () => ({
   getPermissionMap: vi.fn(),
   isPermitted: vi.fn(),
   isPermittedCached: vi.fn(),
 }));
 
-import { getPermissionMap, isPermitted, isPermittedCached } from '../../rbac/api';
+import { getPermissionMap, isPermitted, isPermittedCached } from '../api';
 
 const mockGetPermissionMap = vi.mocked(getPermissionMap);
 const mockIsPermitted = vi.mocked(isPermitted);
@@ -49,9 +49,9 @@ describe('usePermissions Hook Stability', () => {
         'create:users': false
       };
 
-      mockGetPermissionMap.mockResolvedValue(mockPermissions);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissions });
 
-      const { result, rerender } = renderHook(() => 
+      const { result, rerender } = renderHook(() =>
         usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId)
       );
 
@@ -89,10 +89,10 @@ describe('usePermissions Hook Stability', () => {
       };
 
       mockGetPermissionMap
-        .mockResolvedValueOnce(initialPermissions)
-        .mockResolvedValueOnce(updatedPermissions);
+        .mockResolvedValueOnce({ ok: true, data: initialPermissions })
+        .mockResolvedValueOnce({ ok: true, data: updatedPermissions });
 
-      const { result, rerender } = renderHook(() => 
+      const { result, rerender } = renderHook(() =>
         usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId)
       );
 
@@ -118,7 +118,7 @@ describe('usePermissions Hook Stability', () => {
 
     it('should not cause infinite re-renders', async () => {
       let renderCount = 0;
-      
+
       mockGetPermissionMap.mockResolvedValue({
         'read:users': true
       });
@@ -140,9 +140,9 @@ describe('usePermissions Hook Stability', () => {
 
   describe('useCan', () => {
     it('should return stable references when data hasn\'t changed', async () => {
-      mockIsPermittedCached.mockResolvedValue(true);
+      mockIsPermittedCached.mockResolvedValue({ ok: true, data: true });
 
-      const { result, rerender } = renderHook(() => 
+      const { result, rerender } = renderHook(() =>
         useCan(mockUserId, mockScope, 'read:users', 'page-123', true, false)
       );
 
@@ -167,10 +167,10 @@ describe('usePermissions Hook Stability', () => {
 
     it('should update references when permission result changes', async () => {
       mockIsPermittedCached
-        .mockResolvedValueOnce(false)
-        .mockResolvedValueOnce(true);
+        .mockResolvedValueOnce({ ok: true, data: false })
+        .mockResolvedValueOnce({ ok: true, data: true });
 
-      const { result, rerender } = renderHook(() => 
+      const { result, rerender } = renderHook(() =>
         useCan(mockUserId, mockScope, 'read:users', 'page-123', true, false)
       );
 
@@ -195,8 +195,8 @@ describe('usePermissions Hook Stability', () => {
 
     it('should not cause infinite re-renders', async () => {
       let renderCount = 0;
-      
-      mockIsPermittedCached.mockResolvedValue(true);
+
+      mockIsPermittedCached.mockResolvedValue({ ok: true, data: true });
 
       const { result } = renderHook(() => {
         renderCount++;
@@ -215,7 +215,7 @@ describe('usePermissions Hook Stability', () => {
 
   describe('Hook Dependencies', () => {
     it('should re-run when userId changes', async () => {
-      mockGetPermissionMap.mockResolvedValue({ 'read:users': true });
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: { 'read:users': true } });
 
       const { result, rerender } = renderHook(
         ({ userId }) => usePermissions(userId, mockOrgId, mockEventId, mockAppId),
@@ -243,7 +243,7 @@ describe('usePermissions Hook Stability', () => {
     });
 
     it('should re-run when scope changes', async () => {
-      mockGetPermissionMap.mockResolvedValue({ 'read:users': true });
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: { 'read:users': true } });
 
       const { result, rerender } = renderHook(
         ({ orgId }) => usePermissions(mockUserId, orgId, mockEventId, mockAppId),

package/src/rbac/hooks/usePermissions.test.ts

@@ -44,8 +44,8 @@ describe('usePermissions Hook', () => {
 
   beforeEach(() => {
     vi.clearAllMocks();
-    // Pre-setup mock responses to avoid async delays
-    mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+    // Pre-setup mock responses (ApiResult shape)
+    mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
   });
 
     describe('Permission Fetching', () => {
@@ -87,7 +87,7 @@ describe('usePermissions Hook', () => {
     });
 
     it('refetches permissions when dependencies change', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result, rerender } = renderHook(
         ({ userId, scope }) => usePermissions(userId, scope),
@@ -112,7 +112,7 @@ describe('usePermissions Hook', () => {
 
   describe('Permission Checking', () => {
     beforeEach(async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
     });
 
     it('hasPermission returns correct boolean', async () => {
@@ -167,7 +167,7 @@ describe('usePermissions Hook', () => {
 
   describe('Refetch Functionality', () => {
     it('refetch function works correctly', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -185,7 +185,7 @@ describe('usePermissions Hook', () => {
 
     it('refetch handles errors correctly', async () => {
       mockGetPermissionMap
-        .mockResolvedValueOnce(mockPermissionMap)
+        .mockResolvedValueOnce({ ok: true, data: mockPermissionMap })
         .mockRejectedValueOnce(new Error('Refetch error'));
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
@@ -215,7 +215,7 @@ describe('usePermissions Hook', () => {
     });
 
     it('shows loading state during refetch', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -248,7 +248,7 @@ describe('usePermissions Hook', () => {
     it('clears error on successful refetch', async () => {
       mockGetPermissionMap
         .mockRejectedValueOnce(new Error('Initial error'))
-        .mockResolvedValueOnce(mockPermissionMap);
+        .mockResolvedValueOnce({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -277,7 +277,7 @@ describe('usePermissions Hook', () => {
         'super_admin': true
       };
 
-      mockGetPermissionMap.mockResolvedValue(superAdminPermissions);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: superAdminPermissions });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -306,7 +306,7 @@ describe('usePermissions Hook', () => {
         'org_admin': true
       };
 
-      mockGetPermissionMap.mockResolvedValue(orgAdminPermissions);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: orgAdminPermissions });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -335,7 +335,7 @@ describe('usePermissions Hook', () => {
         'event_admin': true
       };
 
-      mockGetPermissionMap.mockResolvedValue(eventAdminPermissions);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: eventAdminPermissions });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -365,7 +365,7 @@ describe('usePermissions Hook', () => {
         'member': true
       };
 
-      mockGetPermissionMap.mockResolvedValue(memberPermissions);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: memberPermissions });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -395,7 +395,7 @@ describe('usePermissions Hook', () => {
         'participant': true
       };
 
-      mockGetPermissionMap.mockResolvedValue(participantPermissions);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: participantPermissions });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -424,7 +424,7 @@ describe('usePermissions Hook', () => {
         'update:events': true
       };
 
-      mockGetPermissionMap.mockResolvedValue(mixedPermissions);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mixedPermissions });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockScope));
 
@@ -450,7 +450,7 @@ describe('usePermissions Hook', () => {
         'update:events': true
       };
 
-      mockGetPermissionMap.mockResolvedValue(mixedPermissions);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mixedPermissions });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockScope));
 
@@ -489,7 +489,7 @@ describe('usePermissions Hook', () => {
     });
 
     it('handles empty scope', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, '', undefined, undefined));
 
@@ -508,7 +508,7 @@ describe('usePermissions Hook', () => {
         largePermissionMap[`permission:${i}`] = i % 2 === 0;
       }
 
-      mockGetPermissionMap.mockResolvedValue(largePermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: largePermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -519,7 +519,7 @@ describe('usePermissions Hook', () => {
     });
 
     it('handles empty permission map', async () => {
-      mockGetPermissionMap.mockResolvedValue({});
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: {} });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -560,7 +560,7 @@ describe('usePermissions Hook', () => {
 
   describe('Performance', () => {
     it('maintains stable references for same permissions', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result, rerender } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -580,7 +580,7 @@ describe('usePermissions Hook', () => {
     });
 
     it('handles rapid permission checks efficiently', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -613,7 +613,7 @@ describe('usePermissions Hook', () => {
 
       // Invalidate and refetch
       const newPermissions = { ...mockPermissionMap, 'new:permission': true };
-      mockGetPermissionMap.mockResolvedValue(newPermissions);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: newPermissions });
       result.current.refetch();
 
       await waitFor(() => {
@@ -622,7 +622,7 @@ describe('usePermissions Hook', () => {
     });
 
     it('handles cache expiration gracefully', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockScope));
 
@@ -640,7 +640,7 @@ describe('usePermissions Hook', () => {
     });
 
     it('maintains consistent permissions across multiple refetches', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockScope));
 
@@ -659,7 +659,7 @@ describe('usePermissions Hook', () => {
 
   describe('[integration] Concurrent Operations', () => {
     it('handles concurrent permission checks', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockScope));
 
@@ -680,7 +680,7 @@ describe('usePermissions Hook', () => {
     });
 
     it('handles rapid scope changes without errors', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result, rerender } = renderHook(
         ({ userId, scope }) => usePermissions(userId, scope),
@@ -711,7 +711,7 @@ describe('usePermissions Hook', () => {
     });
 
     it('handles mixed permission and refetch operations', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockScope));
 
@@ -734,7 +734,7 @@ describe('usePermissions Hook', () => {
 
   describe('[integration] Permission Resolution Edge Cases', () => {
     it('handles organization context changes', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result, rerender } = renderHook(
         ({ userId, scope }) => usePermissions(userId, scope),
@@ -757,7 +757,7 @@ describe('usePermissions Hook', () => {
     });
 
     it('handles event context changes', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result, rerender } = renderHook(
         ({ userId, scope }) => usePermissions(userId, scope),
@@ -781,7 +781,7 @@ describe('usePermissions Hook', () => {
 
     it('handles permission resolution with partial scope', async () => {
       const partialScope = { organisationId: mockScope.organisationId };
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, partialScope));
 
@@ -797,7 +797,7 @@ describe('usePermissions Hook', () => {
     it('recovers from transient errors', async () => {
       mockGetPermissionMap
         .mockRejectedValueOnce(new Error('Transient error'))
-        .mockResolvedValueOnce(mockPermissionMap);
+        .mockResolvedValueOnce({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockScope));
 
@@ -831,7 +831,7 @@ describe('usePermissions Hook', () => {
     it('maintains stability after error recovery', async () => {
       mockGetPermissionMap
         .mockRejectedValueOnce(new Error('Error'))
-        .mockResolvedValue(mockPermissionMap);
+        .mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockScope));
 
@@ -853,7 +853,7 @@ describe('usePermissions Hook', () => {
 
   describe('[integration] State Management', () => {
     it('preserves state across hook re-renders', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result, rerender } = renderHook(() => usePermissions(mockUserId, mockScope));
 
@@ -885,7 +885,7 @@ describe('usePermissions Hook', () => {
     });
 
     it('handles null event ID', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, 'org-123', null, undefined));
 
@@ -911,7 +911,7 @@ describe('usePermissions Hook', () => {
     });
 
     it('handles undefined appId', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, undefined));
 
@@ -923,7 +923,7 @@ describe('usePermissions Hook', () => {
     });
 
     it('handles null appId', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, null as any));
 
@@ -938,7 +938,7 @@ describe('usePermissions Hook', () => {
       let fetchCount = 0;
       mockGetPermissionMap.mockImplementation(() => {
         fetchCount++;
-        return Promise.resolve(mockPermissionMap);
+        return Promise.resolve({ ok: true, data: mockPermissionMap });
       });
 
       const { rerender } = renderHook(
@@ -983,7 +983,7 @@ describe('usePermissions Hook', () => {
         'delete:users': false,
       };
 
-      mockGetPermissionMap.mockResolvedValue(allFalsePermissions);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: allFalsePermissions });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -1005,7 +1005,7 @@ describe('usePermissions Hook', () => {
         'delete:users': true,
       };
 
-      mockGetPermissionMap.mockResolvedValue(mixedPermissions);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mixedPermissions });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -1020,7 +1020,7 @@ describe('usePermissions Hook', () => {
     });
 
     it('handles hasAnyPermission with empty array', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -1033,7 +1033,7 @@ describe('usePermissions Hook', () => {
     });
 
     it('handles hasAllPermissions with empty array', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -1046,7 +1046,7 @@ describe('usePermissions Hook', () => {
     });
 
     it('handles hasAnyPermission with single permission', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -1059,7 +1059,7 @@ describe('usePermissions Hook', () => {
     });
 
     it('handles hasAllPermissions with single permission', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -1103,7 +1103,7 @@ describe('usePermissions Hook', () => {
 
     it('preserves permissions during refetch error', async () => {
       mockGetPermissionMap
-        .mockResolvedValueOnce(mockPermissionMap)
+        .mockResolvedValueOnce({ ok: true, data: mockPermissionMap })
         .mockRejectedValueOnce(new Error('Refetch error'));
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
@@ -1124,7 +1124,7 @@ describe('usePermissions Hook', () => {
     });
 
     it('handles concurrent refetch calls', async () => {
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -1151,7 +1151,7 @@ describe('usePermissions Hook', () => {
         largePermissionMap[`permission:${i}`] = i % 2 === 0;
       }
 
-      mockGetPermissionMap.mockResolvedValue(largePermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: largePermissionMap });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -1167,8 +1167,8 @@ describe('usePermissions Hook', () => {
 
     it('handles permission map transition from empty to populated', async () => {
       mockGetPermissionMap
-        .mockResolvedValueOnce({})
-        .mockResolvedValueOnce(mockPermissionMap);
+        .mockResolvedValueOnce({ ok: true, data: {} })
+        .mockResolvedValueOnce({ ok: true, data: mockPermissionMap });
 
       const { result, rerender } = renderHook(
         ({ userId, orgId }) => usePermissions(userId, orgId, mockEventId, mockAppId),
@@ -1193,8 +1193,8 @@ describe('usePermissions Hook', () => {
 
     it('handles permission map transition from populated to empty', async () => {
       mockGetPermissionMap
-        .mockResolvedValueOnce(mockPermissionMap)
-        .mockResolvedValueOnce({});
+        .mockResolvedValueOnce({ ok: true, data: mockPermissionMap })
+        .mockResolvedValueOnce({ ok: true, data: {} });
 
       const { result, rerender } = renderHook(
         ({ userId, orgId }) => usePermissions(userId, orgId, mockEventId, mockAppId),
@@ -1223,7 +1223,7 @@ describe('usePermissions Hook', () => {
         'read:users': false,
       };
 
-      mockGetPermissionMap.mockResolvedValue(wildcardPermissions);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: wildcardPermissions });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -1241,7 +1241,7 @@ describe('usePermissions Hook', () => {
         '*': true,
       };
 
-      mockGetPermissionMap.mockResolvedValue(wildcardPermissions);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: wildcardPermissions });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -1258,7 +1258,7 @@ describe('usePermissions Hook', () => {
         '*': true,
       };
 
-      mockGetPermissionMap.mockResolvedValue(wildcardPermissions);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: wildcardPermissions });
 
       const { result } = renderHook(() => usePermissions(mockUserId, mockOrgId, mockEventId, mockAppId));
 
@@ -1274,7 +1274,7 @@ describe('usePermissions Hook', () => {
       let fetchCount = 0;
       mockGetPermissionMap.mockImplementation(() => {
         fetchCount++;
-        return Promise.resolve(mockPermissionMap);
+        return Promise.resolve({ ok: true, data: mockPermissionMap });
       });
 
       const { rerender } = renderHook(
@@ -1313,7 +1313,7 @@ describe('usePermissions Hook', () => {
       }, { timeout: 4000 });
 
       // Provide organisation context
-      mockGetPermissionMap.mockResolvedValue(mockPermissionMap);
+      mockGetPermissionMap.mockResolvedValue({ ok: true, data: mockPermissionMap });
       rerender({ userId: mockUserId, orgId: mockOrgId });
 
       await waitFor(() => {