docs(hook-development): Add guidance on properly disabling hooks #332
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Hooks are loaded from the plugin manifest, not auto-discovered by file presence. Renaming or deleting a hook file doesn't disable it - you must remove the hook entry from plugin.json/config.json. Adds: - New "Disabling Hooks" subsection with correct procedure - Warning in DON'T best practices list Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
Thanks for your interest! This repo only accepts contributions from Anthropic team members. If you'd like to submit a plugin to the marketplace, please submit your plugin here. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Context
Discovered through real usage: renaming or deleting a hook script file doesn't disable it - Claude Code still tries to load the hook from the manifest and fails with a missing file error. The correct approach is to remove the hook entry from
plugin.json/config.json.Test plan
🤖 Generated with Claude Code