docs: simplify README and make landing docs agent-agnostic#4552
Conversation
Move duplicated quickstart and Model Router content into the docs site, retire the experimental Hermes label, and sync the platform matrix to prerequisites.mdx with MDX-safe sentinels.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis PR rebrands docs from OpenClaw to NemoClaw/OpenShell, adds explicit OpenClaw and Hermes quickstarts, relocates the generated platform matrix into prerequisites.mdx (with script and pre-commit updates), removes the Hermes experimental warning, and expands Model Router inference guidance with examples and credential-flow notes. ChangesDocumentation Rebranding and Structure Updates
🎯 2 (Simple) | ⏱️ ~10 minutes Possibly Related PRs
Suggested Labels
Suggested Reviewers
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)
Comment |
|
🌿 Preview your docs: https://nvidia-preview-pr-4552.docs.buildwithfern.com/nemoclaw |
E2E Advisor RecommendationRequired E2E: None Full advisor summaryE2E Recommendation AdvisorBase: Required E2E
Optional E2E
New E2E recommendations
|
E2E Scenario Advisor RecommendationRequired scenario E2E: None Full scenario advisor summaryE2E Scenario AdvisorBase: Required scenario E2E
Optional scenario E2E
Relevant changed files
|
PR Review AdvisorFindings: 0 needs attention, 3 worth checking, 0 nice ideas Review findings🛠️ Needs attention
🔎 Worth checking
🌱 Nice ideas
Since last review detailsCurrent findings:
This is an automated advisory review. A human maintainer must make the final merge decision. |
Bring back the Page | Description table with all doc links so the simplified README still points readers to the full docs site map.
There was a problem hiding this comment.
🧹 Nitpick comments (1)
docs/about/overview.mdx (1)
8-8: ⚡ Quick winNormalize branded term casing in keywords.
Use
OpenClaw,NVIDIA OpenShell, andNVIDIAcapitalization inkeywordsto match the project word list.As per coding guidelines: “NVIDIA must be all caps” and “NemoClaw, OpenClaw, and OpenShell must use correct casing.”
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@docs/about/overview.mdx` at line 8, Update the keywords array so branded terms use the project's canonical casing: change "nemoclaw overview" to "NemoClaw overview", "openclaw always-on assistants" to "OpenClaw always-on assistants", and apply all-caps to NVIDIA where present (e.g., "nvidia openshell" → "NVIDIA OpenShell", "nvidia nemotron" → "NVIDIA Nemotron"); ensure the keywords array uses these exact corrected strings and leave unrelated entries (like "hermes agent") unchanged.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@docs/about/overview.mdx`:
- Line 8: Update the keywords array so branded terms use the project's canonical
casing: change "nemoclaw overview" to "NemoClaw overview", "openclaw always-on
assistants" to "OpenClaw always-on assistants", and apply all-caps to NVIDIA
where present (e.g., "nvidia openshell" → "NVIDIA OpenShell", "nvidia nemotron"
→ "NVIDIA Nemotron"); ensure the keywords array uses these exact corrected
strings and leave unrelated entries (like "hermes agent") unchanged.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: 71318e8c-071b-493e-a8a6-0febc55773f8
📒 Files selected for processing (9)
.pre-commit-config.yamlREADME.mddocs/about/how-it-works.mdxdocs/about/overview.mdxdocs/get-started/prerequisites.mdxdocs/get-started/quickstart-hermes.mdxdocs/index.mdxdocs/inference/inference-options.mdxscripts/generate-platform-docs.py
💤 Files with no reviewable changes (1)
- docs/get-started/quickstart-hermes.mdx
ericksoa
left a comment
There was a problem hiding this comment.
LGTM for the docs/README simplification. I reviewed the README/docs generator changes on c5f186f and locally verified the generated platform docs are in sync with python3 scripts/generate-platform-docs.py --check; diff whitespace checks are clean as well. CodeRabbit's metadata keyword-casing note is non-blocking.
Format the alpha notice as a GitHub blockquote alert and list supported agents as bullets instead of inline prose.
Summary
Simplifies the GitHub README into a short landing page and updates the docs site intro pages to describe NemoClaw as an agent-agnostic sandbox stack. OpenClaw and Hermes are both listed as supported agents, and Hermes is no longer labeled experimental.
Related Issue
None.
Changes
README.mdto point at docs for prerequisites, quickstarts, and deep topics instead of duplicating them inline.docs/index.mdx,docs/about/overview.mdx, anddocs/about/how-it-works.mdxwith agent-agnostic framing and links to both OpenClaw and Hermes quickstarts.docs/get-started/quickstart-hermes.mdx.docs/inference/inference-options.mdx.README.mdtodocs/get-started/prerequisites.mdxusing MDX{/* */}sentinels so Fern builds cleanly.Type of Change
Verification
npx prek run --all-filespasses (pre-commit hooks on commit)npm testpassesnpm run docsbuilds without warnings (doc changes only)Signed-off-by: Miyoung Choi miyoungc@nvidia.com
Summary by CodeRabbit
Documentation
Chores