[ptr727]

Summary

Implements #3457 (the closing keyword goes on the promotion pull request).

• scripts/pr_review_digest.py is a new pure reader module. Its render takes the payload, the normalized checks, the run log's rate-limit sentence, and the seen set as arguments, and makes no call. It imports only the other readers and the reviewer adapter, never the gate.
• digest stays in pr_review.py as the shell that does the reads (full_read, check_nodes, and confirmed_quota when a refusal is present), so the tests that patch it there keep working.
• The status header is an ordered list of (key, value) pairs joined by one space, a None value leaving the field out, so a new field is one entry.
• The window, threads-truncated, and reviews-truncated advisory lines print from an ordered (value, template) table, in the order they printed before.
• confirmed_quota is now read once per digest rather than up to twice, so refusal=QUOTA and the RATE LIMIT line can no longer disagree across two run-log reads.
• The new module joins the readers' import guard, the whole-source guards, and the size ceilings, and scripts/README.md names it.

Verification

• Every digest output the existing suite produces, 256 renderings across tests/test_pr_review.py, was recorded before and after the move and compared byte for byte, identical.
• TestDigestRenderIsPinned pins a crowded and a quiet digest whole, with text captured from the pre-move code.
• The full suite passes, python -m unittest discover -s tests, 2963 tests, 7 skipped. Ruff, mypy, and the pre-commit hook are clean.
• A local strict review ran two rounds. Round 1 raised two introduced findings, the new helper's name colliding with pr_review_rounds.refusal_of and the pin test's docstring claiming more than it pins, both fixed in 023589f. Round 2 raised none.

Pre-existing findings, not filed

• scripts/README.md:9, scripts/README.md:190, scripts/pr_review.py:16: pre-existing, each an older sentence over the opt-in sentence-length check's 25 words, flagged only because its line is in this diff.

The comments label is applied because four comment lines moved verbatim from pr_review.py into the new module.

🤖 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: digest-render-leaf
• File: .coderabbit.yaml
• Commit: 3eb58d6
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: d34fae91-404c-4928-b9af-f941b70d4efc

• 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.}

[ptr727]

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

[Copilot]

🟡 Changes recommended

Address the moderate refusal-recomputation finding and the two documentation/ownership clarification nits.

1 open issue

• Clarify read-shell ownership in documentation · New

What changed in this PR

This pull request extracts digest rendering into a pure pr_review_digest.py module while keeping data reads in pr_review.py and preserving existing output behavior.

Changes:

• Adds ordered digest header and advisory rendering.
• Keeps digest as the read-and-delegate shell and consolidates quota-log reads.
• Expands source guards, size limits, documentation, and byte-for-byte output tests.

File	Summary
tests/​test_pr_review.py	Updates reader guards, adapter references, and pinned digest output tests.
tests/​test_pr_review_size.py	Adds a size ceiling for the new digest module.
scripts/​README.md	Documents the new reader module; remove the incidental reader count to avoid stale documentation.
scripts/​pr_review.py	Retains digest input acquisition and delegates rendering to the pure module.
scripts/​pr_review_digest.py	Provides pure digest rendering; clarify read-shell ownership and avoid recomputing the derived refusal during rendering.

_{Review effort: Lite}

[ptr727]

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.07%. Comparing base (f87bf62) to head (3eb58d6).

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3493      +/-   ##
===========================================
+ Coverage    81.04%   81.07%   +0.02%     
===========================================
  Files           25       26       +1     
  Lines         8922     8934      +12     
===========================================
+ Hits          7231     7243      +12     
  Misses        1691     1691              

Flag	Coverage Δ
python-3.13	81.07% <100.00%> (+0.02%)	⬆️

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.