Skip to content

chore: add CI workflow to validate plugin manifest on every PR#73

Merged
gringolito merged 2 commits into
masterfrom
chore/add-ci-plugin-manifest-validation-33
May 6, 2026
Merged

chore: add CI workflow to validate plugin manifest on every PR#73
gringolito merged 2 commits into
masterfrom
chore/add-ci-plugin-manifest-validation-33

Conversation

@gringolito
Copy link
Copy Markdown
Owner

@gringolito gringolito commented May 6, 2026

Summary

  • Adds .github/workflows/ci.yaml that installs Claude Code and runs claude plugin validate against both .claude-plugin/plugin.json and .claude-plugin/marketplace.json
  • Triggers on every pull request (any branch) and on push to master
  • Uses permissions: contents: read (least-privilege — checkout only, no write access needed)
  • Adds $schema to both manifests for IDE validation support
  • Adds description to marketplace.json (resolves claude plugin validate warning about missing marketplace description)

Acceptance Criteria

  • .github/workflows/ci.yaml exists and runs on pull_request and push to master
  • Least privilege principle applied (permissions: contents: read)
  • Workflow installs Claude Code (npm install -g @anthropic-ai/claude-code) and runs claude plugin validate
  • A deliberately broken plugin.json causes the workflow to fail — verify by temporarily removing a required field (e.g. name) and pushing; claude plugin validate exits non-zero
  • A clean repo passes the workflow — both manifests currently pass locally
  • marketplace.json validation included and exercisable — verify by introducing a JSON syntax error in the file

Closes #33

🤖 Generated with Claude Code

@gringolito gringolito self-assigned this May 6, 2026
Adds `.github/workflows/ci.yaml` that installs Claude Code and runs
`claude plugin validate` against both plugin.json and marketplace.json
on every pull request and push to master. Converts manifest correctness
from a reviewer-enforced soft invariant into a hard CI gate.

Refs #33

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@gringolito gringolito force-pushed the chore/add-ci-plugin-manifest-validation-33 branch from 543e8cc to 5fca666 Compare May 6, 2026 13:49
- Add $schema to plugin.json and marketplace.json for IDE validation
- Add marketplace description (resolves claude plugin validate warning)
- Add `claude --version` step to CI for install verification

Refs #33

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@gringolito gringolito merged commit 439066f into master May 6, 2026
1 check passed
@gringolito gringolito deleted the chore/add-ci-plugin-manifest-validation-33 branch May 6, 2026 14:10
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.

Add CI workflow to validate plugin manifest on every PR

1 participant