Skip to content

chore: drop the PreToolUse code-review gate - #31

Merged
YonatanHen merged 1 commit into
masterfrom
dev/drop-code-review-hook
Sep 23, 2026
Merged

YonatanHen merged 1 commit into
masterfrom
dev/drop-code-review-hook

Conversation

@YonatanHen

Copy link
Copy Markdown
Owner

What

Removes the PreToolUse hook on gh pr create from .claude/settings.json.

Why

The hook was type: prompt, and a prompt hook only receives the tool-input JSON — it cannot see the conversation transcript. So it had no way to tell whether /code-review had already run, and it blocked gh pr create even right after a review. PR #30 had to work around it.

The PostToolUse technical-writer docs check is unchanged.

🤖 Generated with Claude Code

The prompt hook could not see the transcript, so it blocked `gh pr create`
even after /code-review had run. The PostToolUse docs check stays.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@YonatanHen
YonatanHen merged commit a911bae into master Sep 23, 2026
5 checks passed
@YonatanHen
YonatanHen deleted the dev/drop-code-review-hook branch September 23, 2026 13:34
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