chrisns / repomanager / 30814957982

86%

Build Type

push

github

Committed by web-flow

Commit Message

Only act when there is something to do (#1620)

The bot was mutating every repo on every tick regardless of state.

Templated files: planFiles diffed against the default branch, but the fix
ships as a PR from `repomanager_files`. While that PR sat unmerged the drift
was permanent, so the applier force-pushed an identical commit every hour on
every repo with an open PR. Skip when the open PR already carries exactly the
desired content — the ball is in the reviewer's court, not ours.

Flags: vulnerabilityAlerts, automatedSecurityFixes and
privateVulnerabilityReporting were unconditional writes. Diff them against
actual state first — same API budget, no mutation once converged. Never
propose privateVulnerabilityReporting on a private repo (public-repo feature),
and treat a private repo with no security_and_analysis block as having no
Advanced Security rather than failing open into a doomed apply every tick.

Consent issues: search is GitHub's most throttled API and octokit serialises
it to ~2s a call. It ran per repo per tick, twice, almost always to discover
nothing existed — now only immediately before creating an issue. Also skip the
closed-issue list on the no-drift path, skip label creation on repos that never
need one, and skip the write entirely when the rendered body already matches:
that identical PUT came straight back as a billed issues.edited webhook.

Claude-Session: https://claude.ai/code/session_013MKCzima2XjGrHTYkY7F2y

Coverage Stats

435 of 553 branches covered (78.66%)

Branch coverage included in aggregate %.

45 of 47 new or added lines in 2 files covered. (95.74%)

3 existing lines in 1 file now uncovered.

699 of 766 relevant lines covered (91.25%)

21.77 hits per line