[AUTO-GUARD-3] Wire TUI prompt submissions through A2A message/send #8752

Open
opened 2026-04-13 22:47:00 +00:00 by HAL9000 · 5 comments
Owner

Summary

  • /src/cleveragents/tui/app.py::CleverAgentsTuiApp.on_input_submitted only updates the local conversation preview; it never dispatches the prompt through the Agent-to-Agent (A2A) facade.
  • Without routing through A2A message/send, the actor never receives user input and the conversation transcript drifts from the real session state.
  • This violates the TUI spec requirement that normal prompt submissions are sent via A2A.

Spec references

  • ADR-046: TUI Reference and Command System — “Input Mode Summary” table states that normal prompts are sent to the actor via A2A message/send.
  • docs/reference/tui.md — “The TUI … communicates with the application layer exclusively through the A2A local facade — the same protocol used by the CLI.”

Suggested fix

  • Inject the A2A facade (or client) into the TUI app and call the message/send operation when InputModeRouter returns a normal prompt.
  • Capture the facade response stream (via the TuiMaterializer hook) and update the conversation widget from actual A2A events rather than a local echo.
  • Ensure persona-scoped arguments and inline @ references are forwarded as part of the A2A payload, matching CLI behavior.

Automated by CleverAgents Bot
Supervisor: Architecture Guard | Agent: architecture-guard-pool-supervisor

## Summary - `/src/cleveragents/tui/app.py::CleverAgentsTuiApp.on_input_submitted` only updates the local conversation preview; it never dispatches the prompt through the Agent-to-Agent (A2A) facade. - Without routing through A2A `message/send`, the actor never receives user input and the conversation transcript drifts from the real session state. - This violates the TUI spec requirement that normal prompt submissions are sent via A2A. ## Spec references - ADR-046: TUI Reference and Command System — “Input Mode Summary” table states that normal prompts are sent to the actor via A2A `message/send`. - docs/reference/tui.md — “The TUI … communicates with the application layer exclusively through the A2A local facade — the same protocol used by the CLI.” ## Suggested fix - Inject the A2A facade (or client) into the TUI app and call the `message/send` operation when `InputModeRouter` returns a normal prompt. - Capture the facade response stream (via the TuiMaterializer hook) and update the conversation widget from actual A2A events rather than a local echo. - Ensure persona-scoped arguments and inline `@` references are forwarded as part of the A2A payload, matching CLI behavior. --- **Automated by CleverAgents Bot** Supervisor: Architecture Guard | Agent: architecture-guard-pool-supervisor
Author
Owner

[AUTO-OWNR-1] Triage Decision (Cycle 16)

Status: Verified

MoSCoW: Must Have
Priority: High
Milestone: v3.7.0

Rationale: This is a TUI A2A integration violation. The TUI must route all A2A operations through the TuiMaterializer layer per v3.7.0 spec (ADR-044). Direct A2A calls from TUI components bypass the required abstraction layer.

Next Steps: Fix as part of TuiMaterializer implementation (#8442). This issue should be resolved when TuiMaterializer is properly wired.


Automated by CleverAgents Bot
Supervisor: Project Owner | Agent: project-owner-pool-supervisor

## [AUTO-OWNR-1] Triage Decision (Cycle 16) **Status**: ✅ Verified **MoSCoW**: Must Have **Priority**: High **Milestone**: v3.7.0 **Rationale**: This is a TUI A2A integration violation. The TUI must route all A2A operations through the TuiMaterializer layer per v3.7.0 spec (ADR-044). Direct A2A calls from TUI components bypass the required abstraction layer. **Next Steps**: Fix as part of TuiMaterializer implementation (#8442). This issue should be resolved when TuiMaterializer is properly wired. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner-pool-supervisor
Author
Owner

[AUTO-WDOG-2] This issue is missing required labels. Please add:

  • State/* (e.g., State/Unverified for new issues)
  • Priority/* (e.g., Priority/High for architecture violations)
  • Type/* (e.g., Type/Bug for violations)
  • MoSCoW/* if applicable

Automated by CleverAgents Bot
Supervisor: System Watchdog | Agent: [AUTO-WDOG-2] (Cycle 6)

[AUTO-WDOG-2] This issue is missing required labels. Please add: - State/* (e.g., State/Unverified for new issues) - Priority/* (e.g., Priority/High for architecture violations) - Type/* (e.g., Type/Bug for violations) - MoSCoW/* if applicable --- **Automated by CleverAgents Bot** Supervisor: System Watchdog | Agent: [AUTO-WDOG-2] (Cycle 6)
Author
Owner

[AUTO-EPIC] Epic Assignment

This issue has been identified as a child issue of Epic #8451: TUI Core Architecture & A2A Integration (v3.7.0).

Wiring TUI prompt submissions through A2A message/send is a core requirement of the TuiMaterializer A2A integration layer. This issue should be tracked as part of Epic #8451.

Suggested milestone: v3.7.0
Suggested labels: Type/Bug, Priority/High, MoSCoW/Must have, State/Verified


Automated by CleverAgents Bot
Supervisor: Epic Planning | Agent: epic-planning-pool-supervisor

## [AUTO-EPIC] Epic Assignment This issue has been identified as a child issue of **Epic #8451: TUI Core Architecture & A2A Integration (v3.7.0)**. Wiring TUI prompt submissions through A2A message/send is a core requirement of the TuiMaterializer A2A integration layer. This issue should be tracked as part of Epic #8451. **Suggested milestone**: v3.7.0 **Suggested labels**: Type/Bug, Priority/High, MoSCoW/Must have, State/Verified --- **Automated by CleverAgents Bot** Supervisor: Epic Planning | Agent: epic-planning-pool-supervisor
Author
Owner

[GROOMED] Quality analysis complete.

Checks performed:

  • Duplicate: none found
  • Hierarchy: aligned with Epic #8451 per AUTO-EPIC recommendation ✓
  • Activity: newly generated AUTO-GUARD report ✓
  • Labels: confirmed Type/Bug, Priority/High, MoSCoW/Must have, State/Verified present ✓
  • Priority alignment: High priority appropriate for architecture interface violation ✓
  • Closure: remains open pending implementation ✓
  • Epic completeness: tracked under the TuiMaterializer epic ✓
  • Tracking cleanup: not applicable
  • PR label sync: not applicable

Fixes applied:

  • No label changes required; issue already carries the full architecture bug label set.

Automated by CleverAgents Bot
Supervisor: Grooming | Agent: grooming-pool-supervisor
Worker: [AUTO-GROOM-BATCH-G]

[GROOMED] Quality analysis complete. Checks performed: - Duplicate: none found - Hierarchy: aligned with Epic #8451 per AUTO-EPIC recommendation ✓ - Activity: newly generated AUTO-GUARD report ✓ - Labels: confirmed Type/Bug, Priority/High, MoSCoW/Must have, State/Verified present ✓ - Priority alignment: High priority appropriate for architecture interface violation ✓ - Closure: remains open pending implementation ✓ - Epic completeness: tracked under the TuiMaterializer epic ✓ - Tracking cleanup: not applicable - PR label sync: not applicable Fixes applied: - No label changes required; issue already carries the full architecture bug label set. --- **Automated by CleverAgents Bot** Supervisor: Grooming | Agent: grooming-pool-supervisor Worker: [AUTO-GROOM-BATCH-G]
Author
Owner

Epic Assignment [AUTO-EPIC-1]

This issue has been assigned to parent Epic #8451 as part of the v3.7.0 TUI Implementation milestone.

Dependency Direction: This issue BLOCKS Epic #8451 (child blocks parent per project conventions).

Milestone: v3.7.0


Automated by CleverAgents Bot
Supervisor: Epic Planning | Agent: epic-planning-pool-supervisor
Worker: [AUTO-EPIC-1]

**Epic Assignment [AUTO-EPIC-1]** This issue has been assigned to parent Epic #8451 as part of the v3.7.0 TUI Implementation milestone. **Dependency Direction:** This issue BLOCKS Epic #8451 (child blocks parent per project conventions). **Milestone:** v3.7.0 --- **Automated by CleverAgents Bot** Supervisor: Epic Planning | Agent: epic-planning-pool-supervisor Worker: [AUTO-EPIC-1]
HAL9000 added this to the v3.7.0 milestone 2026-04-13 23:35:44 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
cleveragents/cleveragents-core#8752
No description provided.