Skip to content

feat(github): add configurable floor for progress-comment edit cadence#759

Open
aparajon wants to merge 1 commit into
armand/progress-comment-decayfrom
armand/progress-comment-min-interval
Open

feat(github): add configurable floor for progress-comment edit cadence#759
aparajon wants to merge 1 commit into
armand/progress-comment-decayfrom
armand/progress-comment-min-interval

Conversation

@aparajon

@aparajon aparajon commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

Why this matters

The built-in edit cadence is a fixed policy. During a GitHub rate-limit incident — or on a server hosting many concurrent long-running applies — operators need one valve that slows every apply's progress-comment traffic at once, without a code change or redeploy of new logic.

What it does

Adds a server config setting that floors the built-in age-decayed cadence:

progress_comment_min_edit_interval: 2m  # Go duration; default unset (no floor)
  • No progress comment is edited more often than the configured interval, regardless of the built-in cadence.
  • State changes still publish immediately — the floor spaces out progress noise, not state transitions.
  • The cadence the progress comment advertises in its "Last updated" footer reflects the floor, so PR readers see the configured interval rather than wondering why updates slowed down.
  • Invalid or negative values are rejected at config validation; an invalid value read at runtime logs a warning and falls back to the built-in cadence.
  • Documented in docs/configuration.md; users needing high-resolution progress use the CLI, which polls the server directly and is unaffected.

Stacked on #758.

🤖 Generated with Claude Code

@aparajon
aparajon force-pushed the armand/progress-comment-decay branch from 3efffda to 8e1ab8f Compare July 18, 2026 21:42
@aparajon
aparajon force-pushed the armand/progress-comment-min-interval branch from 370766e to 4394bda Compare July 18, 2026 21:42
@aparajon
aparajon force-pushed the armand/progress-comment-decay branch from 8e1ab8f to 0710f5b Compare July 18, 2026 23:14
@aparajon
aparajon force-pushed the armand/progress-comment-min-interval branch from 4394bda to a6ac3f4 Compare July 18, 2026 23:14
Add progress_comment_min_edit_interval server config: a Go duration
string that floors the built-in age-decayed progress-comment edit
cadence. Operators can set it during a GitHub rate-limit incident (or
permanently on servers hosting many concurrent long-running applies) to
bound EditComment traffic without a code change. State changes still
publish immediately; the floor only spaces out progress-percentage
edits. Empty or unset means no floor, and an invalid value is rejected
at config validation.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@aparajon
aparajon force-pushed the armand/progress-comment-decay branch from 0710f5b to 389522a Compare July 19, 2026 00:04
@aparajon
aparajon force-pushed the armand/progress-comment-min-interval branch from a6ac3f4 to b436d45 Compare July 19, 2026 00:06
@aparajon
aparajon marked this pull request as ready for review July 20, 2026 02:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant