Skip to content

chore(cli): remove mcp-replaced modules — gh/jira/sentry/slack/pd/eks/pipeline#2

Merged
aladac merged 1 commit into
masterfrom
chore/drop-mcp-replaced
May 24, 2026
Merged

chore(cli): remove mcp-replaced modules — gh/jira/sentry/slack/pd/eks/pipeline#2
aladac merged 1 commit into
masterfrom
chore/drop-mcp-replaced

Conversation

@aladac
Copy link
Copy Markdown
Member

@aladac aladac commented May 24, 2026

Summary

Removes 7 modules that have stable MCP server replacements, significantly slimming the hu binary.

Removed modules → MCP replacements

Module Replacement
gh github/github-mcp-server
jira sooperset/mcp-atlassian (5.2k★)
sentry getsentry/sentry-mcp (702★)
slack zencoderai/slack-mcp-server
pagerduty PagerDuty/pagerduty-mcp-server
eks rohitg00/kubectl-mcp-server
pipeline aws/agent-toolkit-for-aws

Kept

newrelic (no viable MCP alternative — top community option: 13★), plus all unique-value modules: context, read, data, docs, cron, shell, mcp, setup, utils.

Dead deps pruned

octocrab, oauth2, axum, open, base64, rand — removed from Cargo.toml. urlencoding and async-trait kept (still used by utils/web_search and util/shell / setup).

Test results

  • 1318 unit tests ✅
  • 13 integration tests ✅
  • 0 warnings in binary build

Safety

legacy branch pinned at d2d4c5f (pre-removal HEAD) for recovery if needed.

…/pipeline

Seven modules have stable MCP server replacements and are removed:
  gh        → github/github-mcp-server
  jira      → sooperset/mcp-atlassian
  sentry    → getsentry/sentry-mcp
  slack     → zencoderai/slack-mcp-server
  pagerduty → PagerDuty/pagerduty-mcp-server
  eks       → rohitg00/kubectl-mcp-server
  pipeline  → aws/agent-toolkit-for-aws

Dead deps also removed: octocrab, oauth2, axum, open, base64, rand.
urlencoding and async-trait kept (still used by utils/web_search and
util/shell + setup).

Credentials structs (GithubCredentials, JiraCredentials) kept in
util/config for backward compat with existing credentials.toml files on
disk; save_credentials / save_credentials_to kept for config tests.
Re-exports of those types removed from util/mod.rs; dead_code lints
suppressed on the write helpers (exercised in tests only).

All 1318 unit tests and 13 integration tests pass.
@aladac aladac merged commit f009a18 into master May 24, 2026
0 of 3 checks passed
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