Skip to content

fix(router-core): don't set isServer if NODE_ENV test#6727

Merged
schiller-manuel merged 2 commits intomainfrom
isServer-test-undefined
Feb 21, 2026
Merged

fix(router-core): don't set isServer if NODE_ENV test#6727
schiller-manuel merged 2 commits intomainfrom
isServer-test-undefined

Conversation

@schiller-manuel
Copy link
Contributor

@schiller-manuel schiller-manuel commented Feb 21, 2026

Summary by CodeRabbit

  • Bug Fixes
    • Server-side detection now correctly returns undefined in test environments instead of always returning a fixed value, improving test accuracy.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 21, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The isServer export in packages/router-core/src/isServer/server.ts is modified from a constant boolean to a conditional value that evaluates to true in non-test environments and undefined when NODE_ENV is 'test', introducing environment-aware behavior.

Changes

Cohort / File(s) Summary
Environment-aware isServer detection
packages/router-core/src/isServer/server.ts
Changed isServer from a fixed boolean constant to a conditional expression that returns true or undefined based on NODE_ENV, enabling test-specific behavior.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Suggested reviewers

  • Sheraff

Poem

🐰 A little bunny hops with glee,
Testing truthfully, setting servers free!
When NODE_ENV says "test," undefined flows,
But truth remains where production goes. ✨

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch isServer-test-undefined

Comment @coderabbitai help to get the list of available commands and usage tips.

@nx-cloud
Copy link

nx-cloud bot commented Feb 21, 2026

View your CI Pipeline Execution ↗ for commit cc299f7

Command Status Duration Result
nx run tanstack-router-e2e-bundle-size:build --... ✅ Succeeded 1m 25s View ↗

☁️ Nx Cloud last updated this comment at 2026-02-21 21:58:21 UTC

@github-actions
Copy link

Bundle Size Benchmarks

  • Commit: a21eab71136a
  • Measured at: 2026-02-21T21:44:29.663Z
  • Baseline source: history:996e18284413
  • Dashboard: bundle-size history
Scenario Current (gzip) Delta vs baseline Raw Brotli Trend
react-router.minimal 86.76 KiB 0 B (0.00%) 272.72 KiB 75.41 KiB ▅▅▅
react-router.full 89.85 KiB 0 B (0.00%) 283.19 KiB 78.06 KiB ▅▅▅
solid-router.minimal 36.04 KiB 0 B (0.00%) 107.87 KiB 32.43 KiB ▅▅▅
solid-router.full 40.39 KiB 0 B (0.00%) 120.96 KiB 36.25 KiB ▅▅▅
vue-router.minimal 51.90 KiB 0 B (0.00%) 147.86 KiB 46.62 KiB ▅▅▅
vue-router.full 56.76 KiB 0 B (0.00%) 163.47 KiB 50.98 KiB ▅▅▅
react-start.minimal 99.25 KiB 0 B (0.00%) 311.71 KiB 85.93 KiB ▅▅▅
react-start.full 102.66 KiB 0 B (0.00%) 321.63 KiB 88.71 KiB ▅▅▅
solid-start.minimal 48.28 KiB 0 B (0.00%) 145.30 KiB 42.70 KiB ▅▅▅
solid-start.full 53.76 KiB 0 B (0.00%) 161.25 KiB 47.34 KiB ▅▅▅

Trend sparkline is historical gzip bytes ending with this PR measurement; lower is better.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 21, 2026

More templates

@tanstack/arktype-adapter

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

@tanstack/eslint-plugin-router

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

@tanstack/history

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

@tanstack/nitro-v2-vite-plugin

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

@tanstack/react-router

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

@tanstack/react-router-devtools

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

@tanstack/react-router-ssr-query

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

@tanstack/react-start

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

@tanstack/react-start-client

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

@tanstack/react-start-server

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

@tanstack/router-cli

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

@tanstack/router-core

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

@tanstack/router-devtools

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

@tanstack/router-devtools-core

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

@tanstack/router-generator

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

@tanstack/router-plugin

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

@tanstack/router-ssr-query-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-ssr-query-core@6727

@tanstack/router-utils

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

@tanstack/router-vite-plugin

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

@tanstack/solid-router

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

@tanstack/solid-router-devtools

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

@tanstack/solid-router-ssr-query

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-router-ssr-query@6727

@tanstack/solid-start

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

@tanstack/solid-start-client

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

@tanstack/solid-start-server

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

@tanstack/start-client-core

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

@tanstack/start-fn-stubs

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-fn-stubs@6727

@tanstack/start-plugin-core

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

@tanstack/start-server-core

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

@tanstack/start-static-server-functions

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

@tanstack/start-storage-context

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-storage-context@6727

@tanstack/valibot-adapter

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

@tanstack/virtual-file-routes

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

@tanstack/vue-router

npm i https://pkg.pr.new/TanStack/router/@tanstack/vue-router@6727

@tanstack/vue-router-devtools

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

@tanstack/vue-router-ssr-query

npm i https://pkg.pr.new/TanStack/router/@tanstack/vue-router-ssr-query@6727

@tanstack/vue-start

npm i https://pkg.pr.new/TanStack/router/@tanstack/vue-start@6727

@tanstack/vue-start-client

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

@tanstack/vue-start-server

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

@tanstack/zod-adapter

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

commit: cc299f7

@schiller-manuel schiller-manuel merged commit fdb8d2e into main Feb 21, 2026
7 of 8 checks passed
@schiller-manuel schiller-manuel deleted the isServer-test-undefined branch February 21, 2026 21:47
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.

1 participant