Skip to content

chore: remove inert .claude/hooks/extensions scaffold - #79

Merged
twistedmelonman merged 1 commit into
mainfrom
claude/chore-remove-claude-hooks-scaffold-5e330f69
Sep 10, 2026
Merged

chore: remove inert .claude/hooks/extensions scaffold#79
twistedmelonman merged 1 commit into
mainfrom
claude/chore-remove-claude-hooks-scaffold-5e330f69

Conversation

@twistedmelonman

@twistedmelonman twistedmelonman commented Sep 10, 2026

Copy link
Copy Markdown
Owner

What

Removes the tracked file .claude/hooks/extensions/example.sh.disabled and replaces .claude/README.md with a corrected copy.

Why

.claude/hooks/extensions/ 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. The committed file was inert and its README lied about it.

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 post-checkout hook exits before any cp -r would re-drop the scaffold.

Evidence

Ran the CI-faithful standards check (standards/run-standards.sh from smartwatermelon/github-workflows@standards-check-v1) locally with an empty HOME (to avoid ~/.shellcheckrc leakage) both before and after this change:

  • Before: exit 0, all enabled linters clean.
  • After: exit 0, all enabled linters clean.

This repo was already passing before this change. The after-run does not prove the removal "fixed" or "cleared" anything — it only demonstrates that the corrected .claude/README.md lints clean.

Advances smartwatermelon/dev-env#110

https://claude.ai/code/session_01TkReZXv8XkcWNbcaiWfcvg

.claude/hooks/extensions/example.sh.disabled was 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.

Removes the tracked file. 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
@claude

This comment has been minimized.

@claude

This comment has been minimized.

@twistedmelonman
twistedmelonman merged commit a915119 into main Sep 10, 2026
9 checks passed
@twistedmelonman
twistedmelonman deleted the claude/chore-remove-claude-hooks-scaffold-5e330f69 branch September 10, 2026 20:18
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