Skip to content

fix(analyze): flag unstable onRouteChange calls - #51

Open
lntutor wants to merge 1 commit into
askrjs:mainfrom
lntutor:fix/on-route-change-analysis
Open

fix(analyze): flag unstable onRouteChange calls#51
lntutor wants to merge 1 commit into
askrjs:mainfrom
lntutor:fix/on-route-change-analysis

Conversation

@lntutor

@lntutor lntutor commented Jul 30, 2026

Copy link
Copy Markdown

Closes #41

Summary

  • include onRouteChange in the render-scoped lifecycle catalog
  • report module-scope and conditional calls through the existing askr/stable-render-call rule
  • add focused analyzer regression coverage

Verification

  • npm run check (201 tests passed)
  • git diff --check

@lntutor
lntutor requested a review from smiggleworth as a code owner July 30, 2026 13:18
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.

analyze: onRouteChange() is render-scoped but missing from RENDER_SCOPED_CONCEPTS

1 participant