Review Agent PRs That Leave Promises Unawaited
Описание и применение
Автоматизированные агенты часто удаляют await, вызывая неотслеживаемые ошибки. Предложенный workflow выявляет, откатывает и тестирует такие изменения, гарантируя надёжность и правильность статуса ответа.
Задача
Automated code agents often remove await statements for perceived speed, causing silent failures and ordering bugs in critical side‑effects.
Результат
The workflow prevents silent failures by ensuring all critical async calls are awaited or properly outboxed, aligning status codes with durability guarantees.
Рабочий процесс
- 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
Доказательства
The agent patch had removed three await keywords for speed.
Revert dropped await on money, mail, auth, or deletion.
- Дата записи: 2026-09-18T00:30:34.745Z