Opus finding
Skipped-by-no-reviewable-files path bypasses comment-already-posted idempotency for a row that prior succeeded with comment but lost cache
lowmaintainabilitylow
- apps/web/lib/review-worker.ts:211-220
If a row reaches processClaimedRow with files.length===0 we mark it succeeded — but this code path overwrites filesReviewed and providerResponses on a row that might have been partially updated by a previous attempt. Since the early gates check prCommentId !== null before claiming, this is mostly safe; but updateReview clobbers fields without an idempotency check. Minor maintainability issue.
Recommendation
Consider scoping the updateReview to only write fields when prior state is pending/empty.