-
Notifications
You must be signed in to change notification settings - Fork 2
chore: Upgrade openai-agents from 0.2.11 to 0.3.3 #133
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-Authored-By: AJ Steers <[email protected]>
Original prompt from AJ Steers
|
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
👋 Greetings, Airbyte Team Member!Here are some helpful tips and reminders for your convenience. Testing This Branch via MCPTo test the changes in this specific branch with an MCP client like Claude Desktop, use the following configuration: {
"mcpServers": {
"connector-builder-mcp-dev": {
"command": "uvx",
"args": ["--from", "git+https://github.com/airbytehq/connector-builder-mcp.git@devin/1760132669-upgrade-openai-agents", "connector-builder-mcp"]
}
}
} Testing This Branch via CLIYou can test this version of the MCP Server using the following CLI snippet: # Run the CLI from this branch:
uvx 'git+https://github.com/airbytehq/connector-builder-mcp.git@devin/1760132669-upgrade-openai-agents#egg=airbyte-connector-builder-mcp' --help PR Slash CommandsAirbyte Maintainers can execute the following slash commands on your PR:
|
📝 WalkthroughWalkthroughUpdated dependency constraint for openai-agents in connector_builder_agents/pyproject.toml from >=0.2.11 to >=0.3.3. No other files or public interfaces changed. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested labels
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
connector_builder_agents/pyproject.toml (1)
10-10
: Tighten openai-agents version range
Add an upper bound to avoid pulling in 0.4.0 prereleases:- "openai-agents>=0.3.3", + "openai-agents>=0.3.3,<0.4.0",All related packages (openai-agents-mcp, openinference-instrumentation-openai-agents, openai 1.109.1) are compatible with 0.3.3; regenerate the lockfile and rerun end-to-end tests.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
connector_builder_agents/uv.lock
is excluded by!**/*.lock
📒 Files selected for processing (1)
connector_builder_agents/pyproject.toml
(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Run Evals (Single Connector)
Best build time yet! Just around 3 minutes! https://github.com/airbytehq/connector-builder-mcp/actions/runs/18419163761/job/52489711554?pr=133 Points to us likely having been held back in efficiency due to library issues prior to now. cc @pedroslopez (Devin, no action needed.) |
openai-agents 0.3.3 (merged in PR #133) includes native MCP support via the agents.mcp module. The separate openai-agents-mcp package is no longer needed. Verified that no code in the codebase imports or uses openai-agents-mcp. See examples in openai-agents repo: https://github.com/openai/openai-agents-python/tree/main/examples/mcp Co-Authored-By: AJ Steers <[email protected]>
chore: Upgrade openai-agents from 0.2.11 to 0.3.3
Summary
Upgrades the
openai-agents
library from version 0.2.11 to 0.3.3 to potentially address coordination issues identified in the agent system. This upgrade includes:Version 0.3.3 includes PR #1662 which adds
AdvancedSQLiteSession
with conversation branching & usage tracking capabilities that may help resolve the handoff tool failures and agent identity confusion issues identified in our investigation.Review & Testing Checklist for Human
report_back_to_manager
tool is available when needed)Test Plan Recommendation
Run the connector builder with a simple API (like JSONPlaceholder) and monitor the execution logs for:
mark_job_success
)Notes
This upgrade was requested as part of investigating coordination issues documented in this Devin session. The original investigation found specific problems with handoff tool availability and agent identity confusion that this library upgrade may help address.
Requested by: @aaronsteers
Summary by CodeRabbit