docs: fix stale links and skill listing#5460
Conversation
Signed-off-by: Glenn-Agent <glenn_agent@163.com>
📝 WalkthroughWalkthroughFixes broken internal documentation links across six pages (how-it-works, release-notes, troubleshooting, commands reference), replaces hardcoded agent version strings with generic ChangesDocumentation Link Fixes, Version Placeholders, and Content Additions
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 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 |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
docs/inference/set-up-sub-agent.mdx (1)
55-55: ⚡ Quick winUse active voice here.
Line 55 reads passively. As per coding guidelines, use active voice and second person in docs.
🤖 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/inference/set-up-sub-agent.mdx` at line 55, Rewrite line 55 in docs/inference/set-up-sub-agent.mdx to use active voice and second person perspective instead of passive voice. The current sentence uses passive constructions like "must be running" and "must be discoverable". Convert it to direct, imperative instructions that address the reader directly using "you" and "your", such as by making it clear what actions the reader must take to ensure the sandbox is properly configured and discoverable.Source: Coding guidelines
🤖 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/inference/set-up-sub-agent.mdx`:
- Line 55: Rewrite line 55 in docs/inference/set-up-sub-agent.mdx to use active
voice and second person perspective instead of passive voice. The current
sentence uses passive constructions like "must be running" and "must be
discoverable". Convert it to direct, imperative instructions that address the
reader directly using "you" and "your", such as by making it clear what actions
the reader must take to ensure the sandbox is properly configured and
discoverable.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: e0f9a93c-ca7b-4d58-bdaa-f600a823c596
📒 Files selected for processing (7)
docs/about/how-it-works.mdxdocs/about/release-notes.mdxdocs/inference/set-up-sub-agent.mdxdocs/reference/commands-nemohermes.mdxdocs/reference/commands.mdxdocs/reference/troubleshooting.mdxdocs/resources/agent-skills.mdx
Summary
Fix several low-risk documentation navigation and discoverability issues reported by doc validation:
nemoclaw-user-agent-skillsin the agent skills page and add an example question for it.Fixes #5445
Fixes #5089
Fixes #5079
Fixes #5077
Fixes #5083
Fixes #5076
Testing
npm run docs:sync-agent-variantsnpm run docs:strict— passed with 0 errors and 2 Fern upgrade warningsgit diff --checknpm run docs:check-agent-variantsgit verify-commit HEADSigned-off-by: Glenn-Agent glenn_agent@163.com
Summary by CodeRabbit
Documentation