fix(code-reviews): expire stale reviews

[alex-alecu]

Summary

• Stale pending and running code reviews now get cancelled by the existing cron after 75 minutes.
• Fresh retry attempts stay active, so a new retry is not stopped just because the first review record is old.
• The cron response now reports how many stale reviews and attempts it cancelled.

Verification

N/A (no manual verification performed).

Visual Changes

N/A

Reviewer Notes

The cron marks rows as cancelled in the database only. It does not stop an already running worker or close an external pull request check.

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

The PR correctly introduces a stale-review expiry step in the dispatch cron — the CTE-based cancellation query is safe, the fresh-retry guard is correct, type definitions are consistently updated across all packages, and the race condition with in-flight workers is already handled by the existing terminal-state guard in the status callback route.

Files Reviewed (7 files)

• apps/web/src/lib/code-reviews/db/code-reviews.ts — new cancelExpiredPendingAndRunningCodeReviews function; CTE logic looks correct
• apps/web/src/lib/code-reviews/dispatch/dispatch-constants.ts — new CRON_STALE_CODE_REVIEW_EXPIRY_MINUTES constant and SQL helper
• apps/web/src/lib/code-reviews/dispatch/dispatch-pending-code-review-owners.ts — orchestrator correctly calls expiry cancellation before dispatching
• apps/web/src/lib/code-reviews/dispatch/dispatch-pending-code-review-owners.test.ts — comprehensive integration tests covering the retry-parent preservation and dispatch-window sparing logic
• apps/web/src/app/api/cron/dispatch-pending-code-reviews/route.test.ts — updated mock to include new summary fields
• packages/db/src/schema-types.ts — dispatch_expired added to both CODE_REVIEW_TERMINAL_REASONS and CODE_REVIEW_BENIGN_TERMINAL_REASONS
• packages/worker-utils/src/cloud-agent-next-client.ts — CloudAgentTerminalReason union updated
• services/code-review-infra/src/types.ts — InternalStatusTerminalReasonSchema updated

---

_{Reviewed by claude-4.6-sonnet-20260217 · 1,122,494 tokens}

_{Review guidance: REVIEW.md from base branch main}