[ptr727]

Summary

A Copilot ccr-overview-v2 round can open on "Approval recommended" and state "No open issues". Its narrative can still name findings in a nested "Review findings remain unresolved:" list and in the file table's Findings column. The reader counted none of them. So the digest printed CLEAN VERDICT HEADLINE for a round it said counts no finding, and wait exited 0 with nothing pointing at the findings.

• The nested label is now a findings-list spelling in scripts/reviewers/copilot.json.
• The file table's Findings column is read through a new table_findings vocabulary key. One table_cells reader replaces table_paths, so the paths and the column come from a single walk.
• A new Reading.listed field holds the larger of the list count and the filled Findings cells, since both name the same findings. unthreaded takes it into account, so FINDINGS WITH NO THREAD (2) prints for the body in Count the Findings Under Copilot's "Review Findings Remain Unresolved" Bullet in pr_review.py #3610.
• Reading.open stays the count the round states. So overview=T/M and its ? reading keep their meaning for the Merge Gate.
• A clean round whose list or table names a finding no longer prints CLEAN VERDICT HEADLINE. A flagging round in the same shape still exits 50 as it did on develop.
• No exit code changes. The documented contract keeps FINDINGS WITH NO THREAD advisory.

Closes on promotion: #3610

Verification

• tests/test_pr_review_copilot.py and tests/test_pr_review.py gain constructed cases for the Count the Findings Under Copilot's "Review Findings Remain Unresolved" Bullet in pr_review.py #3610 shape, the list alone, the column alone, a stated count beside the lists, and a flagging verdict that still exits 50. Each new case fails against the base reader.
• The whole suite passes (python -m unittest discover -s tests), tests/test_pr_review_size.py included.
• ruff format, ruff check, mypy, the prose gate, build_dist.py --check, and the eol gate pass.

Local pass findings

The first push's spent edit budget left four introduced findings open. The next push fixed each:

• scripts/pr_review_digest.py dropped the headline under a flagging verdict carrying a findings list. Fixed in 872b4ed, which also added the test assertion for it.
• scripts/pr_review.py --help said the headline alone names what the round flags. Fixed in 872b4ed.
• Lines this change edited ran past the wrap width. Rewrapped in 872b4ed.
• The flagging wording named only the findings list, while listed can come from the file table alone. The pr-review-conduct skill still called the headline the one place. Fixed in dd213c6.

A later pass raised one more introduced finding, also fixed:

• scripts/pr_review_copilot.py the Reading docstring said listed is the findings the Findings column names, while a cell counts once however many it names. Fixed in fc8eb49, which states the possible undercount. The pass over fc8eb49 raised no finding.

🤖 Generated with Claude Code

[ptr727]

A recorded local strict-review pass covers head 2b7304d2e0bde2e727c3ccd97fd39675c9799c58, the content this pull request carries at that commit against develop, and it recorded 4 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: feature/auto-3610
• File: .coderabbit.yaml
• Commit: fc8eb49
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 8ac2db5a-7437-4836-aada-5d8a561af8e0

• 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]

🟡 Changes recommended

Several active findings remain, including a moderate issue that drops the flagging headline from output and a missing test assertion, along with documentation, status-message, and formatting inconsistencies.

6 open issues

• Preserve the flagging headline with findings lists · New
• Assert emission of the flagging headline · New
• Document findings-list flagging in the help contract · New
• Wrap help text to the configured line width · New
• Mention file-table findings in the exit-50 status message · New
• Wrap the updated docstring to the configured width · New

What changed in this PR

This pull request updates Copilot review parsing so findings named in unresolved lists or file-table Findings columns are counted and surfaced without changing existing exit codes.

Changes:

• Adds listed finding tracking and unified table-cell parsing.
• Updates verdict handling, digest output, reviewer vocabularies, and documentation.
• Adds regression coverage for list-only, table-only, clean, flagging, and stated-count scenarios.
• Synchronizes the review-conduct skill distributions.

This pull request requires changes.

File	Summary
tests/​test_pr_review.py	Adds end-to-end coverage for listed findings and verdict output.
tests/​test_pr_review_copilot.py	Tests nested finding lists, Findings columns, and stated counts.
scripts/​reviewers/​qodo.json	Adds the shared table_findings vocabulary key.
scripts/​reviewers/​copilot.json	Adds unresolved-list and Findings-column vocabulary.
scripts/​reviewers/​coderabbit.json	Adds the shared table_findings vocabulary key.
scripts/​README.md	Documents list and file-table finding sources.
scripts/​pr_review.py	Updates status help and verdict messaging.
scripts/​pr_review_rounds.py	Accounts for listed findings in verdict classification.
scripts/​pr_review_digest.py	Renders advisory output for unthreaded listed findings.
scripts/​pr_review_copilot.py	Parses finding lists and file-table cells into Reading.listed.
.github/​skills/​pr-review-conduct/​SKILL.md	Updates generated review-conduct guidance.
.claude-plugin/​fleet-skills/​skills/​pr-review-conduct/​SKILL.md	Updates the Claude skill distribution.
.claude-plugin/​fleet-skills/​.source-digests/​pr-review-conduct	Refreshes the skill source digest.
.agents/​skills/​pr-review-conduct/​SKILL.md	Updates the canonical review-conduct guidance.

_{Review effort: Lite}

[codecov]

Codecov Report

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

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3642      +/-   ##
===========================================
+ Coverage    81.60%   81.70%   +0.09%     
===========================================
  Files           26       26              
  Lines         9070     9119      +49     
===========================================
+ Hits          7402     7451      +49     
  Misses        1668     1668              

Flag	Coverage Δ
python-3.13	81.70% <100.00%> (+0.09%)	⬆️

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.

[ptr727]

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

[ptr727]

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