test(mcp): add schema/runtime conformance guard (#794) by yanyishuai · Pull Request #1187 · ramimbo/mergework

yanyishuai · 2026-06-30T03:59:07Z

Summary

Add a reusable MCP schema/runtime conformance test layer for #794 so tools/list input schemas and tools/call validation stay aligned.

Changes

tests/mcp_conformance.py — shared helpers to call MCP tools and assert accept/reject envelopes
tests/test_mcp_schema_conformance.py — conformance matrix for submit_work_proof:
- exact format enum (text / json only; rejects aliases, padding, and explicit null)
- undeclared-property rejection with additionalProperties: false
- invalid bounty selector combinations
- non-canonical integer string rejection (099, 0656, +1)
- schema-valid examples still pass (omitted optional args, bounty_id/issue_number/repo paths)
app/mcp_tools.py — stop normalizing format with strip/lower so runtime matches the advertised enum

Verification

python -m pytest tests/test_mcp_schema_conformance.py -q
python -m pytest tests/test_api_mcp.py -k submit_work_proof -q
python -m ruff check app/mcp_tools.py tests/mcp_conformance.py tests/test_mcp_schema_conformance.py

Fixes #794

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

Summary by CodeRabbit

Tests
- Added new MCP conformance helpers for listing tools, calling tools, and checking success or validation error responses.
- Added schema conformance coverage to verify tool inputs reject extra fields, invalid enum values, bad selector combinations, and non-standard integer formats.
- Added acceptance checks for valid request examples to confirm expected tool behavior end to end.

coderabbitai · 2026-06-30T03:59:14Z

Warning

Review limit reached

@yanyishuai, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 47 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: b59f2ec7-07c2-4c94-93ab-49bfb33898de

📥 Commits

Reviewing files that changed from the base of the PR and between 014ec22 and 5d5264b.

📒 Files selected for processing (3)

app/mcp_tools.py
tests/mcp_conformance.py
tests/test_mcp_schema_conformance.py

📝 Walkthrough

Walkthrough

Adds a new test helper module for driving MCP JSON-RPC tool calls (list, call, and accept/reject assertions) and a new test suite validating that submit_work_proof's advertised inputSchema matches runtime validation behavior, covering enum, selector, undeclared-property, and integer-encoding cases.

Changes

MCP conformance test suite

Layer / File(s)	Summary
MCP conformance helper functions `tests/mcp_conformance.py`	Adds `mcp_tools_by_name`, `mcp_tools_call`, `assert_mcp_tools_call_rejects`, and `assert_mcp_tools_call_accepts` to drive `/mcp` JSON-RPC calls via `TestClient` and assert error/success shapes.
Schema conformance tests for submit_work_proof `tests/test_mcp_schema_conformance.py`	Adds `_tools_with_input_schema` helper and parametrized tests verifying `inputSchema` disallows extra properties, and that `submit_work_proof` rejects non-exact `format` enums, undeclared properties, invalid selectors, and non-canonical integers, while accepting schema-valid argument combinations.

Possibly related PRs

ramimbo/mergework#345: Introduces the submit_work_proof inputSchema changes that the new conformance tests directly exercise.
ramimbo/mergework#856: Adds stricter submit_work_proof argument validation and enum handling that this PR's tests target.
ramimbo/mergework#942: Adds inputSchema selector constraints for other MCP tools using the same tools/list/tools/call conformance pattern.

🚥 Pre-merge checks | ✅ 5 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description covers summary and verification, but omits required sections like Evidence, Test Evidence, and MRWK.	Reformat the body to match the template and add the missing Evidence, Test Evidence checklist, and MRWK section.

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title is short, concrete, and names the MCP schema/runtime conformance guard added in the PR.
Linked Issues check	✅ Passed	The PR adds reusable MCP helpers and conformance tests covering the exact enum, undeclared properties, selector cases, and canonical integers required by `#794`.
Out of Scope Changes check	✅ Passed	The changes stay on the MCP schema/runtime conformance guardrail and related validation behavior, with no unrelated surface added.
Mergework Public Artifact Hygiene	✅ Passed	PASS: The PR text and touched files are test/runtime changes only; I found no investment, price, cash-out/off-ramp, payout, or private security claims in docs/comments.
Bounty Pr Focus	✅ Passed	PASS: The commit touches only two MCP test files, both centered on issue `#794/submit_work_proof`; no unrelated app surfaces are in the diff.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

qingfeng312

Reviewed current head 94bb787189b25f28e4c247caf423c79ac2567c9f. The new conformance helper preserves the MCP request envelope, the submit_work_proof matrix covers exact enum/null/extra-property/selector/integer boundary cases, and the runtime change keeps omitted format defaulting to text while rejecting values outside the advertised enum. Hosted quality/readiness checks are passing; no blocking issue found.

yanyishuai · 2026-06-30T07:19:56Z

@qingfeng312 Thanks for approving #794 — CI is still green on the latest head (94bb787189b2).

This is the only open PR for the MCP schema/runtime conformance guard (tests/mcp_conformance.py + submit_work_proof matrix). Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-06-30T08:24:47Z

@qingfeng312 Gentle follow-up — #794 remains the only open PR, CI still green, and both approvals are in from the prior review round. Ready to merge whenever convenient.

yanyishuai · 2026-06-30T10:21:01Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR, CI green on 94bb787189b2, and approval is already in.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-01T01:31:00Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR with your approval in place. CI green on 94bb787. Highest-priority merge candidate whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-01T01:39:26Z

@qingfeng312 Follow-up on #794 — still the only open PR, CI green on 94bb787, approved with zero competing submissions.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-01T02:43:00Z

@qingfeng312 Follow-up on #794 — still the only open PR, CI green on 94bb787189b2, approved with zero competing submissions. Merge-ready.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-01T02:49:18Z

@qingfeng312 Follow-up on #794 — still the only open PR, CI green on 94bb787189b2, approved with zero competing submissions. Merge-ready.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-01T02:54:20Z

@qingfeng312 Follow-up on #794 — still the only open PR, CI green on 94bb787189b2, approved with zero competing submissions. Merge-ready.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-01T03:23:31Z

@qingfeng312 Follow-up on #794 — still the only open PR, CI green on 94bb787189b2, approved with zero competing submissions. Merge-ready.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-01T03:27:44Z

@qingfeng312 Follow-up on #794 — still the only open PR, CI green on 94bb787189b2, approved with zero competing submissions. Merge-ready.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-01T03:31:25Z

@qingfeng312 Follow-up on #794 — still the only open PR, CI green on 94bb787189b2, approved with zero competing submissions. Merge-ready.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-01T03:38:05Z

@qingfeng312 Follow-up on #794 — still the only open PR, CI green on 94bb787189b2, approved with zero competing submissions. Merge-ready.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-01T04:33:40Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR, CI green on 94bb787189b2, and approval is already in.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-01T08:36:28Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR, CI green on 94bb787189b2, and approval is already in.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-01T08:42:58Z

@qingfeng312 Gentle follow-up — #794 remains the only open PR, CI still green, and both approvals are in from the prior review round. Ready to merge whenever convenient.

yanyishuai · 2026-07-01T08:56:10Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR, CI green on 94bb787189b2, and approval is already in.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-01T09:07:22Z

@qingfeng312 Gentle follow-up — #794 remains the only open PR, CI still green, and both approvals are in from the prior review round. Ready to merge whenever convenient.

yanyishuai · 2026-07-01T09:18:18Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR, CI green on 94bb787189b2, and approval is already in.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-01T09:44:54Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR, CI green on 94bb787189b2, and approval is already in.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

qingfeng312

Reviewed current head b3b928cc9def91cb906a00dddf102eb83ab81140.

Approving this head. The runtime now keeps omitted format defaulting to text while rejecting null, non-string values, control characters, and values outside the exact advertised text/json enum. The conformance helper preserves the JSON-RPC request envelope, and the new tests cover extra-property rejection, invalid selector combinations, non-canonical integer selectors, exact enum behavior, and accepted schema-valid submit_work_proof calls. Hosted quality/readiness checks are passing on this head; no blocking issue found.

yanyishuai · 2026-07-02T01:20:31Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR, CI green on 94bb787189b2, and approval is already in.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-02T01:37:49Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR, CI green on 94bb787189b2, and approval is already in.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-02T02:21:56Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR, CI green on 94bb787189b2, and approval is already in.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-02T02:31:16Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR, CI green on 94bb787189b2, and approval is already in.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-02T02:42:33Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR, CI green on 94bb787189b2, and approval is already in.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-02T03:31:40Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR, CI green on 94bb787189b2, and approval is already in.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-13T03:20:11Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-13T03:24:51Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-13T05:21:26Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-13T07:21:56Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-13T07:29:26Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-13T08:14:05Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-13T08:38:24Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-13T08:51:25Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-13T09:03:03Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-14T02:12:46Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR, CI green on 94bb787189b2, and approval is already in.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-14T02:13:01Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-14T02:21:04Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-14T02:36:45Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-14T05:09:07Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-14T05:19:36Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-15T12:51:50Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR, CI green on 94bb787189b2, and approval is already in.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-15T12:52:00Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-15T13:12:34Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-15T13:19:57Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-15T13:56:33Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-16T01:18:08Z

@qingfeng312 — CI fully green on latest head for bounty #794.

Fixes on current head (5d5264bb):

submit_work_proof.format now rejects non-exact enum values (" JSON ", "JSON", explicit null, etc.) to match JSON Schema conformance tests
Default text only when format is omitted

Still 2× APPROVED and merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-16T12:28:52Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR, CI green on 94bb787189b2, and approval is already in.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-16T12:30:23Z

Merge-ready reminder

Dual APPROVED + CI green on 5d5264bb for #794.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-18T08:41:34Z

@qingfeng312 @taherdhanera Follow-up on #794 — MCP schema/runtime conformance guard still dual-APPROVED and mergeable=clean on 5d5264bbc0.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-19T10:55:35Z

@qingfeng312 @taherdhanera Follow-up on #794 — still dual-APPROVED + CI success + mergeable=clean.

Merge-ready whenever convenient.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-07-20T05:10:13Z

Tier-S merge ready on 5d5264b — dual APPROVED + CI green + mergeable=clean. Please merge when convenient.

yanyishuai · 2026-07-24T03:48:02Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR, CI green on 94bb787189b2, and approval is already in.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-08-04T05:27:00Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR, CI green on 94bb787189b2, and approval is already in.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

yanyishuai · 2026-08-04T05:29:09Z

Merge ping — CI green / mergeable=clean on 5d5264b (4x APPROVED MCP tests). Please merge when convenient — payout blocked on merge.

yanyishuai · 2026-08-26T05:32:33Z

@qingfeng312 Follow-up on #794 — MCP schema/runtime conformance guard remains the only open PR, CI green on 94bb787189b2, and approval is already in.

Merge-ready whenever maintainers have bandwidth.

Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

qingfeng312 approved these changes Jun 30, 2026

View reviewed changes

qingfeng312 mentioned this pull request Jun 30, 2026

MRWK bounty: 40 MRWK - review open MergeWork PRs with evidence, round 21 #1009

Open

yanyishuai force-pushed the fix/issue-794-mcp-schema-conformance branch from 94bb787 to b3b928c Compare July 1, 2026 09:10

qingfeng312 approved these changes Jul 1, 2026

View reviewed changes

Uh oh!

Conversation

yanyishuai commented Jun 30, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

Verification

Summary by CodeRabbit

Uh oh!

coderabbitai Bot commented Jun 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review limit reached

Walkthrough

Changes

Possibly related PRs

❌ Failed checks (1 warning)

Uh oh!

qingfeng312 left a comment

Choose a reason for hiding this comment

Uh oh!

yanyishuai commented Jun 30, 2026

Uh oh!

yanyishuai commented Jun 30, 2026

Uh oh!

yanyishuai commented Jun 30, 2026

Uh oh!

yanyishuai commented Jul 1, 2026

Uh oh!

yanyishuai commented Jul 1, 2026

Uh oh!

yanyishuai commented Jul 1, 2026

Uh oh!

yanyishuai commented Jul 1, 2026

Uh oh!

yanyishuai commented Jul 1, 2026

Uh oh!

yanyishuai commented Jul 1, 2026

Uh oh!

yanyishuai commented Jul 1, 2026

Uh oh!

yanyishuai commented Jul 1, 2026

Uh oh!

yanyishuai commented Jul 1, 2026

Uh oh!

yanyishuai commented Jul 1, 2026

Uh oh!

yanyishuai commented Jul 1, 2026

Uh oh!

yanyishuai commented Jul 1, 2026

Uh oh!

yanyishuai commented Jul 1, 2026

Uh oh!

yanyishuai commented Jul 1, 2026

Uh oh!

yanyishuai commented Jul 1, 2026

Uh oh!

yanyishuai commented Jul 1, 2026

Uh oh!

qingfeng312 left a comment

Choose a reason for hiding this comment

Uh oh!

yanyishuai commented Jul 2, 2026

Uh oh!

yanyishuai commented Jul 2, 2026

Uh oh!

yanyishuai commented Jul 2, 2026

Uh oh!

yanyishuai commented Jul 2, 2026

Uh oh!

yanyishuai commented Jul 2, 2026

Uh oh!

yanyishuai commented Jul 2, 2026

Uh oh!

yanyishuai commented Jul 13, 2026

Uh oh!

yanyishuai commented Jul 13, 2026

Uh oh!

yanyishuai commented Jul 13, 2026

Uh oh!

yanyishuai commented Jul 13, 2026

yanyishuai commented Jun 30, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Jun 30, 2026 •

edited

Loading