Commit Graph

7 Commits

Author SHA1 Message Date
c3f2a5f5ef fix(pre-commit): autofix JSON formatting instead of failing
Add --autofix to pretty-format-json so the hook rewrites files in place
rather than requiring manual intervention. Also applies the resulting key
ordering fixes to settings.json files.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-06-28 15:37:51 +00:00
098fc7315e chore(pre-commit): add missing hooks for TOML, Python, merge conflicts, secrets, and meta-validation
## Why
Five gap areas were identified when auditing the config against tracked file extensions and the hooks reference:
- No TOML validator despite 2 `.toml` files tracked
- No Python AST check despite 10 `.py` files tracked
- No merge-conflict marker detection
- No PEM private key block detection (gitleaks covers high-entropy strings but not raw PEM)
- No meta-validation to catch hooks that match no files or useless exclude patterns

## Impact
All five new hooks pass on `--all-files` run.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01P87CiC58Ru2PPWYTeXtjHT
2026-06-28 09:07:50 +00:00
8b8cb33df6 chore(pre-commit): validate marketplace manifest on pre-push
## Why
The root marketplace.json is the index that ties all plugins together.
claude plugin validate --strict covers schema-level checks that
check-manifests.sh does not, so it warrants its own hook alongside
validate-plugins.

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-06-27 19:47:22 +00:00
acc6eb9edd chore(pre-commit): validate plugins on pre-push
## Why
`claude plugin validate --strict` catches structural issues in plugin
manifests that `check-manifests.sh` does not cover (e.g. schema
violations, unrecognised fields). Running it at pre-push ensures all
plugins in `plugins/*/` stay valid before reaching the remote.

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-06-27 19:45:38 +00:00
c0e54a11fa chore: migrate pre-push hook to pre-commit framework
Add pre-push stage hooks:
- run-tests: execute all test-*.sh files and bats suite
- check-manifests: validate marketplace.json and plugin.json paths

Remove hardcoded .git/hooks/pre-push.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-06-27 19:22:32 +00:00
5da0d34690 chore(pre-commit): activate conventional commits validation hook 2026-06-27 19:20:30 +00:00
d98d0dae18 chore: migrate legacy pre-commit hook to .pre-commit-config.yaml
Replaces shell script (.git/hooks/pre-commit.legacy) with ecosystem-managed pre-commit framework:
- gitleaks/gitleaks: secret scanning
- jumanjihouse/pre-commit-hooks: shellcheck wrapper
- pre-commit/pre-commit-hooks: JSON/YAML validation, end-of-file-fixer, trailing-whitespace
- local hooks: SKILL.md frontmatter validation

Uses pinned versions for reproducibility across environments. Includes auto-fixes from hook runs (formatting, trailing whitespace, JSON beautification).

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-06-27 18:54:37 +00:00