docs(inference): split guides into nested topics - #6659
Merged
Conversation
Signed-off-by: Miyoung Choi <miyoungc@nvidia.com>
miyoungc
added
area: docs
Contributor
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughThe PR reorganizes NemoClaw inference documentation into focused provider, model, setup, configuration, validation, and management pages. It updates navigation, redirects, cross-references, generated-doc targets, adds agent configuration pages, and extends published-route validation tests. ChangesInference documentation restructure
Estimated code review effort: 4 (Complex) | ~60 minutes Possibly related PRs
Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Contributor
Code Coverage OverviewLanguages: TypeScript TypeScript / code-coverage/pluginThe overall coverage remains at 96%, unchanged from the TypeScript / code-coverage/cliThe overall coverage in the Show a code coverage summary of the most impacted files.
Updated |
Contributor
|
🌿 Preview your docs: https://nvidia-preview-pr-6659.docs.buildwithfern.com/nemoclaw |
miyoungc
marked this pull request as draft
July 10, 2026 21:36
|
Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually. Contributors can view more details about this message here. |
miyoungc
marked this pull request as ready for review
July 10, 2026 21:37
![github-code-quality[bot]](https://avatars.githubusercontent.com/u/9919?s=60&v=4){kind=small}
miyoungc
marked this pull request as draft
July 10, 2026 21:38
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 12
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
docs/inference/use-openrouter.mdx (1)
58-58: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winAdd the missing period.
This related-topic sentence should end with a period to comply with the documentation guidelines.
-- [Understand Provider Validation](../validate-and-troubleshoot/understand-provider-validation) describes provider validation behavior. +- [Understand Provider Validation](../validate-and-troubleshoot/understand-provider-validation) describes provider validation behavior.🤖 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/use-openrouter.mdx` at line 58, Add a period to the end of the related-topic sentence in the documentation.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.
Inline comments:
In `@docs/about/overview.mdx`:
- Line 73: Correct the inference-provider links in the overview table, including
the entries corresponding to the “Routed inference” and the other affected rows,
by replacing ../inference/learn-and-choose/choose-inference-provider with
../inference/choose-inference-provider.
In `@docs/configure-agents/understand-context-compaction.mdx`:
- Line 35: Update the model-limits reference near “Configure Model Limits” to
use the relative link ../inference/configure-model-limits, removing the
incorrect manage-inference path.
In `@docs/get-started/quickstart-hermes.mdx`:
- Line 87: Update the inference-provider links in quickstart-hermes.mdx,
including the references near the related repeated locations, to use
../inference/choose-inference-provider so they resolve to the existing page
instead of the nonexistent nested route.
In `@docs/inference/fix-tool-calling-failures.mdx`:
- Line 52: The onboarding instructions use the wrong provider option for an
already-running parser-aware vLLM server. Update this section to select “Local
vLLM (already running)” and reserve “Local vLLM [experimental]” for the
managed-install workflow, or explicitly distinguish both setup paths.
In `@docs/inference/set-up-ollama.mdx`:
- Around line 96-107: Update the Windows Ollama setup and restart guidance to
avoid exposing the unauthenticated API on all interfaces: either require and
document a Windows Firewall rule that restricts port 11434 to WSL/Docker
traffic, or configure OLLAMA_HOST to bind only to the adapter/address required
by host.docker.internal. Apply the restriction consistently to every
Windows-host launch path.
In `@docs/inference/set-up-openai-compatible-endpoint.mdx`:
- Around line 44-52: Clarify the onboarding validation statement to distinguish
endpoint types: state that a real inference request is performed before sandbox
creation only for host-routable endpoints, while the sandbox-internal
host.openshell.internal alias skips that probe and must be verified through the
runtime route after onboarding.
In `@docs/inference/set-up-vllm.mdx`:
- Around line 118-123: Update the `HF_TOKEN` example in the vLLM setup
instructions to quote the placeholder value, using valid shell syntax such as
`export HF_TOKEN="<your-hf-token>"`, so users can copy the command without
triggering redirection parsing.
In `@docs/inference/understand-provider-validation.mdx`:
- Line 84: The OpenAI-compatible endpoint link in the inference documentation
points to the wrong location; update the link target in the relevant markdown
entry to the sibling path set-up-openai-compatible-endpoint, removing the
../custom-endpoints/ prefix.
In `@docs/inference/use-nvidia-endpoints.mdx`:
- Around line 50-51: Update the links in the inference documentation list to use
the local targets `choose-model` and `understand-provider-validation`, while
preserving their existing link text.
In `@docs/inference/view-active-inference-route.mdx`:
- Line 73: Update the “Verify the Sandbox Inference Route” link in the relevant
documentation to target the sibling page `verify-inference-route` instead of the
incorrect `../validate-and-troubleshoot/verify-inference-route` path.
In `@docs/manage-sandboxes/workspace-files.mdx`:
- Line 108: Update the “Set Up Task-Specific Sub-Agents” link in the
workspace-files documentation to target ../inference/set-up-sub-agent, unless
the intended fix is to add the missing configure-agents page.
In `@test/model-capability-audit-doc.test.ts`:
- Around line 73-77: Add negative assertions in the model-capability audit test
alongside the existing positive link checks, verifying that the exact superseded
inference-provider and tool-calling hrefs are absent from markdown. Keep the
assertions focused on the old paths so the migration test fails if stale links
remain.
---
Outside diff comments:
In `@docs/inference/use-openrouter.mdx`:
- Line 58: Add a period to the end of the related-topic sentence in the
documentation.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: e6db73f3-664f-4d85-b8dc-b8e956efff26
📒 Files selected for processing (75)
.agents/skills/nemoclaw-contributor-update-docs/SKILL.md.pre-commit-config.yamlREADME.mddocs/about/ecosystem-deepagents.mdxdocs/about/how-it-works.mdxdocs/about/overview.mdxdocs/about/release-notes.mdxdocs/configure-agents/configure-agent-heartbeats.mdxdocs/configure-agents/progressive-tool-disclosure.mdxdocs/configure-agents/understand-context-compaction.mdxdocs/deployment/brev-web-ui.mdxdocs/deployment/deploy-to-remote-gpu.mdxdocs/get-started/quickstart-hermes.mdxdocs/get-started/quickstart-langchain-deepagents-code.mdxdocs/get-started/quickstart.mdxdocs/get-started/windows-preparation.mdxdocs/index.ymldocs/inference/choose-compatible-inference-api.mdxdocs/inference/choose-inference-provider.mdxdocs/inference/choose-local-inference-server.mdxdocs/inference/choose-model.mdxdocs/inference/configure-inference-timeouts.mdxdocs/inference/configure-model-capabilities.mdxdocs/inference/configure-model-limits.mdxdocs/inference/custom-endpoint-security.mdxdocs/inference/declarative-agents-manifest.mdxdocs/inference/fix-tool-calling-failures.mdxdocs/inference/how-inference-routing-works.mdxdocs/inference/inference-options.mdxdocs/inference/inference-overview.mdxdocs/inference/local-compatible-inference-setup.mdxdocs/inference/model-capability-audit.mdxdocs/inference/set-up-anthropic-compatible-endpoint.mdxdocs/inference/set-up-model-router.mdxdocs/inference/set-up-nvidia-nim.mdxdocs/inference/set-up-ollama.mdxdocs/inference/set-up-openai-compatible-endpoint.mdxdocs/inference/set-up-sub-agent.mdxdocs/inference/set-up-vllm.mdxdocs/inference/switch-inference-providers.mdxdocs/inference/switch-models.mdxdocs/inference/switch-providers.mdxdocs/inference/tool-calling-reliability.mdxdocs/inference/understand-provider-validation.mdxdocs/inference/use-anthropic.mdxdocs/inference/use-google-gemini.mdxdocs/inference/use-hermes-provider.mdxdocs/inference/use-local-inference.mdxdocs/inference/use-nvidia-endpoints.mdxdocs/inference/use-openai.mdxdocs/inference/use-openrouter.mdxdocs/inference/verify-inference-route.mdxdocs/inference/view-active-inference-route.mdxdocs/manage-sandboxes/lifecycle.mdxdocs/manage-sandboxes/runtime-controls.mdxdocs/manage-sandboxes/workspace-files.mdxdocs/monitoring/monitor-sandbox-activity.mdxdocs/reference/architecture.mdxdocs/reference/cli-selection-guide.mdxdocs/reference/commands.mdxdocs/reference/enterprise-readiness.mdxdocs/reference/platform-support.mdxdocs/reference/troubleshooting.mdxdocs/security/best-practices.mdxdocs/security/credential-rotation.mdxfern/docs.ymlscripts/bench/lib.tsscripts/check-docs-published-routes.tsscripts/generate-platform-docs.pysrc/lib/onboard/install-ollama-linux.tssrc/lib/onboard/model-router-python.tstest/check-docs-published-routes.test.tstest/generate-platform-docs.test.tstest/inference-options-docs.test.tstest/model-capability-audit-doc.test.ts
💤 Files with no reviewable changes (5)
- docs/inference/use-local-inference.mdx
- docs/inference/inference-options.mdx
- docs/inference/local-compatible-inference-setup.mdx
- docs/inference/switch-inference-providers.mdx
- docs/inference/tool-calling-reliability.mdx
Contributor
E2E Advisor RecommendationRequired E2E: Dispatch hint: Full advisor summaryE2E Recommendation AdvisorBase: Required E2E
Optional E2E
New E2E recommendations
Dispatch hint
|
Contributor
E2E Target RecommendationRequired E2E targets: Dispatch required E2E targets:
Full E2E target advisor summaryE2E Target AdvisorBase: Required E2E targets
Optional E2E targets
Relevant changed files
|
Contributor
PR Review Advisor — No blocking findingsMerge posture: No blocking advisor findings Action checklist
Test follow-ups to resolve or justifyIf these cover changed behavior, prefer adding them in this PR; otherwise state why existing coverage is enough or link the follow-up.
This is an automated, non-binding review; it still expects maintainers and agents to respond to each required or warning item. Treat suggestions as current-PR improvements when they touch changed code; defer only with maintainer rationale or a linked follow-up. A human maintainer must make the final merge decision. |
Contributor
PR Review Advisor (Nemotron Ultra) — Changes requestedMerge posture: Do not merge yet Action checklist
Findings index
🚨 Required before mergeAddress these before merging unless a maintainer explicitly overrides the advisor with rationale.
|
Signed-off-by: Miyoung Choi <miyoungc@nvidia.com>
miyoungc
marked this pull request as ready for review
July 10, 2026 21:44
miyoungc
marked this pull request as draft
July 10, 2026 21:53
wmanning-docker
left a comment
wmanning-docker
left a comment
There was a problem hiding this comment.
Thanks for the nested inference docs split. I found a blocking route issue for the Deep Agents variant:
fern/docs.ymlredirects/user-guide/deepagents/inference/use-local-inferenceto/user-guide/deepagents/inference/local-inference/set-up-ollama, but the Deep Agents navigation intentionally does not publishset-up-ollama(only vLLM/NIM under Local Inference). That legacy Deep Agents URL will redirect to a non-published route/404. Please make the:variantredirect conditional or add an explicit Deep Agents redirect to/user-guide/deepagents/inference/local-inference(or another published Deep Agents page).- The shared release notes now link to
../inference/local-inference/set-up-ollamain entries such as v0.0.79/v0.0.78. When rendered for Deep Agents, those links also resolve to the same unpublished Deep Agents Ollama route. Please wrap Ollama-only references inAgentOnlyor point shared text at the published Local Inference landing page.
I verified with the published route index: the generated Deep Agents release notes produce unresolved /user-guide/deepagents/inference/local-inference/set-up-ollama links.
Signed-off-by: Miyoung Choi <miyoungc@nvidia.com>
Signed-off-by: Miyoung Choi <miyoungc@nvidia.com>
Signed-off-by: Miyoung Choi <miyoungc@nvidia.com>
Signed-off-by: Miyoung Choi <miyoungc@nvidia.com>
miyoungc
marked this pull request as ready for review
July 10, 2026 23:08
Signed-off-by: Miyoung Choi <miyoungc@nvidia.com>
miyoungc
marked this pull request as draft
July 10, 2026 23:21
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
test/check-docs-published-routes.test.ts (1)
117-135: 🎯 Functional Correctness | 🔵 Trivial | ⚡ Quick winCover destination-only variant expansion.
The fixture places
:variantin both the redirect source and destination, so it does not verify redirects with a fixed source and variant-specific destination. Add that behavioral case to prevent regressions where only one side is expanded.As per path instructions, keep the test focused on observable route-validation behavior rather than implementation details.
🤖 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 `@test/check-docs-published-routes.test.ts` around lines 117 - 135, Extend the test “validates variant redirect destinations independently” with a redirect whose source is fixed and whose destination contains :variant, then assert findBrokenPublishedRedirects reports the expected expanded destination and variant. Keep the assertion focused on observable validation output and preserve the existing source-and-destination variant coverage.Source: Path instructions
test/inference-options-docs.test.ts (1)
222-227: 🔒 Security & Privacy | 🔵 Trivial | ⚡ Quick winCover equivalent
--hostsyntax in the bind-address test.The current assertion misses forms such as
--host=0.0.0.0, allowing a non-loopback example to regress while the test still passes. Extract the documented--hostvalues and assert that every value is127.0.0.1.Proposed test adjustment
- expect(markdown).toContain("--host 127.0.0.1"); - expect(markdown).not.toContain("--host 0.0.0.0"); + const hostValues = [...markdown.matchAll(/--host(?:=|\s+)([^\s`]+)/g)].map( + ([, value]) => value, + ); + expect(hostValues).toEqual(["127.0.0.1"]);🤖 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 `@test/inference-options-docs.test.ts` around lines 222 - 227, Update the loopback bind test in the test named "uses a loopback-only bind for the local vLLM server example" to extract host values from both `--host VALUE` and `--host=VALUE` syntax, then assert every extracted value equals `127.0.0.1` instead of relying on substring checks.Source: Path instructions
🤖 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.
Inline comments:
In `@scripts/check-docs-published-routes.ts`:
- Around line 290-297: Update findBrokenPublishedInferenceRoutes to match both
inference section roots and nested routes: accept resolved paths ending with
“/inference” as well as paths containing “/inference/”, preserving the existing
filtering behavior.
---
Nitpick comments:
In `@test/check-docs-published-routes.test.ts`:
- Around line 117-135: Extend the test “validates variant redirect destinations
independently” with a redirect whose source is fixed and whose destination
contains :variant, then assert findBrokenPublishedRedirects reports the expected
expanded destination and variant. Keep the assertion focused on observable
validation output and preserve the existing source-and-destination variant
coverage.
In `@test/inference-options-docs.test.ts`:
- Around line 222-227: Update the loopback bind test in the test named "uses a
loopback-only bind for the local vLLM server example" to extract host values
from both `--host VALUE` and `--host=VALUE` syntax, then assert every extracted
value equals `127.0.0.1` instead of relying on substring checks.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: 0a02b5e6-9a48-4367-a19b-c5cd4a104043
📒 Files selected for processing (15)
.agents/skills/nemoclaw-contributor-update-docs/SKILL.mddocs/about/release-notes.mdxdocs/index.ymldocs/inference/choose-local-inference-server.mdxdocs/inference/how-inference-routing-works.mdxdocs/inference/set-up-model-router.mdxdocs/inference/set-up-nvidia-nim.mdxdocs/inference/set-up-ollama.mdxdocs/inference/set-up-vllm.mdxdocs/inference/use-hermes-provider.mdxdocs/inference/use-openrouter.mdxfern/docs.ymlscripts/check-docs-published-routes.tstest/check-docs-published-routes.test.tstest/inference-options-docs.test.ts
✅ Files skipped from review due to trivial changes (8)
- docs/inference/use-hermes-provider.mdx
- docs/inference/set-up-nvidia-nim.mdx
- docs/inference/use-openrouter.mdx
- docs/inference/set-up-model-router.mdx
- docs/inference/set-up-vllm.mdx
- .agents/skills/nemoclaw-contributor-update-docs/SKILL.md
- docs/inference/how-inference-routing-works.mdx
- docs/about/release-notes.mdx
🚧 Files skipped from review as they are similar to previous changes (3)
- docs/index.yml
- fern/docs.yml
- docs/inference/set-up-ollama.mdx
Signed-off-by: Miyoung Choi <miyoungc@nvidia.com>
Signed-off-by: Miyoung Choi <miyoungc@nvidia.com>
miyoungc
marked this pull request as ready for review
July 10, 2026 23:35
Contributor
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 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.
Inline comments:
In `@docs/inference/model-capability-audit.mdx`:
- Around line 159-168: Fix the provider link in the “Next Steps” section by
updating the link target in “Choose an Inference Provider” to
../choose-inference-provider, removing the nonexistent learn-and-choose segment
while preserving the link text.
In `@test/inference-options-docs.test.ts`:
- Around line 247-252: Prevent the troubleshooting section extraction in “keeps
tool-calling remediation canonical in troubleshooting” from using -1 as the
slice endpoint. After locating the next heading with indexOf, use the markdown
length when no subsequent “###” section exists, so the final section is captured
completely and the test does not silently truncate it.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: bd24bc5d-d13d-4f57-ae4d-20985fde6656
📒 Files selected for processing (30)
docs/about/release-notes.mdxdocs/configure-agents/progressive-tool-disclosure.mdxdocs/index.ymldocs/inference/choose-compatible-inference-api.mdxdocs/inference/choose-local-inference-server.mdxdocs/inference/choose-model.mdxdocs/inference/configure-inference-timeouts.mdxdocs/inference/custom-endpoint-security.mdxdocs/inference/how-inference-routing-works.mdxdocs/inference/model-capability-audit.mdxdocs/inference/set-up-anthropic-compatible-endpoint.mdxdocs/inference/set-up-nvidia-nim.mdxdocs/inference/set-up-ollama.mdxdocs/inference/set-up-openai-compatible-endpoint.mdxdocs/inference/set-up-vllm.mdxdocs/inference/switch-models.mdxdocs/inference/understand-provider-validation.mdxdocs/inference/use-anthropic.mdxdocs/inference/use-google-gemini.mdxdocs/inference/use-nvidia-endpoints.mdxdocs/inference/use-openai.mdxdocs/inference/use-openrouter.mdxdocs/inference/verify-inference-route.mdxdocs/inference/view-active-inference-route.mdxdocs/reference/architecture.mdxdocs/reference/commands.mdxdocs/reference/troubleshooting.mdxfern/docs.ymltest/inference-options-docs.test.tstest/model-capability-audit-doc.test.ts
✅ Files skipped from review due to trivial changes (22)
- docs/inference/use-anthropic.mdx
- docs/inference/choose-model.mdx
- docs/inference/verify-inference-route.mdx
- docs/configure-agents/progressive-tool-disclosure.mdx
- docs/inference/use-openai.mdx
- docs/reference/architecture.mdx
- docs/reference/commands.mdx
- docs/inference/choose-local-inference-server.mdx
- docs/inference/configure-inference-timeouts.mdx
- docs/inference/use-openrouter.mdx
- docs/inference/switch-models.mdx
- docs/inference/view-active-inference-route.mdx
- docs/inference/use-google-gemini.mdx
- docs/inference/use-nvidia-endpoints.mdx
- docs/inference/how-inference-routing-works.mdx
- docs/inference/understand-provider-validation.mdx
- docs/inference/set-up-nvidia-nim.mdx
- docs/inference/custom-endpoint-security.mdx
- docs/inference/set-up-openai-compatible-endpoint.mdx
- docs/inference/set-up-anthropic-compatible-endpoint.mdx
- docs/about/release-notes.mdx
- docs/inference/set-up-vllm.mdx
🚧 Files skipped from review as they are similar to previous changes (4)
- test/model-capability-audit-doc.test.ts
- docs/index.yml
- fern/docs.yml
- docs/inference/set-up-ollama.mdx
Signed-off-by: Miyoung Choi <miyoungc@nvidia.com>
miyoungc
marked this pull request as draft
July 10, 2026 23:46
Contributor
There was a problem hiding this comment.
🧹 Nitpick comments (1)
test/inference-options-docs.test.ts (1)
151-153: 🎯 Functional Correctness | 🔵 Trivial | ⚡ Quick winClosing-fence backreference doesn't allow longer closing fences.
^\1[ \t]*$requires the closing fence to match the opening fence's exact string. CommonMark only requires the closing fence to have at least as many backticks/tildes as the opening one — a common pattern when nesting fenced examples with a longer outer fence. If anydocs/inference/*.mdxfile uses this pattern, the block won't be stripped and the compact-list-item test could false-positive on content that's actually inside a code fence.🔧 Proposed fix
- return markdown.replace(/^(`{3,}|~{3,})[^\n]*\n[\s\S]*?^\1[ \t]*$/gm, ""); + return markdown.replace(/^(`{3,}|~{3,})[^\n]*\n[\s\S]*?^\1+[ \t]*$/gm, "");🤖 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 `@test/inference-options-docs.test.ts` around lines 151 - 153, Update stripFencedCodeBlocks to capture the fence character and opening length separately, then accept a closing fence consisting of at least that many matching backticks or tildes with only trailing spaces or tabs. Preserve support for both fence styles and ensure longer closing fences are stripped correctly.
🤖 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 `@test/inference-options-docs.test.ts`:
- Around line 151-153: Update stripFencedCodeBlocks to capture the fence
character and opening length separately, then accept a closing fence consisting
of at least that many matching backticks or tildes with only trailing spaces or
tabs. Preserve support for both fence styles and ensure longer closing fences
are stripped correctly.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: 13e0fb7d-b3d0-469d-84a0-e0ee858e9aff
📒 Files selected for processing (2)
.coderabbit.yamltest/inference-options-docs.test.ts
Signed-off-by: Miyoung Choi <miyoungc@nvidia.com>
miyoungc
marked this pull request as ready for review
July 10, 2026 23:52
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
21 tasks
Collaborator
|
Follow-up review completed on
Verification:
|
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
Collaborator
|
Advisor follow-up for the current head:
|
cv
approved these changes
Jul 11, 2026
Hadar301
pushed a commit
to Hadar301/NemoClaw-OpenShift
that referenced
this pull request
Jul 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Reorganize the inference documentation into nested, task-focused pages so readers no longer have to navigate five oversized guides.
Add variant-aware navigation, stable redirects, and route validation that keeps the new published paths correct across OpenClaw, Hermes, and Deep Agents.
Changes
Type of Change
Quality Gates
Verification
Verifiedin GitHubpre-commit,commit-msg, andpre-pushhooks passed, ornpm run check:diffpassed when hooks were skipped or unavailablenpx vitest run test/check-docs-published-routes.test.ts test/model-capability-audit-doc.test.ts test/inference-options-docs.test.tspassed with 25 tests.npm testfor broad runtime/test-harness changes;npm run checkfor repo-wide validation/coverage changes — command/result:npm run docsbuilds without warnings (doc changes only) — build passed with 0 errors and two warnings: unauthenticated redirect checking and the existing light-mode accent contrast warning.Signed-off-by: Miyoung Choi miyoungc@nvidia.com
Summary by CodeRabbit