-
Notifications
You must be signed in to change notification settings - Fork 12
Adding A2A platform #142
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
Adding A2A platform #142
Conversation
1. Updated Agent class to include AgentCard for all agents 2. Added a protocol folder and other modules. For now, agent card with JSON-RPC, server to act on agent and client to contact the server.
Added functions for unsynchronous execution with status update as PENDING, RUNNING, DONE, ERROR. Now the platform can support multiple executions in parallel.
Trying to execute task in background, but if there is any error, it will be catched by exception and there can be a warning - "Task exception was never retrieved". In such cases, this should be uncommented and the test script need to changed accordingly.
Moved add capabilities, description and so on to the bottom, as it makes sense to add them only after the full agent is developed. Also, the methods are captured from the script directly.
Providing proper JSON codes for reporting errors. This makes the platform's error more standardised.
Using error_middleware to catch the errors, match it with existing Exceptions. If noting matches, then it will report as Internal Error. All in JSON RPC format so that the reporting structure is always constant.
Invalid parameters, invalid request, parse error, method not found, Internal error, task not found, task is still pending, task failed
Updated agent card to include methods automatically from the methods registered in the router. Also, kept the functionality for manually adding it.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
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 28 out of 28 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
|
@copilot summary of changed docs |
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 28 out of 28 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Created a new platform for agent interaction through standardised Agent2Agent protocol.
It supports registering the agent, its methods, hosting the server endpoint, displaying agent cards, submitting agent jobs as tasks, and monitoring their live execution status (PENDING/RUNNING/DONE/ERROR) through SSE or HTTP polling.