Let a Copilot File Table Carry Forward on an Identical File Set

[ptr727]

Summary

Implements the maintainer's answer on #2272, option 1. A full Copilot file table on an earlier round now counts as coverage of a later head whose rounds carry no table, when the pull request changes exactly the same set of files at both commits. That is the bound carry_holds already applies to a carried coverage statement.

• carried_table picks the newest non-refusal round carrying a table. table_reading consults it only where no round covering the head carries a table of its own, and only after the partial-on-record and review-window guards pass. The carry also requires that table to name exactly the changed files and carry_holds to confirm the change set at both commits.
• status prints coverage=carried:table with the round's commit and the delta. report_verdict and the digest share the reading, and the memoized compare keeps them in agreement.
• A partial table, a partial on record anywhere on the pull request, a review history past the window, or a change set that moved or could not be read each still goes to the maintainer.
• table_covers had no callers left once table_reading existed, so it is gone and its reasoning moved into table_reading.
• One sentence states the carry on each surface that states the Merge Gate's table reading: GOVERNANCE.md, pr-review-conduct, the Copilot runbook, and scripts/README.md.

Verification

• python3 -m unittest tests.test_pr_review: 507 tests pass. TestFileTableCarriesForward covers the carry, a moved and an unreadable change set, a partial table, a partial on record, a truncated history, a head table that misses the diff, the newest-table choice, and a refusal carrying a table.
• Each new test was shown to fail with its behavior reverted or mutated.
• Live check against Return Zero From install-tools.sh Tool Selection and Hide the Home Directory in Notes #2275, an open PR in this exact shape: develop's pr_review.py exits 45 there, and this branch reads coverage=carried:table and exits 0.
• Two local strict-review passes ran. Round 1 raised 7 findings, 6 fixed and 1 declined: the newest table is picked by submission time, the same rule carried_coverage uses. Round 2 raised 4 low findings, all fixed.

Closes #2272

🤖 Generated with Claude Code

[coderabbitai]

Important

• 🔍 Trigger review

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

⚙️ Run configuration

Configuration used: Repository: ptr727/ProjectTemplate/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: f4bb8a62-209a-48f5-adad-32c1f5d58682

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

Autopilot is currently an internal CodeRabbit preview.

---

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

[codecov]

Codecov Report

❌ Patch coverage is 97.22222% with 1 line in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (develop@f92076e). Learn more about missing BASE report.

Files with missing lines	Patch %	Lines
scripts/pr_review.py	97.22%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             develop    #2276   +/-   ##
==========================================
  Coverage           ?   57.45%           
==========================================
  Files              ?       16           
  Lines              ?     7639           
  Branches           ?        0           
==========================================
  Hits               ?     4389           
  Misses             ?     3250           
  Partials           ?        0           

Flag	Coverage Δ
python-3.13	57.45% <97.22%> (?)

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.

[Copilot]

Copilot review overview

🔵 Needs a closer look

The review-gate logic and distributed guidance changes warrant final human approval.

Review effort: Lite
Findings: None

What changed in this PR

This PR adds bounded carry-forward support for Copilot file tables when the changed-file set is identical across commits.

Changes:

• Implements carried-table evaluation and synchronized reporting.
• Adds comprehensive regression tests and safeguards.
• Updates governance, documentation, and distributed skill guidance.

File	Description
tests/​test_pr_review.py	Tests carry-forward behavior and safeguards.
scripts/​README.md	Documents updated coverage rules.
scripts/​pr_review.py	Implements carried file-table evaluation.
GOVERNANCE.md	Updates the review contract.
.github/​skills/​pr-review-conduct/​SKILL.md	Updates generated GitHub guidance.
.github/​copilot-instructions.md	Updates Copilot review guidance.
.claude-plugin/​fleet-skills/​skills/​pr-review-conduct/​SKILL.md	Updates generated Claude guidance.
.claude-plugin/​fleet-skills/​.source-digests/​pr-review-conduct	Refreshes the source digest.
.agents/​skills/​pr-review-conduct/​SKILL.md	Updates canonical review guidance.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.