drop the AbortController patch

[vicb]

This patch was added in #441

The AbortController is meant for the client side but somehow ends up in the server bundler. It was safe to remove it - workerd errors when instantiating an AbortController at top level.

After #497, no more Next code is executed at top level so it is safe to remove the patch.

[changeset-bot]

⚠️ No Changeset found

Latest commit: d0ca42c

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

[pkg-pr-new]

Open in StackBlitz

pnpm add https://pkg.pr.new/@opennextjs/cloudflare@519

commit: d0ca42c

[conico974]

LGTM.
We should probably keep a small comment somewhere though about this with the link to the workerd issue

[vicb]

LGTM. We should probably keep a small comment somewhere though about this with the link to the workerd issue

I was wondering if we should keep the patch (but not use it) and maybe remove it later.

WDYT?

[conico974]

Yeah we can keep it for now and remove it once the fix is made in workerd, we could just add a small comment to explain why it's not needed anymore

[vicb]

I update the PR:

• restored the rule and added comments on the rule
• stop applying the patch for now

[vicb]

Thanks Nico!