chore: inline self-contained house Renovate config#8
Conversation
Replace bare config:recommended with full inlined house config built on public presets only (config:best-practices + dependencyDashboard + semanticCommits + vuln alerts). Self-contained — no cross-repo refs.
Walkthrough
ChangesRenovate Configuration Overhaul
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@renovate.json`:
- Line 11: Remove the redundant empty ignoreDeps array from the renovate.json
configuration file. The ignoreDeps property with an empty array is the default
behavior in Renovate, so including it explicitly only adds configuration noise
without providing functional value. Simply delete the "ignoreDeps": [] entry
from the configuration object to clean up the configuration file.
- Line 4: The renovate.json configuration has redundant label configuration due
to the `:enableVulnerabilityAlertsWithLabel(security)` preset on line 4 already
setting the "security" label for vulnerability alerts. Locate the
`vulnerabilityAlerts` object in the configuration and remove the `"labels":
["security"]` property from it, while preserving the `schedule` and `automerge`
properties that should remain in that object.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: b2b0b9fe-8c36-4a1a-bb89-000597764714
📒 Files selected for processing (1)
renovate.json
| "config:recommended" | ||
| ] | ||
| "description": "Inlined house Renovate config — self-contained, public presets only.", | ||
| "extends": ["config:best-practices", ":dependencyDashboard", ":semanticCommits", ":enableVulnerabilityAlertsWithLabel(security)"], |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Description: Check Renovate documentation for the enableVulnerabilityAlertsWithLabel preset
# Search for any local documentation or comments about vulnerability alert configuration
rg -i -C3 'vulnerabilityAlerts|enableVulnerabilityAlerts' --glob '*.md'Repository: baker-scripts/Scripts
Length of output: 47
🌐 Web query:
What does the Renovate preset :enableVulnerabilityAlertsWithLabel(security) configure by default?
💡 Result:
The Renovate preset:enableVulnerabilityAlertsWithLabel(security) enables vulnerability alerts and configures Renovate to add a PR label list containing exactly "security" (i.e., it replaces any existing list of PR labels with ["security"]). [1] Concretely, the preset sets vulnerabilityAlerts.enabled = true and vulnerabilityAlerts.labels = ["{{arg0}}"], so with arg0 = security it becomes vulnerabilityAlerts.labels = ["security"]. [1]
Citations:
🏁 Script executed:
cat -n renovate.json | head -20Repository: baker-scripts/Scripts
Length of output: 1777
Remove redundant "labels": ["security"] from line 12.
The preset :enableVulnerabilityAlertsWithLabel(security) on line 4 already configures vulnerability alerts with the "security" label. The explicit vulnerabilityAlerts configuration on line 12 redundantly specifies "labels": ["security"] again. Keep the schedule and automerge settings, but remove the redundant labels property.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@renovate.json` at line 4, The renovate.json configuration has redundant label
configuration due to the `:enableVulnerabilityAlertsWithLabel(security)` preset
on line 4 already setting the "security" label for vulnerability alerts. Locate
the `vulnerabilityAlerts` object in the configuration and remove the `"labels":
["security"]` property from it, while preserving the `schedule` and `automerge`
properties that should remain in that object.
| "labels": ["dependencies", "renovate"], | ||
| "rangeStrategy": "bump", "separateMajorMinor": true, "separateMinorPatch": false, | ||
| "automerge": false, "platformAutomerge": false, "rebaseWhen": "behind-base-branch", | ||
| "branchPrefix": "renovate/", "ignoreDeps": [], |
There was a problem hiding this comment.
🧹 Nitpick | 🔵 Trivial | 💤 Low value
Remove redundant empty ignoreDeps array.
The ignoreDeps: [] declaration is redundant since an empty array is the default value. Unless this is included explicitly for documentation purposes or to facilitate future modifications, consider removing it to reduce configuration noise.
♻️ Proposed cleanup
- "branchPrefix": "renovate/", "ignoreDeps": [],
+ "branchPrefix": "renovate/",🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@renovate.json` at line 11, Remove the redundant empty ignoreDeps array from
the renovate.json configuration file. The ignoreDeps property with an empty
array is the default behavior in Renovate, so including it explicitly only adds
configuration noise without providing functional value. Simply delete the
"ignoreDeps": [] entry from the configuration object to clean up the
configuration file.
Replace bare
config:recommendedwith the full inlined house Renovate config.config:best-practices,:dependencyDashboard,:semanticCommits, vuln alerts).github>.../local>...cross-repo references.Validated with
renovate-config-validator.Summary by CodeRabbit
Chores