fix(ci): publish only from main when version is unpublished#15
fix(ci): publish only from main when version is unpublished#15coffeexcoin merged 1 commit intomainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
PR SummaryMedium Risk Overview It adds a Written by Cursor Bugbot for commit 085f8c3. This will update automatically on new commits. Configure here. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
| should_publish=false | ||
| if [ -z "$published_version" ] || [ "$published_version" != "$local_version" ]; then | ||
| should_publish=true | ||
| fi |
There was a problem hiding this comment.
Version check compares against latest, not specific version
Low Severity
The npm view @abstract-foundation/mpp version command returns only the latest dist-tag version, not whether the specific local version exists on the registry. The intent (per PR description and RELEASING.md) is to publish "when the local version is not yet published," but this check actually tests "when the local version differs from latest." If a version bump is ever reverted on main, the local version could be an already-published older version that differs from latest, causing the workflow to attempt a redundant publish that fails. Using npm view @abstract-foundation/mpp@$local_version version would correctly check whether the exact local version is already published.


Summary
Testing