fix: bump plugin manifests to 5.7.0 and enforce version consistency in CI#28
Merged
Merged
Conversation
…ement Agent-Logs-Url: https://github.com/GoCodeAlone/claude-superpowers/sessions/ea13db6a-9b67-4e1a-a465-d28aa43995b9 Co-authored-by: intel352 <77607+intel352@users.noreply.github.com>
Copilot created this pull request from a session on behalf of
intel352
May 8, 2026 13:51
View session
Contributor
There was a problem hiding this comment.
Pull request overview
Bumps the declared plugin version to 5.7.0 across the repository’s plugin manifests and adds automation (script + test + CI workflow) to prevent future version drift.
Changes:
- Updated plugin manifest
"version"fields to5.7.0in three JSON manifests. - Added
scripts/bump-version.shto update versions across the allowlisted manifests. - Added
tests/version-check.shplus a GitHub Actions workflow to enforce consistency (and tag alignment on tag pushes).
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
tests/version-check.sh |
New consistency check script with optional git-tag verification mode. |
scripts/bump-version.sh |
New helper script to bump version across the three manifest files. |
.github/workflows/version-check.yml |
CI workflow to run the version check on PRs/pushes and on tag pushes. |
.cursor-plugin/plugin.json |
Manifest version bump to 5.7.0. |
.claude-plugin/plugin.json |
Manifest version bump to 5.7.0. |
.claude-plugin/marketplace.json |
Manifest version bump to 5.7.0. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ) | ||
|
|
||
| extract_version() { | ||
| grep '"version"' "$1" | head -1 | sed 's/.*"version": *"\([^"]*\)".*/\1/' |
Comment on lines
+67
to
+76
| if [[ "$CHECK_TAG" == true ]]; then | ||
| # Get the latest semver tag (strip the leading 'v') | ||
| LATEST_TAG=$(git -C "$REPO_ROOT" tag --sort=-v:refname | grep -E '^v[0-9]+\.[0-9]+\.[0-9]+$' | head -1 || true) | ||
| if [[ -z "$LATEST_TAG" ]]; then | ||
| echo "WARNING: No semver git tags found; skipping tag check." >&2 | ||
| else | ||
| TAG_VERSION="${LATEST_TAG#v}" | ||
| if [[ "$FIRST_VERSION" != "$TAG_VERSION" ]]; then | ||
| echo "ERROR: Plugin version '$FIRST_VERSION' does not match latest git tag '$LATEST_TAG'." >&2 | ||
| echo " Run: scripts/bump-version.sh ${TAG_VERSION}" >&2 |
Comment on lines
+38
to
+42
| OLD_VERSION=$(grep '"version"' "$PRIMARY" | head -1 | sed 's/.*"version": *"\([^"]*\)".*/\1/') | ||
| if [[ -z "$OLD_VERSION" ]]; then | ||
| echo "ERROR: Could not detect current version from $PRIMARY" >&2 | ||
| exit 1 | ||
| fi |
| echo "WARNING: $rel_path not found, skipping" >&2 | ||
| continue | ||
| fi | ||
| sed -i.bak "s/\"version\": \"${OLD_VERSION}\"/\"version\": \"${NEW_VERSION}\"/" "$abs_path" && rm -f "${abs_path}.bak" |
Contributor
|
@copilot apply changes based on the comments in this thread |
Agent-Logs-Url: https://github.com/GoCodeAlone/claude-superpowers/sessions/161a7b0e-4bce-4a7f-80d3-01ae6578a602 Co-authored-by: intel352 <77607+intel352@users.noreply.github.com>
Agent-Logs-Url: https://github.com/GoCodeAlone/claude-superpowers/sessions/161a7b0e-4bce-4a7f-80d3-01ae6578a602 Co-authored-by: intel352 <77607+intel352@users.noreply.github.com>
Contributor
Author
Applied all 4 review comments in cb562b8:
|
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.
scripts/bump-version.sh— add|| truetogrep|head|sedpipeline so the[[ -z "$OLD_VERSION" ]]guard is reachable underset -euo pipefailscripts/bump-version.sh— pre-check that the old version string exists in each file before runningsed; accumulate errors and exit non-zero if any file was skipped, preventing silent "Updated" without actual change; clarify error message to say "cannot update" instead of "skipping" since the script will ultimately failtests/version-check.sh— add|| trueto thegrep|head|sedpipeline insideextract_version()so the[[ -z "$ver" ]]guard is reachabletests/version-check.sh— replacegit tag --sort=... | grep | head(latest tag) withgit describe --tags --exact-match HEADso--check-tagcompares against the tag at HEAD, not the globally newest tag