Skip to content

fix: add coding_agent_rl dependencies to requirements.txt - #216

Merged
aoshen02 merged 1 commit into
mainfrom
sync-107-gaps
Jun 9, 2026
Merged

fix: add coding_agent_rl dependencies to requirements.txt#216
aoshen02 merged 1 commit into
mainfrom
sync-107-gaps

Conversation

@aoshen02

@aoshen02 aoshen02 commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

Test plan

  • pip install -r requirements.txt succeeds with the new deps

🤖 Generated with Claude Code

#148 synced the coding_agent_rl subsystem (slime #1923 et al.) but
missed the corresponding requirements: anthropic, e2b, openai,
openai-agents.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request adds several new dependencies to requirements.txt, including anthropic, e2b, openai, and openai-agents. The reviewer suggested specifying a minimum version constraint for e2b (e.g., e2b>=1.0.0) to avoid potential runtime ImportError issues with older versions.

Comment thread requirements.txt
blobfile
cloudpickle
datasets
e2b

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.

medium

The codebase imports CommandExitException from e2b.sandbox.commands.command_handle in vime/agent/sandbox.py (line 214), which is only available in newer versions of the e2b SDK (v1.0.0+). Without specifying a minimum version constraint, an older version of e2b might be installed, leading to a runtime ImportError. Consider specifying a minimum version constraint like e2b>=1.0.0.

e2b>=1.0.0

@aoshen02
aoshen02 merged commit e5caafe into main Jun 9, 2026
0 of 2 checks passed
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.

1 participant