diff options
| author | Craig Jennings <c@cjennings.net> | 2026-04-14 01:17:58 -0400 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-04-14 01:17:58 -0400 |
| commit | f8c593791ae051b07dba2606c18f1deb7589825e (patch) | |
| tree | 22dc96a5eeba6250f4697b2daf573289e9fea8f2 /Makefile | |
| parent | f894e07ea3c39620e37245f1e1bf829e853bbe5b (diff) | |
| download | rulesets-f8c593791ae051b07dba2606c18f1deb7589825e.tar.gz rulesets-f8c593791ae051b07dba2606c18f1deb7589825e.zip | |
chore: remove non-functional hooks template
The hooks/settings.json template was broken since day one:
- PostEditTool is not a valid Claude hook event
- PreCommit is not a Claude concept at all
- matcher was used as a glob, not a tool-name regex
- $FILE was never substituted from stdin JSON
Hooks never fired. Formatting and secret-scanning belong in CI and
pre-commit frameworks, not per-developer Claude config. Remove the
template and its install-hooks Makefile target.
Diffstat (limited to 'Makefile')
| -rw-r--r-- | Makefile | 16 |
1 files changed, 1 insertions, 15 deletions
@@ -3,7 +3,7 @@ RULES_DIR := $(HOME)/.claude/rules SKILLS := c4-analyze c4-diagram debug add-tests respond-to-review review-pr fix-issue security-check RULES := $(wildcard claude-rules/*.md) -.PHONY: install uninstall install-hooks list +.PHONY: install uninstall list install: @mkdir -p $(SKILLS_DIR) $(RULES_DIR) @@ -34,20 +34,6 @@ install: @echo "" @echo "done" -install-hooks: -ifndef TARGET - $(error Usage: make install-hooks TARGET=/path/to/project) -endif - @mkdir -p $(TARGET)/.claude - @if [ -e "$(TARGET)/.claude/settings.json" ]; then \ - echo " WARN $(TARGET)/.claude/settings.json already exists — not overwriting"; \ - echo " Compare with: diff $(CURDIR)/hooks/settings.json $(TARGET)/.claude/settings.json"; \ - else \ - cp "$(CURDIR)/hooks/settings.json" "$(TARGET)/.claude/settings.json"; \ - echo " copy settings.json → $(TARGET)/.claude/settings.json"; \ - fi - @echo "done" - uninstall: @echo "Skills:" @for skill in $(SKILLS); do \ |