docs: catch up documentation for v0.0.17 changes#1968
Conversation
Document Ollama network exposure warning, snapshot restore liveness preflight and clean restore behavior, Jetson BSP R39+ support, custom Dockerfile permission error handling, and bump doc versions through 0.0.17. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughThe pull request updates various nemoclaw documentation files and project metadata: clarifies onboarding with Docker build-context unreadable files, snapshot restore requirements and behavior, adds Jetson BSP R39+ handling note, adds an Ollama network-exposure warning during onboarding, and inserts a new preferred docs version 0.0.17. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 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 unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/reference/troubleshooting.md`:
- Line 522: The sentence in docs/reference/troubleshooting.md that states "The
warning is suppressed on WSL, where Ollama binds to `127.0.0.1` instead." should
be changed to clarify this is the default behavior and not a guarantee: reword
it to say that on WSL Ollama defaults to binding to 127.0.0.1 but explicit
OLLAMA_HOST environment variables are respected (as implemented in
bin/lib/onboard.js), so the warning may not be suppressed if a user has exported
OLLAMA_HOST; keep the explanation concise and reference the behavior rather than
asserting it as absolute.
🪄 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: Pro Plus
Run ID: 24ca997e-5f33-4ffe-a51d-fb039ad8da68
📒 Files selected for processing (5)
.agents/skills/nemoclaw-user-reference/references/commands.md.agents/skills/nemoclaw-user-reference/references/troubleshooting.mddocs/reference/commands.mddocs/reference/troubleshooting.mddocs/versions1.json
| On trusted private networks, the warning is informational. | ||
| On shared or public networks (airports, coffee shops), any adjacent device can send prompts to and enumerate models on your Ollama instance. | ||
|
|
||
| The warning is suppressed on WSL, where Ollama binds to `127.0.0.1` instead. |
There was a problem hiding this comment.
Avoid absolute WSL binding wording here.
Line 522 currently implies WSL always binds Ollama to 127.0.0.1. That is not always true if a user explicitly exports OLLAMA_HOST, so this should be phrased as default behavior rather than a guarantee.
Based on learnings: In bin/lib/onboard.js (NemoClaw project), the WSL2 Ollama startup path intentionally does NOT clear an inherited OLLAMA_HOST environment variable and respects explicit user configuration.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/reference/troubleshooting.md` at line 522, The sentence in
docs/reference/troubleshooting.md that states "The warning is suppressed on WSL,
where Ollama binds to `127.0.0.1` instead." should be changed to clarify this is
the default behavior and not a guarantee: reword it to say that on WSL Ollama
defaults to binding to 127.0.0.1 but explicit OLLAMA_HOST environment variables
are respected (as implemented in bin/lib/onboard.js), so the warning may not be
suppressed if a user has exported OLLAMA_HOST; keep the explanation concise and
reference the behavior rather than asserting it as absolute.
Summary
--fromDockerfile permission error handling (from fix: catch permission errors when copying custom Dockerfile build context (#1808) #1931)Test plan
make docsbuilds without warnings🤖 Generated with Claude Code
Summary by CodeRabbit