fix: narrow comment firewall to paragraph workarounds - #406
Conversation
Narrows #403 to the failure mode Bun/RoboBun actually targets. - challenge only standalone paragraphs when the staged change contributes 3+ non-structural comment lines - group adjacent line comments and adjacent one-line block comments without merging separate multi-line blocks - treat multiline block explanations opened after code or closed before structural punctuation/JSX closers as reviewable, while comments followed by executable text remain inline - pass one/two-line staged changes, inline comments, untouched comments, deletions, and pure renames - store rationale metadata in Git-local state with pre-change-blob migration, per-worktree ownership, conflict detection, CAS pruning, shared review reads, and private review writes - bound semantic review to one 200-finding/120k-character batch; overflow exits deterministically - allow no content-keyword bypasses Prior art: oven-sh/bun#37948 and oven-sh/bun#39166. Validation: 58 focused tests; typecheck, lint, format, build, and benchmark registry checks. Full suite: 3925 passed; one unrelated process-reaping timing fixture failed under full load and passed immediately alone. AI review bypass explicitly authorized by the user after independent correctness, completeness, and duplicate/clone agent reviews passed; the Claude CLI weekly quota was exhausted.
|
Warning Review limit reached
Next review available in: 57 minutes Limit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?Wait for the limit to reset, then comment An organization admin can change what happens after included review limits in Billing. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (9)
📒 Files selected for processing (20)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
1 participant
Narrows #403 to the failure mode Bun/RoboBun actually targets.
Prior art: oven-sh/bun#37948 and oven-sh/bun#39166.
Validation: 58 focused tests; typecheck, lint, format, build, and benchmark registry checks. Full suite: 3925 passed; one unrelated process-reaping timing fixture failed under full load and passed immediately alone.
AI review bypass explicitly authorized by the user after independent correctness, completeness, and duplicate/clone agent reviews passed; the Claude CLI weekly quota was exhausted.