docs: add v0.0.96 release follow-up - #7649
Conversation
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
📝 WalkthroughWalkthroughThe v0.0.96 changelog documents policy exclusion persistence, diagnostic and error handling, onboarding and recovery flows, backup/restore safeguards, and OpenClaw base-image selection rules. Changesv0.0.96 operational behavior
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
|
🌿 Preview your docs: https://nvidia-preview-pr-7649.docs.buildwithfern.com/nemoclaw |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (1)
docs/changelog/2026-07-25.mdx (1)
62-62: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winUse active voice in the recovery and image-selection clauses.
Rewrite the passive constructions so the product or operator is the subject, for example: “NemoClaw rejects corrupt persisted Shields state…” and “onboarding refreshes an incompatible cached exact release once, then uses a current local build…”
As per coding guidelines, MDX documentation should use active voice and direct language.
Also applies to: 72-72
🤖 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/changelog/2026-07-25.mdx` at line 62, Rewrite the recovery and image-selection clauses in the changelog entry using active voice and direct language, making NemoClaw or the operator the subject; update both the corrupt-state statement and the incompatible cached-release/onboarding statement while preserving their existing behavior and conditions.Sources: Coding guidelines, Learnings
🤖 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/changelog/2026-07-25.mdx`:
- Line 62: Rewrite the recovery and image-selection clauses in the changelog
entry using active voice and direct language, making NemoClaw or the operator
the subject; update both the corrupt-state statement and the incompatible
cached-release/onboarding statement while preserving their existing behavior and
conditions.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: af73db3e-174a-48b7-a7e7-639a5feaf3de
📒 Files selected for processing (1)
docs/changelog/2026-07-25.mdx
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
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/changelog/2026-07-25.mdx`:
- Around line 62-63: Rephrase the recovery sentence in the NemoClaw changelog
entry using third-person historical product-behavior wording, such as stating
that operators must restore the state from a trusted host backup. Preserve the
existing warning that “shields up” and an ordinary rebuild do not replace
corrupt persisted state.
🪄 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: f3788f36-4b81-4325-bb8c-9b343c626426
📒 Files selected for processing (1)
docs/changelog/2026-07-25.mdx
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 6 optional E2E recommendations
This automated review informs maintainers. Warnings and suggestions do not require a response. A maintainer decides whether to merge. |
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
1 participant
Summary
docs/changelog/2026-07-25.mdxnow includes the user-facing fixes that merged after #7607 and before the v0.0.96 tag.The follow-up covers safer bulk backup and clone restore behavior, policy and inference repairs, cleaner onboarding diagnostics, and OpenClaw base-image validation while leaving test-only and maintainer-internal merges out of the release entry.
Changes
backup-allflow from fix(backup): preserve Shields during backup-all #7557 and the clone-specific restore pairing publication from fix(restore): restart gateway after pairing approval #7608.Type of Change
Quality Gates
Documentation Writer Review
docs-updated29316da26, a Codex Desktop documentation writer revieweddocs/changelog/2026-07-25.mdxagainstAGENTS.md,WRITING.md, anddocs/CONTRIBUTING.md. The review confirmed that the full entry accurately reflects the merged user-visible behavior, retains fix(images): support Docker Engine gateway builds #7622's existing wording, appropriately excludes internal and test-only PRs, and uses conforming terminology, structure, links, and release classification. It also confirmed that the review follow-ups use active third-person release-entry voice, name the actor and recovery requirement directly, and accurately preserve the trusted-backup, cached-release refresh, and local-build fallback constraints. The changelog test passed 6/6, and the docs build completed with 0 errors and 2 pre-existing hidden warnings.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.tspassed 6/6 tests after the final review fix.npm testfor broad runtime/test-harness changes;npm run checkfor repo-wide validation/coverage changes — command/result: Not applicable to this prose-only changelog change.npm run docsbuilds without warnings (doc changes only) — exited 0 with 0 errors and 2 pre-existing hidden warnings after the final review fix.Signed-off-by: Carlos Villela cvillela@nvidia.com
Summary by CodeRabbit
policy exclude/policy restorebehavior across rebuilds and snapshot restores, including reporting on removed endpoints and exclusion consistency.claude-codepreset guidance to allow the npm-installed OpenShell launcher path while maintaining endpoint/HTTP method scope.:latestfallback.