Conversation
…he global soldr Remove the stale `soldr_version: 0.7.11` pin written by `clud optimize` (its old hardcoded default, zackees/clud#349). Since zackees/clud 09b4fc0 (clud#1025) clud reconciles an explicit pin on every launch with `uv tool install --force soldr==0.7.11`, downgrading the user's global soldr. Without the key clud uses the installed soldr and installs the latest only when soldr is missing. `install_soldr` and `use_soldr_shims` stay enabled. Add tests/test_clud_settings.py, a no-subprocess guard that fails if `soldr_version` reappears anywhere in the file. Closes #1212 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Fg9EZEUZcM8uc6ibJc3WSX
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What was wrong
.clud/settings.jsonpinned"soldr_version": "0.7.11"underoptimize.rust. The pin was added on 2026-06-16 (1ec6ecd), whenclud optimizewrote its old hardcoded--soldr-versiondefault (zackees/clud#349). Until recently the pin did nothing. zackees/clud 09b4fc0 (clud#1025) changed that: clud now checks an explicit pin on every launch and runsuv tool install --force soldr==0.7.11whenever the installed version differs. That downgrades the global soldr for anyone with a newer one. On 2026-09-14, the same pin in FastLED/fbuild replaced soldr 0.9.15 with 0.7.11.Fix
soldr_versionkey. clud now uses whatever soldr is installed, and installs the latestsoldronly when it's missing.install_soldr: trueanduse_soldr_shims: truestay as they are.tests/test_clud_settings.py. It loads the JSON and fails ifsoldr_versionappears at any depth, and it also asserts that install and shims stay enabled. It is pure file I/O and spawns no processes.RED -> GREEN
RED (test added, pin still present):
GREEN (key removed):
Both runs used
running-process -- python -m pytest tests/test_clud_settings.py -v. On the new file,ruff checkpasses andruff format --checkreports it already formatted.FastLED/fbuild has the same pin and is being fixed in a separate PR.
Closes #1212
🤖 Generated with Claude Code
https://claude.ai/code/session_01Fg9EZEUZcM8uc6ibJc3WSX