watch: track worker thread entry files in --watch mode

[SudhansuBandha]

Currently, --watch mode only tracks dependencies from the main module graph (require/import). Worker thread entry points created via new Worker() are not included, so changes to worker files do not trigger restarts.

This change hooks into Worker initialization and registers the worker entry file with the watch mode, ensuring restarts when worker files change.

Fixes: #62275

[clemyan]

This solves the issue for the worker module itself, but not its dependencies. I.e. if a worker module imports/requires another module, modifying the latter still does not trigger re-run.

[SudhansuBandha]

I have updated changes to watch dependencies for worker as well, so changes in imported/required modules will now trigger a re-run.
@clemyan

[clemyan]

This is the wrong way to solve this, for a myriad of reasons: (based on a cursory glance of the code. Did not bother to actual test it)

1. It is impossible to properly parse JS with regex. This approach cannot even distinguish if the "import" is part of a string or a comment
2. It only works for direct dependencies of the worker module, not transitively. (I.e. it does not watch dependencies of dependencies, and their dependencies, etc.)
3. It only works for literals specifiers that are relative paths, not for absolute paths require('/path/to/file.js'), packages require('my-pkg/sub/path'), dynamic specifier require('./messages/'+language+'/string.json'), or paths that are resolved by customization hooks
4. What about the worker module itself? If it is loaded by customization hooks, there isn't necessarily a file on disk to readSync from.

Any solution must be integrated into Node's loading pipeline

[SudhansuBandha]

I will revert back these updates and work on the suggestions
@clemyan

[franciscoarturorivera371-cyber]

Submit the change.

[SudhansuBandha]

@clemyan
I have updated handling of dependencies from worker thread. For this I have also reverted my changes from the initial commit as well f9beab0
I have implemented the solution for both cjs and esm files.
Do let me know your feedback on the proposed solution and whether it makes sense to proceed further in this direction.

[aduh95]

The added test already passes on main, that means either the code change is not necessary, or that it's not actually testing what you're trying to achieve

[SudhansuBandha]

Sure!! @aduh95 I will update the tests for this. Thanks for your feedback!!

[SudhansuBandha]

@aduh95 Please do review the updated tests for the proposed changes based on behaviour of worker file and it's dependencies in --watch mode for both cjs and esm modules.

[aduh95]

Can you not put the tests in test/sequential/test-watch-mode.mjs? That file is enormous, which is becoming a maintenance issue/

[SudhansuBandha]

@aduh95 As suggested I have removed tests from test/sequential/test-watch-mode.mjs to a new file and made few lint fixes.
Do let me know if any further changes are required