@envive-ai/react-hooks 0.3.17 → 0.3.19

package/src/contexts/userIdentityContext/__tests__/userIdentityContext.test.tsx

@@ -8,9 +8,9 @@ import { UserIdentityService } from 'src/services/userIdentityService';
 import { UserIdentityProvider, useUserIdentity } from '../userIdentityContext';
 
 // Mock the Logger to avoid console output in tests
-vi.spyOn(Logger, 'logInfo').mockImplementation(() => {});
-vi.spyOn(Logger, 'logWarn').mockImplementation(() => {});
-vi.spyOn(Logger, 'logError').mockImplementation(() => {});
+vi.spyOn(Logger.prototype, 'logInfo').mockImplementation(() => {});
+vi.spyOn(Logger.prototype, 'logWarn').mockImplementation(() => {});
+vi.spyOn(Logger.prototype, 'logError').mockImplementation(() => {});
 
 // Mock uuid to have predictable values in tests
 vi.mock('uuid', () => ({
@@ -262,7 +262,7 @@ describe('UserIdentityProvider', () => {
     });
 
     it('should log info when setting userId override', async () => {
-      const logSpy = vi.spyOn(Logger, 'logInfo');
+      const logSpy = vi.spyOn(Logger.prototype, 'logInfo');
 
       const ComponentWithSet: React.FC = () => {
         const context = useUserIdentity();
@@ -283,7 +283,7 @@ describe('UserIdentityProvider', () => {
     });
 
     it('should log info when clearing userId override', async () => {
-      const logSpy = vi.spyOn(Logger, 'logInfo');
+      const logSpy = vi.spyOn(Logger.prototype, 'logInfo');
 
       const ComponentWithClear: React.FC = () => {
         const context = useUserIdentity();

package/src/contexts/userIdentityContext/userIdentityContext.tsx

@@ -13,6 +13,8 @@ import { useSetAtom } from 'jotai';
 import { userIdAtom } from 'src/atoms/app';
 import { UserIdentityService } from 'src/services/userIdentityService';
 
+const logger = new Logger('userIdentityContext');
+
 export interface UserIdentityContextType {
   getUserIdOrDefault: () => Promise<string>;
   getUserIdOverrideFromLocalStorage: () => string | undefined;
@@ -53,7 +55,7 @@ export const UserIdentityProvider: React.FC<UserIdentityProviderProps> = ({
 
   const setUserIdOverrideInLocalStorage = useCallback(
     (userId: string): string => {
-      Logger.logInfo(`setUserIdOverrideInLocalStorage - Setting user_id=${userId}`);
+      logger.logInfo(`setUserIdOverrideInLocalStorage - Setting user_id=${userId}`);
       setItem(USER_ID_OVERRIDE_KEY, userId);
       // window.dispatchEvent is handled by useLocalStorage now
       return userId;
@@ -62,7 +64,7 @@ export const UserIdentityProvider: React.FC<UserIdentityProviderProps> = ({
   );
 
   const clearUserIdOverrideInLocalStorage = useCallback(() => {
-    Logger.logInfo(`clearUserIdOverrideInLocalStorage - Clearing user_id`);
+    logger.logInfo(`clearUserIdOverrideInLocalStorage - Clearing user_id`);
     // LocalStorageService.getLocalStorage()?.removeItem(USER_ID_OVERRIDE_KEY);
     // window.dispatchEvent is handled by useLocalStorage now
     setItem(USER_ID_OVERRIDE_KEY, ''); // Set to empty string to clear
@@ -80,15 +82,15 @@ export const UserIdentityProvider: React.FC<UserIdentityProviderProps> = ({
 
       if (!enviveUserId) {
         throw new Error(
-          '[spiffy-ai] EnviveAnalytics.resolveEnviveUserId() returned null/undefined - this should never happen',
+          'EnviveAnalytics.resolveEnviveUserId() returned null/undefined - this should never happen',
         );
       }
 
       return enviveUserId;
     } catch (error: unknown) {
-      Logger.logError('[spiffy-ai] CRITICAL: Failed to get user ID from EnviveAnalytics', error);
+      logger.logError('CRITICAL: Failed to get user ID from EnviveAnalytics', error);
       const errorMessage = error instanceof Error ? error.message : 'Unknown error';
-      throw new Error(`[spiffy-ai] CRITICAL: User ID resolution failed: ${errorMessage}`);
+      throw new Error(`CRITICAL: User ID resolution failed: ${errorMessage}`);
     }
   }, [getUserIdOverrideFromLocalStorage]);
 
@@ -112,7 +114,6 @@ export const UserIdentityProvider: React.FC<UserIdentityProviderProps> = ({
     if (isReady && !localUserId) {
       const setUserIdStates = async () => {
         const newUserId = await getUserIdOrDefault();
-        console.log('useUserIdentity useEffect - setting localUserId', newUserId);
         setLocalUserId(newUserId);
         setUserId(newUserId);
       };

package/src/contexts/widgetConfigContext/__tests__/widgetConfigContext.test.tsx

@@ -7,9 +7,9 @@ import { WidgetConfigProvider, useWidgetConfig } from '../widgetConfigContext';
 import { WidgetConfigV3, WidgetTypeV3 } from '../../typesV3';
 
 // Mock the Logger to avoid console output in tests
-vi.spyOn(Logger, 'logInfo').mockImplementation(() => {});
-vi.spyOn(Logger, 'logWarn').mockImplementation(() => {});
-vi.spyOn(Logger, 'logError').mockImplementation(() => {});
+vi.spyOn(Logger.prototype, 'logInfo').mockImplementation(() => {});
+vi.spyOn(Logger.prototype, 'logWarn').mockImplementation(() => {});
+vi.spyOn(Logger.prototype, 'logError').mockImplementation(() => {});
 
 // Mock the useNewOrgConfig hook
 vi.mock('src/hooks/NewOrgConfig', () => ({
@@ -394,7 +394,7 @@ describe('WidgetConfigProvider', () => {
 
   describe('Logger Integration', () => {
     it('should log info when retrieving widget config', async () => {
-      const logInfoSpy = vi.spyOn(Logger, 'logInfo');
+      const logDebugSpy = vi.spyOn(Logger.prototype, 'logDebug');
       const mockWidgetConfig: WidgetConfigV3 = {
         widgetConfigId: 'test-config-id',
         type: WidgetTypeV3.PromptCarouselV3,
@@ -425,12 +425,12 @@ describe('WidgetConfigProvider', () => {
       });
 
       await waitFor(() => {
-        expect(logInfoSpy).toHaveBeenCalledWith('widgetConfigContext | retrieving widget config', {
+        expect(logDebugSpy).toHaveBeenCalledWith('retrieving widget config', {
           widgetConfigId: 'test-config-id',
           widgetType: WidgetTypeV3.PromptCarouselV3,
         });
-        expect(logInfoSpy).toHaveBeenCalledWith(
-          'widgetConfigContext | returning widget config response',
+        expect(logDebugSpy).toHaveBeenCalledWith(
+          'returning widget config response',
           mockWidgetConfig,
         );
       });

package/src/contexts/widgetConfigContext/widgetConfigContext.tsx

@@ -3,6 +3,8 @@ import Logger from 'src/application/logging/logger';
 import { useNewOrgConfig } from 'src/hooks/NewOrgConfig';
 import { WidgetConfigV3, WidgetTypeV3 } from '../typesV3';
 
+const logger = new Logger('widgetConfigContext');
+
 interface WidgetConfigRequest {
   widgetType: WidgetTypeV3;
   widgetConfigId: string;
@@ -22,7 +24,7 @@ export const WidgetConfigProvider: React.FC<{ children: ReactNode }> = ({ childr
   const getWidgetConfig = useCallback(
     async (request: WidgetConfigRequest): Promise<WidgetConfigV3> => {
       setIsLoadingWidgetConfig(true);
-      Logger.logInfo('widgetConfigContext | retrieving widget config', {
+      logger.logDebug('retrieving widget config', {
         widgetConfigId: request.widgetConfigId,
         widgetType: request.widgetType,
       });
@@ -33,7 +35,7 @@ export const WidgetConfigProvider: React.FC<{ children: ReactNode }> = ({ childr
 
       const { widgetConfigId, widgetType } = request;
       const response = frontendConfig?.widgetConfigs?.[widgetConfigId];
-      Logger.logInfo('widgetConfigContext | returning widget config response', response);
+      logger.logDebug('returning widget config response', response);
 
       if (widgetConfigId === 'fake-widget-config-id') {
         return {} as WidgetConfigV3;

package/src/hooks/CustomerSupportHandoff/useCustomerSupportHandoff.ts

@@ -1,6 +1,8 @@
 import Logger from 'src/application/logging/logger';
 import { useCallback } from 'react';
 
+const logger = new Logger('useCustomerSupportHandoff');
+
 /**
  * Hook to call the `click` method of the merchant's customer support chat widget.
  *
@@ -17,14 +19,14 @@ export const useCustomerSupportHandoff = (onSwitchToAgent?: () => void) => {
     const kustomerElement = document.getElementById('kustomer-ui-sdk-iframe');
 
     if (kustomerElement == null || !(kustomerElement instanceof HTMLIFrameElement)) {
-      Logger.logError('[spiffy-ai] Kustomer iFrame element not found', undefined);
+      logger.logError('Kustomer iFrame element not found', undefined);
       return;
     }
 
     const kustomerButton = kustomerElement.contentWindow?.document?.getElementById('rootChatIcon');
 
     if (kustomerButton == null) {
-      Logger.logError('[spiffy-ai] Kustomer button not found', undefined);
+      logger.logError('Kustomer button not found', undefined);
       return;
     }
 

package/src/hooks/Search/__tests__/useSearch.test.tsx

@@ -135,14 +135,20 @@ vi.mock('@spiffy-ai/commerce-api-client', () => ({
 }));
 
 // Mock Logger
-vi.mock('src/application/logging/logger', () => ({
-  default: {
-    logInfo: vi.fn(),
-    logWarn: vi.fn(),
-    logError: vi.fn(),
-    logDebug: vi.fn(),
-  },
-}));
+vi.mock('src/application/logging/logger', () => {
+  const MockLogger = vi.fn(function MockLogger() {
+    return {
+      logInfo: vi.fn(),
+      logWarn: vi.fn(),
+      logError: vi.fn(),
+      logDebug: vi.fn(),
+    };
+  });
+
+  return {
+    default: MockLogger,
+  };
+});
 
 // Mock search service adapter
 vi.mock('src/atoms/search/searchServiceAdapter', () => ({

package/src/hooks/WidgetInteraction/useWidgetInteraction.ts

@@ -6,12 +6,15 @@ import { v4 as uuid } from 'uuid';
 import { InternalWidgetInteraction, WidgetInteraction } from './types';
 import { extractPageContext } from './utils';
 
+const getInteractionId = () => uuid();
+
 export const useWidgetInteraction = () => {
   const { trackEvent } = useAmplitude();
   const variantInfo = useAtomValue(pageVariantInfoAtom);
 
   const trackWidgetInteraction = (props: WidgetInteraction) => {
     if (props.trigger) {
+      // eslint-disable-next-line no-param-reassign
       props.trigger.interaction_id = props.trigger.interaction_id || getInteractionId();
     }
 
@@ -26,8 +29,6 @@ export const useWidgetInteraction = () => {
     });
   };
 
-  const getInteractionId = () => uuid();
-
   return {
     trackWidgetInteraction,
     getInteractionId,

package/src/hooks/WidgetInteraction/utils.ts

@@ -3,9 +3,9 @@ import {
   FullPageSalesAgentVariantInfo,
   HomeVariantInfo,
   OtherVariantInfo,
-  PageVariantInfo,
   PDPPageVariantInfo,
   PLPPageVariantInfo,
+  PageVariantInfo,
 } from '../../contexts/pageContext/types';
 import { WidgetInteractionContext, WidgetInteractionPageType } from './types';
 
@@ -38,6 +38,8 @@ export const extractPageContext = (
         page_id: (pageVariant as FullPageSalesAgentVariantInfo).url,
         page_type: WidgetInteractionPageType.FULL_PAGE,
       };
+    default:
+      return null;
   }
 
   return null;

package/src/services/amplitudeService/__tests__/amplitudeService.test.ts

@@ -281,7 +281,7 @@ describe('AmplitudeService', () => {
         featureFlagService,
       });
 
-      const logWarnSpy = vi.spyOn(Logger, 'logWarn');
+      const logWarnSpy = vi.spyOn(Logger.prototype, 'logWarn');
 
       await service.trackEvent({
         eventName: SpiffyMetricsEventName.ChatUserMessageInput,
@@ -304,7 +304,7 @@ describe('AmplitudeService', () => {
         throw error;
       });
 
-      const logErrorSpy = vi.spyOn(Logger, 'logError');
+      const logErrorSpy = vi.spyOn(Logger.prototype, 'logError');
 
       await service.trackEvent({
         eventName: SpiffyMetricsEventName.ChatUserMessageInput,
@@ -312,7 +312,7 @@ describe('AmplitudeService', () => {
       });
 
       expect(logErrorSpy).toHaveBeenCalledWith(
-        '[spiffy-ai] Error tracking event',
+        'Error tracking event',
         error,
         expect.objectContaining({
           eventName: SpiffyMetricsEventName.ChatUserMessageInput,

package/src/services/amplitudeService/amplitudeService.ts

@@ -17,6 +17,8 @@ import { EnviveMetricsEventName, SpiffyMetricsEventName } from './eventNames';
 // Re-export event names for convenience
 export { EnviveMetricsEventName, SpiffyMetricsEventName };
 
+const logger = new Logger('amplitudeService');
+
 export interface TrackEventParams {
   eventName: SpiffyMetricsEventName | EnviveMetricsEventName;
   eventProps?: Record<string, unknown>;
@@ -217,7 +219,7 @@ export class AmplitudeService {
     );
 
     if (!isReady) {
-      Logger.logDebug('AmplitudeService is not ready', {
+      logger.logDebug('AmplitudeService is not ready', {
         isReady,
         userId: this.config.userId,
         hasFeatureFlagService: !!this.config.featureFlagService,
@@ -351,12 +353,12 @@ export class AmplitudeService {
     eventGroups,
     alsoSendToGoogleAnalytics = false,
   }: TrackEventParams): Promise<void> {
-    Logger.logDebug('Submitting event', eventName);
+    logger.logDebug('Submitting event', eventName);
     try {
       const decoratedEventName = AmplitudeService.decorateEventName(eventName);
 
       if (!this.amplitudeClient) {
-        Logger.logWarn('amplitude client undefined', undefined, {
+        logger.logWarn('amplitude client undefined', undefined, {
           event_name: decoratedEventName,
         });
         return;
@@ -377,7 +379,7 @@ export class AmplitudeService {
       const hashArray = Array.from(new Uint8Array(hashBuffer));
       const currentInsertId = hashArray.map(b => b.toString(16).padStart(2, '0')).join('');
 
-      Logger.logDebug(`amplitude tracking ${decoratedEventName}`, null, {
+      logger.logDebug(`amplitude tracking ${decoratedEventName}`, null, {
         event_name: decoratedEventName,
         props: mappedEventProps,
       });
@@ -404,7 +406,7 @@ export class AmplitudeService {
         };
         projectToGA4(eventName as EnviveMetricsEventName, mergedPropsForGA4);
       } else if (alsoSendToGoogleAnalytics && this.config.orgGaConfig) {
-        Logger.logDebug('[spiffy-ai] GA tracking', decoratedEventName);
+        logger.logDebug('GA tracking', decoratedEventName);
         if (typeof window !== 'undefined' && window.dataLayer) {
           // eslint-disable-next-line @typescript-eslint/no-explicit-any
           (window.dataLayer as any[]).push({
@@ -414,7 +416,7 @@ export class AmplitudeService {
         }
       }
     } catch (err) {
-      Logger.logError('[spiffy-ai] Error tracking event', err, {
+      logger.logError('Error tracking event', err, {
         eventName,
         eventProps,
       });

package/src/services/ga4ProjectionService/ga4ProjectionService.ts

@@ -2,6 +2,8 @@ import Logger from 'src/application/logging/logger';
 import { EnviveMetricsEventName } from '../amplitudeService/eventNames';
 import { GA4ProjectedEventConfig, GA4_EVENT_SCHEMA } from './ga4EventSchema';
 
+const logger = new Logger('ga4ProjectionService');
+
 const filterToSchema = (
   eventProps: Record<string, unknown>,
   allowedFields: readonly string[],
@@ -130,7 +132,7 @@ export const projectToGA4 = (
       ...truncatedParams,
     });
   } catch (err) {
-    Logger.logError('[envive-ai] Error projecting event to GA4', err, {
+    logger.logError('Error projecting event to GA4', err, {
       eventName,
     });
   }

package/src/services/userIdentityService/userIdentityService.ts

@@ -4,6 +4,8 @@ import { getAtomStore } from 'src/atoms/atomStore';
 import { userIdAtom } from 'src/atoms/app';
 import { ExecutionContext } from '@envive-ai/analytics/dist/analytics/EnviveAnalytics';
 
+const logger = new Logger('userIdentityService');
+
 type UserIdentityServiceProps = {
   apiKey: string;
 };
@@ -18,7 +20,7 @@ export class UserIdentityService {
   }
 
   getUserIdOverrideFromLocalStorage(): string | undefined {
-    Logger.logDebug('[spiffy-ai] Getting userID from localStorage');
+    logger.logDebug('Getting userID from localStorage');
     return localStorage.getItem(this.USER_ID_OVERRIDE_KEY) ?? undefined;
   }
 
@@ -29,7 +31,7 @@ export class UserIdentityService {
     }
 
     try {
-      Logger.logDebug('Getting userID via EnviveAnalytics in UserIdentityService');
+      logger.logDebug('Getting userID via EnviveAnalytics in UserIdentityService');
       const enviveAnalytics = new EnviveAnalytics({
         apiKey: this.apiKey,
         environmentInfo: { executionContext: ExecutionContext.EnviveReactComponentsV3 },
@@ -37,16 +39,14 @@ export class UserIdentityService {
       const enviveUserId = await enviveAnalytics.resolveEnviveUserId();
 
       if (!enviveUserId) {
-        throw new Error(
-          '[spiffy-ai] EnviveAnalytics.resolveEnviveUserId() returned null/undefined',
-        );
+        throw new Error('EnviveAnalytics.resolveEnviveUserId() returned null/undefined');
       }
 
       return enviveUserId;
     } catch (error: unknown) {
-      Logger.logError('[spiffy-ai] CRITICAL: Failed to get user ID:', error);
+      logger.logError('CRITICAL: Failed to get user ID:', error);
       const errorMessage = error instanceof Error ? error.message : 'Unknown error';
-      throw new Error(`[spiffy-ai] CRITICAL: User ID resolution failed: ${errorMessage}`);
+      throw new Error(`CRITICAL: User ID resolution failed: ${errorMessage}`);
     }
   }
 
@@ -57,7 +57,7 @@ export class UserIdentityService {
       const store = atomStore || getAtomStore();
       store.set(userIdAtom, userId);
     } catch (error) {
-      Logger.logError('[spiffy-ai] CRITICAL: Failed to initialize user ID:', error);
+      logger.logError('CRITICAL: Failed to initialize user ID:', error);
       throw error;
     }
   }