gitmem-mcp 1.4.1 → 1.4.2

package/CHANGELOG.md

@@ -7,6 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.4.2] - 2026-02-22
+
+### Fixed
+- **Scar usage `execution_successful` nulls eliminated**: N_A confirmations now record `true` (was null/undefined). Q6 text matches now include `execution_successful: true` (was omitted). Fixes 80% null rate in scar effectiveness data.
+- **Auto-bridge fires on all session closes**: Previously required Q6 `scars_applied` to be non-empty. Now fires whenever no explicit `scars_to_record` is provided, ensuring confirmations from `confirm_scars` always get recorded.
+- **Surfaced scars survive MCP restart**: `getSurfacedScars()` now recovers from the active-sessions registry when `currentSession` is null after MCP restart. Scars surfaced early in a session are no longer silently lost.
+- **Session close display shows scar titles**: `reference_context` now leads with the scar title instead of boilerplate. Display uses +/! indicators for applied/refuted scars.
+
 ## [1.4.1] - 2026-02-22
 
 ### Added

package/dist/services/session-state.js

@@ -12,7 +12,9 @@
  * This allows recall() to always assign variants even without explicit parameters.
  */
 import fs from "fs";
+import * as os from "os";
 import { getSessionPath } from "./gitmem-dir.js";
+import { listActiveSessions } from "./active-sessions.js";
 // Global session state (single active session per MCP server instance)
 let currentSession = null;
 /**
@@ -105,7 +107,7 @@ export function getSurfacedScars() {
     if (currentSession?.surfacedScars && currentSession.surfacedScars.length > 0) {
         return currentSession.surfacedScars;
     }
-    // Fallback: recover from per-session file if in-memory was lost (MCP restart)
+    // Fallback 1: recover from per-session file if in-memory was lost (MCP restart)
     if (currentSession?.sessionId) {
         try {
             const sessionFilePath = getSessionPath(currentSession.sessionId, "session.json");
@@ -122,6 +124,32 @@ export function getSurfacedScars() {
             console.warn("[session-state] Failed to recover surfaced scars from file:", error);
         }
     }
+    // Fallback 2: if currentSession is null (MCP restarted completely), find session from registry.
+    // After MCP restart the PID changes, but the active-sessions registry may still have
+    // the previous session entry (pruneStale adopts dead-PID sessions on same hostname).
+    if (!currentSession) {
+        try {
+            const hostname = os.hostname();
+            const sessions = listActiveSessions();
+            const candidates = sessions
+                .filter(s => s.hostname === hostname)
+                .sort((a, b) => new Date(b.started_at).getTime() - new Date(a.started_at).getTime());
+            if (candidates.length > 0) {
+                const recoveredId = candidates[0].session_id;
+                const sessionFilePath = getSessionPath(recoveredId, "session.json");
+                if (fs.existsSync(sessionFilePath)) {
+                    const data = JSON.parse(fs.readFileSync(sessionFilePath, "utf-8"));
+                    if (data.surfaced_scars && Array.isArray(data.surfaced_scars) && data.surfaced_scars.length > 0) {
+                        console.error(`[session-state] Recovered ${data.surfaced_scars.length} surfaced scars from registry session ${recoveredId.slice(0, 8)}`);
+                        return data.surfaced_scars;
+                    }
+                }
+            }
+        }
+        catch (error) {
+            console.warn("[session-state] Failed to recover surfaced scars from registry:", error);
+        }
+    }
     return [];
 }
 /**

package/dist/tools/session-close.js

@@ -288,15 +288,12 @@ function formatCloseDisplay(sessionId, compliance, params, learningsCount, succe
             lines.push(`  ${dimText("d")} ${truncate(d.title, 68)}`);
         }
     }
-    // Scars applied — compact table, only if any were applied
+    // Scars applied — show titles with status indicator
     if (scarsApplied > 0) {
         lines.push("");
         for (const s of params.scars_to_record.filter(s => s.reference_type !== "none")) {
-            const ref = s.reference_type === "explicit" ? "applied" :
-                s.reference_type === "implicit" ? "implicit" :
-                    s.reference_type === "acknowledged" ? "ack'd" :
-                        s.reference_type === "refuted" ? `${ANSI.yellow}REFUTED${ANSI.reset}` : (s.reference_type || "?");
-            lines.push(`  ${dimText(ref.padEnd(8))} ${truncate(s.reference_context || "", 60)}`);
+            const indicator = s.reference_type === "refuted" ? `${ANSI.yellow}!${ANSI.reset}` : STATUS.pass;
+            lines.push(`  ${indicator} ${truncate(s.reference_context || s.scar_identifier || "", 70)}`);
         }
     }
     // Transcript — only on failure
@@ -599,12 +596,13 @@ function bridgeScarsToUsageRecords(normalizedScarsApplied, sessionId, agentIdent
                 if (reflection) {
                     // Reflection provides definitive signal
                     executionSuccessful = reflection.outcome === "OBEYED" ? true : false;
-                    context = `Confirmed: ${confirmation.decision} → Reflected: ${reflection.outcome} — ${reflection.evidence.slice(0, 80)}`;
+                    context = `${scar.scar_title.slice(0, 60)} (${reflection.outcome})`;
                 }
                 else {
-                    // Fall back to confirmation-based default (Option A)
-                    executionSuccessful = confirmation.decision === "APPLYING" ? true : undefined;
-                    context = `Confirmed via confirm_scars: ${confirmation.decision} — ${confirmation.evidence.slice(0, 100)}`;
+                    // Fall back to confirmation-based default
+                    // APPLYING/N_A = task proceeded normally (true), REFUTED = outcome unknown (null)
+                    executionSuccessful = confirmation.decision === "REFUTED" ? undefined : true;
+                    context = `${scar.scar_title.slice(0, 60)} (${confirmation.decision})`;
                 }
                 autoBridgedScars.push({
                     scar_identifier: scar.scar_id,
@@ -636,7 +634,8 @@ function bridgeScarsToUsageRecords(normalizedScarsApplied, sessionId, agentIdent
                     agent: agentIdentity,
                     surfaced_at: match.surfaced_at,
                     reference_type: "acknowledged",
-                    reference_context: `Auto-bridged from Q6 answer: "${scarApplied}"`,
+                    reference_context: `${match.scar_title.slice(0, 60)} (Q6 match)`,
+                    execution_successful: true,
                     variant_id: match.variant_id,
                 });
             }
@@ -651,7 +650,7 @@ function bridgeScarsToUsageRecords(normalizedScarsApplied, sessionId, agentIdent
                     agent: agentIdentity,
                     surfaced_at: scar.surfaced_at,
                     reference_type: "none",
-                    reference_context: `Surfaced during ${scar.source} but not mentioned in closing reflection`,
+                    reference_context: `${scar.scar_title.slice(0, 60)} (not addressed)`,
                     execution_successful: false,
                     variant_id: scar.variant_id,
                 });
@@ -1046,8 +1045,9 @@ export async function sessionClose(params) {
     }
     // Auto-bridge Q6 answers to scar_usage records
     const normalizedScarsApplied = normalizeScarsApplied(params.closing_reflection?.scars_applied);
-    if ((!params.scars_to_record || params.scars_to_record.length === 0) &&
-        normalizedScarsApplied.length > 0) {
+    // Auto-bridge surfaced scars to usage records whenever no explicit scars_to_record.
+    // Fires even when Q6 is empty — Pass 1 matches confirmations, Pass 3 catches ignored scars.
+    if (!params.scars_to_record || params.scars_to_record.length === 0) {
         const bridgedScars = bridgeScarsToUsageRecords(normalizedScarsApplied, sessionId, agentIdentity);
         if (bridgedScars.length > 0) {
             params = { ...params, scars_to_record: bridgedScars };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gitmem-mcp",
-  "version": "1.4.1",
+  "version": "1.4.2",
   "mcpName": "io.github.gitmem-dev/gitmem",
   "description": "Persistent learning memory for AI coding agents. Memory that compounds.",
   "type": "module",