Skip to content

Release v1.7.1: suppress public failure comments; retry bad JSON - #11

Merged
tllongdev merged 1 commit into
mainfrom
develop
Jul 27, 2026
Merged

Release v1.7.1: suppress public failure comments; retry bad JSON#11
tllongdev merged 1 commit into
mainfrom
develop

Conversation

@tllongdev

Copy link
Copy Markdown
Owner

Summary

  • Stop posting a public "Automated review could not complete" comment as the user by default. A malformed-JSON model response on a coworker's PR triggered exactly this. Failures are now logged locally and the PR is left untouched. Opt back in with --post-failure-comment or VIGILANT_POST_FAILURE_COMMENT=1.
  • Retry once on a JSON parse failure (call_model_json) with a strict-JSON repair reminder before giving up. A fresh sample almost always fixes a transient formatting slip. Inference/config errors still fail fast.
  • Bump to 1.7.1.

Test plan

  • PYTHONPATH=src pytest (225 passed), ruff check, mypy src all green.
  • New tests: retry succeeds on second sample, exhausts attempts, and does not retry config/inference errors.

A malformed-JSON model response made the watcher post an "Automated review
could not complete" comment as the user on a coworker's PR. Suppress that by
default (opt in with --post-failure-comment / VIGILANT_POST_FAILURE_COMMENT)
and retry once with a strict-JSON reminder before giving up, since a fresh
sample almost always fixes a transient formatting slip. Bump to 1.7.1.
@tllongdev
tllongdev merged commit 089fb40 into main Jul 27, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant