@jmruthers/pace-core 0.5.135 → 0.5.137

package/src/components/ProtectedRoute/ProtectedRoute.test.tsx

@@ -98,7 +98,21 @@ describe('ProtectedRoute Component', () => {
     isLoading: false,
   };
 
+  let originalMode: string | undefined;
+
   beforeEach(() => {
+    // Ensure logger is enabled by setting MODE to development
+    originalMode = import.meta.env.MODE;
+    (import.meta.env as any).MODE = 'development';
+    
+    import('../../utils/core/logger').then(({ Logger, LogLevel }) => {
+      Logger.configure({
+        level: LogLevel.DEBUG,
+        includeTimestamp: false,
+        includeComponent: true,
+      });
+    });
+    
     vi.clearAllMocks();
     console.debug = vi.fn();
     console.warn = vi.fn();
@@ -112,6 +126,11 @@ describe('ProtectedRoute Component', () => {
   afterEach(() => {
     console.debug = originalConsoleDebug;
     console.warn = originalConsoleWarn;
+    
+    // Restore original mode
+    if (originalMode !== undefined) {
+      (import.meta.env as any).MODE = originalMode;
+    }
   });
 
   describe('Rendering', () => {
@@ -124,6 +143,8 @@ describe('ProtectedRoute Component', () => {
     });
 
     it('renders outlet when events exist but none selected (allows event selector visibility)', () => {
+      const consoleDebugSpy = vi.spyOn(console, 'debug').mockImplementation(() => {});
+      
       mockUseEvents.mockReturnValue({
         selectedEvent: null,
         events: [{ id: 'event-1', name: 'Test Event' }],
@@ -133,8 +154,8 @@ describe('ProtectedRoute Component', () => {
       renderWithProviders(<ProtectedRoute />);
 
       expect(screen.getByTestId('outlet')).toBeInTheDocument();
-      expect(console.debug).toHaveBeenCalledWith(
-        '[ProtectedRoute] Events available but none selected - allowing render so selector is visible'
+      expect(consoleDebugSpy).toHaveBeenCalledWith(
+        expect.stringContaining('[DEBUG] [ProtectedRoute] Events available but none selected - allowing render so selector is visible')
       );
     });
 
@@ -222,6 +243,8 @@ describe('ProtectedRoute Component', () => {
     });
 
     it('warns when redirecting due to session restoration timeout', () => {
+      const consoleWarnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {});
+      
       mockUseUnifiedAuth.mockReturnValue({
         isAuthenticated: false,
         authLoading: false,
@@ -236,8 +259,8 @@ describe('ProtectedRoute Component', () => {
 
       renderWithProviders(<ProtectedRoute />);
 
-      expect(console.warn).toHaveBeenCalledWith(
-        '[ProtectedRoute] Session restoration failed, redirecting to login',
+      expect(consoleWarnSpy).toHaveBeenCalledWith(
+        expect.stringContaining('[WARN] [ProtectedRoute] Session restoration failed, redirecting to login'),
         expect.objectContaining({
           timedOut: true,
         })
@@ -259,10 +282,12 @@ describe('ProtectedRoute Component', () => {
         hasTimedOut: false,
       });
 
+      const consoleWarnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {});
+
       renderWithProviders(<ProtectedRoute />);
 
-      expect(console.warn).toHaveBeenCalledWith(
-        '[ProtectedRoute] Session restoration failed, redirecting to login',
+      expect(consoleWarnSpy).toHaveBeenCalledWith(
+        expect.stringContaining('[WARN] [ProtectedRoute] Session restoration failed, redirecting to login'),
         expect.objectContaining({
           error: 'Session restoration failed',
           timedOut: false,
@@ -408,11 +433,13 @@ describe('ProtectedRoute Component', () => {
         isLoading: false,
       });
 
+      const consoleDebugSpy = vi.spyOn(console, 'debug').mockImplementation(() => {});
+
       renderWithProviders(<ProtectedRoute requireEvent={true} />);
 
       expect(screen.getByTestId('outlet')).toBeInTheDocument();
-      expect(console.debug).toHaveBeenCalledWith(
-        '[ProtectedRoute] Events available but none selected - allowing render so selector is visible'
+      expect(consoleDebugSpy).toHaveBeenCalledWith(
+        expect.stringContaining('[DEBUG] [ProtectedRoute] Events available but none selected - allowing render so selector is visible')
       );
     });
 

package/src/components/ProtectedRoute/ProtectedRoute.tsx

@@ -76,6 +76,7 @@ import { useEvents } from '../../hooks/useEvents';
 import { LoadingSpinner } from '../LoadingSpinner/LoadingSpinner';
 import { SessionRestorationLoader } from '../SessionRestorationLoader';
 import { Alert, AlertDescription, AlertTitle } from '../Alert/Alert';
+import { logger } from '../../utils/core/logger';
 
 export interface ProtectedRouteProps {
   /**
@@ -175,7 +176,7 @@ export function ProtectedRoute({
   // Redirect to login if not authenticated
   if (!isAuthenticated) {
     if (sessionRestoration.hasTimedOut || sessionRestoration.restorationError) {
-      console.warn('[ProtectedRoute] Session restoration failed, redirecting to login', {
+      logger.warn('ProtectedRoute', 'Session restoration failed, redirecting to login', {
         timedOut: sessionRestoration.hasTimedOut,
         error: sessionRestoration.restorationError?.message
       });
@@ -217,7 +218,7 @@ export function ProtectedRoute({
   // The event selector will be visible and user can select, or auto-selection will kick in
   if (!selectedEvent) {
     // Log for debugging - this is expected behavior, not an error
-    console.debug('[ProtectedRoute] Events available but none selected - allowing render so selector is visible');
+    logger.debug('ProtectedRoute', 'Events available but none selected - allowing render so selector is visible');
     return <Outlet />;
   }
 

package/src/components/PublicLayout/PublicErrorBoundary.tsx

@@ -40,7 +40,7 @@
  */
 
 import React, { Component, ErrorInfo, ReactNode } from 'react';
-import { cn } from '../../utils/cn';
+import { cn } from '../../utils/core/cn';
 
 export interface PublicErrorBoundaryProps {
   /** Child components to wrap */

package/src/components/PublicLayout/PublicLoadingSpinner.tsx

@@ -37,7 +37,7 @@
  */
 
 import React from 'react';
-import { cn } from '../../utils/cn';
+import { cn } from '../../utils/core/cn';
 
 export interface PublicLoadingSpinnerProps {
   /** Loading message to display */

package/src/components/PublicLayout/PublicPageContextChecker.tsx

@@ -10,6 +10,7 @@
 
 import React, { useEffect } from 'react';
 import { usePublicPageContext } from './PublicPageProvider';
+import { logger } from '../../utils/core/logger';
 
 export interface PublicPageContextCheckerProps {
   /** Whether to enable checking */
@@ -23,88 +24,88 @@ export interface PublicPageContextCheckerProps {
  * 
  * This component will immediately log to console if authentication context
  * is being triggered in a public page.
+ * 
+ * Only enabled in development mode when VITE_ENABLE_DEBUG_LOGS is 'true'
  */
 export function PublicPageContextChecker({ enabled = true, label = 'PublicPage' }: PublicPageContextCheckerProps) {
+  // Check if debug logging is enabled
+  // Allow in development mode with VITE_ENABLE_DEBUG_LOGS, or in test mode
+  // Also check for VITEST environment variable which is set by Vitest
+  const isDebugEnabled = 
+    (import.meta.env.MODE === 'development' && import.meta.env.VITE_ENABLE_DEBUG_LOGS === 'true') ||
+    import.meta.env.MODE === 'test' ||
+    import.meta.env.VITEST === true ||
+    (typeof process !== 'undefined' && process.env.NODE_ENV === 'test');
+
+  // Early return if not in debug mode
+  if (!isDebugEnabled || !enabled) {
+    return null;
+  }
+
   useEffect(() => {
-    if (!enabled) return;
 
-    console.group(`🚨 [${label}] PUBLIC PAGE CONTEXT CHECK`);
+    logger.debug('PublicPageContextChecker', `🚨 [${label}] PUBLIC PAGE CONTEXT CHECK`);
     
     // Check for authentication context
     try {
       // This will throw if we're not in UnifiedAuthProvider
       const { isAuthenticated } = require('../../providers/UnifiedAuthProvider').useUnifiedAuth();
-      console.error(`❌ [${label}] AUTHENTICATION CONTEXT DETECTED!`);
-      console.error(`❌ [${label}] isAuthenticated:`, isAuthenticated);
-      console.error(`❌ [${label}] This public page is inside UnifiedAuthProvider - THIS IS WRONG!`);
-      console.error(`❌ [${label}] SOLUTION: Move public routes outside of authentication providers`);
+      logger.error('PublicPageContextChecker', `❌ [${label}] AUTHENTICATION CONTEXT DETECTED!`);
+      logger.error('PublicPageContextChecker', `❌ [${label}] isAuthenticated:`, isAuthenticated);
+      logger.error('PublicPageContextChecker', `❌ [${label}] This public page is inside UnifiedAuthProvider - THIS IS WRONG!`);
+      logger.error('PublicPageContextChecker', `❌ [${label}] SOLUTION: Move public routes outside of authentication providers`);
     } catch (error) {
-      console.log(`✅ [${label}] No authentication context detected (GOOD!)`);
+      logger.debug('PublicPageContextChecker', `✅ [${label}] No authentication context detected (GOOD!)`);
     }
 
     // Check for organisation context
     try {
       // This will throw if we're not in OrganisationProvider
       const { selectedOrganisation } = require('../../providers/OrganisationProvider').useOrganisations();
-      console.error(`❌ [${label}] ORGANISATION CONTEXT DETECTED!`);
-      console.error(`❌ [${label}] selectedOrganisation:`, selectedOrganisation);
-      console.error(`❌ [${label}] This public page is inside OrganisationProvider - THIS IS WRONG!`);
-      console.error(`❌ [${label}] SOLUTION: Move public routes outside of authentication providers`);
+      logger.error('PublicPageContextChecker', `❌ [${label}] ORGANISATION CONTEXT DETECTED!`);
+      logger.error('PublicPageContextChecker', `❌ [${label}] selectedOrganisation:`, selectedOrganisation);
+      logger.error('PublicPageContextChecker', `❌ [${label}] This public page is inside OrganisationProvider - THIS IS WRONG!`);
+      logger.error('PublicPageContextChecker', `❌ [${label}] SOLUTION: Move public routes outside of authentication providers`);
     } catch (error) {
-      console.log(`✅ [${label}] No organisation context detected (GOOD!)`);
+      logger.debug('PublicPageContextChecker', `✅ [${label}] No organisation context detected (GOOD!)`);
     }
 
     // Check for event context
     try {
       // This will throw if we're not in EventProvider
       const { events } = require('../../providers/EventProvider').useEvents();
-      console.error(`❌ [${label}] EVENT CONTEXT DETECTED!`);
-      console.error(`❌ [${label}] events:`, events);
-      console.error(`❌ [${label}] This public page is inside EventProvider - THIS IS WRONG!`);
-      console.error(`❌ [${label}] SOLUTION: Move public routes outside of authentication providers`);
+      logger.error('PublicPageContextChecker', `❌ [${label}] EVENT CONTEXT DETECTED!`);
+      logger.error('PublicPageContextChecker', `❌ [${label}] events:`, events);
+      logger.error('PublicPageContextChecker', `❌ [${label}] This public page is inside EventProvider - THIS IS WRONG!`);
+      logger.error('PublicPageContextChecker', `❌ [${label}] SOLUTION: Move public routes outside of authentication providers`);
     } catch (error) {
-      console.log(`✅ [${label}] No event context detected (GOOD!)`);
+      logger.debug('PublicPageContextChecker', `✅ [${label}] No event context detected (GOOD!)`);
     }
 
     // Check for PublicPageProvider context
     try {
       const { isPublicPage } = usePublicPageContext();
       if (isPublicPage) {
-        console.log(`✅ [${label}] Public page context detected (GOOD!)`);
+        logger.debug('PublicPageContextChecker', `✅ [${label}] Public page context detected (GOOD!)`);
       } else {
-        console.warn(`⚠️ [${label}] Not in PublicPageProvider context`);
-        console.warn(`⚠️ [${label}] SOLUTION: Wrap your public page in <PublicPageProvider>`);
+        logger.warn('PublicPageContextChecker', `⚠️ [${label}] Not in PublicPageProvider context`);
+        logger.warn('PublicPageContextChecker', `⚠️ [${label}] SOLUTION: Wrap your public page in <PublicPageProvider>`);
       }
     } catch (error) {
-      console.warn(`⚠️ [${label}] Not in PublicPageProvider context`);
-      console.warn(`⚠️ [${label}] SOLUTION: Wrap your public page in <PublicPageProvider>`);
+      logger.warn('PublicPageContextChecker', `⚠️ [${label}] Not in PublicPageProvider context`);
+      logger.warn('PublicPageContextChecker', `⚠️ [${label}] SOLUTION: Wrap your public page in <PublicPageProvider>`);
     }
 
-    console.groupEnd();
-
     // Provide immediate guidance
-    console.group(`📖 [${label}] IMMEDIATE ACTION REQUIRED`);
-    console.log(`If you see any ❌ errors above, your public page is inside authentication context.`);
-    console.log(`This will cause infinite loading loops and authentication errors.`);
-    console.log(``);
-    console.log(`🔧 SOLUTION:`);
-    console.log(`1. Check your main App.tsx file`);
-    console.log(`2. Make sure public routes are completely separate from authentication providers`);
-    console.log(`3. Follow the architecture in: packages/core/docs/public-pages-guide.md`);
-    console.log(``);
-    console.log(`✅ CORRECT ARCHITECTURE:`);
-    console.log(`<BrowserRouter>`);
-    console.log(`  <Routes>`);
-    console.log(`    <Route path="/events/*" element={<PublicPageApp />} />`);
-    console.log(`    <Route path="/*" element={<AuthenticatedApp />} />`);
-    console.log(`  </Routes>`);
-    console.log(`</BrowserRouter>`);
-    console.groupEnd();
+    logger.debug('PublicPageContextChecker', `📖 [${label}] IMMEDIATE ACTION REQUIRED`);
+    logger.debug('PublicPageContextChecker', `If you see any ❌ errors above, your public page is inside authentication context.`);
+    logger.debug('PublicPageContextChecker', `This will cause infinite loading loops and authentication errors.`);
+    logger.debug('PublicPageContextChecker', `🔧 SOLUTION: Check your main App.tsx file`);
+    logger.debug('PublicPageContextChecker', `Make sure public routes are completely separate from authentication providers`);
+    logger.debug('PublicPageContextChecker', `Follow the architecture in: packages/core/docs/public-pages-guide.md`);
 
   }, [enabled, label]);
 
-  if (!enabled) return null;
-
   return (
     <div style={{ 
       position: 'fixed', 

package/src/components/PublicLayout/PublicPageFooter.tsx

@@ -42,7 +42,7 @@
  */
 
 import React from 'react';
-import { cn } from '../../utils/cn';
+import { cn } from '../../utils/core/cn';
 import type { Event } from '../../types/unified';
 
 export interface PublicPageFooterProps {

package/src/components/PublicLayout/PublicPageHeader.tsx

@@ -55,7 +55,7 @@ import type { Event } from '../../types/unified';
 import { FileDisplay } from '../FileDisplay/FileDisplay';
 import { FileCategory } from '../../types/file-reference';
 import { useAppConfig } from '../../hooks/useAppConfig';
-import { cn } from '../../utils/cn';
+import { cn } from '../../utils/core/cn';
 
 export interface PublicPageHeaderProps {
   /** The event data for this public page */
@@ -145,20 +145,6 @@ export function PublicPageHeader({
             <>
               {customEventLogo || (
                 <>
-                  {/* Log organisation_id derivation chain for debugging */}
-                  {(() => {
-                    console.log('[PublicPageHeader] Organisation ID Derivation Chain:', {
-                      eventCode: eventCode,
-                      eventId: event.event_id,
-                      eventName: event.event_name,
-                      organisationId: event.organisation_id,
-                      organisationIdType: typeof event.organisation_id,
-                      organisationIdValid: !!event.organisation_id && event.organisation_id !== '',
-                      derivation: 'URL → eventCode → usePublicEvent → event.organisation_id → FileDisplay',
-                      note: 'Organisation ID is derived from event data fetched using event code from URL'
-                    });
-                    return null;
-                  })()}
                   <FileDisplay
                     table_name="event"
                     record_id={event.event_id}

package/src/components/PublicLayout/PublicPageProvider.tsx

@@ -38,6 +38,7 @@ import React, { createContext, useContext, ReactNode, useMemo } from 'react';
 import { createClient } from '@supabase/supabase-js';
 import type { Database } from '../../types/database';
 import { PublicErrorBoundary } from './PublicErrorBoundary';
+import { logger } from '../../utils/core/logger';
 
 interface PublicPageContextType {
   isPublicPage: true;
@@ -93,11 +94,11 @@ export function PublicPageProvider({ children, appName }: PublicPageProviderProp
   // Create Supabase client if environment variables are available
   const supabase = useMemo(() => {
     if (!supabaseUrl || !supabaseKey) {
-      console.warn('[PublicPageProvider] Missing Supabase environment variables. Please ensure VITE_SUPABASE_URL and VITE_SUPABASE_ANON_KEY are set in your environment.');
+      logger.warn('PublicPageProvider', 'Missing Supabase environment variables. Please ensure VITE_SUPABASE_URL and VITE_SUPABASE_ANON_KEY are set in your environment.');
       return null;
     }
     const client = createClient<Database>(supabaseUrl, supabaseKey);
-    console.log('[PublicPageProvider] Supabase client created successfully for public pages');
+    logger.info('PublicPageProvider', 'Supabase client created successfully for public pages');
     return client;
   }, [supabaseUrl, supabaseKey]);
 

package/src/components/PublicLayout/__tests__/PublicPageContextChecker.test.tsx

@@ -20,6 +20,8 @@ describe('PublicPageContextChecker', () => {
   const consoleErrorSpy = vi.spyOn(console, 'error');
 
   beforeEach(() => {
+    // Component now automatically detects test mode via process.env.NODE_ENV === 'test'
+    // No need to manually set environment variables
     vi.clearAllMocks();
     consoleGroupSpy.mockImplementation(() => {});
     consoleGroupEndSpy.mockImplementation(() => {});

package/src/components/PublicLayout/__tests__/PublicPageFooter.test.tsx

@@ -12,7 +12,7 @@ import { PublicPageFooter } from '../PublicPageFooter';
 import type { Event } from '../../../types/unified';
 
 // Mock the cn utility
-vi.mock('../../../utils/cn', () => ({
+vi.mock('../../../utils/core/cn', () => ({
   cn: vi.fn((...classes) => classes.filter(Boolean).join(' '))
 }));
 

package/src/components/PublicLayout/index.ts

@@ -21,8 +21,6 @@ export { PublicPageLayout, usePublicPageContext } from './PublicPageLayout';
 export { PublicPageHeader } from './PublicPageHeader';
 export { PublicPageProvider, usePublicPageContext as usePublicPageProviderContext, useIsPublicPage } from './PublicPageProvider';
 export { PublicPageFooter } from './PublicPageFooter';
-export { PublicPageDebugger } from './PublicPageDebugger';
-export { PublicPageDiagnostic } from './PublicPageDiagnostic';
 export { PublicPageContextChecker } from './PublicPageContextChecker';
 
 
@@ -40,6 +38,10 @@ export {
   PublicLoadingSkeleton 
 } from './PublicLoadingSpinner';
 
+// === EVENT LOGO COMPONENT ===
+export { EventLogo, EventLogoCompact, EventLogoLarge } from './EventLogo';
+export type { EventLogoProps } from './EventLogo';
+
 // === TYPES ===
 export type { PublicPageLayoutProps } from './PublicPageLayout';
 export type { PublicPageHeaderProps } from './PublicPageHeader';

package/src/components/Select/Select.test.tsx

@@ -42,7 +42,7 @@ vi.mock('../Button/Button', () => ({
 }));
 
 // Mock cn utility
-vi.mock('../../utils/cn', () => ({
+vi.mock('../../utils/core/cn', () => ({
   cn: (...classes: any[]) => classes.filter(Boolean).join(' '),
 }));
 

package/src/components/Select/Select.tsx

@@ -15,7 +15,7 @@
 import * as React from "react";
 import { Search, X, ChevronDown, Check } from "lucide-react";
 import { Button, type ButtonProps } from "../Button/Button";
-import { cn } from "../../utils/cn";
+import { cn } from "../../utils/core/cn";
 import { 
   useSelectState, 
   useSelectEvents, 
@@ -33,11 +33,13 @@ export interface SelectContextValue extends SelectState {
   actions: SelectActions;
   registerItem?: (value: string, text: string) => void;
   unregisterItem?: (value: string) => void;
+  direction?: 'up' | 'down';
 }
 
 export interface SelectProps extends Omit<React.HTMLAttributes<HTMLFormElement>, 'onChange' | 'onKeyDown' | 'onFocus' | 'onBlur'> {
   children: React.ReactNode;
   className?: string;
+  direction?: 'up' | 'down';
   // State props are in UseSelectStateProps (via & intersection)
 }
 
@@ -168,6 +170,7 @@ export const Select = React.forwardRef<HTMLFormElement, SelectProps & UseSelectS
   ({ 
     children,
     className,
+    direction = 'down',
     ...selectProps
   }, ref) => {
     const internalRef = React.useRef<HTMLFormElement>(null);
@@ -317,7 +320,8 @@ export const Select = React.forwardRef<HTMLFormElement, SelectProps & UseSelectS
       actions,
       registerItem,
       unregisterItem,
-    }), [state, actions, registerItem, unregisterItem]);
+      direction,
+    }), [state, actions, registerItem, unregisterItem, direction]);
 
     return (
       <form
@@ -341,7 +345,8 @@ Select.displayName = "Select";
 
 export const SelectTrigger = React.forwardRef<HTMLButtonElement, SelectTriggerProps>(
   ({ children, className, variant = "outline", size = "default", asChild = false, ...props }, ref) => {
-    const { open, disabled, value, actions } = useSelectContext();
+    const { open, disabled, value, actions, direction = 'down' } = useSelectContext();
+    const opensUpward = direction === 'up';
 
     const handleClick = () => {
       actions.setOpen(!open);
@@ -377,7 +382,8 @@ export const SelectTrigger = React.forwardRef<HTMLButtonElement, SelectTriggerPr
       className: cn(
         "!justify-between relative w-full",
         "[&_svg]:pointer-events-none",
-        open && "!rounded-b-none !border-b-0",
+        open && !opensUpward && "!rounded-b-none !border-b-0",
+        open && opensUpward && "!rounded-t-none !border-t-0",
         className
       ),
       style: {
@@ -438,7 +444,8 @@ export const SelectTrigger = React.forwardRef<HTMLButtonElement, SelectTriggerPr
         className={cn(
           "!justify-between relative w-full",
           "[&_svg]:pointer-events-none",
-          open && "!rounded-b-none !border-b-0",
+          open && !opensUpward && "!rounded-b-none !border-b-0",
+          open && opensUpward && "!rounded-t-none !border-t-0",
           className
         )}
         style={{
@@ -493,10 +500,10 @@ export const SelectContent = React.forwardRef<HTMLUListElement, SelectContentPro
     className, 
     searchable = false,
     searchPlaceholder = "Search...",
-    maxHeight = "20rem",
+    maxHeight = "max(20rem, 50vh)",
     style
   }, ref) => {
-    const { open, actions } = useSelectContext();
+    const { open, actions, direction = 'down' } = useSelectContext();
     const { searchTerm, setSearchTerm, filteredChildren, searchInputRef } = useSelectSearch({
       children,
       searchable,
@@ -525,16 +532,21 @@ export const SelectContent = React.forwardRef<HTMLUListElement, SelectContentPro
       );
     }
 
+    const opensUpward = direction === 'up';
+    
     return (
       <ul
         ref={ref}
         className={cn(
-          "absolute z-[99999] w-full overflow-y-auto rounded-b-md border border-t-0 border-main-300 bg-main-50 shadow-lg",
+          "absolute z-[99999] w-full overflow-y-auto border border-main-300 bg-main-50 shadow-lg",
           "list-none p-0 m-0",
+          opensUpward 
+            ? "rounded-t-md border-b-0" 
+            : "rounded-b-md border-t-0",
           className
         )}
         style={{ 
-          top: '100%',
+          [opensUpward ? 'bottom' : 'top']: '100%',
           left: 0,
           right: 0,
           maxHeight,

package/src/components/{SessionRestorationLoader.tsx → SessionRestorationLoader/SessionRestorationLoader.tsx}

@@ -1,7 +1,7 @@
 /**
  * @file SessionRestorationLoader Component
  * @package @jmruthers/pace-core
- * @module Components
+ * @module Components/SessionRestorationLoader
  * @since 0.1.0
  *
  * Displays a consistent loading state while the authentication service
@@ -9,7 +9,7 @@
  */
 
 import React from 'react';
-import { LoadingSpinner } from './LoadingSpinner/LoadingSpinner';
+import { LoadingSpinner } from '../LoadingSpinner/LoadingSpinner';
 
 export interface SessionRestorationLoaderProps {
   /** Customise the status message displayed under the spinner */
@@ -46,3 +46,4 @@ export const SessionRestorationLoader: React.FC<SessionRestorationLoaderProps> =
     </div>
   );
 };
+

package/src/components/SessionRestorationLoader/index.ts

@@ -0,0 +1,3 @@
+export { SessionRestorationLoader } from './SessionRestorationLoader';
+export type { SessionRestorationLoaderProps } from './SessionRestorationLoader';
+

package/src/components/Switch/Switch.tsx

@@ -69,7 +69,7 @@
 
 import * as React from "react";
 import * as SwitchPrimitive from "@radix-ui/react-switch";
-import { cn } from "../../utils/cn";
+import { cn } from "../../utils/core/cn";
 
 /**
  * Switch component props

package/src/components/Table/Table.tsx

@@ -91,7 +91,7 @@
 
 import * as React from "react"
 
-import { cn } from "../../utils/cn"
+import { cn } from "../../utils/core/cn"
 
 /**
  * Table component

package/src/components/Table/__tests__/Table.test.tsx

@@ -20,7 +20,7 @@ import {
 } from '../Table';
 
 // Mock the cn utility
-vi.mock('../../../utils/cn', () => ({
+vi.mock('../../../utils/core/cn', () => ({
   cn: vi.fn((...classes) => classes.filter(Boolean).join(' '))
 }));
 

package/src/components/Toast/Toast.tsx

@@ -88,7 +88,7 @@
 import * as React from "react"
 import * as ToastPrimitives from "@radix-ui/react-toast"
 import { X } from "lucide-react"
-import { cn } from "../../utils/cn"
+import { cn } from "../../utils/core/cn"
 import { useToast } from "../../hooks/useToast"
 
 const ToastProvider = ToastPrimitives.Provider

package/src/components/Tooltip/Tooltip.tsx

@@ -73,7 +73,7 @@
 
 import * as React from "react";
 import * as TooltipPrimitive from "@radix-ui/react-tooltip";
-import { cn } from "../../utils/cn";
+import { cn } from "../../utils/core/cn";
 
 const TooltipProvider = TooltipPrimitive.Provider;
 

package/src/components/index.ts

@@ -49,6 +49,9 @@ export type { LabelProps } from './Label';
 export { Alert, AlertTitle, AlertDescription } from './Alert';
 export { Avatar, AvatarImage, AvatarFallback } from './Avatar';
 
+export { Badge } from './Badge';
+export type { BadgeProps, BadgeVariant } from './Badge';
+
 export { Checkbox } from './Checkbox';
 export { Switch } from './Switch';
 export type { SwitchProps } from './Switch';
@@ -151,17 +154,11 @@ export type { DataRecord } from './DataTable/types';
 
 export { 
   Form, 
-  FormField, 
-  FormErrorSummary, 
-  FormLiveRegion, 
-  FormFieldset 
+  FormField
 } from './Form';
 export type { 
   FormProps, 
-  FormFieldProps, 
-  FormErrorSummaryProps, 
-  FormLiveRegionProps, 
-  FormFieldsetProps 
+  FormFieldProps
 } from './Form';
 
 // LoginForm - ensure it's exported
@@ -212,6 +209,7 @@ export { ErrorBoundary } from './ErrorBoundary';
 export type { ErrorBoundaryProps, ErrorBoundaryState } from './ErrorBoundary';
 export { LoadingSpinner } from './LoadingSpinner';
 export { SessionRestorationLoader } from './SessionRestorationLoader';
+export type { SessionRestorationLoaderProps } from './SessionRestorationLoader';
 
 // ============================================================================
 // EVENT MANAGEMENT
@@ -223,8 +221,7 @@ export { EventSelector } from './EventSelector';
 // AUTHENTICATION FORMS
 // ============================================================================
 
-// Password Component exports
-export { PasswordResetForm } from './PasswordReset';
+// Password Component exports (PasswordResetForm removed - unused)
 
 // ============================================================================
 // STORAGE COMPONENTS