Skip to content

fix: cap GitHub Actions annotations at 50 with overflow notice#293

Merged
mcarvin8 merged 1 commit into
mainfrom
fix/github-actions-annotation-limit
May 29, 2026
Merged

fix: cap GitHub Actions annotations at 50 with overflow notice#293
mcarvin8 merged 1 commit into
mainfrom
fix/github-actions-annotation-limit

Conversation

@mcarvin8
Copy link
Copy Markdown
Owner

Summary

  • generateGitHubActions previously emitted one ::warning per uncovered line with no upper bound
  • GitHub Actions silently truncates annotations beyond its per-step limit, giving no indication to the user that lines were dropped
  • Annotations are now capped at 50. When the total exceeds the cap, a ::notice is appended stating how many additional uncovered lines were not shown
  • The summary ::notice and handler data are unaffected — the full count is still reported accurately in the summary line

Test plan

  • All 107 unit tests pass (npm run test:only)
  • 100% branch coverage (196/196)
  • New test: 60 uncovered lines → 50 ::warning lines + truncation notice citing 10 remaining
  • New test: 51 uncovered lines → singular "line" in truncation notice
  • New test: ≤50 uncovered lines → no truncation notice emitted

🤖 Generated with Claude Code

generateGitHubActions previously emitted one ::warning per uncovered
line with no upper bound. GitHub Actions silently truncates annotations
beyond its per-step limit, giving no indication to the user that lines
were dropped.

Annotations are now capped at 50. When the total exceeds the cap, a
::notice is appended stating how many additional lines were not shown.
The summary ::notice and handler data are unaffected — the full count
is still reported in the summary line.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
...c/transformers/generators/generateGitHubActions.ts 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mcarvin8 mcarvin8 merged commit ad3bdff into main May 29, 2026
20 checks passed
@mcarvin8 mcarvin8 deleted the fix/github-actions-annotation-limit branch May 29, 2026 13:47
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.

2 participants