Skip to content

fix(init): detect TanStack Start instead of treating it as a Vite SPA - #828

Open
kushals256 wants to merge 6 commits into
reticlehq:mainfrom
kushals256:fix/init-tanstack-start
Open

fix(init): detect TanStack Start instead of treating it as a Vite SPA#828
kushals256 wants to merge 6 commits into
reticlehq:mainfrom
kushals256:fix/init-tanstack-start

Conversation

@kushals256

@kushals256 kushals256 commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • reticle init classified @tanstack/react-start / @tanstack/start as Vite, wired the plugin, and reported every step green ("also injects connect()"). Start SSRs <html> from __root.tsx and never sends Vite's index.html, so the transform never runs and no session appears.
  • Start is now detected before Vite (not Query or Router alone). The plugin is kept with inject: false so source stamps still land; connect is a printed client-only useEffect (a static import on that module SSRs and 500s). The plan says this path is ungated.
  • Feedback stack detection names tanstack-start rather than collapsing it into react/vite.

Fixes #773

Test plan

  • packages/server unit tests: detect Start vs Query/Router/Next; plan is MANUAL connect + inject: false plugin write; connect-step title fails init; unverified NOTICE
  • pnpm --filter @reticlehq/server test:unit (6554 passed locally; the two docs-em-dash failures were from this branch and are fixed)
  • Reviewer: reticle init --dry-run on a TanStack Start app should not report Vite / "also injects connect()"

kushals256 and others added 2 commits September 7, 2026 12:51
Start SSRs its own HTML, so the plugin's connect injection never fires.
Classifying it as Vite reported a green install over an app that could
never connect. Detect Start before Vite, keep the plugin with inject:
false so stamps still land, and print a client-only connect rather than
writing a static import that SSRs and 500s.

Signed-off-by: Kushal S <skushal.mys@gmail.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: Kushal S <skushal.mys@gmail.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@divshekhar
divshekhar enabled auto-merge (squash) September 8, 2026 09:16
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.

init classifies TanStack Start as a Vite SPA: connect() never runs, the install reports green, the daemon waits forever

2 participants