[ptr727]

Part of #3531.

Splits the over-length sentences in the comments and docstrings of pr_review*.py, pr_cleanup.py, gh_graphql.py, host_gate.py, and their two test files. Comments and docstrings only, no code or asserted string changed. scripts/pr_review.py's pinned docstring phrases are unchanged.

Sentence-length count for the slice went from 186 to 0.

Tests: uv run --no-project --with pytest --with markdown-it-py python -m pytest tests/test_pr_review.py tests/test_host_gate.py tests/test_pr_review_size.py tests/test_pr_cleanup.py tests/test_gh_graphql.py passes (696 passed).

The pr_cleanup.py "A branch is finished when" sentence uses the wording scripts/README.md already carries.

Copies left for the comment sweep

None. The two scripts/README.md table cells that restate pieces of the status and wait exit codes are worded differently and already read zero.

Local strict review: two passes, the first raised drift items that the last commit restores, the second found none.

🤖 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: feature/3531-cmt-pr-review-host-gate
• File: .coderabbit.yaml
• Commit: 69f7bb1
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: f663c1db-630c-4db4-bd3e-4781f36241fd

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

[Copilot]

🟡 Changes recommended

Two active nit-level findings remain open: an unclear remedy clause in the probe() docstring in scripts/host_gate.py, and missing blank lines before separate rules in the help text in scripts/pr_review.py.

2 open issues

• Clarify probe() docstring remedy subjects · New
• Separate help-text rules from bullet lists · New

What changed in this PR

This PR splits over-length sentences in the comments and docstrings of the PR review scripts, the cleanup script, the GraphQL helper, and the host gate, along with two of their test files. It is part of a wider effort to bring the repository's sentence-length check to zero. It changes no code and no asserted string, so behavior stays the same. The author reports that the sentence-length count for this slice went from 186 to 0.

Changes:

• Long sentences in comments and docstrings are split into shorter ones.
• Some run-on lists of conditions and exit codes become bullet points.
• The pr_cleanup.py branch-finished sentence is reworded to match the wording in scripts/README.md.

File	Description
tests/​test_pr_review.py	Rewraps long test docstrings and comments into shorter sentences. Assertions are unchanged.
tests/​test_host_gate.py	Splits long docstring and comment sentences. Two schema-comparison comments are reworded.
scripts/​pr_review.py	Splits the help text into shorter sentences and turns skip and exit-code conditions into bullets. Pinned docstring phrases are unchanged.
scripts/​pr_review_rounds.py	Splits docstrings for the table matcher, thread matcher, and local-pass check. The local-pass check now lists its conditions.
scripts/​pr_review_copilot.py	Splits one module docstring sentence and one entry-parsing docstring sentence.
scripts/​pr_review_checks.py	Splits long sentences in the check normalization docstring.
scripts/​pr_cleanup.py	Turns the refusal conditions into a list and splits the sweep's finished-branch definition.
scripts/​host_gate.py	Splits long module, probe, merge, and overlay docstrings and comments. Some lists become bullets.
scripts/​gh_graphql.py	Splits the module docstring and the request and walker docstrings into shorter sentences.

_{Review effort: Lite}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.60%. Comparing base (0d4e6f1) to head (69f7bb1).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #3606   +/-   ##
========================================
  Coverage    81.60%   81.60%           
========================================
  Files           26       26           
  Lines         9070     9070           
========================================
  Hits          7402     7402           
  Misses        1668     1668           

Flag	Coverage Δ
python-3.13	81.60% <ø> (ø)

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