Add failsafe if bog-analytics fails

[UZ9]

Addresses #221

[greptile-apps]

Disclaimer: This comment was AI-generated and is not necessarily completely accurate. Please take code comments with a grain of salt.

Greptile Overview

Greptile Summary

Added null/undefined response checks after calling bog-analytics logging methods (logClickEvent, logInputEvent, logVisitEvent, logCustomEvent) to prevent accessing properties on potentially null/undefined responses. When bog-analytics fails or returns falsy values due to invalid configuration, the service now throws a descriptive RpcException with INVALID_ARGUMENT status code instead of causing runtime errors.

• Added identical failsafe checks in all four logging methods
• Each check throws an exception with message "Invalid analytics configuration" if response is falsy
• Prevents downstream errors when trying to access properties like response._id, response.category, etc.

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• The changes add defensive programming checks that prevent null/undefined access errors. The implementation is consistent across all four affected methods, uses appropriate error codes and messages, and doesn't modify any existing logic or introduce breaking changes
• No files require special attention

Important Files Changed

File Analysis

Filename	Score	Overview
packages/analytics-service/src/modules/analytics/analytics.service.ts	5/5	Added null/undefined response checks after bog-analytics logging calls to handle configuration failures gracefully

Sequence Diagram

sequenceDiagram
    participant Client
    participant AnalyticsService
    participant AnalyticsLogger
    participant BogAnalytics
    
    Client->>AnalyticsService: logClickEvent(event)
    AnalyticsService->>AnalyticsService: getAnalyticsLogger()
    AnalyticsService->>AnalyticsLogger: authenticate(clientKey)
    AnalyticsService->>BogAnalytics: logClickEvent(event)
    
    alt Response exists
        BogAnalytics-->>AnalyticsService: response object
        AnalyticsService->>AnalyticsService: Check if response exists
        AnalyticsService-->>Client: Return formatted response
    else Response is null/undefined (NEW)
        BogAnalytics-->>AnalyticsService: null/undefined
        AnalyticsService->>AnalyticsService: Check if response exists
        AnalyticsService-->>Client: RpcException: Invalid analytics configuration
    end

Loading

_{1 file reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}