fix: use empty JSON instead of invalid \"allow\" decision value

Claude Code only recognizes \"block\" as a top-level decision value.
\"allow\" is a permissionDecision value for PreToolUse hooks, not a
valid top-level decision. The correct way to not block is to return
empty JSON. Caught by #872.

Author: mvalentsev

mempalace/hooks_cli.py

@@ -36,11 +36,6 @@
     "Save everything to MemPalace, then allow compaction to proceed."
 )
 
-PRECOMPACT_ALLOW_REASON = (
-    "MemPalace pre-compaction save. Your conversation has been saved "
-    "in the background. Compaction can proceed safely."
-)
-
 
 def _sanitize_session_id(session_id: str) -> str:
     """Only allow alnum, dash, underscore to prevent path traversal."""
@@ -239,7 +234,7 @@ def hook_precompact(data: dict, harness: str):
     # Mine synchronously so data lands before compaction proceeds
     _mine_sync(transcript_path)
 
-    _output({"decision": "allow", "reason": PRECOMPACT_ALLOW_REASON})
+    _output({})
 
 
 def run_hook(hook_name: str, harness: str):

tests/test_hooks_cli.py

@@ -10,7 +10,6 @@
 from mempalace.hooks_cli import (
     SAVE_INTERVAL,
     STOP_BLOCK_REASON,
-    PRECOMPACT_ALLOW_REASON,
     _count_human_messages,
     _get_mine_dir,
     _log,
@@ -212,8 +211,7 @@ def test_precompact_allows(tmp_path):
         {"session_id": "test"},
         state_dir=tmp_path,
     )
-    assert result["decision"] == "allow"
-    assert result["reason"] == PRECOMPACT_ALLOW_REASON
+    assert result == {}
 
 
 # --- _log ---
@@ -382,7 +380,7 @@ def test_precompact_with_mempal_dir(tmp_path):
                 {"session_id": "test"},
                 state_dir=tmp_path,
             )
-    assert result["decision"] == "allow"
+    assert result == {}
     mock_run.assert_called_once()
 
 
@@ -397,7 +395,7 @@ def test_precompact_with_mempal_dir_oserror(tmp_path):
                 {"session_id": "test"},
                 state_dir=tmp_path,
             )
-    assert result["decision"] == "allow"
+    assert result == {}
 
 
 def test_precompact_with_timeout(tmp_path):
@@ -412,7 +410,7 @@ def test_precompact_with_timeout(tmp_path):
             result = _capture_hook_output(
                 hook_precompact, {"session_id": "test"}, state_dir=tmp_path
             )
-    assert result["decision"] == "allow"
+    assert result == {}
 
 
 def test_precompact_mines_transcript_dir(tmp_path, monkeypatch):
@@ -426,7 +424,7 @@ def test_precompact_mines_transcript_dir(tmp_path, monkeypatch):
             {"session_id": "test", "transcript_path": str(transcript)},
             state_dir=tmp_path,
         )
-    assert result["decision"] == "allow"
+    assert result == {}
     mock_run.assert_called_once()
     # Verify mine dir is the transcript's parent
     call_args = mock_run.call_args[0][0]
@@ -471,9 +469,7 @@ def test_run_hook_dispatches_precompact(tmp_path):
         with patch("mempalace.hooks_cli.STATE_DIR", tmp_path):
             with patch("mempalace.hooks_cli._output") as mock_output:
                 run_hook("precompact", "claude-code")
-    mock_output.assert_called_once()
-    call_args = mock_output.call_args[0][0]
-    assert call_args["decision"] == "allow"
+    mock_output.assert_called_once_with({})
 
 
 def test_run_hook_unknown_hook():