trace node packages

[vicb]

In the cloudflare adapter, we want to copy files used via the "workerd" build conditions.

To be able to do this this PR updates copyTracedFiles to return a map of traced node packages.
It maps the original path of the package to the path in .open-next.

As a follow up, we might integrate that better with the patch system.
Today, applyCodePatches assumes that a patch will update file content.
We could modify applyCodePatches to take a list of Patch.
The Patch class would be instantiated with options, tracedFiles, manifests, nodePackages, nextVersion and have a public apply method.
We could then have derived PatchFileContent, PatchCopyFile, ... classes.

[changeset-bot]

🦋 Changeset detected

Latest commit: b218400

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

[pkg-pr-new]

Open in StackBlitz

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

commit: b218400

[github-actions]

Coverage Report

Status	Category	Percentage	Covered / Total
🔵	Lines	26.54%	2465 / 9287
🔵	Statements	26.54%	2465 / 9287
🔵	Functions	52.86%	129 / 244
🔵	Branches	73.4%	599 / 816

File Coverage

File	Stmts	Branches	Functions	Lines	Uncovered Lines
Changed Files
packages/open-next/src/build/copyTracedFiles.ts	0%	0%	0%	0%	1-370

Generated in workflow #1162 for commit b218400 by the Vitest Coverage Report Action

[conico974]

As a follow up, we might integrate that better with the patch system.
Today, applyCodePatches assumes that a patch will update file content.
We could modify applyCodePatches to take a list of Patch.
The Patch class would be instantiated with options, tracedFiles, manifests, nodePackages, nextVersion and have a public apply method.
We could then have derived PatchFileContent, PatchCopyFile, ... classes.

That's a good idea yeah.
One other improvements that would be great to have would be for ast-grep to avoid parsing the same files multiple times, (but i think it involves changes in ast-grep itself)

[conico974]

LGTM Thanks

[vicb]

Thanks for the review Nico!
When do you plan the next release?

[conico974]

@vicb Later today, just let me know if you want to add something in it