Add contributor docs, CI validation, and repo hygiene#1
Conversation
- README: fix marketplace name in the install/update commands to match marketplace.json (govkit@aipos); add a Contributing section - Add CONTRIBUTING.md covering skill authoring, local validation, and PR flow - Add GitHub Actions workflow running `claude plugin validate` on every PR (and pushes to main); structural check, no API key required - Add .gitignore for OS, editor, Node, and local Claude cruft Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
PR Summary by QodoAdd contributor docs and CI validation for Claude plugin marketplace
AI Description
Diagram
High-Level Assessment
Files changed (4)
|
Code Review by Qodo
1.
|
Address code-review feedback on the validate workflow: - Pin actions/checkout and actions/setup-node to commit SHAs (v4.3.1, v4.4.0 as comments) instead of moving major tags. - Pin @anthropic-ai/claude-code to 2.1.199 instead of implicit latest, so the structural gate is deterministic. - Add a top-level `permissions: contents: read` block; the job only reads code and runs local validation. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Summary
Repository hygiene and contributor-onboarding pass. No changes to the
govkitplugin or skill content.marketplace.jsonnames the marketplaceaipos, but the README told users to rungovkit@accelerated-innovation/marketplace update accelerated-innovation, which would not resolve. Aligned both commands toaipos. Also added a Contributing section.CONTRIBUTING.md. Short, repo-specific guide: layout, how to add a skill (via the template), a pre-PR checklist (claude plugin validate ., JSON validity, version bump), the CI note, and MIT licensing of contributions..github/workflows/validate.yml). Runsclaude plugin validate .on every PR and on pushes tomain. Installs@anthropic-ai/claude-codeon Node 22. Structural validation only — no API key / secret, so it also works for fork PRs. Includes aconcurrencygroup to cancel superseded runs..gitignorefor OS/editor/Node/local-Claude cruft. Scoped the Claude rule to.claude/so it does not catch the committed.claude-plugin/directory (verified withgit check-ignore).Notes
aiposwas itself an unintended marketplace name, the alternative fix is to renamemarketplace.json'snametoaccelerated-innovationinstead — happy to flip that direction.plugin validateneeds no auth or interactive onboarding in CI but don't state it verbatim; this PR's own CI run is the end-to-end confirmation.🤖 Generated with Claude Code