lib: deserialize to native errors in error_serdes

Author: Renegade334

lib/internal/error_serdes.js

@@ -6,7 +6,7 @@ const {
  EvalError,
  FunctionPrototypeCall,
  ObjectAssign,
- ObjectCreate,
+ ObjectDefineProperties,
  ObjectDefineProperty,
  ObjectGetOwnPropertyDescriptor,
  ObjectGetOwnPropertyNames,
@@ -15,11 +15,12 @@ const {
  ObjectPrototypeToString,
  RangeError,
  ReferenceError,
+ ReflectConstruct,
+ ReflectDeleteProperty,
  SafeSet,
  StringFromCharCode,
  StringPrototypeSubstring,
  SymbolFor,
- SymbolToStringTag,
  SyntaxError,
  TypeError,
  TypedArrayPrototypeGetBuffer,
@@ -28,6 +29,8 @@ const {
  URIError,
 } = primordials;
 
+const assert = require('internal/assert');
+
 const { Buffer } = require('buffer');
 const { inspect: { custom: customInspectSymbol } } = require('util');
 
@@ -166,16 +169,17 @@ function deserializeError(error) {
  switch (error[0]) {
  case kSerializedError: {
  const { constructor, properties } = deserialize(error.subarray(1));
- const ctor = errors[constructor];
- ObjectDefineProperty(properties, SymbolToStringTag, {
- __proto__: null,
- value: { __proto__: null, value: 'Error', configurable: true },
- enumerable: true,
- });
+ assert(errorConstructorNames.has(constructor), 'Invalid constructor');
  if ('cause' in properties && 'value' in properties.cause) {
  properties.cause.value = deserializeError(properties.cause.value);
  }
- return ObjectCreate(ctor.prototype, properties);
+ // Invoke the Error constructor to gain an object with an [[ErrorData]] internal slot
+ const ret = ReflectConstruct(Error, [], errors[constructor]);
+ // Delete any properties defined by the Error constructor before assigning from source
+ ArrayPrototypeForEach(ObjectGetOwnPropertyNames(ret), (key) => {
+ ReflectDeleteProperty(ret, key);
+ });
+ return ObjectDefineProperties(ret, properties);
  }
  case kSerializedObject:
  return deserialize(error.subarray(1));
@@ -196,7 +200,7 @@ function deserializeError(error) {
  [customInspectSymbol]: () => '[Circular object]',
  };
  }
- require('assert').fail('This should not happen');
+ assert.fail('Unknown serializer flag');
 }
 
 module.exports = { serializeError, deserializeError };

test/es-module/test-esm-loader-with-syntax-error.mjs

@@ -13,7 +13,7 @@ describe('ESM: loader with syntax error', { concurrency: !process.env.TEST_PARAL
  path('print-error-message.js'),
  ]);
 
- match(stderr, /SyntaxError \[Error\]:/);
+ match(stderr, /SyntaxError:/);
  ok(!stderr.includes('Bad command or file name'));
  notStrictEqual(code, 0);
  });

test/parallel/test-worker-error-serdes.js

@@ -0,0 +1,34 @@
+'use strict';
+const common = require('../common');
+const assert = require('assert');
+const { isNativeError } = require('util/types');
+const { Worker } = require('worker_threads');
+
+const validateError = (error, ctor) => {
+ assert.strictEqual(error.constructor, ctor);
+ assert.strictEqual(Object.getPrototypeOf(error), ctor.prototype);
+ assert(isNativeError(error));
+};
+
+{
+ const w = new Worker('throw new Error()', { eval: true });
+ w.on('error', common.mustCall((error) => {
+ validateError(error, Error);
+ }));
+}
+
+{
+ const w = new Worker('throw new RangeError()', { eval: true });
+ w.on('error', common.mustCall((error) => {
+ validateError(error, RangeError);
+ }));
+}
+
+{
+ const w = new Worker('throw new Error(undefined, { cause: new TypeError() })', { eval: true });
+ w.on('error', common.mustCall((error) => {
+ validateError(error, Error);
+ assert.notStrictEqual(error.cause, undefined);
+ validateError(error.cause, TypeError);
+ }));
+}