chore(install): bump OpenShell version to 0.0.32 - #2307
Conversation
Bumps the pinned OpenShell version range from 0.0.29 to 0.0.32 so fresh NemoClaw installs pick up sandbox hardening and TLS improvements from the last three OpenShell releases. Notable upstream changes 0.0.29 → 0.0.32: - 0.0.30: network policy deny rules, preserve ownership on read_write paths, disable child core dumps, escape control chars in SSE errors, fix silent truncation of large streaming responses (NVIDIA/OpenShell#822, #827, #821, #842, #834) - 0.0.31: inference routed-request header allowlist (NVIDIA/OpenShell#826) - 0.0.32: load system CA certs for upstream TLS connections (NVIDIA/OpenShell#862) Changes: - nemoclaw-blueprint/blueprint.yaml: min and max openshell version - scripts/install-openshell.sh: MIN_VERSION and MAX_VERSION - scripts/brev-launchable-ci-cpu.sh: default OPENSHELL_VERSION - src/lib/onboard.ts: blueprint-fallback min version - test/onboard.test.ts, test/install-openshell-version-check.test.ts: fixtures updated; "above MAX" test case moved from 0.0.30 to 0.0.33 Historical "m-dev" comments referencing 0.0.29 left in place — they describe a self-report quirk the sidecar fallback still handles. Signed-off-by: Prekshi Vyas <prekshiv@nvidia.com> Co-Authored-By: Claude Opus 4.7 (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)
📝 WalkthroughWalkthroughConsistent OpenShell version bump from 0.0.29 to 0.0.32 across blueprint metadata, install/CI scripts, onboarding preflight fallback, and corresponding tests. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
|
This bump may close:
Signed-off-by: Prekshi Vyas prekshiv@nvidia.com |
ericksoa
left a comment
There was a problem hiding this comment.
Clean version bump, all references consistent. Good rationale for stopping at 0.0.32 and deferring 0.0.34+. LGTM.
Summary
Bumps the pinned OpenShell version range from
0.0.29→0.0.32so fresh NemoClaw installs pick up sandbox hardening and TLS improvements from the last three OpenShell releases.Notable upstream changes
0.0.30 (NVIDIA/OpenShell@v0.0.29...v0.0.30)
read_writepaths (OpenShell#827)0.0.31 (NVIDIA/OpenShell@v0.0.30...v0.0.31)
0.0.32 (NVIDIA/OpenShell@v0.0.31...v0.0.32)
openshell-gatewaybinaries (OpenShell#853)Changes
nemoclaw-blueprint/blueprint.yaml:min_openshell_versionandmax_openshell_version→0.0.32scripts/install-openshell.sh:MIN_VERSIONandMAX_VERSION→0.0.32(PIN_VERSIONfollowsMAX)scripts/brev-launchable-ci-cpu.sh: defaultOPENSHELL_VERSION→v0.0.32src/lib/onboard.ts: blueprint-fallback min version →0.0.32test/onboard.test.ts,test/install-openshell-version-check.test.ts: fixtures updated; "above MAX" test case moved from0.0.30to0.0.33Historical
m-devcomments referencing0.0.29left in place — they describe a self-report quirk the sidecar fallback still handles.Why not 0.0.33+?
0.0.34introduced incremental sandbox policy updates and L7 request-target canonicalization — changes with larger surface area against how NemoClaw delivers policy via gRPC. Worth a follow-up PR rather than bundling here.0.0.35released hours before this PR was cut — too fresh.Type of Change
Testing
npx vitest run test/install-openshell-version-check.test.ts— 9 passedNotes
test/onboard.test.tsreproduce on cleanmainand are unrelated to this bump.Signed-off-by: Prekshi Vyas prekshiv@nvidia.com
🤖 Generated with Claude Code
Summary by CodeRabbit
Chores
Tests