Skip to content

React router fixes and production server fixes#270

Merged
AlexanderKaran merged 16 commits into
mainfrom
react-router-fixes
Jun 28, 2026
Merged

React router fixes and production server fixes#270
AlexanderKaran merged 16 commits into
mainfrom
react-router-fixes

Conversation

@AlexanderKaran

@AlexanderKaran AlexanderKaran commented Jun 28, 2026

Copy link
Copy Markdown
Collaborator
  • Ensure each frameworks uses its official production server
  • Also made sure the CSR harness returns 404 for missing assets
  • Updated React Router to use ClientLoader instead, which makes the SPA build not needed
  • Type check for React Router
  • As TanStack is now the only app using a SPA build, set it up to be like others and left a comment in methodology. Still renderes client side though

Sorry the large PR had to merge to due to a few server bugs that cropped up in the CI run

Few more edits:

  • Set production server to production
  • Moved handlers for throughput test into their own things so it clearer. I know we might remove this one but makes it easier if we do too
  • Temp removed Mastro from SSR tests will fix up in its own PR

@AlexanderKaran AlexanderKaran requested a review from 43081j as a code owner June 28, 2026 08:15
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 28, 2026

Copy link
Copy Markdown

Deploying framework-tracker with  Cloudflare Pages  Cloudflare Pages

Latest commit: 81f82af
Status: ✅  Deploy successful!
Preview URL: https://c1d296cf.framework-tracker.pages.dev
Branch Preview URL: https://react-router-fixes.framework-tracker.pages.dev

View logs

@AlexanderKaran AlexanderKaran changed the title React router fixes React router fixes and production server fixes Jun 28, 2026
Comment thread packages/stats-generator/src/serve/react-router.ts Outdated
@AlexanderKaran AlexanderKaran merged commit ee79b7a into main Jun 28, 2026
4 checks passed
@AlexanderKaran AlexanderKaran deleted the react-router-fixes branch June 28, 2026 13:09
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