refactor bump script - #45
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 4283e7f. Configure here.
| inserted = 1 | ||
| # Marker is followed by a blank line; skip the consumed blank. | ||
| next | ||
| } |
There was a problem hiding this comment.
Changelog entry can delete lines
Medium Severity
The CHANGELOG.md awk path always getlines and then nexts after the Semantic Versioning line, discarding that next line unconditionally. The readme.txt path only drops a following blank. If the blank after the marker is missing, the following heading or content is removed during a bump. The old script required the blank marker and failed instead of deleting data.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 4283e7f. Configure here.


Note
Low Risk
Dev-only release tooling with no runtime plugin impact; the main caveat is macOS-specific
sedand narrowerpackage-lockedits, which could behave differently on Linux CI than the removed Node script.Overview
Replaces
scripts/bump-version.jswithbin/bump.shand wiresnpm run bumptobash bin/bump.sh..distignoreand.gitattributesnow ignore/bininstead of/scripts.The new shell script keeps the same release workflow: resolve
patch/minor/majoror an explicitx.y.zfrom.plugin-data, then update JSON versions (plugin data,package.json, blockblock.jsonfiles),package-lock.json,blockparty-tabs.php,readme.txt,CHANGELOG.md, the Playground blueprintref, andREADME.mdwhen the old version appears. It documents macOSsed -i ''and prints a summary of touched files at the end.Reviewed by Cursor Bugbot for commit 4283e7f. Bugbot is set up for automated code reviews on this repo. Configure here.