fix: handle outdated message in channel queue

[jedlikowski]

Fixes #183

[changeset-bot]

⚠️ No Changeset found

Latest commit: 9f59e96

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

[JounQin]

What's the issue here? Can you provide more information? Reproduction or failing test for example.

[jedlikowski]

Hi, sorry, it was meant to be a draft PR ;) I've linked in the description to the issue this PR means to fix.

[jedlikowski]

Hi, I've added one more change to avoid sending the worker task result to parent if parent isn't waiting for the result anymore. Let me know what you think of this PR and I'll be happy to carry it to the end :)

I also wondered how we could pass some sort of EventEmitter to the task function itself, to let it know that the task has been aborted. This would allow it for example to cancel a HTTP request, but I couldn't figure out a way to do it without changing the library API.

This chart aims visualize the current behavior (dotted line means Atomics.wait() timeout on parent process):

While this one shows the updated behavior:

[jedlikowski]

Hi, @JounQin, what do you think of this PR? Would you be interested in such a contribution?