gitmem-mcp 1.0.4 → 1.0.6

package/CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.0.6] - 2026-02-16
+
+### Fixed
+- **Session close crash on malformed scars_to_record**: Agents writing `{title, description, severity}` (create_learning shape) instead of `{scar_identifier, reference_type, reference_context}` (ScarUsageEntry shape) in closing payload caused `Cannot read properties of undefined (reading 'length')` crash in `formatCloseDisplay`. Now auto-coerces salvageable entries and drops invalid ones with warnings.
+- **Defensive property access in formatCloseDisplay**: Guard against undefined `scar_identifier`, `reference_type`, and `reference_context` as belt-and-suspenders protection.
+
 ## [1.0.3] - 2026-02-15
 
 ### Changed

package/dist/tools/session-close.js

@@ -297,9 +297,10 @@ function formatCloseDisplay(sessionId, compliance, params, learningsCount, succe
             const ref = s.reference_type === "explicit" ? "applied" :
                 s.reference_type === "implicit" ? "implicit" :
                     s.reference_type === "acknowledged" ? "ack'd" :
-                        s.reference_type === "refuted" ? "REFUTED" : s.reference_type;
-            const id = s.scar_identifier.length > 12 ? s.scar_identifier.slice(0, 8) : s.scar_identifier;
-            lines.push(`  ${id}  ${ref.padEnd(8)}  ${truncate(s.reference_context, 50)}`);
+                        s.reference_type === "refuted" ? "REFUTED" : (s.reference_type || "?");
+            const scarId = s.scar_identifier || "(unknown)";
+            const id = scarId.length > 12 ? scarId.slice(0, 8) : scarId;
+            lines.push(`  ${id}  ${ref.padEnd(8)}  ${truncate(s.reference_context || "", 50)}`);
         }
     }
     // Transcript status
@@ -737,6 +738,35 @@ export async function sessionClose(params) {
     catch (error) {
         console.warn("[session_close] Failed to read closing-payload.json:", error);
     }
+    // Sanitize scars_to_record: agents frequently write create_learning shape
+    // ({title, description, severity}) instead of ScarUsageEntry shape
+    // ({scar_identifier, reference_type, reference_context}).
+    // Filter out entries missing required fields to prevent crashes in formatCloseDisplay.
+    if (Array.isArray(params.scars_to_record) && params.scars_to_record.length > 0) {
+        const valid = [];
+        for (const entry of params.scars_to_record) {
+            if (entry && typeof entry === "object" && typeof entry.scar_identifier === "string" && entry.scar_identifier.length > 0) {
+                valid.push(entry);
+            }
+            else {
+                // Try to salvage: if agent wrote {title, description} instead of {scar_identifier, reference_context}
+                const raw = entry;
+                if (raw && typeof raw.title === "string" && raw.title.length > 0) {
+                    valid.push({
+                        scar_identifier: raw.title,
+                        reference_type: (typeof raw.reference_type === "string" ? raw.reference_type : "acknowledged"),
+                        reference_context: (typeof raw.description === "string" ? raw.description : "auto-coerced from payload"),
+                        surfaced_at: new Date().toISOString(),
+                    });
+                    console.error(`[session_close] Coerced malformed scar entry "${raw.title}" → scar_identifier`);
+                }
+                else {
+                    console.error(`[session_close] Dropped malformed scars_to_record entry: ${JSON.stringify(entry).slice(0, 100)}`);
+                }
+            }
+        }
+        params.scars_to_record = valid.length > 0 ? valid : undefined;
+    }
     // Normalize closing_reflection field aliases (q1_broke → what_broke, etc.)
     // Agents frequently guess field names instead of using canonical keys from CLOSING_QUESTIONS.
     if (params.closing_reflection && typeof params.closing_reflection === "object") {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gitmem-mcp",
-  "version": "1.0.4",
+  "version": "1.0.6",
   "description": "Institutional memory for AI coding agents. Memory that compounds.",
   "type": "module",
   "main": "dist/index.js",