Add ACPAgent remote runtime example docs#392
Conversation
Document examples/02_remote_agent_server/09_acp_agent_with_remote_runtime.py in the ACP agent guide page. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
all-hands-bot
left a comment
There was a problem hiding this comment.
🟢 Good taste
Verified against upstream PR software-agent-sdk#2190 - code matches exactly, imports check out, follows documentation conventions. Clean, straightforward addition.
VERDICT: ✅ Merge it
No unnecessary complexity. No breaking changes. Just documents a real feature with working code. This is how docs PRs should be done.
|
@OpenHands please follow the docs in this PR and try to use the OPENHANDS_API_KEY, LLM_API_KEY and LLM_BASE_URL provided to you in secrets and try to run this example, and tell me if it works |
|
I'm on it! xingyaoww can track my progress at all-hands.dev |
|
I tried running the example Setup
Result: ❌ Failed with 401 UnauthorizedThe error message from the API: I also tried Root CauseThe example requires a Summary of secrets needed vs. available
To run this example, I would need the actual |
SummaryI attempted to run the ACP agent remote runtime example ( Steps Taken
Result: ❌ Failed with 401 UnauthorizedThe Runtime API ( Root CauseThe Actions
|
|
@OpenHands please follow the docs in this PR and try to use the RUNTIME_API_KEY (i just gave you that!), LLM_API_KEY and LLM_BASE_URL provided to you in secrets and try to run this example, and tell me if it works |
|
I'm on it! xingyaoww can track my progress at all-hands.dev |
Results of Running the ACP Remote Runtime ExampleI ran the example Run 1: Using
|
| Aspect | Status |
|---|---|
RUNTIME_API_KEY authentication |
✅ Works |
LLM_API_KEY / LLM_BASE_URL setup |
✅ Configured correctly |
| Runtime API pod provisioning | ✅ Works (with valid images) |
| ACPAgent on remote runtime | ❌ Blocked — needs a server image with ACPAgent support |
Bottom line: The infrastructure and authentication all work correctly. The example itself cannot complete end-to-end until the SDK PR (#2190) is merged to main (so the main image includes ACPAgent), or until CI runs on the SDK PR and publishes the Docker image with the PR's commit SHA. This is expected for a pre-merge test — the example is designed to work in CI where GITHUB_SHA points to a just-built image.
SummaryI ran the ACP agent remote runtime example ( ResultsRun 1 — Using
Run 2 — Using PR-specific image (
ConclusionThe infrastructure and all credentials work correctly. The example cannot complete end-to-end right now because there is no Docker image available that includes ACPAgent support. This will resolve once either:
This is expected behavior — the example is designed for CI where No code changes were made — this was purely a test run. I posted the detailed findings as a comment on PR #392. |
Summary
examples/02_remote_agent_server/09_acp_agent_with_remote_runtime.pyto the ACP agent guide pageThis is the companion docs PR for OpenHands/software-agent-sdk#2190, which adds ACPAgent support on RemoteRuntime.
Test plan
check-examplesCI check passes on the agent-sdk PR after this branch exists🤖 Generated with Claude Code