ensure Error.captureStackTrace is only used in the correct envs

paritytech · Jan 17, 2024 · dbbcb47 · dbbcb47
1 parent 0c61913
commit dbbcb47
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/src/errors/BaseError.ts b/src/errors/BaseError.ts
@@ -1,5 +1,8 @@
 // Copyright 2023 Parity Technologies (UK) Ltd.
 
+import { JS_ENV } from '../consts';
+import { WindowChrome } from './types';
+
 /**
  * Errors that may be returned by the API.
  */
@@ -76,6 +79,9 @@ export class BaseError extends Error {
 	constructor(message: string, error?: BaseErrorsEnum) {
 		super(message);
 		this.name = error || this.constructor.name;
-		Error.captureStackTrace(this, this.constructor);
+		// Only runtimes built on v8 will have `Error.captureStackTrace`.
+		if (JS_ENV === 'node' || (window && (window as WindowChrome).chrome)) {
+			Error.captureStackTrace(this, this.constructor);
+		}
 	}
 }
diff --git a/src/errors/types.ts b/src/errors/types.ts
@@ -8,3 +8,7 @@ export interface CheckXcmTxInputsOpts {
 	isLimited?: boolean;
 	weightLimit?: { refTime?: string; proofSize?: string };
 }
+
+export interface WindowChrome extends Window {
+	chrome?: unknown;
+}