chore: disable worker-docs PR hooks#146
Merged
Merged
Conversation
Unregister both worker-docs Claude Code hooks from .claude/settings.json: - PostToolUse update-worker-docs.sh auto-ran generate_worker_docs.py and committed the result on every `gh pr create`/`edit`, sweeping unrelated worker-description drift (stale summaries for workers the PR never touched) into otherwise-focused diffs. - PreToolUse validate-worker-docs.sh blocked any PR that changed a worker without also updating REGISTRY.md, forcing that same noisy regeneration into logic-only PRs just to pass the check. The hook scripts are kept in .claude/hooks/ (with a DISABLED note at the top of each) so they can be re-registered later; CLAUDE.md is updated to mark both hooks disabled and explain why. Worker docs/REGISTRY.md are now a manual `python generate_worker_docs.py` step, run when a worker's interface or labels actually change. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MgYphJWw3jgQoLx5MXz95h
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
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.
Why
The two worker-docs Claude Code hooks repeatedly caused friction on PRs:
update-worker-docs.sh(PostToolUse) re-rangenerate_worker_docs.pyand committed the result on everygh pr create/edit. Because the committedREGISTRY.mddrifts from the generator's current output, this swept unrelated one-line description changes (for workers the PR never touched) into otherwise-focused diffs.validate-worker-docs.sh(PreToolUse) blocked any PR that modified a worker without also updatingREGISTRY.md— forcing that same noisy regeneration into logic-only PRs just to pass the check.(Concretely: the polygon-geometry fix in #145 is a pure logic change, yet these hooks wanted ~21 lines of unrelated
REGISTRY.mddescription churn committed alongside it.)What
.claude/settings.json(now{ "hooks": {} })..claude/hooks/with aDISABLEDnote at the top of each, so they can be re-registered later.CLAUDE.mdto mark both hook sections DISABLED and explain why.Worker docs /
REGISTRY.mdbecome a manual step: runpython generate_worker_docs.pywhen a worker's interface or labels actually change.Not changed
generate_worker_docs.pyitself is untouched — only the automatic invocation/validation is removed.🤖 Generated with Claude Code
https://claude.ai/code/session_01MgYphJWw3jgQoLx5MXz95h