safeword 0.6.2 → 0.6.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "safeword",
-  "version": "0.6.2",
+  "version": "0.6.3",
   "description": "CLI for setting up and managing safeword development environments",
   "type": "module",
   "bin": {
@@ -19,18 +19,6 @@
   "engines": {
     "node": ">=18"
   },
-  "scripts": {
-    "build": "tsup",
-    "dev": "tsup --watch",
-    "test": "vitest run",
-    "test:e2e": "vitest run tests/e2e/",
-    "test:watch": "vitest",
-    "test:coverage": "vitest run --coverage",
-    "typecheck": "tsc --noEmit",
-    "lint": "eslint src tests",
-    "clean": "rm -rf dist",
-    "prepublishOnly": "npm audit --audit-level=high && npm run build && npm test"
-  },
   "dependencies": {
     "commander": "^12.1.0"
   },
@@ -48,5 +36,16 @@
     "claude-code"
   ],
   "author": "",
-  "license": "MIT"
-}
+  "license": "MIT",
+  "scripts": {
+    "build": "tsup",
+    "dev": "tsup --watch",
+    "test": "vitest run",
+    "test:e2e": "vitest run tests/e2e/",
+    "test:watch": "vitest",
+    "test:coverage": "vitest run --coverage",
+    "typecheck": "tsc --noEmit",
+    "lint": "eslint src tests",
+    "clean": "rm -rf dist"
+  }
+}

package/templates/hooks/stop-quality.sh

@@ -42,21 +42,31 @@ if [ -z "$msg_text" ]; then
   exit 0
 fi
 
-# Extract the JSON blob from the end of the message
-# Pattern: {"proposedChanges": boolean, "madeChanges": boolean, "askedQuestion": boolean}
-json_blob=$(echo "$msg_text" | grep -oE '\{"proposedChanges":\s*(true|false)\s*,\s*"madeChanges":\s*(true|false)\s*,\s*"askedQuestion":\s*(true|false)\s*\}' | tail -1)
+# Extract JSON blob containing our required fields (order-independent)
+# Strategy: Use jq to find and validate the response summary object
+# Look for object with exactly our three boolean fields anywhere in the text
+json_blob=$(echo "$msg_text" | grep -oE '\{[^}]+\}' | while IFS= read -r candidate; do
+  if echo "$candidate" | jq -e '
+    type == "object" and
+    (.proposedChanges | type) == "boolean" and
+    (.madeChanges | type) == "boolean" and
+    (.askedQuestion | type) == "boolean"
+  ' >/dev/null 2>&1; then
+    echo "$candidate"
+  fi
+done | tail -1)
 
 if [ -z "$json_blob" ]; then
-  # No JSON blob found - remind about required format
+  # No valid JSON blob found - remind about required format
   echo "SAFEWORD: Response missing required JSON summary. Add to end of response:" >&2
   echo '{"proposedChanges": boolean, "madeChanges": boolean, "askedQuestion": boolean}' >&2
   exit 0
 fi
 
-# Parse the boolean values
-proposed_changes=$(echo "$json_blob" | jq -r '.proposedChanges // false' 2>/dev/null)
-made_changes=$(echo "$json_blob" | jq -r '.madeChanges // false' 2>/dev/null)
-asked_question=$(echo "$json_blob" | jq -r '.askedQuestion // false' 2>/dev/null)
+# Parse the boolean values (already validated, safe to extract)
+proposed_changes=$(echo "$json_blob" | jq -r '.proposedChanges')
+made_changes=$(echo "$json_blob" | jq -r '.madeChanges')
+asked_question=$(echo "$json_blob" | jq -r '.askedQuestion')
 
 # If asked a question, don't trigger review (waiting for user input)
 if [ "$asked_question" = "true" ]; then
@@ -67,7 +77,9 @@ fi
 if [ "$proposed_changes" = "true" ] || [ "$made_changes" = "true" ]; then
   echo "SAFEWORD Quality Review:" >&2
   echo "" >&2
-  echo "Double check and critique your work:" >&2
+  echo "Double check and critique your work again just in case." >&2
+  echo "Assume you've never seen it before." >&2
+  echo "" >&2
   echo "- Is it correct?" >&2
   echo "- Is it elegant?" >&2
   echo "- Does it follow latest docs/best practices?" >&2