Skip to content

[codex] Forward Northbound agent runs to Runtime - #3707

Merged
hhhhsc701 merged 0 commit into
dev/helm-release-splitfrom
dev/northbound-runtime-forwarding
Aug 19, 2026
Merged

[codex] Forward Northbound agent runs to Runtime#3707
hhhhsc701 merged 0 commit into
dev/helm-release-splitfrom
dev/northbound-runtime-forwarding

Conversation

@hhhhsc701

Copy link
Copy Markdown
Contributor

What changed

  • Add Runtime internal agent run and stop endpoints with strict request validation.
  • Add a managed Northbound Runtime HTTP client with request ID propagation, SSE proxying, timeout/error mapping, and deterministic upstream cleanup.
  • Route REST Chat and A2A synchronous, streaming, and cancellation execution through Runtime without forwarding external authorization or falling back to local execution.
  • Add the shared agent SSE interruption contract and focused unit/integration-style tests.

Why

Northbound should remain the authentication and orchestration boundary while all agent execution happens in Runtime, including multi-replica deployments.

Impact

Agent run and stop operations require Runtime availability. Runtime 4xx responses are preserved, 5xx responses map to 502, and connection failures map to 503.

Validation

  • 131 targeted backend tests passed.
  • git diff --check passed.

Known follow-up

  • A Runtime failure before the first A2A event should persist the already-created task as TASK_STATE_FAILED before returning the proxy error.

Stack

Stacked on dev/helm-release-split; review this PR against that branch.

@hhhhsc701 hhhhsc701 closed this Aug 19, 2026
@hhhhsc701
hhhhsc701 force-pushed the dev/northbound-runtime-forwarding branch from e451e6d to ab86966 Compare August 19, 2026 06:50
@hhhhsc701
hhhhsc701 merged commit ab86966 into dev/helm-release-split Aug 19, 2026
@hhhhsc701
hhhhsc701 force-pushed the dev/helm-release-split branch from bfa5450 to 03652f7 Compare August 19, 2026 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant