SREP-4769 chore: Remove backplane MCP subcommand

[samanthajayasinghe]

SREP-4769 feat: Remove backplane MCP subcommand

What type of PR is this?

• fix (Bug Fix)
• deprecate
• docs (Documentation)
• test (Test Coverage)
• chore (Clean Up / Maintenance Tasks)
• other (Anything that doesn't fit the above)

What this PR does / Why we need it?

Removing backplane MCP subcommand in favour of backplane skills

Which Jira/Github issue(s) does this PR fix?

• Related Issue #
• Closes # https://redhat.atlassian.net/browse/SREP-4769

Special notes for your reviewer

Unit Test Coverage

Guidelines

• If it's a new sub-command or new function to an existing sub-command, please cover at least 50% of the code
• If it's a bug fix for an existing sub-command, please cover 70% of the code

Test coverage checks

• Added unit tests
• Created jira card to add unit test
• This PR may not need unit tests

Pre-checks (if applicable)

• Ran unit tests locally
• Validated the changes in a cluster
• Included documentation changes with PR
• Backward compatible

/label tide/merge-method-squash

Summary by CodeRabbit

• Features Removed

  • Removed MCP (Model Context Protocol) server functionality and the mcp command.
  • Removed associated tools for cloud console, cluster resource queries, console management, login, and system information previously available through the MCP server.

• Chores

  • Removed MCP-related dependencies from the project.

[coderabbitai]

Walkthrough

This PR removes the MCP (Model Context Protocol) feature: the mcp CLI command, all MCP tool implementations and their tests, MCP wiring in the root command, and related go.mod dependencies.

Changes

MCP Feature Removal

Layer / File(s)	Summary
Command Setup cmd/ocm-backplane/mcp/mcp.go	Deleted exported MCPCmd Cobra command, its flags (--http, --port), and runMCP implementation (MCP server creation, tool registration, HTTP/stdio transport selection).
Root Command Wiring cmd/ocm-backplane/root.go	Removed mcp package import and rootCmd.AddCommand(mcp.MCPCmd) registration.
Tool Implementations pkg/ai/mcp/backplane_info.go, pkg/ai/mcp/backplane_login.go, pkg/ai/mcp/backplane_console.go, pkg/ai/mcp/backplane_cluster_resource.go, pkg/ai/mcp/backplane_cloud_console.go	Deleted five MCP tool implementations and their input types and exported handler functions (info, login, console, cluster-resource, cloud-console).
Tests / Suites cmd/ocm-backplane/mcp/*_test.go, pkg/ai/mcp/*_test.go	Removed all MCP-related test files: command tests, per-tool unit tests, suite bootstraps, and integration tests.
Dependency Cleanup go.mod	Removed direct dependency github.com/modelcontextprotocol/go-sdk v1.6.0 and several indirect requirements (github.com/google/jsonschema-go v0.4.3, github.com/segmentio/asm v1.1.3, github.com/segmentio/encoding v0.5.4, github.com/yosida95/uritemplate/v3 v3.0.2).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The PR title 'SREP-4769 feat: Remove backplane MCP subcommand' directly and clearly describes the main change: removing the MCP subcommand from backplane.
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	Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

• Generate code and open pull requests
• Plan features and break down work
• Investigate incidents and troubleshoot customer tickets together
• Automate recurring tasks and respond to alerts with triggers
• Summarize progress and report instantly

Built for teams:

• Shared memory across your entire org—no repeating context
• Per-thread sandboxes to safely plan and execute work
• Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started

---

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

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 54.00%. Comparing base (a6e43bd) to head (44fffb2).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #936      +/-   ##
==========================================
+ Coverage   53.97%   54.00%   +0.02%     
==========================================
  Files          88       82       -6     
  Lines        6662     6307     -355     
==========================================
- Hits         3596     3406     -190     
+ Misses       2597     2455     -142     
+ Partials      469      446      -23     

Files with missing lines	Coverage Δ
cmd/ocm-backplane/root.go	82.60% <ø> (-0.73%)	⬇️

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[MitaliBhalla]

/lgtm

[openshift-merge-bot]

/retest-required

Remaining retests: 0 against base HEAD a6e43bd and 2 for PR HEAD 380cb92 in total

[openshift-ci]

@samanthajayasinghe: all tests passed!

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

[xiaoyu74]

Thanks Sam.

small nit: since this removes an pilot/experimental backplane MCP wrapper rather than adding new functionality, chore may fit better than feat in the PR title for categorizing release note?

[xiaoyu74]

/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: MitaliBhalla, samanthajayasinghe, xiaoyu74

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [MitaliBhalla,samanthajayasinghe,xiaoyu74]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment