Conversation
… server helpers - Added `agentflow.a2a_integration` module to bridge agentflow with A2A protocol. - Introduced `AgentFlowExecutor` to handle execution of agentflow graphs as A2A agents. - Created client helpers for calling remote A2A agents, including `delegate_to_a2a_agent` and `create_a2a_client_node`. - Developed server helpers to expose agentflow graphs as A2A-compliant HTTP endpoints, including `create_a2a_server` and `make_agent_card`. - Enhanced message handling in `Message` class to support various content types. - Added tests for handling malformed tool messages and ensuring robustness in message conversion.
… and add tests for text extraction
- Implemented stricter validation for SkillMeta fields including name, description, triggers, resources, tags, and priority using Pydantic field validators. - Added path traversal protection for resource paths in discover_skills and load_resource functions. - Improved error handling and logging for skill discovery and loading processes. - Introduced a new unregister method in SkillsRegistry to allow removal of skills. - Added comprehensive tests for SkillMeta validation, loader functions, and SkillsRegistry operations to ensure robustness and correctness.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request removes the agent-to-agent (A2A) communication functionality from the
agentflow/communicationmodule. Specifically, it deletes the main A2A communication manager, the agent communicator mixin, and the module's public API definitions. These changes indicate a significant refactor or deprecation of the A2A communication subsystem.The most important changes are:
Removal of A2A Communication Core:
A2ACommunicationManagerfroma2a.py, which handled agent registration, messaging, broadcasting, pub-sub, and agent registry management.AgentCommunicatormixin/wrapper fromagent_communicator.py, which provided agent classes with easy access to A2A communication features.Module API Cleanup:
communication/__init__.py, eliminating references toA2ACommunicationManager,AgentRegistry,AgentRegistryEntry, andMessageRouterfrom the module's__all__.