forked from cleveragents/cleveragents-core
freemo
52bfb1658a
Every finite retry cap in the agent system has been replaced with infinite retry + diagnostic logging. The system now self-corrects indefinitely rather than giving up and leaving work incomplete. Changes across 9 agent definitions: - ca-pr-self-reviewer: Merge retry changed from `for attempt in 1..3` to `WHILE True` with exponential backoff (10s → 5 min cap). Only exits the retry loop on success or merge conflict (which requires implementor rebase, not more retries). - ca-continuous-pr-reviewer: Removed the `attempts >= 5` give-up block that posted "manual intervention required" and stopped retrying. PRs are now retried indefinitely with a diagnostic comment every 10 attempts. - issue-implementor: Removed the "2 retries then permanently failed" budget. Failed issues are always re-queued. Every 3 consecutive failures posts a diagnostic comment on the Forgejo issue and resets the approach (clears prior attempt context to break out of repeating failure patterns). Removed the `skipped` list entirely — no issue is ever skipped. - product-builder: Forgejo API retry changed from "3 times then halt" to "indefinitely with exponential backoff, cap 5 min." Only halts on auth revocation (HTTP 401/403). Worker failure docs updated to reflect the issue-implementor's infinite retry policy. - ca-pr-checker, ca-timeline-updater, ca-spec-updater, ca-project-bootstrapper, ca-docs-writer: Git push conflict retry changed from "up to 3 times" to "indefinitely with rebase." Added reclone fallback: after every 5 consecutive push failures, the clone is deleted and recreated fresh to recover from corrupted git state. NOT changed (already correct): - ca-subtask-loop: 3-retry transient error classification is a heuristic for deciding when to escalate model tier. The loop itself already says "NEVER give up" and runs indefinitely at opus tier. - ca-human-liaison: Already says "Never exit voluntarily."