Advertise MCP proof and ledger schemas

[elianguitarra]

Summary

• add explicit MCP input schemas for get_proof and get_ledger_entry
• document supported proof hash and ledger sequence arguments for agent/tool clients
• add conformance coverage that verifies both schemas are advertised

Validation

• .\.venv\Scripts\python.exe -m pytest tests\test_api_mcp.py tests\test_mcp_tools.py -> 131 passed, 1 warning
• .\.venv\Scripts\ruff.exe check app\mcp.py tests\test_api_mcp.py -> All checks passed
• .\.venv\Scripts\ruff.exe format --check app\mcp.py tests\test_api_mcp.py -> 2 files already formatted
• git diff --check origin/main..HEAD -> clean

Scope

• Bounty MRWK bounty: 150 MRWK - MCP structured output and conformance suite #946 MCP structured output/conformance surface only.
• No MCP tool runtime behavior, ledger mutation, payout execution, treasury/proposal execution, wallet behavior, bridge/exchange/cash-out behavior, MRWK price behavior, private data, or secrets changed.

Summary by CodeRabbit

• Improvements

  • Enhanced input validation for ledger entry and proof lookup tools with stricter parameter constraints, preventing invalid requests and improving data reliability.

• Tests

  • Added comprehensive validation tests to verify tools enforce proper input constraints.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: d075c9e6-cfc5-45fe-8c7c-d1bc55447a43

📥 Commits

Reviewing files that changed from the base of the PR and between 8bdd282 and 6b396dd.

📒 Files selected for processing (2)

• app/mcp.py
• tests/test_api_mcp.py

---

📝 Walkthrough

Walkthrough

This PR adds JSON input schema validation to two MCP tools: get_ledger_entry now requires an integer sequence parameter with a minimum value of 1, and get_proof now requires a hash string parameter constrained to exactly 64 lowercase hexadecimal characters. Test assertions were extended to validate both schema structures and constraint properties.

Changes

MCP Tool Input Schemas

Layer / File(s)	Summary
Tool input schemas and validations app/mcp.py, tests/test_api_mcp.py	get_ledger_entry and get_proof MCP tool definitions replaced with schema-validated input formats; sequence integer and hash 64-hex-character string are now required with explicit constraint rules. Test assertions extended to validate schema structure, required fields, type constraints, and pattern/length properties for both tools.

Possibly related PRs

• ramimbo/mergework#937: Both PRs update app/mcp.py's MCP_TOOLS to add/expand MCP inputSchema validation for tool arguments and extend tests/test_api_mcp.py's tools/list assertions accordingly.
• ramimbo/mergework#526: Both PRs modify the MCP get_proof tool contract: main PR adds strict input JSON schema validation for the hash argument, while retrieved PR changes get_proof handling of malformed stored Proof.public_json.
• ramimbo/mergework#926: Both PRs extend app/mcp.py's MCP_TOOLS tool entries with JSON inputSchema and update tests/test_api_mcp.py's tools/list assertions to validate those schemas.

---

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

[elianguitarra]

Closing this unclaimed PR to avoid overlapping with the already-open MCP proof/ledger schema surface in #974. I will look for a non-duplicate #946 surface instead.