Skip to content

Add executor_builder argument to A2AAgentServer#119

Merged
weilixu merged 1 commit into
developfrom
executor-builder-argument
May 29, 2026
Merged

Add executor_builder argument to A2AAgentServer#119
weilixu merged 1 commit into
developfrom
executor-builder-argument

Conversation

@jslane-h

Copy link
Copy Markdown
Collaborator
  • Adds executor_builder argument to A2AAgentServer in order to have custom AgentExecutors
  • Moved langgraph_chatagent_test.py from agents to agent_test folder

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an optional executor_builder callable to A2AAgentServer so callers can plug in custom AgentExecutor implementations instead of always using GenericAgentExecutor. Also relocates langgraph_chatagent_test.py from agents/ to the agent_test/ folder.

Changes:

  • Add executor_builder: Callable[[BaseAgent], AgentExecutor] | None parameter to A2AAgentServer.__init__, defaulting to a lambda that constructs GenericAgentExecutor.
  • Use self.executor_builder(self._agent) when constructing the DefaultRequestHandler in run().
  • Move langgraph_chatagent_test.py into the agent_test/ directory.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
automa_ai/common/agent_registry.py Adds executor_builder constructor argument and uses it when creating the request handler.
automa_ai/agent_test/langgraph_chatagent_test.py Relocated test file (contents unchanged) to the dedicated test folder.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jslane-h jslane-h changed the base branch from master to develop May 28, 2026 20:21
@weilixu weilixu merged commit 286b306 into develop May 29, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants