posthog-node 4.5.0 → 4.5.2

package/lib/index.d.ts

@@ -389,7 +389,7 @@ interface Mechanism {
 declare class ErrorTracking {
     private client;
     private _exceptionAutocaptureEnabled;
-    static captureException(client: PostHog, error: unknown, distinctId: string, hint: EventHint, additionalProperties?: Record<string | number, any>): Promise<void>;
+    static captureException(client: PostHog, error: unknown, hint: EventHint, distinctId?: string, additionalProperties?: Record<string | number, any>): Promise<void>;
     constructor(client: PostHog, options: PostHogOptions);
     private startAutocaptureIfEnabled;
     private onException;
@@ -472,7 +472,7 @@ declare class PostHog extends PostHogCoreStateless implements PostHogNodeV1 {
     reloadFeatureFlags(): Promise<void>;
     shutdown(shutdownTimeoutMs?: number): Promise<void>;
     private addLocalPersonAndGroupProperties;
-    captureException(error: unknown, distinctId: string, additionalProperties?: Record<string | number, any>): void;
+    captureException(error: unknown, distinctId?: string, additionalProperties?: Record<string | number, any>): void;
 }
 
 /**

package/lib/index.esm.js

@@ -3,7 +3,7 @@ import { createReadStream } from 'node:fs';
 import { createInterface } from 'node:readline';
 import { posix, dirname, sep } from 'node:path';
 
-var version = "4.5.0";
+var version = "4.5.2";
 
 var PostHogPersistedProperty;
 (function (PostHogPersistedProperty) {
@@ -2874,10 +2874,12 @@ function getLastStackFrame(arr) {
 
 const SHUTDOWN_TIMEOUT = 2000;
 class ErrorTracking {
-  static async captureException(client, error, distinctId, hint, additionalProperties) {
+  static async captureException(client, error, hint, distinctId, additionalProperties) {
     const properties = {
       ...additionalProperties
     };
+    // Given stateless nature of Node SDK we capture exceptions using personless processing when no
+    // user can be determined because a distinct_id is not provided e.g. exception autocapture
     if (!distinctId) {
       properties.$process_person_profile = false;
     }
@@ -2903,11 +2905,7 @@ class ErrorTracking {
     }
   }
   onException(exception, hint) {
-    // Given stateless nature of Node SDK we capture exceptions using personless processing
-    // when no user can be determined e.g. in the case of exception autocapture
-    ErrorTracking.captureException(this.client, exception, uuidv7(), hint, {
-      $process_person_profile: false
-    });
+    ErrorTracking.captureException(this.client, exception, hint);
   }
   async onFatalError() {
     await this.client.shutdown(SHUTDOWN_TIMEOUT);
@@ -2998,8 +2996,12 @@ class PostHog extends PostHogCoreStateless {
         // return await super.getFeatureFlagsStateless(distinctId, groups, undefined, undefined, disableGeoip)
         return await _getFlags(distinctId, groups, disableGeoip);
       }
+      if (event === '$feature_flag_called') {
+        // If we're capturing a $feature_flag_called event, we don't want to enrich the event with cached flags that may be out of date.
+        return {};
+      }
       if ((this.featureFlagsPoller?.featureFlags?.length || 0) > 0) {
-        // Otherwise we may as well check for the flags locally and include them if there
+        // Otherwise we may as well check for the flags locally and include them if they are already loaded
         const groupsWithStringValues = {};
         for (const [key, value] of Object.entries(groups || {})) {
           groupsWithStringValues[key] = String(value);
@@ -3245,9 +3247,9 @@ class PostHog extends PostHogCoreStateless {
   }
   captureException(error, distinctId, additionalProperties) {
     const syntheticException = new Error('PostHog syntheticException');
-    ErrorTracking.captureException(this, error, distinctId, {
+    ErrorTracking.captureException(this, error, {
       syntheticException
-    }, additionalProperties);
+    }, distinctId, additionalProperties);
   }
 }
 
@@ -3368,7 +3370,7 @@ function setupExpressErrorHandler(_posthog, app) {
     };
     // Given stateless nature of Node SDK we capture exceptions using personless processing
     // when no user can be determined e.g. in the case of exception autocapture
-    ErrorTracking.captureException(_posthog, error, uuidv7(), hint, {
+    ErrorTracking.captureException(_posthog, error, hint, uuidv7(), {
       $process_person_profile: false
     });
     next(error);