Skip to content

Add CI to validate plugin and marketplace manifests#6

Open
edmundmiller wants to merge 1 commit into
claude/marketplace-NM8lcfrom
claude/ci-validate-NM8lc
Open

Add CI to validate plugin and marketplace manifests#6
edmundmiller wants to merge 1 commit into
claude/marketplace-NM8lcfrom
claude/ci-validate-NM8lc

Conversation

@edmundmiller
Copy link
Copy Markdown
Member

Summary

  • Add a Validate GitHub Actions workflow (push to master + all PRs).
  • manifests job installs Claude Code and runs claude plugin validate --strict on .claude-plugin/plugin.json, and on .claude-plugin/marketplace.json when present.
  • shellcheck job lints any hooks/scripts/*.sh (no-op when none exist).

Why validate each manifest file instead of claude plugin validate .

claude plugin validate . picks a single manifest, and once marketplace.json exists it validates only the marketplace, silently skipping the plugin + hooks. Pointing it at each manifest file checks both. --strict (recommended for CI) treats warnings — missing metadata, misspelled fields — as failures.

Verified locally (claude CLI v2.1.152)

  • plugin.json → passes (skills + auto-discovered hooks/hooks.json validated)
  • marketplace.json → passes after the description fix
  • hook script → shellcheck clean (v0.10.0)

Test plan

  • Workflow runs green on this PR
  • Confirm npm install -g @anthropic-ai/claude-code + claude plugin validate run without auth in CI

https://claude.ai/code/session_01BDX5Wh7CXx1hXbfJCyBmMN


Generated by Claude Code

Run `claude plugin validate --strict` on the plugin and marketplace
manifests, and shellcheck the hook scripts, on push to master and on
pull requests.

https://claude.ai/code/session_01BDX5Wh7CXx1hXbfJCyBmMN
@edmundmiller edmundmiller changed the base branch from master to claude/marketplace-NM8lc May 27, 2026 16:11
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