Skip to content

billing: remove deprecated Remote MCP references#595

Merged
micheleRP merged 1 commit into
mainfrom
remove-remote-mcp
May 21, 2026
Merged

billing: remove deprecated Remote MCP references#595
micheleRP merged 1 commit into
mainfrom
remove-remote-mcp

Conversation

@micheleRP
Copy link
Copy Markdown
Contributor

@micheleRP micheleRP commented May 21, 2026

Description

Resolves: no linked issue — incidental cleanup of pages missed by #562 (Remove deprecated ADP content) and b4b87469 (Remove deprecated Remote MCP documentation).
Review deadline: none

Removes the == Remote MCP billing metrics section from the Billing page and strips incidental Remote MCP mentions from the Manage payment methods and View billing activity pages. Remote MCP was deprecated and removed from Redpanda Cloud; these three billing pages were missed in earlier cleanup PRs.

Page previews

Checks

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

Remote MCP has been removed from Redpanda Cloud (see #562, b4b8746).
Drop the Remote MCP billing section from the Billing page and remove
incidental Remote MCP mentions from Manage payment methods and View
billing activity.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@micheleRP micheleRP requested a review from a team as a code owner May 21, 2026 20:46
@netlify
Copy link
Copy Markdown

netlify Bot commented May 21, 2026

Deploy Preview for rp-cloud ready!

Name Link
🔨 Latest commit 79f3cf3
🔍 Latest deploy log https://app.netlify.com/projects/rp-cloud/deploys/6a0f6f3bfca34a00080d5b1e
😎 Deploy Preview https://deploy-preview-595--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.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 21, 2026

Review Change Stack

📝 Walkthrough

Walkthrough

This pull request updates billing documentation across three Asciidoc pages to modify Remote MCP references. The main billing page removes the entire "Remote MCP billing metrics" section, causing the "Support plans" section to follow directly after the "Redpanda Connect billing metrics" table. The payment methods page removes "Remote MCP servers" from the list of billable resources covered by credit cards. The billing activity page adds "Remote MCP server" to the described resources included in the per-resource cost breakdown.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • Feediver1
  • kbatuigas
  • towfiqa
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: removing deprecated Remote MCP references from billing 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.
Description check ✅ Passed The pull request description is complete and well-structured, following the template requirements with linked context, clear purpose, page previews, and appropriate category selection.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch remove-remote-mcp

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

Copy link
Copy Markdown
Contributor

@Feediver1 Feediver1 left a comment

Choose a reason for hiding this comment

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

PR Review: billing: remove deprecated Remote MCP references (#595)

Files reviewed: 3 .adoc files (2 additions / 21 deletions)
Overall assessment: Clean, minimal deletion-only cleanup. No issues. Safe to merge.

What this PR does

Removes residual mentions of the deprecated "Remote MCP" feature from three cloud-docs billing pages that earlier cleanup work (#562 and commit b4b87469) missed:

  • billing.adoc: drops the entire == Remote MCP billing metrics section (19 lines).
  • manage-payment-methods.adoc: strips ", Remote MCP servers" from one inline list.
  • view-billing-activity.adoc: strips ", Remote MCP server" from one inline list.

This brings the billing pages in line with the already-announced deprecation in whats-new-cloud.adoc (April 2026, "Remote MCP: Deprecated").

Jira ticket alignment

Ticket: None — PR body explicitly states "no linked issue — incidental cleanup of pages missed by #562 and b4b87469." Treated as housekeeping; no ticket needed.

Critical issues (must fix)

None.

Suggestions (should consider)

None of substance. The diff is correct, minimal, and removes only what should be removed.

Impact on other files

  • No incoming xrefs: grep across modules/ finds zero references to a #remote-mcp-billing-metrics anchor or "Remote MCP billing" content anywhere in cloud-docs. The removal won't break any cross-references.
  • No leftover "Remote MCP" mentions elsewhere in cloud-docs except the intentional deprecation announcement at modules/get-started/pages/whats-new-cloud.adoc:44–46. That's the correct place for it to stay — it's announcing the deprecation, not promoting the feature.
  • No nav.adoc update needed — no pages added/removed/renamed, just intra-page deletions.
  • No What's New update needed — the deprecation was already announced in the April 2026 section.
  • No cross-repo (single-source) riskmodules/billing/ is cloud-only, not single-sourced from redpanda-data/docs.

CodeRabbit findings worth considering

None. CodeRabbit's check passed with no review summary or comments to triage.

What works well

  • Clean Oxford-comma list rewrites. Both inline-list edits leave the surrounding sentence well-formed — "Serverless, Dedicated, and BYOC clusters, Redpanda Connect pipelines, and your support plan" reads naturally with no orphan commas.
  • Scope is tight. The PR removes only Remote MCP references in billing/, doesn't touch the intentional deprecation announcement, doesn't try to fold in other unrelated cleanups.
  • PR body is informative. Cross-references the prior cleanup PR (#562) and commit (b4b87469) so reviewers (and future archaeologists) have the deprecation provenance.
  • CI is green and the diff is small enough that the Netlify preview links in the PR body cover all three changed pages.

Final-pass review via /docs-team-standards:pr-review.

Copy link
Copy Markdown
Contributor

@Feediver1 Feediver1 left a comment

Choose a reason for hiding this comment

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

Claude likes it!

@micheleRP micheleRP merged commit 728110b into main May 21, 2026
5 checks passed
@micheleRP micheleRP deleted the remove-remote-mcp branch May 21, 2026 21:47
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