updateProgress() should be more permissive on progress type.

[nullndr]

Currently Job.updateProgress() is declared as:

/**
 * MinimalJob
 */
export interface MinimalJob<
  DataType = any,
  ReturnType = any,
  NameType extends string = string,
> {
  ...
  /**
   * Updates a job's progress
   *
   * @param progress - number or object to be saved as progress.
   */
  updateProgress(progress: number | object): Promise<void>;
  ...
}

Is there a specific reason for which progress must be a number | object?

From what I can see here progress is serialized using JSON.stringify(), so would it be possible to support other types like string in order for the job to emit simpler messages?

[manast]

Yes, it could be a string too. As long as it is JSON serialisable it will work.

[nullndr]

Are you open to a PR?

[nullndr]

I am wondering if progress should be just unknown, that it is the type returned from JSON.parse().

In this way you are obligated to check the type in the worker.on("progress", callback) callback

[manast]

No, the proper type should be something that represents JSON serializability, using unknown will not really improve things. If you just add "string" as possible type I think we are good for now.

[nullndr]

Closing after #3187