chore: remove inert .claude/hooks/extensions scaffold, correct README - #40
Merged
twistedmelonman merged 1 commit intoSep 10, 2026
Merged
Conversation
.claude/hooks/extensions/example.sh.disabled is documented by its own README as being "automatically discovered and run by global hooks." That claim is false and was never true: git log -S'.claude/hooks/extensions' -- git/hooks/ in the dotfiles repo returns empty. Live extension discovery is .project-hooks/ only (dotfiles/git/hooks/pre-commit:92, pre-push:840). The scaffold dates to 2026-01-12; the .project-hooks/ seam was named in dotfiles#243 on 2026-08-24. Removal is safe: dotfiles/git/template/hooks/post-checkout:111 guards on .claude/ EXISTING, not on the example file. This repo keeps a tracked .claude/README.md and config.sh.template, so the hook exits before the cp -r that would re-drop the scaffold. Also replaces .claude/README.md with a corrected copy that no longer makes the false discovery claim. Advances #110 Claude-Session: https://claude.ai/code/session_01TkReZXv8XkcWNbcaiWfcvg
|
This PR updates No blocking issues found. VERDICT: PASS |
twistedmelonman
deleted the
claude/chore-remove-claude-hooks-scaffold-5e330f69
branch
September 10, 2026 20:16
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.
Summary
.claude/hooks/extensions/example.sh.disabled. Its own README claimed it is "automatically discovered and run by global hooks" — that claim is false and was never true.git log -S'.claude/hooks/extensions' -- git/hooks/in the dotfiles repo returns empty. Live extension discovery is.project-hooks/only (dotfiles/git/hooks/pre-commit:92,pre-push:840). The scaffold dates to 2026-01-12; the.project-hooks/seam was named in dotfiles#243 on 2026-08-24..claude/README.mdwith a corrected copy that no longer makes the false discovery claim.Removal is safe:
dotfiles/git/template/hooks/post-checkout:111guards on.claude/EXISTING, not on the example file. This repo keeps a tracked.claude/README.mdandconfig.sh.template, so the hook exits before thecp -rthat would re-drop the scaffold. This was verified by trial clone (DIRTY_LINES=0, nothing re-dropped).Evidence
The CI-faithful standards check (
run-standards.shatgithub-workflows@standards-check-v1, run with an emptyHOMEto avoid~/.shellcheckrcleakage) was run both before and after this change:The repo was already passing before this change — the "before" run was green because it already carried the passing variant of
example.sh.disabled. So the "after" green run does not demonstrate that the removal fixed or cleared anything. The only thing this run legitimately demonstrates is that the corrected.claude/README.mdlints clean (markdownlint:.claude/README.mdlinted, 0 issues).Advances smartwatermelon/dev-env#110
https://claude.ai/code/session_01TkReZXv8XkcWNbcaiWfcvg