Skip to content

feat: investigate Dependabot security alerts#35

Draft
groovecoder wants to merge 6 commits intomainfrom
investigate-dependabot-security-alerts
Draft

feat: investigate Dependabot security alerts#35
groovecoder wants to merge 6 commits intomainfrom
investigate-dependabot-security-alerts

Conversation

@groovecoder
Copy link
Copy Markdown
Member

Add alert discovery in sweep, investigation workflow, and post-action routing (bump PR for non-affected, private fork for affected).

Closes #4

Add alert discovery in sweep, investigation workflow, and post-action
routing (bump PR for non-affected, private fork for affected).

Closes #4
Repos can opt in via `investigate.dismiss_not_affected: true` in their
.blender/blender.yml. When enabled, BLEnder dismisses Dependabot alerts
as "inaccurate" instead of opening bump PRs for non-affected packages.

Every investigation now posts a row to a tracking issue on the target
repo. The issue is created on the first run, then reused. Each row is
a comment (not a body edit) to avoid races when many workflows run
concurrently.
Comment thread scripts/sweep.py
Comment thread scripts/sweep.py Outdated
Comment thread scripts/sweep.py
Comment thread scripts/gather-alert-context.sh Outdated
Comment thread scripts/post_alert_action.py Outdated
Comment thread scripts/post_alert_action.py Outdated
Comment thread scripts/post_alert_action.py Outdated
Replace tracking issue with JSON artifact to avoid exposing alert
details on public repos. Remove open_bump_pr (no file changes made it
useless). Rename "not affected" to "unaffected" everywhere. Extract
shared sanitize_for_prompt into scripts/sanitize.sh. Fix "dedup"
jargon in sweep.py comments.
Comment thread scripts/post_alert_action.py Outdated
Replace JSON artifact with a styled HTML report. The report shows
alert metadata, verdict, and source code snippets for each vulnerable
path identified during investigation. Styled like a coverage report
for easy in-browser review.
Comment thread scripts/post_alert_action.py Outdated
Move read_code_snippet, render_html, and write_summary out of
post_alert_action.py into a dedicated module. Keeps the action
script focused on orchestration.
zizmor flagged workflow-level actions:write as overly broad. Only the
investigate job needs it for upload-artifact.
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.

feat(security): add workflow to investigate impact of dependabot security alerts

1 participant