Skip to content

fix: treat PR scan fetch errors as inconclusive#7

Merged
homanp merged 3 commits into
mainfrom
homanp/pr-scan-fetch-error-inconclusive-a2a5
Apr 27, 2026
Merged

fix: treat PR scan fetch errors as inconclusive#7
homanp merged 3 commits into
mainfrom
homanp/pr-scan-fetch-error-inconclusive-a2a5

Conversation

@homanp
Copy link
Copy Markdown
Contributor

@homanp homanp commented Apr 25, 2026

Summary

  • Classify Brin PR scan_error findings as inconclusive instead of blocking
  • Forward the GitHub installation token to the Brin PR scan API in the X-GitHub-token header so private PRs can be fetched
  • Clear stale PR scan labels and comments when a scan becomes inconclusive
  • Add focused policy, label, and Brin API request tests

Testing

  • npm test
  • npm run typecheck
Open in Web Open in Cursor 

cursoragent and others added 2 commits April 25, 2026 21:18
Co-authored-by: Ismail Pelaseyed <homanp@users.noreply.github.com>
Co-authored-by: Ismail Pelaseyed <homanp@users.noreply.github.com>
@homanp homanp marked this pull request as ready for review April 25, 2026 21:25
@superagent-security superagent-security Bot added contributor:verified Contributor passed trust analysis. pr:verified PR passed security analysis. labels Apr 25, 2026
@homanp homanp changed the title Treat PR scan fetch errors as inconclusive fix: treat PR scan fetch errors as inconclusive Apr 27, 2026
@homanp
Copy link
Copy Markdown
Contributor Author

homanp commented Apr 27, 2026

@cursor review

@homanp homanp self-assigned this Apr 27, 2026
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 5574309. Configure here.

@homanp homanp merged commit 688bfe4 into main Apr 27, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor:verified Contributor passed trust analysis. pr:verified PR passed security analysis.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants