diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-09 16:52:49 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-09 16:52:49 -0500 |
| commit | 05a074633edfd320233f6dc065945f6d9a858674 (patch) | |
| tree | c8b14c358679cc2d61edeaa569c36abb32231316 | |
| parent | 0572c985a919824b7a28f124cc38c31c50f948b3 (diff) | |
| download | dotemacs-05a074633edfd320233f6dc065945f6d9a858674.tar.gz dotemacs-05a074633edfd320233f6dc065945f6d9a858674.zip | |
chore(hooks): drop PROJECT_ROOT guard from validate-el.sh
The PROJECT_ROOT guard skipped any .el file outside the project on the theory that out-of-tree files are owned by their own project's hooks. In practice, when I edit an .el file in another repo from a Claude session in this one, the other project often doesn't have its own hook running, so the file was getting no validation at all. I dropped the guard so any .el file edited from a Claude session here gets validated, regardless of which repo it lives in.
| -rwxr-xr-x | .claude/hooks/validate-el.sh | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/.claude/hooks/validate-el.sh b/.claude/hooks/validate-el.sh index b811bcc2b..803badf82 100755 --- a/.claude/hooks/validate-el.sh +++ b/.claude/hooks/validate-el.sh @@ -35,13 +35,6 @@ f="$(jq -r '.tool_input.file_path // .tool_response.filePath // empty')" [ -z "$f" ] && exit 0 [ "${f##*.}" = "el" ] || exit 0 -# Only validate in-tree files. Files outside PROJECT_ROOT are owned by -# their own project's hooks (when a Claude session runs there). -case "$f" in - "$PROJECT_ROOT"/*) ;; # in-tree, continue - *) exit 0 ;; # out-of-tree, skip -esac - MAX_AUTO_TEST_FILES=20 # skip if more matches than this (large test suites) # --- Phase 1: syntax + byte-compile --- |