Skip to content

Restore rpk cloud mcp command documentation#1688

Merged
JakeSCahill merged 5 commits intomainfrom
feature/remove-mcp-server-docs
May 1, 2026
Merged

Restore rpk cloud mcp command documentation#1688
JakeSCahill merged 5 commits intomainfrom
feature/remove-mcp-server-docs

Conversation

@JakeSCahill
Copy link
Copy Markdown
Contributor

Summary

  • Restores the rpk cloud mcp install and rpk cloud mcp stdio command documentation that was removed in error
  • These commands are still needed for the Redpanda Cloud management MCP server

Test plan

  • Verify the pages render correctly in the documentation site

🤖 Generated with Claude Code

JakeSCahill and others added 3 commits April 24, 2026 09:19
The rpk connect mcp-server commands are deprecated:
- Remove rpk-connect-mcp-server.adoc
- Remove rpk-connect-mcp-server-init.adoc
- Remove rpk-connect-mcp-server-lint.adoc
- Update nav.adoc to remove entries

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Remove the following partial files that are no longer included by cloud-docs:
- rpk-cloud-mcp.adoc
- rpk-cloud-mcp-install.adoc
- rpk-cloud-mcp-proxy.adoc
- rpk-cloud-mcp-stdio.adoc

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
These pages were removed in error. The rpk cloud mcp commands are still
needed for the Redpanda Cloud management MCP server.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@JakeSCahill JakeSCahill requested a review from a team as a code owner April 30, 2026 08:51
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 30, 2026

Deploy Preview for redpanda-docs-preview ready!

Name Link
🔨 Latest commit a0d476f
🔍 Latest deploy log https://app.netlify.com/projects/redpanda-docs-preview/deploys/69f4ced5f5821e00089cd87d
😎 Deploy Preview https://deploy-preview-1688--redpanda-docs-preview.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.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 30, 2026

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.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: a507cd55-eee8-4070-90d3-d2e1433f29be

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

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

This pull request removes documentation for MCP (Message Collaboration Protocol) server integration features. The changes delete five AsciiDoc reference pages covering rpk connect mcp-server, rpk connect mcp-server init, rpk connect mcp-server lint, rpk cloud mcp, and rpk cloud mcp-proxy commands. Additionally, corresponding navigation entries in nav.adoc are removed. No code changes or functional modifications are included.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • paulohtb6
  • birdayz
🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title mentions 'Restore rpk cloud mcp command documentation' but the changeset primarily shows removals of MCP-related documentation files (rpk-connect-mcp-server and rpk-cloud-mcp files), which contradicts the stated intent to restore documentation. Clarify the title to accurately reflect whether this PR removes or restores MCP documentation, or provide additional context about which specific files are being restored.
Description check ❓ Inconclusive The description is incomplete against the template requirements. It lacks the JIRA ticket link in the Resolves field, missing page preview links, and the Checks section is not properly filled out with any selected options. Fill in the JIRA ticket reference, add page preview links for the restored documentation pages, and mark the appropriate checkbox under Checks (likely Content gap or Support Follow-up).
✅ Passed checks (3 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/remove-mcp-server-docs

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.

JakeSCahill and others added 2 commits May 1, 2026 15:59
Add the parent command documentation for rpk cloud mcp to complete the
restoration of the rpk cloud mcp command documentation.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Resolve conflicts by keeping the rpk cloud mcp documentation files that
were removed in main but are being restored in this PR.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@micheleRP micheleRP left a comment

Choose a reason for hiding this comment

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

lgtm

@JakeSCahill JakeSCahill merged commit f45b216 into main May 1, 2026
7 checks passed
@JakeSCahill JakeSCahill deleted the feature/remove-mcp-server-docs branch May 1, 2026 18:20
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