[ddaspit]

Quick summary

The review now lists bugs shared with the C# code in its summary and skips rebases that change nothing. local_check.sh now fails when any step fails, not only pytest. Nothing in machine/ or tests/ changes.

Where to look

• The new changes job compares the PR's diff at the last Reviewed at commit with the diff at the head. I ran it on the review rounds from Convert USFM versification before updating from rows #390, Limit zip file entries to 100MB by default #406, and Add try_get_chapters to scripture parsing #407. It skips the three rebases and still reviews the real change on Limit zip file entries to 100MB by default #406 (89685bd to bdce2ab).
• local_check.sh runs every step and ends with local_check: PASSED or local_check: FAILED (<steps>). The pyright errors that failed CI on Limit zip file entries to 100MB by default #406 reproduce locally at 6a53b79, so the earlier "868 passed" hid a pyright failure.
• The pr-review skill no longer posts line comments for upstream defects. A round with nothing new posts a one-line summary.

Deliberately not included

The two Reviewable mentions on #406 and #407 that got no response are not fixed. The action logged "No trigger was met for @claude" even though the review began with @claude. I could not find the cause. A new step prints the review body when this happens, so the next miss should show why. A top-level PR comment works in the meantime.

Validation

• ./local_check.sh --agent-strict -- local_check: PASSED (894 passed, 3 skipped)
• ./local_check.sh with a stub poetry whose pyright fails -- local_check: FAILED (pyright), exit 1
• git diff --check <merge-base>...HEAD -- clean
• The workflow changes have not run in Actions yet. This PR's own review runs the new changes job.

---

Decisions, and why

The rebase check compares with the last reviewed commit rather than the previous push. The review's concurrency group cancels an in-progress round, so a quick rebase after a fix could otherwise cancel the review of the fix and then skip.

The review does not run local_check.sh because black and isort rewrite files. It cites gh pr checks instead, so its summaries stop listing local_check as unverified.

🤖 Generated with Claude Code

---

This change is 

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.18%. Comparing base (0e78bb8) to head (79bc02d).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #409   +/-   ##
=======================================
  Coverage   92.18%   92.18%           
=======================================
  Files         398      398           
  Lines       25267    25267           
=======================================
  Hits        23293    23293           
  Misses       1974     1974           

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

[Enkidu93]

@Enkidu93 reviewed 5 files and all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on ddaspit).