MRWK bounty: 40 MRWK - review open MergeWork PRs with evidence, round 20

[ramimbo]

MergeWork status: live bounty

Reserved on MergeWork: https://mrwk.online/bounties/114

Claims are open only while the public bounty row remains open and effective award capacity remains. Use the public bounty page or API as the authoritative current status.

MRWK Bounty

Reward: 40 MRWK per accepted award
Max awards: 30

Work Needed

Review open MergeWork pull requests with concrete current-head evidence. This round is proposed early so review capacity is ready before the current review rounds empty.

Priority Targets

• Clean, mergeable contributor PRs that are close to acceptance.
• PRs that affect treasury, bounty lifecycle, admin flow, deployment, documentation, MCP behavior, public bounty discovery, or queue safety.
• Blocker reviews on stale-looking, dirty, or workflow-gated PRs where the review gives maintainers a clear close, fix, approve-workflow, or rebase decision.

Acceptance Criteria

Accepted reviews must:

• be by someone other than the PR author;
• link a GitHub PR review or concise PR comment;
• review the current head, or clearly state the reviewed head SHA;
• include concrete evidence such as files inspected, behavior checked, commands run, CI reviewed, mergeability checked, duplicate or overlap checked, or a specific actionable finding;
• avoid treating proposed, pending, closed, exhausted, or unsupported bounty references as claimable.

How To Submit

Comment here with the PR review URL or PR comment URL and include Bounty #<this issue number> or Refs #<this issue number>.

Evidence or Tests Required

Use a short review packet:

• Reviewed PR:
• Head commit:
• Files inspected:
• Verdict:
• Validation:

Out of Scope

• Rubber-stamp approvals, vague comments, self-reviews, stale-head reviews that do not account for current code, or duplicate reviews that add no new evidence.
• Reviews that call bad work merge-ready while missing a central behavior problem.
• Reviews containing private secrets, private vulnerability details, fabricated payout claims, price claims, investment claims, exchange/liquidity claims, or bridge/cash-out claims.

Duplicate and Stale Work Rules

Duplicate work is judged by the first useful, evidence-backed review for the same PR head and core finding. Stale reviews may be skipped if the PR head changes materially before acceptance. Reviews may identify blockers and still qualify if they give maintainers concrete action.

[ramimbo]

Status: pending create_bounty proposal. This issue is not claimable yet.

Proposal: https://api.mrwk.online/api/v1/treasury/proposals/149
Earliest execution: 2026-06-06T09:28:29.201633Z

Wait until proposal execution creates the public bounty row and finalizes this GitHub issue with mrwk:bounty and the claims-open comment before treating this as live bounty work.

[ramimbo]

Reserved on MergeWork: https://mrwk.online/bounties/114

Claims are now open for accepted work that matches this bounty's criteria.

[elianguitarra]

Bounty #933 review submission

• Reviewed PR: Reject unsupported work discovery query parameters #949
• Review URL: Reject unsupported work discovery query parameters #949 (review)
• Head commit: 05e139f
• Files inspected: app/bounty_api.py, app/query_validation.py, tests/test_query_validation.py, tests/test_work_discovery.py
• Verdict: useful change, but current-head rebase/conflict resolution is required before maintainers can merge it cleanly.
• Validation: local merge-tree against current origin/main reports content conflict in app/bounty_api.py; focused PR-head pytest checks passed: tests/test_work_discovery.py::test_work_discovery_rejects_unsupported_filter_query_params and tests/test_query_validation.py::test_unknown_query_guard_rejects_unsupported_names -> 2 passed.

[elianguitarra]

Bounty #933 review submission

• Reviewed PR: Add MCP schemas for proof and ledger lookups #974
• Review URL: Add MCP schemas for proof and ledger lookups #974 (review)
• Head commit: 0adcbc3
• Files inspected: app/mcp.py, tests/test_api_mcp.py, docs/agent-guide.md
• Verdict: useful and merge-ready from the reviewed surfaces. The PR adds missing tools/list schemas for get_ledger_entry and get_proof without changing runtime behavior; advertised constraints match current dispatcher/path validation expectations.
• Validation: .venv\Scripts\python.exe -m pytest tests/test_api_mcp.py::test_mcp_tools_list_and_call tests/test_api_mcp.py::test_mcp_get_ledger_entry_includes_payment_proof_hash tests/test_api_mcp.py::test_mcp_get_ledger_entry_rejects_non_positive_sequence tests/test_api_mcp.py::test_mcp_get_proof_returns_public_proof_details tests/test_api_mcp.py::test_mcp_get_proof_reports_unknown_hash tests/test_api_mcp.py::test_mcp_get_proof_rejects_malformed_hash -> 6 passed, 1 existing Starlette/httpx warning; git diff --check 8bdd2823340d9704298e4829f42f53558aa83133..HEAD -> clean.
• Mergeability/overlap checked: GitHub reports Add MCP schemas for proof and ledger lookups #974 open, non-draft, mergeable; no human reviews were present before this review; CodeRabbit generated no actionable comments.

[aosoficial]

Bounty #933 review submission

• Reviewed PR: Align MCP bounty sort schema with runtime #976
• Review URL: Align MCP bounty sort schema with runtime #976 (review)
• Head commit: 9c8af4befe1f5685b586df14131102f3c613e0af
• Files inspected: app/mcp.py, tests/test_api_mcp.py, plus runtime context in app/bounty_sorting.py / bounty API sort routing.
• Verdict: approved. The PR aligns the MCP list_bounties advertised sort enum with the runtime-supported newest, reward, available, and awards modes, with focused schema regression coverage and no runtime behavior change.

Validation:

• uv run --python 3.12 --extra dev python -m pytest tests/test_api_mcp.py::test_mcp_tools_list_and_call tests/test_api_mcp.py::test_mcp_list_bounties_honors_sort_argument -q -> 2 passed, 1 existing Starlette/httpx warning.
• uv run --python 3.12 --extra dev ruff check app/mcp.py tests/test_api_mcp.py -> passed.
• uv run --python 3.12 --extra dev ruff format --check app/mcp.py tests/test_api_mcp.py -> 2 files already formatted.
• uv run --python 3.12 --extra dev mypy app/mcp.py -> success.
• uv run --python 3.12 --extra dev python scripts/docs_smoke.py -> docs smoke ok.
• git diff --check origin/main...HEAD -> clean.
• git merge-tree --write-tree origin/main HEAD -> clean tree b3e8f0387fa96e644496ef1e9826eb9f78f0d2eb.

GitHub state before review: open, non-draft, mergeable, CodeRabbit success, and no human reviews on current head. Scope was read-only PR review of MCP schema/test changes; no ledger, wallet, treasury, payout, admin, price, bridge, exchange, cash-out, secrets, or private data used.