docs: add pre-bounty preflight checks and rejection recovery guide [Bounty #383]#387
Conversation
Bounty ramimbo#383 — agent submission hygiene and claim-window docs Adds two complementary updates for the 2nd award slot: 1. docs/api-examples.md: Pre-Bounty Preflight Checks section with live API examples for checking award capacity, active attempts, open PR overlap, and stale/exhausted round detection. 2. docs/agent-guide.md: Recovering from Rejection section under Bounty Submission Checklist, explaining how to respond to mrwk:rejected and mrwk:needs-info labels. These changes complement PR ramimbo#386 which focuses on the agent checklist and PR template. This PR adds the API-level preflight workflow and post-submission recovery guidance. Refs ramimbo#383
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughAdds two documentation sections: pre-bounty read-only preflight checks (API and GitHub verification) and a “Recovering from Rejection” subsection explaining labels, when not to resubmit, and next steps after rejection. ChangesAgent submission guidelines
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related issues
Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/api-examples.md`:
- Around line 589-592: The curl currently queries the issues endpoint which
returns both issues and PRs; change the preflight query to the PR-specific
endpoint by replacing "issues?state=open" with "pulls?state=open&per_page=50"
(or, if you must keep using the issues endpoint, filter the returned items by
presence of the pull_request field before applying the "Bounty `#N`"/"Refs `#N`"
checks) so only actual pull requests are scanned for overlapping scope.
- Line 586: Update the sentence in docs/api-examples.md that currently reads
“Open multiple PRs for the same bounty issue from different contributors is
normal for multi-award bounties, but you should avoid overlapping scope” to use
correct subject-verb agreement; change it to either “Opening multiple PRs for
the same bounty issue from different contributors is normal for multi-award
bounties, but you should avoid overlapping scope” or “Multiple open PRs for the
same bounty issue from different contributors are normal for multi-award
bounties, but you should avoid overlapping scope.”
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: a52f328b-5794-4e6f-93b4-065004a7d762
📒 Files selected for processing (2)
docs/agent-guide.mddocs/api-examples.md
aiautotool
left a comment
There was a problem hiding this comment.
I found two small documentation issues that should be fixed before this lands. The overall direction is useful for #383, and the changed docs pass the smoke check, but the PR example should avoid sending agents to an endpoint that mixes issues and PRs.
Findings:
docs/api-examples.md:586has a subject/verb agreement typo: “Open multiple PRs ... is normal”. Use “Opening multiple PRs ... is normal” or “Multiple open PRs ... are normal”.docs/api-examples.md:590-592says to check open PRs, but the example queries/issues?state=open&per_page=50. GitHub’s issues endpoint returns both issues and pull requests, so agents following this literally can treat regular issues as overlapping PRs. Since this section is specifically PR preflight, use/pulls?state=open&per_page=50, or explicitly filter/issuesresults to objects withpull_requestbefore scanning bodies.
Evidence checked:
docs/api-examples.mdadded capacity, active-attempt, open-PR, and stale-round preflight guidance.docs/agent-guide.mdadded rejection/needs-info recovery guidance.- The changes are docs-only and scoped to
docs/api-examples.mdanddocs/agent-guide.md.
Local validation from a clean PR worktree:
/Users/vkct/Documents/Codex/2026-05-26/s-d-ng-ki-n-th/mergework/.venv/bin/python scripts/docs_smoke.py-> docs smoke okgit diff --check origin/main...HEAD-> passed
No secrets, private security details, payout credentials, or MRWK price claims found in the reviewed diff.
|
Fixed both review items:
Please re-review when convenient. 🙏 |
Summary
Add pre-bounty preflight checks and rejection recovery guidance to the docs.
Changes
docs/api-examples.md (+80 lines)
New Pre-Bounty Preflight Checks section with live API examples:
/api/v1/bounties,/api/v1/bounties/summary)docs/agent-guide.md (+20 lines)
New Recovering from Rejection subsection:
mrwk:rejectedlabels as diagnostic feedbackmrwk:needs-inforequests promptlyEvidence
api.mrwk.ltclab.sitehostTest Evidence
ruff format --check .ruff check .mypy app(no Python code changes)pytest(no test changes)python scripts/docs_smoke.py— docs-only change, template and examples updatedMRWK
Refs #383
Summary by CodeRabbit