ci(renovate): group @connectrpc/* so the exact peer moves as one PR - #64
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds a
@connectrpc/**group rule to the fleet Renovate preset so@connectrpc/connectand@connectrpc/connect-webmove as one PR.@connectrpc/connect-webdeclares an exact peer on@connectrpc/connect(2.2.0 requires exactly 2.2.0), so a PR bumping either half alone cannot
regenerate the lockfile: it is born red on
renovate/artifactsand no rebasefixes it. Two such PRs are open on trellis right now (#428 since 2026-09-10,
#447 since 2026-09-14), both red, both consuming the repo's two-open-PR cap.
This is the same failure shape as
@vitest/**, already grouped withvitestin this file for exactly this reason; the new rule's
descriptionrecords thecause so the next reader does not have to rediscover it.
Renovate closes #428 and #447 on its next tick and raises one grouped PR in
their place — they are not expected to self-heal.
Linked Issue
Fixes #63
Type of Change
Risk
Low, and confined to how Renovate batches npm PRs. The rule only adds a
groupName; it changes no version, no automerge policy (the existingpatch/minor automerge rule still applies to the group) and no other package.
@bufbuild/protobufis deliberately not in the group — its peer is acaret range (
^2.7.0), so it resolves independently and does not need to movein lockstep.
Testing Evidence
The peer that causes the failure:
The validator CI runs (
ci.yml:76), on this branch:The
WARN: Config migration necessaryis pre-existing and untouched by thischange — the same validator on
origin/main's copy of the file:(It is the
fileMatch→managerFilePatternsrename in the twocustomType: regexmanagers, out of scope here.)JSON well-formedness:
Security and Release Checklist
default.jsononly; no.github/workflowsfile touched)