Skip to content

Replace static parameters in route pattern for telemetry - #2

Open
tomerqodo wants to merge 8 commits into
cursor_only-issues-20260113-cursor_completion_base_replace_static_parameters_in_route_pattern_for_telemetry_pr20from
cursor_only-issues-20260113-cursor_completion_head_replace_static_parameters_in_route_pattern_for_telemetry_pr20
Open

tomerqodo wants to merge 8 commits into
cursor_only-issues-20260113-cursor_completion_base_replace_static_parameters_in_route_pattern_for_telemetry_pr20from
cursor_only-issues-20260113-cursor_completion_head_replace_static_parameters_in_route_pattern_for_telemetry_pr20

Conversation

@tomerqodo

@tomerqodo tomerqodo commented Jan 20, 2026 •

Copy link
Copy Markdown

Benchmark PR from qodo-benchmark#20


Note

Improves route diagnostics/debug output by substituting required values into route patterns and refining constraint rendering.

  • Add RoutePatternDebugStringFormatter to format RoutePattern debug strings by replacing parameters with matching RequiredValues, preserving literals, constraints, and leading slashes; returns / when empty
  • Change RoutePattern.DebuggerToString() to use the new formatter; minor cleanup (remove LINQ and unused separator constant)
  • Improve RoutePatternParameterPart.DebuggerToString() to prefer constraint.Content, render RegexRouteConstraint as regex(pattern), and fall back to policy type when needed
  • Test Add comprehensive unit tests for formatter behaviors (substitutions, catch-alls, literals, constraints, leading slash) and update MVC EndpointMetadataProviderTest expected route to reflect substituted values

Written by Cursor Bugbot for commit 4404a33. Configure here.

@tomerqodo

Copy link
Copy Markdown
Author

@cursor review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants