Infer Agent from AgentPlatform by aggregate name with default agent fallback by Copilot · Pull Request #328 · elasticsoftwarefoundation/akces-framework

Copilot · 2026-04-10T11:28:33Z

Refactors the agentic command/event handler adapters to resolve the Agent at runtime from AgentPlatform using the aggregate name, with a default agent fallback when no matching agent is deployed. This reduces startup-time coupling to Spring bean naming and enables dynamic agent deployments.

Changes Made

Updated AgenticCommandHandlerFunctionAdapter and AgenticEventHandlerFunctionAdapter to take aggregateName and resolve the agent lazily (exact match or {name}Agent suffix match), with a default agent fallback (first available agent from the platform) when no name match is found.
Removed eager Agent bean lookup from AgenticAggregateRuntimeFactory and passed aggregate name into adapters instead.
Removed Autonomy.chooseAndAccomplishGoal() fallback — it blocks the thread until completion, causing Kafka consumer timeouts with the tick() approach. Replaced with non-blocking default agent selection.
resolveAgentByName now returns Agent directly (not Optional<Agent>), throwing IllegalStateException only when no agents are deployed on the platform at all.
Added unit tests covering agent resolution scenarios (exact match, suffix match, default fallback, no-agents-deployed error).
Fixed javadoc in AgenticAggregateRuntimeFactory — removed stale {@link Agent} references that broke javadoc:jar.

Agent-Logs-Url: https://github.com/elasticsoftwarefoundation/akces-framework/sessions/e6ed61c0-c5ac-4bbc-9cab-5e7276244968 Co-authored-by: jwijgerd <914840+jwijgerd@users.noreply.github.com>

Replace direct Agent injection in AgenticCommandHandlerFunctionAdapter and AgenticEventHandlerFunctionAdapter with runtime agent resolution based on the aggregate name. If no matching agent is found, fall back to Autonomy.chooseAndAccomplishGoal() to let Embabel decide. Remove resolveAgent() from AgenticAggregateRuntimeFactory; pass aggregate name (String) instead of Agent to adapter constructors. Agent-Logs-Url: https://github.com/elasticsoftwarefoundation/akces-framework/sessions/e6ed61c0-c5ac-4bbc-9cab-5e7276244968 Co-authored-by: jwijgerd <914840+jwijgerd@users.noreply.github.com>

Add AgenticCommandHandlerFunctionAdapterTest (12 tests) and AgenticEventHandlerFunctionAdapterTest (7 tests) covering: - Exact name match resolution - Suffix name match resolution - Autonomy chooseAndAccomplishGoal() fallback - Exception wrapping for Autonomy failures - Exact match precedence over suffix match - resolveAgentByName static helper - Basic adapter properties Agent-Logs-Url: https://github.com/elasticsoftwarefoundation/akces-framework/sessions/e6ed61c0-c5ac-4bbc-9cab-5e7276244968 Co-authored-by: jwijgerd <914840+jwijgerd@users.noreply.github.com>

…x test mocks - Narrow exception catch from generic Exception to ProcessExecutionException - Use doThrow() for Kotlin final method stubbing in tests - Provide non-null AgentProcess to ProcessExecutionFailedException constructor Agent-Logs-Url: https://github.com/elasticsoftwarefoundation/akces-framework/sessions/e6ed61c0-c5ac-4bbc-9cab-5e7276244968 Co-authored-by: jwijgerd <914840+jwijgerd@users.noreply.github.com>

Copilot

Pull request overview

This PR refactors the agentic command/event handler adapters to resolve the Agent at runtime from AgentPlatform using the aggregate name, with an Autonomy.chooseAndAccomplishGoal() fallback when no matching agent is deployed. This reduces startup-time coupling to Spring bean naming and enables dynamic agent deployments.

Changes:

Updated AgenticCommandHandlerFunctionAdapter and AgenticEventHandlerFunctionAdapter to take aggregateName and resolve the agent lazily (exact or {name}Agent suffix), with Autonomy fallback and exception wrapping.
Removed eager Agent bean lookup from AgenticAggregateRuntimeFactory and passed aggregate name into adapters instead.
Added new unit tests covering agent resolution scenarios, fallback behavior, and exception wrapping.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
plans/agent-resolution-refactoring.md	Documents the refactor plan, motivation, and testing/risk notes.
main/agentic/src/main/java/org/elasticsoftware/akces/agentic/runtime/AgenticCommandHandlerFunctionAdapter.java	Implements runtime agent resolution + Autonomy fallback for agent-handled commands; adds helper + tick safety method.
main/agentic/src/main/java/org/elasticsoftware/akces/agentic/runtime/AgenticEventHandlerFunctionAdapter.java	Implements runtime agent resolution + Autonomy fallback for agent-handled external events; adds tick safety method.
main/agentic/src/main/java/org/elasticsoftware/akces/agentic/beans/AgenticAggregateRuntimeFactory.java	Removes eager Agent resolution and wires adapters with aggregate name instead of Agent instance.
main/agentic/src/test/java/org/elasticsoftware/akces/agentic/runtime/AgenticCommandHandlerFunctionAdapterTest.java	Adds unit tests for exact/suffix resolution, precedence, fallback, exception wrapping, and helper behavior.
main/agentic/src/test/java/org/elasticsoftware/akces/agentic/runtime/AgenticEventHandlerFunctionAdapterTest.java	Adds unit tests for exact/suffix resolution, fallback, exception wrapping, and basic adapter properties.

...ain/java/org/elasticsoftware/akces/agentic/runtime/AgenticCommandHandlerFunctionAdapter.java

.../main/java/org/elasticsoftware/akces/agentic/runtime/AgenticEventHandlerFunctionAdapter.java

…ate-name

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

jwijgerd

Fix build issues

…FunctionAdapter The code review suggestion commit introduced getAgents() which does not exist on the AgentPlatform interface. The correct method is agents(). Also restored single-pass agent resolution logic with suffix match support that was lost in the code review suggestion. Agent-Logs-Url: https://github.com/elasticsoftwarefoundation/akces-framework/sessions/e6bbbe64-67fe-4997-9046-7e96a9a43702 Co-authored-by: jwijgerd <914840+jwijgerd@users.noreply.github.com>

Copilot · 2026-04-10T12:17:42Z

Fix build issues

Fixed in f1d995e. The code review suggestion commit introduced getAgents() which doesn't exist on AgentPlatform — replaced with agents() and restored the single-pass resolution logic with suffix match support. All 63 tests pass.

jwijgerd

there is a javadoc issue, see: https://github.com/elasticsoftwarefoundation/akces-framework/actions/runs/24242498376/job/70780279849?pr=328

fix it so the CI build runs again

…eRuntimeFactory The class-level javadoc still referenced {@link Agent} which was no longer imported after agent resolution moved to the adapters. Updated the paragraph to describe the current lazy-resolution behavior. Agent-Logs-Url: https://github.com/elasticsoftwarefoundation/akces-framework/sessions/d46ab258-b06a-4d2a-b1ed-ccc2c1db2aae Co-authored-by: jwijgerd <914840+jwijgerd@users.noreply.github.com>

Copilot · 2026-04-10T12:43:21Z

there is a javadoc issue, see: https://github.com/elasticsoftwarefoundation/akces-framework/actions/runs/24242498376/job/70780279849?pr=328

fix it so the CI build runs again

Fixed in 1595976. The class-level javadoc in AgenticAggregateRuntimeFactory still had {@link Agent} references from before the refactor. Replaced the stale paragraph with documentation that describes the current lazy-resolution behavior. Javadoc now generates cleanly and all 63 tests pass.

jwijgerd · 2026-04-10T13:04:24Z

...ain/java/org/elasticsoftware/akces/agentic/runtime/AgenticCommandHandlerFunctionAdapter.java

+                    aggregateName);
+            Autonomy autonomy = agentPlatform.getPlatformServices().autonomy();
+            try {
+                AgentProcessExecution execution = autonomy.chooseAndAccomplishGoal(


in hindsight, this approach doesn't work well with the tick() approach (since the Autonomy solution runs to completion and will block the thread for a long time causing kafka to time out. Let's create a default agent that will always be present and that will be returned when no other agent matches

Replaced the Autonomy fallback with a default agent in 9477ef6. resolveAgentByName now returns Agent directly (not Optional) — it tries exact match, then {name}Agent suffix match, then falls back to the first available agent from the platform. IllegalStateException is thrown only when no agents are deployed at all. All 63 tests pass.

The Autonomy.chooseAndAccomplishGoal() approach blocks the thread until completion, causing Kafka consumer timeouts. Replace it with a default agent fallback: when no agent matches the aggregate name (exact or suffix), the first available agent from the platform is used instead. - Remove Autonomy imports and fallback code from both adapters - Change resolveAgentByName to return Agent directly (not Optional) - Throw IllegalStateException only when no agents are deployed at all - Update tests to verify default agent and no-agents-deployed scenarios Agent-Logs-Url: https://github.com/elasticsoftwarefoundation/akces-framework/sessions/8cf48950-f35e-46f6-971b-ec135e834c54 Co-authored-by: jwijgerd <914840+jwijgerd@users.noreply.github.com>

Copilot AI and others added 4 commits April 10, 2026 10:42

Add plan for agent resolution refactoring

4364638

Agent-Logs-Url: https://github.com/elasticsoftwarefoundation/akces-framework/sessions/e6ed61c0-c5ac-4bbc-9cab-5e7276244968 Co-authored-by: jwijgerd <914840+jwijgerd@users.noreply.github.com>

Copilot AI assigned Copilot and jwijgerd Apr 10, 2026

Copilot created this pull request from a session on behalf of jwijgerd April 10, 2026 11:28 View session

jwijgerd marked this pull request as ready for review April 10, 2026 11:29

Copilot AI review requested due to automatic review settings April 10, 2026 11:29

Copilot started reviewing on behalf of jwijgerd April 10, 2026 11:29 View session

Copilot AI reviewed Apr 10, 2026

View reviewed changes

jwijgerd and others added 2 commits April 10, 2026 14:07

Merge branch 'main' into copilot/infer-agent-instance-based-on-aggreg…

3026106

…ate-name

Apply suggestions from code review

5316887

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

jwijgerd requested changes Apr 10, 2026

View reviewed changes

Copilot started work on behalf of jwijgerd April 10, 2026 12:11 View session

Copilot finished work on behalf of jwijgerd April 10, 2026 12:19

Copilot AI requested a review from jwijgerd April 10, 2026 12:19

jwijgerd requested changes Apr 10, 2026

View reviewed changes

Copilot started work on behalf of jwijgerd April 10, 2026 12:40 View session

Copilot AI requested a review from jwijgerd April 10, 2026 12:46

Copilot finished work on behalf of jwijgerd April 10, 2026 12:46

jwijgerd requested changes Apr 10, 2026

View reviewed changes

Copilot started work on behalf of jwijgerd April 10, 2026 13:04 View session

Copilot AI changed the title ~~Infer Agent from AgentPlatform by aggregate name with Autonomy fallback~~ Infer Agent from AgentPlatform by aggregate name with default agent fallback Apr 10, 2026

Copilot finished work on behalf of jwijgerd April 10, 2026 13:17

Copilot AI requested a review from jwijgerd April 10, 2026 13:17

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Infer Agent from AgentPlatform by aggregate name with default agent fallback#328

Infer Agent from AgentPlatform by aggregate name with default agent fallback#328
Copilot wants to merge 9 commits intomainfrom
copilot/infer-agent-instance-based-on-aggregate-name

Copilot AI commented Apr 10, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

jwijgerd left a comment

Uh oh!

Copilot AI commented Apr 10, 2026

Uh oh!

jwijgerd left a comment

Uh oh!

Copilot AI commented Apr 10, 2026

Uh oh!

jwijgerd Apr 10, 2026

Uh oh!

Copilot AI Apr 10, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

Copilot AI commented Apr 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changes Made

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

jwijgerd left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented Apr 10, 2026

Uh oh!

jwijgerd left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented Apr 10, 2026

Uh oh!

jwijgerd Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Copilot AI commented Apr 10, 2026 •

edited

Loading