docs: complete v0.0.96 post-tag audit fixes - #7758
Conversation
Signed-off-by: Miyoung Choi <miyoungc@nvidia.com>
|
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. |
|
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)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughDocumentation updates clarify endpoint authentication rules, Ollama onboarding and proxy-token behavior, a Windows setup reference, and sandbox dependency review headings. ChangesDocumentation updates
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
🌿 Preview your docs: https://nvidia-preview-pr-7758.docs.buildwithfern.com/nemoclaw |
PR Review Advisor — No blocking findings reportedAdvisor assessment: No blocking advisor findings reported Model lanes
Nemotron output stays in workflow artifacts and does not change the assessment above. E2E guidanceAdvisory only. E2E / PR Gate selects and runs jobs independently. Recommended E2E: None 3 optional E2E recommendations
This automated review informs maintainers. Warnings and suggestions do not require a response. A maintainer decides whether to merge. |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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/custom-endpoint-security.mdx`:
- Around line 20-22: Update the non-interactive onboarding sentence in the
custom endpoint security documentation to explicitly scope
NEMOCLAW_COMPATIBLE_AUTH_MODE=none to OpenAI-compatible endpoints. Preserve the
existing localhost no-auth guidance and the documented distinction that
Anthropic-compatible onboarding requires a non-empty authentication value.
🪄 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: 4b2c6a1f-20fc-4936-b226-e8971f71e301
📒 Files selected for processing (4)
docs/changelog/2026-07-25.mdxdocs/inference/custom-endpoint-security.mdxdocs/inference/set-up-ollama.mdxdocs/security/sandbox-base-2026-07-25-dependency-review.md
Signed-off-by: Julie Yaunches <jyaunches@nvidia.com>
Summary
This PR corrects post-tag v0.0.96 documentation drift for exact-loopback no-auth compatible endpoints and local Ollama onboarding.
It also aligns the Windows changelog link label and source-only dependency review headings with their canonical titles and writing rules.
Changes
Type of Change
Quality Gates
Documentation Writer Review
docs-updateddocs/changelog/2026-07-25.mdx,docs/inference/custom-endpoint-security.mdx,docs/inference/set-up-ollama.mdx, anddocs/security/sandbox-base-2026-07-25-dependency-review.mdagainstdocs/CONTRIBUTING.mdandWRITING.md. The exact-head review found no remaining findings.DGX Station Hardware Evidence
Verification
Signed-off-by:line and every commit appears asVerifiedin GitHubpre-commit,commit-msg, andpre-pushhooks passed, ornpm run check:diffpassed when hooks were skipped or unavailablenpx vitest run test/changelog-docs.test.ts test/check-docs-links.test.tspassed 21 tests.npm testfor broad runtime/test-harness changes;npm run checkfor repo-wide validation/coverage changes — not run because this is a focused documentation-only change.npm run docsbuilds without warnings (doc changes only). The command passed with 0 errors and two Fern warnings.Signed-off-by: Miyoung Choi miyoungc@nvidia.com
Summary by CodeRabbit