Merge pull request #885 from MemPalace/fix/hook-invalid-decision

fix: replace invalid 'decision: allow' with {} in hooks (closes #872)
This commit is contained in:
Igor Lins e Silva
2026-04-14 16:56:30 -03:00
committed by GitHub
2 changed files with 6 additions and 13 deletions
+3 -7
View File
@@ -68,10 +68,6 @@ if [ -n "$MEMPAL_DIR" ] && [ -d "$MEMPAL_DIR" ]; then
python3 -m mempalace mine "$MEMPAL_DIR" >> "$STATE_DIR/hook.log" 2>&1 python3 -m mempalace mine "$MEMPAL_DIR" >> "$STATE_DIR/hook.log" 2>&1
fi fi
# Notify — compaction is about to happen but filing is handled in background # Silent: return empty JSON to not block. "decision": "allow" is invalid —
cat << 'HOOKJSON' # only "block" or {} are recognized.
{ echo '{}'
"decision": "allow",
"reason": "MemPalace pre-compaction save. Your full conversation has been saved verbatim in the background — no action needed. Compaction can proceed safely."
}
HOOKJSON
+3 -6
View File
@@ -168,12 +168,9 @@ if [ "$SINCE_LAST" -ge "$SAVE_INTERVAL" ] && [ "$EXCHANGE_COUNT" -gt 0 ]; then
} }
HOOKJSON HOOKJSON
else else
cat << 'HOOKJSON' # Silent mode: return empty JSON to not block. "decision": "allow" is
{ # not a valid value — only "block" or {} are recognized.
"decision": "allow", echo '{}'
"reason": "MemPalace auto-save checkpoint. Your conversation is being saved verbatim in the background — no action needed from you. Continue working."
}
HOOKJSON
fi fi
else else
# Not time yet — let the AI stop normally # Not time yet — let the AI stop normally