Skip to content

fix: bound the sweep issue body to the GitHub limit - #139

Merged
abrichr merged 1 commit into
mainfrom
fix/sweep-issue-body-cap
Aug 22, 2026
Merged

fix: bound the sweep issue body to the GitHub limit#139
abrichr merged 1 commit into
mainfrom
fix/sweep-issue-body-cap

Conversation

@abrichr

@abrichr abrichr commented Aug 22, 2026

Copy link
Copy Markdown
Member

The first scheduled sweep died at createIssue: GraphQL rejected the body over 65536 characters, because the org holds ~50 stranded branches and the table was unbounded.

  • Cap the remote table at 40 oldest rows with a per-repo summary of the rest.
  • Hard-guard the final body at 60000 chars with an explicit truncation note.
  • Tests: cap fires and hides nothing silently; a pathological wide fixture stays under the API limit.

Verified against live data: body 6370 chars, 40 shown, +10 summarized.

@abrichr
abrichr merged commit 365453f into main Aug 22, 2026
7 checks passed
@abrichr
abrichr deleted the fix/sweep-issue-body-cap branch August 22, 2026 16:48
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