feat: write back to Dependabot when security findings are auto-dismissed#440
Merged
jeanduplessis merged 3 commits intomainfrom Feb 24, 2026
Merged
feat: write back to Dependabot when security findings are auto-dismissed#440jeanduplessis merged 3 commits intomainfrom
jeanduplessis merged 3 commits intomainfrom
Conversation
Contributor
Code Review SummaryStatus: No Issues Found | Recommendation: Merge The PR adds Dependabot writeback functionality to the auto-dismiss service. When findings are auto-dismissed (via triage, sandbox, or bulk), the corresponding Dependabot alert on GitHub is also dismissed via the API. Key observations:
Files Reviewed (2 files)
|
a9df5bd to
966713a
Compare
When the security agent auto-dismisses a finding (via triage or sandbox analysis), also dismiss the corresponding Dependabot alert on GitHub. Previously, only manual dismissals wrote back to Dependabot. This change adds the same writeback to the auto-dismiss flow (maybeAutoDismissAnalysis) and the bulk auto-dismiss flow (autoDismissEligibleFindings). The writeback is wrapped in error handling so that a GitHub API failure does not break the auto-dismiss flow — errors are logged to Sentry but the local dismissal still succeeds.
5b08274 to
8daba2f
Compare
marius-kilocode
approved these changes
Feb 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
maybeAutoDismissAnalysisand the bulkautoDismissEligibleFindingsflowChanges
auto-dismiss-service.ts: AddedwritebackDependabotDismissal(fetches finding + integration, callsdismissDependabotAlert) andsafeWritebackDependabotDismissal(error-safe wrapper). Integrated into sandbox auto-dismiss, triage auto-dismiss, and bulk auto-dismiss paths.auto-dismiss-service.test.ts: New test file covering writeback for sandbox/triage auto-dismiss, skip conditions (non-dependabot source, missing integration, invalid source_id, bad repo name), and resilience when GitHub API fails.