refactor: minor updates

[vicb]

No description provided.

[changeset-bot]

⚠️ No Changeset found

Latest commit: 322194c

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

[github-actions]

Coverage Report

Status	Category	Percentage	Covered / Total
🔵	Lines	22.4%	1657 / 7397
🔵	Statements	22.4%	1657 / 7397
🔵	Functions	56.32%	98 / 174
🔵	Branches	70.42%	412 / 585

File Coverage

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

Generated in workflow #755 for commit 322194c by the Vitest Coverage Report Action

[pkg-pr-new]

Open in Stackblitz

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

commit: 322194c

[conico974]

Just a little comment, other than that LGTM

[conico974]

We could do it in 1 filter instead of 2, it avoids having to iterate over everything twice

[vicb]

yes we could.
however I like to have "simple" filters.
I don't think the time lost iterating a list is significant vs i/o (copy the files).

Happy to change if you feel strongly about this one.

[conico974]

Ok fine for me then, i don't have that strong of an opinion on this