chore(docker, openclaw): Updating openclaw to 2026.3.13 as per https:…#880
chore(docker, openclaw): Updating openclaw to 2026.3.13 as per https:…#880buddingengineers12345 wants to merge 3 commits intoNVIDIA:mainfrom
Conversation
…#328 To fix the issues mentioned in NVIDIA#328, the openclaw is getting updated
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughOpenClaw CLI version references were bumped from 2026.3.11 to 2026.3.13 across the Docker runtime image, package manifest, sandbox metadata, and related tests; no other logic, control flow, or public API declarations were changed. 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 `@Dockerfile`:
- Around line 71-72: Dockerfile currently pins openclaw@2026.3.13 while
package.json and test/install-preflight.test.js use 2026.3.11, causing
environment drift; fix by reconciling versions: either update package.json
dependency and the five literal fixtures in test/install-preflight.test.js to
2026.3.13 so scripts/install.sh (which derives the version from package.json)
and Dockerfile match, or refactor Dockerfile to accept a build ARG (e.g.,
OPENCLAW_VERSION) and use that arg in the RUN installing openclaw, then set that
ARG at build time to the version from package.json; ensure all references
(Dockerfile RUN installing openclaw, package.json dependency, and the five
fixtures in test/install-preflight.test.js) use the same canonical version
string.
…#328 To fix the issues mentioned in NVIDIA#328, the openclaw is getting updated updating multiple files
Fixed the review comments in the second commit |
…//github.com//issues/328
To fix the issues mentioned in #328, the openclaw is getting updated
Summary
Related Issue
Changes
Type of Change
Testing
npx prek run --all-filespasses (or equivalentlymake check).npm testpasses.make docsbuilds without warnings. (for doc-only changes)Checklist
General
Code Changes
npx prek run --all-filesauto-fixes formatting (ormake formatfor targeted runs).Doc Changes
update-docsagent skill to draft changes while complying with the style guide. For example, prompt your agent with "/update-docscatch up the docs for the new changes I made in this PR."Summary by CodeRabbit