@crimson-education/browser-logger 5.0.2 → 5.0.4

package/src/reporters/posthogReporter.test.ts

@@ -145,10 +145,73 @@ describe('posthogReporter', () => {
 
     reporter.recordSession?.();
     reporter.recordSessionStop?.();
-    expect(posthogMock.startSessionRecording).toHaveBeenCalledWith(true);
+    expect(posthogMock.startSessionRecording).toHaveBeenCalledWith({
+      event_trigger: true,
+      linked_flag: true,
+      sampling: true,
+      url_trigger: true,
+    });
     expect(posthogMock.stopSessionRecording).toHaveBeenCalled();
   });
 
+  it('suppresses configured trackEvent messages from PostHog analytics', () => {
+    const reporter = posthogReporter(
+      {
+        service: 'crimson-app-frontend',
+        application: 'crimson-app',
+        environment: 'production',
+        version: '1.2.3',
+      },
+      {
+        apiKey: 'ph_key',
+        apiHost: 'https://eu.i.posthog.com',
+        ignoreTrackEventPatterns: ['feedback fetch success'],
+      },
+    );
+
+    reporter.trackEvent({
+      message: 'feedback fetch success',
+      level: LogLevel.Info,
+    });
+
+    expect(posthogMock.capture).not.toHaveBeenCalled();
+
+    reporter.trackEvent({
+      message: 'message-sent',
+      level: LogLevel.Info,
+    });
+
+    expect(posthogMock.capture).toHaveBeenCalledWith('message-sent', {
+      message: 'message-sent',
+      level: LogLevel.Info,
+    });
+  });
+
+  it('does not suppress trackEvent messages unless configured', () => {
+    const reporter = posthogReporter(
+      {
+        service: 'crimson-app-frontend',
+        application: 'crimson-app',
+        environment: 'production',
+        version: '1.2.3',
+      },
+      {
+        apiKey: 'ph_key',
+        apiHost: 'https://eu.i.posthog.com',
+      },
+    );
+
+    reporter.trackEvent({
+      message: 'feedback fetch success',
+      level: LogLevel.Info,
+    });
+
+    expect(posthogMock.capture).toHaveBeenCalledWith('feedback fetch success', {
+      message: 'feedback fetch success',
+      level: LogLevel.Info,
+    });
+  });
+
   it('adds compatibility properties in before_send', () => {
     const beforeSend = jest.fn((captureResult) => captureResult);
 

package/src/reporters/posthogReporter.ts

@@ -10,6 +10,7 @@ import {
   ReportUser,
   ServiceInfo,
 } from '../types';
+import { shouldSuppressAnalyticsEvent } from './analyticsNoise';
 
 type PostHogValue = Properties[string];
 
@@ -283,6 +284,9 @@ export function posthogReporter(info: ServiceInfo, config: PostHogReporterConfig
   const reporter: IReporter = {
     ...config,
     trackEvent: function (event: ReporterEvent): void {
+      if (shouldSuppressAnalyticsEvent(event.message, config.ignoreTrackEventPatterns)) {
+        return;
+      }
       posthog.capture(event.message, {
         message: event.message,
         level: event.level,
@@ -363,7 +367,12 @@ export function posthogReporter(info: ServiceInfo, config: PostHogReporterConfig
       });
     },
     recordSession: function (): void {
-      posthog.startSessionRecording(true);
+      posthog.startSessionRecording({
+        event_trigger: true,
+        linked_flag: true,
+        sampling: true,
+        url_trigger: true,
+      });
     },
     recordSessionStop: function (): void {
       posthog.stopSessionRecording();

package/src/types/reporter.ts

@@ -163,6 +163,12 @@ export interface ReporterConfigBase {
    * This is useful to prevent data of no use for the reporter, or if the data keys are too long.
    */
   ignoreMetadataPatterns?: (string | RegExp)[];
+
+  /**
+   * Ignore specific trackEvent messages from analytics-style reporters.
+   * This keeps technical log noise out of product analytics/RUM while preserving log reporter output.
+   */
+  ignoreTrackEventPatterns?: (string | RegExp)[];
 }
 
 /**