Skip to content

Treat review limit as successful dispatch no-op - #83

Merged
oshaughnessy-junior merged 1 commit into
masterfrom
codex/dispatch-elevation-noop
Aug 12, 2026
Merged

Treat review limit as successful dispatch no-op#83
oshaughnessy-junior merged 1 commit into
masterfrom
codex/dispatch-elevation-noop

Conversation

@oshaughnessy-junior

Copy link
Copy Markdown
Owner

The review coordinator intentionally returns HTTP 409 after the PR-wide review claim ceiling is reached. The existing workflow mapped that expected human-closeout state to a failed CI job and retried it four times.

This change:

  • accepts only the exact sanitized REVIEW_GENERATION_LIMIT response for the current head as a successful no-op
  • emits an Actions notice and summary stating that no reviewer launched and human closeout is required
  • keeps every other 409, authentication, transport, identity, malformed-response, and server error red
  • avoids retrying the intentional ceiling response

The response handler is byte-equivalent to the handler covered by interhost-cross-review PR #31 (apart from this repository's endpoint). Shell syntax and diff checks pass. Merge remains manual.

@oshaughnessy-junior
oshaughnessy-junior had a problem deploying to private-review-dispatch-rift August 12, 2026 12:07 — with GitHub Actions Error
@oshaughnessy-junior
oshaughnessy-junior marked this pull request as draft August 12, 2026 12:07
@oshaughnessy-junior
oshaughnessy-junior temporarily deployed to private-review-dispatch-rift August 12, 2026 12:07 — with GitHub Actions Inactive
@oshaughnessy-junior
oshaughnessy-junior marked this pull request as ready for review August 12, 2026 12:09
@oshaughnessy-junior
oshaughnessy-junior temporarily deployed to private-review-dispatch-rift August 12, 2026 12:09 — with GitHub Actions Inactive
@oshaughnessy-junior
oshaughnessy-junior merged commit 0d651a2 into master Aug 12, 2026
2 of 3 checks passed
@oshaughnessy-junior
oshaughnessy-junior temporarily deployed to private-review-dispatch-rift August 12, 2026 13:40 — with GitHub Actions Inactive
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.

2 participants