Skip to content

chore(deps): align Node and advisor SDK requirements - #6729

Merged
cv merged 3 commits into
mainfrom
codex/node-22-19-pi-0-80-6
Jul 13, 2026
Merged

chore(deps): align Node and advisor SDK requirements#6729
cv merged 3 commits into
mainfrom
codex/node-22-19-pi-0-80-6

Conversation

@cv

@cv cv commented Jul 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

Raises NemoClaw's supported Node.js minimum from 22.16 to 22.19 so the host and contributor contract matches the current OpenClaw runtime and advisor SDK requirements.
Upgrades both advisor workflows to Pi SDK 0.80.6, removes the SDK advisories present in 0.74.x, and pins TypeBox explicitly where the PR advisor imports it directly.

Changes

  • Align package engines, installer and developer checks, the exact-minimum CI lane, fixtures, tests, and user-facing prerequisites on Node.js 22.19.
  • Upgrade the PR and E2E advisor SDK pins to @earendil-works/pi-coding-agent@0.80.6 with a reviewed July 11 dependency cutoff.
  • Declare typebox@1.1.38 directly and install it explicitly in the PR advisor workflow because Pi 0.80.6 shrinkwraps its own nested dependency copy.
  • Extend the PR advisor workflow boundary and install-step test to protect the direct TypeBox runtime dependency.

Type of Change

  • Code change (feature, bug fix, or refactor)
  • Code change with doc updates
  • Doc only (prose changes, no code sample modifications)
  • Doc only (includes code sample changes)

Quality Gates

  • Tests added or updated for changed behavior
  • Existing tests cover changed behavior — justification:
  • Tests not applicable — justification:
  • Docs updated for user-facing behavior changes
  • Docs not applicable — justification:
  • Sensitive paths changed (security, policy, credentials, preflight, onboarding, inference, runner, sandbox, or messaging)
  • Sensitive-path review completed or maintainer-approved waiver recorded — reviewer/approval link/justification: Pending maintainer review; no waiver requested.
  • Non-success, skipped, or missing CI check accepted by maintainer — check name, approval link, and follow-up issue: None.

Verification

  • PR description includes the DCO sign-off declaration and every commit appears as Verified in GitHub
  • Normal pre-commit, commit-msg, and pre-push hooks passed, or npm run check:diff passed when hooks were skipped or unavailable
  • Targeted behavior tests pass for the current change set, or tests are marked not applicable above — Node.js 22.19: CLI typecheck passed; 3 CLI files/11 tests, 17 integration files/318 tests, and 2 installer-integration files/96 tests passed; isolated PR and E2E advisor runtime imports passed.
  • Applicable broad gate passed — npm test for broad runtime/test-harness changes; npm run check for repo-wide validation/coverage changes — command/result: Not run; the focused version, installer, workflow, and advisor lanes cover this dependency change.
  • Quality Gates section completed with required justifications or waivers
  • No secrets, API keys, or credentials committed
  • npm run docs builds without warnings (doc changes only) — Build passed with 0 errors; Fern reported an unauthenticated redirects warning and the existing light-theme contrast warning.
  • Doc pages follow the style guide (doc changes only)
  • New doc pages include SPDX header and frontmatter (new pages only)

Signed-off-by: Carlos Villela cvillela@nvidia.com

Summary by CodeRabbit

  • Compatibility

    • Raised the minimum supported Node.js version to 22.19 across installer/runtime checks, fixtures, and tooling.
  • Maintenance

    • Updated pinned runtime Node.js versions in workflows to 22.19.
    • Upgraded the Pi SDK to 0.80.6 and pinned TypeBox where required.
    • Hardened the advisor workflow by cleaning up analysis workspace symlinks to prevent unsafe reads.
  • Documentation

    • Updated prerequisites and “Node.js too old” troubleshooting guidance to require 22.19.
  • Tests

    • Updated installer/workflow validations and assertions to expect 22.19 behavior, including advisor workflow boundary checks.

Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv cv added dependencies Pull requests that update a dependency file area: ci CI workflows, checks, release automation, or GitHub Actions area: docs Documentation, examples, guides, or docs build labels Jul 12, 2026
@cv cv self-assigned this Jul 12, 2026
@coderabbitai

coderabbitai Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 53a66eac-1990-4eaa-979a-34f8ea31c655

📥 Commits

Reviewing files that changed from the base of the PR and between fd28b0c and a866513.

📒 Files selected for processing (4)
  • .github/workflows/pr-review-advisor.yaml
  • ci/source-shape-test-budget.json
  • test/pr-review-advisor-workflow-boundary.test.ts
  • tools/pr-review-advisor/workflow-boundary.mts

📝 Walkthrough

Walkthrough

The pull request raises the repository’s minimum Node.js version from 22.16 to 22.19, updates installer validation and tests, refreshes documentation and package engines, and pins Pi SDK and TypeBox versions in advisor workflows.

Changes

Runtime baseline and validation

Layer / File(s) Summary
Node.js minimum enforcement
scripts/install.sh, scripts/dev-setup.sh, src/lib/domain/installer/version.ts, package.json, nemoclaw/package.json, agents/openclaw/mcporter-runtime/package.json, test/e2e/fixtures/plugins/weather/package.json, .github/workflows/installer-hash-check.yaml, test/pr-workflow-contract.test.ts, scripts/check-node-version.js
The minimum supported Node.js version and related runtime checks are updated to 22.19.0.
Node.js requirement documentation
AGENTS.md, CONTRIBUTING.md, docs/..., scripts/bench/README.md
Development, setup, troubleshooting, starter prompt, and benchmark guidance now require Node.js 22.19 or later.
Installer and runtime test updates
src/lib/.../*.test.ts, test/...
Fixtures and expectations use Node.js 22.19.0, with unsupported-runtime cases updated to Node.js 22.18.0 where applicable.

Advisor runtime and workspace protection

Layer / File(s) Summary
Advisor dependency pinning
.github/workflows/e2e-advisor.yaml, .github/workflows/pr-review-advisor.yaml, package.json, tools/pr-review-advisor/workflow-boundary.mts, test/pr-review-advisor-workflow-boundary.test.ts, docs/security/...
Pi SDK is pinned to 0.80.6, TypeBox 1.1.38 is installed explicitly, and the corresponding workflow boundary checks and dependency review record are updated.
Analysis workspace symlink protection
.github/workflows/pr-review-advisor.yaml, tools/pr-review-advisor/workflow-boundary.mts, test/pr-review-advisor-workflow-boundary.test.ts, ci/source-shape-test-budget.json
The workflow removes analysis-workspace symlinks before review execution, while validation and tests enforce the cleanup command and step ordering.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested labels: chore

Suggested reviewers: ericksoa, jyaunches

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 36.36% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main changes: raising Node requirements and updating advisor SDK dependencies.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/node-22-19-pi-0-80-6

Comment @coderabbitai help to get the list of available commands.

@github-code-quality

github-code-quality Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Code Coverage Overview

Languages: TypeScript

TypeScript / code-coverage/plugin

The overall coverage remains at 96%, unchanged from the main branch.

TypeScript / code-coverage/cli

The overall coverage in the codex/node-22-19-pi-... branch remains at 79%, unchanged from the main branch.

Show a code coverage summary of the most impacted files.
File main 13bece6 codex/node-22-19-pi-... a866513 +/-
src/lib/actions...l-validation.ts 95% 89% -6%
src/lib/actions...-add-restart.ts 19% 14% -5%
src/lib/securit...ntial-filter.ts 99% 97% -2%
src/lib/policy/index.ts 60% 61% +1%
src/lib/actions...e-validation.ts 88% 90% +2%
src/lib/state/config-io.ts 90% 93% +3%
src/lib/security/redact.ts 95% 100% +5%
src/lib/messagi...ridge-health.ts 91% 96% +5%
src/lib/messagi...onfig-prompt.ts 91% 97% +6%
src/lib/messagi.../token-paste.ts 82% 92% +10%

Updated July 13, 2026 00:23 UTC
Code Coverage is in Public Preview. Learn more and provide us with your feedback.

@github-actions

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

E2E Advisor Recommendation

Required E2E: cloud-onboard
Optional E2E: None

Dispatch hint: cloud-onboard

Workflow run

Full advisor summary

E2E Recommendation Advisor

Base: origin/main
Head: HEAD
Confidence: high

Required E2E

  • cloud-onboard (high): Required by the deterministic platform-install risk plan. It validates that the public installer works on a clean supported host with the pinned runtime dependencies and reaches a usable hosted sandboxed agent after the Node.js runtime-floor change.

Optional E2E

  • None.

New E2E recommendations

  • None.

Dispatch hint

  • Workflow: E2E
  • jobs input: cloud-onboard

@github-actions

github-actions Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

PR Review Advisor — No blocking findings

Merge posture: No blocking advisor findings
Primary next action: No advisor follow-up required beyond maintainer review.
Findings: 0 required · 0 warnings · 0 optional suggestions
Since last review: 1 prior item resolved · 0 still apply · 0 new items found

Workflow run details

This is an automated review. Required findings need action before merge. Warnings and optional suggestions do not require a response or follow-up. A human maintainer makes the final merge decision.

@cv cv added the v0.0.82 label Jul 12, 2026
cv added 2 commits July 12, 2026 15:56
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
cv merged commit deed1aa into main Jul 13, 2026
57 checks passed
@cv
cv deleted the codex/node-22-19-pi-0-80-6 branch July 13, 2026 00:47
cv pushed a commit that referenced this pull request Jul 14, 2026
<!-- markdownlint-disable MD041 -->
## Summary

Release-prep documentation for v0.0.82 now summarizes user-facing
changes merged since v0.0.81.
It also closes stale wording in the stopped-sandbox backup,
snapshot-clone, Ollama selection, and custom-policy authoring guidance.

## Changes

- Add the `v0.0.82` section to `docs/about/release-notes.mdx` with links
to the focused user guides.
- Document that snapshot clones receive a destination-owned dashboard
port before destructive replacement begins.
- Align `backup-all` guidance with eligible stopped Docker-driver
sandboxes that NemoClaw starts temporarily.
- Describe the running and stopped Ollama menu states without claiming
one fixed label.
- Document runtime rejection of catch-all hosts in custom policy files.

### Source summary

- [#6748](#6748) ->
`docs/about/release-notes.mdx`, `docs/manage-sandboxes/lifecycle.mdx`,
and `docs/reference/commands.mdx`: Summarize non-destructive sandbox
`stop` and `start` commands.
- [#6723](#6723) ->
`docs/about/release-notes.mdx`,
`docs/manage-sandboxes/backup-restore.mdx`, and
`docs/reference/commands.mdx`: Record temporary startup and cleanup for
eligible stopped-sandbox backups.
- [#6749](#6749) ->
`docs/about/release-notes.mdx` and
`docs/manage-sandboxes/backup-restore.mdx`: Document destination-owned
dashboard ports for snapshot clones.
- [#6764](#6764) ->
`docs/about/release-notes.mdx`: Summarize installer handling of
route-only onboarding placeholders.
- [#6771](#6771) ->
`docs/about/release-notes.mdx`, `docs/inference/set-up-vllm.mdx`,
`docs/inference/choose-inference-provider.mdx`,
`docs/reference/commands.mdx`, and
`docs/reference/platform-support.mdx`: Summarize managed-vLLM storage
gates, immutable image digests, and the explicit override boundary.
- [#6759](#6759) ->
`docs/about/release-notes.mdx`: Record early, actionable OpenShell
gateway-port conflict diagnostics.
- [#6753](#6753) ->
`docs/about/release-notes.mdx` and `docs/inference/set-up-ollama.mdx`:
Document truthful running and stopped Ollama menu states.
- [#6776](#6776) ->
`docs/about/release-notes.mdx`: Summarize proxy-independent loopback
readiness checks.
- [#6769](#6769) ->
`docs/about/release-notes.mdx`: Record compatible endpoint and agent
guidance when Chat Completions is unavailable.
- [#6730](#6730) ->
`docs/about/release-notes.mdx`: Summarize bounded reuse of an eligible
successful Chat Completions check.
- [#6768](#6768) ->
`docs/about/release-notes.mdx`: Record route-reservation repair during
resumed onboarding.
- [#6742](#6742) ->
`docs/about/release-notes.mdx`: Summarize pre-mutation resolution of
secret-free sandbox create intent.
- [#6721](#6721) ->
`docs/about/release-notes.mdx` and
`docs/get-started/quickstart-langchain-deepagents-code.mdx`: Record
bounded cleanup of completed managed Deep Agents headless sessions.
- [#6731](#6731) ->
`docs/about/release-notes.mdx` and
`docs/network-policy/customize-network-policy.mdx`: Document runtime
rejection of catch-all custom-policy destinations.
- [#6729](#6729) ->
`docs/about/release-notes.mdx` and `docs/get-started/prerequisites.mdx`:
Record the Node.js 22.19 minimum.
- [#6735](#6735) ->
`docs/about/release-notes.mdx` and
`docs/reference/platform-support.mdx`: Summarize the Ubuntu 26.04
userspace contract without claiming pending host or live validation.
- [#6775](#6775) ->
`docs/about/release-notes.mdx` and
`docs/resources/community-contributions.mdx`: Route independent
solutions outside canonical supported-product documentation.
- [#6740](#6740) ->
`docs/about/release-notes.mdx`: Summarize the semantic
dependency-upgrade contributor workflow.
- [#6777](#6777) ->
`docs/about/release-notes.mdx` and `docs/CONTRIBUTING.md`: Summarize the
route-safe documentation-refactor workflow.
- [#6741](#6741) ->
`docs/about/release-notes.mdx` and
`docs/security/openclaw-2026.6.10-dependency-review.md`: Summarize
reviewed npm archive verification and audit enforcement.
- [#6739](#6739) ->
`docs/about/release-notes.mdx` and
`docs/security/openclaw-2026.6.10-dependency-review.md`: Record the
locked offline dependency graph for the managed OpenClaw WeChat runtime.
- [#6737](#6737) ->
`docs/about/release-notes.mdx`: Record removal of the messaging build
plan from final OpenClaw and Hermes image environments.
- [#6733](#6733) ->
`docs/about/release-notes.mdx`: Summarize cached plugin dependency
layers for source and blueprint rebuilds.

### Skipped from docs-skip

- None. No commit or changed path in `v0.0.81..origin/main` matched
`openclaw-sandbox-permissive.yaml` or `config-show`, and the drafted
content contains none of the configured skip terms.

## 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)

## Quality Gates

- [ ] Tests added or updated for changed behavior
- [ ] Existing tests cover changed behavior — justification:
- [x] Tests not applicable — justification: This is a documentation-only
release-prep update; behavior is protected by the merged source PRs, and
the documentation build validates the changed routes and agent variants.
- [x] Docs updated for user-facing behavior changes
- [ ] Docs not applicable — justification:
- [ ] Sensitive paths changed (security, policy, credentials, preflight,
onboarding, inference, runner, sandbox, or messaging)
- [ ] Sensitive-path review completed or maintainer-approved waiver
recorded — reviewer/approval link/justification:
- [ ] Non-success, skipped, or missing CI check accepted by maintainer —
check name, approval link, and follow-up issue:

## Verification

- [x] PR description includes the DCO sign-off declaration and every
commit appears as `Verified` in GitHub
- [x] Normal `pre-commit`, `commit-msg`, and `pre-push` hooks passed, or
`npm run check:diff` passed when hooks were skipped or unavailable
- [x] Targeted behavior tests pass for the current change set, or tests
are marked not applicable above — tests are not applicable for this
documentation-only change.
- [ ] Applicable broad gate passed — `npm test` for broad
runtime/test-harness changes; `npm run check` for repo-wide
validation/coverage changes — command/result: not run for this
documentation-only change.
- [x] Quality Gates section completed with required justifications or
waivers
- [x] No secrets, API keys, or credentials committed
- [ ] `npm run docs` builds without warnings (doc changes only) — 0
errors; two pre-existing Fern warnings remain.
- [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)
— no new pages.

---
Signed-off-by: Charan Jagwani <cjagwani@nvidia.com>


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Documentation**
* Updated release notes with improvements to sandbox recovery,
onboarding, session management, policy validation, storage checks, and
system requirements.
  * Clarified Ollama setup instructions and status labels.
* Documented safer snapshot restoration, including dedicated ports and
protection against destructive failures.
* Expanded `backup-all` coverage to include eligible stopped sandboxes.
* Added guidance rejecting broad or catch-all network destinations in
custom policies.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Signed-off-by: Charan Jagwani <cjagwani@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ci CI workflows, checks, release automation, or GitHub Actions area: docs Documentation, examples, guides, or docs build dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants