no-yolo-commits 1.0.0 → 1.0.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "no-yolo-commits",
-  "version": "1.0.0",
+  "version": "1.0.2",
   "description": "AI pre-commit review (via the claude CLI) + a guard that auto-branches you off main/master instead of letting a direct commit land there.",
   "keywords": [
     "husky",
@@ -34,5 +34,11 @@
   },
   "publishConfig": {
     "access": "public"
+  },
+  "devDependencies": {
+    "husky": "^9.1.7"
+  },
+  "scripts": {
+    "prepare": "husky"
   }
 }

package/templates/pre-commit.sh

@@ -13,13 +13,18 @@ Do NOT flag stylistic preferences. Do NOT flag pre-existing patterns already use
 
   review_tmpfile="$(mktemp)"
 
-  (claude -p "$review_prompt" \
+  # No subshell wrapper here on purpose: `(claude ...; exit 0) &` would make
+  # $! the wrapper's PID, not claude's — since the wrapper has more to do
+  # after claude exits, the shell can't tail-exec it, so killing $review_pid
+  # on timeout would kill the empty wrapper and leave claude itself running
+  # orphaned. Backgrounding claude directly makes $! the real PID.
+  claude -p "$review_prompt" \
     --output-format json \
     --json-schema '{"type":"object","properties":{"blocking_issues":{"type":"array","items":{"type":"string"}}},"required":["blocking_issues"]}' \
     --allowedTools "Bash(git diff*)" "Bash(git show*)" "Bash(git log*)" "Read" "Grep" "Glob" \
     --disallowedTools "Write" "Edit" "MultiEdit" "NotebookEdit" \
     --permission-mode bypassPermissions \
-    >"$review_tmpfile" 2>/dev/null; exit 0) &
+    >"$review_tmpfile" 2>/dev/null &
   review_pid=$!
   (sleep 120; kill "$review_pid" >/dev/null 2>&1; exit 0) >/dev/null 2>&1 &
   review_watcher_pid=$!
@@ -33,7 +38,13 @@ Do NOT flag stylistic preferences. Do NOT flag pre-existing patterns already use
     const fs = require("fs")
     try {
       const data = JSON.parse(fs.readFileSync(process.argv[1], "utf-8"))
-      const issues = (data.structured_output && data.structured_output.blocking_issues) || []
+      // An API/session error from claude itself is valid JSON with no
+      // structured_output — falling back to [] there made a failed review
+      // look identical to "no issues found" instead of "unavailable".
+      if (data.is_error || !data.structured_output) {
+        process.exit(2)
+      }
+      const issues = data.structured_output.blocking_issues || []
       if (issues.length > 0) {
         console.error("")
         console.error("⛔ AI review found blocking issues:")
@@ -83,7 +94,9 @@ if command -v claude >/dev/null 2>&1; then
     tmpfile="$(mktemp)"
     prompt="Summarize the following staged git diff as ONE short phrase in kebab-case (lowercase words separated by hyphens, no punctuation, no quotes), at most 8 words, describing what changed. Output ONLY the phrase, nothing else, no explanation."
 
-    (printf '%s' "$diff" | claude -p "$prompt" >"$tmpfile" 2>/dev/null; exit 0) &
+    # Same reasoning as the review call above: background claude directly,
+    # no subshell wrapper, so $! is the real PID the timeout can kill.
+    printf '%s' "$diff" | claude -p "$prompt" >"$tmpfile" 2>/dev/null &
     claude_pid=$!
     (sleep 25; kill "$claude_pid" >/dev/null 2>&1; exit 0) >/dev/null 2>&1 &
     watcher_pid=$!
@@ -113,13 +126,14 @@ timestamp="$(date +%s)"
 new_branch="__PREFIX__-${timestamp}-${slug}"
 new_branch="$(printf '%s' "$new_branch" | cut -c1-100)"
 
-if ! git checkout -b "$new_branch" 2>/tmp/no-yolo-commits-error; then
+branch_err_file="$(mktemp)"
+if ! git checkout -b "$new_branch" 2>"$branch_err_file"; then
   echo "✗ Failed to create branch '$new_branch':"
-  cat /tmp/no-yolo-commits-error 2>/dev/null || true
-  rm -f /tmp/no-yolo-commits-error 2>/dev/null || true
+  cat "$branch_err_file" 2>/dev/null || true
+  rm -f "$branch_err_file" 2>/dev/null || true
   echo "Commit aborted — create/switch to a feature branch manually and retry."
   exit 1
 fi
-rm -f /tmp/no-yolo-commits-error 2>/dev/null || true
+rm -f "$branch_err_file" 2>/dev/null || true
 
 echo "✓ Switched to '$new_branch' — continuing commit there."