[PR #21] [MERGED] Restore clean MCP setup (revert experimental changes) #25

Closed
opened 2026-02-27 13:59:33 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/GH05TCREW/pentestagent/pull/21
Author: @giveen
Created: 1/21/2026
Status: Merged
Merged: 1/22/2026
Merged by: @GH05TCREW

Base: mainHead: mcp-cleanup


📝 Commits (10+)

  • c737453 chore(mcp): add visibility note for mcp-cleanup branch
  • c5c6fee chore(mcp): disable MCP auto-start and auto-connect; document manual install
  • f3f3b09 mcp: add daemon mode + disconnect; harden StdioTransport cleanup
  • 080a32a mcp: add unix socket control + tools call/list via daemon
  • 84203c0 mcp: auto-connect when configured; add socket control and tools call
  • d29e1f7 tui: instantiate MCPManager at startup to auto-connect and register MCP tools
  • 1a54dea mcp: allow background connect without registering tools; TUI starts background connect(register=False)
  • d6c5d8e tui: register MCP tools quietly at startup (register=True, quiet=True)
  • 30e2476 chore(mcp): remove vendored HexStrike files
  • db06dcf chore(mcp): remove hexstrike/metasploit adapters and helper scripts

📊 Changes

37 files changed (+741 additions, -25400 deletions)

View changed files

📝 .env.example (+9 -15)
MCP-CLEANUP-NOTE.md (+15 -0)
📝 README.md (+8 -4)
dupe-workspace.tar.gz (+0 -0)
expimp-workspace.tar.gz (+0 -0)
📝 pentestagent/interface/cli.py (+4 -15)
📝 pentestagent/interface/main.py (+373 -2)
📝 pentestagent/interface/tui.py (+23 -15)
pentestagent/mcp/example_adapter.py (+83 -0)
pentestagent/mcp/hexstrike_adapter.py (+0 -338)
📝 pentestagent/mcp/manager.py (+4 -315)
📝 pentestagent/mcp/mcp_servers.json (+1 -27)
pentestagent/mcp/metasploit_adapter.py (+0 -414)
pentestagent/mcp/stdio_adapter.py (+181 -0)
📝 pentestagent/mcp/transport.py (+5 -287)
📝 pentestagent/workspaces/utils.py (+1 -1)
requirements-hexstrike.txt (+0 -3)
scripts/add_hexstrike_subtree.sh (+0 -32)
scripts/add_metasploit_subtree.sh (+0 -84)
scripts/install_hexstrike_deps.ps1 (+0 -45)

...and 17 more files

📄 Description

I wanted to apologize for the earlier MCP changes I pushed in.

After working with them a bit more, I wasn’t happy with the direction or the amount of noise they introduced into the project.

Rather than trying to patch around it, I decided it was cleaner and safer to roll the MCP setup back to commit 49c0b93.

This PR restores the original structure from that commit and removes the experimental MCP additions. From here, we can re‑introduce MCP support in a more controlled and incremental way if we decide to bring it back.

Thanks for your patience while I cleaned this up.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/GH05TCREW/pentestagent/pull/21 **Author:** [@giveen](https://github.com/giveen) **Created:** 1/21/2026 **Status:** ✅ Merged **Merged:** 1/22/2026 **Merged by:** [@GH05TCREW](https://github.com/GH05TCREW) **Base:** `main` ← **Head:** `mcp-cleanup` --- ### 📝 Commits (10+) - [`c737453`](https://github.com/GH05TCREW/pentestagent/commit/c73745304f855fde9f73bb91b7c2f48184638648) chore(mcp): add visibility note for mcp-cleanup branch - [`c5c6fee`](https://github.com/GH05TCREW/pentestagent/commit/c5c6fee8da1e58559dbb12242b10a26921814ad3) chore(mcp): disable MCP auto-start and auto-connect; document manual install - [`f3f3b09`](https://github.com/GH05TCREW/pentestagent/commit/f3f3b0956bc413fb5316ffe6804bd611d4b12443) mcp: add daemon mode + disconnect; harden StdioTransport cleanup - [`080a32a`](https://github.com/GH05TCREW/pentestagent/commit/080a32a8fa081e0f4bff51209cdd31f2ed7d32ab) mcp: add unix socket control + tools call/list via daemon - [`84203c0`](https://github.com/GH05TCREW/pentestagent/commit/84203c0435049be6662b75056308c1c6e2367c1b) mcp: auto-connect when configured; add socket control and tools call - [`d29e1f7`](https://github.com/GH05TCREW/pentestagent/commit/d29e1f78e6f271b5fbf9d256a534411e3d37eed4) tui: instantiate MCPManager at startup to auto-connect and register MCP tools - [`1a54dea`](https://github.com/GH05TCREW/pentestagent/commit/1a54dea061bc26b8b6147cee445644be159de1ed) mcp: allow background connect without registering tools; TUI starts background connect(register=False) - [`d6c5d8e`](https://github.com/GH05TCREW/pentestagent/commit/d6c5d8ebd2e8feb0cfb9e084b415a6cd38b0dbed) tui: register MCP tools quietly at startup (register=True, quiet=True) - [`30e2476`](https://github.com/GH05TCREW/pentestagent/commit/30e2476d01644c960c258c6de9c5f33138e52a0c) chore(mcp): remove vendored HexStrike files - [`db06dcf`](https://github.com/GH05TCREW/pentestagent/commit/db06dcfa63ac6ed31761c86862a5f4a1044959e1) chore(mcp): remove hexstrike/metasploit adapters and helper scripts ### 📊 Changes **37 files changed** (+741 additions, -25400 deletions) <details> <summary>View changed files</summary> 📝 `.env.example` (+9 -15) ➕ `MCP-CLEANUP-NOTE.md` (+15 -0) 📝 `README.md` (+8 -4) ➖ `dupe-workspace.tar.gz` (+0 -0) ➖ `expimp-workspace.tar.gz` (+0 -0) 📝 `pentestagent/interface/cli.py` (+4 -15) 📝 `pentestagent/interface/main.py` (+373 -2) 📝 `pentestagent/interface/tui.py` (+23 -15) ➕ `pentestagent/mcp/example_adapter.py` (+83 -0) ➖ `pentestagent/mcp/hexstrike_adapter.py` (+0 -338) 📝 `pentestagent/mcp/manager.py` (+4 -315) 📝 `pentestagent/mcp/mcp_servers.json` (+1 -27) ➖ `pentestagent/mcp/metasploit_adapter.py` (+0 -414) ➕ `pentestagent/mcp/stdio_adapter.py` (+181 -0) 📝 `pentestagent/mcp/transport.py` (+5 -287) 📝 `pentestagent/workspaces/utils.py` (+1 -1) ➖ `requirements-hexstrike.txt` (+0 -3) ➖ `scripts/add_hexstrike_subtree.sh` (+0 -32) ➖ `scripts/add_metasploit_subtree.sh` (+0 -84) ➖ `scripts/install_hexstrike_deps.ps1` (+0 -45) _...and 17 more files_ </details> ### 📄 Description I wanted to apologize for the earlier MCP changes I pushed in. After working with them a bit more, I wasn’t happy with the direction or the amount of noise they introduced into the project. Rather than trying to patch around it, I decided it was cleaner and safer to roll the MCP setup back to commit 49c0b93. This PR restores the original structure from that commit and removes the experimental MCP additions. From here, we can re‑introduce MCP support in a more controlled and incremental way if we decide to bring it back. Thanks for your patience while I cleaned this up. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-27 13:59:33 -05:00
yindo closed this issue 2026-02-27 13:59:36 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: yindo/pentestagent#25