dhi: add mcp server#25589
Conversation
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟡 NEEDS ATTENTION
1 medium finding in new content added by this PR.
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
Verified redirect (alias) coverage for all 21 moved/renamed pages — every deleted or relocated page has a correct aliases: entry in its new location. The new mcp.md page is well-structured, uses proper front matter, avoids style-guide violations (no hedge words, no AI-isms, no marketing language), and the content is accurate. Two minor style suggestions are left as inline comments.
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
The PR adds a new DHI MCP server documentation page and reorganizes the DHI navigation structure (adding a tools section, moving core-concepts into explore, moving troubleshoot into how-to). All reviewed items look correct:
- Redirects: All moved/removed pages have proper
aliases:entries covering the old URLs (core-concepts → security-concepts, troubleshoot, feedback, cli, features) - New mcp.md: Front matter is complete, content is accurate, Docker Agent YAML config (
remote:+transport_type: streamable) matches the vendored Docker Agent configuration reference - Style: No hedge words, AI-isms, or style violations found in the new content
- Tool count: Stated "ten tools" matches the ten-row table
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟡 NEEDS ATTENTION
One style guide violation found in the new MCP server documentation page. All other areas are clean: front matter is complete, shortcode syntax is correct, the security warning callout is appropriate, the base64 command is accurate, link text is descriptive, and no hedge words or AI-isms were found.
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
Description
Added topic for DHI mcp server. (main point of this PR)
Added a "tools" section in order to have a home for the mcp topic. It includes:
Re-orged topics because adding "tools" made the top-level nav overloaded:
The how-to section is still a bit too much. That's a harder problem and this PR already scope-creeped enough.
Related issues or tickets
ENGDOCS-3337
Reviews