Conversation
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
github-actions
bot
added
area/desktop
![docker-agent[bot]](https://avatars.githubusercontent.com/in/2787757?s=60&v=4){kind=small}
There was a problem hiding this comment.
Review Summary
Assessment: 🟢 APPROVE
This PR successfully renames "cagent" to "Docker Agent" across 30 documentation files. The changes are comprehensive and well-executed. I found 2 minor issues that could be improved, but they don't block the PR:
- Typo: "Alternativey" should be "Alternatively"
- Version clarity: The version information could be clearer about when the name change takes effect
Both are minor wording improvements.
Findings
Minor Issues
Typo in installation instructions
- File: content/manuals/ai/docker-agent/_index.md:76
- Severity: Low
- Issue: The word "Alternativey" should be "Alternatively"
Version information could be clearer
- File: content/manuals/ai/docker-agent/_index.md:64-65
- Severity: Low
- Issue: The text states "Docker Agent is included in Docker Desktop 4.63 and later" then "cagent is included in Docker Desktop 4.49 and later." Consider clarifying that versions 4.49-4.62 included cagent (the old name) and 4.63+ includes Docker Agent (the renamed version).
CLI plugin vs standalone binary usage
- File: content/manuals/ai/docker-agent/integrations/mcp.md:96
- Severity: Low
- Issue: The example shows using the docker binary with CLI plugin approach, but the description mentions "path to docker-agent if not using the docker CLI plugin" which could be clearer about when to use which approach.
✅ No vendored or generated files were modified
✅ Proper aliases added for moved pages
✅ No broken links or command errors detected
✅ Markdown formatting is correct
|
|
||
| Before configuring MCP integration, you need: | ||
|
|
||
| - **cagent installed** - See the [installation guide](../_index.md#installation) |
| "mcpServers": { | ||
| "myagent": { | ||
| "command": "/usr/local/bin/cagent", | ||
| "command": "/usr/local/bin/docker", |
There was a problem hiding this comment.
For DD users, this is a symlink (on macos, to /Applications/Docker.app/Contents/Resources/bin/docker). Will it still work? For docker-ce users, the cli is usually installed to /usr/bin/docker (and I think for rootless it's under ~/.local/bin/docker)
should this point to the docker binary or the docker-agent binary?
There was a problem hiding this comment.
updated to just docker that should be in the path
| releases](https://github.com/docker/cagent/releases) | ||
| - **From source**: See the [cagent | ||
| - **From source**: See the [Docker Agent | ||
| repository](https://github.com/docker/cagent?tab=readme-ov-file#build-from-source) |
There was a problem hiding this comment.
a few more cagent mentions.
will the repo stay as docker/cagent?
There was a problem hiding this comment.
For now, the cagent repo is still named cagent. Also, the homebrew formula and winget packages are not yet renamed, will update the docs later on that
There was a problem hiding this comment.
This file is auto-generated from the sandbox CLI plugin
There was a problem hiding this comment.
Removed that change for now, since the sandbox CLI still mentions cagent, will need to update (and regenerate) that part when sandbox has been updated
gtardif
force-pushed
the
cagent_rename
branch
2 times, most recently
from
8e38623 to
c67b2c4
Compare
Signed-off-by: Guillaume Tardif <guillaume.tardif@gmail.com>
Signed-off-by: Guillaume Tardif <guillaume.tardif@gmail.com>
…49, docker agent is available in desktop 4.63 and later Signed-off-by: Guillaume Tardif <guillaume.tardif@gmail.com>
Signed-off-by: Guillaume Tardif <guillaume.tardif@gmail.com>
2 participants
Description
cagent is now docker-agent, available as a standalone command of a docker CLI plugin (by default installed with Desktop since 4.63)
Reviews