Skip to content

docs: add contribution guidelines - #317

Open
MohammadWasi wants to merge 1 commit into
anthropics:mainfrom
MohammadWasi:fix/hooks-manifest-schema
Open

docs: add contribution guidelines#317
MohammadWasi wants to merge 1 commit into
anthropics:mainfrom
MohammadWasi:fix/hooks-manifest-schema

Conversation

@MohammadWasi

@MohammadWasi MohammadWasi commented Aug 14, 2026

Copy link
Copy Markdown

Summary

  • Add CONTRIBUTING.md with repository-specific setup, validation, security, plugin-versioning, and PR expectations.
  • Link the guide from the root README’s Contributing section.
  • Document the required valid empty hook-manifest shape: { "hooks": {} }.

Reproduction and resolution

Contributors previously had to infer the repository workflow from several files. In particular, the source-of-truth rule for skills, required plugin version bumps, managed-agent safety boundaries, and pre-PR validation steps were not documented in one place.

This PR adds a concise contribution guide that provides those instructions and a PR-body template.

The hook-manifest repair is intentionally excluded from this PR because it has already been merged into the current upstream base.

Validation

  • python3 scripts/check.py
  • python3 scripts/version_bump.py --check --base upstream/main
  • bash scripts/test-cookbooks.sh — all 10 cookbooks passed
  • Verified the README link resolves to CONTRIBUTING.md

Compatibility

  • Plugin versions bumped: none
  • No API, credential, MCP-provider, or managed-agent configuration changes
  • Documentation-only change

@MohammadWasi
MohammadWasi marked this pull request as draft August 14, 2026 01:48
@MohammadWasi
MohammadWasi force-pushed the fix/hooks-manifest-schema branch from 98c52be to 1f18989 Compare August 14, 2026 01:51
@MohammadWasi MohammadWasi changed the title Fix/hooks manifest schema docs: add contribution guidelines Aug 14, 2026
@MohammadWasi
MohammadWasi marked this pull request as ready for review August 14, 2026 01:55
@MohammadWasi

Copy link
Copy Markdown
Author

Hi @dougbarth @tss Can you please review and provide your feedback?

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.

1 participant