fix: partially reverts 644

[vicb]

No description provided.

[changeset-bot]

🦋 Changeset detected

Latest commit: 979ea33

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

This PR includes changesets to release 3 packages

Name	Type
@opennextjs/aws	Patch
app-pages-router	Patch
app-router	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]

Coverage Report

Status	Category	Percentage	Covered / Total
🔵	Lines	22.91%	1723 / 7519
🔵	Statements	22.91%	1723 / 7519
🔵	Functions	56.59%	103 / 182
🔵	Branches	71%	436 / 614

File Coverage

File	Stmts	Branches	Functions	Lines	Uncovered Lines
Changed Files
packages/open-next/src/adapters/middleware.ts	0%	0%	0%	0%	1-99
packages/open-next/src/core/requestHandler.ts	0%	0%	0%	0%	1-239
packages/open-next/src/core/routingHandler.ts	0%	0%	0%	0%	1-238
packages/open-next/src/core/routing/middleware.ts	96.89%	85.29%	50%	96.89%	34-37, 120
packages/open-next/src/overrides/converters/aws-cloudfront.ts	77.45%	80.76%	100%	77.45%	103-106, 151-191
packages/open-next/src/overrides/converters/edge.ts	0%	0%	0%	0%	1-113
packages/open-next/src/overrides/wrappers/cloudflare-edge.ts	0%	100%	100%	0%	8-70
packages/open-next/src/types/open-next.ts	0%	0%	0%	0%

Generated in workflow #802 for commit 979ea33 by the Vitest Coverage Report Action

[pkg-pr-new]

Open in Stackblitz

pnpm add https://pkg.pr.new/@opennextjs/aws@660

commit: 979ea33

[vicb]

Also doing some renaming, no strong opinion, open to anything better

[conico974]

LGTM Thanks

[conico974]

It is failing to build https://github.com/opennextjs/opennextjs-aws/actions/runs/12135452530/job/33834536340?pr=660

[vicb]

fixed!
Thanks @conico974 for all the reviews this morning 🙏

[conico974]

LGTM Thanks