Skip to content

Issue #103: Claude ACP qualification を adapter version 固定から実挙動ベースにする - #104

Merged
geeen-jp merged 2 commits into
mainfrom
aidw/exec/exec-98ef2266119313548c32926df0990d1e
Oct 11, 2026
Merged

geeen-jp merged 2 commits into
mainfrom
aidw/exec/exec-98ef2266119313548c32926df0990d1e

Conversation

@geeen-jp

Copy link
Copy Markdown
Owner

This PR was prepared by the workflow promotion pipeline. Merge and Issue closure remain outside workflow authority.

@github-actions

Copy link
Copy Markdown

Endpoint audit

⚠️ 18 actionable Agent Server contract divergence(s) · report-only

Contract: pinned release artifact

Category Count
Actionable client-only calls 0
Actionable server-only operations 18
Documented non-divergences 13
Agent Server contract operations 129
Audited handwritten client endpoints 119

Actionable client-only calls (0)

none

Actionable server-only operations (18)

  • DELETE /api/canvas-extensions/installed/{}
  • DELETE /api/llm/provider-connections/{}
  • GET /api/canvas-extensions/installed
  • GET /api/canvas-extensions/installed/{}
  • GET /api/canvas-extensions/installed/{}/bundle
  • GET /api/conversations/{}/events
  • GET /api/file/archive
  • GET /api/git/commits
  • GET /api/git/commits/{}/changes
  • GET /api/init
  • GET /api/llm/provider-connections
  • PATCH /api/canvas-extensions/installed/{}
  • PATCH /api/llm/provider-connections/{}
  • POST /api/canvas-extensions/install
  • POST /api/conversations/{}/load_plugin
  • POST /api/file/create_directory
  • POST /api/init
  • POST /api/llm/provider-connections
Documented non-divergences (13)

Client calls intentionally absent from the filtered contract (11)

  • GET /
  • GET /alive
  • GET /health
  • GET /ready
  • GET /server_info

Reason: Operational Agent Server endpoints intentionally excluded from the filtered public release artifact.
Owner: OpenHands runtime maintainers

  • DELETE /api/meta-profiles/{}
  • GET /api/meta-profiles
  • GET /api/meta-profiles/{}
  • POST /api/meta-profiles/{}
  • POST /api/meta-profiles/{}/activate

Reason: Client-ahead API stacked on the pending Agent Server meta-profiles implementation.
Owner: OpenHands SDK maintainers
Tracking: OpenHands#3744

  • POST /api/profiles/{}/validate

Reason: Client-ahead API stacked on the pending Agent Server pre-flight LLM validation endpoint.
Owner: OpenHands TypeScript client maintainers
Tracking: OpenHands#4422

Server operations covered by an exposed browser URL (2)

  • GET /api/conversations/{}/workspace
  • GET /api/conversations/{}/workspace/{}

Reason: RemoteWorkspace.startWorkspaceSession exposes these authenticated URLs for browser iframe and file requests; they are not HttpClient method calls.
Owner: OpenHands TypeScript client maintainers

@github-actions

Copy link
Copy Markdown

Python API breakage checks — ✅ PASSED

Result: ✅ PASSED

Action log

@github-actions

Copy link
Copy Markdown

REST API breakage checks (OpenAPI) — ✅ PASSED

Result: ✅ PASSED

Action log

@github-actions

Copy link
Copy Markdown

Coverage

Coverage Report •
FileStmtsMissCoverMissing
openhands-sdk/openhands/sdk/agent
   acp_agent.py259433187%268, 338–341, 382, 414, 543, 550, 563, 588, 605, 608, 688, 695, 698–699, 795, 802, 822, 864, 867, 941, 1029–1030, 1086–1087, 1091, 1101–1104, 1110–1113, 1116, 1123, 1137, 1143, 1149, 1156, 1158, 1199, 1287, 1406, 1456, 1572, 1576, 1578, 1580, 1609, 1612, 1614, 1618, 1621, 1630, 1713, 1719, 1742, 1800, 1810–1816, 1842–1844, 1855–1856, 1893, 1896, 1913–1917, 2146, 2150, 2186, 2192, 2200–2201, 2204, 2220–2221, 2237, 2240, 2243–2248, 2253, 2263, 2337, 2362–2372, 2381, 2421–2422, 2477, 2479, 2483, 2487, 2503, 2645, 2792, 2940, 2943–2944, 2961–2962, 2978, 2995, 3000, 3024, 3055, 3068, 3082, 3103, 3161, 3173, 3222, 3230, 3233, 3237–3240, 3251–3253, 3275, 3283–3284, 3287–3290, 3299–3301, 3307–3309, 3317–3318, 3330–3335, 3338–3340, 3347–3350, 3355–3359, 3371, 3397–3398, 3402–3403, 3411–3414, 3421, 3473, 3478, 3571–3572, 3595–3596, 3696, 3703–3704, 3796, 4056–4057, 4060, 4064–4066, 4069–4073, 4295, 4396, 4510, 4568, 4574, 4589–4590, 4601–4604, 4630, 4638, 4811–4813, 4825–4827, 4854, 4869–4870, 4873–4875, 4882, 4886, 4897–4898, 4906, 4911, 4923–4924, 4938, 4945, 4950–4953, 4955–4958, 4960–4963, 4985, 4997–4999, 5034–5035, 5038, 5046–5048, 5052, 5054, 5058, 5061, 5070–5072, 5076, 5110–5112, 5114, 5119–5121, 5130–5134, 5137, 5142–5144, 5147–5148, 5163–5165, 5220–5222, 5460–5462, 5501, 5505, 5516, 5526, 5534, 5564, 5597–5599, 5613, 5624–5627, 5722–5723, 5764–5765, 5780–5781, 5789, 5792–5793, 5798, 5800, 5834–5835, 5851–5852
TOTAL430861188172% 

@geeen-jp
geeen-jp merged commit c4fa8b9 into main Oct 11, 2026
31 checks passed
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