Reduce amount of rules in routes json file

[veitbjarsch]

Changes

Cloudflare has a limit of 100 includes/exclude rules. In many applications this amount is reached quite fast.
This PR add the functionality to compare include and exclude rules and reduce the amount of rules wherever possible.
Therefore it:

• adds a wildcard if a folder only belogs to include or exclude rules
• skips all subfolders where a wildcard is already present (leave them untouched)

Testing

Tested in out own project using cloudflare as well wrote a unit test here.

Docs

[changeset-bot]

🦋 Changeset detected

Latest commit: 342d0a3

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

This PR includes changesets to release 10 packages

Name	Type
@astrojs/cloudflare	Minor
@test/astro-cloudflare-astro-dev-platform	Patch
@test/astro-cloudflare-astro-env	Patch
@test/astro-cloudflare-compile-image-service	Patch
@test/astro-cloudflare-external-image-service	Patch
@test/astro-cloudflare-wasm	Patch
@test/astro-cloudflare-no-output	Patch
@test/astro-cloudflare-routes-json	Patch
@test/astro-cloudflare-with-solid-js	Patch
@test/astro-cloudflare-wrangler-preview-platform	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

[alexanderniebuhr]

@veitbjarsch Thank you for this contribution, I'll check it later to see if it works with all requirements outlined in this issue #374

[alexanderniebuhr]

adds a wildcard if a folder only belogs to include or exclude rules

I have to check this internally, but I think we can't tell this upfront, either in all cases or in specific cases.

skips all subfolders where a wildcard is already present (leave them untouched)

That's a good addition, we need to see how it handles the routes if you have all static in one folder and then one dynamic route in a nested subfolder. I'll take a look at your testcase.

[veitbjarsch]

@alexanderniebuhr Thanks for your fast response.

skips all subfolders where a wildcard is already present (leave them untouched)

That's a good addition, we need to see how it handles the routes if you have all static in one folder and then one dynamic route in a nested subfolder. I'll take a look at your testcase.

The code compares both includes and excludes in 2 ways. This means if the same folder (partial path) is present in both PathTrie objects, it will leave them untouched. Only if a folder is not present in the other PathTrie object we can savely assume that all children belongs to the current PathTrie. Only in this case we add the wildcard flag for this folder.

This way we can reduce the amount of rules in our project by more than 50% automatically, without the need to create and handle the routes.jsnon manually.

[alexanderniebuhr]

Sounds good, I did a quick test and it seems to be working with all the cases. I'll give it a proper review later :)

[veitbjarsch]

Thanks

[alexanderniebuhr]

@veitbjarsch can you add an changeset with pnpm exec changeset

[veitbjarsch]

@alexanderniebuhr added the changeset.

[veitbjarsch]

@alexanderniebuhr Hey, is it planned to merge this PR into the main branch?