-
Notifications
You must be signed in to change notification settings - Fork 2
chore: Remove redundant openai-agents-mcp dependency #134
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
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]>
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:
|
📝 WalkthroughWalkthroughRemoved the dependency "openai-agents-mcp>=0.0.8" from connector_builder_agents/pyproject.toml. No other files or public interfaces were 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)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
⏰ 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). (2)
Comment |
👋 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/1760133611-remove-openai-agents-mcp", "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/1760133611-remove-openai-agents-mcp#egg=airbyte-connector-builder-mcp' --help PR Slash CommandsAirbyte Maintainers can execute the following slash commands on your PR:
|
chore: Remove redundant openai-agents-mcp dependency
Summary
Removes the
openai-agents-mcp>=0.0.8
dependency as it's now redundant. Investigation confirmed that openai-agents 0.3.3 (merged in PR #133) includes native MCP support via theagents.mcp
module, making the separate package unnecessary.Key findings:
openai-agents-mcp
from agents.mcp import MCPServer, MCPServerStreamableHttp
Review & Testing Checklist for Human
Notes
This change aligns the project with the openai-agents 0.3.3 upgrade that included native MCP support. The root pyproject.toml already has openai-agents-mcp commented out (line 40), indicating this cleanup was anticipated.
Link to Devin run: https://app.devin.ai/sessions/6acb99e3b34a4918af907764ca9b55c3
Requested by: @aaronsteers
Summary by CodeRabbit