fix: resolve 6 high-severity bugs#33
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 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 |
Add comprehensive test coverage for additional tools: - Bash tool: command execution, stderr, exit codes, timeout, dangerous command warnings - Todo tools: add, update, remove, clear, read, filter by status - Undo tool: backup creation, restoration, listing - Diff tool: file comparison, content comparison, identical detection - Git tools: status, diff, commit with auto-generated and custom messages Increases test count from 83 to 116 tests. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- hooks/executor.ts: Track hook errors in metadata, stop execution for critical hooks (priority < 10) on failure instead of silently continuing - rpc/client.ts: Log JSON parse errors with content preview instead of silently ignoring; add cleanupStaleRequests() to prevent memory leaks from unbounded pendingRequests map - rpc/types.ts: Add createdAt timestamp to PendingRequest for cleanup - skills/parser.ts: Capture full error details (stderr, exit code) for command failures; add security documentation for command execution - background.ts: Add automatic cleanup of completed tasks (>1hr old), cap total tasks at 100 to prevent unbounded memory growth Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
550f515 to
7a2bb65
Compare
Summary
Fixes 6 high-severity bugs identified during code review:
createdAttimestamp andcleanupStaleRequests()to prevent unbounded pendingRequests growthTest plan
🤖 Generated with Claude Code