fix(hooks): Update hook schema to use permissionDecision

- Replace 'decision: allow/block' with 'permissionDecision: allow/deny'
- Use 'updatedInput' for PreToolUse input modification
- Fix auto-background-hook and sms-response-handler schemas

Author: jonathanpeterwu

templates/claude-hooks/auto-background-hook.js

@@ -110,19 +110,19 @@ process.stdin.on('end', () => {
     // Only process Bash tool
     if (tool_name !== 'Bash') {
       // Allow other tools through unchanged
-      console.log(JSON.stringify({ decision: 'allow' }));
+      console.log(JSON.stringify({ permissionDecision: 'allow' }));
       return;
     }
 
     const command = tool_input?.command;
     if (!command) {
-      console.log(JSON.stringify({ decision: 'allow' }));
+      console.log(JSON.stringify({ permissionDecision: 'allow' }));
       return;
     }
 
     // Already backgrounded
     if (tool_input.run_in_background === true) {
-      console.log(JSON.stringify({ decision: 'allow' }));
+      console.log(JSON.stringify({ permissionDecision: 'allow' }));
       return;
     }
 
@@ -135,22 +135,23 @@ process.stdin.on('end', () => {
         );
       }
 
-      // Modify the tool input to add run_in_background
+      // Modify the tool input to add run_in_background using correct schema
       console.log(
         JSON.stringify({
-          decision: 'modify',
-          tool_input: {
+          hookEventName: 'PreToolUse',
+          permissionDecision: 'allow',
+          updatedInput: {
             ...tool_input,
             run_in_background: true,
           },
         })
       );
     } else {
-      console.log(JSON.stringify({ decision: 'allow' }));
+      console.log(JSON.stringify({ permissionDecision: 'allow' }));
     }
   } catch (err) {
     // On error, allow the command through unchanged
     console.error('[auto-bg] Error:', err.message);
-    console.log(JSON.stringify({ decision: 'allow' }));
+    console.log(JSON.stringify({ permissionDecision: 'allow' }));
   }
 });

templates/claude-hooks/sms-response-handler.js

@@ -139,13 +139,9 @@ process.stdin.on('end', () => {
 
         clearLatestResponse();
 
-        console.log(
-          JSON.stringify({
-            decision: 'allow',
-            context: context,
-            user_message: `[SMS Response] User replied: "${latestResponse.response}"`,
-          })
-        );
+        // Log context to stderr for visibility, allow the tool
+        console.error(`[sms-hook] Context: ${JSON.stringify(context)}`);
+        console.log(JSON.stringify({ permissionDecision: 'allow' }));
         return;
       }
 
@@ -162,24 +158,17 @@ process.stdin.on('end', () => {
           )
           .join('\n');
 
-        console.log(
-          JSON.stringify({
-            decision: 'allow',
-            context: {
-              type: 'sms_actions_executed',
-              results,
-            },
-            user_message: `[SMS Actions] Executed queued actions:\n${summary}`,
-          })
-        );
+        // Log results to stderr for visibility, allow the tool
+        console.error(`[sms-hook] Actions summary:\n${summary}`);
+        console.log(JSON.stringify({ permissionDecision: 'allow' }));
         return;
       }
     }
 
     // Default: allow everything
-    console.log(JSON.stringify({ decision: 'allow' }));
+    console.log(JSON.stringify({ permissionDecision: 'allow' }));
   } catch (err) {
     console.error('[sms-hook] Error:', err.message);
-    console.log(JSON.stringify({ decision: 'allow' }));
+    console.log(JSON.stringify({ permissionDecision: 'allow' }));
   }
 });