Skip to content

[Tech Debt] Warn when an action declares a return value #834

Description

@vishnuprakaz

Search before asking

  • I searched in the issues and found nothing similar.

Description

Actions communicate by sending events via ctx.send_event(...); any value an
action returns is silently discarded by the framework.

Today this fails quietly
if a user writes return result in an action, nothing happens and there is no
error or warning, which is a confusing trap.

python/flink_agents/plan/actions/action.py already carries a TODO for this:

Proposed fix:

when an agent plan is built, emit a warning for any @action-
decorated method that declares a non-None return type, pointing the user to
ctx.send_event(...). Implements the existing TODO; warning only, no change to
execution behavior

Are you willing to submit a PR?

  • I'm willing to submit a PR!

Metadata

Metadata

Assignees

Labels

priority/majorDefault priority of the PR or issue.tech debt[Issue Type] User-unaware issues, such as code refactor and infrastructure maintenance.

Type

No type

Fields

No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions