@umituz/web-traffic 1.0.10 → 1.0.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@umituz/web-traffic",
-  "version": "1.0.10",
+  "version": "1.0.12",
   "description": "Web analytics tracking library. Event tracking, pageviews, sessions, device info, and UTM parameter support.",
   "main": "./src/index.ts",
   "types": "./src/index.ts",

package/src/domains/affiliate/value-objects/site-id.vo.ts

@@ -27,7 +27,10 @@ export class SiteId {
   }
 
   static generate(): SiteId {
-    const id = `site-${Date.now()}-${Math.random().toString(36).substring(2, 11)}`;
+    const uniqueId = typeof crypto !== 'undefined' && crypto.randomUUID
+      ? crypto.randomUUID()
+      : `${Date.now()}-${Math.random().toString(36).substring(2, 11)}`;
+    const id = `site-${uniqueId}`;
     return new SiteId(id);
   }
 }

package/src/domains/tracking/application/tracking-command.service.ts

@@ -10,8 +10,6 @@ import { Session } from '../../tracking/aggregates/session.aggregate';
 import { Event } from '../../tracking/entities/event.entity';
 import { Pageview } from '../../tracking/entities/pageview.entity';
 import type { ISessionRepository, IEventRepository, IPageviewRepository } from '../../tracking/repositories/event.repository.interface';
-import type { EventTracked } from '../events/event-tracked.domain-event';
-import type { PageviewTracked } from '../events/pageview-tracked.domain-event';
 
 export interface TrackingCommandResult {
   success: boolean;

package/src/domains/tracking/repositories/event.repository.interface.ts

@@ -4,11 +4,12 @@
  */
 
 import type { Event } from '../entities/event.entity';
+import type { Pageview } from '../entities/pageview.entity';
 import type { EventId } from '../value-objects/event-id.vo';
 import type { SessionId } from '../value-objects/session-id.vo';
 
 export interface IEventRepository {
-  save(event: Event): Promise<void>;
+  save(event: Event | Pageview): Promise<void>;
   findById(id: EventId): Promise<Event | null>;
   findBySessionId(sessionId: SessionId): Promise<Event[]>;
   delete(id: EventId): Promise<void>;

package/src/domains/tracking/value-objects/device-info.vo.ts

@@ -148,9 +148,12 @@ function parseOS(userAgent: string): OSInfo {
 }
 
 function detectDeviceType(userAgent: string, screenWidth?: number): DeviceType {
+  const MOBILE_BREAKPOINT = 768;
+  const TABLET_BREAKPOINT = 1024;
+
   if (screenWidth) {
-    if (screenWidth < 768) return 'mobile';
-    if (screenWidth < 1024) return 'tablet';
+    if (screenWidth < MOBILE_BREAKPOINT) return 'mobile';
+    if (screenWidth < TABLET_BREAKPOINT) return 'tablet';
     return 'desktop';
   }
 

package/src/domains/tracking/value-objects/event-id.vo.ts

@@ -30,7 +30,10 @@ export class EventId {
   }
 
   static generate(): EventId {
-    const id = `event-${Date.now()}-${Math.random().toString(36).substring(2, 11)}`;
+    const uniqueId = typeof crypto !== 'undefined' && crypto.randomUUID
+      ? crypto.randomUUID()
+      : `${Date.now()}-${Math.random().toString(36).substring(2, 11)}`;
+    const id = `event-${uniqueId}`;
     return new EventId(id);
   }
 }

package/src/domains/tracking/value-objects/session-id.vo.ts

@@ -30,7 +30,10 @@ export class SessionId {
   }
 
   static generate(): SessionId {
-    const id = `session-${Date.now()}-${Math.random().toString(36).substring(2, 11)}`;
+    const uniqueId = typeof crypto !== 'undefined' && crypto.randomUUID
+      ? crypto.randomUUID()
+      : `${Date.now()}-${Math.random().toString(36).substring(2, 11)}`;
+    const id = `session-${uniqueId}`;
     return new SessionId(id);
   }
 }

package/src/infrastructure/analytics/http-analytics.repository.impl.ts

@@ -32,13 +32,11 @@ export class HTTPAnalyticsRepository implements IAnalyticsRepository {
       });
 
       if (!response.ok) {
-        console.error('Analytics fetch failed:', response.statusText);
         return null;
       }
 
       return await response.json();
-    } catch (error) {
-      console.error('Analytics error:', error);
+    } catch {
       return null;
     }
   }

package/src/infrastructure/repositories/http-event.repository.impl.ts

@@ -24,17 +24,23 @@ export class HTTPEventRepository implements IEventRepository {
   private flushTimer: ReturnType<typeof setInterval> | null = null;
   private readonly FLUSH_INTERVAL = 30000; // 30 seconds
   private readonly MAX_QUEUE_SIZE = 100;
+  private readonly FLUSH_THRESHOLD = 10; // Flush when queue reaches this size
+  private beforeUnloadHandler: (() => void) | null = null;
+  private isFlushing = false;
 
   constructor(private readonly config: HTTPRepositoryConfig) {
     this.startFlushTimer();
     if (typeof window !== 'undefined') {
-      window.addEventListener('beforeunload', () => this.flush());
+      this.beforeUnloadHandler = () => {
+        void this.flush();
+      };
+      window.addEventListener('beforeunload', this.beforeUnloadHandler);
     }
   }
 
   async save(event: Event): Promise<void> {
     this.queue.push({ event, timestamp: Date.now() });
-    if (this.queue.length >= 10) {
+    if (this.queue.length >= this.FLUSH_THRESHOLD) {
       await this.flush();
     }
   }
@@ -60,8 +66,11 @@ export class HTTPEventRepository implements IEventRepository {
   }
 
   private async flush(): Promise<void> {
-    if (this.queue.length === 0) return;
+    if (this.queue.length === 0 || this.isFlushing) {
+      return;
+    }
 
+    this.isFlushing = true;
     const items = [...this.queue];
     this.queue = [];
 
@@ -79,20 +88,25 @@ export class HTTPEventRepository implements IEventRepository {
       });
 
       if (!response.ok) {
-        console.error('Tracking failed:', response.statusText);
         this.queue.unshift(...items);
       }
-    } catch (error) {
-      console.error('Tracking error:', error);
+    } catch {
       this.queue.unshift(...items);
+    } finally {
+      this.isFlushing = false;
     }
   }
 
   destroy(): void {
     if (this.flushTimer) {
       clearInterval(this.flushTimer);
+      this.flushTimer = null;
+    }
+    if (this.beforeUnloadHandler && typeof window !== 'undefined') {
+      window.removeEventListener('beforeunload', this.beforeUnloadHandler);
+      this.beforeUnloadHandler = null;
     }
-    this.flush();
+    void this.flush();
   }
 }
 
@@ -104,7 +118,7 @@ export class HTTPPageviewRepository implements IPageviewRepository {
   }
 
   async save(pageview: Pageview): Promise<void> {
-    await this.eventRepo.save(pageview as any);
+    await this.eventRepo.save(pageview);
   }
 
   async findById(id: EventId): Promise<Pageview | null> {
@@ -126,7 +140,11 @@ export class LocalSessionRepository implements ISessionRepository {
   async save(session: Session): Promise<void> {
     this.sessions.set(session.id.toString(), session);
     if (typeof window !== 'undefined') {
-      localStorage.setItem('wt_session', JSON.stringify(session.toJSON()));
+      try {
+        localStorage.setItem('wt_session', JSON.stringify(session.toJSON()));
+      } catch {
+        // Storage unavailable - continue without localStorage persistence
+      }
     }
   }
 
@@ -137,15 +155,19 @@ export class LocalSessionRepository implements ISessionRepository {
   async findActive(deviceId: string, timeoutMs: number): Promise<Session | null> {
     if (typeof window === 'undefined') return null;
 
-    const stored = localStorage.getItem('wt_session');
-    if (!stored) return null;
+    try {
+      const stored = localStorage.getItem('wt_session');
+      if (!stored) return null;
 
-    const data = JSON.parse(stored);
-    const sessionId = new SessionId(data.id);
+      const data = JSON.parse(stored);
+      const sessionId = new SessionId(data.id);
 
-    const session = this.sessions.get(sessionId.toString());
-    if (session && session.isActive(timeoutMs)) {
-      return session;
+      const session = this.sessions.get(sessionId.toString());
+      if (session && session.isActive(timeoutMs)) {
+        return session;
+      }
+    } catch {
+      // Storage unavailable or corrupted data
     }
 
     return null;
@@ -154,7 +176,11 @@ export class LocalSessionRepository implements ISessionRepository {
   async delete(id: SessionId): Promise<void> {
     this.sessions.delete(id.toString());
     if (typeof window !== 'undefined') {
-      localStorage.removeItem('wt_session');
+      try {
+        localStorage.removeItem('wt_session');
+      } catch {
+        // Storage unavailable - continue without cleanup
+      }
     }
   }
 }

package/src/infrastructure/tracking/web-traffic.service.ts

@@ -32,10 +32,12 @@ class WebTrafficService {
   private eventRepo: HTTPEventRepository | null = null;
   private pageviewRepo: HTTPPageviewRepository | null = null;
   private sessionRepo: LocalSessionRepository | null = null;
+  private originalPushState: typeof history.pushState | null = null;
+  private originalReplaceState: typeof history.replaceState | null = null;
+  private popStateHandler: (() => void) | null = null;
 
   initialize(config: WebTrafficConfig): void {
     if (this.initialized) {
-      console.warn('WebTrafficService already initialized');
       return;
     }
 
@@ -53,8 +55,9 @@ class WebTrafficService {
       this.pageviewRepo
     );
 
-    // Get or create session
-    this.initializeSession();
+    // Get or create session - this is async but we don't await it
+    // The service will return "not initialized" errors if called before session is ready
+    void this.initializeSession();
 
     // Setup auto-tracking if enabled
     if (config.autoTrack && typeof window !== 'undefined') {
@@ -129,13 +132,23 @@ class WebTrafficService {
   private getOrCreateDeviceId(): string {
     if (typeof window === 'undefined') return '';
 
-    let deviceId = localStorage.getItem('wt_device_id');
-    if (!deviceId) {
-      deviceId = `device-${Date.now()}-${Math.random().toString(36).substring(2, 11)}`;
-      localStorage.setItem('wt_device_id', deviceId);
+    try {
+      let deviceId = localStorage.getItem('wt_device_id');
+      if (!deviceId) {
+        const uniqueId = typeof crypto !== 'undefined' && crypto.randomUUID
+          ? crypto.randomUUID()
+          : `${Date.now()}-${Math.random().toString(36).substring(2, 11)}`;
+        deviceId = `device-${uniqueId}`;
+        localStorage.setItem('wt_device_id', deviceId);
+      }
+      return deviceId;
+    } catch {
+      // Storage unavailable - generate temporary device ID
+      const uniqueId = typeof crypto !== 'undefined' && crypto.randomUUID
+        ? crypto.randomUUID()
+        : `${Date.now()}-${Math.random().toString(36).substring(2, 11)}`;
+      return `device-${uniqueId}`;
     }
-
-    return deviceId;
   }
 
   private getUTMFromURL() {
@@ -159,30 +172,52 @@ class WebTrafficService {
     if (typeof window === 'undefined') return;
 
     // Track initial pageview
-    this.trackPageView();
+    void this.trackPageView();
 
     // Track SPA navigation
-    const originalPushState = history.pushState;
-    const originalReplaceState = history.replaceState;
+    this.originalPushState = history.pushState;
+    this.originalReplaceState = history.replaceState;
 
     history.pushState = (...args) => {
-      originalPushState.apply(history, args);
-      this.trackPageView();
+      this.originalPushState!.apply(history, args);
+      void this.trackPageView();
     };
 
     history.replaceState = (...args) => {
-      originalReplaceState.apply(history, args);
-      this.trackPageView();
+      this.originalReplaceState!.apply(history, args);
+      void this.trackPageView();
     };
 
-    window.addEventListener('popstate', () => {
-      this.trackPageView();
-    });
+    this.popStateHandler = () => {
+      void this.trackPageView();
+    };
+    window.addEventListener('popstate', this.popStateHandler);
   }
 
   destroy(): void {
+    // Restore original history methods
+    if (this.originalPushState && typeof history !== 'undefined') {
+      history.pushState = this.originalPushState;
+      this.originalPushState = null;
+    }
+    if (this.originalReplaceState && typeof history !== 'undefined') {
+      history.replaceState = this.originalReplaceState;
+      this.originalReplaceState = null;
+    }
+
+    // Remove popstate event listener
+    if (this.popStateHandler && typeof window !== 'undefined') {
+      window.removeEventListener('popstate', this.popStateHandler);
+      this.popStateHandler = null;
+    }
+
     this.eventRepo?.destroy();
     this.initialized = false;
+    this.commandService = null;
+    this.currentSession = null;
+    this.eventRepo = null;
+    this.pageviewRepo = null;
+    this.sessionRepo = null;
   }
 }
 

package/src/presentation/context.tsx

@@ -5,7 +5,8 @@
 
 import React, { createContext, useContext, useEffect, useMemo, type ReactNode } from 'react';
 import { webTrafficService, type WebTrafficConfig } from '../infrastructure/tracking/web-traffic.service';
-import type { WebTrafficContextValue, TrackingCommandResult } from './hooks';
+import type { WebTrafficContextValue } from './hooks';
+import type { TrackingCommandResult } from '../domains/tracking/application/tracking-command.service';
 
 const TrackingContext = createContext<WebTrafficContextValue | null>(null);
 

package/src/presentation/hooks.ts

@@ -9,13 +9,7 @@ import { webTrafficService } from '../infrastructure/tracking/web-traffic.servic
 import { HTTPAnalyticsRepository } from '../infrastructure/analytics/http-analytics.repository.impl';
 import type { AnalyticsQuery } from '../domains/analytics/repositories/analytics.repository.interface';
 import type { AnalyticsData } from '../domains/analytics/entities/analytics.entity';
-
-export interface TrackingCommandResult {
-  success: boolean;
-  eventId?: string;
-  error?: string;
-}
-
+import type { TrackingCommandResult } from '../domains/tracking/application/tracking-command.service';
 import type { WebTrafficConfig } from '../infrastructure/tracking/web-traffic.service';
 
 export interface WebTrafficContextValue {
@@ -29,17 +23,7 @@ export function useWebTraffic(): WebTrafficContextValue {
   const context = useContext(WebTrafficContext);
 
   if (!context) {
-    // If no context, use service directly
-    return {
-      trackEvent: useCallback(async (name: string, properties?: Record<string, unknown>) => {
-        return webTrafficService.trackEvent(name, properties);
-      }, []),
-      trackPageView: useCallback(async (path?: string) => {
-        return webTrafficService.trackPageView(path);
-      }, []),
-      isInitialized: webTrafficService.isInitialized(),
-      config: { apiKey: '' }, // Fallback - context should always be used
-    };
+    throw new Error('useWebTraffic must be used within WebTrafficProvider');
   }
 
   return context;