gitmem-mcp 1.4.4 → 1.6.0

package/dist/tools/session-close.js

@@ -10,9 +10,9 @@ import { v4 as uuidv4 } from "uuid";
 import { detectAgent } from "../services/agent-detection.js";
 import * as supabase from "../services/supabase-client.js";
 import { embed, isEmbeddingAvailable } from "../services/embedding.js";
-import { hasSupabase, getTableName } from "../services/tier.js";
+import { hasSupabase, hasProInsights, getTableName } from "../services/tier.js";
 import { getStorage } from "../services/storage.js";
-import { clearCurrentSession, getSurfacedScars, getConfirmations, getReflections, getObservations, getChildren, getThreads, getSessionActivity } from "../services/session-state.js";
+import { clearCurrentSession, getSurfacedScars, getConfirmations, getReflections, getObservations, getChildren, getThreads, getSessionActivity, isRecallCalled } from "../services/session-state.js";
 import { normalizeThreads, mergeThreadStates, migrateStringThread, saveThreadsFile } from "../services/thread-manager.js"; // 
 import { deduplicateThreadList } from "../services/thread-dedup.js";
 import { syncThreadsToSupabase, loadOpenThreadEmbeddings } from "../services/thread-supabase.js";
@@ -20,6 +20,7 @@ import { validateSessionClose, buildCloseCompliance, } from "../services/complia
 import { normalizeReflectionKeys } from "../constants/closing-questions.js";
 import { Timer, recordMetrics, buildPerformanceData, updateRelevanceData, } from "../services/metrics.js";
 import { wrapDisplay, truncate, productLine, dimText, STATUS, ANSI } from "../services/display-protocol.js";
+import { queryScarUsageByDateRange, enrichScarUsageTitles, formatBlindspotSnippet } from "../services/analytics.js";
 import { recordScarUsageBatch } from "./record-scar-usage-batch.js";
 import { getEffectTracker } from "../services/effect-tracker.js";
 import { saveTranscript } from "./save-transcript.js";
@@ -264,7 +265,7 @@ async function sessionCloseFree(params, timer) {
         };
     }
 }
-function formatCloseDisplay(sessionId, compliance, params, learningsCount, success, errors, transcriptStatus) {
+function formatCloseDisplay(sessionId, compliance, params, learningsCount, success, errors, transcriptStatus, blindspotSnippet) {
     const lines = [];
     // Header: branded product line
     const status = success ? STATUS.complete : STATUS.failed;
@@ -320,6 +321,11 @@ function formatCloseDisplay(sessionId, compliance, params, learningsCount, succe
             lines.push(`  ${indicator} ${truncate(s.reference_context || s.scar_identifier || "", 70)}`);
         }
     }
+    // Pro: blindspot section
+    if (blindspotSnippet) {
+        lines.push("");
+        lines.push(blindspotSnippet);
+    }
     // Transcript — only on failure
     if (transcriptStatus && !transcriptStatus.saved) {
         lines.push("");
@@ -858,13 +864,56 @@ export async function sessionClose(params) {
                 `(${Math.round(activity.duration_min)} min, no substantive activity). ` +
                 `Proceeding with standard as requested.`);
         }
+        // Hard gate: quick close requires session under 30 minutes
+        if (params.close_type === "quick" && activity.duration_min >= 30) {
+            return {
+                success: false,
+                session_id: params.session_id || "",
+                close_compliance: {
+                    close_type: "quick",
+                    agent: detectAgent().agent,
+                    checklist_displayed: false,
+                    questions_answered_by_agent: false,
+                    human_asked_for_corrections: false,
+                    learnings_stored: 0,
+                    scars_applied: 0,
+                },
+                validation_errors: [
+                    `Session has been active for ${Math.round(activity.duration_min)} minutes. ` +
+                        `Quick close requires sessions under 30 minutes. Use close_type: "standard".`,
+                ],
+                performance: buildPerformanceData("session_close", timer.stop(), 0),
+            };
+        }
         if (params.close_type === "quick" && recommendedLevel === "full") {
-            // Warn but don't reject — agent chose quick on a substantive session
+            // Warn but don't reject — agent chose quick on a substantive session under 30 min
             console.error(`[session_close] Warning: "quick" close on substantive session ` +
                 `(${Math.round(activity.duration_min)} min, ${activity.recall_count} recalls, ` +
                 `${activity.observation_count} observations). Consider "standard" close.`);
         }
     }
+    // Hard gate: standard close requires at least one recall() call
+    // Exemptions: quick (micro sessions), autonomous (CODA-1), hasReflection (agent already wrote full reflection)
+    if (params.close_type === "standard" && !isRecallCalled() && !hasReflection) {
+        return {
+            success: false,
+            session_id: params.session_id || "",
+            close_compliance: {
+                close_type: "standard",
+                agent: detectAgent().agent,
+                checklist_displayed: false,
+                questions_answered_by_agent: false,
+                human_asked_for_corrections: false,
+                learnings_stored: 0,
+                scars_applied: 0,
+            },
+            validation_errors: [
+                `No recall() was run this session. Standard close requires at least one recall. ` +
+                    `Run recall("session close ceremony") first, then retry session_close.`,
+            ],
+            performance: buildPerformanceData("session_close", timer.stop(), 0),
+        };
+    }
     // Free tier: simple local persistence, skip Supabase recovery and compliance
     if (!hasSupabase()) {
         return sessionCloseFree(params, timer);
@@ -1104,11 +1153,31 @@ export async function sessionClose(params) {
             params = { ...params, scars_to_record: bridgedScars };
         }
     }
+    // Pro: fetch blindspot data in parallel with session persistence
+    let blindspotSnippet = null;
+    const blindspotPromise = (async () => {
+        if (!hasProInsights())
+            return;
+        try {
+            const endDate = new Date().toISOString();
+            const startDate = new Date(Date.now() - 30 * 86400000).toISOString();
+            const project = isRetroactive ? "default" : existingSession?.project || "default";
+            const rawUsages = await queryScarUsageByDateRange(startDate, endDate, project);
+            const usages = await enrichScarUsageTitles(rawUsages);
+            blindspotSnippet = formatBlindspotSnippet(usages);
+        }
+        catch (error) {
+            console.error("[session_close] Blindspot fetch failed (non-fatal):", error);
+        }
+    })();
     // 6. Persist to Supabase (direct REST API, bypasses ww-mcp)
     try {
         // Upsert session WITHOUT embedding (fast path)
         // Embedding + thread detection run fire-and-forget after
-        await supabase.directUpsert(getTableName("sessions"), sessionData);
+        await Promise.all([
+            supabase.directUpsert(getTableName("sessions"), sessionData),
+            blindspotPromise,
+        ]);
         // Tracked fire-and-forget embedding generation + session update + thread detection
         if (isEmbeddingAvailable()) {
             getEffectTracker().track("embedding", "session_close", async () => {
@@ -1219,7 +1288,7 @@ export async function sessionClose(params) {
             }
             catch { /* already gone */ }
         }
-        const display = formatCloseDisplay(sessionId, closeCompliance, params, learningsCount, true, validation.warnings.length > 0 ? validation.warnings : undefined, transcriptStatus);
+        const display = formatCloseDisplay(sessionId, closeCompliance, params, learningsCount, true, validation.warnings.length > 0 ? validation.warnings : undefined, transcriptStatus, blindspotSnippet);
         return {
             success: true,
             session_id: sessionId,
@@ -1235,7 +1304,7 @@ export async function sessionClose(params) {
         const perfData = buildPerformanceData("session_close", latencyMs, 0);
         // Clear session state even on error (session is done either way)
         clearCurrentSession();
-        const errorDisplay = formatCloseDisplay(sessionId, closeCompliance, params, learningsCount, false, [`Failed to persist session: ${errorMessage}`], transcriptStatus);
+        const errorDisplay = formatCloseDisplay(sessionId, closeCompliance, params, learningsCount, false, [`Failed to persist session: ${errorMessage}`], transcriptStatus, blindspotSnippet);
         return {
             success: false,
             session_id: sessionId,

package/dist/tools/session-start.js

@@ -17,7 +17,7 @@ import { detectAgent } from "../services/agent-detection.js";
 import * as supabase from "../services/supabase-client.js";
 // Scar search removed from start pipeline (loads on-demand via recall)
 import { ensureInitialized } from "../services/startup.js";
-import { hasSupabase, getTableName } from "../services/tier.js";
+import { hasSupabase, hasProInsights, getTableName } from "../services/tier.js";
 import { getStorage } from "../services/storage.js";
 import { Timer, recordMetrics, calculateContextBytes, buildPerformanceData, buildComponentPerformance, } from "../services/metrics.js";
 import { setCurrentSession, getCurrentSession, addSurfacedScars, getSurfacedScars } from "../services/session-state.js";
@@ -29,6 +29,7 @@ import { registerSession, findSessionByHostPid, pruneStale, migrateFromLegacy }
 import * as os from "os";
 import { formatDate } from "../services/timezone.js";
 import { productLine, dimText, boldText } from "../services/display-protocol.js";
+import { querySessionsByDateRange, queryScarUsageByDateRange, enrichScarUsageTitles, computeLightweightSummary, } from "../services/analytics.js";
 /**
  * Closing payload schema — returned in session_start/refresh so agents
  * know the exact field names for closing-payload.json without guessing.
@@ -252,6 +253,32 @@ async function loadRecentDecisions(project, limit = 5) {
         };
     }
 }
+/**
+ * Load lightweight analytics for Pro insights snippet.
+ * Reuses cached analytics queries — ~200ms on cache miss, near-instant on hit.
+ * Returns null on any error (never blocks session start).
+ */
+async function loadLightweightAnalytics(project) {
+    if (!hasProInsights() || !hasSupabase())
+        return null;
+    try {
+        const endDate = new Date().toISOString();
+        const startDate = new Date(Date.now() - 30 * 86400000).toISOString();
+        const [sessions, rawUsages] = await Promise.all([
+            querySessionsByDateRange(startDate, endDate, project),
+            queryScarUsageByDateRange(startDate, endDate, project),
+        ]);
+        // Skip if not enough data for meaningful insights
+        if (sessions.length < 5)
+            return null;
+        const usages = await enrichScarUsageTitles(rawUsages);
+        return computeLightweightSummary(sessions, usages);
+    }
+    catch (error) {
+        console.error("[session_start] Lightweight analytics failed (non-fatal):", error);
+        return null;
+    }
+}
 // loadRecentWins removed — wins available via search/log on-demand
 /**
  * Create a new session record
@@ -666,7 +693,7 @@ function writeSessionFiles(sessionId, agent, project, surfacedScars, threads, re
 function stripThreadPrefix(text) {
     return text.replace(/^t-[a-f0-9]+:\s*/i, "");
 }
-function formatStartDisplay(result, displayInfoMap, isFirstSession) {
+function formatStartDisplay(result, displayInfoMap, isFirstSession, analytics) {
     const visual = [];
     // Line 1: branded product line + session state
     const stateLabel = result.refreshed ? "refreshed" : (result.resumed ? "resumed" : "active");
@@ -676,6 +703,20 @@ function formatStartDisplay(result, displayInfoMap, isFirstSession) {
     if (result.project)
         parts.push(result.project);
     visual.push(dimText(parts.join(" · ")));
+    // Line 3: duration + surfaced scars for resumed/refreshed sessions
+    if (result.resumed || result.refreshed) {
+        const session = getCurrentSession();
+        if (session?.startedAt) {
+            const durationMs = Date.now() - session.startedAt.getTime();
+            const totalMin = Math.floor(durationMs / 60000);
+            const hours = Math.floor(totalMin / 60);
+            const mins = totalMin % 60;
+            const durationStr = hours > 0 ? `${hours}h ${mins}m` : `${mins}m`;
+            const scarCount = session.surfacedScars?.length || 0;
+            const scarSuffix = scarCount > 0 ? ` · ${scarCount} scars loaded from earlier` : "";
+            visual.push(dimText(`Session active for: ${durationStr}${scarSuffix}`));
+        }
+    }
     // Threads section — top 5 by vitality, truncated to 60 chars
     const hasThreads = result.open_threads && result.open_threads.length > 0;
     const hasDecisions = result.recent_decisions && result.recent_decisions.length > 0;
@@ -711,6 +752,16 @@ function formatStartDisplay(result, displayInfoMap, isFirstSession) {
         visual.push("");
         visual.push("No threads or decisions.");
     }
+    // Pro insights snippet — 30-day analytics summary
+    if (analytics) {
+        visual.push("");
+        const appPct = Math.round(analytics.application_rate * 100);
+        visual.push(boldText("Pro Insights (30d)"));
+        visual.push(`  ${analytics.total_sessions} sessions · ${analytics.scars_surfaced} scars surfaced · ${appPct}% applied`);
+        if (analytics.top_blindspot) {
+            visual.push(`  Top blindspot: "${analytics.top_blindspot.title}" (ignored ${analytics.top_blindspot.times} times)`);
+        }
+    }
     // First-session nudge — agent sees this once, internalizes to PMEM
     if (isFirstSession) {
         visual.push(FIRST_SESSION_NUDGE);
@@ -770,12 +821,13 @@ export async function sessionStart(params) {
     if (!hasSupabase()) {
         return sessionStartFree(params, env, agent, project, timer, metricsId, existingSession?.sessionId, existingSession?.startedAt || forceCarryStartedAt, priorSession ? { surfacedScars: forceCarrySurfacedScars, observations: forceCarryObservations, children: forceCarryChildren } : undefined);
     }
-    // 2. Load last session + decisions in parallel (was sequential)
+    // 2. Load last session + decisions + analytics in parallel (was sequential)
     // Scars and wins removed from pipeline — load on-demand via recall/search
     // Rapport loading disabled — recording kept in session_close but not injected
-    const [lastSessionResult, decisionsResult] = await Promise.all([
+    const [lastSessionResult, decisionsResult, analyticsResult] = await Promise.all([
         loadLastSession(agent, project),
         loadRecentDecisions(project, 3),
+        loadLightweightAnalytics(project),
     ]);
     const lastSession = lastSessionResult.session;
     const decisions = decisionsResult.decisions;
@@ -910,7 +962,7 @@ export async function sessionStart(params) {
         displayInfoMap.set(info.thread.id, info);
     }
     const isFirstSession = !isResuming && !slimLastSession;
-    result.display = formatStartDisplay(result, displayInfoMap, isFirstSession);
+    result.display = formatStartDisplay(result, displayInfoMap, isFirstSession, analyticsResult);
     // Write display to per-session dir
     try {
         const sessionFilePath = getSessionPath(sessionId, "session.json");

package/package.json

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