Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions e2e/solid-start/basic-cloudflare/src/routes/index.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
import { createFileRoute } from '@tanstack/solid-router'
import { createServerFn } from '@tanstack/solid-start'
import { getRequestHeader } from '@tanstack/solid-start/server'
import { env } from 'cloudflare:workers'

export const Route = createFileRoute('/')({
Expand All @@ -11,6 +12,7 @@ const getData = createServerFn().handler(() => {
return {
message: `Running in ${navigator.userAgent}`,
myVar: env.MY_VAR,
requestHeader: getRequestHeader('x-solid-start-context'),
}
})

Expand All @@ -22,6 +24,7 @@ function Home() {
<h3>Welcome Home!!!</h3>
<p data-testid="message">{data().message}</p>
<p data-testid="myVar">{data().myVar}</p>
<p data-testid="requestHeader">{data().requestHeader}</p>
</div>
)
}
10 changes: 10 additions & 0 deletions e2e/solid-start/basic-cloudflare/tests/app.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,16 @@ test('returns the correct value from a Cloudflare binding', async ({
await expect(page.getByTestId('myVar')).toHaveText('Hello from Cloudflare')
})

test('preserves request context with nodejs_als', async ({ page }) => {
await page.setExtraHTTPHeaders({
'x-solid-start-context': 'request-context-works',
})
await page.goto('/')
await expect(page.getByTestId('requestHeader')).toHaveText(
'request-context-works',
)
})

test('prerender with Cloudflare Workers runtime', async ({ page }) => {
// Verify the static page was prerendered during build
const distDir = join(process.cwd(), 'dist', 'client')
Expand Down
10 changes: 2 additions & 8 deletions e2e/solid-start/basic-cloudflare/worker-configuration.d.ts
Original file line number Diff line number Diff line change
@@ -1,18 +1,12 @@
/* eslint-disable */
// Generated by Wrangler by running `wrangler types` (hash: b11df627d8b3c51b1bf3230a546b0f20)
// Runtime types generated with workerd@1.20260317.1 2025-09-24 nodejs_compat
// Generated by Wrangler by running `wrangler types` (hash: 24a3fe972e2c4aaa57bb30eaba75ee7a)
// Runtime types generated with workerd@1.20260317.1 2025-09-24 nodejs_als
declare namespace Cloudflare {
interface Env {
MY_VAR: "Hello from Cloudflare";
}
}
interface Env extends Cloudflare.Env {}
type StringifyValues<EnvType extends Record<string, unknown>> = {
[Binding in keyof EnvType]: EnvType[Binding] extends string ? EnvType[Binding] : string;
};
declare namespace NodeJS {
interface ProcessEnv extends StringifyValues<Pick<Cloudflare.Env, "MY_VAR">> {}
}

// Begin runtime types
/*! *****************************************************************************
Expand Down
2 changes: 1 addition & 1 deletion e2e/solid-start/basic-cloudflare/wrangler.jsonc
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
"$schema": "node_modules/wrangler/config-schema.json",
"name": "tanstack-start-app",
"compatibility_date": "2025-09-24",
"compatibility_flags": ["nodejs_compat"],
"compatibility_flags": ["nodejs_als"],
"main": "@tanstack/solid-start/server-entry",
"vars": {
"MY_VAR": "Hello from Cloudflare",
Expand Down
10 changes: 2 additions & 8 deletions examples/solid/start-basic-cloudflare/worker-configuration.d.ts
Original file line number Diff line number Diff line change
@@ -1,18 +1,12 @@
/* eslint-disable */
// Generated by Wrangler by running `wrangler types` (hash: b11df627d8b3c51b1bf3230a546b0f20)
// Runtime types generated with workerd@1.20260317.1 2025-09-24 nodejs_compat
// Generated by Wrangler by running `wrangler types` (hash: 24a3fe972e2c4aaa57bb30eaba75ee7a)
// Runtime types generated with workerd@1.20260317.1 2025-09-24 nodejs_als
declare namespace Cloudflare {
interface Env {
MY_VAR: "Hello from Cloudflare";
}
}
interface Env extends Cloudflare.Env {}
type StringifyValues<EnvType extends Record<string, unknown>> = {
[Binding in keyof EnvType]: EnvType[Binding] extends string ? EnvType[Binding] : string;
};
declare namespace NodeJS {
interface ProcessEnv extends StringifyValues<Pick<Cloudflare.Env, "MY_VAR">> {}
}

// Begin runtime types
/*! *****************************************************************************
Expand Down
2 changes: 1 addition & 1 deletion examples/solid/start-basic-cloudflare/wrangler.jsonc
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
"$schema": "node_modules/wrangler/config-schema.json",
"name": "tanstack-start-app",
"compatibility_date": "2025-09-24",
"compatibility_flags": ["nodejs_compat"],
"compatibility_flags": ["nodejs_als"],
"main": "@tanstack/solid-start/server-entry",
"vars": {
"MY_VAR": "Hello from Cloudflare",
Expand Down
3 changes: 1 addition & 2 deletions packages/react-router/src/ssr/renderRouterToStream.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,10 @@ import ReactDOMServer from 'react-dom/server'
import { isbot } from 'isbot'
import {
createSsrStreamResponse,
transformPipeableStreamWithRouter,
transformReadableStreamWithRouter,
} from '@tanstack/router-core/ssr/server'
import { transformPipeableStreamWithRouter } from './transform-pipeable-stream-with-router'
import type { AnyRouter } from '@tanstack/router-core'
import type { ReadableStream } from 'node:stream/web'
import type { ReactNode } from 'react'

const noop = () => {}
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
import { Readable } from 'node:stream'
import { transformStreamWithRouter } from '@tanstack/router-core/ssr/server'
import type { TransformStreamWithRouterOptions } from '@tanstack/router-core/ssr/server'
import type { AnyRouter } from '@tanstack/router-core'
import type { ReadableStream as NodeReadableStream } from 'node:stream/web'

export function transformPipeableStreamWithRouter(
router: AnyRouter,
routerStream: Readable,
opts?: TransformStreamWithRouterOptions,
) {
const appStream = Readable.toWeb(routerStream) as unknown as ReadableStream
const transformedStream = transformStreamWithRouter(router, appStream, opts)

return Readable.fromWeb(transformedStream as unknown as NodeReadableStream)
}
33 changes: 33 additions & 0 deletions packages/react-router/tests/renderRouterToStream.test.tsx
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
import { PassThrough } from 'node:stream'
import { afterEach, describe, expect, onTestFinished, test, vi } from 'vitest'
import { attachRouterServerSsrUtils } from '@tanstack/router-core/ssr/server'
import { createMemoryHistory, createRootRoute, createRouter } from '../src'
import { transformPipeableStreamWithRouter } from '../src/ssr/transform-pipeable-stream-with-router'

const reactDomServerMocks = vi.hoisted(() => ({
renderToReadableStream: undefined as undefined | (() => unknown),
Expand Down Expand Up @@ -261,4 +263,35 @@ describe('renderRouterToStream - pipeable sync errors', () => {
router.serverSsr?.cleanup()
}
})

test('destroying the pipeable response cancels the React adapter', async () => {
const router = await buildRouter()
const cleanup = vi.spyOn(router.serverSsr!, 'cleanup')
const input = new PassThrough()
let aborts = 0
const output = transformPipeableStreamWithRouter(router, input, {
onAbort: () => {
aborts++
},
})
output.on('error', () => {})

try {
input.write('<p>x</p>')
await new Promise((resolve) => setImmediate(resolve))

output.destroy(new Error('client gone'))
await new Promise((resolve) => setImmediate(resolve))
await Promise.resolve()

expect(aborts).toEqual(1)
expect(input.destroyed).toEqual(true)
expect(cleanup).toHaveBeenCalledOnce()
} finally {
if (!input.destroyed) {
input.destroy()
}
router.serverSsr?.cleanup()
}
})
Comment thread
coderabbitai[bot] marked this conversation as resolved.
})
1 change: 0 additions & 1 deletion packages/router-core/src/ssr/server.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ export type {
SsrResponse,
} from './handlerCallback'
export {
transformPipeableStreamWithRouter,
transformStreamWithRouter,
transformReadableStreamWithRouter,
} from './transformStreamWithRouter'
Expand Down
12 changes: 0 additions & 12 deletions packages/router-core/src/ssr/transformStreamWithRouter.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
import { ReadableStream } from 'node:stream/web'
import { Readable } from 'node:stream'
import { TSR_SCRIPT_BARRIER_ID } from './constants'
import type { AnyRouter } from '../router'

Expand Down Expand Up @@ -28,16 +26,6 @@ export function transformReadableStreamWithRouter(
return transformStreamWithRouter(router, routerStream, opts)
}

export function transformPipeableStreamWithRouter(
router: AnyRouter,
routerStream: Readable,
opts?: TransformStreamWithRouterOptions,
) {
return Readable.fromWeb(
transformStreamWithRouter(router, Readable.toWeb(routerStream), opts),
)
}

// Minimum length of a valid closing tag: </a> = 4 characters
const MIN_CLOSING_TAG_LENGTH = 4

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@
// transformStreamWithRouter calls controller.enqueue unconditionally,
// allowing controller.[[queue]] to accumulate Uint8Array chunks (external
// memory) when the consumer drains slower than the producer.
import { ReadableStream } from 'node:stream/web'
import { describe, expect, it } from 'vitest'
import { transformStreamWithRouter } from '../src/ssr/transformStreamWithRouter'

Expand Down
32 changes: 0 additions & 32 deletions packages/router-core/tests/transformStreamWithRouter.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,16 +4,13 @@
// (TanStack/router#7402) without relying on GC, timing of real I/O, or
// process.memoryUsage(). On-demand backpressure/external-memory
// assertions live in transformStreamBackpressure.perf.test.ts.
import { ReadableStream } from 'node:stream/web'
import { PassThrough } from 'node:stream'
import { describe, expect, test, vi } from 'vitest'
import { createMemoryHistory } from '@tanstack/history'
import { BaseRootRoute, BaseRoute } from '../src'
import { GLOBAL_TSR, TSR_SCRIPT_BARRIER_ID } from '../src/ssr/constants'
import { createSsrStreamResponse } from '../src/ssr/handlerCallback'
import { attachRouterServerSsrUtils } from '../src/ssr/ssr-server'
import {
transformPipeableStreamWithRouter,
transformReadableStreamWithRouter,
transformStreamWithRouter,
} from '../src/ssr/transformStreamWithRouter'
Expand Down Expand Up @@ -1027,35 +1024,6 @@ describe('transformStreamWithRouter — injected HTML ordering', () => {
expect(aborts).toBe(0)
})

test('onAbort: fires when pipeable wrapper consumer destroys', async () => {
const { router, finishSerialization } = makeRouter()
finishSerialization()

const pass = new PassThrough()
let aborts = 0
const out = transformPipeableStreamWithRouter(router as any, pass, {
onAbort: () => aborts++,
})
// Swallow expected error emission from destroy().
out.on('error', () => {})

// Push something so the read loop is engaged.
pass.write('<p>x</p>')
// Wait a tick to let reader start.
await new Promise((r) => setImmediate(r))

// Destroy downstream — simulates Node response being closed by client.
out.destroy(new Error('client gone'))
// Allow microtasks + readable webstream cancel propagation.
await new Promise((r) => setImmediate(r))
await Promise.resolve()

expect(aborts).toBe(1)

// Cleanup: destroy upstream so we don't leak.
if (!pass.destroyed) pass.destroy()
})

test('onAbort: lifetime timeout triggers abort exactly once', async () => {
vi.useFakeTimers()
const errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
Expand Down
1 change: 0 additions & 1 deletion packages/solid-router/src/ssr/renderRouterToStream.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import {
} from '@tanstack/router-core/ssr/server'
import { makeSsrSerovalPlugin } from '@tanstack/router-core'
import type { JSXElement } from 'solid-js'
import type { ReadableStream } from 'node:stream/web'
import type { AnyRouter } from '@tanstack/router-core'

const noop = () => {}
Expand Down
1 change: 0 additions & 1 deletion packages/start-server-core/src/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ export {
createRequestHandler,
defineHandlerCallback,
transformReadableStreamWithRouter,
transformPipeableStreamWithRouter,
} from '@tanstack/router-core/ssr/server'
export type { HandlerCallback } from '@tanstack/router-core/ssr/server'

Expand Down
11 changes: 5 additions & 6 deletions packages/vue-router/src/ssr/renderRouterToStream.tsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
import { ReadableStream as NodeReadableStream } from 'node:stream/web'
import * as Vue from 'vue'
import { pipeToWebWritable, renderToString } from 'vue/server-renderer'
import { isbot } from 'isbot'
Expand All @@ -15,8 +14,8 @@ const isAbortError = (request: Request, error: unknown) =>
(error as any)?.code === 'ABORT_ERR'

function prependDoctype(
readable: globalThis.ReadableStream,
): NodeReadableStream<Uint8Array> {
readable: ReadableStream<Uint8Array>,
): ReadableStream<Uint8Array> {
Comment thread
coderabbitai[bot] marked this conversation as resolved.
const encoder = new TextEncoder()
let sentDoctype = false
let reader: ReadableStreamDefaultReader<Uint8Array> | undefined
Expand All @@ -29,7 +28,7 @@ function prependDoctype(
reader = undefined
}

return new NodeReadableStream<Uint8Array>({
return new ReadableStream<Uint8Array>({
start() {
reader = readable.getReader()
},
Expand Down Expand Up @@ -123,7 +122,7 @@ export const renderRouterToStream = async ({
}
}

const { writable, readable } = new TransformStream()
const { writable, readable } = new TransformStream<Uint8Array, Uint8Array>()
const innerWriter = writable.getWriter()
let writerDone = false
const releaseWriter = () => {
Expand Down Expand Up @@ -159,7 +158,7 @@ export const renderRouterToStream = async ({
throw err
}

const vueWritable = new WritableStream({
const vueWritable = new WritableStream<Uint8Array>({
write(chunk) {
if (writerDone) {
return
Expand Down