Skip to content

ci: reduce pr-severity bot comment spam#10553

Open
Roasbeef wants to merge 1 commit intomasterfrom
claude-chill
Open

ci: reduce pr-severity bot comment spam#10553
Roasbeef wants to merge 1 commit intomasterfrom
claude-chill

Conversation

@Roasbeef
Copy link
Member

@Roasbeef Roasbeef commented Feb 4, 2026

The PR severity classifier currently posts a new comment on every
push to a PR, even when the classification hasn't changed. On active
PRs with many pushes this creates a wall of identical bot comments.

This change updates the classifier prompt to check for existing bot
comments (via the pr-severity-bot HTML marker) and compare the newly
determined severity against the PR's current severity label before
deciding whether to comment.

A comment is now only posted when the bot hasn't commented before or
when the severity actually changed. Labels are still always kept in
sync regardless of whether a comment is posted.

When severity does change between classifications, the new comment
includes a banner showing the old and new severity levels so
reviewers can see what shifted and why.

Only post a severity classification comment when the bot hasn't
commented before or when the severity actually changed. Previously
every push (synchronize event) would post a new comment even if the
classification was identical.

The prompt now instructs the classifier to:
- Check for existing bot comments via the pr-severity-bot marker
- Compare the new severity against the existing severity label
- Skip commenting if both match, while still ensuring labels are correct
- Include a severity changed banner when re-commenting due to a change
@gemini-code-assist
Copy link

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@lightninglabs-deploy
Copy link
Collaborator

🟢 PR Severity: LOW

CI/CD Configuration | 1 file | 37 lines changed

🟢 Low (1 file)
  • .github/workflows/pr-severity.yml - CI/CD workflow configuration for PR severity bot

Analysis

This PR modifies only the GitHub Actions workflow configuration for the PR severity bot itself. Changes to CI/CD configuration files in .github/* are classified as LOW severity as they:

  • Don't affect runtime behavior of lnd
  • Don't touch any critical paths or consensus logic
  • Are limited to automation/tooling improvements
  • Can be tested and validated through the CI pipeline itself

The change appears to reduce comment spam from the severity bot, which is a pure quality-of-life improvement for the development workflow.

No severity bumps applied: Single file change with <500 lines.


To override, add a severity-override-{critical,high,medium,low} label.

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