Skip to content

WIP: feat: full document hydration for solid #4341

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 137 commits into from

Conversation

brenelz
Copy link
Contributor

@brenelz brenelz commented Jun 7, 2025

Basically we want to allow a user to have full access to the html document. Alexis mentioned to me that the html tag can't be hydrated on the client so had to take a bit of a different approach of customizing the server entry.

tannerlinsley and others added 30 commits May 10, 2025 14:54
RELEASE_ALL
RELEASE_ALL
Currently, none of the packages (i.e `packages/*`) need `vite`
bundled as a `dependency`. So its been moved to the
`devDependencies` and had the version opened using
`"peerDependencies": { "vite": ">=6.0.0" }`.

Additionally, `vite` has been added as a `dependency` to
the Start end-to-end sandboxes.

In a later PR, `vite` will need to be added to all the
TanStack Start examples (i.e. `examples/**/start-*`).
…anStack#4137)

Only import `routeCode` if `node.fullPath` is available.
…t + matching server function handler route on Windows (TanStack#4146)
@brenelz brenelz changed the title feat: full document hydration for solid WIP: feat: full document hydration for solid Jun 7, 2025
@brenelz brenelz marked this pull request as draft June 7, 2025 22:37
Copy link

nx-cloud bot commented Jun 7, 2025

View your CI Pipeline Execution ↗ for commit 77f6ff8.

Command Status Duration Result
nx affected --targets=test:eslint,test:unit,tes... ❌ Failed 6m 38s View ↗
nx run-many --target=build --exclude=examples/*... ✅ Succeeded 20s View ↗

☁️ Nx Cloud last updated this comment at 2025-06-08 05:01:55 UTC

Copy link

pkg-pr-new bot commented Jun 7, 2025

More templates

@tanstack/arktype-adapter

npm i https://pkg.pr.new/TanStack/router/@tanstack/arktype-adapter@4341

@tanstack/directive-functions-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/directive-functions-plugin@4341

@tanstack/eslint-plugin-router

npm i https://pkg.pr.new/TanStack/router/@tanstack/eslint-plugin-router@4341

@tanstack/history

npm i https://pkg.pr.new/TanStack/router/@tanstack/history@4341

@tanstack/react-router

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-router@4341

@tanstack/react-router-devtools

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-router-devtools@4341

@tanstack/react-router-with-query

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-router-with-query@4341

@tanstack/react-start

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-start@4341

@tanstack/react-start-client

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-start-client@4341

@tanstack/react-start-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-start-plugin@4341

@tanstack/react-start-server

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-start-server@4341

@tanstack/router-cli

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-cli@4341

@tanstack/router-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-core@4341

@tanstack/router-devtools

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-devtools@4341

@tanstack/router-devtools-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-devtools-core@4341

@tanstack/router-generator

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-generator@4341

@tanstack/router-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-plugin@4341

@tanstack/router-utils

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-utils@4341

@tanstack/router-vite-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-vite-plugin@4341

@tanstack/server-functions-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/server-functions-plugin@4341

@tanstack/solid-router

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-router@4341

@tanstack/solid-router-devtools

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-router-devtools@4341

@tanstack/solid-start

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-start@4341

@tanstack/solid-start-client

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-start-client@4341

@tanstack/solid-start-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-start-plugin@4341

@tanstack/solid-start-server

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-start-server@4341

@tanstack/start-client-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-client-core@4341

@tanstack/start-plugin-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-plugin-core@4341

@tanstack/start-server-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-server-core@4341

@tanstack/start-server-functions-client

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-server-functions-client@4341

@tanstack/start-server-functions-fetcher

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-server-functions-fetcher@4341

@tanstack/start-server-functions-server

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-server-functions-server@4341

@tanstack/valibot-adapter

npm i https://pkg.pr.new/TanStack/router/@tanstack/valibot-adapter@4341

@tanstack/virtual-file-routes

npm i https://pkg.pr.new/TanStack/router/@tanstack/virtual-file-routes@4341

@tanstack/zod-adapter

npm i https://pkg.pr.new/TanStack/router/@tanstack/zod-adapter@4341

commit: 77f6ff8

}: {
createRouter: () => TRouter
RootDocument?: any
Copy link
Contributor Author

Choose a reason for hiding this comment

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

i wasn't sure how to type this since its a Solid component but its in the core package

@@ -5,6 +5,7 @@ export interface HandlerCallback<TRouter extends AnyRouter> {
request: Request
router: TRouter
responseHeaders: Headers
RootDocument?: any
Copy link
Contributor Author

Choose a reason for hiding this comment

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

same here

@schiller-manuel schiller-manuel force-pushed the alpha branch 2 times, most recently from de916e0 to a8e467c Compare August 8, 2025 20:55
@brenelz brenelz changed the base branch from alpha to main August 8, 2025 21:25
@brenelz
Copy link
Contributor Author

brenelz commented Aug 8, 2025

This is so out of date i will just close

@brenelz brenelz closed this Aug 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.