[ptr727]

Read Copilot's Open findings: label, drop the duplicate pr_review_rounds.null_node (callers use gh_graphql.null_node), and bring three scripts/README.md passages in line with the code on develop (hosted-reviewer hold in wait exit 30, merge's CHECKS_PENDING refusal with pending=, the Qodo dismiss remedy).

Closes on promotion: #3824, #3817, #3805

Pre-existing findings, not filed:

• scripts/pr_review_checks.py limited_since: a queued check with no start time reads any past rate limit as newer (pre-existing, code outside this diff).
• scripts/README.md: "neither passed nor failed" mirrors the code docstrings, which also count a failed check whose newer run is unfinished (pre-existing wording, low severity).

Local strict review: attested below after the push.

🤖 Generated with Claude Code

[coderabbitai]

Important

• Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

• Source mode: Auto
• Repository: ptr727/ProjectTemplate
• Branch: burndown/3824-pr-review-readers
• File: .coderabbit.yaml
• Commit: ab6fd85
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 5bb88200-1714-46ab-a3ef-9c15c0322d8f

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

• X
• Mastodon
• Reddit
• LinkedIn

[ptr727]

A recorded local strict-review pass covers head ab6fd852c08dc6e3c60a62dd986bf7a51358567f, the content this pull request carries at that commit against develop, and it recorded 1 finding.

[Copilot]

🟢 Approval recommended

No active review issues remain, with the parser update covered by regression tests and the related documentation and cleanup changes addressed.

No open issues 🎉

What changed in this PR

This pull request updates Copilot findings parsing and Merge Gate documentation, removes an unused duplicate GraphQL helper, and adds regression coverage.

Changes:

• Recognizes Open findings: in Copilot review output.
• Adds parser test coverage for the new label spelling.
• Removes the duplicate pr_review_rounds.null_node helper.
• Aligns scripts/README.md with current wait, merge, and Qodo behavior.

File	Description
tests/​test_pr_review_copilot.py	Adds coverage for Open findings: in supported findings-list contexts.
scripts/​reviewers/​copilot.json	Adds the new Copilot findings-label spelling to the parser vocabulary.
scripts/​README.md	Documents current hosted-review, merge, pending-check, and Qodo remediation behavior.
scripts/​pr_review_rounds.py	Removes the unused duplicate null_node helper.

_{Review effort: Lite}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.97%. Comparing base (b38e084) to head (ab6fd85).
⚠️ Report is 77 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3828      +/-   ##
===========================================
+ Coverage    81.60%   82.97%   +1.36%     
===========================================
  Files           26       29       +3     
  Lines         9070     9741     +671     
===========================================
+ Hits          7402     8083     +681     
+ Misses        1668     1658      -10     

Flag	Coverage Δ
python-3.13	82.97% <ø> (+1.36%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.