docs: turn pro quick start into a gateway#2862
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
WalkthroughThe Pro Quick Start guide in the OSS documentation tree was replaced with a concise gateway page. The comprehensive installation and configuration walkthrough was removed and replaced with links to canonical Pro resources, including Pro installation, upgrade guides, OSS-vs-Pro comparison, and feature documentation. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
Review: Documentation gateway refactor This is a clean, focused documentation change. Verified:
The approach is sound: preserving the URL while replacing duplicated content with a gateway avoids the info-architecture problem described in the PR without breaking any existing inbound links. One minor suggestion: the page intro refers to "the Pro docs section" without a hyperlink. Linking that phrase directly (e.g. to Otherwise LGTM — no code changes, no security surface, and no behavioral risk. |
Greptile SummaryThis PR converts Confidence Score: 5/5Safe to merge — documentation-only change with all links verified Single markdown file touched; every relative link resolves to an existing file and correct anchor; no code or config changes; the URL path is unchanged so existing deep-links continue to work No files require special attention Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A["OSS discovery path\n(pro-quick-start.md)"] --> B{Where are you?}
B -->|New Pro app| C["docs/pro/installation.md\n#fresh-installation"]
B -->|Upgrading from OSS| D["docs/pro/upgrading-to-pro.md"]
B -->|Comparing OSS vs Pro| E["docs/oss/getting-started/oss-vs-pro.md"]
A --> F[After Setup]
F --> G["docs/pro/react-server-components/index.md"]
F --> H["docs/pro/node-renderer.md"]
F --> I["docs/oss/getting-started/common-issues.md"]
Reviews (1): Last reviewed commit: "docs: turn pro quick start into a gatewa..." | Re-trigger Greptile |
| # React on Rails Pro: Start Here | ||
|
|
||
| This guide walks you through creating a complete React on Rails Pro application with server-side rendering via the Node Renderer, from an empty directory to a running app. | ||
| React on Rails Pro has its canonical installation and upgrade guides in the Pro docs section. This |
There was a problem hiding this comment.
The phrase "the Pro docs section" would be more helpful as a hyperlink. Readers who aren't sure where that section lives will benefit from a direct link to the Pro docs landing page.
| React on Rails Pro has its canonical installation and upgrade guides in the Pro docs section. This | |
| React on Rails Pro has its canonical installation and upgrade guides in the [Pro docs section](../../pro/home-pro.md). This |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
docs/oss/getting-started/pro-quick-start.md (1)
7-7: Optional wording polish for the heading“Choose the Right Pro Path” is clear, but if you want to reduce repeated “right” nearby, consider “Choose Your Pro Path.”
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/oss/getting-started/pro-quick-start.md` at line 7, Heading wording “Choose the Right Pro Path” is fine but could avoid repeating “right” elsewhere; change the heading text string "Choose the Right Pro Path" to "Choose Your Pro Path" in the markdown so it reads more concise and natural (update the header token that contains "Choose the Right Pro Path" to "Choose Your Pro Path").
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@docs/oss/getting-started/pro-quick-start.md`:
- Line 7: Heading wording “Choose the Right Pro Path” is fine but could avoid
repeating “right” elsewhere; change the heading text string "Choose the Right
Pro Path" to "Choose Your Pro Path" in the markdown so it reads more concise and
natural (update the header token that contains "Choose the Right Pro Path" to
"Choose Your Pro Path").
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: e3156587-f1ae-4fb0-b79e-90c962aa8381
📒 Files selected for processing (1)
docs/oss/getting-started/pro-quick-start.md
…olve-2833 * origin/main: docs: fix profiling node renderer command (#2863) generators: point Pro install fallback to upgrade guide (#2868) Add RSC Flight payload optimization guide (Article 7) (#2827) Migrate from deprecated Async::Variable to Async::Promise (#2832) docs: turn pro quick start into a gateway (#2862) Fix upload-assets endpoint duplicating bundles across directories (#2768) docs: fix stale docs links and help URLs (#2850) docs: replace dead pro.reactonrails.com links (#2851) docs: refresh generator and helper URLs (#2852) Add standalone RSC upgrade guide for existing Pro apps (#2831) Raise docs version floor to 16.4.0 in install/demo guidance (#2610) # Conflicts: # CHANGELOG.md
…olve-2834 * origin/main: docs: fix profiling node renderer command (#2863) generators: point Pro install fallback to upgrade guide (#2868) Add RSC Flight payload optimization guide (Article 7) (#2827) Migrate from deprecated Async::Variable to Async::Promise (#2832) docs: turn pro quick start into a gateway (#2862) Fix upload-assets endpoint duplicating bundles across directories (#2768) docs: fix stale docs links and help URLs (#2850) docs: replace dead pro.reactonrails.com links (#2851) docs: refresh generator and helper URLs (#2852) # Conflicts: # CHANGELOG.md # react_on_rails/lib/react_on_rails/utils.rb
…olve-2849 * origin/main: docs: fix profiling node renderer command (#2863) generators: point Pro install fallback to upgrade guide (#2868) Add RSC Flight payload optimization guide (Article 7) (#2827) Migrate from deprecated Async::Variable to Async::Promise (#2832) docs: turn pro quick start into a gateway (#2862) Fix upload-assets endpoint duplicating bundles across directories (#2768) docs: fix stale docs links and help URLs (#2850) docs: replace dead pro.reactonrails.com links (#2851) docs: refresh generator and helper URLs (#2852) Add standalone RSC upgrade guide for existing Pro apps (#2831) Raise docs version floor to 16.4.0 in install/demo guidance (#2610) # Conflicts: # CHANGELOG.md
…olve-2835 * origin/main: (21 commits) docs: fix profiling node renderer command (#2863) generators: point Pro install fallback to upgrade guide (#2868) Add RSC Flight payload optimization guide (Article 7) (#2827) Migrate from deprecated Async::Variable to Async::Promise (#2832) docs: turn pro quick start into a gateway (#2862) Fix upload-assets endpoint duplicating bundles across directories (#2768) docs: fix stale docs links and help URLs (#2850) docs: replace dead pro.reactonrails.com links (#2851) docs: refresh generator and helper URLs (#2852) Add standalone RSC upgrade guide for existing Pro apps (#2831) Raise docs version floor to 16.4.0 in install/demo guidance (#2610) Fix release script: require changelog, fix RC version computation (#2848) Bump version to 16.5.0 Bump version to 16.5.0.rc.0 Update CHANGELOG.md for 16.5.0.rc.0 (#2847) Docs: add memory leak prevention guide for Node Renderer SSR (#2845) Docs: fix RSC migration gaps found during real-world migration (#2842) Add common mistakes sections to RSC migration guides (#2826) fix: preserve runtime env vars across Bundler.with_unbundled_env (#2836) Skip Pro CI workflows for Dependabot PRs (#2825) ... # Conflicts: # CHANGELOG.md
## Summary - Stamped `### [16.5.1] - 2026-03-27` with two Pro fixes (PRs #2872, #2768) - Removed the `### [16.5.0.rc.0]` section (was already released as 16.5.0 stable) - Consolidated the 16.5.0 section with full entries (previously just said "no changes from rc.0") - Updated diff links at bottom of file ## Skipped PRs (docs/internal only) #2856, #2860, #2857, #2859, #2864, #2870, #2863, #2868, #2827, #2862, #2850, #2851, #2852, #2831, #2610, #2848 ## Test plan - [ ] Verify CHANGELOG.md formatting and diff links are correct - [ ] Run `rake release` (no args) after merge to publish 16.5.1 🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Low risk: changes are limited to `CHANGELOG.md` release notes and version/compare links with no runtime code modifications. > > **Overview** > Documents the `16.5.1` release by adding a new section under *Unreleased* with two **[Pro]** fixes (missing packaged rake tasks and avoiding duplicated bundles during remote renderer asset uploads). > > Cleans up the `16.5.0` entry by removing the `16.5.0.rc.0` section and updating the compare-link footer so `unreleased` now starts from `v16.5.1` and `16.5.0` compares from `v16.4.0`. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 9a6b224. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Documentation** * Updated changelog to document version 16.5.1 release with consolidated version history and updated comparison links. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>
## Summary - Stamped `### [16.5.1] - 2026-03-27` with two Pro fixes (PRs #2872, #2768) - Removed the `### [16.5.0.rc.0]` section (was already released as 16.5.0 stable) - Consolidated the 16.5.0 section with full entries (previously just said "no changes from rc.0") - Updated diff links at bottom of file ## Skipped PRs (docs/internal only) #2856, #2860, #2857, #2859, #2864, #2870, #2863, #2868, #2827, #2862, #2850, #2851, #2852, #2831, #2610, #2848 ## Test plan - [ ] Verify CHANGELOG.md formatting and diff links are correct - [ ] Run `rake release` (no args) after merge to publish 16.5.1 🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Low risk: changes are limited to `CHANGELOG.md` release notes and version/compare links with no runtime code modifications. > > **Overview** > Documents the `16.5.1` release by adding a new section under *Unreleased* with two **[Pro]** fixes (missing packaged rake tasks and avoiding duplicated bundles during remote renderer asset uploads). > > Cleans up the `16.5.0` entry by removing the `16.5.0.rc.0` section and updating the compare-link footer so `unreleased` now starts from `v16.5.1` and `16.5.0` compares from `v16.4.0`. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 9a6b224. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Documentation** * Updated changelog to document version 16.5.1 release with consolidated version history and updated comparison links. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>
Summary
Resolves #2861 by reducing
docs/oss/getting-started/pro-quick-start.mdto a short OSS-side gateway page instead of maintaining a second full Pro installation guide in the OSS getting-started tree.Changes in this PR:
Why
The old page duplicated Pro setup guidance from the OSS subtree and competed with the canonical Pro docs under
/docs/pro/. That made the information architecture harder to follow and increased the chance that users or agents would land on the wrong entry point first.Test plan
/Users/justin/codex/react_on_rails/node_modules/.bin/prettier --check docs/oss/getting-started/pro-quick-start.mdgit diff --checklychee --offline --no-progress --format compact docs/oss/getting-started/pro-quick-start.mdNote
Low Risk
Low risk: documentation-only change that removes duplicated setup instructions and replaces them with links to canonical Pro docs.
Overview
Replaces
docs/oss/getting-started/pro-quick-start.mdwith a short gateway that preserves the existing URL but stops duplicating Pro setup instructions.The page now directs users to the canonical Pro installation and upgrade guides, plus follow-up links for React Server Components, the Node Renderer architecture, and OSS-side troubleshooting.
Written by Cursor Bugbot for commit 06026b6. This will update automatically on new commits. Configure here.
Summary by CodeRabbit