Skip to content

Conversation

@designcode
Copy link
Collaborator

@designcode designcode commented Jan 23, 2026

Note

Refines release process to avoid cross-package releases and ensures CI dry-run tests the correct branch.

  • Update release.config.base.cjs to apply semantic-release rules only to the package scope and block releases from other scopes using negation rules
  • Adjust PR workflow .github/workflows/pr.yaml to checkout github.head_ref and run release:dry-run with GITHUB_TOKEN
  • Enhance scripts/release.sh dry-run to use current/PR branch, skip npm verification, verify only GitHub conditions, and unset GITHUB_ACTIONS

Written by Cursor Bugbot for commit b03ba16. This will update automatically on new commits. Configure here.

@greptile-apps
Copy link

greptile-apps bot commented Jan 23, 2026

Greptile Overview

Greptile Summary

Fixes monorepo release isolation by preventing commits scoped to one package from triggering releases in other packages. Previously, a feat(iam) commit incorrectly released all four packages (storage, iam, keyv-tigris, react) as evidenced by commit b1326cd.

  • Release config: Adds negation rules !(scope) to explicitly prevent other scopes from triggering releases for each package
  • PR workflow: Checks out PR branch head instead of merge commit, adds GITHUB_TOKEN for dry-run validation
  • Release script: Configures dry-run to target current branch, unsets GITHUB_ACTIONS to avoid CI environment interference, skips npm verification during dry-run

Confidence Score: 4/5

  • Safe to merge with minor consideration for unscoped commits
  • The changes correctly address the monorepo release isolation issue. The negation pattern in release.config.base.cjs prevents cross-package releases. However, commits without a scope will still fall through to semantic-release's default rules, which may trigger releases in all packages. The project's commitlint config requires scopes, which mitigates this risk.
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/pr.yaml Adds PR branch checkout and GITHUB_TOKEN for dry-run release validation
release.config.base.cjs Adds negation rules to prevent cross-package releases in monorepo
scripts/release.sh Enhances dry-run mode to work with PR branches and skip npm verification

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

No files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@designcode designcode marked this pull request as draft January 23, 2026 12:51
@designcode designcode force-pushed the chore/fix-release-config branch from f987faf to 31e13da Compare January 23, 2026 13:07
@designcode designcode force-pushed the chore/fix-release-config branch from 31e13da to b03ba16 Compare January 23, 2026 13:10
@designcode designcode marked this pull request as ready for review January 23, 2026 13:12
@designcode designcode merged commit 453a4ea into main Jan 23, 2026
2 checks passed
@designcode designcode deleted the chore/fix-release-config branch January 23, 2026 13:14
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

No files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@github-actions
Copy link

🎉 This PR is included in version 1.1.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions
Copy link

🎉 This PR is included in version 1.2.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions
Copy link

github-actions bot commented Feb 5, 2026

🎉 This PR is included in version 2.12.1 🎉

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