🦋 Changeset detected

Latest commit: 9bdc5ad

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

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

CodSpeed Performance Report

Merging #14161 will not alter performance

_{Comparing fix/routing-validation (92632a6) with main (34e6b3a)¹}

Summary

✅ 6 untouched benchmarks

What would be the behaviour now, if one of these invalid redirects was created?

As mentioned in the bug report, the error is generic:

[ERROR] [build] Failed to call getStaticPaths for /categories/[category]
[GetStaticPathsRequired] `getStaticPaths()` function is required for dynamic routes. Make sure that you `export` a `getStaticPaths` function from your dynamic route.

This still doesn't test for the example in the issue, and I'm not sure it would catch it. The original had the same number of dynamic segments on each side:

redirects: {
  '/categories/[category]': '/category/[category]/1',
}

Sorry, I missed that. I'll add a test for that too

@ematipico what are your plans for this PR?

@florian-lefebvre It requires some thinking about how to catch the edge case and throw an error. Unfortunately I'm focused on other things, and jumping on this one again would require some time to fix

Closing due to inactivity, I assume that with Astro 6 this might require some code change as well.