Skip to content

chore(shared): fix the release config for breaking changes#80

Merged
designcode merged 1 commit intomainfrom
fix/release-config
Apr 8, 2026
Merged

chore(shared): fix the release config for breaking changes#80
designcode merged 1 commit intomainfrom
fix/release-config

Conversation

@designcode
Copy link
Copy Markdown
Collaborator

@designcode designcode commented Apr 8, 2026

Note

Medium Risk
Changes semantic-release commit analysis rules, which can alter versioning/publishing behavior across packages. Risk is mainly around missed or unexpected releases if breaking-change commits are scoped differently than assumed.

Overview
Updates release.config.base.cjs semantic-release releaseRules to explicitly ignore breaking-change commits whose scope does not match the current package.

This prevents a breaking change in another package’s scope from triggering a major release for this package.

Reviewed by Cursor Bugbot for commit 4cba432. Bugbot is set up for automated code reviews on this repo. Configure here.

@greptile-apps
Copy link
Copy Markdown

greptile-apps bot commented Apr 8, 2026

Greptile Summary

This PR adds a missing catch-all releaseRules entry so that breaking changes scoped to other packages (e.g. feat(cli)!:) don't accidentally trigger a major version bump for the current package. The new rule { scope: \!(${scope})`, breaking: true, release: false }mirrors the already-existing positive-scope rule{ scope, breaking: true, release: 'major' }` and is consistent with the rest of the negation block.

Confidence Score: 5/5

Safe to merge — the change is a one-line addition that completes an existing symmetrical pattern with no side effects.

No P0 or P1 issues found. The rule is logically correct, mirrors the established pattern for scope-negation rules, and all four package-level configs delegate to this base, so the fix propagates consistently across the monorepo.

No files require special attention.

Vulnerabilities

No security concerns identified.

Important Files Changed

Filename Overview
release.config.base.cjs Adds a negation catch-all rule so breaking changes from other package scopes don't trigger a major release in the current package; mirrors the existing positive-scope pattern correctly.

Reviews (1): Last reviewed commit: "chore(shared): fix the release config fo..." | Re-trigger Greptile

@designcode designcode merged commit 03b00af into main Apr 8, 2026
2 checks passed
@designcode designcode deleted the fix/release-config branch April 8, 2026 18:38
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

🎉 This PR is included in version 2.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 3.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants