Skip to content

fix: rename cagent binary to docker-agent#109

Merged
derekmisler merged 1 commit intodocker:mainfrom
derekmisler:vk/5f3d-i-need-to-rename
Mar 23, 2026
Merged

fix: rename cagent binary to docker-agent#109
derekmisler merged 1 commit intodocker:mainfrom
derekmisler:vk/5f3d-i-need-to-rename

Conversation

@derekmisler
Copy link
Copy Markdown
Contributor

@derekmisler derekmisler commented Mar 23, 2026

Summary

  • Rename the cagent binary to docker-agent in download URLs, cache paths, and execution paths — the old asset names were dropped in v1.36.1
  • Rename CAGENT_VERSION file to DOCKER_AGENT_VERSION and update all internal env vars/step IDs
  • Use correct product name ("Docker Agent") and CLI command (docker agent) conventions in user-facing text
  • Bump version to v1.36.1
  • Fix docker/cagentdocker/docker-agent repo references in version update workflow

Does NOT rename the action itself, output names (cagent-version), secret names, or <!-- cagent-review --> HTML markers — deferred to the full action rename.

Test plan

  • All existing tests pass locally (test-output-extraction.sh, test-job-summary.sh, test-exploits.sh)
  • Verified v1.36.1 download URL resolves: https://github.com/docker/docker-agent/releases/download/v1.36.1/docker-agent-linux-amd64
  • CI pirate agent test validates end-to-end binary download and execution

Closes docker/gordon#266

@derekmisler derekmisler requested a review from a team as a code owner March 23, 2026 16:05
Copy link
Copy Markdown
Contributor

@docker-agent docker-agent bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assessment: 🟢 APPROVE

All changed code has been reviewed. No bugs found in the changes.

Summary:

  • ✅ Binary download URLs correctly use docker-agent-* naming (verified by PR author)
  • ✅ Workflow renaming is consistent across all files
  • ✅ Version file renamed from CAGENT_VERSION to DOCKER_AGENT_VERSION

This PR successfully renames cagentdocker-agent across the codebase with no functional issues.

@derekmisler derekmisler self-assigned this Mar 23, 2026
Signed-off-by: Derek Misler <derek.misler@docker.com>
@derekmisler derekmisler force-pushed the vk/5f3d-i-need-to-rename branch from 8f922c4 to f3ddc2c Compare March 23, 2026 16:16
@derekmisler derekmisler merged commit ab2c594 into docker:main Mar 23, 2026
8 checks passed
derekmisler added a commit to derekmisler/cagent-action that referenced this pull request Mar 24, 2026
The binary is installed as docker-agent (standalone), not as a
docker CLI plugin. The command should be docker-agent, not docker agent.

This was broken in docker#109 when the binary was renamed from cagent to
docker-agent — the install step was updated but the invocation was
changed to 'docker agent' (CLI plugin syntax) instead of 'docker-agent'
(standalone binary syntax).

Assisted-By: docker-agent
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