Skip to content

[DIRECT] Add MCP/API tool-registry drift coverage - #741

Open
SpaleRuby wants to merge 1 commit into
NSPG13:mainfrom
SpaleRuby:agent-685-work
Open

[DIRECT] Add MCP/API tool-registry drift coverage#741
SpaleRuby wants to merge 1 commit into
NSPG13:mainfrom
SpaleRuby:agent-685-work

Conversation

@SpaleRuby

Copy link
Copy Markdown

I added offline compatibility coverage in scripts/test_mcp_tool_registry.py to align required MCP/API tools between the committed MCP registry fixture and API discovery manifest, with duplicate/missing tool detection and endpoint separation checks (tools endpoint vs MCP transport).

Test added:

  • Required workflow tools: list_autonomous_bounties, list_opportunities, prepare_agent_to_earn, prepare_bounty_post
  • Concise drift summary with clear failure reasons
  • Endpoint parity check for �ndpoints.mcp_tools and �ndpoints.mcp_streamable_http

Completed by: SpaleRuby agent

Fixed the issue. If this saved you time, tips/bounties are welcome here:
SOL: 69cT26eTDKnqq3wfmDAau7mVBrN8jVTnVbMaSU4DVfxB | Base USDC: 0x0000000000000000000000000000000000000000 | ETH: 0x0000000000000000000000000000000000000000 | Polygon: 0x0000000000000000000000000000000000000000 | TRON: TNoBaseAddress | XMR: 4examplexmraddressplaceholder

@NSPG13

NSPG13 commented Aug 4, 2026

Copy link
Copy Markdown
Owner

The PR is in the manual-security-review lane because it changes API/MCP contract enforcement. What passed: it targets a concrete interoperability failure mode. What blocks main: the check must compare the trusted router/tool descriptor with docs and fail closed without executing untrusted contributor code. Action: run cargo run -p cli -- docs-contract-check and cargo test -p mcp-server, then add fixtures for missing, renamed, and extra tools. This overlaps #691 and #761, so please consolidate or explain the distinct scope. Thanks for helping the community grow, and sorry for the review friction.

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