@jmruthers/pace-core 0.6.10 → 0.6.11

package/src/types/database.generated.ts

@@ -2154,7 +2154,6 @@ export type Database = {
       }
       core_forms: {
         Row: {
-          allow_multiple_submissions: boolean | null
           closes_at: string | null
           confirmation_message: string | null
           context_id: string | null
@@ -2179,7 +2178,6 @@ export type Database = {
           updated_by: string | null
         }
         Insert: {
-          allow_multiple_submissions?: boolean | null
           closes_at?: string | null
           confirmation_message?: string | null
           context_id?: string | null
@@ -2204,7 +2202,6 @@ export type Database = {
           updated_by?: string | null
         }
         Update: {
-          allow_multiple_submissions?: boolean | null
           closes_at?: string | null
           confirmation_message?: string | null
           context_id?: string | null
@@ -6744,10 +6741,9 @@ export type Database = {
       }
       app_cake_mealplan_bulk_delete: {
         Args: {
-          p_event_id?: string
-          p_organisation_id?: string
           p_unit_id: string
-          p_user_id?: string
+          p_organisation_id: string
+          p_event_id?: string
         }
         Returns: {
           deleted_count: number
@@ -6755,6 +6751,34 @@ export type Database = {
           success: boolean
         }[]
       }
+      app_cake_diners_bulk_import: {
+        Args: {
+          p_diners_data: Json
+          p_organisation_id: string
+          p_event_id?: string
+        }
+        Returns: {
+          errors: Json
+          failed: number
+          imported: number
+          message: string
+          success: boolean
+        }[]
+      }
+      app_cake_mealplans_bulk_import: {
+        Args: {
+          p_mealplans_data: Json
+          p_organisation_id: string
+          p_event_id?: string
+        }
+        Returns: {
+          errors: Json
+          failed: number
+          imported: number
+          message: string
+          success: boolean
+        }[]
+      }
       app_cake_mealplan_create: {
         Args: {
           p_mealplan_dish_id: string
@@ -6851,8 +6875,8 @@ export type Database = {
       app_cake_meals_bulk_import: {
         Args: {
           p_meals_data: Json
-          p_organisation_id?: string
-          p_user_id?: string
+          p_organisation_id: string
+          p_event_id?: string
         }
         Returns: {
           errors: Json
@@ -7331,9 +7355,9 @@ export type Database = {
       }
       app_cake_units_bulk_import: {
         Args: {
-          p_organisation_id?: string
           p_units_data: Json
-          p_user_id?: string
+          p_organisation_id: string
+          p_event_id?: string
         }
         Returns: {
           errors: Json
@@ -8356,6 +8380,17 @@ export type Database = {
           updated_by: string
         }[]
       }
+      data_cake_event_items_supply_stats: {
+        Args: {
+          p_event_id: string
+          p_organisation_id?: string
+          p_user_id?: string
+        }
+        Returns: {
+          items_total: number
+          items_with_supply_details: number
+        }[]
+      }
       data_cake_unit_details: {
         Args: {
           p_event_id?: string

package/src/types/event.ts

@@ -28,28 +28,30 @@ export interface EventTheme {
   light: ThemeColors;
 }
 
-/**
- * Event interface matching the database schema
- * All events must have an organisation_id for security
- */
-export interface Event {
-  // Primary identification
+/** Primary identification for Event (ISP). */
+export interface EventIdentity {
   id: string;
   event_id: string;
   event_name: string;
   event_code?: string;
-  
-  // Organisation context - MANDATORY for security
+}
+
+/** Organisation context for Event - MANDATORY for security (ISP). */
+export interface EventOrganisation {
   organisation_id: OrganisationId;
-  
-  // Event details
+}
+
+/** Event details (ISP). */
+export interface EventDetails {
   event_date?: string;
   event_days?: number;
   event_venue?: string;
   event_participants?: number;
   is_visible?: boolean;
-  
-  // Visual branding
+}
+
+/** Visual branding for Event (ISP). */
+export interface EventBranding {
   /** @deprecated Use file_references table instead */
   event_logo?: string | null;
   event_colours?: Record<string, unknown> | null;
@@ -59,8 +61,10 @@ export interface Event {
   event_col_m2?: string;
   event_col_l1?: string;
   event_col_l2?: string;
-  
-  // Additional metadata
+}
+
+/** Additional metadata for Event (ISP). */
+export interface EventMetadata {
   event_news?: string;
   event_billing?: string;
   event_catering_email?: string;
@@ -69,19 +73,35 @@ export interface Event {
   event_rounddown?: number;
   event_youthmultiplier?: number;
   event_typicalunit?: number;
-  
-  // Timestamps
+}
+
+/** Timestamps for Event (ISP). */
+export interface EventTimestamps {
   created_at: string;
   updated_at: string;
   created_by?: string;
   updated_by?: string;
-  
-  // Additional optional fields
+}
+
+/** Additional optional fields for Event (ISP). */
+export interface EventOptional {
   description?: string;
   end_date?: string;
   theme_colors?: EventTheme;
 }
 
+/**
+ * Event type matching the database schema (ISP: composed from focused interfaces).
+ * All events must have an organisation_id for security.
+ */
+export type Event = EventIdentity
+  & EventOrganisation
+  & EventDetails
+  & EventBranding
+  & EventMetadata
+  & EventTimestamps
+  & EventOptional;
+
 /**
  * Event context type for event management hooks and providers
  */

package/src/types/{__tests__/file-reference.test.ts → file-reference.test.ts}

@@ -1,7 +1,7 @@
 /**
  * @file File Reference Type Tests
  * @package @jmruthers/pace-core
- * @module Types/__tests__
+ * @module Types
  * @since 1.0.0
  */
 
@@ -16,7 +16,7 @@ import {
   type FileUrlInfo,
   type FileReferenceWithUrl,
   type UploadProgress
-} from '../file-reference';
+} from './file-reference';
 
 describe('[types] File Reference Types', () => {
   describe('FileReference interface', () => {
@@ -319,17 +319,17 @@ describe('[types] File Reference Types', () => {
   describe('FileReferenceService interface', () => {
     it('validates FileReferenceService has all required methods', () => {
       const service: FileReferenceService = {
-        createFileReference: async () => ({} as FileReference),
-        getFileReference: async () => null,
-        getFileReferenceById: async () => null,
-        getFileUrl: async () => null,
-        getSignedUrl: async () => null,
-        updateFileReference: async () => ({} as FileReference),
-        deleteFileReference: async () => false,
-        listFileReferences: async () => [],
-        getFilesByCategory: async () => [],
-        getFileCount: async () => 0,
-        uploadMultipleFiles: async () => ({ success: [], failed: [] })
+        createFileReference: async () => ({ ok: true, data: {} as FileReference }),
+        getFileReference: async () => ({ ok: true, data: null }),
+        getFileReferenceById: async () => ({ ok: true, data: null }),
+        getFileUrl: async () => ({ ok: true, data: null }),
+        getSignedUrl: async () => ({ ok: true, data: null }),
+        updateFileReference: async () => ({ ok: true, data: {} as FileReference }),
+        deleteFileReference: async () => ({ ok: true, data: true }),
+        listFileReferences: async () => ({ ok: true, data: [] }),
+        getFilesByCategory: async () => ({ ok: true, data: [] }),
+        getFileCount: async () => ({ ok: true, data: 0 }),
+        uploadMultipleFiles: async () => ({ ok: true, data: { success: [], failed: [] } })
       };
 
       expect(service).toHaveProperty('createFileReference');

package/src/types/file-reference.ts

@@ -2,6 +2,7 @@
 // These types define the structure for the centralized file reference system
 
 import type { AppId } from './core';
+import type { ApiResult } from './api-result';
 
 export interface FileReference {
   id: string;
@@ -74,20 +75,44 @@ export interface FileUploadOptions {
 }
 
 
-export interface FileReferenceService {
-  createFileReference(options: FileUploadOptions, file: File): Promise<FileReference>;
-  getFileReference(table_name: string, record_id: string, organisation_id?: string): Promise<FileReference | null>;
-  getFileReferenceById(id: string, organisation_id?: string): Promise<FileReference | null>;
-  getFileUrl(table_name: string, record_id: string, organisation_id?: string): Promise<string | null>;
-  getSignedUrl(table_name: string, record_id: string, organisation_id?: string, expires_in?: number): Promise<string | null>;
-  updateFileReference(id: string, updates: Partial<FileReference>): Promise<FileReference>;
-  deleteFileReference(table_name: string, record_id: string, organisation_id?: string, delete_file?: boolean): Promise<boolean>;
-  listFileReferences(table_name: string, record_id: string, organisation_id?: string): Promise<FileReference[]>;
-  getFilesByCategory(table_name: string, record_id: string, category: FileCategory, organisation_id?: string): Promise<FileReference[]>;
-  getFileCount(table_name: string, record_id: string, organisation_id?: string): Promise<number>;
-  uploadMultipleFiles(options: FileUploadOptions, files: File[]): Promise<BulkUploadResult>;
+/** Create/read operations for FileReferenceService (ISP). */
+export interface FileReferenceServiceCreateRead {
+  createFileReference(options: FileUploadOptions, file: File): Promise<ApiResult<FileReference>>;
+  getFileReference(table_name: string, record_id: string, organisation_id?: string): Promise<ApiResult<FileReference | null>>;
+  getFileReferenceById(id: string, organisation_id?: string): Promise<ApiResult<FileReference | null>>;
 }
 
+/** URL operations for FileReferenceService (ISP). */
+export interface FileReferenceServiceUrls {
+  getFileUrl(table_name: string, record_id: string, organisation_id?: string): Promise<ApiResult<string | null>>;
+  getSignedUrl(table_name: string, record_id: string, organisation_id?: string, expires_in?: number): Promise<ApiResult<string | null>>;
+}
+
+/** Write operations for FileReferenceService (ISP). */
+export interface FileReferenceServiceWrite {
+  updateFileReference(id: string, updates: Partial<FileReference>): Promise<ApiResult<FileReference>>;
+  deleteFileReference(table_name: string, record_id: string, organisation_id?: string, delete_file?: boolean): Promise<ApiResult<boolean>>;
+}
+
+/** List/count operations for FileReferenceService (ISP). */
+export interface FileReferenceServiceList {
+  listFileReferences(table_name: string, record_id: string, organisation_id?: string): Promise<ApiResult<FileReference[]>>;
+  getFilesByCategory(table_name: string, record_id: string, category: FileCategory, organisation_id?: string): Promise<ApiResult<FileReference[]>>;
+  getFileCount(table_name: string, record_id: string, organisation_id?: string): Promise<ApiResult<number>>;
+}
+
+/** Bulk upload for FileReferenceService (ISP). */
+export interface FileReferenceServiceBulk {
+  uploadMultipleFiles(options: FileUploadOptions, files: File[]): Promise<ApiResult<BulkUploadResult>>;
+}
+
+/** FileReferenceService (ISP: composed from focused interfaces). */
+export type FileReferenceService = FileReferenceServiceCreateRead
+  & FileReferenceServiceUrls
+  & FileReferenceServiceWrite
+  & FileReferenceServiceList
+  & FileReferenceServiceBulk;
+
 export interface StorageUploadOptions {
   orgId: string;
   isPublic?: boolean;

package/src/types/{__tests__/guards.test.ts → guards.test.ts}

@@ -8,8 +8,8 @@
  */
 
 import { describe, it, expect } from 'vitest';
-import { isAuthErrorCode, isUser, isSession } from '../guards';
-import { AuthErrorCode } from '../core';
+import { isAuthErrorCode, isUser, isSession } from './guards';
+import { AuthErrorCode } from './core';
 
 describe('Type Guards', () => {
   describe('isAuthErrorCode', () => {

package/src/types/{__tests__/index.test.ts → index.test.ts}

@@ -1,7 +1,7 @@
 /**
  * @file Types Index Barrel File Tests
  * @package @jmruthers/pace-core
- * @module Types/__tests__
+ * @module Types
  * @since 1.0.0
  *
  * Comprehensive tests for the types barrel file (src/types/index.ts) ensuring
@@ -9,7 +9,7 @@
  */
 
 import { describe, it, expect } from 'vitest';
-import * as TypeExports from '../index';
+import * as TypeExports from './index';
 
 // Following testing standards: use timeout parameter to prevent hanging
 const TEST_TIMEOUT = 10000; // 10 seconds per test

package/src/types/index.ts

@@ -18,6 +18,9 @@
 // Core types (branded types, errors, primitives)
 export * from './core';
 
+// API result shape (standard for async APIs)
+export * from './api-result';
+
 // Domain-specific types
 export * from './auth';
 export * from './event';

package/src/types/{__tests__/organisation.roles.test.ts → organisation.roles.test.ts}

@@ -1,5 +1,5 @@
 import { describe, expect, it } from 'vitest';
-import { ORGANISATION_ROLE_PERMISSIONS, type OrganisationPermission } from '../organisation';
+import { ORGANISATION_ROLE_PERMISSIONS, type OrganisationPermission } from './organisation';
 
 const EXPECTED_ROLE_ORDER = ['supporter', 'member', 'leader', 'org_admin'] as const;
 const ALLOWED_PERMISSIONS: OrganisationPermission[] = [

package/src/types/{__tests__/organisation.test.ts → organisation.test.ts}

@@ -1,7 +1,7 @@
 /**
  * @file Organisation Type Tests
  * @package @jmruthers/pace-core
- * @module Types/__tests__
+ * @module Types
  * @since 1.0.0
  */
 
@@ -15,7 +15,6 @@ import type {
   OrganisationInvitation,
   OrganisationMembership,
   OrganisationContext,
-  OrganisationProviderProps,
   OrganisationHookReturn,
   OrganisationEvent,
   OrganisationTheme,
@@ -468,35 +467,8 @@ describe('[types] Organisation Context Types', () => {
     });
   });
 
-  describe('OrganisationProviderProps interface', () => {
-    it('validates OrganisationProviderProps type structure', () => {
-      const props: OrganisationProviderProps = {
-        children: 'Test children',
-        organisationId: 'org-123',
-        userId: 'user-123',
-        autoRefresh: true,
-        refreshInterval: 30000,
-        onOrganisationChange: () => {},
-        onError: () => {}
-      };
-
-      expect(props).toHaveProperty('children');
-      expect(props).toHaveProperty('organisationId');
-      expect(props).toHaveProperty('userId');
-      expect(props).toHaveProperty('autoRefresh');
-      expect(props).toHaveProperty('refreshInterval');
-      expect(props).toHaveProperty('onOrganisationChange');
-      expect(props).toHaveProperty('onError');
-
-      expect(typeof props.children).toBe('string');
-      expect(typeof props.organisationId).toBe('string');
-      expect(typeof props.userId).toBe('string');
-      expect(typeof props.autoRefresh).toBe('boolean');
-      expect(typeof props.refreshInterval).toBe('number');
-      expect(typeof props.onOrganisationChange).toBe('function');
-      expect(typeof props.onError).toBe('function');
-    });
-  });
+  // OrganisationProviderProps removed - OrganisationProvider was removed
+  // Test removed as OrganisationProvider is no longer part of the codebase
 
   describe('OrganisationHookReturn interface', () => {
     it('validates OrganisationHookReturn type structure', () => {

package/src/types/organisation.ts

@@ -105,33 +105,31 @@ export interface OrganisationSecurityError extends Error {
   userId?: string;
 }
 
-export interface OrganisationContextType {
-  // Current organisation context
+/** Organisation context state (ISP). */
+export interface OrganisationContextState {
   selectedOrganisation: Organisation | null;
   organisations: Organisation[];
   userMemberships: OrganisationMembership[];
-  
-  // Loading and error states
   isLoading: boolean;
   error: Error | null;
   hasValidOrganisationContext: boolean;
   isContextReady: boolean;
-  
-  // Organisation management
+}
+
+/** Organisation context methods (ISP). */
+export interface OrganisationContextMethods {
   setSelectedOrganisation: (org: Organisation | null) => void;
   switchOrganisation: (orgId: string) => Promise<void>;
-  
-  // Security helpers
   getUserRole: (orgId?: string) => string;
   validateOrganisationAccess: (orgId: string) => boolean;
   ensureOrganisationContext: () => Organisation;
-  
-  // Data management
   refreshOrganisations: () => Promise<void>;
   getPrimaryOrganisation: () => Organisation | null;
   isOrganisationSecure: () => boolean;
-  
-  // Enhanced security features
+}
+
+/** Enhanced organisation context features (ISP). */
+export interface OrganisationContextEnhanced {
   getOrganisationHierarchy?: (orgId: string) => Promise<OrganisationHierarchy[]>;
   canAccessChildOrganisations?: (orgId: string) => boolean;
   validateUserAccess?: (userId: string, orgId: string) => Promise<boolean>;
@@ -139,9 +137,11 @@ export interface OrganisationContextType {
   logOrganisationAccess?: (action: string, details?: unknown) => Promise<void>;
 }
 
-export interface OrganisationProviderProps {
-  children: React.ReactNode;
-}
+/** Organisation context type (ISP: composed from focused interfaces). */
+export type OrganisationContextType = OrganisationContextState
+  & OrganisationContextMethods
+  & OrganisationContextEnhanced;
+
 
 export interface OrganisationHierarchy {
   organisation: Organisation;

package/src/types/supabase.ts

@@ -39,19 +39,28 @@ export interface QueryResult<T = unknown> {
 // AUTH OPERATIONS
 // ============================================================================
 
-/**
- * Authentication operations interface
- */
-export interface AuthOperations {
+/** Sign-in/sign-up auth operations (ISP). */
+export interface AuthSignInSignUpOperations {
   signIn(email: string, password: string): Promise<{ data: { user: User; session: Session } | null; error: AuthError | null }>;
   signUp(email: string, password: string): Promise<{ data: { user: User } | null; error: AuthError | null }>;
+}
+
+/** Session auth operations (ISP). */
+export interface AuthSessionOperations {
   signOut(): Promise<{ error: AuthError | null }>;
   getSession(): Promise<{ data: { session: Session } | null; error: AuthError | null }>;
   refreshSession(): Promise<{ data: { session: Session } | null; error: AuthError | null }>;
+}
+
+/** Password/user update auth operations (ISP). */
+export interface AuthPasswordOperations {
   resetPassword(email: string): Promise<{ data: unknown; error: AuthError | null }>;
   updateUser(attributes: Partial<User>): Promise<{ data: { user: User } | null; error: AuthError | null }>;
 }
 
+/** Authentication operations (ISP: composed from focused interfaces). */
+export type AuthOperations = AuthSignInSignUpOperations & AuthSessionOperations & AuthPasswordOperations;
+
 // ============================================================================
 // PERMISSION OPERATIONS
 // ============================================================================

package/src/types/{__tests__/theme.test.ts → theme.test.ts}

@@ -1,7 +1,7 @@
 /**
  * @file Theme Type Tests
  * @package @jmruthers/pace-core
- * @module Types/__tests__
+ * @module Types
  * @since 1.0.0
  */
 

package/src/types/{__tests__/type-validation.test.ts → type-validation.test.ts}

@@ -1,7 +1,7 @@
 /**
  * @file Type Validation Tests
  * @package @jmruthers/pace-core
- * @module Types/__tests__
+ * @module Types
  * @since 1.0.0
  */
 

package/src/types/{__tests__/validation.test.ts → validation.test.ts}

@@ -1,7 +1,7 @@
 /**
  * @file Validation Schemas Tests
  * @package @jmruthers/pace-core
- * @module Types/Validation/__tests__
+ * @module Types/Validation
  * @since 0.1.0
  * 
  * Comprehensive tests for validation schemas and utility functions.
@@ -27,7 +27,7 @@ import {
   contactFormSchema,
   pickSchema,
   combineSchemas,
-} from '../validation';
+} from './validation';
 
 describe('[types] Validation Schemas', () => {
   describe('emailSchema', () => {