Skip to content

fix: sync acp-providers.json with openhands-sdk ACP_PROVIDERS - #328

Merged
VascoSch92 merged 1 commit into
bump-agent-server-1.41.0from
fix/acp-providers-drift-1.41.0
Aug 6, 2026
Merged

fix: sync acp-providers.json with openhands-sdk ACP_PROVIDERS#328
VascoSch92 merged 1 commit into
bump-agent-server-1.41.0from
fix/acp-providers-drift-1.41.0

Conversation

@VascoSch92

Copy link
Copy Markdown
Member

Summary

Fixes the failing validate-acp-providers check on #327.

scripts/check-acp-drift.py compares src/models/acp-providers.json against ACP_PROVIDERS in openhands-sdk (installed from software-agent-sdk@main). After the v1.41.0 SDK bump in #327, the upstream provider list had moved and this file's mirror was stale:

  • Added the new claude-opus-5 model entry
  • Dropped version numbers from Claude model labels (Claude Opus 4.8 (1M)Claude Opus (1M), Claude Sonnet 4.6Claude Sonnet, Claude Haiku 4.5Claude Haiku)
  • Bumped pinned @agentclientprotocol/claude-agent-acp from 0.44.0 to 0.63.0
  • Bumped pinned @agentclientprotocol/codex-acp from 1.1.2 to 1.1.7

Test plan

  • python scripts/check-acp-drift.py passes locally against openhands-sdk@main
  • npx prettier --check src/models/acp-providers.json passes
  • CI green on this branch (validates against the same SDK install CI uses)

The v1.41.0 SDK bump brought ACP_PROVIDERS changes upstream that
src/models/acp-providers.json wasn't updated for, failing the
validate-acp-providers CI check:

- add claude-opus-5 model entry
- drop version numbers from Claude model labels
- bump pinned @agentclientprotocol/claude-agent-acp to 0.63.0
- bump pinned @agentclientprotocol/codex-acp to 1.1.7
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Endpoint audit

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

Contract: pinned release artifact

Category Count
Actionable client-only calls 0
Actionable server-only operations 7
Documented non-divergences 12
Agent Server contract operations 117
Audited handwritten client endpoints 118

Actionable client-only calls (0)

none

Actionable server-only operations (7)

  • GET /api/conversations/{}/events
  • GET /api/file/archive
  • GET /api/git/commits
  • GET /api/git/commits/{}/changes
  • GET /api/init
  • POST /api/conversations/{}/load_plugin
  • POST /api/init
Documented non-divergences (12)

Client calls intentionally absent from the filtered contract (10)

  • 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/software-agent-sdk#3744

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

@VascoSch92
VascoSch92 merged commit 61ad179 into bump-agent-server-1.41.0 Aug 6, 2026
3 of 4 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.

2 participants