Skip to content

perf: remove unused query runtime - #11

Merged
effinrich merged 2 commits into
mainfrom
effinrich-performance-quick-win
Oct 3, 2026
Merged

effinrich merged 2 commits into
mainfrom
effinrich-performance-quick-win

Conversation

@effinrich

Copy link
Copy Markdown
Owner

Summary

  • remove unused React Query and TanStack Router SSR Query runtime integration
  • shrink the initial client entry from 364,075 B to 337,151 B (−7.4%)
  • shrink gzip size from 114,542 B to 108,250 B (−5.5%)

Validation

  • bun run --filter @richtillman/web lint
  • bun run --filter @richtillman/web typecheck
  • bun run --filter @richtillman/web test (26 passed)
  • bun run --filter @richtillman/web build
  • bun install --frozen-lockfile
  • local production trace: CLS 0.04, no console errors

effinrich and others added 2 commits October 3, 2026 14:24
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings October 3, 2026 18:26
@chromatic-com

chromatic-com Bot commented Oct 3, 2026 •

Copy link
Copy Markdown

Tip

All tests passed and all changes approved!

🟢 UI Tests: 78 visual and accessibility changes accepted as baselines
🟢 UI Review: Approved by Rich Tillman
Storybook icon Storybook Publish: 79 stories published

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

Review effort: Balanced
Findings: None


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

@effinrich
effinrich merged commit cdbf511 into main Oct 3, 2026
6 checks passed
@effinrich
effinrich deleted the effinrich-performance-quick-win branch October 3, 2026 19:50
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.

2 participants