Skip to content
This repository was archived by the owner on Nov 29, 2025. It is now read-only.

Conversation

@double16
Copy link

Fixes #49

  • MCP servers supported
  • stdio runs locally, still needs docker isolation
  • Tools are namespaced
  • Output is saved in files and only file references are returned for content > 4K
  • Changed from numeric toolLimit to list of allowed tools

Review needs

  • Is the tool prompt sufficient for the agent?
  • Is the list_mcp_tools full catalog sufficient for the agent?
  • UI, again, it's vibe coded because I lack experience with react

@double16 double16 changed the base branch from main to release-0.1.4 November 12, 2025 17:15
@double16
Copy link
Author

I'm seeing the agent often passing incorrect types to the tool.

Error executing tool port_scan: 2 validation errors for port_scanArguments\nip_addresses\n  Input should be a valid list [type=list_type, input_value='192.168.1.84', input_type=str]\n    For further information visit https://errors.pydantic.dev/2.11/v/list_type\nports\n  Input should be a valid list [type=list_type, input_value='4001', input_type=str]

Here is the tool context prompt:

## MCP TOOLS

Available general MCP tools:
- list_mcp_tools()  # full MCP tool catalog including input schema, output schema, description
- shyhurricane_directory_buster(url: str = None, depth: int = 3, method: str = 'GET', wordlist: Optional[str] = None, cookies: Optional[dict] = None, params: Optional[dict] = None, extensions: Optional[list[str]] = None, ignored_response_codes: Optional[list[int]] = None, additional_hosts: Optional[dict] = None, user_agent: Optional[str] = None, request_headers: Optional[dict] = None, timeout_seconds: Optional[int] = None)
- shyhurricane_port_scan(hostnames: Optional[list[str]] = None, ip_addresses: Optional[list[str]] = None, ip_subnets: Optional[list[str]] = None, ports: Optional[list[int]] = None, port_range_low: Optional[int] = None, port_range_high: Optional[int] = None, additional_hosts: Optional[dict] = None, timeout_seconds: Optional[int] = None, retry: bool = False)

@double16 double16 force-pushed the feature/operational-mcp-support-49-M2 branch from fa488c9 to 4eaaa42 Compare November 14, 2025 12:59
@westonbrown westonbrown added the enhancement New feature or request label Nov 15, 2025
@westonbrown westonbrown added this to the release 0.1.4 milestone Nov 15, 2025
@westonbrown
Copy link
Owner

Successfully integrated MCP support into release-0.1.4 after resolving config type ordering (MCPConnection forward reference) and logger import path conflicts from the config refactoring. Tested with HTB MCP server - agent successfully discovered 13 tools and executed list_events, retrieve_teams, and other MCP tools with outputs saved to artifacts.

@westonbrown
Copy link
Owner

Thank you @double16 for this excellent MCP integration work! Since you're unavailable and we needed this feature for release-0.1.4, we've manually integrated your changes with the necessary fixes for our refactored config system (commit 07e1219). The MCP support is now live and tested with HTB - closing this PR as completed.

@westonbrown
Copy link
Owner

Integrated manually into release-0.1.4 with config compatibility fixes

@double16 double16 deleted the feature/operational-mcp-support-49-M2 branch November 20, 2025 14:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature: Operational Plugin MCP Support (Toggleable, Configurable, Composable)

2 participants