[ptr727]

Summary

The fourth promotion review of #3484 found the rewrite of the wait help's exit 0 text dropping its caveat that exit 0 does not prove the required checks ran.

• The help now says, right after its opening exit 0 line, that a review-present exit 0 does not poll the checks, so it does not prove the required checks ran. The README's exit 0 row says the same.
• The help's claim that every workflow run on the head has completed is scoped to an open attested held head, the one case that polls its runs.

Behavior is unchanged.

Part of #3471, which the promotion pull request closes.

Pre-existing findings, not filed

• scripts/pr_review.py help, the exit 0 "only under" sentence (pre-existing): its subject is "Exit 0", tied to the attested held head only by "then", so read alone it reads as covering a review-present exit 0 too.

Dropped findings

• scripts/README.md, the exit 0 row (style): "on that review" reaches back past the held-head clause.
• scripts/pr_review.py help (style): "held head whose commit the payload carries" reads the commit as separate from the head.

Test plan

• prose_lint.py --check sentence-length is clean over the diff, and the pre-commit gates pass.
• The recorded local pass checked each changed sentence against the end of wait and found it true.

🤖 Generated with Claude Code

[ptr727]

A recorded local strict-review pass covers head 8c65e4d85d34e1ed10ccc8eef7a045a00211609a, the content this pull request carries at that commit against develop, and it recorded 3 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: fix/3471-exit-0-review-caveat
• File: .coderabbit.yaml
• Commit: 8c65e4d
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 117a185e-a5ee-4d1e-b73e-ad1a46d50dd3

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

🟢 Approval recommended

The documentation changes are fully reviewed, consistent, and have no unresolved blocking issues.

No open issues 🎉

What changed in this PR

This pull request restores documentation clarifying that a review-present wait exit 0 does not poll or prove required checks ran, while limiting workflow-completion claims to the attested held-head path. Runtime behavior is unchanged.

Changes:

• Restores the exit 0 caveat in scripts/pr_review.py help text.
• Mirrors the caveat in the scripts/README.md exit-code table.
• Scopes workflow-run completion claims to open, attested, held heads.

File	Description
scripts/​README.md	Documents that a review-present exit 0 does not prove required checks ran.
scripts/​pr_review.py	Restores the help caveat and scopes workflow-run completion claims.

_{Review effort: Lite}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.04%. Comparing base (578a265) to head (8c65e4d).

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #3491   +/-   ##
========================================
  Coverage    81.04%   81.04%           
========================================
  Files           25       25           
  Lines         8922     8922           
========================================
  Hits          7231     7231           
  Misses        1691     1691           

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