Skip to content

fix: disable overlapping route view transitions - #19

Merged
slate-rehm merged 1 commit into
masterfrom
feature/fix/disable-route-view-transitions
Aug 1, 2026
Merged

slate-rehm merged 1 commit into
masterfrom
feature/fix/disable-route-view-transitions

Conversation

@slate-rehm

@slate-rehm slate-rehm commented Aug 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Disable TanStack Router view transitions to prevent overlapping updates from throwing unhandled InvalidStateError exceptions.
  • Keep route changes as standard React transitions.

Testing

  • Not run

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

TanStack Router disables browser view transitions. Route changes use standard React transitions. This prevents overlapping updates from throwing unhandled InvalidStateError exceptions.

Tests were not run.

- Prevent unhandled InvalidStateError during route changes
@coderabbitai

coderabbitai Bot commented Aug 1, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a5cad3ba-6b26-4449-a081-1e447a1ca335

📥 Commits

Reviewing files that changed from the base of the PR and between 834cd14 and d009af9.

📒 Files selected for processing (1)
  • src/router.tsx

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


📝 Walkthrough

Walkthrough

TanStack Router now sets defaultViewTransition to false in its default configuration.

Changes

Router view-transition configuration

Layer / File(s) Summary
Disable default view transitions
src/router.tsx
The router changes defaultViewTransition from true to false.

Estimated code review effort: 1 (Trivial) | ~2 minutes

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch

Comment @coderabbitai help to get the list of available commands.

@slate-rehm
slate-rehm merged commit 12af216 into master Aug 1, 2026
2 of 4 checks passed
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.

1 participant