Skip to content

dhi: add mcp server#25589

Open
craig-osterhout wants to merge 7 commits into
docker:mainfrom
craig-osterhout:docs/dhi-mcp-server
Open

dhi: add mcp server#25589
craig-osterhout wants to merge 7 commits into
docker:mainfrom
craig-osterhout:docs/dhi-mcp-server

Conversation

@craig-osterhout

@craig-osterhout craig-osterhout commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

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:

  • mcp (new)
  • cli (moved from how-to)
  • hub (new, moved relevant content from explore how-to)
  • terrarform (new, aggregates content from how-tos)

Re-orged topics because adding "tools" made the top-level nav overloaded:

  • Moved core concepts into explore to reduce top-level navigation overload. Keeps most of the conceptual stuff in one place.
  • Moved troubleshoot into how-to to reduce top-level navigation overload. Keeps most of the tasks in one place.
  • Merged feedback into additional resources as it is awkward under explore. Been meaning to do it for awhile.
  • Merged features into main index page to reduce top-level navigation overload. Removed a lot of the fluff. Keeps most of the marketing stuff in one place.

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

  • Technical review
  • Editorial review
  • Product review

Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
@netlify

netlify Bot commented Jul 17, 2026

Copy link
Copy Markdown

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit d481ea2
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6a5a9d2baa66d400089e9a0a
😎 Deploy Preview https://deploy-preview-25589--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assessment: 🟡 NEEDS ATTENTION

1 medium finding in new content added by this PR.

Comment thread content/manuals/dhi/tools/mcp.md Outdated
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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.

Comment thread content/manuals/dhi/tools/mcp.md Outdated
Comment thread content/manuals/dhi/tools/mcp.md
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
@craig-osterhout
craig-osterhout requested a review from a team July 17, 2026 19:12
@craig-osterhout craig-osterhout added the status/review Pull requests that are ready for review label Jul 17, 2026

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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 docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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.

Comment thread content/manuals/dhi/tools/mcp.md
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
Comment on lines +12 to +19
## Search the catalog

You can browse, search, or filter images by category in the [Docker Hub
catalog](https://hub.docker.com/hardened-images/catalog). For details about
the catalog interface, see [Docker Hub](/dhi/tools/hub/).

Alternatively, use the [DHI CLI](/dhi/tools/cli/) to browse the catalog from
the command line:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think we could also mention the MCP server here too right?

@dvdksn dvdksn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM - left a comment but feel free to ignore if that's not the right place to mention the MCP server.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/dhi area/guides status/review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants