fix(recipe-runner): auto-update rust runner binary on version mismatch (#4189)#4199
fix(recipe-runner): auto-update rust runner binary on version mismatch (#4189)#4199
Conversation
ensure_rust_recipe_runner() previously only installed the binary when absent, but silently used a stale/incompatible version if it was already present. Now checks the version via check_runner_version() and triggers a re-install via cargo if the binary is outdated. Fixes #4159 (recipe-runner-rs not auto-updated on version mismatch) Atlas layer 8 (ast-lsp-bindings) refreshed for changed modules. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Repo Guardian - PassedAll 3 changed files passed inspection — no ephemeral content or temporary scripts detected.
|
PR Triage ReportRisk: 🟢 Low | Priority: Medium | Type: Bug fix AssessmentTargeted fix for
RecommendationHold pending #4203 review. If #4203 subsumes this fix, close this PR. Otherwise, rebase and merge independently.
|
Summary
Fixes
ensure_rust_recipe_runner()to upgrade the binary when it's present but outdated, not just when it's absent.Root Cause
ensure_rust_recipe_runner()previously returnedTrueimmediately if any binary was found viais_rust_runner_available(), even ifcheck_runner_version()would have returnedFalse.Changes
src/amplihack/recipes/rust_runner.py: Split the early-return check into two cases:Trueimmediately (unchanged behaviour)cargo installto upgradedocs/atlas/ast-lsp-bindings/: Refreshed atlas layer 8 to clear CI staleness checkCloses #4159
Related: #4188, #4169