Skip to content

Add manifest version-consistency check and CI - #10

Merged
maxkilla merged 1 commit into
mainfrom
fix/version-consistency-check
Oct 11, 2026
Merged

maxkilla merged 1 commit into
mainfrom
fix/version-consistency-check

Conversation

@maxkilla

Copy link
Copy Markdown
Owner

What

  • scripts/check-versions.js: fails if package.json, plugin.yaml, .claude-plugin/plugin.json and marketplace.json disagree on version; optional release-tag argument must match too.
  • test/versions.test.js: 4 tests (repo consistent, stale manifest, tag mismatch, missing marketplace entry).
  • .github/workflows/test.yml: first CI for comb. Runs JS tests, Python tests, and the version check (against the tag on v* tags).

Why

ponytail v4.8.0 shipped with stale manifests, so plugins kept reporting the old version after update. Comb has four version files and the same risk.

Testing

Locally: 33 JS + 19 Python tests pass. The workflow has not run on GitHub yet.

🤖 Generated with Claude Code

https://claude.ai/code/session_016eSubvA6StrmXkwgww4tMk


Generated by Claude Code

Fails if package.json, plugin.yaml, plugin.json and marketplace.json disagree,
or if a release tag does not match them (the stale-manifest bug ponytail hit in
v4.8.0). Runs in npm test and in a new GitHub Actions workflow.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016eSubvA6StrmXkwgww4tMk
@maxkilla
maxkilla merged commit 19289b4 into main Oct 11, 2026
2 checks passed
@maxkilla
maxkilla deleted the fix/version-consistency-check branch October 11, 2026 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants