Budget PR review effort by PR size

[markphelps]

Summary

• add tiered PR review budgets based on changed files and line counts
• enforce review wall-clock timeout with deterministic timeout comments
• pass reviewBudget into the code-review skill and document budget rules

Test Plan

• node --test test/*.test.mjs
• pnpm build
• npx -p typescript tsc --noEmit -p tsconfig.json

[github-actions]

Verdict: approve

Well-defined PR review budget policies are introduced and enforced, with sound tier rules and deterministic timeout handling. All changes are clear; only tiny nits could further polish minor behavior. No changes required for correctness or clarity.

lib/review-timeout.ts

• nit (L80): In the timeoutReviewResult function, template literals are used to format the duration in seconds, but Math.round(timeoutMs / 1000) will lose milliseconds (e.g., 1599ms rounds down to 1s). For consistency with test/review-timeout.test.mjs (which expects 2s for 1500ms) you may want to consistently round up (Math.ceil()). Minor, discretionary.

workflows/pr-review.ts

• nit (L136): The fallback stats used before reviewStats are fetched (changedFiles: 999, additions: 1000, deletions: 0) are highly conservative, which is intentional, but may lead to over-budgeting if stats fetch fails. Consider annotating the code with rationale or tuning the fallback numbers based on most common PR size range. No action needed unless it's affecting review behavior.

🤖 Automated review by the Flipt PR review agent.