fix(miniflare): Fix Durable Object RPC calls from Node.js blocking the event loop#11663
fix(miniflare): Fix Durable Object RPC calls from Node.js blocking the event loop#11663
Conversation
🦋 Changeset detectedLatest commit: ab328ee The changes in this PR will be included in the next version bump. 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 |
|
Claude finished @NuroDev's task in 36s —— View job Changeset Review
|
create-cloudflare
@cloudflare/kv-asset-handler
miniflare
@cloudflare/pages-shared
@cloudflare/unenv-preset
@cloudflare/vite-plugin
@cloudflare/vitest-pool-workers
@cloudflare/workers-editor-shared
@cloudflare/workers-utils
wrangler
commit: |
petebacondarwin
left a comment
There was a problem hiding this comment.
Don't merge this unless someone else also approves, as this is a hairy bit of the code base - but LGTM!
Co-authored-by: Pete Bacon Darwin <pete@bacondarwin.com>
ascorbic
left a comment
There was a problem hiding this comment.
For the same reason as Pete I won't give this a final approval as I don't know that part of the codebase at all, but the code itself looks good.
Co-authored-by: Dario Piotrowicz <dario@cloudflare.com>
dario-piotrowicz
left a comment
There was a problem hiding this comment.
Lol I've reviewed the changes but I'm in the same boat as Pete and Matt 😅
If this works locally anyways and passes all the tests I think it'd be safe to merge 🙂
…ng the event loop (#11664) * Add Miniflare logic to mark DO stubs as async * Added tests to check for blocking DO stubs * Added changeset * Updated backport tests * Updated changeset description

Fixes #11122.
This fixes Durable Object RPC calls from Node.js blocking the event loop, preventing
Promise.race()and timeouts from working correctly.A picture of a cute animal (not mandatory, but encouraged)