GH#3702: Fix critical quality-debt in code-audit-helper.sh#3720
GH#3702: Fix critical quality-debt in code-audit-helper.sh#3720marcusquinn merged 1 commit intomainfrom
Conversation
…solidate DB queries Sanitise jq-parsed values with integer validation before SQL interpolation and arithmetic, preventing syntax errors from malformed SonarCloud API responses. Consolidate three separate DB queries for previous snapshot into a single query returning all columns. Closes #3702
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
|
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. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
🔍 Code Quality Report�[0;35m[MONITOR]�[0m Code Review Monitoring Report �[0;34m[INFO]�[0m Latest Quality Status: �[0;34m[INFO]�[0m Recent monitoring activity: 📈 Current Quality Metrics
Generated on: Sat Mar 7 21:22:13 UTC 2026 Generated by AI DevOps Framework Code Review Monitoring |
|
|
@coderabbitai review Review bots were rate-limited when this PR was created (affected: coderabbitai gemini-code-assist). Requesting a review retry. |
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
|
🧠 Learnings used✅ Actions performedReview triggered.
|
|
This PR appears orphaned — no active worker process found and no activity for 7+ hours. Flagging for re-dispatch. |
|
This PR appears orphaned — no active worker process found and no activity for 14+ hours. Flagging for re-dispatch. If work is still in progress, remove the |
|
Merging via pulse supervisor.
|



Summary
total,critical,high,medium,low) before SQL interpolation and bash arithmetic — prevents SQL syntax errors and arithmetic failures from malformed SonarCloud API responsesSELECTqueries for the previous regression snapshot with a single query returning all columns via pipe-separated output, reducing DB round-tripsContext
Addresses critical and major findings from CodeRabbit review on PR #1418:
Duplicate— already fixed in current codecheck-regressioncase arm (dead code)— pulse.sh is archived, not applicableunified_task_creatorvariable reuse in pulse.shCloses #3702