[ptr727]

Answers the Copilot finding raised on promotion #3667 against scripts/pr_review_copilot.py, which #3642 introduced.

• A Copilot Findings cell now counts each finding it names. The file table's Findings column counted one finding per filled cell, so a second finding in the same cell went uncounted. On a clean-verdict round that leaves it out of triage. A new per-reviewer vocabulary pattern, table_finding_marker, is \[P\d\] for Copilot and empty for CodeRabbit and Qodo. A filled cell counts each marker it carries, and once where it carries none, so no count drops below what it was before.
• The Reading docstring states the new counting.

Verification

• The full suite passes, with 3123 tests run and 7 skipped.
• The new subtest, a cell carrying two markers beside a cell carrying none, counts 3. It fails against develop's adapter, which counts 2.
• ruff, mypy, the diff-scoped prose gate, and the eol gate are clean.

Dropped findings

• scripts/pr_review_copilot.py: a [Pn] written inside a code span or in prose about another finding also counts. The only effect is a higher count in the digest's no-thread line, a loud failure rather than a silent one, so it is left as is under "Trust Boundaries and Hardening Effort".

Closes on promotion: none. The finding was raised on #3667 and has no issue of its own.

🤖 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: fix/findings-cell-count
• File: .coderabbit.yaml
• Commit: a455d82
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 450e3430-84c4-4b08-ad4c-77b0d6e4a9b6

• 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 a455d825eb89af6042ab92f0cf6afbda1cfcbf20, the content this pull request carries at that commit against develop, and it recorded 0 findings.

[Copilot]

🟢 Approval recommended

The review evidence identifies no active issues in the updated parsing logic, regression coverage, configuration, or documentation.

No open issues 🎉

What changed in this PR

Updates Copilot findings parsing so multiple severity markers in a single file-table cell are counted individually, preventing findings from being omitted during triage.

Changes:

• Adds reviewer vocabulary support for table finding markers.
• Counts each [P#] marker in Copilot Findings cells while preserving one count for markerless cells.
• Adds regression coverage for multiple markers in one cell.
• Documents the updated counting behavior.

File	Description
tests/​test_pr_review_copilot.py	Verifies that multiple markers and markerless cells are counted correctly.
scripts/​reviewers/​qodo.json	Adds an empty marker pattern for schema consistency.
scripts/​reviewers/​copilot.json	Defines Copilot’s severity-marker pattern.
scripts/​reviewers/​coderabbit.json	Adds an empty marker pattern for schema consistency.
scripts/​pr_review_copilot.py	Compiles the marker pattern and counts findings per marker.

_{Review effort: Lite}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.72%. Comparing base (1a42d34) to head (a455d82).

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #3669   +/-   ##
========================================
  Coverage    81.72%   81.72%           
========================================
  Files           27       27           
  Lines         9126     9128    +2     
========================================
+ Hits          7458     7460    +2     
  Misses        1668     1668           

Flag	Coverage Δ
python-3.13	81.72% <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.