Fix logger scope for worker

[jcapcik]

Description

This change fixes #418, which was a simple bug introduced where the workerId was no longer being set on the logger scope (it was being set on a nested scope property), which resulted in the default logger not outputting as expected, and the logFactory function not being passed the values as defined in the documentation.

When using the default logger, the output before would be something like this:

[job] INFO: Starting task

After:

[job(worker-2c04dcd6ba5728c822: someTask{1})] INFO: Starting task

Performance impact

None

Security impact

None

Checklist

• My code matches the project's code style and yarn lint:fix passes.
• I've added tests for the new feature, and yarn test passes.
• I have detailed the new feature in the relevant documentation.
• I have added this feature to 'Pending' in the RELEASE_NOTES.md file (if one exists).
• If this is a breaking change I've explained why.

[benjie]

Great catch; thanks!