fix: correct route path for MenuEntry

[tltv]

MenuEntry should not have route parameters included in the path for client routes. It should omit route path in same way as with Hilla automatic menu. Menu should not include nested routes for excluded menu item or for route with required parameter.

[sonarcloud]

Quality Gate passed

Issues
4 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[github-actions]

Test Results

1 142 files  ± 0  1 142 suites  ±0   1h 26m 29s ⏱️ -44s
7 459 tests ± 0  7 409 ✅ ± 0  50 💤 ±0  0 ❌ ±0 
7 781 runs   - 47  7 721 ✅  - 47  60 💤 ±0  0 ❌ ±0 

Results for commit a006792. ± Comparison against base commit 4924daa.

[caalador]

Does this change break Hilla?
I feel the filtering should be in MenuConfiguration and not in MenuRegistry as at the moment registry returns what Hilla is expecting.

[tltv]

Does this change break Hilla?
I feel the filtering should be in MenuConfiguration and not in MenuRegistry as at the moment registry returns what Hilla is expecting.

It doesn't because Hilla uses collectClientMenuItems(...). This change affects only collectMenuItems() and collectMenuItemsList().