pmcelhaney / counterfact / 22882848157 / 1

83%

main: 87%

Pull #1524

github

Fix root middleware not being executed for all routes

The recurse() function in registry.ts traversed paths as:
/admin/users → /admin → "" → null, never checking "/".
Root middleware is registered at key "/" by the module-loader,
so it was never applied.

Fix: change nextPath calculation to traverse through "/" instead
of "", so traversal goes: /admin/users → /admin → / → null.

Also update tests to use "/" key for root middleware (consistent
with how module-loader registers it), and add a test specifically
for the root middleware case described in the issue.

Co-authored-by: pmcelhaney <51504+pmcelhaney@users.noreply.github.com>

1422 of 1583 branches covered (89.83%)

Branch coverage included in aggregate %.

8106 of 10000 relevant lines covered (81.06%)

30.02 hits per line