fix(routing): keep scroll on search-param updates

[chiptus]

Adds resetScroll: false to search-param-only replace navigations (vote sort-lock, vote/timeline/admin filters) so they no longer scroll the window to top. Closes #386; the invite-flow site from that audit was already refactored into a real page navigation and needs no change.
Also guards the global router.invalidate() so Supabase's refocus auth events (same user) no longer re-run all route loaders — which could swap in the pending fallback and reset scroll on tab refocus.

Verification

• Scroll down the sets list and vote on a set (first vote, unsorted-locked state); the list stays in place and the sort locks.
• Scroll down the schedule list view and toggle a day/stage/vote filter chip; no jump to top.
• In admin artists, scroll down and change sort/filter; no jump to top.
• Click "clear filters" on the vote tab; page scrolls to top (intentional, unchanged).
• While logged in, scroll down, switch to another tab and back; page stays in place. Sign out and back in; route data still refreshes.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
upline	Ready	Preview	Aug 31, 2026 5:57pm

[Copilot]

🟢 Approval recommended

The changes are narrowly scoped, consistent with existing resetScroll: false usage in the codebase, and preserve intentional scroll-reset behavior on clear-filters actions.

Pull request overview

This PR addresses an app-wide UX issue where search-param-only replace navigations in TanStack Router reset the window scroll position to the top. It updates the relevant URL-state hooks so filter/sort/vote state changes no longer cause unintended scroll jumps, aligning with the fix pattern previously applied for timeline scroll sync.

Changes:

• Add resetScroll: false to search-param-only replace navigations for admin artists filter/sort updates.
• Add resetScroll: false to the generic sets-page URL-state update helper (used for vote sort-lock / filters).
• Add resetScroll: false to timeline/list schedule filter updates (day/time/stages/votes/types).

File summaries

File	Description
src/pages/admin/ArtistsManagement/hooks/useAdminArtistsUrlState.ts	Prevents scroll-to-top on admin artists search-param updates by disabling scroll reset.
src/hooks/useUrlState.ts	Prevents scroll-to-top on sets page search-param updates while preserving existing “clear filters” scroll behavior.
src/hooks/useTimelineUrlState.ts	Prevents scroll-to-top on schedule timeline/list filter chip updates by disabling scroll reset.

Review details

• Files reviewed: 3/3 changed files
• Comments generated: 0
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[github-actions]

Playwright test results

 65 passed

Details

 65 tests across 21 suites
 1 minute, 29 seconds
 cab2bb9

[Copilot]

🟢 Approval recommended

The focused navigation options correctly address the reported behavior and follow the established fix.

Review details

• Files reviewed: 3/3 changed files
• Comments generated: 0 new
• Review effort level: Balanced

[Copilot]

🟢 Approval recommended

The targeted navigation sites are covered and the auth guard preserves invalidation when user identity changes.

Review details

• Files reviewed: 4/4 changed files
• Comments generated: 0 new
• Review effort level: Balanced