docs(bounties): Add recent paid proof rows#448
Conversation
Add newly accepted paid bounty rows for PRs 445 and 446 to the public paid-bounties index. Guard the new PR and proof links with a focused docs test so the index does not silently lose these recent payments. Refs ramimbo#426 Co-Authored-By: OpenAI Codex <codex@openai.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis PR updates the paid bounties documentation table by adding two new accepted bounty entries for 2026-05-26 (issues ChangesBounty Documentation Update
Possibly related issues
Possibly related PRs
🚥 Pre-merge checks | ✅ 5 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
|
This opened after the 2026-05-26 maintainer batch was already merged and deployed. Holding it for the next pass: it needs current-head review coverage and mergeability before acceptance/payment. |
There was a problem hiding this comment.
Actionable comments posted: 1
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 8f496654-b451-416c-abd4-e7c613d215a7
📒 Files selected for processing (2)
docs/paid-bounties.mdtests/test_docs_public_urls.py
GHX5T-SOL
left a comment
There was a problem hiding this comment.
I reviewed current head 179b3e82e1dc7847c7eb24fb86df8c8670ec0b68.
The documentation rows themselves match the recent payment evidence for PR #445 and PR #446, and the diff is tightly scoped to docs/paid-bounties.md plus tests/test_docs_public_urls.py. I agree with CodeRabbit's test-strengthening comment and would address it before merge because this PR's regression guard is meant to protect the PR/proof mapping, not just the presence of two PR URLs and two proof hashes somewhere in the file.
The current test_paid_bounties_lists_recent_docs_payments uses four independent substring assertions. A local mutation that swaps the two new rows' proof labels and proof hashes still satisfies those four assertions, while exact row-level assertions for PR #445 -> f5b9b286... and PR #446 -> bba27216... fail. Please assert row-level fragments or full expected rows so the test proves each accepted work item is tied to the right proof.
Validation on this head:
.venv/bin/python -m pytest tests/test_docs_public_urls.py::test_paid_bounties_lists_recent_docs_payments -q-> 1 passed..venv/bin/python -m pytest tests/test_docs_public_urls.py -q-> 23 passed..venv/bin/python scripts/docs_smoke.py-> docs smoke ok..venv/bin/python -m ruff check tests/test_docs_public_urls.py-> passed..venv/bin/python -m ruff format --check tests/test_docs_public_urls.py-> 1 file already formatted.git diff --check origin/main...HEAD-> passed.git diff origin/main...HEAD | gitleaks stdin --no-banner --redact --exit-code 1-> no leaks found.
No wallet material, private keys, private data, or price/off-ramp claims were used.
adliebe
left a comment
There was a problem hiding this comment.
Reviewed PR #448 at head 179b3e82e1dc7847c7eb24fb86df8c8670ec0b68. No blockers found.
Evidence checked:
- The diff is intentionally small:
docs/paid-bounties.mdplustests/test_docs_public_urls.py, 11 inserted lines total. - The two new paid-bounty rows match the live public proof API:
bba272164648c1bca33710226173a350d002a22f23aa1bf24c46ec594cf2cc9c=>kind=bounty_payment, issue#426, amount50, recipientgithub:campersurfer, submissionPR #446, ledger715.f5b9b286c4cfefe6de73198dd7be5b8d5e1013264623573b7e49488dd6efaecb=>kind=bounty_payment, issue#411, amount75, recipientgithub:campersurfer, submissionPR #445, ledger699.
- The copy stays in the existing public-proof index format and does not add price, liquidity, exchange, bridge, off-ramp, or payout-guarantee claims.
Validation run locally on this head:
.venv\Scripts\python.exe -m pytest tests/test_docs_public_urls.py::test_paid_bounties_lists_recent_docs_payments -q-> 1 passed.venv\Scripts\python.exe -m pytest tests/test_docs_public_urls.py -q-> 23 passed.venv\Scripts\python.exe scripts/docs_smoke.py-> docs smoke ok.venv\Scripts\ruff.exe check docs/paid-bounties.md tests/test_docs_public_urls.py-> passed.venv\Scripts\ruff.exe format --check tests/test_docs_public_urls.py-> 1 file already formattedgit diff --check 12c3dbd462d6647d50bfc8115b852e7cb338bb5b..179b3e82e1dc7847c7eb24fb86df8c8670ec0b68-> clean
Assert row-level PR-to-proof mappings for the recent paid bounty rows so swapped proof hashes fail the regression guard. Refs ramimbo#426 Co-Authored-By: OpenAI Codex <noreply@openai.com>
|
Updated in commit 31b7a5f to address the row-integrity review finding. The test now asserts each full row fragment, including the expected bounty issue, PR, recipient, amount, proof label, and proof URL, so a swapped PR/proof mapping fails.\n\nValidation:\n- Temporarily swapped the two new proof rows locally; the tightened test failed on the expected missing #426/PR #446 -> bba27216 row. Reverted the mutation before committing.\n- /Users/juliebush/mergework/.venv/bin/python -m pytest tests/test_docs_public_urls.py::test_paid_bounties_lists_recent_docs_payments -q -> 1 passed\n- /Users/juliebush/mergework/.venv/bin/python -m pytest tests/test_docs_public_urls.py -q -> 23 passed\n- /Users/juliebush/mergework/.venv/bin/python scripts/docs_smoke.py -> docs smoke ok\n- /Users/juliebush/mergework/.venv/bin/ruff check tests/test_docs_public_urls.py -> passed\n- /Users/juliebush/mergework/.venv/bin/ruff format --check tests/test_docs_public_urls.py -> 1 file already formatted\n- git diff --check -> passed |
GHX5T-SOL
left a comment
There was a problem hiding this comment.
I re-reviewed current head 31b7a5f1109ef1cfca4dcbdf05a0c1bdb37bc2ea; this resolves my earlier row-integrity change request.
The updated test now asserts full row fragments for the two new paid-bounty entries, including issue link, PR link, recipient, MRWK amount, proof label, and proof URL. That covers the PR-to-proof mapping problem I flagged on the prior head, so swapped proof labels/hashes would no longer satisfy the regression guard.
Validation on this head:
.venv/bin/python -m pytest tests/test_docs_public_urls.py::test_paid_bounties_lists_recent_docs_payments -q-> 1 passed..venv/bin/python -m pytest tests/test_docs_public_urls.py -q-> 23 passed..venv/bin/python scripts/docs_smoke.py-> docs smoke ok..venv/bin/python -m ruff check tests/test_docs_public_urls.py-> passed..venv/bin/python -m ruff format --check tests/test_docs_public_urls.py-> 1 file already formatted.git diff --check origin/main...HEAD-> passed.git diff origin/main...HEAD | gitleaks stdin --no-banner --redact --exit-code 1-> no leaks found.- GitHub CI run
26469101907-> success.
No wallet material, private keys, private data, or price/off-ramp claims were used.
adliebe
left a comment
There was a problem hiding this comment.
Follow-up review on current head 31b7a5f1109ef1cfca4dcbdf05a0c1bdb37bc2ea.
I re-checked the update after PR #448 moved from 179b3e82e1dc7847c7eb24fb86df8c8670ec0b68 to 31b7a5f1109ef1cfca4dcbdf05a0c1bdb37bc2ea. The only post-review delta is in tests/test_docs_public_urls.py, where the recent paid-proof assertions were strengthened.
Current-head validation:
pytest tests/test_docs_public_urls.py::test_paid_bounties_lists_recent_docs_payments -q-> 1 passedpytest tests/test_docs_public_urls.py -q-> 23 passedpython scripts/docs_smoke.py-> docs smoke okruff check docs/paid-bounties.md tests/test_docs_public_urls.py-> All checks passedruff format --check tests/test_docs_public_urls.py-> already formattedgit diff --check 12c3dbd462d6647d50bfc8115b852e7cb338bb5b..31b7a5f1109ef1cfca4dcbdf05a0c1bdb37bc2ea-> clean
No new blockers found.
Summary: Add newly accepted MRWK payout rows to
docs/paid-bounties.md, including the just-paid documentation bounty PR 446, with proof links and a docs test guard.Evidence: The live accepted-work API for
github:campersurferreportsaccepted_awards=4andaccepted_mrwk="400". It lists PR 445 paid 75 MRWK at ledger sequence 699 with prooff5b9b286c4cfefe6de73198dd7be5b8d5e1013264623573b7e49488dd6efaecb, and PR 446 paid 50 MRWK under the documentation bounty at ledger sequence 715 with proofbba272164648c1bca33710226173a350d002a22f23aa1bf24c46ec594cf2cc9c.Validation: Added the docs test first and confirmed it failed before updating
docs/paid-bounties.md. After the docs update,python -m pytest tests/test_docs_public_urls.py::test_paid_bounties_lists_recent_docs_payments -q,python scripts/docs_smoke.py,python -m pytest tests/test_docs_public_urls.py -q,ruff check tests/test_docs_public_urls.py,ruff format --check tests/test_docs_public_urls.py, andgit diff --checkall passed locally.This is a focused public documentation update for paid bounty proof indexing. It does not claim cash value, liquidity, redemption, bridge, exchange, off-ramp, price, or private-key behavior.
Bounty #426
Summary by CodeRabbit
Documentation
Tests