ci(release): bump plugin manifests on release; fix marketplace docs#111
Open
simonfaltum wants to merge 1 commit into
Open
ci(release): bump plugin manifests on release; fix marketplace docs#111simonfaltum wants to merge 1 commit into
simonfaltum wants to merge 1 commit into
Conversation
The Release workflow now runs scripts/bump_version.py to set the version in .claude-plugin/plugin.json and .cursor-plugin/plugin.json to the release tag, regenerate manifest.json, commit, then tag. Claude Code's marketplace keys plugin updates on the version field, so releases that did not bump it left marketplace clients on the cached copy (latest tag v0.2.1 vs manifest 0.2.0). Docs: fix the Claude Code install command to databricks@databricks-agent-skills, correct the example plugin cache path, soften the GPG-signing claim to match reality, and document the release flow plus the cli-compat.json follow-up in CONTRIBUTING. Co-authored-by: Isaac Signed-off-by: simon <simon.faltum@databricks.com>
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.
Summary
Makes the Release workflow keep the plugin version in sync with the release tag, and fixes a few Claude Code marketplace docs.
Claude Code's plugin marketplace keys updates on the
versionfield in.claude-plugin/plugin.json. Releases never bumped it (the workflow only tagged and created a GitHub release), so marketplace clients stay on the cached copy and never see new skills. The drift is already live: the latest tag isv0.2.1but the manifest still says0.2.0.Changes
Release pipeline
scripts/bump_version.py <vX.Y.Z>: setsversionin.claude-plugin/plugin.jsonand.cursor-plugin/plugin.jsonto the release version and regeneratesmanifest.json. Reusesskills.pygeneration and is stdlib-only, so it runs on the protected runner that can't reach pypi.release.ymlnow: validate tag, run the bump script, commit the bump tomain, then create the tag and release from that commit. The tag therefore points at the bumped manifests.Docs
databricks@databricks-agent-skills(wasdatabricks-skills, which is the Cursor install identifier). Also corrected the example plugin cache path. Cursor's/add-plugin databricks-skillsis left unchanged (it is correct, per.cursor-plugin/NOTES.md).cli-compat.jsonfollow-up PR in the CLI repo.This PR does not itself bump the version (no release is being cut here). The existing
0.2.0vsv0.2.1drift self-corrects on the next release via the new workflow.Reviewer notes
mainstep needs CI push access tomain. Ifmainis branch-protected, add a bypass (or a PAT) or the push step will fail.github-actions[bot]with-s(DCO sign-off). If a DCO app is enforced org-wide, allowlist the bot.Documentation safety checklist
No skill examples were added or changed in this PR.
ALL PRIVILEGES, admin tokens, or broad scopes)Test plan
python3 scripts/skills.py validatepasses before and after the change.bump_version.py v0.2.1: produced single-line version bumps in both plugin manifests,manifest.jsonunchanged (it keys on per-skill versions); reverted afterward.mainpush access is confirmed.This pull request and its description were written by Isaac.