Skip to content

Migrate to mini-SWE-agent v2#109

Merged
john-b-yang merged 3 commits into
mainfrom
john/mini-v2
Jun 24, 2026
Merged

Migrate to mini-SWE-agent v2#109
john-b-yang merged 3 commits into
mainfrom
john/mini-v2

Conversation

@john-b-yang

Copy link
Copy Markdown
Contributor

Migrating CodeClash to use mini-SWE-agent with version 2.0 upgrades.

Key changes:

  • mini-SWE-agent changes execute to take a dictionary of {"command": <action>}, whereas it formerly took a string. To minimize amount of changes to rest of codebase, a small ClashDockerEnvironment class was added that wraps the execute to take in either a string or a dictionary. string inputs are then cast into the dictionary format before being passed to the execute call of the underlying DockerEnvironment class defined by mini-SWE-agent.
  • Solved some lingering small bugs (temp directory not working on AWS, work around for DeterministicModel)
  • Update viewer to handle both v1 and v2 style mini-SWE-agent trajectories.
  • Remove dangling trajectory.html
  • Update pyproject, dependency versioning.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@Muhtasham

Copy link
Copy Markdown
Contributor

@john-b-yang I pushed a small follow-up on this branch to fix the mini-SWE-agent v2 model config migration:

  • normalized the README-linked example configs from Portkey-style @provider/... names to LiteLLM provider names
  • changed Gemini in configs/models.yaml from google/gemini-2.5-pro to gemini/gemini-2.5-pro
  • kept PORTKEY_API_KEY documented in .env.example for legacy configs that still set model_class: portkey

Verified through the actual CodeClash mini-SWE-agent v2 import path: minisweagent.models.get_model(...) + LiteLLM provider resolution validated all 25 example/source-roster model names. Also reran local SCML/CybORG/dummy arena tests, and CI is green.

@Muhtasham Muhtasham 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.

Approved. I reviewed the mini-SWE-agent v2 migration and pushed a small follow-up fixing the model config names in the README-linked examples/source roster. Provider resolution was verified through the actual CodeClash mini-SWE-agent v2 path, local SCML/CybORG/dummy tests pass, and CI is green.

@john-b-yang

Copy link
Copy Markdown
Contributor Author

You are amazing, thank you so much @Muhtasham, merging!

@john-b-yang john-b-yang merged commit d77984c into main Jun 24, 2026
5 checks passed
@john-b-yang john-b-yang deleted the john/mini-v2 branch June 24, 2026 22:27
This was referenced Jun 24, 2026
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