From 5dacc814b880c5a3814a057bbbf52c8f167d7272 Mon Sep 17 00:00:00 2001
From: jacob-wang <jiayuw794@gmail.com>
Date: Tue, 21 Apr 2026 05:04:19 +0800
Subject: [PATCH] fix(careful): skip false positives in text-output command
 segments
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

/careful hook was triggering on patterns inside git commit -m messages,
echo arguments, and other text-output commands — even though those strings
are not executed as destructive operations.

Split command on shell operators (&&, ||, ;) and skip segments that are
purely text-output commands (git commit -m, echo, printf, cat). Only
check the remaining executable segments for destructive patterns.

Fixes #1060.
---
 careful/bin/check-careful.sh | 32 ++++++++++++++++++++++++++++++++
 1 file changed, 32 insertions(+)

diff --git a/careful/bin/check-careful.sh b/careful/bin/check-careful.sh
index c8bc2c7ab..f5e00f527 100755
--- a/careful/bin/check-careful.sh
+++ b/careful/bin/check-careful.sh
@@ -22,6 +22,38 @@ if [ -z "$CMD" ]; then
   exit 0
 fi
 
+# Split command on shell operators and filter out text-only segments.
+# Segments that start with git commit, echo, cat, or printf produce text output,
+# not destructive actions — skip their content to avoid false positives.
+FILTERED_CMD=""
+IFS_SAVE="$IFS"
+IFS="$(printf '\n\t')"
+for segment in $(printf '%s' "$CMD" | sed 's/&&/\n/g; s/||/\n/g; s/;/ /g'); do
+  trimmed=$(printf '%s' "$segment" | sed 's/^[[:space:]]*//;s/[[:space:]]*$//')
+  # Skip segments that are purely text-output commands (their args may mention
+  # dangerous patterns without actually executing them)
+  case "$trimmed" in
+    git\ commit\ -m\ *|git\ commit\ --message\ *|echo\ *|printf\ *|cat\ <*)
+      continue
+      ;;
+    *)
+      if [ -n "$FILTERED_CMD" ]; then
+        FILTERED_CMD="$FILTERED_CMD && "
+      fi
+      FILTERED_CMD="${FILTERED_CMD}${trimmed}"
+      ;;
+  esac
+done
+IFS="$IFS_SAVE"
+
+# Use filtered command for pattern checks; if all segments were filtered out, allow
+if [ -z "$FILTERED_CMD" ]; then
+  echo '{}'
+  exit 0
+fi
+
+CMD="$FILTERED_CMD"
+
 # Normalize: lowercase for case-insensitive SQL matching
 CMD_LOWER=$(printf '%s' "$CMD" | tr '[:upper:]' '[:lower:]')