Skip to content

[ci] Retry transient empty NAT responses - #124

Merged
antoniomtz merged 2 commits into
mainfrom
codex/retry-empty-nat-response
Aug 4, 2026
Merged

[ci] Retry transient empty NAT responses#124
antoniomtz merged 2 commits into
mainfrom
codex/retry-empty-nat-response

Conversation

@antoniomtz

@antoniomtz antoniomtz commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Retry a NAT functional request once, after two seconds, only when NAT returns the known empty-LLM 422 workflow_error.
  • Emit a visible GitHub Actions warning when the targeted retry is used.
  • Keep timeouts, other HTTP errors, malformed responses, and response-contract failures single-attempt.

Root cause

After #123 merged, Blueprint QA reached the search agent but the upstream model returned neither content nor tool calls. NAT surfaced that transient condition as HTTP 422. A rerun on the same commit passed all NAT checks and the UI suite.

Validation

  • pytest tests/: 394 passed
  • Ruff lint and format checks passed for backend, tests, and the NAT checker
  • Pyright passed for the NAT checker and backend targets
  • Blueprint QA passed all NAT functional checks and the UI suite with the retry implementation
  • git diff --check passed

Follow-up to #123.

@antoniomtz
antoniomtz marked this pull request as ready for review August 4, 2026 15:48
@antoniomtz antoniomtz self-assigned this Aug 4, 2026
@antoniomtz
antoniomtz merged commit 03fc55e into main Aug 4, 2026
7 checks passed
@antoniomtz
antoniomtz deleted the codex/retry-empty-nat-response branch August 4, 2026 17:20
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