Skip to content

Bump direct Go dependencies#2586

Merged
dgageot merged 2 commits intodocker:mainfrom
dgageot:board/bump-direct-go-dependencies-f19df3ce
Apr 29, 2026
Merged

Bump direct Go dependencies#2586
dgageot merged 2 commits intodocker:mainfrom
dgageot:board/bump-direct-go-dependencies-f19df3ce

Conversation

@dgageot
Copy link
Copy Markdown
Member

@dgageot dgageot commented Apr 29, 2026

Bump direct Go dependencies.

Module From To Notes
github.com/openai/openai-go/v3 v3.32.0 v3.33.0 drop-in
github.com/coder/acp-go-sdk v0.6.3 v0.12.2 required code adaptation in pkg/acp/agent.go (see below)

acp-go-sdk v0.12.2 adaptation

The new acp.Agent interface adds several methods. Implemented them in pkg/acp/agent.go:

  • CloseSession — cancels in-flight work and removes the session from the in-memory map.
  • ListSessions, ResumeSession, SetSessionConfigOption — stubs since we don't advertise these capabilities.

The acp.RequestPermissionToolCall type was removed. The three existing usages (and the buildToolCallUpdate return type) were switched to the unified acp.ToolCallUpdate, which is what RequestPermissionRequest.ToolCall now expects.

Validation: mise lint and mise test pass.

dgageot added 2 commits April 29, 2026 07:49
Adapt pkg/acp to the new Agent interface, which adds the CloseSession,

ListSessions, ResumeSession and SetSessionConfigOption methods. The

RequestPermissionToolCall type was removed; permission requests now use

the unified ToolCallUpdate type.

Assisted-By: docker-agent
@dgageot dgageot requested a review from a team as a code owner April 29, 2026 06:10
@dgageot dgageot merged commit d46fbe4 into docker:main Apr 29, 2026
9 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