[ptr727]

Summary

Implements #3460. The promotion pull request carries the closing line.

The pr_review readers spelled three payload accessors out at each site. This adds one definition each to scripts/pr_review_rounds.py and moves every lenient site onto them.

• commit_oid(node) reads a node's commit oid, or None where the payload names none.
• page_flag(pr, field, key) reads one pageInfo flag of a connection, false where absent.
• head_nodes(pr, nodes) filters nodes to those carrying the current head's commit.

Each strict pr["reviewThreads"]["nodes"] read is unchanged, so a malformed payload still ends the run. Two sites stay as they are on purpose. head_review_done needs each review's position, so it uses commit_oid rather than the filter. page_files follows a cursor only on an explicit True, which is stricter than page_flag.

pr_review_checks.py now imports the accessors from pr_review_rounds.py, an edge READER_IMPORTS already allows. head_commit collapses onto commit_oid, which leaves the file at 444 lines, under its 450 ceiling.

Verification

• The unit suite passes unchanged: python -m unittest discover -s tests ran 3081 tests, OK (skipped=7).
• The refactor's behavior was recorded rather than inferred. Every touched function plus main was wrapped, and the test_pr_review* suites ran on the base tree and on this branch. Both runs made 10608 calls. Once wall-clock timestamps and the snapshot directory name are normalized, every return value and printed output is identical.
• That harness caught a name collision in an earlier draft, where the filter was called on_head and render holds a local of that name. It is now head_nodes.
• The local strict review ran two rounds at the frontier tier. Round one found one missed single-quoted site in render, fixed in the second commit. Round two found nothing.

🤖 Generated with Claude Code

[ptr727]

A recorded local strict-review pass covers head 29c176c545de8fc572db0852a166da1206b18326, the content this pull request carries at that commit against develop, and it recorded 0 findings.

[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: pr-review-accessors
• File: .coderabbit.yaml
• Commit: 29c176c
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 75473812-d32e-48ee-a2c7-05ff84d2eec9

• 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

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

[Copilot]

🟢 Approval recommended

No active issues were identified, and the refactor preserves strict/lenient behavior and explicit pagination while consolidating repeated payload access.

No open issues 🎉

What changed in this PR

This pull request centralizes repeated payload access in the pr_review reader modules while preserving existing strict and lenient behavior.

Changes:

• Adds shared commit_oid, page_flag, and head_nodes accessors.
• Migrates repeated commit, pagination, and head-review reads to the shared accessors.
• Reuses the accessors across review, digest, and check processing.
• Preserves strict review-thread reads and explicit pagination behavior.

File	Description
scripts/​pr_review.py	Uses commit_oid when reading refusal commit identifiers.
scripts/​pr_review_rounds.py	Defines and applies the shared payload accessors.
scripts/​pr_review_digest.py	Uses shared commit and head-review readers during digest rendering.
scripts/​pr_review_checks.py	Reuses shared commit and pagination readers for check processing.

_{Review effort: Lite}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.61%. Comparing base (487deaa) to head (29c176c).

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #3624   +/-   ##
========================================
  Coverage    81.60%   81.61%           
========================================
  Files           26       26           
  Lines         9070     9073    +3     
========================================
+ Hits          7402     7405    +3     
  Misses        1668     1668           

Flag	Coverage Δ
python-3.13	81.61% <100.00%> (+<0.01%)	⬆️

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.