@jmruthers/pace-core 0.5.125 → 0.5.127

package/src/hooks/useSecureDataAccess.ts

@@ -199,12 +199,6 @@ export function useSecureDataAccess(): SecureDataAccessReturn {
   ): Promise<T[]> => {
     validateContext();
     const organisationId = getCurrentOrganisationId();
-    
-    console.log('[useSecureDataAccess] Executing secure query:', {
-      table,
-      organisationId,
-      filters
-    });
 
     // Set organisation context in database session
     await setOrganisationContextInSession(organisationId);
@@ -273,11 +267,6 @@ export function useSecureDataAccess(): SecureDataAccessReturn {
       throw error;
     }
 
-    console.log('[useSecureDataAccess] Query successful:', {
-      table,
-      resultCount: data?.length || 0
-    });
-
     // NEW: Phase 1 - Record successful data access attempt
     recordDataAccess(table, 'read', true, `SELECT ${columns} FROM ${table}`, filters);
 
@@ -290,11 +279,6 @@ export function useSecureDataAccess(): SecureDataAccessReturn {
   ): Promise<T> => {
     validateContext();
     const organisationId = getCurrentOrganisationId();
-    
-    console.log('[useSecureDataAccess] Executing secure insert:', {
-      table,
-      organisationId
-    });
 
     // Set organisation context in database session
     await setOrganisationContextInSession(organisationId);
@@ -316,11 +300,6 @@ export function useSecureDataAccess(): SecureDataAccessReturn {
       throw error;
     }
 
-    console.log('[useSecureDataAccess] Insert successful:', {
-      table,
-      id: insertData?.id
-    });
-
     return insertData as T;
   }, [validateContext, getCurrentOrganisationId, setOrganisationContextInSession, supabase]);
 
@@ -331,12 +310,6 @@ export function useSecureDataAccess(): SecureDataAccessReturn {
   ): Promise<T[]> => {
     validateContext();
     const organisationId = getCurrentOrganisationId();
-    
-    console.log('[useSecureDataAccess] Executing secure update:', {
-      table,
-      organisationId,
-      filters
-    });
 
     // Set organisation context in database session
     await setOrganisationContextInSession(organisationId);
@@ -377,11 +350,6 @@ export function useSecureDataAccess(): SecureDataAccessReturn {
       throw error;
     }
 
-    console.log('[useSecureDataAccess] Update successful:', {
-      table,
-      updatedCount: updateData?.length || 0
-    });
-
     return (updateData as T[]) || [];
   }, [validateContext, getCurrentOrganisationId, setOrganisationContextInSession, supabase]);
 
@@ -391,12 +359,6 @@ export function useSecureDataAccess(): SecureDataAccessReturn {
   ): Promise<void> => {
     validateContext();
     const organisationId = getCurrentOrganisationId();
-    
-    console.log('[useSecureDataAccess] Executing secure delete:', {
-      table,
-      organisationId,
-      filters
-    });
 
     // Set organisation context in database session
     await setOrganisationContextInSession(organisationId);
@@ -443,10 +405,6 @@ export function useSecureDataAccess(): SecureDataAccessReturn {
       console.error('[useSecureDataAccess] Delete failed:', error);
       throw error;
     }
-
-    console.log('[useSecureDataAccess] Delete successful:', {
-      table
-    });
   }, [validateContext, getCurrentOrganisationId, setOrganisationContextInSession, supabase]);
 
   const secureRpc = useCallback(async <T = any>(
@@ -455,11 +413,6 @@ export function useSecureDataAccess(): SecureDataAccessReturn {
   ): Promise<T> => {
     validateContext();
     const organisationId = getCurrentOrganisationId();
-    
-    console.log('[useSecureDataAccess] Executing secure RPC:', {
-      functionName,
-      organisationId
-    });
 
     // Set organisation context in database session
     await setOrganisationContextInSession(organisationId);
@@ -543,18 +496,6 @@ export function useSecureDataAccess(): SecureDataAccessReturn {
       secureParams.p_event_id = selectedEvent.event_id;
     }
 
-    // Debug logging for items list to help diagnose issues
-    if (functionName === 'data_cake_items_list') {
-      console.log('[useSecureDataAccess] Calling data_cake_items_list with params:', {
-        p_user_id: secureParams.p_user_id,
-        p_organisation_id: secureParams.organisation_id || secureParams.p_organisation_id,
-        p_event_id: secureParams.p_event_id,
-        hasEvent: !!selectedEvent?.event_id,
-        eventId: selectedEvent?.event_id,
-        allParams: secureParams
-      });
-    }
-
     const { data, error } = await supabase!.rpc(functionName, secureParams);
 
     if (error) {
@@ -562,10 +503,6 @@ export function useSecureDataAccess(): SecureDataAccessReturn {
       throw error;
     }
 
-    console.log('[useSecureDataAccess] RPC successful:', {
-      functionName
-    });
-
     return data as T;
   }, [validateContext, getCurrentOrganisationId, setOrganisationContextInSession, supabase, selectedEvent?.event_id, user?.id]);
 

package/src/services/EventService.ts

@@ -133,15 +133,6 @@ export class EventService extends BaseService implements IEventService {
     if (event) {
       // SECURITY: Validate event belongs to current organisation
       try {
-        console.log('[EventService] Event selection validation:', {
-          eventId: event.event_id,
-          eventName: event.event_name,
-          eventOrganisationId: event.organisation_id,
-          selectedOrganisationId: this.selectedOrganisation?.id,
-          selectedOrganisationName: this.selectedOrganisation?.display_name,
-          match: event.organisation_id === this.selectedOrganisation?.id
-        });
-        
         if (this.selectedOrganisation && event.organisation_id !== this.selectedOrganisation.id) {
           console.error('[EventService] Event organisation_id does not match selected organisation', {
             eventOrganisationId: event.organisation_id,
@@ -355,12 +346,6 @@ export class EventService extends BaseService implements IEventService {
 
       if (isMounted) {
         const eventsData = data || [];
-        console.log('[EventService] Loaded events:', eventsData.map((event: any) => ({
-          eventId: event.event_id,
-          eventName: event.event_name,
-          organisationId: event.organisation_id,
-          selectedOrganisationId: this.selectedOrganisation?.id
-        })));
 
         // Transform the data to match our Event interface
         const transformedEvents: Event[] = eventsData.map((event: any) => ({
@@ -381,10 +366,6 @@ export class EventService extends BaseService implements IEventService {
         }));
 
         this.events = transformedEvents;
-        console.log('[EventService] Set events in service:', {
-          count: transformedEvents.length,
-          events: transformedEvents.map(e => ({ id: e.event_id, name: e.event_name }))
-        });
         this.error = null;
 
         // Reset auto-selection ref for new events