fix(#3482): exclude vulnerability PRs from grouped patch updates - #3485
fix(#3482): exclude vulnerability PRs from grouped patch updates#3485fullsend-ai-coder[bot] wants to merge 1 commit into
Conversation
Add packageRules entry to renovate.json that sets groupName and groupSlug to null for vulnerability alerts on gomod packages. This prevents Renovate from deduplicating standalone security PRs against grouped patch bundles that may have failing CI. Also add vulnerabilityAlerts config with security/renovate labels and unrestricted scheduling so security fixes are processed immediately. This addresses a recurring pattern where approved security PRs (e.g., #3473-3475 for CVE-2026-71556/CVE-2026-71557, and earlier #3250-3253 for go-git v5.19.1) were autoclosed by Renovate in favor of months-stale grouped PRs with failing checks. Note: pre-commit could not run in sandbox (network access restricted). No Go code changes — config-only change. Closes #3482
|
🤖 Finished Review · ✅ Success · Started 2:29 PM UTC · Completed 2:42 PM UTC Commit: |
Codecov Report✅ All modified and coverable lines are covered by tests.
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
ReviewFindingsMedium
Low
Labels: PR modifies Renovate configuration for vulnerability alert handling |
| "helpers:pinGitHubActionDigests" | ||
| ] | ||
| ], | ||
| "packageRules": [ |
There was a problem hiding this comment.
[medium] logic error
The packageRules entry setting groupName: null and groupSlug: null for isVulnerabilityAlert: true is redundant. Renovate already internally forces groupName: null for vulnerability alerts via its built-in force block. The actual problem (autoclosing of standalone security PRs) is caused by Renovate's update deduplication logic, not by grouping configuration. This rule will not prevent the autoclosing behavior described in issue #3482.
Suggested fix: Investigate whether the autoclosing is caused by Renovate's branch/update deduplication logic. Consider: (1) filing a Renovate issue about deduplication between vulnerability PRs and grouped PRs with failing CI, (2) splitting the stalled grouped PR (#3019), or (3) keeping this rule with documentation noting its redundancy.
| ], | ||
| "packageRules": [ | ||
| { | ||
| "description": "Keep vulnerability/security PRs standalone so they are not grouped with stalled patch bundles", |
There was a problem hiding this comment.
[low] scope limitation
The packageRules entry restricts the vulnerability ungrouping override to matchManagers: [gomod] only, while the parent preset groups dependencies across multiple managers. Since the entire packageRules entry is redundant, this scope limitation is moot.
What:
Add a
packageRulesentry torenovate.jsonthat excludes vulnerability/security PRs from being grouped with patch update bundles. Also addvulnerabilityAlertsconfiguration for proper labeling and scheduling.Why:
Standalone security PRs (e.g., #3473-3475 for CVE-2026-71556 and CVE-2026-71557) have been repeatedly autoclosed by Renovate's deduplication logic because the same version bump exists in a grouped patch PR (#3019) that has been failing CI for months. This is a recurring pattern — the same thing happened with go-git v5.19.1 in April 2026 (#3250-3253 autoclosed, manual PR #3335 needed weeks later). The fix ensures security PRs remain standalone and can merge independently of stalled grouped PRs.
Tickets:
#3482
Closes #3482
Post-script verification
agent/3482-exclude-vuln-from-group)61ed8a355ace58359e1255b0ea2ce01a9d378f50..HEAD)