fix(cloudflare-pages): use predefined wildcards in routes.exclude
#2319
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
π Linked issue
β Type of change
π Description
When using Cloudflare Pages with Nitro all static files need to be present in the _routes.json file's
exclude
array.Since only 100 entries are allowed in the file, projects can use wildcards to reduce the number entries (eg. excluding
"/_nuxt/*"
excludes all subpaths).Nitro handles this well for
nitro.options.publicAssets
by adding the wildcard path and then not adding the individual files matching it. But if you just add a wildcard tonitro.options.cloudflare.pages.routes.exclude
it still includes the individual files like so:That's what this PR fixes. It handles both cases using
routes.exclude
(thatexplicitPublicAssets
has already been appended to) for the globby ignore list, replacing the"/*"
syntax used on Cloudflare Pages with the"/**"
syntax that globby uses.I added a test that fails without the fix.
π Checklist