@jmruthers/pace-core 0.6.6 → 0.6.7

package/src/providers/__tests__/UnifiedAuthProvider.test.tsx

@@ -58,15 +58,15 @@ const TestComponent = () => {
   const auth = useUnifiedAuth();
   
   return (
-    <div data-testid="test-component">
-      <div data-testid="hasUser">{auth.user !== null ? 'true' : 'false'}</div>
-      <div data-testid="hasSession">{auth.session !== null ? 'true' : 'false'}</div>
-      <div data-testid="hasSignIn">{typeof auth.signIn === 'function' ? 'true' : 'false'}</div>
-      <div data-testid="hasSignOut">{typeof auth.signOut === 'function' ? 'true' : 'false'}</div>
-      <div data-testid="hasOrganisation">{typeof auth.selectedOrganisation === 'object' && auth.selectedOrganisation !== null ? 'true' : 'false'}</div>
-      <div data-testid="hasEvents">{Array.isArray(auth.events) ? 'true' : 'false'}</div>
-      <div data-testid="hasAppName">{typeof auth.appName === 'string' ? 'true' : 'false'}</div>
-    </div>
+    <section data-testid="test-component">
+      <p data-testid="hasUser">{auth.user !== null ? 'true' : 'false'}</p>
+      <p data-testid="hasSession">{auth.session !== null ? 'true' : 'false'}</p>
+      <p data-testid="hasSignIn">{typeof auth.signIn === 'function' ? 'true' : 'false'}</p>
+      <p data-testid="hasSignOut">{typeof auth.signOut === 'function' ? 'true' : 'false'}</p>
+      <p data-testid="hasOrganisation">{typeof auth.selectedOrganisation === 'object' && auth.selectedOrganisation !== null ? 'true' : 'false'}</p>
+      <p data-testid="hasEvents">{Array.isArray(auth.events) ? 'true' : 'false'}</p>
+      <p data-testid="hasAppName">{typeof auth.appName === 'string' ? 'true' : 'false'}</p>
+    </section>
   );
 };
 
@@ -91,7 +91,7 @@ describe('UnifiedAuthProvider', () => {
         </UnifiedAuthProvider>
       );
 
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
+      expect(screen.getByTestId('test-component')).toBeDefined();
     });
 
     it('renders with custom app name', () => {
@@ -101,7 +101,7 @@ describe('UnifiedAuthProvider', () => {
         </UnifiedAuthProvider>
       );
 
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
+      expect(screen.getByTestId('test-component')).toBeDefined();
     });
 
     it('renders without supabase client', () => {
@@ -111,7 +111,7 @@ describe('UnifiedAuthProvider', () => {
         </UnifiedAuthProvider>
       );
 
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
+      expect(screen.getByTestId('test-component')).toBeDefined();
     });
   });
 
@@ -148,9 +148,9 @@ describe('UnifiedAuthProvider', () => {
       const TestHookComponent = () => {
         try {
           const auth = useUnifiedAuth();
-          return <div data-testid="hook-success">Hook works</div>;
+          return <p data-testid="hook-success">Hook works</p>;
         } catch (error) {
-          return <div data-testid="hook-error">Hook failed</div>;
+          return <p data-testid="hook-error">Hook failed</p>;
         }
       };
 
@@ -160,7 +160,7 @@ describe('UnifiedAuthProvider', () => {
         </UnifiedAuthProvider>
       );
 
-      expect(screen.getByTestId('hook-success')).toBeInTheDocument();
+      expect(screen.getByTestId('hook-success')).toBeDefined();
     });
   });
 
@@ -177,7 +177,7 @@ describe('UnifiedAuthProvider', () => {
         </UnifiedAuthProvider>
       );
 
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
+      expect(screen.getByTestId('test-component')).toBeDefined();
     });
 
     it('uses default configuration', () => {
@@ -187,7 +187,7 @@ describe('UnifiedAuthProvider', () => {
         </UnifiedAuthProvider>
       );
 
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
+      expect(screen.getByTestId('test-component')).toBeDefined();
     });
   });
 
@@ -197,12 +197,12 @@ describe('UnifiedAuthProvider', () => {
         const auth = useUnifiedAuth();
         
         return (
-          <div>
-            <div data-testid="composition-success">composed</div>
-            <div data-testid="has-all-contexts">
+          <section>
+            <p data-testid="composition-success">composed</p>
+            <p data-testid="has-all-contexts">
               {auth.user && auth.session && auth.isAuthenticated ? 'yes' : 'no'}
-            </div>
-          </div>
+            </p>
+          </section>
         );
       };
 
@@ -225,13 +225,13 @@ describe('UnifiedAuthProvider', () => {
         const auth = useUnifiedAuth();
         
         return (
-          <div>
-            <div data-testid="user-context">{auth.user ? 'present' : 'missing'}</div>
-            <div data-testid="session-context">{auth.session ? 'present' : 'missing'}</div>
-            <div data-testid="organisation-context">{auth.selectedOrganisation ? 'present' : 'missing'}</div>
-            <div data-testid="event-context">{auth.selectedEvent ? 'missing' : 'present'}</div>
-            <div data-testid="app-context">{auth.appName ? 'present' : 'missing'}</div>
-          </div>
+          <section>
+            <p data-testid="user-context">{auth.user ? 'present' : 'missing'}</p>
+            <p data-testid="session-context">{auth.session ? 'present' : 'missing'}</p>
+            <p data-testid="organisation-context">{auth.selectedOrganisation ? 'present' : 'missing'}</p>
+            <p data-testid="event-context">{auth.selectedEvent ? 'missing' : 'present'}</p>
+            <p data-testid="app-context">{auth.appName ? 'present' : 'missing'}</p>
+          </section>
         );
       };
 
@@ -258,10 +258,10 @@ describe('UnifiedAuthProvider', () => {
         const auth = useUnifiedAuth();
         
         return (
-          <div>
-            <div data-testid="provider-hierarchy">valid</div>
-            <div data-testid="auth-available">{auth ? 'yes' : 'no'}</div>
-          </div>
+          <section>
+            <p data-testid="provider-hierarchy">valid</p>
+            <p data-testid="auth-available">{auth ? 'yes' : 'no'}</p>
+          </section>
         );
       };
 
@@ -282,14 +282,14 @@ describe('UnifiedAuthProvider', () => {
         const auth = useUnifiedAuth();
         
         return (
-          <div>
-            <div data-testid="loading-orchestration">
+          <section>
+            <p data-testid="loading-orchestration">
               {auth.isLoading ? 'loading' : 'ready'}
-            </div>
-            <div data-testid="context-ready">
+            </p>
+            <p data-testid="context-ready">
               {auth.user && auth.session ? 'ready' : 'initializing'}
-            </div>
-          </div>
+            </p>
+          </section>
         );
       };
 
@@ -299,8 +299,8 @@ describe('UnifiedAuthProvider', () => {
         </UnifiedAuthProvider>
       );
 
-      expect(screen.getByTestId('loading-orchestration')).toBeInTheDocument();
-      expect(screen.getByTestId('context-ready')).toBeInTheDocument();
+      expect(screen.getByTestId('loading-orchestration')).toBeDefined();
+      expect(screen.getByTestId('context-ready')).toBeDefined();
     });
 
     it('coordinates loading across multiple context sources', () => {
@@ -308,12 +308,12 @@ describe('UnifiedAuthProvider', () => {
         const auth = useUnifiedAuth();
         
         return (
-          <div>
-            <div data-testid="coordination">{auth.isLoading ? 'coordinating' : 'coordinated'}</div>
-            <div data-testid="all-contexts-loaded">
+          <section>
+            <p data-testid="coordination">{auth.isLoading ? 'coordinating' : 'coordinated'}</p>
+            <p data-testid="all-contexts-loaded">
               {auth.user && auth.session && !auth.isLoading ? 'loaded' : 'loading'}
-            </div>
-          </div>
+            </p>
+          </section>
         );
       };
 
@@ -323,8 +323,8 @@ describe('UnifiedAuthProvider', () => {
         </UnifiedAuthProvider>
       );
 
-      expect(screen.getByTestId('coordination')).toBeInTheDocument();
-      expect(screen.getByTestId('all-contexts-loaded')).toBeInTheDocument();
+      expect(screen.getByTestId('coordination')).toBeDefined();
+      expect(screen.getByTestId('all-contexts-loaded')).toBeDefined();
     });
   });
 
@@ -334,12 +334,12 @@ describe('UnifiedAuthProvider', () => {
         const auth = useUnifiedAuth();
         
         return (
-          <div>
-            <div data-testid="error-state">{auth.error ? 'has-error' : 'no-error'}</div>
-            <div data-testid="still-functional">
+          <section>
+            <p data-testid="error-state">{auth.error ? 'has-error' : 'no-error'}</p>
+            <p data-testid="still-functional">
               {auth.user || auth.session ? 'functional' : 'broken'}
-            </div>
-          </div>
+            </p>
+          </section>
         );
       };
 
@@ -349,8 +349,8 @@ describe('UnifiedAuthProvider', () => {
         </UnifiedAuthProvider>
       );
 
-      expect(screen.getByTestId('error-state')).toBeInTheDocument();
-      expect(screen.getByTestId('still-functional')).toBeInTheDocument();
+      expect(screen.getByTestId('error-state')).toBeDefined();
+      expect(screen.getByTestId('still-functional')).toBeDefined();
     });
 
     it('provides error recovery mechanisms', () => {
@@ -358,12 +358,12 @@ describe('UnifiedAuthProvider', () => {
         const auth = useUnifiedAuth();
         
         return (
-          <div>
-            <div data-testid="recovery-available">
+          <section>
+            <p data-testid="recovery-available">
               {auth.refreshSession ? 'available' : 'unavailable'}
-            </div>
-            <div data-testid="error-handled">{auth.error ? 'handled' : 'no-error'}</div>
-          </div>
+            </p>
+            <p data-testid="error-handled">{auth.error ? 'handled' : 'no-error'}</p>
+          </section>
         );
       };
 
@@ -373,8 +373,8 @@ describe('UnifiedAuthProvider', () => {
         </UnifiedAuthProvider>
       );
 
-      expect(screen.getByTestId('recovery-available')).toBeInTheDocument();
-      expect(screen.getByTestId('error-handled')).toBeInTheDocument();
+      expect(screen.getByTestId('recovery-available')).toBeDefined();
+      expect(screen.getByTestId('error-handled')).toBeDefined();
     });
 
     it('maintains stability during error states', () => {
@@ -382,12 +382,12 @@ describe('UnifiedAuthProvider', () => {
         const auth = useUnifiedAuth();
         
         return (
-          <div>
-            <div data-testid="stable-during-error">
+          <section>
+            <p data-testid="stable-during-error">
               {auth ? 'stable' : 'unstable'}
-            </div>
-            <div data-testid="re-render-safe">safe</div>
-          </div>
+            </p>
+            <p data-testid="re-render-safe">safe</p>
+          </section>
         );
       };
 
@@ -397,7 +397,7 @@ describe('UnifiedAuthProvider', () => {
         </UnifiedAuthProvider>
       );
 
-      expect(screen.getByTestId('stable-during-error')).toBeInTheDocument();
+      expect(screen.getByTestId('stable-during-error')).toBeDefined();
       
       // Re-render during error state
       rerender(
@@ -416,14 +416,14 @@ describe('UnifiedAuthProvider', () => {
         const auth = useUnifiedAuth();
         
         return (
-          <div>
-            <div data-testid="hook-access">
+          <section>
+            <p data-testid="hook-access">
               {auth.user && auth.session && auth.isAuthenticated ? 'full' : 'partial'}
-            </div>
-            <div data-testid="hook-methods">
+            </p>
+            <p data-testid="hook-methods">
               {auth.signIn && auth.signOut && auth.refreshSession ? 'complete' : 'incomplete'}
-            </div>
-          </div>
+            </p>
+          </section>
         );
       };
 
@@ -447,12 +447,12 @@ describe('UnifiedAuthProvider', () => {
         const auth = useUnifiedAuth();
         
         return (
-          <div>
-            <div data-testid="user-stable">{auth.user ? 'stable' : 'unstable'}</div>
-            <div data-testid="methods-stable">
+          <section>
+            <p data-testid="user-stable">{auth.user ? 'stable' : 'unstable'}</p>
+            <p data-testid="methods-stable">
               {auth.signIn && auth.signOut ? 'stable' : 'unstable'}
-            </div>
-          </div>
+            </p>
+          </section>
         );
       };
 
@@ -486,10 +486,10 @@ describe('UnifiedAuthProvider', () => {
         const isAuthenticated = auth.isAuthenticated;
         
         return (
-          <div>
-            <div data-testid="concurrent-safe">safe</div>
-            <div data-testid="all-reads">{user && session && isAuthenticated ? 'success' : 'fail'}</div>
-          </div>
+          <section>
+            <p data-testid="concurrent-safe">safe</p>
+            <p data-testid="all-reads">{user && session && isAuthenticated ? 'success' : 'fail'}</p>
+          </section>
         );
       };
 
@@ -514,13 +514,13 @@ describe('UnifiedAuthProvider', () => {
         const auth = useUnifiedAuth();
         
         return (
-          <div>
-            <div data-testid="custom-app-name">{auth.appName}</div>
-            <div data-testid="app-name-present">{auth.appName ? 'present' : 'missing'}</div>
-            <div data-testid="app-name-consistent">
+          <section>
+            <p data-testid="custom-app-name">{auth.appName}</p>
+            <p data-testid="app-name-present">{auth.appName ? 'present' : 'missing'}</p>
+            <p data-testid="app-name-consistent">
               {auth.appName && auth.appName.length > 0 ? 'consistent' : 'inconsistent'}
-            </div>
-          </div>
+            </p>
+          </section>
         );
       };
 
@@ -542,14 +542,14 @@ describe('UnifiedAuthProvider', () => {
         const auth = useUnifiedAuth();
         
         return (
-          <div>
-            <div data-testid="rbac-enabled">
+          <section>
+            <p data-testid="rbac-enabled">
               {auth.isAuthenticated ? 'enabled' : 'disabled'}
-            </div>
-            <div data-testid="session-available">
+            </p>
+            <p data-testid="session-available">
               {auth.session ? 'available' : 'unavailable'}
-            </div>
-          </div>
+            </p>
+          </section>
         );
       };
 
@@ -564,7 +564,7 @@ describe('UnifiedAuthProvider', () => {
         expect(screen.getByTestId('rbac-enabled')).toHaveTextContent('enabled');
       }, { timeout: 3000 });
       
-      expect(screen.getByTestId('session-available')).toBeInTheDocument();
+      expect(screen.getByTestId('session-available')).toBeDefined();
     });
 
     it('configures inactivity tracking when enabled', () => {
@@ -572,14 +572,14 @@ describe('UnifiedAuthProvider', () => {
         const auth = useUnifiedAuth();
         
         return (
-          <div>
-            <div data-testid="inactivity-config">
+          <section>
+            <p data-testid="inactivity-config">
               {auth.isLoading ? 'configuring' : 'configured'}
-            </div>
-            <div data-testid="context-complete">
+            </p>
+            <p data-testid="context-complete">
               {auth.user && auth.session ? 'complete' : 'incomplete'}
-            </div>
-          </div>
+            </p>
+          </section>
         );
       };
 
@@ -589,7 +589,7 @@ describe('UnifiedAuthProvider', () => {
         </UnifiedAuthProvider>
       );
 
-      expect(screen.getByTestId('inactivity-config')).toBeInTheDocument();
+      expect(screen.getByTestId('inactivity-config')).toBeDefined();
       expect(screen.getByTestId('context-complete')).toHaveTextContent('complete');
     });
   });

package/src/providers/services/__tests__/AuthServiceProvider.integration.test.tsx

@@ -67,12 +67,12 @@ function TestComponent() {
   }, [authService]);
   
   return (
-    <div>
-      <div data-testid="user-id">{authService.getUser()?.id || 'no-user'}</div>
-      <div data-testid="is-authenticated">{authService.isAuthenticated().toString()}</div>
-      <div data-testid="is-loading">{authService.isLoading().toString()}</div>
-      <div data-testid="error">{authService.getError()?.message || 'no-error'}</div>
-    </div>
+    <section>
+      <p data-testid="user-id">{authService.getUser()?.id || 'no-user'}</p>
+      <p data-testid="is-authenticated">{authService.isAuthenticated().toString()}</p>
+      <p data-testid="is-loading">{authService.isLoading().toString()}</p>
+      <p data-testid="error">{authService.getError()?.message || 'no-error'}</p>
+    </section>
   );
 }
 
@@ -198,7 +198,7 @@ describe('AuthServiceProvider Integration', () => {
       // The subscription is set up in useEffect, so we need to wait for that
       await waitFor(() => {
         // Check if subscription is set up by verifying the component has rendered
-        expect(screen.getByTestId('user-id')).toBeInTheDocument();
+        expect(screen.getByTestId('user-id')).toBeDefined();
       }, { interval: 10, timeout: 1000 });
 
       // Wait a bit more to ensure subscription is fully set up

package/src/providers/services/__tests__/UnifiedAuthProvider.integration.test.tsx

@@ -61,16 +61,16 @@ function TestComponent() {
   const auth = useUnifiedAuth();
   
   return (
-    <div>
-      <div data-testid="user-id">{auth.user?.id || 'no-user'}</div>
-      <div data-testid="is-authenticated">{auth.isAuthenticated.toString()}</div>
-      <div data-testid="is-loading">{auth.isLoading.toString()}</div>
-      <div data-testid="app-name">{auth.appName}</div>
-       <div data-testid="permissions-count">{0}</div>
-       <div data-testid="roles-count">{0}</div>
-      <div data-testid="organisations-count">{auth.organisations.length}</div>
-      <div data-testid="events-count">{auth.events.length}</div>
-    </div>
+    <section>
+      <p data-testid="user-id">{auth.user?.id || 'no-user'}</p>
+      <p data-testid="is-authenticated">{auth.isAuthenticated.toString()}</p>
+      <p data-testid="is-loading">{auth.isLoading.toString()}</p>
+      <p data-testid="app-name">{auth.appName}</p>
+       <p data-testid="permissions-count">{0}</p>
+       <p data-testid="roles-count">{0}</p>
+      <p data-testid="organisations-count">{auth.organisations.length}</p>
+      <p data-testid="events-count">{auth.events.length}</p>
+    </section>
   );
 }
 

package/src/styles/core.css

@@ -206,6 +206,13 @@
   dialog::backdrop {
     background-color: oklch(from var(--color-main-700) l c h / 0.5);
   }
+
+  progress::indeterminate {
+    
+    background: var(--color-acc-600);
+    /*
+    background: linear-gradient(to right, var(--color-acc-700) 10%, var(--color-sec-200) 30%, var(--color-main-500) 70%, var(--color-acc-200) 90%); */
+  }
   
   .appGradient {
     background: linear-gradient(145deg, var(--color-main-100) 10%, var(--color-main-200) 30%, var(--color-main-200) 70%, var(--color-main-300) 90%);

package/src/theming/__tests__/parseEventColours.test.ts

@@ -88,7 +88,7 @@ describe('parseAndNormalizeEventColours', () => {
   });
 
   describe('Standard format only', () => {
-    it('returns null for legacy format with ev-main, ev-sec, ev-acc keys', () => {
+    it('normalizes legacy format with ev-main, ev-sec, ev-acc keys to standard format', () => {
       const input = {
         'ev-main': { 500: { L: 0.5, C: 0.2, H: 0 } },
         'ev-sec': { 500: { L: 0.5, C: 0.2, H: 120 } },
@@ -96,8 +96,14 @@ describe('parseAndNormalizeEventColours', () => {
       };
 
       const result = parseAndNormalizeEventColours(input);
-      // Legacy format is no longer supported - returns null
-      expect(result).toBeNull();
+      // Legacy format is normalized to standard format (ev-main -> main, etc.)
+      expect(result).not.toBeNull();
+      expect(result).toHaveProperty('main');
+      expect(result).toHaveProperty('sec');
+      expect(result).toHaveProperty('acc');
+      expect(result?.main).toHaveProperty('500');
+      expect(result?.sec).toHaveProperty('500');
+      expect(result?.acc).toHaveProperty('500');
     });
 
     it('uses standard keys when provided', () => {

package/src/theming/parseEventColours.ts

@@ -8,15 +8,13 @@
  * Handles multiple input formats and ensures consistent palette structure.
  */
 
-import { createLogger } from '../utils/core/logger';
-
-const log = createLogger('ParseEventColours');
 
 /**
  * Parse and normalize event_colours to PaletteData
  * 
  * Supports input formats:
- * - Object with 'main', 'sec', 'acc' keys
+ * - Object with 'main', 'sec', 'acc' keys (standard format)
+ * - Object with 'ev-main', 'ev-sec', 'ev-acc' keys (database format with prefix)
  * - JSON string that will be parsed
  * 
  * Only includes explicitly defined color values. Does not fill
@@ -38,6 +36,18 @@ const log = createLogger('ParseEventColours');
  * // Returns: { main: { 500: {...}, raw: {...} }, sec: { 500: {...} }, acc: { 500: {...} } }
  * ```
  * 
+ * @example
+ * ```ts
+ * // Database format with ev- prefix
+ * const colours = {
+ *   'ev-main': { 500: { L: 0.5, C: 0.2, H: 0 } },
+ *   'ev-sec': { 500: { L: 0.5, C: 0.2, H: 120 } },
+ *   'ev-acc': { 500: { L: 0.5, C: 0.2, H: 240 } }
+ * };
+ * const palette = parseAndNormalizeEventColours(colours);
+ * // Returns: { main: { 500: {...} }, sec: { 500: {...} }, acc: { 500: {...} } }
+ * ```
+ * 
  */
 export function parseAndNormalizeEventColours(input: unknown): { main: any; sec: any; acc: any } | null {
   try {
@@ -55,13 +65,16 @@ export function parseAndNormalizeEventColours(input: unknown): { main: any; sec:
       return null;
     }
 
-    // Use standard 'main'/'sec'/'acc' keys
-    const main = obj?.main || null;
-    const sec  = obj?.sec  || null;
-    const acc  = obj?.acc  || null;
+    // Support both standard keys (main/sec/acc) and prefixed keys (ev-main/ev-sec/ev-acc)
+    // Database may store with ev-* prefix, but we normalize to standard keys
+    const main = obj?.main || obj?.['ev-main'] || null;
+    const sec  = obj?.sec  || obj?.['ev-sec']  || null;
+    const acc  = obj?.acc  || obj?.['ev-acc'] || null;
     
     // If no palette data found, return null
-    if (!main && !sec && !acc) return null;
+    if (!main && !sec && !acc) {
+      return null;
+    }
 
     // Helper: only include explicitly defined color values
     // This ensures we don't include undefined shades in the palette
@@ -88,7 +101,6 @@ export function parseAndNormalizeEventColours(input: unknown): { main: any; sec:
       acc: fill(acc) 
     };
   } catch (error) {
-    log.warn('Failed to parse/normalize event colours:', error);
     return null;
   }
 }