Fixed interception on a catch-all route

[Reflex2468]

For Contributors

Fixing a bug

Fixes #72523
Related to b914ad8
Related to #51787

For Maintainers

• Fixes catch-all segments in intercepted routes.

What?

The regex replacement, also replaces ... with _. Which causes the check below, which handles catch-all segments, to always return false.

const paramName = capture.replace(/\W+/g, '_');

Why?

Currently when you have an intercepted route with a catch-all segment, it only matches the first segment. As soon as you navigate to /multiple/segments the intercept fails because the rewrite is not capturing this correctly.

How?

The changes introduced in b914ad8 added a replace(/\W+/g, '_') which causes the check below to always fail, since if it started with ... that would at that point be replaced with _.

Todo

• write tests to verify the return value of rewrites has :path* instead of :path

[ijjk]

Allow CI Workflow Run

• approve CI run for commit: 050a720

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer