gitmem-mcp 1.0.3 → 1.0.5

package/dist/tools/graph-traverse.js

@@ -17,29 +17,31 @@ import { Timer, buildPerformanceData, } from "../services/metrics.js";
 import { wrapDisplay } from "../services/display-protocol.js";
 // --- Constants ---
 const SELECT_COLS = "id,subject,predicate,object,event_time,decay_weight,source_type,source_id,source_linear_issue,created_by";
-const KNOWN_AGENTS = ["CLI", "DAC", "CODA-1", "Brain_Local", "Brain_Cloud"];
+const KNOWN_AGENTS = ["cli", "desktop", "autonomous", "local", "cloud", "CLI", "DAC", "CODA-1", "Brain_Local", "Brain_Cloud"];
 /** Persona names are resolved dynamically via triple-writer's normalizePersonaLabel */
 const KNOWN_PERSONAS = [];
 function normalizeNode(input) {
     const trimmed = input.trim();
-    // Already prefixed: "Scar: Title", "Issue: OD-466", etc.
-    if (/^(Scar|Win|Decision|Pattern|Anti-Pattern|Issue|Agent|Persona|Thread):/.test(trimmed)) {
-        return { pattern: `*${trimmed}*`, type: trimmed.split(":")[0] };
+    // Sanitize: strip PostgREST structural chars that could escape ilike/or expressions
+    const safe = trimmed.replace(/[(),\0]/g, "");
+    // Already prefixed: "Scar: Title", "Issue: PROJ-123", etc.
+    if (/^(Scar|Win|Decision|Pattern|Anti-Pattern|Issue|Agent|Persona|Thread):/.test(safe)) {
+        return { pattern: `*${safe}*`, type: safe.split(":")[0] };
     }
-    // OD-XXX issue pattern
-    if (/^OD-\d+$/i.test(trimmed)) {
-        return { pattern: `*${trimmed.toUpperCase()}*`, type: "Issue" };
+    // Issue identifier pattern (e.g., PROJ-123, ENG-456)
+    if (/^[A-Z]+-\d+$/i.test(safe)) {
+        return { pattern: `*${safe.toUpperCase()}*`, type: "Issue" };
     }
     // Known agent names
-    if (KNOWN_AGENTS.includes(trimmed)) {
-        return { pattern: `*Agent: ${trimmed}*`, type: "Agent" };
+    if (KNOWN_AGENTS.includes(safe)) {
+        return { pattern: `*Agent: ${safe}*`, type: "Agent" };
     }
     // Known persona names
-    if (KNOWN_PERSONAS.includes(trimmed)) {
-        return { pattern: `*Persona: ${trimmed}*`, type: "Persona" };
+    if (KNOWN_PERSONAS.includes(safe)) {
+        return { pattern: `*Persona: ${safe}*`, type: "Persona" };
     }
     // Fallback: fuzzy search
-    return { pattern: `*${trimmed}*`, type: "Unknown" };
+    return { pattern: `*${safe}*`, type: "Unknown" };
 }
 /** Detect the type label from a subject/object string */
 function detectNodeType(label) {
@@ -283,13 +285,13 @@ async function stats(params, timer) {
     for (const t of allTriples) {
         objectCounts[t.object] = (objectCounts[t.object] || 0) + 1;
     }
-    // Issues by learning count (objects matching "Issue: OD-*")
+    // Issues by learning count (objects matching "Issue: *")
     const issueCounts = {};
     for (const t of allTriples) {
-        if (t.object.startsWith("Issue: OD-")) {
+        if (t.object.startsWith("Issue: ")) {
             issueCounts[t.object] = (issueCounts[t.object] || 0) + 1;
         }
-        if (t.subject.startsWith("Issue: OD-")) {
+        if (t.subject.startsWith("Issue: ")) {
             issueCounts[t.subject] = (issueCounts[t.subject] || 0) + 1;
         }
     }
@@ -359,7 +361,7 @@ export async function graphTraverse(params) {
     }
     // Validate: node required for all lenses except stats
     if (lens !== "stats" && !params.node) {
-        const missingNodeMsg = `The '${lens}' lens requires a 'node' parameter (e.g., "OD-466", "CLI", "Scar: Done ≠ Deployed")`;
+        const missingNodeMsg = `The '${lens}' lens requires a 'node' parameter (e.g., "PROJ-123", "cli", "Scar: Done ≠ Deployed")`;
         return {
             success: false,
             lens,

package/dist/tools/list-threads.d.ts

@@ -1,10 +1,10 @@
 /**
- * list_threads Tool (OD-thread-lifecycle, OD-622)
+ * list_threads Tool
  *
  * List open threads across recent sessions. Shows unresolved work items
  * that carry over between sessions, with IDs for resolution.
  *
- * OD-622: Primary read from Supabase (source of truth).
+ * Primary read from Supabase (source of truth).
  * Falls back to session-based aggregation (same as session_start),
  * then in-memory session state, then .gitmem/threads.json.
  *

package/dist/tools/list-threads.js

@@ -1,10 +1,10 @@
 /**
- * list_threads Tool (OD-thread-lifecycle, OD-622)
+ * list_threads Tool
  *
  * List open threads across recent sessions. Shows unresolved work items
  * that carry over between sessions, with IDs for resolution.
  *
- * OD-622: Primary read from Supabase (source of truth).
+ * Primary read from Supabase (source of truth).
  * Falls back to session-based aggregation (same as session_start),
  * then in-memory session state, then .gitmem/threads.json.
  *
@@ -14,7 +14,7 @@ import { v4 as uuidv4 } from "uuid";
 import { hasSupabase } from "../services/tier.js";
 import { getProject } from "../services/session-state.js";
 import { aggregateThreads, loadThreadsFile, mergeThreadStates } from "../services/thread-manager.js";
-import { deduplicateThreadList } from "../services/thread-dedup.js"; // OD-641
+import { deduplicateThreadList } from "../services/thread-dedup.js";
 import { listThreadsFromSupabase } from "../services/thread-supabase.js";
 import * as supabase from "../services/supabase-client.js";
 import { Timer, recordMetrics, buildPerformanceData, } from "../services/metrics.js";
@@ -67,7 +67,7 @@ export async function listThreads(params) {
     const project = params.project || getProject() || "default";
     let allThreads = null;
     let source = "file";
-    // OD-622: Try Supabase first (source of truth)
+    // Try Supabase first (source of truth)
     const supabaseThreads = await listThreadsFromSupabase(project, {
         statusFilter: includeResolved ? undefined : statusFilter,
         includeResolved,

package/dist/tools/log.d.ts

@@ -1,5 +1,5 @@
 /**
- * log Tool (OD-561)
+ * log Tool
  *
  * Chronological listing of recent learnings, like `git log`.
  * Pure read-only operation with no side effects.

package/dist/tools/log.js

@@ -1,5 +1,5 @@
 /**
- * log Tool (OD-561)
+ * log Tool
  *
  * Chronological listing of recent learnings, like `git log`.
  * Pure read-only operation with no side effects.

package/dist/tools/prepare-context.d.ts

@@ -1,5 +1,5 @@
 /**
- * prepare_context Tool (GitMem v2 Phase 1 — OD-593)
+ * prepare_context Tool
  *
  * Bridge between global memory and agents that can't call recall directly.
  * The lead agent calls this to generate a portable memory payload that

package/dist/tools/prepare-context.js

@@ -1,5 +1,5 @@
 /**
- * prepare_context Tool (GitMem v2 Phase 1 — OD-593)
+ * prepare_context Tool
  *
  * Bridge between global memory and agents that can't call recall directly.
  * The lead agent calls this to generate a portable memory payload that

package/dist/tools/recall.d.ts

@@ -4,12 +4,12 @@
  * Check institutional memory for relevant scars before taking action.
  * Core GitMem MVP tool that enables "check before act" workflow.
  *
- * Performance target: <2000ms response time (OD-429)
+ * Performance target: <2000ms response time
  *
- * OD-489: Uses local vector search when available for speed,
+ * Uses local vector search when available for speed,
  * falls back to Supabase scar_search when local cache isn't ready.
  *
- * OD-466: After scar search, fetches related knowledge triples and
+ * After scar search, fetches related knowledge triples and
  * includes them in the formatted response for relationship context.
  */
 import type { KnowledgeTriple } from "../services/supabase-client.js";
@@ -26,7 +26,7 @@ export interface RecallParams {
     similarity_threshold?: number;
 }
 /**
- * Required verification block for enforcement gate (OD-487)
+ * Required verification block for enforcement gate
  */
 interface RequiredVerification {
     when: string;

package/dist/tools/recall.js

@@ -4,12 +4,12 @@
  * Check institutional memory for relevant scars before taking action.
  * Core GitMem MVP tool that enables "check before act" workflow.
  *
- * Performance target: <2000ms response time (OD-429)
+ * Performance target: <2000ms response time
  *
- * OD-489: Uses local vector search when available for speed,
+ * Uses local vector search when available for speed,
  * falls back to Supabase scar_search when local cache isn't ready.
  *
- * OD-466: After scar search, fetches related knowledge triples and
+ * After scar search, fetches related knowledge triples and
  * includes them in the formatted response for relationship context.
  */
 import * as supabase from "../services/supabase-client.js";
@@ -19,8 +19,8 @@ import { getProject } from "../services/session-state.js";
 import { getStorage } from "../services/storage.js";
 import { Timer, recordMetrics, buildPerformanceData, buildComponentPerformance, calculateContextBytes, } from "../services/metrics.js";
 import { getOrAssignVariant, formatVariantEnforcement, } from "../services/variant-assignment.js";
-import { addSurfacedScars, getCurrentSession } from "../services/session-state.js"; // OD-552
-import { getAgentIdentity } from "../services/agent-detection.js"; // OD-547
+import { addSurfacedScars, getCurrentSession } from "../services/session-state.js";
+import { getAgentIdentity } from "../services/agent-detection.js";
 import { v4 as uuidv4 } from "uuid";
 import * as fs from "fs";
 import { getSessionPath } from "../services/gitmem-dir.js";
@@ -35,7 +35,7 @@ function formatResponse(scars, plan, dismissals) {
 
 No past lessons match this plan closely enough. Scars accumulate as you work — create learnings during session close to build institutional memory.`;
     }
-    // OD-487: Check if any scars have required_verification (blocking gates)
+    // Check if any scars have required_verification (blocking gates)
     const scarsWithVerification = scars.filter((s) => s.required_verification?.blocking);
     const lines = [
         "🧠 INSTITUTIONAL MEMORY ACTIVATED",
@@ -43,7 +43,7 @@ No past lessons match this plan closely enough. Scars accumulate as you work —
         `Found ${scars.length} relevant scar${scars.length === 1 ? "" : "s"} for your plan:`,
         "",
     ];
-    // OD-487: Display blocking verification requirements FIRST and prominently
+    // Display blocking verification requirements FIRST and prominently
     if (scarsWithVerification.length > 0) {
         lines.push("═══════════════════════════════════════════════════════════════");
         lines.push("🚨 **VERIFICATION REQUIRED BEFORE PROCEEDING**");
@@ -84,7 +84,7 @@ No past lessons match this plan closely enough. Scars accumulate as you work —
                 lines.push(`  _[${counts.dismissed}x dismissed — consider archiving with gm-archive]_`);
             }
         }
-        // OD-525: Use variant enforcement text if available (blind to variant name)
+        // Use variant enforcement text if available (blind to variant name)
         if (scar.variant_info?.has_variants && scar.variant_info.variant) {
             const variantText = formatVariantEnforcement(scar.variant_info.variant, scar.title);
             lines.push(variantText);
@@ -104,7 +104,7 @@ No past lessons match this plan closely enough. Scars accumulate as you work —
             lines.push("");
             lines.push("*Applies when:* " + scar.applies_when.slice(0, 3).join(", "));
         }
-        // OD-530: Render LLM-cooperative enforcement fields (OD-508)
+        // Render LLM-cooperative enforcement fields
         if (scar.why_this_matters) {
             lines.push("");
             lines.push(`**Why this matters:** ${scar.why_this_matters}`);
@@ -123,7 +123,7 @@ No past lessons match this plan closely enough. Scars accumulate as you work —
                 lines.push(`  - [ ] ${criterion}`);
             }
         }
-        // OD-466: Render related knowledge triples
+        // Render related knowledge triples
         if (scar.related_triples && scar.related_triples.length > 0) {
             lines.push("");
             lines.push("*Related knowledge:*");
@@ -170,8 +170,8 @@ export async function recall(params) {
     }
     const project = params.project || getProject() || "default";
     const matchCount = params.match_count || 3;
-    const issueId = params.issue_id; // OD-525: For variant assignment
-    // OD-686: Similarity threshold — suppress weak matches
+    const issueId = params.issue_id; // For variant assignment
+    // Similarity threshold — suppress weak matches
     const defaultThreshold = hasSupabase() ? 0.35 : 0.4;
     const similarityThreshold = params.similarity_threshold ?? defaultThreshold;
     // Free tier: use local keyword search
@@ -191,7 +191,7 @@ export async function recall(params) {
                 applies_when: [],
                 similarity: scar.similarity || 0,
             }))
-                // OD-686: Filter below threshold
+                // Filter below threshold
                 .filter((scar) => scar.similarity >= similarityThreshold);
             const latencyMs = timer.stop();
             const perfData = buildPerformanceData("recall", latencyMs, scars.length, {
@@ -253,7 +253,7 @@ export async function recall(params) {
         let cache_age_ms;
         let search_mode = "remote";
         let network_call = true; // Assume network call until proven otherwise
-        // OD-489: Try local vector search first (fast, no Supabase hit)
+        // Try local vector search first (fast, no Supabase hit)
         // Falls back to Supabase scar_search if local cache isn't ready
         const searchTimer = new Timer();
         if (isLocalSearchReady(project)) {
@@ -275,7 +275,7 @@ export async function recall(params) {
             cache_age_ms = supabaseResult.cache_age_ms;
         }
         const searchLatencyMs = searchTimer.stop();
-        // OD-547: Assign variants for A/B testing (dev tier only)
+        // Assign variants for A/B testing (dev tier only)
         // Agent identity is always available, so variants are always assigned
         const variantTimer = new Timer();
         const variantResults = new Map();
@@ -296,7 +296,7 @@ export async function recall(params) {
             for (const { scarId, variantInfo } of results) {
                 variantResults.set(scarId, variantInfo);
             }
-            // OD-525: Record enforcement metrics for variants (dev only)
+            // Record enforcement metrics for variants (dev only)
             if (hasMetrics()) {
                 const metricsPromises = results
                     .filter(({ variantInfo }) => variantInfo.has_variants && variantInfo.variant)
@@ -322,7 +322,7 @@ export async function recall(params) {
             }
         }
         const variantLatencyMs = variantTimer.stop();
-        // OD-466: Fetch related knowledge triples for surfaced scars
+        // Fetch related knowledge triples for surfaced scars
         const tripleTimer = new Timer();
         const scarIds = rawScars.map((s) => s.id);
         const triplesMap = await supabase.fetchRelatedTriples(scarIds);
@@ -351,19 +351,19 @@ export async function recall(params) {
             source_issue: scar.source_linear_issue,
             similarity: scar.similarity || 0,
             required_verification: scar.required_verification,
-            variant_info: variantResults.get(scar.id), // OD-525
-            // OD-508: LLM-cooperative enforcement fields
+            variant_info: variantResults.get(scar.id),
+            // LLM-cooperative enforcement fields
             why_this_matters: scar.why_this_matters,
             action_protocol: scar.action_protocol,
             self_check_criteria: scar.self_check_criteria,
-            // OD-466: Knowledge triples
+            // Knowledge triples
             related_triples: triplesMap.get(scar.id),
             // Behavioral decay
             decay_multiplier: scar.decay_multiplier,
         }))
-            // OD-686: Filter below threshold
+            // Filter below threshold
             .filter((scar) => scar.similarity >= similarityThreshold);
-        // OD-552: Track surfaced scars for auto-bridge at session close
+        // Track surfaced scars for auto-bridge at session close
         const recallSurfacedAt = new Date().toISOString();
         const recallSurfacedScars = scars.map((scar) => ({
             scar_id: scar.id,
@@ -374,7 +374,7 @@ export async function recall(params) {
             variant_id: variantResults.get(scar.id)?.assignment?.variant_id,
         }));
         addSurfacedScars(recallSurfacedScars);
-        // OD-552: Update per-session dir with accumulated surfaced scars
+        // Update per-session dir with accumulated surfaced scars
         try {
             const session = getCurrentSession();
             if (session) {
@@ -393,7 +393,7 @@ export async function recall(params) {
         const latencyMs = timer.stop();
         const memoriesSurfaced = scars.map((s) => s.id);
         const similarityScores = scars.map((s) => s.similarity);
-        // OD-489: Build detailed performance breakdown for test harness
+        // Build detailed performance breakdown for test harness
         const breakdown = {
             scar_search: buildComponentPerformance(searchLatencyMs, search_mode === "local" ? "local_cache" : "supabase", network_call, network_call ? "miss" : "hit"),
         };
@@ -436,7 +436,7 @@ export async function recall(params) {
                 cache_hit,
                 cache_age_ms,
                 search_mode,
-                // OD-489: Detailed instrumentation
+                // Detailed instrumentation
                 network_calls_made: perfData.network_calls_made,
                 fully_local: perfData.fully_local,
             },

package/dist/tools/record-scar-usage-batch.js

@@ -94,8 +94,8 @@ export async function recordScarUsageBatch(params) {
                 scar_id: scarId,
                 issue_id: entry.issue_id || null,
                 issue_identifier: entry.issue_identifier || null,
-                session_id: entry.session_id || null, // OD-552: Session tracking
-                agent: entry.agent || null, // OD-552: Agent identity
+                session_id: entry.session_id || null, // Session tracking
+                agent: entry.agent || null, // Agent identity
                 surfaced_at: entry.surfaced_at,
                 acknowledged_at: entry.acknowledged_at || null,
                 referenced: entry.reference_type !== "none",

package/dist/tools/record-scar-usage.d.ts

@@ -4,7 +4,7 @@
  * Track scar application for effectiveness measurement.
  * Records to scar_usage table.
  *
- * Performance target: <1000ms (OD-429)
+ * Performance target: <1000ms
  */
 import type { RecordScarUsageParams, RecordScarUsageResult } from "../types/index.js";
 /**

package/dist/tools/record-scar-usage.js

@@ -4,7 +4,7 @@
  * Track scar application for effectiveness measurement.
  * Records to scar_usage table.
  *
- * Performance target: <1000ms (OD-429)
+ * Performance target: <1000ms
  */
 import { v4 as uuidv4 } from "uuid";
 import { wrapDisplay } from "../services/display-protocol.js";
@@ -24,8 +24,8 @@ export async function recordScarUsage(params) {
         scar_id: params.scar_id,
         issue_id: params.issue_id || null,
         issue_identifier: params.issue_identifier || null,
-        session_id: params.session_id || null, // OD-552: Session tracking
-        agent: params.agent || null, // OD-552: Agent identity
+        session_id: params.session_id || null, // Session tracking
+        agent: params.agent || null, // Agent identity
         surfaced_at: params.surfaced_at,
         acknowledged_at: params.acknowledged_at || null,
         referenced: params.reference_type !== "none",

package/dist/tools/resolve-thread.d.ts

@@ -1,11 +1,11 @@
 /**
- * resolve_thread Tool (OD-thread-lifecycle, OD-621)
+ * resolve_thread Tool
  *
  * Mark an open thread as resolved. Supports resolution by:
  * - thread_id: exact match (preferred)
  * - text_match: case-insensitive substring match (fallback)
  *
- * OD-621: Updates Supabase (source of truth) + local file (cache).
+ * Updates Supabase (source of truth) + local file (cache).
  * Falls back to local-only if Supabase is unavailable or thread
  * doesn't exist in Supabase.
  *

package/dist/tools/resolve-thread.js

@@ -1,11 +1,11 @@
 /**
- * resolve_thread Tool (OD-thread-lifecycle, OD-621)
+ * resolve_thread Tool
  *
  * Mark an open thread as resolved. Supports resolution by:
  * - thread_id: exact match (preferred)
  * - text_match: case-insensitive substring match (fallback)
  *
- * OD-621: Updates Supabase (source of truth) + local file (cache).
+ * Updates Supabase (source of truth) + local file (cache).
  * Falls back to local-only if Supabase is unavailable or thread
  * doesn't exist in Supabase.
  *
@@ -82,7 +82,7 @@ export async function resolveThread(params) {
     }
     // Persist to local file (cache)
     saveThreadsFile(threads);
-    // OD-621: Update Supabase (source of truth) — graceful fallback on failure
+    // Update Supabase (source of truth) — graceful fallback on failure
     let supabaseSynced = false;
     const supabaseSuccess = await resolveThreadInSupabase(resolved.id, {
         resolvedAt: resolved.resolved_at,

package/dist/tools/save-transcript.d.ts

@@ -4,7 +4,7 @@
  * Save full session transcript to Supabase storage for training data,
  * post-mortems, and pattern mining.
  *
- * Issue: OD-467
+ *
  */
 import type { Project, PerformanceData } from "../types/index.js";
 export interface SaveTranscriptParams {

package/dist/tools/save-transcript.js

@@ -4,7 +4,7 @@
  * Save full session transcript to Supabase storage for training data,
  * post-mortems, and pattern mining.
  *
- * Issue: OD-467
+ *
  */
 import { detectAgent } from "../services/agent-detection.js";
 import * as supabase from "../services/supabase-client.js";

package/dist/tools/search.d.ts

@@ -1,5 +1,5 @@
 /**
- * search Tool (OD-560)
+ * search Tool
  *
  * Semantic search of institutional memory without action context.
  * Unlike recall (which is "I'm about to do X, warn me"), search is

package/dist/tools/search.js

@@ -1,5 +1,5 @@
 /**
- * search Tool (OD-560)
+ * search Tool
  *
  * Semantic search of institutional memory without action context.
  * Unlike recall (which is "I'm about to do X, warn me"), search is

package/dist/tools/session-close.d.ts

@@ -4,7 +4,7 @@
  * Persist session with compliance validation.
  * Validates that required fields are present based on close type.
  *
- * Performance target: <3000ms (OD-429)
+ * Performance target: <3000ms
  */
 import type { SessionCloseParams, SessionCloseResult } from "../types/index.js";
 /**