Skip to content

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

@vishnuprakaz

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

No one assigned

    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
    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