You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This commit was created on GitHub.com and signed with GitHub’s verified signature.
Changed
An agent that cannot finish now exits 2. It used to print a warning and
count as zero findings, so an OpenRouter outage or a garbled reply passed
as a clean review (the pre-commit hook included). Findings from the agents
that did finish are still printed and posted, and the posted review names
the ones that did not. --advisory, and so the default review.yml, still
exits 0 (#41).
Fixed
--stdin crashed on httpx.ReadTimeout over a large diff. Timeouts and
dropped connections are now retried with the same backoff as a 429 (#41).
review.yml failed on diffs containing escape sequences.gh pr diff
refuses such a diff unless told otherwise, printed nothing, and the job
reported No changes to review. before exiting 1. The diff is now fetched
with --allow-escape-sequences into a file, and a failed fetch says so
(#44).
The workflow guard missed renames. It read gh pr diff --name-only,
which lists only a renamed file's new path, so a fork could move a workflow
out of .github/workflows/ (deleting it) and pass. The guard in examples/gatehouse.yml and in this repo now reads both filename and previous_filename from the pull-request files API. Found by Gatehouse
reviewing its own fleet rollout (RT #1507); repos that copied the example
should re-copy it.
A reply could clear a failed workflow guard. The guard ran only on pull_request_target, so on a pull_request_review_comment run it reported skipped, and a required check counts skipped as passing. It now runs on
both events, and posts its explanation only once.