feat: retry logic, credit guard, and cost estimation for Higgsfield automator (t236.1) #962
Conversation
- Add withRetry() wrapper with exponential backoff for transient failures (network timeouts, page load errors). Non-transient errors (content policy, missing assets) are not retried. Video/lipsync get max 1 retry (expensive). - Add credit cost map per operation type (image:2, video:20, lipsync:10, etc.) - Add credit guard that checks cached credits before expensive operations. Blocks if insufficient credits, warns if low. Use --force to override. - Cache credit info for 10 minutes after 'credits' command runs - Add --force, --dry-run, --no-retry CLI flags - Wrap all 18 generation commands with retry in main dispatch - Free commands (login, credits, screenshot, download, assets) skip guard/retry
|
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. ✨ 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: Tue Feb 10 18:13:37 UTC 2026 Generated by AI DevOps Framework Code Review Monitoring |
|
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
…s-contamination
The session_id, worktree, branch, log_file, and pr_url variables in
cmd_transition() were not declared as local. When the pulse cycle calls
cmd_transition() in a loop for multiple tasks, these variables leaked
between calls via the shell's global scope. A call with --branch or
--pr-url would set the global, and subsequent calls without those flags
would still see the stale value via ${branch:-} / ${pr_url:-} checks,
overwriting unrelated tasks' DB rows.
This was the root cause of all PR cross-contamination (6 historical
duplicates, plus ongoing backlog-19 contamination where PR #962 was
linked to 5 unrelated tasks).
…s-contamination (#967) The session_id, worktree, branch, log_file, and pr_url variables in cmd_transition() were not declared as local. When the pulse cycle calls cmd_transition() in a loop for multiple tasks, these variables leaked between calls via the shell's global scope. A call with --branch or --pr-url would set the global, and subsequent calls without those flags would still see the stale value via ${branch:-} / ${pr_url:-} checks, overwriting unrelated tasks' DB rows. This was the root cause of all PR cross-contamination (6 historical duplicates, plus ongoing backlog-19 contamination where PR #962 was linked to 5 unrelated tasks).
1 participant
Summary
Phase 1 of Higgsfield automator production hardening (t236). Adds three reliability features:
withRetry()wrapper handles transient failures (network timeouts, page load errors). Non-transient errors (content policy violations, missing assets) are not retried. Video/lipsync operations get max 1 retry due to high credit cost.--forceto override.New CLI flags
--force— Override credit guard (proceed even with low credits)--dry-run— Navigate and configure but don't click Generate (Phase 5 will implement)--no-retry— Disable automatic retry on failureTesting
CREDIT_GUARD: Insufficient credits. Need ~2, have 1.--forcebypasses the guard and proceedscreditscommand, valid for 10 minutes