fix: cache handler patch not matching require.resolve

[james-elicx]

If the the adapters api fixRequire patch is applied, __require is replaced with require, which results in the patch to replace the cache handler paths failing to match.

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/@opennextjs/cloudflare@1

npm i https://pkg.pr.new/@opennextjs/aws@1

commit: db9b613

[vicb]

Is this really required for the adapters API?

We used to do that to override the cache handler.
But with the adapters API, we should be able to set the cache handler(s) directly in the the Next config

[vicb]

I'm not against merging that if it helps for now but there should be a follow up issue to clean that up.

But I think this is already handled at https://github.com/opennextjs/opennextjs-cloudflare/blob/adapters-api/packages/cloudflare/src/cli/adapter.ts#L80

[james-elicx]

To be honest I didn't go far enough to check that, it was just from when I was seeing if the e2es still ran with no changes. I'll close this if the thinking is that cache handler will no longer be an issue once adapters api is used properly.