refactor

apps/api/src/modules/analytics/AnalyticsService.ts

@@ -1,83 +1,26 @@
 import { Injectable, Inject } from '@nestjs/common';
 import { RecordEngagementInput, RecordEngagementOutput, RecordPageViewInput, RecordPageViewOutput } from './dto/AnalyticsDto';
-import { IPageViewRepository } from '@gridpilot/analytics/application/repositories/IPageViewRepository';
-import { IEngagementRepository } from '@gridpilot/analytics/domain/repositories/IEngagementRepository';
 import { Logger } from '@gridpilot/shared/logging/Logger';
-import { PageView } from '@gridpilot/analytics/domain/entities/PageView';
-import { EngagementEvent } from '@gridpilot/analytics/domain/entities/EngagementEvent';
+import { RecordPageViewUseCase } from './use-cases/RecordPageViewUseCase';
+import { RecordEngagementUseCase } from './use-cases/RecordEngagementUseCase';
 
 const Logger_TOKEN = 'Logger_TOKEN';
-const IPAGE_VIEW_REPO_TOKEN = 'IPageViewRepository_TOKEN';
-const IENGAGEMENT_REPO_TOKEN = 'IEngagementRepository_TOKEN';
+const RECORD_PAGE_VIEW_USE_CASE_TOKEN = 'RecordPageViewUseCase_TOKEN';
+const RECORD_ENGAGEMENT_USE_CASE_TOKEN = 'RecordEngagementUseCase_TOKEN';
 
 @Injectable()
 export class AnalyticsService {
   constructor(
-    @Inject(IPAGE_VIEW_REPO_TOKEN) private readonly pageViewRepository: IPageViewRepository,
-    @Inject(IENGAGEMENT_REPO_TOKEN) private readonly engagementRepository: IEngagementRepository,
+    @Inject(RECORD_PAGE_VIEW_USE_CASE_TOKEN) private readonly recordPageViewUseCase: RecordPageViewUseCase,
+    @Inject(RECORD_ENGAGEMENT_USE_CASE_TOKEN) private readonly recordEngagementUseCase: RecordEngagementUseCase,
     @Inject(Logger_TOKEN) private readonly logger: Logger,
   ) {}
 
   async recordPageView(input: RecordPageViewInput): Promise<RecordPageViewOutput> {
-    this.logger.debug('Executing RecordPageViewUseCase', { input });
-    try {
-      const pageViewId = `pv-${Date.now()}-${Math.random().toString(36).substr(2, 9)}`;
-
-      const baseProps: Omit<Parameters<typeof PageView.create>[0], 'timestamp'> = {
-        id: pageViewId,
-        entityType: input.entityType as any, // Cast to any to bypass strict type checking, will resolve with proper domain layer alignment
-        entityId: input.entityId,
-        visitorType: input.visitorType as any, // Cast to any to bypass strict type checking, will resolve with proper domain layer alignment
-        sessionId: input.sessionId,
-      };
-
-      const pageView = PageView.create({
-        ...baseProps,
-        ...(input.visitorId !== undefined ? { visitorId: input.visitorId } : {}),
-        ...(input.referrer !== undefined ? { referrer: input.referrer } : {}),
-        ...(input.userAgent !== undefined ? { userAgent: input.userAgent } : {}),
-        ...(input.country !== undefined ? { country: input.country } : {}),
-      });
-
-      await this.pageViewRepository.save(pageView);
-      this.logger.info('Page view recorded successfully', { pageViewId, input });
-      return { pageViewId };
-    } catch (error) {
-      this.logger.error('Error recording page view', error, { input });
-      throw error;
-    }
+    return await this.recordPageViewUseCase.execute(input);
   }
 
   async recordEngagement(input: RecordEngagementInput): Promise<RecordEngagementOutput> {
-    this.logger.debug('Executing RecordEngagementUseCase', { input });
-    try {
-      const eventId = `eng-${Date.now()}-${Math.random().toString(36).substr(2, 9)}`;
-
-      const baseProps: Omit<Parameters<typeof EngagementEvent.create>[0], 'timestamp'> = {
-        id: eventId,
-        action: input.action as any, // Cast to any to bypass strict type checking, will resolve with proper domain layer alignment
-        entityType: input.entityType as any, // Cast to any to bypass strict type checking, will resolve with proper domain layer alignment
-        entityId: input.entityId,
-        actorType: input.actorType,
-        sessionId: input.sessionId,
-      };
-
-      const event = EngagementEvent.create({
-        ...baseProps,
-        ...(input.actorId !== undefined ? { actorId: input.actorId } : {}),
-        ...(input.metadata !== undefined ? { metadata: input.metadata } : {}),
-      });
-
-      await this.engagementRepository.save(event);
-      this.logger.info('Engagement recorded successfully', { eventId, input });
-
-      return {
-        eventId,
-        engagementWeight: event.getEngagementWeight(),
-      };
-    } catch (error) {
-      this.logger.error('Error recording engagement', error, { input });
-      throw error;
-    }
+    return await this.recordEngagementUseCase.execute(input);
   }
 }