nodejs / diagnostics

Node.js Diagnostics Working Group
MIT License
537 stars 78 forks source link

TC39 proposal AsyncContext #608

Open legendecas opened 1 year ago

legendecas commented 1 year ago

The TC39 proposal AsyncContext https://github.com/tc39/proposal-async-context has reached Stage 2 today. I'd like to bring the topic up again at the diagnostics meeting to discuss the integration of Node.js with the proposal and the next steps for it.

This is a revival of https://github.com/nodejs/diagnostics/issues/375.

Related:

mhdawson commented 1 year ago

Good to see that progressing in TC39.

legendecas commented 1 year ago

During the May 3rd TC39 AsyncContext call, there were no objections to using the reject's call time context for unhandledRejection: https://github.com/tc39/proposal-async-context/issues/16#issuecomment-1531941423.

We would like to know if it is feasible for Node.js to change the context of the unhandledRejection event and if there are any concerns regarding the decision.

I would like to bring this up to the next diag call and @jridgewell would like to join too.

Qard commented 1 year ago

Will you be at the collab summit next week? We have a diagnostics session on the agenda so we could bring it up there if you're around. :)

legendecas commented 1 year ago

Unfortunately, I'm unable to attend the summit next week :(.

Flarna commented 1 year ago

I will be also not there. I haven't even noticed that there is a summit and a diagnostics session... Seems I monitor the wrong channels/repos.

Qard commented 1 year ago

You can watch https://github.com/openjs-foundation/summit/ for future summits. :)

legendecas commented 1 year ago

Untagging https://github.com/nodejs/diagnostics/labels/diag-agenda as the issue is being discussed at https://github.com/tc39/proposal-async-context/issues/16.

github-actions[bot] commented 1 year ago

This issue is stale because it has been open many days with no activity. It will be closed soon unless the stale label is removed or a comment is made.