Skip to content

fix(v1): pin nano-rlm to 83ef01f - #2379

Merged
hallerite merged 2 commits into
mainfrom
fix/nano-rlm-repo-url
Aug 24, 2026
Merged

fix(v1): pin nano-rlm to 83ef01f#2379
hallerite merged 2 commits into
mainfrom
fix/nano-rlm-repo-url

Conversation

@hallerite

@hallerite hallerite commented Aug 16, 2026

Copy link
Copy Markdown
Member

Pin the rlm harness default version to nano-rlm 83ef01f instead of tracking main, and drop the version = "main" override from configs/gsm8k_rlm.toml so the example uses it. An explicit version still selects any branch, tag, or commit.

🤖 Generated with Claude Code

@macroscopeapp

macroscopeapp Bot commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 5ec5c65

Macroscope's review found this PR approvable — This is a small, isolated reproducibility fix that pins the existing RLM installation flow to an immutable nano-rlm commit and removes a redundant example override. No new workflow, schema, infrastructure, or sensitive-area behavior is introduced.

You can add or adjust custom eligibility rules. Learn more.

macroscopeapp[bot]
macroscopeapp Bot previously approved these changes Aug 16, 2026
Default RLMHarnessConfig.version to commit 83ef01f7a6c97328919387343bd30cf4edaac20d
instead of tracking `main`, so every sandbox installs the same harness. An explicit
`version` override still selects any branch, tag, or commit.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@hallerite
hallerite force-pushed the fix/nano-rlm-repo-url branch from 9608fcd to d28b1de Compare August 24, 2026 20:13
@hallerite hallerite changed the title fix(v1): point RLM_REPO at the renamed nano-rlm repo fix(v1): pin nano-rlm to 83ef01f Aug 24, 2026
The rlm harness default is now pinned; the example should use it rather
than tracking nano-rlm main.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@hallerite
hallerite merged commit b2e4e81 into main Aug 24, 2026
13 checks passed
@hallerite
hallerite deleted the fix/nano-rlm-repo-url branch August 24, 2026 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants