Skip to content

More doc fixing with "agent serve mcp"#2130

Merged
dgageot merged 1 commit intodocker:mainfrom
gtardif:fix_more_serve_docs
Mar 16, 2026
Merged

More doc fixing with "agent serve mcp"#2130
dgageot merged 1 commit intodocker:mainfrom
gtardif:fix_more_serve_docs

Conversation

@gtardif
Copy link
Contributor

@gtardif gtardif commented Mar 16, 2026

No description provided.

Signed-off-by: Guillaume Tardif <guillaume.tardif@gmail.com>
@gtardif gtardif requested a review from a team as a code owner March 16, 2026 17:01
Copy link

@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.

Review Summary

Assessment: 🟢 APPROVE

Summary

This PR correctly fixes command-line syntax in documentation examples:

  1. docs/features/acp/index.md: Fixed the JavaScript spawn() call to properly pass arguments as an array in the second parameter
  2. docs/features/mcp-mode/index.md: Added the missing "serve" argument to the MCP configuration args array

Both corrections align with the documented command syntax (docker agent serve acp and docker agent serve mcp). The changes are accurate and improve the documentation quality.

Findings

No bugs detected. The documentation fixes are correct.

@dgageot dgageot merged commit 2aa5a46 into docker:main Mar 16, 2026
8 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