Skip to content

Remove deprecated rpk connect mcp-server documentation#1682

Merged
JakeSCahill merged 2 commits intomainfrom
feature/remove-mcp-server-docs
Apr 28, 2026
Merged

Remove deprecated rpk connect mcp-server documentation#1682
JakeSCahill merged 2 commits intomainfrom
feature/remove-mcp-server-docs

Conversation

@JakeSCahill
Copy link
Copy Markdown
Contributor

Summary

Remove the deprecated rpk connect mcp-server command documentation.

Changes

  • 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 these entries

Context

The rpk connect mcp-server commands have been deprecated along with the Remote MCP feature.

Test plan

  • Build succeeds without mcp-server pages
  • Nav renders correctly without mcp-server entries

🤖 Generated with Claude Code

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>
@JakeSCahill JakeSCahill requested a review from a team as a code owner April 24, 2026 08:20
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 24, 2026

Deploy Preview for redpanda-docs-preview ready!

Name Link
🔨 Latest commit a6c592d
🔍 Latest deploy log https://app.netlify.com/projects/redpanda-docs-preview/deploys/69ef85244cf0f000081c0e1e
😎 Deploy Preview https://deploy-preview-1682--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 24, 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: 95e0a766-9bbd-426a-9881-3bcae2beb109

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 pages and navigation references for the rpk connect mcp-server command. The changes include deletion of the main rpk-connect-mcp-server.adoc page, two subcommand documentation pages (rpk-connect-mcp-server-init.adoc and rpk-connect-mcp-server-lint.adoc), and their corresponding navigation entries in modules/ROOT/nav.adoc.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • paulohtb6
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description covers the changes made and context, but lacks required template sections like JIRA ticket link, page previews, and review deadline. Add the missing template sections: Resolves link (JIRA ticket), page previews, and review deadline. Ensure the description follows the repository's standard format.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and specifically describes the main change: removing deprecated rpk connect mcp-server documentation.
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.

@micheleRP
Copy link
Copy Markdown
Contributor

Review findings

Critical issues

None.

Suggestions

None for the scope as defined.

What works well

  • Cleanly scoped, minimal footprint (3 page deletions + nav).
  • Nav and deletions align.
  • No dangling xrefs to the deleted pages anywhere in the repo.

Impact on other files (cross-repo)

Not in scope for this PR as defined, but worth flagging as a follow-up:

  • modules/reference/partials/rpk-cloud/rpk-cloud-mcp.adoc, rpk-cloud-mcp-install.adoc, rpk-cloud-mcp-proxy.adoc, rpk-cloud-mcp-stdio.adoc — These 4 partials are the single-source content included by cloud-docs rpk-cloud-mcp* pages that cloud-docs FIPS Doc #562 deletes. Once that PR merges, these partials become orphans (not included anywhere) and also contain internal xrefs to ai-agents:mcp/remote/quickstart.adoc which will no longer exist.
    • Recommendation: Add a follow-up PR to this repo to delete these 4 orphan partials, OR extend this PR's scope to include them.

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

@micheleRP
Copy link
Copy Markdown
Contributor

actually, one thing: should you add the deprecation to the RPCN what's new?

@prakhargarg105
Copy link
Copy Markdown

lgtm!

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>
@JakeSCahill JakeSCahill merged commit 2e2a98b into main Apr 28, 2026
7 checks passed
@JakeSCahill JakeSCahill deleted the feature/remove-mcp-server-docs branch April 28, 2026 15:38
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.

3 participants