@posthog/core 1.48.3 → 1.48.4

package/dist/error-tracking/coercers/error-coercer.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"error-coercer.d.ts","sourceRoot":"","sources":["../../../src/error-tracking/coercers/error-coercer.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,eAAe,EAAE,oBAAoB,EAAE,aAAa,EAAE,MAAM,UAAU,CAAA;AAE/E,qBAAa,YAAa,YAAW,oBAAoB,CAAC,KAAK,CAAC;IAC9D,KAAK,CAAC,GAAG,EAAE,OAAO,GAAG,GAAG,IAAI,KAAK;IAIjC,MAAM,CAAC,GAAG,EAAE,KAAK,EAAE,GAAG,EAAE,eAAe,GAAG,aAAa;IAUvD,OAAO,CAAC,OAAO;IAIf,OAAO,CAAC,UAAU;IAUlB,OAAO,CAAC,QAAQ;CAGjB"}
+{"version":3,"file":"error-coercer.d.ts","sourceRoot":"","sources":["../../../src/error-tracking/coercers/error-coercer.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,eAAe,EAAE,oBAAoB,EAAE,aAAa,EAAE,MAAM,UAAU,CAAA;AAE/E,qBAAa,YAAa,YAAW,oBAAoB,CAAC,KAAK,CAAC;IAC9D,KAAK,CAAC,GAAG,EAAE,OAAO,GAAG,GAAG,IAAI,KAAK;IAIjC,MAAM,CAAC,GAAG,EAAE,KAAK,EAAE,GAAG,EAAE,eAAe,GAAG,aAAa;IAgBvD,OAAO,CAAC,OAAO;IAIf,OAAO,CAAC,UAAU;IAUlB,OAAO,CAAC,QAAQ;CAGjB"}

package/dist/error-tracking/coercers/error-coercer.js

@@ -32,12 +32,14 @@ class ErrorCoercer {
         return (0, type_utils_js_namespaceObject.isError)(err);
     }
     coerce(err, ctx) {
+        const stack = this.getStack(err);
+        const synthetic = void 0 === stack;
         return {
             type: this.getType(err),
             value: this.getMessage(err, ctx),
-            stack: this.getStack(err),
+            stack: stack ?? ctx.syntheticException?.stack,
             cause: err.cause ? ctx.next(err.cause) : void 0,
-            synthetic: false
+            synthetic
         };
     }
     getType(err) {

package/dist/error-tracking/coercers/error-coercer.mjs

@@ -4,12 +4,14 @@ class ErrorCoercer {
         return isError(err);
     }
     coerce(err, ctx) {
+        const stack = this.getStack(err);
+        const synthetic = void 0 === stack;
         return {
             type: this.getType(err),
             value: this.getMessage(err, ctx),
-            stack: this.getStack(err),
+            stack: stack ?? ctx.syntheticException?.stack,
             cause: err.cause ? ctx.next(err.cause) : void 0,
-            synthetic: false
+            synthetic
         };
     }
     getType(err) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@posthog/core",
-  "version": "1.48.3",
+  "version": "1.48.4",
   "bugs": {
     "url": "https://github.com/PostHog/posthog-js/issues"
   },

package/src/error-tracking/coercers/error-coercer.ts

@@ -7,12 +7,18 @@ export class ErrorCoercer implements ErrorTrackingCoercer<Error> {
   }
 
   coerce(err: Error, ctx: CoercingContext): ExceptionLike {
+    const stack = this.getStack(err)
+    // Some errors carry no usable stack (for example a Firefox network `fetch`
+    // `TypeError`). Fall back to the synthetic exception captured at the call
+    // site so the frames still point at the caller. Mark it synthetic so the
+    // parser trims the SDK's own frames.
+    const synthetic = stack === undefined
     return {
       type: this.getType(err),
       value: this.getMessage(err, ctx),
-      stack: this.getStack(err),
+      stack: stack ?? ctx.syntheticException?.stack,
       cause: err.cause ? ctx.next(err.cause) : undefined,
-      synthetic: false,
+      synthetic,
     }
   }
 

package/src/error-tracking/error-properties-builder.coerce.spec.ts

@@ -112,6 +112,21 @@ describe('ErrorPropertiesBuilder', () => {
       })
     })
 
+    it('should fall back to the synthetic stack when the error has no stack', () => {
+      // Firefox rejects a network `fetch` with a `TypeError` that carries no
+      // stack, so the frames must come from the synthetic exception instead.
+      const errorObject = new CustomTestError('My special error')
+      errorObject.stack = ''
+      const syntheticError = new Error()
+      const exception = coerceInput(errorObject, syntheticError)
+      expect(exception).toMatchObject({
+        type: 'CustomTestError',
+        value: 'My special error',
+        stack: syntheticError.stack,
+        synthetic: true,
+      })
+    })
+
     it('should preserve a cross-realm error', () => {
       const crossRealmError = runInNewContext(
         `new TypeError('cross-realm error', {