Skip to content

chore: track pre-commit hooks in source control#24

Open
tannn wants to merge 1 commit into
mainfrom
chore/track-pre-commit-hooks
Open

chore: track pre-commit hooks in source control#24
tannn wants to merge 1 commit into
mainfrom
chore/track-pre-commit-hooks

Conversation

@tannn
Copy link
Copy Markdown
Owner

@tannn tannn commented Apr 13, 2026

Summary

  • Adds pre-commit (orchestrator), pre-commit-agent-check, and pre-commit-encoding-check to git-hooks/ so setup-hooks.sh distributes the complete set of pre-commit safety guards to all contributors
  • All three files are byte-identical to their .git/hooks/ counterparts
  • setup-hooks.sh already iterates all files in git-hooks/ generically — no script changes needed

Test plan

  • Run bash scripts/setup-hooks.sh on a fresh clone and verify all four hooks (commit-msg, pre-commit, pre-commit-agent-check, pre-commit-encoding-check) are installed in .git/hooks/
  • Attempt to commit a file with a staged .claude/ path and confirm pre-commit-agent-check blocks it
  • Confirm a normal commit proceeds without errors

🤖 Generated with Claude Code

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

Adds pre-commit, pre-commit-agent-check, and pre-commit-encoding-check
to git-hooks/ so setup-hooks.sh distributes the complete set of
pre-commit safety guards to all contributors.

- pre-commit: orchestrator that runs all checks in order
- pre-commit-agent-check: blocks committing agent credential directories
- pre-commit-encoding-check: validates UTF-8 encoding in markdown files

Nightshift-Task: commit-normalize
Nightshift-Ref: https://github.com/marcus/nightshift

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@tannn tannn force-pushed the chore/track-pre-commit-hooks branch from 5e4bcf1 to 3086cbb Compare April 13, 2026 06:45
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