superlocalmemory 4.1.19 → 4.1.21

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "superlocalmemory",
-  "version": "4.1.19",
+  "version": "4.1.21",
   "description": "Local-first agent memory with MCP and an agent-native CLI. Documented clients include Claude Code, Cursor, and Windsurf.",
   "keywords": [
     "ai-memory",

package/plugin/.claude-plugin/plugin.json

@@ -15,5 +15,5 @@
   "mcpServers": "./.mcp.json",
   "name": "superlocalmemory",
   "repository": "https://github.com/qualixar/superlocalmemory",
-  "version": "4.1.19"
+  "version": "4.1.21"
 }

package/plugin/.mcp.json

@@ -1,8 +1,13 @@
 {
   "mcpServers": {
     "superlocalmemory": {
-      "command": "${CLAUDE_PLUGIN_ROOT}/scripts/slm-launch",
-      "args": [],
+      "command": "${ComSpec:-${CLAUDE_PLUGIN_ROOT}/scripts/slm-launch}",
+      "args": [
+        "/d",
+        "/s",
+        "/c",
+        "where.exe /q slm || (echo SLM plugin: slm is not on PATH. On Windows the plugin runs the SuperLocalMemory you installed. Install it first with: pipx install superlocalmemory  or: pip install superlocalmemory  then restart your editor. 1>&2 & exit 1) & slm serve start 1>&2 & slm mcp"
+      ],
       "env": {
         "SLM_AGENT_ID": "claude_code"
       }

package/plugin/CLAUDE.md

@@ -1,4 +1,4 @@
-<!-- BEGIN SuperLocalMemory v4.1.19 -->
+<!-- BEGIN SuperLocalMemory v4.1.21 -->
 
 ## SuperLocalMemory (SLM) — Agent Rules
 
@@ -40,6 +40,6 @@ slm-recall · slm-remember · slm-session · slm-status · slm-cache · slm-comp
 ### Subagents
 slm-memory-advisor (memory decisions, session hygiene, scope/profile guidance) · slm-optimize-advisor (context compression + KV cache) · slm-governance-advisor (scope/roles/compliance/GDPR)
 
-<!-- END SuperLocalMemory v4.1.19 -->
+<!-- END SuperLocalMemory v4.1.21 -->
 
-SuperLocalMemory v4.1.19 · Qualixar · AGPL-3.0-or-later
+SuperLocalMemory v4.1.21 · Qualixar · AGPL-3.0-or-later

package/plugin/agents/slm-governance-advisor.md

@@ -77,4 +77,4 @@ slm-scope · slm-governance · slm-profile · slm-remember · slm-recall
 # What NOT to do
 Never session_init twice; never forget without dry-run preview; never store secrets; never bypass role checks; never claim an erasure succeeded without verifying via recall.
 
-SuperLocalMemory v4.1.19 · Qualixar · AGPL-3.0-or-later
+SuperLocalMemory v4.1.21 · Qualixar · AGPL-3.0-or-later

package/plugin/agents/slm-loop-runner.md

@@ -68,4 +68,4 @@ assessment. The gate is the authority.
 
 ---
 
-SuperLocalMemory v4.1.19 · Qualixar · AGPL-3.0-or-later
+SuperLocalMemory v4.1.21 · Qualixar · AGPL-3.0-or-later

package/plugin/agents/slm-memory-advisor.md

@@ -47,4 +47,4 @@ slm-recall · slm-remember · slm-session · slm-scope · slm-profile · slm-gov
 # What NOT to do
 Never session_init twice; never forget dry_run=False without reporting preview; never dump a whole file into remember; never invent a memory; never claim "saved" without success:true / clean CLI exit; never bypass scope or governance restrictions.
 
-SuperLocalMemory v4.1.19 · Qualixar · AGPL-3.0-or-later
+SuperLocalMemory v4.1.21 · Qualixar · AGPL-3.0-or-later

package/plugin/agents/slm-optimize-advisor.md

@@ -41,4 +41,4 @@ slm-compress · slm-cache · slm-status · slm-profile
 # What NOT to do
 Never compress code-for-edit/JSON-to-parse/<500 chars; never store secrets/ccr_ids; never let optimize failure block/alter the task; never claim a specific savings %; never carry ccr_ids across profile switches.
 
-SuperLocalMemory v4.1.19 · Qualixar · AGPL-3.0-or-later
+SuperLocalMemory v4.1.21 · Qualixar · AGPL-3.0-or-later

package/plugin/hooks/hooks.json

@@ -5,7 +5,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "slm hook mandate 2>/dev/null || true",
+            "command": "\"${CLAUDE_PLUGIN_ROOT}/scripts/slm-run\" hook mandate || true",
             "timeout": 5000
           }
         ]
@@ -22,7 +22,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "slm hook start 2>/dev/null || true",
+            "command": "\"${CLAUDE_PLUGIN_ROOT}/scripts/slm-run\" hook start || true",
             "timeout": 15000
           }
         ]
@@ -34,7 +34,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "slm hook checkpoint 2>/dev/null || true",
+            "command": "\"${CLAUDE_PLUGIN_ROOT}/scripts/slm-run\" hook checkpoint || true",
             "timeout": 5000
           }
         ]
@@ -44,7 +44,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "slm hook post_tool_outcome 2>/dev/null || true",
+            "command": "\"${CLAUDE_PLUGIN_ROOT}/scripts/slm-run\" hook post_tool_outcome || true",
             "timeout": 5000
           }
         ]
@@ -55,7 +55,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "slm hook user_prompt_rehash 2>/dev/null || true",
+            "command": "\"${CLAUDE_PLUGIN_ROOT}/scripts/slm-run\" hook user_prompt_rehash || true",
             "timeout": 5000
           }
         ]
@@ -64,7 +64,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "slm hook topic_shift 2>/dev/null || true",
+            "command": "\"${CLAUDE_PLUGIN_ROOT}/scripts/slm-run\" hook topic_shift || true",
             "timeout": 3000
           }
         ]
@@ -75,12 +75,12 @@
         "hooks": [
           {
             "type": "command",
-            "command": "slm hook stop 2>/dev/null || true",
+            "command": "\"${CLAUDE_PLUGIN_ROOT}/scripts/slm-run\" hook stop || true",
             "timeout": 10000
           },
           {
             "type": "command",
-            "command": "slm hook stop_outcome 2>/dev/null || true",
+            "command": "\"${CLAUDE_PLUGIN_ROOT}/scripts/slm-run\" hook stop_outcome || true",
             "timeout": 10000
           }
         ]
@@ -92,7 +92,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "slm hook before_web 2>/dev/null || true",
+            "command": "\"${CLAUDE_PLUGIN_ROOT}/scripts/slm-run\" hook before_web || true",
             "timeout": 5000
           }
         ]

package/plugin/requirements.txt

@@ -1 +1 @@
-superlocalmemory==4.1.19
+superlocalmemory==4.1.21

package/plugin/scripts/ensure-venv.sh

@@ -8,7 +8,7 @@
 #   CLAUDE_PLUGIN_ROOT  — plugin installation dir (ephemeral, contains scripts/ + requirements.txt)
 #   CLAUDE_PLUGIN_DATA  — persistent data dir (venv lives here, survives plugin updates)
 #
-# Exit codes: 0 = venv ready   non-0 = failure (logged to stderr)
+# Exit codes: 0 = venv ready, or not needed   non-0 = failure (logged to stderr)
 # All output goes to stderr only (stdout reserved for MCP stdio protocol).
 
 set -euo pipefail
@@ -24,6 +24,28 @@ set -euo pipefail
 # ---------------------------------------------------------------------------
 exec 1>&2
 
+# ---------------------------------------------------------------------------
+# Only build a venv that will be used.
+#
+# slm-launch prefers an slm that is already installed, and SLM_LAUNCHER can name
+# one outright. In those cases the plugin venv is never run, so neither its
+# Python requirement nor its install cost should land on this session. The
+# question is answered by slm-resolve.sh, the same code the launcher and the
+# hooks use, so the three can never disagree about which slm is in play.
+# ---------------------------------------------------------------------------
+# shellcheck source=slm-resolve.sh
+. "$(dirname "${BASH_SOURCE[0]}")/slm-resolve.sh"
+if ! slm_choose; then
+    # A venv cannot fix a launcher that is configured to use something else.
+    # Report it, and leave the session alone: the MCP server reports it again.
+    echo "SLM plugin: venv not needed — ${SLM_RESOLVE_ERROR}" >&2
+    exit 0
+fi
+if [ "${SLM_CHOICE}" != "plugin" ]; then
+    echo "SLM plugin: venv not needed — using ${SLM_CHOICE_BIN} (${SLM_CHOICE})." >&2
+    exit 0
+fi
+
 # ---------------------------------------------------------------------------
 # Python >= 3.12 guard
 # ---------------------------------------------------------------------------

package/plugin/scripts/slm-launch

@@ -31,7 +31,12 @@
 # user's decision, and a plugin that narrows it takes away tools they configured.
 
 _SLM_DATA="${SLM_DATA_DIR:-$HOME/.superlocalmemory}"
-_PLUGIN_BIN="${CLAUDE_PLUGIN_DATA:+${CLAUDE_PLUGIN_DATA}/venv/bin/slm}"
+
+# Which slm to run is decided in ONE place, shared with the hooks and the venv
+# bootstrap, so the three can never disagree about which install is in play.
+# shellcheck source=slm-resolve.sh
+. "$(dirname "${BASH_SOURCE[0]}")/slm-resolve.sh"
+_PLUGIN_BIN="$(slm_plugin_bin)"
 
 # Is a daemon already up for this data dir? If so we join it rather than boot a
 # second one — one writer for every session that shares the store.
@@ -41,91 +46,78 @@ if [ -f "${_SLM_DATA}/daemon.pid" ] \
     _daemon_running=1
 fi
 
-# Expand a leading ~ WITHOUT eval — eval on an environment value is a
-# command-injection foot-gun. Only tilde expansion; pass expanded paths for vars.
-_expand_tilde() {
-    case "$1" in
-        "~"*) printf '%s' "${HOME}${1#\~}" ;;
-        *)    printf '%s' "$1" ;;
-    esac
-}
-
 # Start a daemon with the chosen binary unless one is already serving.
 _ensure_daemon() {
     [ "${_daemon_running}" -eq 1 ] && return 0
     "$1" serve start >&2
 }
 
-case "${SLM_LAUNCHER:-auto}" in
-    auto)
-        # 1. An slm already on PATH — the pip, pipx or npm install the user has.
-        #    This is the common case and the one that must not be forked.
-        if command -v slm >/dev/null 2>&1; then
-            # Both orderings end up here, and both must converge on ONE
-            # environment:
-            #   installed first, plugin second -> nothing was ever forked;
-            #   plugin first, installed second -> the plugin venv is now the
-            #     older copy and must stop being used.
-            # They already share one store, because this launcher sets no
-            # SLM_DATA_DIR and both resolve ~/.superlocalmemory. What is left to
-            # go wrong is two DIFFERENT VERSIONS serving that one store: whoever
-            # starts the daemon first decides which code the other one talks to.
-            # So when a leftover plugin venv is found beside a real install, say
-            # so, with both versions named. Silent duplication is how a store
-            # ends up served by a release nobody thinks they are running.
-            if [ -n "${_PLUGIN_BIN}" ] && [ -x "${_PLUGIN_BIN}" ]; then
-                _sys_v="$(slm --version 2>/dev/null | awk '{print $NF}')"
-                _plg_v="$("${_PLUGIN_BIN}" --version 2>/dev/null | awk '{print $NF}')"
-                if [ -n "${_plg_v}" ] && [ "${_sys_v}" != "${_plg_v}" ]; then
-                    echo "SLM plugin: using the installed slm ${_sys_v:-?} and IGNORING a leftover plugin venv at ${_PLUGIN_BIN} (${_plg_v})." >&2
-                    echo "  One store, one version. Remove the venv to stop it drifting:  rm -rf \"${CLAUDE_PLUGIN_DATA}/venv\"" >&2
-                fi
-            fi
-            _ensure_daemon "$(command -v slm)" || \
-                echo "SLM plugin: daemon did not start; continuing as a direct MCP writer." >&2
-            exec slm mcp
-        fi
-        # 2. No system install. Use the plugin's own venv if this host gave us
-        #    somewhere to keep one.
+if ! slm_choose; then
+    echo "${SLM_RESOLVE_ERROR}" >&2
+    exit 1
+fi
+
+case "${SLM_LAUNCHER:-auto}:${SLM_CHOICE}" in
+    auto:system)
+        # An slm already on PATH — the pip, pipx or npm install the user has.
+        # This is the common case and the one that must not be forked.
+        #
+        # Both orderings end up here, and both must converge on ONE
+        # environment:
+        #   installed first, plugin second -> nothing was ever forked;
+        #   plugin first, installed second -> the plugin venv is now the
+        #     older copy and must stop being used.
+        # They already share one store, because this launcher sets no
+        # SLM_DATA_DIR and both resolve ~/.superlocalmemory. What is left to
+        # go wrong is two DIFFERENT VERSIONS serving that one store: whoever
+        # starts the daemon first decides which code the other one talks to.
+        # So when a leftover plugin venv is found beside a real install, say
+        # so, with both versions named. Silent duplication is how a store
+        # ends up served by a release nobody thinks they are running.
         if [ -n "${_PLUGIN_BIN}" ] && [ -x "${_PLUGIN_BIN}" ]; then
-            if ! _ensure_daemon "${_PLUGIN_BIN}"; then
-                echo "SLM plugin: unable to start the owned daemon; refusing a direct MCP writer." >&2
-                exit 1
+            _sys_v="$(slm --version 2>/dev/null | awk '{print $NF}')"
+            _plg_v="$("${_PLUGIN_BIN}" --version 2>/dev/null | awk '{print $NF}')"
+            if [ -n "${_plg_v}" ] && [ "${_sys_v}" != "${_plg_v}" ]; then
+                echo "SLM plugin: using the installed slm ${_sys_v:-?} and IGNORING a leftover plugin venv at ${_PLUGIN_BIN} (${_plg_v})." >&2
+                echo "  One store, one version. Remove the venv to stop it drifting:  rm -rf \"${CLAUDE_PLUGIN_DATA}/venv\"" >&2
             fi
-            exec "${_PLUGIN_BIN}" mcp
         fi
-        # 3. Neither. Say which two things were looked for, so the fix is obvious.
-        echo "SLM plugin: no SuperLocalMemory found." >&2
-        echo "  Looked for: 'slm' on PATH, and ${_PLUGIN_BIN:-a plugin venv (this host sets no plugin data dir)}." >&2
-        echo "  Install it with:  pipx install superlocalmemory" >&2
-        exit 1
-        ;;
-    system)
-        if ! command -v slm >/dev/null 2>&1; then
-            echo "SLM_LAUNCHER=system but no 'slm' on PATH." >&2
-            exit 1
-        fi
-        _ensure_daemon "$(command -v slm)" || true
+        _ensure_daemon "${SLM_CHOICE_BIN}" || \
+            echo "SLM plugin: daemon did not start; continuing as a direct MCP writer." >&2
         exec slm mcp
         ;;
-    plugin)
-        if [ -z "${_PLUGIN_BIN}" ]; then
-            echo "SLM_LAUNCHER=plugin but this host sets no CLAUDE_PLUGIN_DATA." >&2
+    auto:plugin)
+        # No system install. Use the plugin's own venv, once SessionStart has
+        # built it. If it is not there either, say which two things were looked
+        # for, so the fix is obvious.
+        if [ "${SLM_CHOICE_READY}" -ne 1 ]; then
+            slm_not_found_message >&2
+            echo >&2
             exit 1
         fi
-        if ! _ensure_daemon "${_PLUGIN_BIN}"; then
+        if ! _ensure_daemon "${SLM_CHOICE_BIN}"; then
             echo "SLM plugin: unable to start the owned daemon; refusing a direct MCP writer." >&2
             exit 1
         fi
-        exec "${_PLUGIN_BIN}" mcp
+        exec "${SLM_CHOICE_BIN}" mcp
         ;;
-    *)
-        _SLM_BIN="$(_expand_tilde "${SLM_LAUNCHER}")"
-        if [ ! -x "${_SLM_BIN}" ]; then
-            echo "SLM_LAUNCHER is not an executable slm binary: ${_SLM_BIN}" >&2
+    system:system)
+        _ensure_daemon "${SLM_CHOICE_BIN}" || true
+        exec slm mcp
+        ;;
+    plugin:plugin)
+        if ! _ensure_daemon "${SLM_CHOICE_BIN}"; then
+            echo "SLM plugin: unable to start the owned daemon; refusing a direct MCP writer." >&2
             exit 1
         fi
-        _ensure_daemon "${_SLM_BIN}" || true
-        exec "${_SLM_BIN}" mcp
+        exec "${SLM_CHOICE_BIN}" mcp
+        ;;
+    *:explicit)
+        _ensure_daemon "${SLM_CHOICE_BIN}" || true
+        exec "${SLM_CHOICE_BIN}" mcp
+        ;;
+    *)
+        echo "SLM plugin: internal error: unexpected launcher choice '${SLM_LAUNCHER:-auto}:${SLM_CHOICE}'." >&2
+        exit 1
         ;;
 esac

package/plugin/scripts/slm-launch.bat

@@ -2,7 +2,16 @@
 :: slm-launch.bat — WP-F SuperLocalMemory MCP launcher (Windows)
 ::
 :: Cross-platform counterpart: slm-launch (POSIX bash)
-:: Referenced by plugin/.mcp.json as the MCP server command (Windows picks .bat automatically).
+::
+:: NOT reached through plugin/.mcp.json. That file names the extensionless
+:: "${CLAUDE_PLUGIN_ROOT}/scripts/slm-launch", and a host that spawns it without
+:: a shell (CreateProcess, Node spawn without shell) never resolves it to this
+:: .bat: CreateProcess cannot start a batch file and only appends .exe to a
+:: name without an extension. This file runs only when started through cmd.exe
+:: (for example a hand-written MCP entry using "cmd /c"). It does not read
+:: SLM_LAUNCHER; see MCP_JSON_NOTES.md and issue #139. Since 4.1.20 the plugin's
+:: .mcp.json starts the INSTALLED slm on Windows through %ComSpec% instead.
+:: Proven on the Windows CI runner: tests/test_plugin/test_windows_mcp_spawn_premise.py
 ::
 :: Resolves the correct venv binary for Windows and joins the namespace daemon
 :: before opening MCP, preserving one writer for parallel Claude sessions.

package/plugin/scripts/slm-resolve.sh

@@ -0,0 +1,128 @@
+#!/usr/bin/env bash
+# slm-resolve.sh — which SuperLocalMemory install does this plugin use?
+#
+# Sourced, never run. ONE answer to that question, shared by every script that
+# needs it:
+#   slm-launch      the MCP server
+#   slm-run         every lifecycle hook
+#   ensure-venv.sh  the plugin venv bootstrap (it only builds what will be used)
+# Before this file each of them decided on its own, and they disagreed: the
+# launcher fell back to the plugin venv, the hooks only ever tried a bare `slm`,
+# and the bootstrap built a venv even when nothing would run it.
+#
+# Environment (all optional):
+#   SLM_LAUNCHER       auto (default) — prefer an installed slm, else the plugin venv
+#                      system         — require the slm on PATH
+#                      plugin         — require the plugin's own venv
+#                      <path>         — an explicit slm binary (leading ~ expanded)
+#   CLAUDE_PLUGIN_DATA where a plugin-owned venv lives, when there is one
+#
+# After slm_choose returns 0:
+#   SLM_CHOICE        system | plugin | explicit
+#   SLM_CHOICE_BIN    the binary that choice names
+#   SLM_CHOICE_READY  1 when SLM_CHOICE_BIN is executable now, else 0
+#                     (a plugin venv that SessionStart has not built yet is 0)
+# After slm_choose returns 1:
+#   SLM_RESOLVE_ERROR what was asked for and why it cannot be honoured
+#
+# Safe under `set -euo pipefail`: every variable read has a default.
+
+# Expand a leading ~ WITHOUT eval — eval on an environment value is a
+# command-injection foot-gun. Only tilde expansion; pass expanded paths for vars.
+_slm_expand_tilde() {
+    case "$1" in
+        "~"*) printf '%s' "${HOME}${1#\~}" ;;
+        *)    printf '%s' "$1" ;;
+    esac
+}
+
+# Where the plugin venv keeps slm. Empty when the host gives the plugin no data
+# directory (only Claude Code sets CLAUDE_PLUGIN_DATA). A venv built on Windows
+# keeps entry points in Scripts\ with an .exe suffix, and Git Bash is where these
+# scripts run there.
+#
+# _slm_plugin_bin_var sets SLM_PLUGIN_BIN without a subshell (hooks run this on
+# every tool call); slm_plugin_bin prints it, for callers that want a value.
+_slm_plugin_bin_var() {
+    SLM_PLUGIN_BIN=""
+    [ -n "${CLAUDE_PLUGIN_DATA:-}" ] || return 0
+    case "${OSTYPE:-}" in
+        msys*|cygwin*|win32*) SLM_PLUGIN_BIN="${CLAUDE_PLUGIN_DATA}/venv/Scripts/slm.exe" ;;
+        *)                    SLM_PLUGIN_BIN="${CLAUDE_PLUGIN_DATA}/venv/bin/slm" ;;
+    esac
+}
+
+slm_plugin_bin() {
+    _slm_plugin_bin_var
+    printf '%s' "${SLM_PLUGIN_BIN}"
+}
+
+# The message for "nothing to run", naming both places that were looked in.
+slm_not_found_message() {
+    local _pbin
+    _pbin="$(slm_plugin_bin)"
+    printf '%s\n' "SLM plugin: no SuperLocalMemory found."
+    printf '%s\n' "  Looked for: 'slm' on PATH, and ${_pbin:-a plugin venv (this host sets no plugin data dir)}."
+    printf '%s'   "  Install it with:  pipx install superlocalmemory"
+}
+
+_slm_set_choice() {
+    SLM_CHOICE="$1"
+    SLM_CHOICE_BIN="$2"
+    if [ -n "$2" ] && [ -x "$2" ]; then SLM_CHOICE_READY=1; else SLM_CHOICE_READY=0; fi
+}
+
+slm_choose() {
+    SLM_CHOICE=""
+    SLM_CHOICE_BIN=""
+    SLM_CHOICE_READY=0
+    SLM_RESOLVE_ERROR=""
+    local _pbin _sys
+    _slm_plugin_bin_var
+    _pbin="${SLM_PLUGIN_BIN}"
+
+    case "${SLM_LAUNCHER:-auto}" in
+        auto)
+            # 1. An slm already on PATH — the pip, pipx or npm install the user
+            #    has. The common case, and the one that must not be forked.
+            if _sys="$(command -v slm 2>/dev/null)" && [ -n "${_sys}" ]; then
+                _slm_set_choice system "${_sys}"
+                return 0
+            fi
+            # 2. No system install: the plugin's own venv, if this host gives the
+            #    plugin somewhere to keep one. It may not be built yet.
+            if [ -n "${_pbin}" ]; then
+                _slm_set_choice plugin "${_pbin}"
+                return 0
+            fi
+            # 3. Neither.
+            SLM_RESOLVE_ERROR="$(slm_not_found_message)"
+            return 1
+            ;;
+        system)
+            if _sys="$(command -v slm 2>/dev/null)" && [ -n "${_sys}" ]; then
+                _slm_set_choice system "${_sys}"
+                return 0
+            fi
+            SLM_RESOLVE_ERROR="SLM_LAUNCHER=system but no 'slm' on PATH."
+            return 1
+            ;;
+        plugin)
+            if [ -z "${_pbin}" ]; then
+                SLM_RESOLVE_ERROR="SLM_LAUNCHER=plugin but this host sets no CLAUDE_PLUGIN_DATA."
+                return 1
+            fi
+            _slm_set_choice plugin "${_pbin}"
+            return 0
+            ;;
+        *)
+            _sys="$(_slm_expand_tilde "${SLM_LAUNCHER}")"
+            if [ ! -x "${_sys}" ]; then
+                SLM_RESOLVE_ERROR="SLM_LAUNCHER is not an executable slm binary: ${_sys}"
+                return 1
+            fi
+            _slm_set_choice explicit "${_sys}"
+            return 0
+            ;;
+    esac
+}

package/plugin/scripts/slm-run

@@ -0,0 +1,76 @@
+#!/usr/bin/env bash
+# slm-run — run slm for a plugin hook, using the slm the MCP server uses.
+#
+#   "${CLAUDE_PLUGIN_ROOT}/scripts/slm-run" hook <name>
+#
+# Hooks used to call a bare `slm`, so on a machine where SuperLocalMemory lives
+# only in the plugin venv every hook was "command not found" — and the error was
+# thrown away, so memory capture stopped with nothing to say why. This wrapper
+# resolves slm with the launcher's own code (slm-resolve.sh), honours the same
+# SLM_LAUNCHER, and passes stdin, stdout and the arguments straight through.
+#
+# A hook must never break the session that runs it, so this ALWAYS exits 0
+# (Claude Code reads exit 2 as "block"). When slm cannot be found, or fails, the
+# reason goes to
+#     ${SLM_DATA_DIR:-~/.superlocalmemory}/logs/plugin-hooks.log
+# and one line naming that file goes to stderr. A hook that succeeds is silent:
+# whatever slm wrote to stderr is discarded, exactly as before.
+
+_slm_args="$*"
+_log_dir="${SLM_DATA_DIR:-${HOME}/.superlocalmemory}/logs"
+_log="${_log_dir}/plugin-hooks.log"
+_LOG_MAX_BYTES=1048576
+
+# Append one entry. Best effort: an unwritable log must not fail the hook.
+_log_entry() {
+    {
+        mkdir -p "${_log_dir}" || return 1
+        if [ -f "${_log}" ] && [ "$(wc -c < "${_log}")" -gt "${_LOG_MAX_BYTES}" ]; then
+            mv -f "${_log}" "${_log}.1"
+        fi
+        printf '%s slm %s: %s\n' "$(date -u '+%Y-%m-%dT%H:%M:%SZ')" "${_slm_args}" "$1" >> "${_log}"
+    } 2>/dev/null
+}
+
+# Record a failure, tell stderr where to look, and leave the session alone.
+_give_up() {
+    if _log_entry "$1"; then
+        echo "SLM plugin: hook 'slm ${_slm_args}' did not run cleanly; see ${_log}" >&2
+    else
+        echo "SLM plugin: hook 'slm ${_slm_args}' did not run cleanly: $1" >&2
+    fi
+    exit 0
+}
+
+_here="${BASH_SOURCE[0]%/*}"
+[ "${_here}" = "${BASH_SOURCE[0]}" ] && _here=.
+# shellcheck source=slm-resolve.sh
+if ! . "${_here}/slm-resolve.sh" 2>/dev/null; then
+    _give_up "the plugin is incomplete: ${_here}/slm-resolve.sh is missing"
+fi
+
+if ! slm_choose; then
+    _give_up "$(printf '%s' "${SLM_RESOLVE_ERROR}" | tr '\n' ' ')"
+fi
+
+if [ "${SLM_CHOICE_READY}" -ne 1 ]; then
+    if [ "${SLM_LAUNCHER:-auto}" = "auto" ]; then
+        _give_up "$(slm_not_found_message | tr '\n' ' ') (A plugin venv is built by SessionStart; the first session can run its hooks before it exists.)"
+    fi
+    _give_up "${SLM_CHOICE} slm is not executable: ${SLM_CHOICE_BIN}"
+fi
+
+# slm's stderr is kept only long enough to report a failure. The trap removes
+# it however this script ends, including a host timeout killing the hook.
+_err="$(mktemp "${TMPDIR:-/tmp}/slm-hook.XXXXXX" 2>/dev/null)" || _err=/dev/null
+trap '[ "${_err}" = /dev/null ] || rm -f "${_err}"' EXIT
+trap 'exit 0' HUP INT TERM
+
+"${SLM_CHOICE_BIN}" "$@" 2>"${_err}"
+_rc=$?
+if [ "${_rc}" -ne 0 ]; then
+    _detail=""
+    [ "${_err}" = /dev/null ] || _detail="$(tail -n 5 "${_err}" 2>/dev/null | tr '\n' ' ')"
+    _give_up "exit ${_rc} from ${SLM_CHOICE_BIN} (${SLM_CHOICE})${_detail:+: ${_detail}}"
+fi
+exit 0

package/plugin/skills/slm-cache/SKILL.md

@@ -145,4 +145,4 @@ These subcommands control daemon-level cache settings. They do not read or write
 
 ---
 
-SuperLocalMemory v4.1.19 · Qualixar · AGPL-3.0-or-later
+SuperLocalMemory v4.1.21 · Qualixar · AGPL-3.0-or-later

package/plugin/skills/slm-compress/SKILL.md

@@ -147,4 +147,4 @@ Content over 1 MB (1 000 000 bytes UTF-8) is processed but `reversible` is force
 
 ---
 
-SuperLocalMemory v4.1.19 · Qualixar · AGPL-3.0-or-later
+SuperLocalMemory v4.1.21 · Qualixar · AGPL-3.0-or-later

package/plugin/skills/slm-governance/SKILL.md

@@ -245,4 +245,4 @@ Before running any destructive operation (`forget`, `compact_memories`):
 
 ---
 
-*SuperLocalMemory v4.1.19 · Qualixar · AGPL-3.0-or-later*
+*SuperLocalMemory v4.1.21 · Qualixar · AGPL-3.0-or-later*

package/plugin/skills/slm-graph/SKILL.md

@@ -312,4 +312,4 @@ profile. See `slm-profile` for the full profile switching workflow.
 
 ---
 
-SuperLocalMemory v4.1.19 · Qualixar · AGPL-3.0-or-later
+SuperLocalMemory v4.1.21 · Qualixar · AGPL-3.0-or-later

package/plugin/skills/slm-loop/SKILL.md

@@ -96,4 +96,4 @@ paused, name the approval needed; when errored, quote the short detail.
 
 ---
 
-SuperLocalMemory v4.1.19 · Qualixar · AGPL-3.0-or-later
+SuperLocalMemory v4.1.21 · Qualixar · AGPL-3.0-or-later

package/plugin/skills/slm-mesh/SKILL.md

@@ -279,4 +279,4 @@ mesh availability.
 
 ---
 
-*SuperLocalMemory v4.1.19 · Qualixar · AGPL-3.0-or-later*
+*SuperLocalMemory v4.1.21 · Qualixar · AGPL-3.0-or-later*

package/plugin/skills/slm-profile/SKILL.md

@@ -32,8 +32,8 @@ project, a client engagement, a production vs staging environment.
 |---------|-------|-------------|
 | `core` | 18 tools — remember, recall, search, session, optimize | Minimal footprint, no code tools |
 | `code` | 38 tools — core + portable Brain evidence + code graph + profile switching + bounded loops + memory-kind management | Default for IDE/coding agents |
-| `full` | 54 tools — code + all memory ops + mesh + bounded loops | Multi-session, team workflows |
-| `power` | 66 tools — full + governance + behavioral tools | Enterprise, admin, audit use cases |
+| `full` | 56 tools — code + all memory ops + mesh + bounded loops | Multi-session, team workflows |
+| `power` | 68 tools — full + governance + behavioral tools | Enterprise, admin, audit use cases |
 | `mesh` | 8 tools — mesh coordination only | Lightweight cross-session signalling |
 
 The profile is set at MCP server startup via `SLM_MCP_PROFILE` in the MCP config.
@@ -146,4 +146,4 @@ Name them differently in your MCP config (e.g. `superlocalmemory-personal` and
 
 ---
 
-*SuperLocalMemory v4.1.19 · Qualixar · AGPL-3.0-or-later*
+*SuperLocalMemory v4.1.21 · Qualixar · AGPL-3.0-or-later*