Skip to content

fix: include release notes in Dependabot PR reviews#74

Merged
Spiev merged 1 commit intomainfrom
fix/pr-review-security-detection
Feb 17, 2026
Merged

fix: include release notes in Dependabot PR reviews#74
Spiev merged 1 commit intomainfrom
fix/pr-review-security-detection

Conversation

@Spiev
Copy link
Owner

@Spiev Spiev commented Feb 17, 2026

Summary

  • Dependabot PR reviews now include release notes, so security advisories (GHSA/CVE) and breaking changes are detected
  • Adds explicit instruction for Claude to check for security-relevant content
  • Increases max_tokens for Dependabot reviews (300 → 500) to avoid truncation

Context: PR #72 (paperless-ngx security fix for GHSA-x395-6h48-wr8v) was incorrectly classified as "Security: None" because the review prompt only included the title and changed files, not the PR body with release notes.

Test plan

  • Merge and verify next Dependabot PR gets reviewed with release notes context
  • Check that security-relevant updates are flagged correctly

🤖 Generated with Claude Code

The review prompt for Dependabot PRs only included the title and changed
files, missing security advisories (GHSA/CVE) and breaking changes from
the PR body. This caused PR #72 (paperless-ngx security fix) to be
classified as "Security: None".

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link

🔍 Code Review (by Claude)

Summary: Adds release notes to Dependabot PR reviews to catch security advisories and breaking changes that were previously missed.

Security: No issues.

Issues:

  • Hardcoded 3000 char truncation (pr_data['body'][:3000]) could cut off critical security info mid-sentence. Consider truncating at word boundaries or increasing limit.

Suggestions:

  • The new prompt instruction is verbose. Consider moving to system prompt or condensing to avoid token waste on every Dependabot PR.
  • 500 tokens may still be insufficient if multiple dependencies are updated. Consider dynamic sizing based on PR complexity.

Review generated by Claude AI. Please use your judgment for final approval.

@Spiev Spiev merged commit 28ba9db into main Feb 17, 2026
5 checks passed
@Spiev Spiev deleted the fix/pr-review-security-detection branch February 17, 2026 09:11
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.

1 participant