Skip to content

fix(openclaw_agent): pin openclaw_version default for reproducibility#2004

Open
j-nolan wants to merge 1 commit into
NVIDIA-NeMo:cmunley1/anyswefrom
j-nolan:jnolan/openclaw-pin-version
Open

fix(openclaw_agent): pin openclaw_version default for reproducibility#2004
j-nolan wants to merge 1 commit into
NVIDIA-NeMo:cmunley1/anyswefrom
j-nolan:jnolan/openclaw-pin-version

Conversation

@j-nolan

@j-nolan j-nolan commented Jul 13, 2026

Copy link
Copy Markdown

The default was None, which falls back to npm install openclaw@latest in the task container and drifts as new OpenClaw releases land, so evals are not reproducible. Pin the default to 2026.6.11 (what @latest currently resolves to, so no behavior change for current runs) and align the example config. Mirrors how hermes_agent pins its agent by a git ref in requirements.txt; overridable per config.

The default was None, which falls back to npm install openclaw@latest in the task container and drifts as new OpenClaw releases land, so evals are not reproducible. Pin the default to 2026.6.11 (what @latest currently resolves to, so no behavior change for current runs) and align the example config. Mirrors how hermes_agent pins its agent by a git ref in requirements.txt; overridable per config.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@copy-pr-bot

copy-pr-bot Bot commented Jul 13, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@nemo-automation-bot nemo-automation-bot Bot added the community-request Issue reported or requested by someone from the community label Jul 13, 2026
@svcnvidia-nemo-ci svcnvidia-nemo-ci added the waiting-on-maintainers Waiting on maintainers to respond label Jul 15, 2026
openclaw_config: dict[str, Any] = Field(default_factory=dict)
openclaw_version: Optional[str] = None
# pin a default so runs are reproducible; None falls back to `npm install openclaw@latest`, which drifts
openclaw_version: Optional[str] = "2026.6.11"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one challenge with pinning the default is any configs which don't specify this will fallback to the repo's default here. if we ever update the default, this would be a breaking change to be safe for reproducibility, should the openclaw version be required instead?

also @cmunley1 for feedback

@cmunley1 cmunley1 Jul 15, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

im fine with either

@ananthsub ananthsub left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the contribution @j-nolan ! any reason why we can't merge this to main directly?

@cmunley1

Copy link
Copy Markdown
Contributor

yes, i would suggest main too

@svcnvidia-nemo-ci svcnvidia-nemo-ci added waiting-on-customer Waiting on the original author to respond and removed waiting-on-maintainers Waiting on maintainers to respond labels Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-request Issue reported or requested by someone from the community waiting-on-customer Waiting on the original author to respond

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants