Skip to content

[Chore] Reduce Renovate review noise#144

Merged
edelauna merged 4 commits into
mainfrom
fix/renovate-review-noise-3s9rcmdfp0p41
May 16, 2026
Merged

[Chore] Reduce Renovate review noise#144
edelauna merged 4 commits into
mainfrom
fix/renovate-review-noise-3s9rcmdfp0p41

Conversation

@roomote

@roomote roomote Bot commented May 16, 2026

Copy link
Copy Markdown
Contributor

What problem this solves

Reviewing dependency updates one by one is noisy in this repo, especially across AI provider SDKs and mixed frontend, tooling, and CI changes.

Running Dependabot alongside Renovate also splits dependency update ownership, and some repo-owned Node version literals had already drifted behind the current 20.20.2 baseline.

Why this change was made

This switches the repo to Renovate as the single updater and teaches it to group updates by operational domain instead of opening a long tail of one-off PRs.

It also aligns the remaining repo-owned Node literals to 20.20.2 now, while adding regex-based managers so those versions stay in sync in future Renovate runs.

User impact

Maintainers should get fewer, better-scoped dependency PRs: grouped AI/provider, frontend/UI, tooling, and GitHub Actions updates, with major updates continuing through the normal PR and CI flow and low-risk @types/* patch updates eligible for automerge.

Node version bumps should also stay consistent across package manifests, setup scripts, Dockerfiles, and GitHub Actions.

@roomote

roomote Bot commented May 16, 2026

Copy link
Copy Markdown
Contributor Author

No actionable issues found in the latest update. The separate major-update thread remains open intentionally because the current product direction is to rely on the normal PR and CI flow for majors. See task

  • Align the newly managed Node literals to the current 20.20.2 baseline in this PR so CI and evals do not stay pinned to 20.19.2 after merge.

@codecov

codecov Bot commented May 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Comment thread renovate.json
Comment thread renovate.json Outdated
"schedule": ["* 21-23 * * 0"]
},
{
"description": "Require approval before opening major React stack updates.",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing the top-level major.dependencyDashboardApproval means every non-React major update will start opening PRs immediately again. That leaves majors for the rest of the frontend/tooling/AI/actions stack ungated, which brings back the review noise this config is trying to reduce.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Leaving this one unresolved intentionally. Per the latest product direction on this PR, major updates should open normally and rely on the usual PR/CI path rather than dashboard approval, so the broad ungated-major behavior is expected here.

@edelauna edelauna left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updating renovate config to see if it helps improve dependency mgmt experience.

@edelauna edelauna merged commit 69ef6a3 into main May 16, 2026
9 checks passed
@edelauna edelauna deleted the fix/renovate-review-noise-3s9rcmdfp0p41 branch May 16, 2026 16:18
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.

2 participants