Fix hook exit code to always return 0 with JSON format

Hooks using JSON output format should always exit with code 0.
The blocking behavior is controlled by the "decision" field in the
JSON output, not the exit code.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: nhorton

src/deepwork/hooks/wrapper.py

@@ -358,7 +358,6 @@ def run_hook(
     output_json = denormalize_output(hook_output, platform, hook_input.event)
     write_stdout(output_json)
 
-    # Return exit code based on decision
-    if hook_output.decision in ("block", "deny"):
-        return 2
+    # Always return 0 when using JSON output format
+    # The decision field in the JSON controls blocking behavior
     return 0

tests/shell_script_tests/test_hook_wrappers.py

@@ -209,7 +209,8 @@ def test_claude_wrapper_with_stop_event(
             env=env,
         )
 
-        assert result.returncode == 2, f"Expected exit code 2 for blocking. stderr: {result.stderr}"
+        # Exit code 0 even when blocking - the JSON decision field controls behavior
+        assert result.returncode == 0, f"Expected exit code 0. stderr: {result.stderr}"
 
         output = json.loads(result.stdout.strip())
         assert output["decision"] == "block"
@@ -242,7 +243,8 @@ def test_gemini_wrapper_with_afteragent_event(
             env=env,
         )
 
-        assert result.returncode == 2, f"Expected exit code 2 for blocking. stderr: {result.stderr}"
+        # Exit code 0 even when blocking - the JSON decision field controls behavior
+        assert result.returncode == 0, f"Expected exit code 0. stderr: {result.stderr}"
 
         output = json.loads(result.stdout.strip())
         # Gemini should get "deny" instead of "block"