Skip to content

🐛 Fix CodeLens route matching and nested router discovery #20

Merged
savannahostrowski merged 5 commits intomainfrom
fix-codelens-discovery
Jan 20, 2026
Merged

🐛 Fix CodeLens route matching and nested router discovery #20
savannahostrowski merged 5 commits intomainfrom
fix-codelens-discovery

Conversation

@savannahostrowski
Copy link
Member

@savannahostrowski savannahostrowski commented Jan 20, 2026

This PR fixes a grab bag of bugs related to CodeLen route matching and nested router discovery:

  • CodeLens now appears for paths like f"{settings.API_V1_STR}/apps/{app.id}/tokens"
  • Routers imported with aliases (from .tokens import router as tokens_router) are now discovered correctly
  • Query strings in test paths no longer cause incorrect route matching
  • Routers that include other routers (router.include_router(child_router)) now have their nested routes discovered
  • Fixed import resolution incorrectly looking for files instead of returning the module containing the variable

@savannahostrowski savannahostrowski added the bug Something isn't working label Jan 20, 2026
@savannahostrowski savannahostrowski merged commit 26d2325 into main Jan 20, 2026
6 of 7 checks passed
@savannahostrowski savannahostrowski deleted the fix-codelens-discovery branch January 20, 2026 23:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant