Skip to content

Conversation

@LadyBluenotes
Copy link
Contributor

@LadyBluenotes LadyBluenotes commented Oct 17, 2025

Summary by CodeRabbit

  • Documentation
    • Added a new side-by-side feature comparison page comparing TanStack Router & Start with Next.js, React Router and Remix, including capability matrices, status keys, and external resource links.
    • Removed an older comparative discussion block and cleaned up minor comment artifacts from framework docs.
    • Added a new "Comparison" entry to the React router documentation navigation.

@github-actions github-actions bot added the documentation Everything documentation related label Oct 17, 2025
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 17, 2025

Walkthrough

Adds a new React framework comparison doc and updates site navigation; removes redundant comparison commentary from React and Solid overview pages. All changes are documentation-only.

Changes

Cohort / File(s) Summary
New documentation page
docs/router/framework/react/comparison.md
Adds a dedicated side-by-side comparison page (front matter, status key, capability matrix comparing TanStack Router/Start, React Router DOM, and Next.js, resource links, and badge/reference footers).
Overview cleanup
docs/router/framework/react/overview.md, docs/router/framework/solid/overview.md
Removes the "WhyChooseTanStackRouter" markdown comment blocks and associated comparison text from both overview pages.
Navigation update
docs/router/config.json
Inserts a new React nav entry: { "label": "Comparison", "to": "framework/react/comparison" } under the React framework children array.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • schiller-manuel
  • birkskyum
  • brenelz

Poem

🐇 I found a new page beneath the log,

Tables aligned like rows of fog.
Old notes hopped away, content refound,
A tidy meadow where features abound.
🥕 Hooray for docs — neat, new, and sound!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The pull request title "docs: Add comparison page back" directly corresponds to the primary change in the changeset—the addition of a new comparison documentation page at docs/router/framework/react/comparison.md that presents a feature comparison between TanStack Router & Start versus Next.js and React Router/Remix. The title is concise, specific, and uses the appropriate "docs:" prefix convention. While the changeset also includes supporting modifications (removal of WhyChooseTanStackRouter section, config updates, and comment removals), the title accurately identifies the main objective without unnecessary detail, which aligns with expected standards for PR titles.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9940112 and e405b14.

📒 Files selected for processing (1)
  • docs/router/config.json (1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
docs/{router,start}/**

📄 CodeRabbit inference engine (AGENTS.md)

Place router docs under docs/router/ and start framework docs under docs/start/

Files:

  • docs/router/config.json
🔇 Additional comments (1)
docs/router/config.json (1)

36-39: ✓ Navigation entry is properly formatted and consistent.

The new "Comparison" item is correctly added to the React framework's Getting Started section with proper JSON structure, consistent naming conventions, and logical placement. The path framework/react/comparison follows the established pattern, and the target documentation file exists at the expected location.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@nx-cloud
Copy link

nx-cloud bot commented Oct 17, 2025

View your CI Pipeline Execution ↗ for commit e405b14

Command Status Duration Result
nx affected --targets=test:eslint,test:unit,tes... ✅ Succeeded <1s View ↗
nx run-many --target=build --exclude=examples/*... ✅ Succeeded 2s View ↗

☁️ Nx Cloud last updated this comment at 2025-10-17 04:04:52 UTC

@pkg-pr-new
Copy link

pkg-pr-new bot commented Oct 17, 2025

More templates

@tanstack/arktype-adapter

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

@tanstack/directive-functions-plugin

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

@tanstack/eslint-plugin-router

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

@tanstack/history

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

@tanstack/nitro-v2-vite-plugin

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

@tanstack/react-router

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

@tanstack/react-router-devtools

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

@tanstack/react-router-ssr-query

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

@tanstack/react-start

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

@tanstack/react-start-client

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

@tanstack/react-start-server

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

@tanstack/router-cli

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

@tanstack/router-core

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

@tanstack/router-devtools

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

@tanstack/router-devtools-core

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

@tanstack/router-generator

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

@tanstack/router-plugin

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

@tanstack/router-ssr-query-core

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

@tanstack/router-utils

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

@tanstack/router-vite-plugin

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

@tanstack/server-functions-plugin

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

@tanstack/solid-router

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

@tanstack/solid-router-devtools

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

@tanstack/solid-start

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

@tanstack/solid-start-client

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

@tanstack/solid-start-server

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

@tanstack/start-client-core

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

@tanstack/start-plugin-core

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

@tanstack/start-server-core

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

@tanstack/start-static-server-functions

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

@tanstack/start-storage-context

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

@tanstack/valibot-adapter

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

@tanstack/virtual-file-routes

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

@tanstack/zod-adapter

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

commit: e405b14

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🧹 Nitpick comments (1)
docs/router/framework/react/comparison.md (1)

12-19: Enhance the feature key formatting for clarity.

The feature key is useful, but the list items could be formatted more clearly with markdown list syntax for better readability and consistency with the rest of the documentation. Consider using proper list formatting:

-Feature/Capability Key:
-
-- ✅ 1st-class, built-in, and ready to use with no added configuration or code
-- 🟡 Partial Support (on a scale of 5)
-- 🟠 Supported via addon/community package
-- 🔶 Possible, but requires custom code/implementation/casting
-- 🛑 Not officially supported
+## Feature/Capability Key
+
+- **✅** 1st-class, built-in, and ready to use with no added configuration or code
+- **🟡** Partial Support (on a scale of 5)
+- **🟠** Supported via addon/community package
+- **🔶** Possible, but requires custom code/implementation/casting
+- **🛑** Not officially supported
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between be6c094 and 9940112.

📒 Files selected for processing (3)
  • docs/router/framework/react/comparison.md (1 hunks)
  • docs/router/framework/react/overview.md (0 hunks)
  • docs/router/framework/solid/overview.md (0 hunks)
💤 Files with no reviewable changes (2)
  • docs/router/framework/react/overview.md
  • docs/router/framework/solid/overview.md
🧰 Additional context used
📓 Path-based instructions (2)
docs/**/*.{md,mdx}

📄 CodeRabbit inference engine (AGENTS.md)

Use internal docs links relative to the docs/ folder (e.g., ./guide/data-loading)

Files:

  • docs/router/framework/react/comparison.md
docs/{router,start}/**

📄 CodeRabbit inference engine (AGENTS.md)

Place router docs under docs/router/ and start framework docs under docs/start/

Files:

  • docs/router/framework/react/comparison.md
🪛 LanguageTool
docs/router/framework/react/comparison.md

[grammar] ~14-~14: There might be a mistake here.
Context: ... use with no added configuration or code - 🟡 Partial Support (on a scale of 5) - ?...

(QB_NEW_EN)


[grammar] ~15-~15: There might be a mistake here.
Context: ...e - 🟡 Partial Support (on a scale of 5) - 🟠 Supported via addon/community package...

(QB_NEW_EN)


[grammar] ~16-~16: There might be a mistake here.
Context: ...🟠 Supported via addon/community package - 🔶 Possible, but requires custom code/im...

(QB_NEW_EN)


[grammar] ~17-~17: There might be a mistake here.
Context: ...uires custom code/implementation/casting - 🛑 Not officially supported | ...

(QB_NEW_EN)


[grammar] ~18-~18: There might be a mistake here.
Context: ...on/casting - 🛑 Not officially supported | ...

(QB_NEW_EN)


[uncategorized] ~22-~22: The official name of this software platform is spelled with a capital “H”.
Context: ...----------------------------------- | | Github Repo / Stars ...

(GITHUB)

🪛 markdownlint-cli2 (0.18.1)
docs/router/framework/react/comparison.md

22-22: Images should have alternate text (alt text)

(MD045, no-alt-text)


22-22: Images should have alternate text (alt text)

(MD045, no-alt-text)


22-22: Images should have alternate text (alt text)

(MD045, no-alt-text)


23-23: Images should have alternate text (alt text)

(MD045, no-alt-text)


23-23: Images should have alternate text (alt text)

(MD045, no-alt-text)


84-84: Link and image reference definitions should be needed
Unused link or image reference definition: "_"

(MD053, link-image-reference-definitions)


90-90: Link and image reference definitions should be needed
Unused link or image reference definition: "bpl-history"

(MD053, link-image-reference-definitions)


91-91: Link and image reference definitions should be needed
Duplicate link or image reference definition: "_"

(MD053, link-image-reference-definitions)


93-93: Link and image reference definitions should be needed
Unused link or image reference definition: "bp-nextjs"

(MD053, link-image-reference-definitions)


96-96: Link and image reference definitions should be needed
Unused link or image reference definition: "bpl-nextjs"

(MD053, link-image-reference-definitions)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Preview
🔇 Additional comments (2)
docs/router/framework/react/comparison.md (2)

1-10: Frontmatter and introduction look good.

The frontmatter is properly formatted, and the introductory text clearly explains the purpose and limitations of the comparison while inviting contributions.


20-79: Comparison table is comprehensive and well-structured.

The table effectively compares capabilities across the three frameworks with clear visual indicators. The breadth of features covered (routing, type safety, full-stack capabilities, etc.) provides useful context for developers evaluating options.


| | TanStack Router / Start | React Router DOM [_(Website)_][router] | Next.JS [_(Website)_][nextjs] |
| ---------------------------------------------- | ------------------------------------------------ | ----------------------------------------------------- | ----------------------------------------------------- |
| Github Repo / Stars | [![][stars-tanstack-router]][gh-tanstack-router] | [![][stars-router]][gh-router] | [![][stars-nextjs]][gh-nextjs] |
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Fix capitalization: "Github" → "GitHub".

The table header uses "Github" but the official branding is "GitHub".

-| Github Repo / Stars                            | [![][stars-tanstack-router]][gh-tanstack-router] | [![][stars-router]][gh-router]                        | [![][stars-nextjs]][gh-nextjs]                        |
+| GitHub Repo / Stars                            | [![][stars-tanstack-router]][gh-tanstack-router] | [![][stars-router]][gh-router]                        | [![][stars-nextjs]][gh-nextjs]                        |
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
| Github Repo / Stars | [![][stars-tanstack-router]][gh-tanstack-router] | [![][stars-router]][gh-router] | [![][stars-nextjs]][gh-nextjs] |
| GitHub Repo / Stars | [![][stars-tanstack-router]][gh-tanstack-router] | [![][stars-router]][gh-router] | [![][stars-nextjs]][gh-nextjs] |
🧰 Tools
🪛 LanguageTool

[uncategorized] ~22-~22: The official name of this software platform is spelled with a capital “H”.
Context: ...----------------------------------- | | Github Repo / Stars ...

(GITHUB)

🪛 markdownlint-cli2 (0.18.1)

22-22: Images should have alternate text (alt text)

(MD045, no-alt-text)


22-22: Images should have alternate text (alt text)

(MD045, no-alt-text)


22-22: Images should have alternate text (alt text)

(MD045, no-alt-text)

🤖 Prompt for AI Agents
In docs/router/framework/react/comparison.md around line 22, the table header
uses incorrect capitalization "Github" — change it to the official branding
"GitHub" by replacing "Github" with "GitHub" in that header cell so the
documentation follows correct product naming.

Comment on lines +80 to +96
[bp-tanstack-router]: https://badgen.net/bundlephobia/minzip/@tanstack/react-router
[bpl-tanstack-router]: https://bundlephobia.com/result?p=@tanstack/react-router
[gh-tanstack-router]: https://github.com/tanstack/router
[stars-tanstack-router]: https://img.shields.io/github/stars/tanstack/router?label=%F0%9F%8C%9F
[_]: _
[router]: https://github.com/remix-run/react-router
[bp-router]: https://badgen.net/bundlephobia/minzip/react-router
[gh-router]: https://github.com/remix-run/react-router
[stars-router]: https://img.shields.io/github/stars/remix-run/react-router?label=%F0%9F%8C%9F
[bpl-router]: https://bundlephobia.com/result?p=react-router
[bpl-history]: https://bundlephobia.com/result?p=history
[_]: _
[nextjs]: https://nextjs.org/docs/routing/introduction
[bp-nextjs]: https://badgen.net/bundlephobia/minzip/next.js?label=All
[gh-nextjs]: https://github.com/vercel/next.js
[stars-nextjs]: https://img.shields.io/github/stars/vercel/next.js?label=%F0%9F%8C%9F
[bpl-nextjs]: https://bundlephobia.com/result?p=next
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Remove unused reference definitions.

Several reference link definitions are defined but never used in the document. Remove the unused entries to clean up the file:

 [bp-tanstack-router]: https://badgen.net/bundlephobia/minzip/@tanstack/react-router
 [bpl-tanstack-router]: https://bundlephobia.com/result?p=@tanstack/react-router
 [gh-tanstack-router]: https://github.com/tanstack/router
 [stars-tanstack-router]: https://img.shields.io/github/stars/tanstack/router?label=%F0%9F%8C%9F
-[_]: _
 [router]: https://github.com/remix-run/react-router
 [bp-router]: https://badgen.net/bundlephobia/minzip/react-router
 [gh-router]: https://github.com/remix-run/react-router
 [stars-router]: https://img.shields.io/github/stars/remix-run/react-router?label=%F0%9F%8C%9F
 [bpl-router]: https://bundlephobia.com/result?p=react-router
-[bpl-history]: https://bundlephobia.com/result?p=history
-[_]: _
 [nextjs]: https://nextjs.org/docs/routing/introduction
-[bp-nextjs]: https://badgen.net/bundlephobia/minzip/next.js?label=All
 [gh-nextjs]: https://github.com/vercel/next.js
 [stars-nextjs]: https://img.shields.io/github/stars/vercel/next.js?label=%F0%9F%8C%9F
-[bpl-nextjs]: https://bundlephobia.com/result?p=next
🧰 Tools
🪛 markdownlint-cli2 (0.18.1)

84-84: Link and image reference definitions should be needed
Unused link or image reference definition: "_"

(MD053, link-image-reference-definitions)


90-90: Link and image reference definitions should be needed
Unused link or image reference definition: "bpl-history"

(MD053, link-image-reference-definitions)


91-91: Link and image reference definitions should be needed
Duplicate link or image reference definition: "_"

(MD053, link-image-reference-definitions)


93-93: Link and image reference definitions should be needed
Unused link or image reference definition: "bp-nextjs"

(MD053, link-image-reference-definitions)


96-96: Link and image reference definitions should be needed
Unused link or image reference definition: "bpl-nextjs"

(MD053, link-image-reference-definitions)

🤖 Prompt for AI Agents
In docs/router/framework/react/comparison.md around lines 80 to 96, several
markdown reference link definitions (e.g., bp-tanstack-router,
bpl-tanstack-router, gh-tanstack-router, stars-tanstack-router, _, router,
bp-router, gh-router, stars-router, bpl-router, bpl-history, nextjs, bp-nextjs,
gh-nextjs, stars-nextjs, bpl-nextjs) are declared but not referenced anywhere in
the document; remove the unused reference definitions and leave only the
reference links that are actually referenced elsewhere in the file (or
consolidate duplicates like repeated [_] entries), ensuring the remaining
references are correct and the file still builds.

@tannerlinsley tannerlinsley merged commit 715763c into TanStack:main Oct 17, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Everything documentation related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants