posthog-node 4.5.0 → 4.5.2

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Next
 
+# 4.5.2 - 2025-02-12
+
+## Fixed
+
+1. fix: Fixed edge case where `$feature_flag_called` events were enriched with additional feature flag data when they shouldn't be.
+
+# 4.5.1 - 2025-02-12
+
+## Fixed
+
+1. Do not require a `distinctId` as an argument to `captureException`
+
 # 4.5.0 - 2025-02-06
 
 ## Added

package/lib/index.cjs.js

@@ -7,7 +7,7 @@ var node_fs = require('node:fs');
 var node_readline = require('node:readline');
 var node_path = require('node:path');
 
-var version = "4.5.0";
+var version = "4.5.2";
 
 var PostHogPersistedProperty;
 (function (PostHogPersistedProperty) {
@@ -2878,10 +2878,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;
     }
@@ -2907,11 +2909,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);
@@ -3002,8 +3000,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);
@@ -3249,9 +3251,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);
   }
 }
 
@@ -3372,7 +3374,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);