Skip to content

Conversation

@paulohtb6
Copy link
Contributor

@paulohtb6 paulohtb6 commented Jul 28, 2025

Description

Do not merge until Redpanda v25.2.3 is released

Related source redpanda-data/docs#1242

Resolves https://github.com/redpanda-data/documentation-private/issues/DOC-1301
Resolves https://github.com/redpanda-data/documentation-private/issues/DOC-1649

This pull request introduces new documentation for the Redpanda Cloud MCP (Model Context Protocol) Server and reorganizes the AI Agents section to focus on this new feature. It adds comprehensive guides for setup, configuration, and usage of the MCP server with AI assistants like Claude and Claude Code. The navigation is updated to reflect these changes, and legacy support agent documentation is removed.

AI Agents & MCP Server Documentation Overhaul:

  • Added a new section for Redpanda Cloud MCP Server, including an overview, quickstart, and detailed configuration guide, targeting integration with AI assistants such as Claude and Claude Code. [1] [2] [3] [4]
  • Removed the old support agent documentation and the previous AI Agents overview page, consolidating the focus on MCP-based integrations. [1] [2]

Navigation and Structure Updates:

  • Updated nav.adoc to replace the old AI Agents links with the new MCP Server documentation, and added new references for rpk cloud mcp commands under the CLI reference. [1] [2]
  • Modified the Antora playbook to include the new docs-site repo and its home start path, supporting the new documentation structure.

Reference Documentation for CLI Integration:

  • Added new reference pages for the rpk cloud mcp install and rpk cloud mcp stdio commands, supporting the setup and troubleshooting instructions in the MCP docs. [1] [2]

Page previews

https://deploy-preview-369--rp-cloud.netlify.app/redpanda-cloud/ai-agents/mcp/local/

Checks

  • New feature
  • Content gap
  • Support Follow-up
  • Small fix (typos, links, copyedits, etc)

@paulohtb6 paulohtb6 requested a review from a team as a code owner July 28, 2025 20:10
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jul 28, 2025

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

📝 Walkthrough

Walkthrough

Added a new MCP-local AI Agents docs set (modules/ai-agents/pages/mcp/local: index.adoc, overview.adoc, quickstart.adoc, configuration.adoc) and two rpk-cloud pages (modules/reference/pages/rpk/rpk-cloud/rpk-cloud-mcp-install.adoc and rpk-cloud-mcp-stdio.adoc). Updated modules/ROOT/nav.adoc to insert an "rpk cloud mcp" subsection referencing the new rpk-cloud pages and relocated AI Agents navigation to ai-agents:mcp/local. Removed two legacy develop agents pages (modules/develop/pages/agents/about.adoc and create-support-agent.adoc). Added a new Antora content source entry in local-antora-playbook.yml.

Sequence Diagram(s)

(omitted — changes are documentation/navigation edits and do not modify runtime control flow)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Assessment against linked issues

Objective (issue#) Addressed Explanation
Document local MCP server and provide a guide (DOC-1649)
Add new rpk MCP commands pages: rpk cloud mcp, rpk cloud mcp install, rpk cloud mcp stdio (DOC-1301)

Out-of-scope changes

Code Change Explanation
Deleted AI Agents overview page (modules/develop/pages/agents/about.adoc) This removal is not referenced in the linked issues which focused on adding MCP docs and rpk commands.
Deleted Support Agent guide (modules/develop/pages/agents/create-support-agent.adoc) Removal of this end-to-end guide is unrelated to documenting the local MCP server or adding rpk commands.
Added Antora content source entry (local-antora-playbook.yml) Build/playbook configuration change is not specified in the linked issues about documenting MCP or rpk commands.

Possibly related PRs

Suggested reviewers

  • Feediver1
  • david-yu
✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch rpk-cloud-mcp

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@netlify
Copy link

netlify bot commented Jul 28, 2025

Deploy Preview for rp-cloud ready!

Name Link
🔨 Latest commit a70b143
🔍 Latest deploy log https://app.netlify.com/projects/rp-cloud/deploys/68bf19e0119e4e00089b52a4
😎 Deploy Preview https://deploy-preview-369--rp-cloud.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.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🧹 Nitpick comments (2)
modules/reference/pages/rpk/rpk-cloud/rpk-cloud-mcp-install.adoc (1)

1-1: Prefer sentence-case titles for consistency

Most Redpanda CLI reference pages use sentence-case command titles (for example, = rpk cloud login).
Consider updating to:

= rpk cloud mcp install

so the initial “rpk” is lowercase but each subsequent noun is lowercase as well, matching the established pattern.

modules/reference/pages/rpk/rpk-cloud/rpk-cloud-mcp-stdio.adoc (1)

1-1: Title style – align with existing CLI docs

For consistency with other rpk command docs, prefer sentence-case:

= rpk cloud mcp stdio

Minor but keeps the rendered ToC coherent.

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6b7dbbd and 17e80cd.

📒 Files selected for processing (3)
  • modules/ROOT/nav.adoc (1 hunks)
  • modules/reference/pages/rpk/rpk-cloud/rpk-cloud-mcp-install.adoc (1 hunks)
  • modules/reference/pages/rpk/rpk-cloud/rpk-cloud-mcp-stdio.adoc (1 hunks)
🧰 Additional context used
🧠 Learnings (3)
📓 Common learnings
Learnt from: micheleRP
PR: redpanda-data/cloud-docs#267
File: modules/manage/pages/maintenance.adoc:91-92
Timestamp: 2025-04-25T01:41:57.162Z
Learning: The notification timeline for Redpanda Cloud deprecations has been deliberately removed from the documentation, even though the PR summary mentioned a 180-day advance notice period.
Learnt from: micheleRP
PR: redpanda-data/cloud-docs#278
File: modules/manage/pages/cluster-maintenance/config-cluster.adoc:75-75
Timestamp: 2025-04-29T18:43:42.666Z
Learning: In AsciiDoc documentation used by Redpanda, empty cross-references (xrefs) without link text (like `xref:manage:rpk/intro-to-rpk.adoc[]`) automatically use the target page's title as the link text.
modules/ROOT/nav.adoc (1)

Learnt from: micheleRP
PR: #334
File: modules/networking/partials/psc-api2.adoc:15-16
Timestamp: 2025-06-18T21:02:38.074Z
Learning: In the Redpanda Cloud documentation system, cross-reference anchors using the format #patch-/v1/clusters/-cluster.id- work correctly for referencing API endpoints, even with dashes instead of curly braces around parameter names.

modules/reference/pages/rpk/rpk-cloud/rpk-cloud-mcp-install.adoc (1)

Learnt from: micheleRP
PR: #278
File: modules/manage/pages/cluster-maintenance/config-cluster.adoc:75-75
Timestamp: 2025-04-29T18:43:42.666Z
Learning: In AsciiDoc documentation used by Redpanda, empty cross-references (xrefs) without link text (like xref:manage:rpk/intro-to-rpk.adoc[]) automatically use the target page's title as the link text.

🔇 Additional comments (2)
modules/reference/pages/rpk/rpk-cloud/rpk-cloud-mcp-install.adoc (1)

3-3: Verify the include target exists and the URI syntax is correct

include::ROOT:reference:partial$rpk-cloud/rpk-cloud-mcp-install.adoc[] is a less common Antora URI form (module family after : and $).
Double-check that:

  1. A partial file actually lives at modules/reference/partials/rpk-cloud/rpk-cloud-mcp-install.adoc.
  2. The family qualifier is correct (partial is the family, not part of the path).
    A more typical pattern is:
include::reference:partial$rpk-cloud/rpk-cloud-mcp-install.adoc[tag=single-source]

If the build fails to resolve the include, adjust accordingly.

modules/reference/pages/rpk/rpk-cloud/rpk-cloud-mcp-stdio.adoc (1)

3-3: Same include-path concern as in the install page

Ensure the partial exists and the Antora URI (ROOT:reference:partial$...) resolves during the site build; otherwise switch to the standard reference:partial$… form.

@paulohtb6 paulohtb6 requested a review from JakeSCahill July 28, 2025 20:13
@JakeSCahill JakeSCahill changed the title docs: add the new rpk cloud mcp reference DOC-1649 Redpanda Cloud MCP Server Sep 3, 2025
@JakeSCahill JakeSCahill requested a review from birdayz September 3, 2025 14:03
@JakeSCahill JakeSCahill changed the title DOC-1649 Redpanda Cloud MCP Server docs: DOC-1649 Redpanda Cloud MCP Server Sep 3, 2025
@JakeSCahill JakeSCahill removed their request for review September 3, 2025 15:57
@JakeSCahill
Copy link
Contributor

@coderabbitai review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 3, 2025

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

♻️ Duplicate comments (1)
modules/ROOT/nav.adoc (1)

490-492: Root page for “rpk cloud mcp” (tracked earlier)

Repeating the earlier suggestion to add a clickable root page or change the group to an xref for consistency with other rpk cloud sub-sections. If the parent command isn’t published by design, feel free to ignore.

Example:

-**** rpk cloud mcp
+**** xref:reference:rpk/rpk-cloud/rpk-cloud-mcp.adoc[rpk cloud mcp]
🧹 Nitpick comments (8)
modules/ai-agents/pages/mcp/local/overview.adoc (3)

35-37: Clarify “install” vs “run” to avoid implying a daemon is started

rpk cloud mcp install configures the client; the server process is started on-demand via rpk cloud mcp stdio (or by the client), not by “install.” Recommend tightening wording.

-. *Local MCP server*: Start the local MCP server using the xref:reference:rpk/rpk-cloud/rpk-cloud-mcp-install.adoc[`rpk cloud mcp install` command]. This process authenticates using the token obtained from `rpk cloud login` and runs the MCP server on your machine.
+. **Local MCP server:** Configure your client using the xref:reference:rpk/rpk-cloud/rpk-cloud-mcp-install.adoc[`rpk cloud mcp install`] command. Your client then starts the server on‑demand via xref:reference:rpk/rpk-cloud/rpk-cloud-mcp-stdio.adoc[`rpk cloud mcp stdio`], authenticating with the token from `rpk cloud login`.

35-37: Consistency: use bold for all step labels

Step 1 uses bold, step 2 uses italics. Make both bold for consistency.

-. *Local MCP server*:
+. **Local MCP server**:

52-52: Cross-ref target may be outside loaded components

xref:home:ROOT:mcp-setup.adoc[] depends on the home component containing this page. Please confirm the page exists and is included by the preview playbook.

I can adjust the link or add the needed content source if it’s missing.

local-antora-playbook.yml (1)

19-21: Include “shared” in start_paths for docs-site in local preview
Add shared alongside home in start_paths so shared assets (e.g., image::shared: includes) resolve during local Antora previews. Keep this change in local-antora-playbook.yml only; it will be reverted to main before merge.

   - url: https://github.com/redpanda-data/docs-site
     branches: [main]
-    start_paths: [home]
+    start_paths: [home, shared]
modules/ai-agents/pages/mcp/local/configuration.adoc (3)

55-68: Verify --allow-delete support on install

Ensure rpk cloud mcp install --client <...> --allow-delete is supported (vs. needing to pass args to stdio). If not, provide the correct flag or a post-install edit step.

I can adjust examples once the CLI behavior is confirmed.


113-114: Typo: duplicate “the”

Small wording fix.

-To update, re-run the the xref:reference:rpk/rpk-cloud/rpk-cloud-mcp-install.adoc[`rpk cloud mcp install` command].
+To update, re-run the xref:reference:rpk/rpk-cloud/rpk-cloud-mcp-install.adoc[`rpk cloud mcp install` command].

148-154: Typo and clarity when starting manually

Fix duplicate “the” and explicitly note the session is ephemeral.

-. [[local]]Start the server manually using the the xref:reference:rpk/rpk-cloud/rpk-cloud-mcp-stdio.adoc[`rpk cloud mcp stdio` command] (one-time login required) to make sure it can reach Redpanda Cloud endpoints:
+. [[local]]Start the server manually using the xref:reference:rpk/rpk-cloud/rpk-cloud-mcp-stdio.adoc[`rpk cloud mcp stdio` command] (one-time login required) to verify connectivity to Redpanda Cloud endpoints:
modules/ai-agents/pages/mcp/local/quickstart.adoc (1)

62-63: Cross-link to deletes section for discoverability

Consider linking to the configuration page’s “Enable delete operations” section to guide users who opt in.

-NOTE: The MCP server does **not** expose API endpoints that result in delete operations by default. Use `--allow-delete` only if you intentionally want to enable the delete operations.
+NOTE: The MCP server does **not** expose API endpoints that result in delete operations by default. Use `--allow-delete` only if you intentionally want to enable delete operations. See xref:ai-agents:mcp/local/configuration.adoc#_enable_delete_operations[Enable delete operations].
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

  • MCP integration is disabled by default for public repositories
  • Jira integration is disabled by default for public repositories
  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 17e80cd and 6b74b22.

⛔ Files ignored due to path filters (1)
  • modules/shared/images/cloud-mcp.gif is excluded by !**/*.gif
📒 Files selected for processing (8)
  • local-antora-playbook.yml (1 hunks)
  • modules/ROOT/nav.adoc (2 hunks)
  • modules/ai-agents/pages/mcp/local/configuration.adoc (1 hunks)
  • modules/ai-agents/pages/mcp/local/index.adoc (1 hunks)
  • modules/ai-agents/pages/mcp/local/overview.adoc (1 hunks)
  • modules/ai-agents/pages/mcp/local/quickstart.adoc (1 hunks)
  • modules/develop/pages/agents/about.adoc (0 hunks)
  • modules/develop/pages/agents/create-support-agent.adoc (0 hunks)
💤 Files with no reviewable changes (2)
  • modules/develop/pages/agents/create-support-agent.adoc
  • modules/develop/pages/agents/about.adoc
✅ Files skipped from review due to trivial changes (1)
  • modules/ai-agents/pages/mcp/local/index.adoc
🧰 Additional context used
🧠 Learnings (2)
📚 Learning: 2025-08-15T04:45:28.695Z
Learnt from: micheleRP
PR: redpanda-data/cloud-docs#390
File: modules/manage/pages/schema-reg/schema-reg-authorization.adoc:4-4
Timestamp: 2025-08-15T04:45:28.695Z
Learning: In the Redpanda documentation system, content is single-sourced across multiple repositories (cloud-docs and docs repos). Include directives in the cloud-docs repo may reference files that exist in separate PRs in the docs repo. These PRs are linked via local-antora-playbook.yml for preview rendering, and the includes resolve correctly during the Antora build process when repositories are merged. The playbook is reverted to main before merging. This cross-repository single sourcing pattern is commonly used, so missing include targets should be verified against linked PRs in other repositories before flagging as errors.

Applied to files:

  • local-antora-playbook.yml
📚 Learning: 2025-08-15T02:29:34.901Z
Learnt from: micheleRP
PR: redpanda-data/cloud-docs#390
File: modules/manage/pages/schema-reg/schema-reg-authorization.adoc:4-4
Timestamp: 2025-08-15T02:29:34.901Z
Learning: In Redpanda Cloud documentation PRs, when CodeRabbit flags missing Asciidoctor tag markers for include directives, the fix may be implemented in the corresponding ROOT module file that contains the actual content being included, rather than in the file where the include directive appears.

Applied to files:

  • modules/ROOT/nav.adoc
🔇 Additional comments (6)
modules/ai-agents/pages/mcp/local/overview.adoc (1)

13-13: Verify image source component (“shared”) is included in playbook

image::shared:cloud-mcp.gif[] relies on a shared component. Ensure the preview playbook includes that component, or the image will 404 in previews.

Would you like me to add shared to start_paths in local-antora-playbook.yml?

modules/ROOT/nav.adoc (1)

72-75: Nice: clear, discoverable MCP Local section

New AI Agents/MCP Local group with index/overview/quickstart/configure reads well and matches the reorg.

modules/ai-agents/pages/mcp/local/configuration.adoc (2)

46-46: Link target accuracy: default config paths

The “default paths” are referenced in rpk cloud login docs. Please verify those paths are indeed documented there; otherwise link to the page that enumerates them.


160-163: Protocol version string: confirm correctness

"protocolVersion":"2025-06-18" looks like a date. Please confirm this is the MCP protocol identifier expected by clients; otherwise switch to the current supported value or omit.

modules/ai-agents/pages/mcp/local/quickstart.adoc (2)

37-38: Good security callout

Clear statement that the token is local and not shared with the assistant.


15-19: Clarify rpk & MCP context for version requirement
Provide the specific rpk binary and a link to this MCP quickstart so we can determine the earliest version supporting install/stdio and align the “minimum version” (e.g. “25.1.2+ (25.2.3+ recommended)”) accordingly.

@JakeSCahill JakeSCahill requested review from birdayz and weeco September 4, 2025 09:08
@JakeSCahill JakeSCahill requested a review from a team September 8, 2025 15:27
Copy link
Contributor

@asimms41 asimms41 left a comment

Choose a reason for hiding this comment

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

Just a couple of small suggestions. No blockers.

@JakeSCahill JakeSCahill merged commit b02aef5 into main Sep 8, 2025
5 checks passed
@JakeSCahill JakeSCahill deleted the rpk-cloud-mcp branch September 8, 2025 18:05
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.

5 participants