René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"[worker_threads]: Main thread receives Error object when the worker throws a primitive value","articleBody":"\u003c!--\r\nThank you for reporting an issue.\r\n\r\nThis issue tracker is for bugs and issues found within Node.js core.\r\nIf you require more general support please file an issue on our help\r\nrepo. https://github.com/nodejs/help\r\n\r\n\r\nPlease fill in as much of the template below as you're able.\r\n\r\nVersion: output of `node -v`\r\nPlatform: output of `uname -a` (UNIX), or version and 32 or 64-bit (Windows)\r\nSubsystem: if known, please specify affected core module name\r\n-->\r\n\r\n* **Version**: v14.7.0\r\n* **Platform**: Darwin suin 18.7.0 Darwin Kernel Version 18.7.0: Mon Feb 10 21:08:45 PST 2020; root:xnu-4903.278.28~1/RELEASE_X86_64 x86_64\r\n* **Subsystem**: N/A\r\n\r\n### What steps will reproduce the bug?\r\n\r\nSummary: Since Node v14.7.0, when primitive values are thrown in a worker, the main thread receives it as Error objects.\r\n\r\nI'm not sure this is actually a bug. But I couldn't find out the information about this breaking change. So I have created this issue. If this is an expected change, please close this issue.\r\n\r\n```js\r\n// main.js\r\nconst { Worker } = require('worker_threads')\r\nconst worker = new Worker('./worker.js')\r\nconsole.log(process.versions.node)\r\nworker.on('error', err => {\r\n  console.error(err)\r\n})\r\n```\r\n\r\n```js\r\n// worker.js\r\nthrow 'Error was thrown in worker'\r\n```\r\n\r\nin Node v14.6.0:\r\n\r\n```\r\n$ node main.js\r\n14.6.0\r\nError was thrown in worker\r\n```\r\n\r\nin Node v14.7.0:\r\n\r\n```\r\n$ node main.js\r\n14.7.0\r\nError [ERR_UNHANDLED_ERROR]: Unhandled error. ('Error was thrown in worker')\r\n    at process.emit (events.js:303:17)\r\n    at emitUnhandledRejectionOrErr (internal/event_target.js:541:11)\r\n    at MessagePort.[nodejs.internal.kHybridDispatch] (internal/event_target.js:356:9)\r\n    at MessagePort.exports.emitMessage (internal/per_context/messageport.js:18:26) {\r\n  code: 'ERR_UNHANDLED_ERROR',\r\n  context: 'Error was thrown in worker'\r\n}\r\n```\r\n\r\nOther primitives:\r\n\r\n\u003cdetails>\r\n\r\n__Undefined__\r\n\r\n```js\r\n// worker.js\r\nthrow undefined\r\n```\r\n\r\n```\r\n$ node main.js\r\n14.6.0\r\nundefined\r\n\r\n$ node main.js\r\n14.7.0\r\nError [ERR_UNHANDLED_ERROR]: Unhandled error. (undefined)\r\n    at process.emit (events.js:303:17)\r\n    at emitUnhandledRejectionOrErr (internal/event_target.js:541:11)\r\n    at MessagePort.[nodejs.internal.kHybridDispatch] (internal/event_target.js:356:9)\r\n    at MessagePort.exports.emitMessage (internal/per_context/messageport.js:18:26) {\r\n  code: 'ERR_UNHANDLED_ERROR',\r\n  context: undefined\r\n}\r\n```\r\n\r\n__Number__\r\n\r\n```js\r\n// worker.js\r\nthrow 0\r\n```\r\n\r\n```\r\n$ node main.js\r\n14.6.0\r\n0\r\n\r\n$ node main.js\r\n14.7.0\r\nError [ERR_UNHANDLED_ERROR]: Unhandled error. (0)\r\n    at process.emit (events.js:303:17)\r\n    at emitUnhandledRejectionOrErr (internal/event_target.js:541:11)\r\n    at MessagePort.[nodejs.internal.kHybridDispatch] (internal/event_target.js:356:9)\r\n    at MessagePort.exports.emitMessage (internal/per_context/messageport.js:18:26) {\r\n  code: 'ERR_UNHANDLED_ERROR',\r\n  context: 0\r\n}\r\n```\r\n\r\n__Symbol__\r\n\r\n```js\r\n// worker.js\r\nthrow Symbol('this is a symbol')\r\n```\r\n\r\n```\r\n$ node main.js\r\n14.6.0\r\nSymbol(this is a symbol)\r\n\r\n$ node main.js\r\n14.7.0\r\nError [ERR_UNHANDLED_ERROR]: Unhandled error. (Symbol(this is a symbol))\r\n    at process.emit (events.js:303:17)\r\n    at emitUnhandledRejectionOrErr (internal/event_target.js:541:11)\r\n    at MessagePort.[nodejs.internal.kHybridDispatch] (internal/event_target.js:356:9)\r\n    at MessagePort.exports.emitMessage (internal/per_context/messageport.js:18:26)\r\n```\r\n\r\n\u003c/details>\r\n\r\n\r\n\r\n\r\n\r\n\r\n### How often does it reproduce? Is there a required condition?\r\n\r\nThere is no condition.\r\n\r\n### What is the expected behavior?\r\n\r\nI'm not sure which is valid behavior, but at point of view of backward compatibilities, it would be the expected behavior that the main thread receives the unhandled primitive error as the primitive value instead of the Error object like the Node v14.6.0 behavior.\r\n\r\n### What do you see instead?\r\n\r\nThe main thread gets the Error object.\r\n\r\n### Additional information\r\n\r\n\u003c!--\r\nTell us anything else you think we should know.\r\n-->\r\n","author":{"url":"https://github.com/suin","@type":"Person","name":"suin"},"datePublished":"2020-10-05T15:23:18Z","url":"https://github.com/nodejs/node/issues/35506"}