-
Notifications
You must be signed in to change notification settings - Fork 8
[PR-1090] Agentic Class #869
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
base: master
Are you sure you want to change the base?
Conversation
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.
Pull request overview
This PR introduces a new AgenticModelClass that extends OpenAIModelClass to enable agentic behavior by integrating LLMs with MCP (Model Context Protocol) servers. The class provides tool discovery, execution, and iterative tool calling capabilities for both chat completions and responses endpoints, supporting both streaming and non-streaming modes.
Key Changes
- Adds MCP server connection and tool discovery with retry logic and resource cleanup
- Implements token usage accumulation across multiple API calls in agentic flows
- Provides streaming and non-streaming support for iterative tool execution with both chat completions and responses API endpoints
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.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 16 comments.
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.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 33 comments.
|
@luv-bansal I've opened a new pull request, #875, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@luv-bansal I've opened a new pull request, #876, to work on those changes. Once the pull request is ready, I'll request review from you. |
…cycle (#875) * Initial plan * Add comprehensive test coverage for MCPConnectionPool Co-authored-by: luv-bansal <[email protected]> * Improve test robustness based on code review feedback Co-authored-by: luv-bansal <[email protected]> * Optimize test execution time and improve mock specifications Co-authored-by: luv-bansal <[email protected]> --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: luv-bansal <[email protected]>
…-python into mcpopenaiclass
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.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 15 comments.
Minimum allowed line rate is |
Pull request overview
This PR introduces a new
AgenticModelClassthat extendsOpenAIModelClassto enable agentic behavior by integrating LLMs with MCP (Model Context Protocol) servers. The class provides tool discovery, execution, and iterative tool calling capabilities for bothchat completionsandresponses endpoints, supporting both streaming and non-streaming modes.LLM with MCP servers client example