Skip to content

docs: cross-link the TanStack Start comparison from the decision guide and Next.js RSC doc#4254

Merged
justin808 merged 1 commit into
mainfrom
jg/tanstack-comparison-crosslinks
Jun 28, 2026
Merged

docs: cross-link the TanStack Start comparison from the decision guide and Next.js RSC doc#4254
justin808 merged 1 commit into
mainfrom
jg/tanstack-comparison-crosslinks

Conversation

@justin808

@justin808 justin808 commented Jun 28, 2026

Copy link
Copy Markdown
Member

What

Two cross-links now that the decision-guide section (#4242) and the RoR Pro vs TanStack Start architecture comparison (#4246) are both on main:

  • The decision guide's "React on Rails vs TanStack Start" section now links the deep architecture comparison — parallel to how it already links the Next.js RSC comparison.
  • The Next.js RSC comparison doc's "Related documentation" now links its TanStack Start sibling.

Closes the asymmetry created by #4242 and #4246 landing as separate PRs.

Gates

  • prettier@3.6.2 ✓ (no reformatting needed)
  • node script/generate-llms-full.mjs
  • lychee ✓ 35 OK / 0 errors (the new relative links resolve to the on-main docs)

Labels: none (docs-only).

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Expanded the React on Rails and TanStack comparison content with an additional link to a deeper architecture overview.
    • Added a new related-docs reference for the React on Rails Pro and TanStack Start comparison.
    • Updated the documentation index so the new comparison page is easier to discover.

…e and Next.js RSC doc

#4242 (decision-guide section) and #4246 (RoR Pro vs TanStack Start
architecture comparison) merged as separate PRs, so neither the decision
guide's TanStack Start section nor the Next.js RSC comparison yet pointed
to the new comparison doc. Add the two cross-links to complete the web:

- Decision guide "React on Rails vs TanStack Start" section now links the
  deep architecture comparison (parallels its existing Next.js RSC link).
- nextjs-comparison.md "Related documentation" now links its TanStack
  Start sibling.

Docs-only; llms-full regenerated.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@coderabbitai

coderabbitai Bot commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 5f297b74-091d-4352-8a10-13051eabb812

📥 Commits

Reviewing files that changed from the base of the PR and between 606241e and 0baf613.

📒 Files selected for processing (4)
  • docs/oss/getting-started/comparing-react-on-rails-to-alternatives.md
  • docs/pro/react-server-components/nextjs-comparison.md
  • llms-full-pro.txt
  • llms-full.txt

Walkthrough

Adds cross-references to the new tanstack-start-comparison.md page in the OSS alternatives comparison doc and the Next.js RSC comparison's related-docs list, and inserts a corresponding entry into both generated LLM index files (llms-full.txt, llms-full-pro.txt).

Changes

TanStack Start comparison cross-links

Layer / File(s) Summary
Docs cross-links and LLM index updates
docs/oss/getting-started/comparing-react-on-rails-to-alternatives.md, docs/pro/react-server-components/nextjs-comparison.md, llms-full.txt, llms-full-pro.txt
Expands the TanStack Start runnable-example sentence to link to the architecture comparison page, adds a related-docs bullet in the Next.js comparison, and inserts the new page entry into both LLM index files.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

documentation

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the docs-only cross-linking update between the decision guide and the Next.js RSC comparison.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jg/tanstack-comparison-crosslinks

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.

@greptile-apps

greptile-apps Bot commented Jun 28, 2026

Copy link
Copy Markdown

Greptile Summary

This PR adds cross-links between the docs comparison pages.

  • Adds a TanStack Start architecture comparison link to the decision guide.
  • Adds the TanStack Start comparison to the Next.js RSC related docs.
  • Updates the generated LLM documentation files to match.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
docs/oss/getting-started/comparing-react-on-rails-to-alternatives.md Adds a cross-link from the TanStack Start section to the Pro architecture comparison.
docs/pro/react-server-components/nextjs-comparison.md Adds the TanStack Start comparison to the related documentation list.
llms-full.txt Mirrors the decision guide link addition in generated documentation text.
llms-full-pro.txt Mirrors the Next.js RSC related-docs link addition in generated documentation text.

Reviews (1): Last reviewed commit: "docs: cross-link the TanStack Start comp..." | Re-trigger Greptile

@claude

claude Bot commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

This PR adds two cross-links: the TanStack Start section of the decision guide now links the deep architecture comparison doc, and the Next.js RSC comparison's Related documentation section now links its TanStack Start sibling. All relative paths resolve correctly, the target file exists, bidirectional link symmetry between tanstack-start-comparison.md and nextjs-comparison.md is now complete, and the llms-full*.txt files faithfully mirror their source markdown.

Two confirmed issues worth addressing:

  1. llms-full.txt cross-bundle reference (see inline on comparing-react-on-rails-to-alternatives.md) — the new sentence directs LLMs to tanstack-start-comparison.md, but that page lives only in llms-full-pro.txt. An LLM given only the OSS bundle sees a "see also" pointer to content that isn't in its context window.

  2. Asymmetric placement in nextjs-comparison.md (see inline) — tanstack-start-comparison.md's "Which should you choose?" section already cross-links to nextjs-comparison.md at the decision-point. The reverse link added here lands only in the lower-prominence "Related documentation" section. Adding a matching entry to nextjs-comparison.md's "Which should you choose?" would close the asymmetry.

Minor nit: the two new links use different text for the same target — comparing-react-on-rails-to-alternatives.md uses the full H1 ("React on Rails Pro and TanStack Start: Two Ways to Own the Full Stack") while nextjs-comparison.md uses the shortened form ("React on Rails Pro and TanStack Start"). Not wrong, but aligning them would be cleaner.

Comment thread docs/pro/react-server-components/nextjs-comparison.md
@justin808

Copy link
Copy Markdown
Member Author

Re-review dispositions (head 0baf61305; all required checks green, mergeStateStatus: CLEAN). Resolving:

  • llms-full cross-bundle link (L139): the decision guide (OSS bundle) already links the Pro nextjs-comparison.md the same way — this PR just adds the parallel link to tanstack-start-comparison.md. The link is a valid reactonrails.com/docs/… path that resolves on the live site (the generator's docs-URL validation passes), and OSS→Pro cross-references are the established pattern here. The OSS/Pro llms-bundle split is orthogonal and not introduced by this link.
  • Placement symmetry (L257): "Related documentation" is the conventional home for a sibling-doc "see also." tanstack-start-comparison.md links back from its "Which should you choose?" selection section; the reciprocal pointer fits naturally under Related documentation here.

Docs-only, required checks green, lychee 0 errors. Beta phase (main), development mode. Enqueuing.

@justin808 justin808 added this pull request to the merge queue Jun 28, 2026
Merged via the queue into main with commit 77e1b02 Jun 28, 2026
45 checks passed
@justin808 justin808 deleted the jg/tanstack-comparison-crosslinks branch June 28, 2026 00:31
justin808 added a commit that referenced this pull request Jun 28, 2026
…ion-eviction-test

* origin/main:
  Make RSC Rspack doctor lazyCompilation warning honest about its limitation (#4249)
  docs: cross-link the TanStack Start comparison from the decision guide and Next.js RSC doc (#4254)
  docs: add "RoR Pro vs TanStack Start" architecture comparison (#4246)
  docs: add "React on Rails vs TanStack Start" to the decision guide (#4242)
justin808 added a commit that referenced this pull request Jun 28, 2026
…pool-warmup

* origin/main:
  Docs: record RSC Rspack client refs investigation (#4269)
  Release train: release-forward-port re-homes changelog to [Unreleased] (#4257)
  Release train: add `release finish` promote + close-out scripts (#4258)
  Release train: /update-changelog release-vs-main target (#4256)
  Release train: add `release start` (auto-create release/X.Y.Z on rc cut) (#4255)
  Release train: ci-changes-detector classifies release-tooling paths (skip full matrix) (#4262)
  Fix release forward-port post-merge review findings (#4261)
  Make RSC Rspack doctor lazyCompilation warning honest about its limitation (#4249)
  docs: cross-link the TanStack Start comparison from the decision guide and Next.js RSC doc (#4254)
  docs: add "RoR Pro vs TanStack Start" architecture comparison (#4246)
  docs: add "React on Rails vs TanStack Start" to the decision guide (#4242)
justin808 added a commit that referenced this pull request Jun 28, 2026
…ession

* origin/main:
  Docs: record RSC Rspack client refs investigation (#4269)
  Release train: release-forward-port re-homes changelog to [Unreleased] (#4257)
  Release train: add `release finish` promote + close-out scripts (#4258)
  Release train: /update-changelog release-vs-main target (#4256)
  Release train: add `release start` (auto-create release/X.Y.Z on rc cut) (#4255)
  Release train: ci-changes-detector classifies release-tooling paths (skip full matrix) (#4262)
  Fix release forward-port post-merge review findings (#4261)
  Make RSC Rspack doctor lazyCompilation warning honest about its limitation (#4249)
  docs: cross-link the TanStack Start comparison from the decision guide and Next.js RSC doc (#4254)
  docs: add "RoR Pro vs TanStack Start" architecture comparison (#4246)
  docs: add "React on Rails vs TanStack Start" to the decision guide (#4242)
justin808 added a commit that referenced this pull request Jun 28, 2026
…r-timing

* origin/main:
  Docs: record RSC Rspack client refs investigation (#4269)
  Release train: release-forward-port re-homes changelog to [Unreleased] (#4257)
  Release train: add `release finish` promote + close-out scripts (#4258)
  Release train: /update-changelog release-vs-main target (#4256)
  Release train: add `release start` (auto-create release/X.Y.Z on rc cut) (#4255)
  Release train: ci-changes-detector classifies release-tooling paths (skip full matrix) (#4262)
  Fix release forward-port post-merge review findings (#4261)
  Make RSC Rspack doctor lazyCompilation warning honest about its limitation (#4249)
  docs: cross-link the TanStack Start comparison from the decision guide and Next.js RSC doc (#4254)
  docs: add "RoR Pro vs TanStack Start" architecture comparison (#4246)
  docs: add "React on Rails vs TanStack Start" to the decision guide (#4242)
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.

1 participant