Skip to content

fix(adk): bridge workflow input through native HITL - #3459

Open
pood1e wants to merge 1 commit into
kagent-dev:mainfrom
pood1e:fix/upstream-workflow-input-hitl
Open

pood1e wants to merge 1 commit into
kagent-dev:mainfrom
pood1e:fix/upstream-workflow-input-hitl

Conversation

@pood1e

@pood1e pood1e commented Oct 31, 2026

Copy link
Copy Markdown
Contributor

Summary

ADK Go 2.x workflow RequestInput emits adk_request_input, while the HITL bridge currently handles tool confirmation calls. A workflow therefore reaches input-required without a public, answerable AskUserRequest.

This maps a single workflow input to the existing HITL question contract, retains its canonical call for resume, and maps the correlated answer back to the SDK workflow dispatcher. Invalid or mixed pauses surface as errors. BuildHITLStatusMessage now returns an error, handled by the executor and its callers.

Verification

  • The SDK workflow/executor round-trip test fails on the current main baseline before the fix.
  • Approve, reject and structured JSON answers resume the real SDK workflow after the fix.
  • Correlation, mixed-pause and non-negotiated prompt tests pass.
  • go test ./adk/... ./api/a2a
  • go test -race ./adk/pkg/a2a ./adk/pkg/tools ./api/a2a
  • make -C go lint AWK=awk — 0 issues.

Related workflow integration: #3344.

Signed-off-by: pood1e <codingpood1e@gmail.com>
@pood1e
pood1e requested a review from supreme-gg-gg as a code owner October 31, 2026 06:14
@github-actions github-actions Bot added the bug Something isn't working label Oct 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant