[ptr727]

Summary

• scripts/local_review.py record takes --report, the file the agent-skill reviewer wrote, and no longer accepts a typed --findings count. The report's first line is contentDigest: <digest>, and each later non-blank line is one finding. record refuses a report that is absent or names another digest. Otherwise it stores the report's sha256 on the pass as reportSha256 and derives the count from the report's lines. There is no dispatch-time stamp, per the recorded decision on Bind a Local Review Record to the Reviewer's Actual Report #3699.
• status gains a reportPath field, the digest-keyed path in the worktree's git directory where the reviewer writes. A record removes reports written for any other digest. The fields pr_review.py reads (covered, receiptProblems, findings) are unchanged.
• The receipt format moves to version 2, so a receipt an earlier engine wrote, its pass carrying a typed count, no longer covers.
• check's printed remedy carries --report with the real path, so it still runs as printed once the reviewer has written its report.
• The local-strict-review skill's brief has the reviewer write that report. "Running It" says to record only once the harness delivers the reviewer's completion notification, and that text in the driver's own turn is never that notification. The handback route and "Recording the Pass" carry --report, and on the handback route the dispatcher reads reportPath from status itself.
• scripts/README.md describes the new record contract and names the uv no-cache exception at both places it promises uvx alone can read the action files (Add the uv No-Cache Exception to the scripts/README.md uvx Promise #3705).

Closes on promotion: #3699, #3705

Local Review Findings

The first push's local-strict-review edit budget ran out with two introduced findings open. Commit d2512ee fixes both, and the pass over that commit found nothing:

• introduced, fixed: "What It Does" said the pass dispatches a read-only subagent, while the brief now has the reviewer write its report. It now says the reviewer needs an agent type that can write a file.
• introduced, fixed: read_report's OSError and UnicodeDecodeError branches only reworded a failure that main() already reports with exit 2, so they are gone.

Filed rather than fixed here: #3804, the typed --findings count canonical_review.py record still takes.

Verification

• python3 tests/test_local_review.py: new cases cover an absent report, a report for another digest, a report with no digest line, the derived count and stored sha256, a header-only report, reportPath sitting in the git directory without moving the digest, pruning, and the removed --findings flag.
• tests/test_pr_review.py, tests/test_canonical_review.py, and tests/test_cli_usage.py pass against the changed engine.
• prose_lint.py --diff origin/develop, ruff format --check, ruff check, repo_gate.py, spec/validate.py, and build_dist.py --check are clean.

🤖 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: burndown/3699-review-record
• File: .coderabbit.yaml
• Commit: 28ceefb
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: fd0af7ae-22d8-4431-bcc7-bb9b94daca2c

• 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

[ptr727]

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

[Copilot]

🟡 Changes recommended

Outstanding correctness issues remain around reviewer write capability, legacy pass invalidation, and concurrent report pruning, so the change cannot yet reliably enforce its report-bound contract.

4 open issues

• Invalidate legacy agent-skill passes without report digests · New
• Prune reports under the receipt lock · New
• Dispatch a reviewer with write access to create the report · New
• Let the top-level handler handle report read failures · New

What changed in this PR

This pull request binds local review receipts to reviewer-generated, digest-specific report files instead of caller-supplied finding counts. It updates the recording engine, tests, documentation, and distributed review skills.

Changes:

• Adds report validation, digest binding, hashing, derived finding counts, and stale-report cleanup.
• Updates status output, check remediation, and the recording workflow to use --report.
• Documents reviewer report creation, completion notification handling, and the uvx no-cache exception.
• Regenerates the distributed local-strict-review skill files and source digest.
• This pull request requires changes.

File	Description
tests/​test_local_review.py	Adds coverage for report validation, derived counts, hashes, paths, pruning, and CLI changes.
scripts/​README.md	Documents the report-based recording contract and uvx cache exception.
scripts/​local_review.py	Implements report parsing, validation, hashing, status paths, cleanup, and report-based recording.
.github/​skills/​local-strict-review/​SKILL.md	Updates the generated Copilot review skill distribution.
.claude-plugin/​fleet-skills/​skills/​local-strict-review/​SKILL.md	Updates the generated Claude review skill distribution.
.claude-plugin/​fleet-skills/​.source-digests/​local-strict-review	Updates the generated skill source digest.
.agents/​skills/​local-strict-review/​SKILL.md	Updates the source review brief and recording workflow for reviewer-generated reports.

_{Review effort: Lite}

[ptr727]

A recorded local strict-review pass covers head d2512ee96c85f665bdd1f80543db390d34a65c83, 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 82.77%. Comparing base (23c7d78) to head (28ceefb).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3807      +/-   ##
===========================================
+ Coverage    82.72%   82.77%   +0.04%     
===========================================
  Files           29       29              
  Lines         9629     9654      +25     
===========================================
+ Hits          7966     7991      +25     
  Misses        1663     1663              

Flag	Coverage Δ
python-3.13	82.77% <100.00%> (+0.04%)	⬆️

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