Bind context to async execution avoiding race-conditions

[enisdenjo]

Like when using graphql-modules with an HTTP server.

[changeset-bot]

🦋 Changeset detected

Latest commit: 0491f4c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
graphql-modules	Patch

Not sure what this means? Click here to learn what changesets are.

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

[github-actions]

🚀 Snapshot Release (alpha)

The latest changes of this PR are available as alpha on npm (based on the declared changesets):

Package	Version	Info
graphql-modules	2.4.1-alpha-20250121160438-0491f4c48e11d9b869a28a3d5dc1f2c339cfdb5c	npm ↗︎ unpkg ↗︎

[github-actions]

💻 Website Preview

The latest changes are available as preview in: https://8faf6ca0.graphql-modules.pages.dev

[Intellicode]

@enisdenjo thanks for fixing this! Is there anything blocking preventing this to be released?

[ardatan]

You dont have to wait.
You can start using the alphas here
#2521 (comment)

[enisdenjo]

@Intellicode there are some benchmarks that got degraded due to this change (which is expected since async_hooks do slow down the execution), there's also removing the dependency on Node (both module and async_hooks cant be imported on root).

Then this can land.

[Intellicode]

You dont have to wait. You can start using the alphas here #2521 (comment)

We indeed tested the alpha build to confirm the absence of the race condition (in our setup), however for production usage it would be nice to have an official release if possible.

@Intellicode there are some benchmarks that got degraded due to this change (which is expected since async_hooks do slow down the execution), there's also removing the dependency on Node (both module and async_hooks cant be imported on root).

Thanks for the update!