Skip to content

perf(console-ui): tune TanStack Router and Query#2555

Draft
malinskibeniamin wants to merge 2 commits into
masterfrom
ben-malinski/no-ticket/tanstack-router-query
Draft

perf(console-ui): tune TanStack Router and Query#2555
malinskibeniamin wants to merge 2 commits into
masterfrom
ben-malinski/no-ticket/tanstack-router-query

Conversation

@malinskibeniamin

Copy link
Copy Markdown
Contributor

Summary

  • Upgrade @tanstack/react-query and its devtools from 5.99.x/5.90.x to 5.101.2.
  • Share one Router plugin configuration between Rsbuild and Vitest, use the current Rspack plugin export, ignore test/helper route files, and emit a stable async lib-tanstack vendor chunk.
  • Enable intent preloading while setting defaultPreloadStaleTime: 0, keeping TanStack Query responsible for cache freshness.
  • Wire the existing pending and error route boundaries into both standalone and embedded routers; make the pending state natively announced to assistive technology.

This keeps the scope to the critical Router/Query integration from TanStack Router and Query. Existing awaited ensureQueryData loaders remain unchanged because they already follow the supported integration pattern.

Commits

  • 06613049d build(console-ui): upgrade tanstack query to 5.101.2
  • 1ba4ca9fd perf(console-ui): tune tanstack router integration

Screenshots / surface review

View Before After Notes
Route pending state Existing spinner and “Loading...” copy Visually unchanged Screenshot skipped: only native status semantics changed; classes and copy are unchanged. Covered by an accessibility-focused integration test.

Visual recap skipped: this is dependency/build/runtime configuration work with no visual delta.

Dependency upgrade path

  • Upgrade evidence: lockfile regenerated with Bun; no call-site migration or codemod was required for Query 5.101.2.
  • Packages: @tanstack/react-query 5.99.2 → 5.101.2; @tanstack/react-query-devtools 5.90.2 → 5.101.2.
  • SemVer confidence: same major version, release notes reviewed, full type/unit/integration/build verification completed.
  • Risk gate: Router remains on 1.170.17. The newer 1.170.18 release is below this repository's 72-hour minimum release-age gate, so this PR does not bypass the supply-chain safeguard.
  • Security notes: bun audit reports existing repository-wide advisories; none are introduced by this focused TanStack update.

Test plan

  • bun install --frozen-lockfile --ignore-scripts
  • bun run type:check
  • Ultracite check on new/changed Router configuration and tests
  • Unit suite: 57 files, 863 tests
  • Integration suite: 115 files, 1,188 tests
  • Production build
  • Bundle comparison: total gzip decreased by about 24 kB; entry bundles stayed effectively flat; TanStack dependencies now form an async cached vendor chunk

Local Node 25 requires --localstorage-file for the pre-existing topic-settings-store integration test; the isolated failure passes with that harness option.

🤖 Generated with OpenAI Codex

malinskibeniamin and others added 2 commits July 14, 2026 16:13
Co-Authored-By: OpenAI Codex <noreply@openai.com>
Co-Authored-By: OpenAI Codex <noreply@openai.com>
@malinskibeniamin malinskibeniamin added the performance Performance is one of the concerns label Jul 14, 2026
@malinskibeniamin malinskibeniamin self-assigned this Jul 14, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Clean — no registry drift, off-token colours, or ad-hoc classes

App: frontend · Scope: diff vs origin/master · Files: 9

Count
⚠️ Outdated registry components 0
🛠 Locally-modified components 0
❓ Unknown to registry 0
🎨 Off-token palette colours 0
🔢 Ad-hoc utility classes 0

Generated by lookout audit-changes.

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

Labels

performance Performance is one of the concerns

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant