[Bug]: DelayError throwing in Child Processor breaks without Message

[jpulec]

Version

v5.29.1

Platform

NodeJS

What happened?

Description

Throwing DelayedError doesn't work properly with child processor sandboxing, if you don't pass a message to the thrown error. This appears to be because when sending a message to the parent that a child failed, it checks if there's an err.message property, and if not, wraps the error with new Error(err) which causes the name property to be overwritten as Error so that the checks for err.name === 'DelayedError in the Worker class are incorrect.

How to reproduce.

// worker.js
import { pathToFileURL } from 'url';

const processorUrl = pathToFileURL(__dirname + '/my_processor.js');

worker = new Worker(queueName, processorUrl);

// my_processor.js
import { SandboxedJob } from 'bullmq';

module.exports = async (job: SandboxedJob) => {
   // Note the absense of a passed in message here
    throw new DelayedError();
};

Relevant log output

No response

Code of Conduct

• I agree to follow this project's Code of Conduct