docs: catch up documentation for v0.0.18 changes#2033
Conversation
Refresh user-facing docs against the 34 commits merged between v0.0.17 and v0.0.18. Highlights: - Replace the Ollama 0.0.0.0 binding guidance with the new authenticated reverse proxy on 127.0.0.1:11435 (#1922). - Document the compatible-endpoint provider defaulting to /v1/chat/completions and the NEMOCLAW_PREFERRED_API=openai-responses opt-in (#1984). - Add the new nemoclaw upgrade-sandboxes command with --check, --auto, and --yes flags (#1943). - Note the cross-sandbox messaging overlap warning and 409 detection in nemoclaw <name> status (#1953). - Document the messaging-token rotation auto-rebuild flow (#1967). - Cover new troubleshooting entries for the Ollama auth proxy, IPv6 localhost resolution, orphan SSH port-forward cleanup on re-onboard, and rotated messaging credentials (#1978, #1950). - Note tar failure exit code for nemoclaw debug --output (#1770) and the orphaned openshell process cleanup in nemoclaw uninstall (#1940). Also: - Extend docs/.docs-skip to exclude the experimental sandbox-mgmt shields and config commands (#1976). - Fix a sphinx-autobuild infinite rebuild loop in docs/conf.py by writing docs/project.json only when its contents change. - Bump the docs version switcher preferred entry to 0.0.18. - Regenerate nemoclaw-user-* agent skills from docs/. Signed-off-by: Miyoung Choi <miyoungc@nvidia.com> Made-with: Cursor
📝 WalkthroughWalkthroughThis pull request updates documentation and configuration across multiple files to reflect changes in NemoClaw's API selection logic, Ollama proxy authentication setup, messaging token management, and introduces a new sandbox upgrade command. Version bumped from 0.0.17 to 0.0.18. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (4)
docs/.docs-skip (1)
37-37: Narrow this skip pattern to avoid hiding stable doc updates.
"src/lib/sandbox-config.ts"will skip any commit touching that file, including future non-experimental changes. Prefer narrower commit-message/path substrings tied specifically to experimental config commands (e.g., rotate-token/show only).🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/.docs-skip` at line 37, The skip entry currently matches the broad literal "src/lib/sandbox-config.ts" which will hide all commits touching that file; narrow it by replacing that exact pattern with a more specific substring tied to experimental commands (e.g., include the command names or TODO markers used in commits such as "rotate-token", "show" or a commit-tag like "sandbox-config:rotate-token") so only commits that change the experimental rotate-token/show code are skipped; update the skip line to reference those unique identifiers (rotate-token, show, sandbox-config:rotate-token) instead of the full filename.docs/inference/switch-inference-providers.md (1)
91-94: Use one sentence per line in source for this paragraph.These sentences are wrapped across multiple lines; docs style requires one full sentence per line for clean diffs.
As per coding guidelines: "One sentence per line in source (makes diffs readable). Flag paragraphs where multiple sentences appear on the same line."
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/inference/switch-inference-providers.md` around lines 91 - 94, The paragraph starting "For the compatible-endpoint provider, NemoClaw uses `/v1/chat/completions` by default, so no env var is required to keep the safe path. To opt in to `/v1/responses` for a backend you have verified end to end, set `NEMOCLAW_PREFERRED_API` before onboarding:" should be reformatted so each sentence is on its own source line; split the two sentences into separate lines (one line with "For the compatible-endpoint provider..." and a second line with "To opt in to `/v1/responses` ... set `NEMOCLAW_PREFERRED_API` before onboarding:") to satisfy the "one sentence per line" docs style.docs/inference/use-local-inference.md (1)
58-84: Apply one-sentence-per-line formatting in the newly added sections.The new prose is wrapped mid-sentence across lines; keep each sentence on one line to match the docs source style.
As per coding guidelines: "One sentence per line in source (makes diffs readable). Flag paragraphs where multiple sentences appear on the same line."
Also applies to: 152-167
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/inference/use-local-inference.md` around lines 58 - 84, The new documentation block describing the NemoClaw/Ollama proxy contains multiple sentences on the same lines (e.g., the paragraph starting "NemoClaw keeps Ollama bound..." and the paragraph mentioning the token file `~/.nemoclaw/ollama-proxy-token`, proxy ports `127.0.0.1:11434`/`0.0.0.0:11435`, and the exempted endpoint `GET /api/tags`), which violates the one-sentence-per-line docs rule; edit those paragraphs so each sentence is on its own line (split any mid-sentence line wraps into full sentences per line) and apply the same fix to the other mentioned block around the section that references the sandbox provider and `OPENAI_API_KEY` so all sentences in these newly added sections are one sentence per line.docs/reference/troubleshooting.md (1)
247-273: Normalize these updated troubleshooting paragraphs to one sentence per line.Several newly added sentences are hard-wrapped across multiple source lines; keep each sentence on a single line to match docs style.
As per coding guidelines: "One sentence per line in source (makes diffs readable). Flag paragraphs where multiple sentences appear on the same line."
Also applies to: 408-425, 569-597
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/reference/troubleshooting.md` around lines 247 - 273, Normalize the hard-wrapped troubleshooting paragraphs so each sentence occupies its own source line: in the paragraph about orphaned SSH port-forward processes (the block starting "After destroying a sandbox and gateway...") and the paragraph about rotated messaging tokens (the block starting "Updated messaging token is not picked up"), reflow lines so every sentence is on a single line; apply the same one-sentence-per-line normalization to the other reported ranges (the blocks referenced as 408-425 and 569-597) to match the docs style guide.
🤖 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/.docs-skip`:
- Line 37: The skip entry currently matches the broad literal
"src/lib/sandbox-config.ts" which will hide all commits touching that file;
narrow it by replacing that exact pattern with a more specific substring tied to
experimental commands (e.g., include the command names or TODO markers used in
commits such as "rotate-token", "show" or a commit-tag like
"sandbox-config:rotate-token") so only commits that change the experimental
rotate-token/show code are skipped; update the skip line to reference those
unique identifiers (rotate-token, show, sandbox-config:rotate-token) instead of
the full filename.
In `@docs/inference/switch-inference-providers.md`:
- Around line 91-94: The paragraph starting "For the compatible-endpoint
provider, NemoClaw uses `/v1/chat/completions` by default, so no env var is
required to keep the safe path. To opt in to `/v1/responses` for a backend you
have verified end to end, set `NEMOCLAW_PREFERRED_API` before onboarding:"
should be reformatted so each sentence is on its own source line; split the two
sentences into separate lines (one line with "For the compatible-endpoint
provider..." and a second line with "To opt in to `/v1/responses` ... set
`NEMOCLAW_PREFERRED_API` before onboarding:") to satisfy the "one sentence per
line" docs style.
In `@docs/inference/use-local-inference.md`:
- Around line 58-84: The new documentation block describing the NemoClaw/Ollama
proxy contains multiple sentences on the same lines (e.g., the paragraph
starting "NemoClaw keeps Ollama bound..." and the paragraph mentioning the token
file `~/.nemoclaw/ollama-proxy-token`, proxy ports
`127.0.0.1:11434`/`0.0.0.0:11435`, and the exempted endpoint `GET /api/tags`),
which violates the one-sentence-per-line docs rule; edit those paragraphs so
each sentence is on its own line (split any mid-sentence line wraps into full
sentences per line) and apply the same fix to the other mentioned block around
the section that references the sandbox provider and `OPENAI_API_KEY` so all
sentences in these newly added sections are one sentence per line.
In `@docs/reference/troubleshooting.md`:
- Around line 247-273: Normalize the hard-wrapped troubleshooting paragraphs so
each sentence occupies its own source line: in the paragraph about orphaned SSH
port-forward processes (the block starting "After destroying a sandbox and
gateway...") and the paragraph about rotated messaging tokens (the block
starting "Updated messaging token is not picked up"), reflow lines so every
sentence is on a single line; apply the same one-sentence-per-line normalization
to the other reported ranges (the blocks referenced as 408-425 and 569-597) to
match the docs style guide.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: b7219321-579b-4adf-9916-c2df34ed7b37
📒 Files selected for processing (13)
.agents/skills/nemoclaw-user-configure-inference/SKILL.md.agents/skills/nemoclaw-user-deploy-remote/SKILL.md.agents/skills/nemoclaw-user-reference/references/commands.md.agents/skills/nemoclaw-user-reference/references/troubleshooting.mddocs/.docs-skipdocs/conf.pydocs/deployment/set-up-telegram-bridge.mddocs/inference/switch-inference-providers.mddocs/inference/use-local-inference.mddocs/project.jsondocs/reference/commands.mddocs/reference/troubleshooting.mddocs/versions1.json
…d reasoning env vars (#2108) ## Summary Documents the `NEMOCLAW_CONTEXT_WINDOW`, `NEMOCLAW_MAX_TOKENS`, and `NEMOCLAW_REASONING` build-time environment variables that landed in #1956. These variables now propagate through the Dockerfile ARG chain into `openclaw.json`, but were not mentioned anywhere in the published docs. ## Changes - Add a "Tune Model Metadata" section to `docs/inference/switch-inference-providers.md` with a variable reference table, defaults, value constraints, an example, and the `nemoclaw onboard --resume --recreate-sandbox` flow for applying changes to existing sandboxes. - Regenerate `.agents/skills/nemoclaw-user-configure-inference/SKILL.md` from the updated source. - Sync `.agents/skills/nemoclaw-user-reference/references/troubleshooting.md` with the current `docs/reference/troubleshooting.md` (brings the autogen reference back in line with the `openclaw update` entry added in #2033). ## Type of Change - [ ] Code change (feature, bug fix, or refactor) - [ ] Code change with doc updates - [x] Doc only (prose changes, no code sample modifications) - [ ] Doc only (includes code sample changes) ## Verification - [x] `npx prek run --all-files` passes - [ ] `npm test` passes - [ ] Tests added or updated for new or changed behavior - [x] No secrets, API keys, or credentials committed - [x] Docs updated for user-facing behavior changes - [x] `make docs` builds without warnings (doc changes only) - [x] Doc pages follow the [style guide](https://github.com/NVIDIA/NemoClaw/blob/main/docs/CONTRIBUTING.md) (doc changes only) - [ ] New doc pages include SPDX header and frontmatter (new pages only) ## AI Disclosure - [x] AI-assisted — tool: Cursor --- Signed-off-by: Miyoung Choi <miyoungc@nvidia.com> Made with [Cursor](https://cursor.com) <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Documentation** * Added guide for tuning model metadata via build-time configuration variables. * Documented three new parameters for context window, max tokens, and reasoning, including accepted values, defaults, and that invalid values are ignored. * Clarified that changes apply during image build and require recreating a sandbox to take effect. * Simplified update-timeout troubleshooting with a single rebuild workflow. * Updated documentation versioning to include the new release. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --------- Signed-off-by: Miyoung Choi <miyoungc@nvidia.com>
Summary
Refresh user-facing documentation against the 34 commits merged between
v0.0.17 and v0.0.18, bump the docs version switcher to v0.0.18, and fix a
sphinx-autobuildinfinite-rebuild loop triggered bydocs/conf.py.Changes
0.0.0.0:11434guidance indocs/inference/use-local-inference.mdwiththe new token-gated proxy on
127.0.0.1:11435, including persisted token,health-check exemption, and sandbox provider wiring. Replace the matching
troubleshooting entry in
docs/reference/troubleshooting.md.compatible-endpoint provider now defaults to
/v1/chat/completionsandupdate
NEMOCLAW_PREFERRED_APIto describeopenai-responsesas theopt-in instead of
openai-completions. Updates inuse-local-inference.md,switch-inference-providers.md, andtroubleshooting.md.nemoclaw upgrade-sandboxescommand (fix(upgrade): pin base image via pull-then-inspect and rebuild stale sandboxes #1943): Add a new referenceentry in
docs/reference/commands.mdcovering--check,--auto, and--yesflags.automatic rebuild behavior and cross-sandbox overlap warning in
docs/deployment/set-up-telegram-bridge.md,commands.md, andtroubleshooting.md.localhost→127.0.0.1IPv6 note (fix(onboard): use 127.0.0.1 instead of localhost for local inference detection #1978)openshellprocess cleanup innemoclaw uninstall(nemoclaw uninstall does not clear all the openshell process #1940)nemoclaw debug --output(fix(cli): detect tar failure in debug --output and set non-zero exit code #1770)docs/.docs-skipto exclude the experimentalsandbox-mgmt shields and config commands feature (feat(sandbox-mgmt): experimental shields and config commands #1976), which was
explicitly merged as not-yet-documented.
docs/conf.pynow writesdocs/project.jsononlywhen contents change, so
make docs-live/sphinx-autobuildno longerdetects its own generated file as a source change and enters an infinite
rebuild loop.
docs/versions1.jsonanddocs/project.jsonpreferred entry to v0.0.18 so this refresh renders under the new version.
nemoclaw-user-*skills fromdocs/withscripts/docs-to-skills.py.Type of Change
Verification
npx prek run --all-filespasses (ran via pre-commit hook on staged files)npm testpassesmake docsbuilds without warnings (doc changes only)AI Disclosure
Signed-off-by: Miyoung Choi miyoungc@nvidia.com
Made with Cursor
Summary by CodeRabbit
Release Notes
New Features
nemoclaw upgrade-sandboxescommand to rebuild sandboxes when base-image digests change.Improvements
/v1/chat/completionsAPI path.Documentation