fix(reconciler): deduplicate checkPRFeedback and checkDispatchCircuitBreaker calls

[jrf0110]

Summary

Eliminates redundant API and SQL calls in two hot paths:

• B4 (actions.ts): Hoists checkPRFeedback above both the auto-resolve and auto-merge blocks in poll_pr. A needsFeedbackCheck flag determines whether the call is needed, and the single result is reused for both code paths. This removes the duplicate GitHub API call that occurred when both auto_resolve_pr_feedback and auto_merge_delay_minutes were configured.

• B5 (reconciler.ts): Moves checkDispatchCircuitBreaker(sql) to the top-level reconcile() function. The boolean result is computed once and passed down to reconcileBeads and reconcileReviewQueue via optional circuitBreakerOpen parameters. Both functions retain backward compatibility by falling back to inline evaluation when called standalone. The notify_mayor action emission is also moved to reconcile() to ensure it fires exactly once per reconcile pass.

Verification

• Typecheck passes
• All existing tests pass (2 pre-existing failures in client.test.ts are unrelated)
• Manual code review verified behavioral equivalence for all edge cases (auto-resolve only, auto-merge only, both, neither)

Visual Changes

N/A

Reviewer Notes

• Both changes are pure optimizations — no behavioral changes expected.
• reconcileBeads and reconcileReviewQueue accept an optional circuitBreakerOpen parameter for backward compatibility when called outside of reconcile().
• The needsFeedbackCheck condition in actions.ts is the logical OR of the two original guard conditions, ensuring checkPRFeedback is called exactly when either block needs it.

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)

• cloudflare-gastown/src/dos/town/actions.ts
• cloudflare-gastown/src/dos/town/reconciler.ts

---

_{Reviewed by gpt-5.4-20260305 · 266,375 tokens}