fix: pin MiniSWEAgent LiteLLM proxy dependency - #2336
Open
xeophon wants to merge 1 commit into
Open
Conversation
ApprovabilityVerdict: Approved 59457c7 This PR pins a dependency version (litellm[proxy]==1.89.2) in script metadata. It's a simple, self-contained dependency constraint with no code logic changes. You can customize Macroscope's approvability policy. Learn more. |
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.
Motivation
litellm[proxy]in the MiniSWEAgent PEP 723 script to eliminate a supply-chain arbitrary-code-execution risk introduced by dynamicuv syncresolution.Description
litellm[proxy]==1.89.2in the harness script atverifiers/v1/harnesses/mini_swe_agent/program.pywhile preserving the existingmini-swe-agent=={version}placeholder and the existing launch flow.Testing
uv run pre-commit install,uv run ruff check --fix verifiers/v1/harnesses/mini_swe_agent/program.py, andpython -m py_compile verifiers/v1/harnesses/mini_swe_agent/program.py, and the targeted lint/compile checks passed; a fullpre-commitrun could not complete because hook dependencies could not be fetched in this environment.Codex Task
Note
Low Risk
Single dependency pin in a harness script with no logic or auth changes. Low risk, and it reduces supply-chain exposure from unpinned installs.
Overview
Pins
litellm[proxy]to==1.89.2in the MiniSWEAgent PEP 723 script (program.py).This removes unpinned runtime resolution of that dependency while keeping the existing
mini-swe-agent=={version}placeholder and launch path unchanged.Reviewed by Cursor Bugbot for commit 59457c7. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Pin
litellm[proxy]to version 1.89.2 in MiniSWEAgent harnessPins the
litellm[proxy]dependency in program.py to version 1.89.2 instead of using the latest available version, preventing breakage from upstream changes.Macroscope summarized 59457c7.