posthog-node 4.11.2 → 4.11.3

package/CHANGELOG.md

@@ -1,8 +1,11 @@
+# 4.11.3 - 2025-04-08
+
+1. fix: do not access `fs` or `readline` in when not available e.g. edge environments
+
 # 4.11.2 - 2025-04-07
 
 1. chore: bump axios to 1.8.2 (fixes [CVE-2025-27152](https://github.com/advisories/GHSA-jr5f-v2jv-69x6))
 
-
 # 4.11.1 - 2025-03-28
 
 ## Fixed
@@ -28,7 +31,6 @@
 1. Fix: only set `platform` on PostHog exception frame properties
 1. Fix: prevent fetch floods when rate-limited.
 
-
 # 4.10.0 – 2025-03-06
 
 1. Attach requestId to $feature_flag_called if present in /decide response
@@ -320,4 +322,4 @@ Breaking changes:
 What's new:
 
 1. You can now evaluate feature flags locally (i.e. without sending a request to your PostHog servers) by setting a personal API key, and passing in groups and person properties to `isFeatureEnabled` and `getFeatureFlag` calls.
-2. Introduces a `getAllFlags` method that returns all feature flags. This is useful for when you want to seed your frontend with some initial flags, given a user ID.
+2. Introduces a `getAllFlags` method that returns all feature flags. This is useful for when you want to seed your frontend with some initial flags, given a user ID.

package/lib/index.cjs.js

@@ -2,8 +2,6 @@
 
 Object.defineProperty(exports, '__esModule', { value: true });
 
-var node_fs = require('node:fs');
-var node_readline = require('node:readline');
 var node_path = require('node:path');
 
 function _interopNamespace(e) {
@@ -24,7 +22,7 @@ function _interopNamespace(e) {
   return Object.freeze(n);
 }
 
-var version = "4.11.2";
+var version = "4.11.3";
 
 var PostHogPersistedProperty;
 (function (PostHogPersistedProperty) {
@@ -2536,6 +2534,26 @@ class ReduceableCache {
   }
 }
 
+const nodeFs = new Lazy(async () => {
+  try {
+    return await Promise.resolve().then(function () { return /*#__PURE__*/_interopNamespace(require('node:fs')); });
+  } catch {
+    return undefined;
+  }
+});
+async function getNodeFs() {
+  return await nodeFs.getValue();
+}
+const nodeReadline = new Lazy(async () => {
+  try {
+    return await Promise.resolve().then(function () { return /*#__PURE__*/_interopNamespace(require('node:readline')); });
+  } catch {
+    return undefined;
+  }
+});
+async function getNodeReadline() {
+  return await nodeReadline.getValue();
+}
 const LRU_FILE_CONTENTS_CACHE = new ReduceableCache(25);
 const LRU_FILE_CONTENTS_FS_READ_FAILED = new ReduceableCache(20);
 const DEFAULT_LINES_OF_CONTEXT = 7;
@@ -2603,70 +2621,79 @@ async function addSourceContext(frames) {
  */
 function getContextLinesFromFile(path, ranges, output) {
   return new Promise(resolve => {
-    // It is important *not* to have any async code between createInterface and the 'line' event listener
-    // as it will cause the 'line' event to
-    // be emitted before the listener is attached.
-    const stream = node_fs.createReadStream(path);
-    const lineReaded = node_readline.createInterface({
-      input: stream
-    });
-    // We need to explicitly destroy the stream to prevent memory leaks,
-    // removing the listeners on the readline interface is not enough.
-    // See: https://github.com/nodejs/node/issues/9002 and https://github.com/getsentry/sentry-javascript/issues/14892
-    function destroyStreamAndResolve() {
-      stream.destroy();
-      resolve();
-    }
-    // Init at zero and increment at the start of the loop because lines are 1 indexed.
-    let lineNumber = 0;
-    let currentRangeIndex = 0;
-    const range = ranges[currentRangeIndex];
-    if (range === undefined) {
-      // We should never reach this point, but if we do, we should resolve the promise to prevent it from hanging.
-      destroyStreamAndResolve();
-      return;
-    }
-    let rangeStart = range[0];
-    let rangeEnd = range[1];
-    // We use this inside Promise.all, so we need to resolve the promise even if there is an error
-    // to prevent Promise.all from short circuiting the rest.
-    function onStreamError() {
-      // Mark file path as failed to read and prevent multiple read attempts.
-      LRU_FILE_CONTENTS_FS_READ_FAILED.set(path, 1);
-      lineReaded.close();
-      lineReaded.removeAllListeners();
-      destroyStreamAndResolve();
-    }
-    // We need to handle the error event to prevent the process from crashing in < Node 16
-    // https://github.com/nodejs/node/pull/31603
-    stream.on('error', onStreamError);
-    lineReaded.on('error', onStreamError);
-    lineReaded.on('close', destroyStreamAndResolve);
-    lineReaded.on('line', line => {
-      lineNumber++;
-      if (lineNumber < rangeStart) {
+    // KLUDGE: edge runtimes do not support node:fs or node:readline
+    // until we have separate packages for each environment this will skip
+    // trying to access the filesystem when not accessible
+    Promise.all([getNodeFs(), getNodeReadline()]).then(([nodeFs, nodeReadline]) => {
+      if (!nodeFs || !nodeReadline) {
+        resolve();
+        return;
+      }
+      // It is important *not* to have any async code between createInterface and the 'line' event listener
+      // as it will cause the 'line' event to
+      // be emitted before the listener is attached.
+      const stream = nodeFs.createReadStream(path);
+      const lineReaded = nodeReadline.createInterface({
+        input: stream
+      });
+      // We need to explicitly destroy the stream to prevent memory leaks,
+      // removing the listeners on the readline interface is not enough.
+      // See: https://github.com/nodejs/node/issues/9002 and https://github.com/getsentry/sentry-javascript/issues/14892
+      function destroyStreamAndResolve() {
+        stream.destroy();
+        resolve();
+      }
+      // Init at zero and increment at the start of the loop because lines are 1 indexed.
+      let lineNumber = 0;
+      let currentRangeIndex = 0;
+      const range = ranges[currentRangeIndex];
+      if (range === undefined) {
+        // We should never reach this point, but if we do, we should resolve the promise to prevent it from hanging.
+        destroyStreamAndResolve();
         return;
       }
-      // !Warning: This mutates the cache by storing the snipped line into the cache.
-      output[lineNumber] = snipLine(line, 0);
-      if (lineNumber >= rangeEnd) {
-        if (currentRangeIndex === ranges.length - 1) {
-          // We need to close the file stream and remove listeners, else the reader will continue to run our listener;
-          lineReaded.close();
-          lineReaded.removeAllListeners();
+      let rangeStart = range[0];
+      let rangeEnd = range[1];
+      // We use this inside Promise.all, so we need to resolve the promise even if there is an error
+      // to prevent Promise.all from short circuiting the rest.
+      function onStreamError() {
+        // Mark file path as failed to read and prevent multiple read attempts.
+        LRU_FILE_CONTENTS_FS_READ_FAILED.set(path, 1);
+        lineReaded.close();
+        lineReaded.removeAllListeners();
+        destroyStreamAndResolve();
+      }
+      // We need to handle the error event to prevent the process from crashing in < Node 16
+      // https://github.com/nodejs/node/pull/31603
+      stream.on('error', onStreamError);
+      lineReaded.on('error', onStreamError);
+      lineReaded.on('close', destroyStreamAndResolve);
+      lineReaded.on('line', line => {
+        lineNumber++;
+        if (lineNumber < rangeStart) {
           return;
         }
-        currentRangeIndex++;
-        const range = ranges[currentRangeIndex];
-        if (range === undefined) {
-          // This should never happen as it means we have a bug in the context.
-          lineReaded.close();
-          lineReaded.removeAllListeners();
-          return;
+        // !Warning: This mutates the cache by storing the snipped line into the cache.
+        output[lineNumber] = snipLine(line, 0);
+        if (lineNumber >= rangeEnd) {
+          if (currentRangeIndex === ranges.length - 1) {
+            // We need to close the file stream and remove listeners, else the reader will continue to run our listener;
+            lineReaded.close();
+            lineReaded.removeAllListeners();
+            return;
+          }
+          currentRangeIndex++;
+          const range = ranges[currentRangeIndex];
+          if (range === undefined) {
+            // This should never happen as it means we have a bug in the context.
+            lineReaded.close();
+            lineReaded.removeAllListeners();
+            return;
+          }
+          rangeStart = range[0];
+          rangeEnd = range[1];
         }
-        rangeStart = range[0];
-        rangeEnd = range[1];
-      }
+      });
     });
   });
 }