Review Agent PRs That Leave Promises Unawaited

Description and use

Automated agents often drop await, causing silent failures. This workflow identifies, reverts, and tests such changes to ensure durability and correct status codes.

Problem

Automated code agents often remove await statements for perceived speed, causing silent failures and ordering bugs in critical side‑effects.

Outcome

The workflow prevents silent failures by ensuring all critical async calls are awaited or properly outboxed, aligning status codes with durability guarantees.

Workflow

• 1. Fetch agent branch and generate diff
• 2. Run scan script to flag unawaited calls
• 3. Review flagged hunks and decide to trust, revert, or test
• 4. Revert dropped await on critical operations
• 5. Add tests for rejection handling, outbox durability, and zero unhandled rejections
• 6. Verify status code matches durable acknowledgment
• 7. Merge after passing tests

Evidence

The agent patch had removed three await keywords for speed.

Source

Revert dropped await on money, mail, auth, or deletion.

Source