[ptr727]

coverage() in scripts/pr_review.py now lets a partial stated on an earlier round outrank a head reading of unknown. A head whose only readings are unbelievable, beside an earlier partial, prints coverage=PARTIAL and exits 42 instead of printing UNKNOWN and exiting 0. A head stating full alone still wins, and unknown with no partial on record is unchanged.

Tests in tests/test_pr_review.py pair an unknown head round with an earlier partial round (fails before the fix), pin a full head beside an earlier partial, and pin the accepted fail-closed outcome for a head carrying both a full and an unknown round. The status help text names the new case.

Local strict review, two edit rounds spent. Remaining introduced findings and their outcomes:

• The status=COVERAGE_IS_PARTIAL line says the covering review "read fewer files than the pull request changed", which contradicts the carry reason in its brackets. The no-statement carry path already had this. Deferred to a filed issue (Fix the COVERAGE_IS_PARTIAL Status Text on the Carry Paths in pr_review.py #3424).
• .agents/skills/pr-review-conduct/SKILL.md says only a stated partial is an exit 42. This is outside this lane's claimed files. Deferred to a filed issue (Update pr-review-conduct to Say an Unbelievable Head Reading Beside a Partial Exits 42 #3425).

Closes on promotion: #3412

🤖 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: pr-review-partial-outranks-unknown
• File: .coderabbit.yaml
• Commit: 3f893b6
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 88452b34-c69d-4209-8417-a8d0feea155f

• 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

Three unresolved review comments remain, including an inaccurate user-facing reason and incomplete or overly broad documentation.

1 open issue

• Clarify carried-partial reason for mixed head readings · New

What changed in this PR

This pull request updates pr_review coverage precedence so an earlier recorded partial review outranks an unknown head reading, while preserving full head coverage precedence. It also adds regression tests and updates status help text.

Changes:

• Returns PARTIAL when an unknown head has an earlier partial review on record.
• Preserves full head coverage as the higher-priority result.
• Adds tests for unknown, full, and mixed coverage scenarios.
• Updates user-facing coverage documentation and status messaging.

Review comments remain on scripts/pr_review.py: an inaccurate mixed-head reason ([P2], 2 votes), incomplete coverage documentation ([P1], 1 vote), and an overly broad partial-fallback description ([P2], 1 vote).

File	Description
tests/​test_pr_review.py	Adds regression tests for partial, unknown, full, and mixed coverage combinations.
scripts/​pr_review.py	Adjusts coverage precedence, status help text, documentation, and partial-coverage reporting.

_{Review effort: Lite}

[Copilot]

🟡 Changes recommended

Unresolved review findings leave coverage diagnostics and the documented exit-code contract inconsistent with the new behavior.

1 open issue

• Document the carried-partial exit path in README · New

1 resolved since last review

• Clarify carried-partial reason for mixed head readings

_{Review effort: Lite}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.81%. Comparing base (9f19b2a) to head (3f893b6).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #3423   +/-   ##
========================================
  Coverage    80.81%   80.81%           
========================================
  Files           23       23           
  Lines         8859     8863    +4     
========================================
+ Hits          7159     7163    +4     
  Misses        1700     1700           

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