Skip to content

feat: add block-no-verify PreToolUse hook to .claude/settings.json#28511

Open
tupe12334 wants to merge 2 commits intocalcom:mainfrom
tupe12334:add-block-no-verify
Open

feat: add block-no-verify PreToolUse hook to .claude/settings.json#28511
tupe12334 wants to merge 2 commits intocalcom:mainfrom
tupe12334:add-block-no-verify

Conversation

@tupe12334
Copy link

Summary

Adds block-no-verify@1.1.2 as a PreToolUse Bash hook in .claude/settings.json, alongside the existing env configuration.

Details

When an agent runs git commit or git push with the hook-bypass flag, it silently disables pre-commit, commit-msg, and pre-push hooks. block-no-verify reads tool_input.command from the Claude Code hook stdin payload, detects the hook-bypass flag across all git subcommands, and exits 2 to block. The existing env.CLAUDE_CODE_EXPERIMENTAL_AGENT_TEAMS is preserved unchanged.

Closes #28510


Disclosure: I am the author and maintainer of block-no-verify.

Prevents agents from bypassing git hooks via the hook-skip flag.

Closes calcom#28510
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

@CLAassistant
Copy link

CLAassistant commented Mar 19, 2026

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@sahitya-chandra sahitya-chandra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you sign the CLA?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: add block-no-verify PreToolUse hook to .claude/settings.json

3 participants