stackmem 1.0.5 → 1.0.7

package/src/index.ts

@@ -12,6 +12,10 @@ import {
   computeMemoryLinks,
   decideSaveAction,
   estimateTokens,
+  indicatesFailedApproach,
+  isAuthRelatedFile,
+  isCodeFile,
+  isQualityMessage,
 } from "./scoring.js";
 import { supabase } from "./storage.js";
 
@@ -422,6 +426,89 @@ server.registerTool(
   }
 );
 
+// --- session_end -----------------------------------------------------------
+
+server.registerTool(
+  "session_end",
+  {
+    title: "Session End",
+    description:
+      "Capture what happened during a coding session and automatically extract memories: unsolved errors, failed approaches, a summary decision, auth-file touches, and an AST re-index.",
+    inputSchema: {
+      project: z.string().describe("Project identifier"),
+      files_touched: z.array(z.string()).describe("Files read or modified during the session"),
+      errors_encountered: z.array(z.string()).describe("Error messages encountered during the session"),
+      approaches_tried: z
+        .array(z.string())
+        .optional()
+        .describe("Things that were attempted during the session"),
+      summary: z.string().optional().describe("One sentence describing what was accomplished"),
+    },
+  },
+  async ({ project, files_touched, errors_encountered, approaches_tried, summary }) => {
+    let memoriesSaved = 0;
+    let errorsLogged = 0;
+
+    for (const problem of errors_encountered) {
+      const { error } = await supabase.from("execution_log").insert({
+        project,
+        problem,
+        solution: null,
+        resolved: false,
+        device_id: getDeviceId(),
+      });
+      if (!error) errorsLogged++;
+    }
+
+    for (const approach of approaches_tried ?? []) {
+      if (!indicatesFailedApproach(approach)) continue;
+      const { error } = await supabase.from("memories").insert({
+        project,
+        type: "rejection",
+        content: approach,
+        device_id: getDeviceId(),
+      });
+      if (!error) memoriesSaved++;
+    }
+
+    if (summary && isQualityMessage(summary)) {
+      const { error } = await supabase.from("memories").insert({
+        project,
+        type: "decision",
+        content: summary.trim(),
+        device_id: getDeviceId(),
+      });
+      if (!error) memoriesSaved++;
+    }
+
+    const authFiles = files_touched.filter(isAuthRelatedFile);
+    if (authFiles.length > 0) {
+      const { error } = await supabase.from("memories").insert({
+        project,
+        type: "discovery",
+        content: `session touched auth files: ${authFiles.join(", ")}`,
+        device_id: getDeviceId(),
+      });
+      if (!error) memoriesSaved++;
+    }
+
+    if (files_touched.some(isCodeFile)) {
+      analyzeCodebase(process.cwd());
+    }
+
+    const message = `Session captured. ${memoriesSaved} memories saved.`;
+
+    return {
+      content: [{ type: "text", text: message }],
+      structuredContent: {
+        memories_saved: memoriesSaved,
+        errors_logged: errorsLogged,
+        message,
+      },
+    };
+  }
+);
+
 // --- transport -------------------------------------------------------------
 
 async function main() {

package/src/scoring.ts

@@ -247,6 +247,57 @@ export function buildSummaryContent(cluster: any[]): string {
   return `[${mostCommonType.toUpperCase()} cluster] ${topic}: ${items}`;
 }
 
+// --- message quality filter ------------------------------------------------
+
+const LOW_QUALITY_PREFIXES = [
+  "wip", "fix", "update", "misc", "temp", "test", "patch", "minor",
+  "tweak", "change", "stuff", "done", "commit", "save", "ok", "m", "x",
+];
+
+const SEMVER_RE = /^\d+\.\d+\.\d+$/;
+
+// Filters out lazy/placeholder messages ("wip", "fix", "m", "...") and bare
+// version bumps ("1.0.4"). Checked against the first word (and the whole
+// message), not a raw prefix match, so a real message like "Migrate auth to
+// JWT" isn't rejected just for starting with the letter "m". The git hook
+// keeps its own inline copy of this same logic since it runs as a standalone
+// script with no access to this module — keep the two in sync by hand if
+// this changes.
+export function isQualityMessage(message: string): boolean {
+  const trimmed = message.trim();
+  if (trimmed.length < 15) return false;
+  if (SEMVER_RE.test(trimmed)) return false;
+  if (!trimmed.includes(" ")) return false;
+
+  const stripped = trimmed.replace(/\s/g, "");
+  if (stripped.length > 0 && new Set(stripped).size === 1) return false;
+
+  const lowerTrimmed = trimmed.toLowerCase();
+  if (LOW_QUALITY_PREFIXES.includes(lowerTrimmed)) return false;
+
+  const firstWord = (lowerTrimmed.match(/^[a-z0-9]+/) || [""])[0];
+  if (LOW_QUALITY_PREFIXES.includes(firstWord)) return false;
+
+  return true;
+}
+
+// --- session-end classification ---------------------------------------------
+
+const FAILURE_PHRASES = ["failed", "didn't work", "reverted", "switched away"];
+
+export function indicatesFailedApproach(text: string): boolean {
+  const lower = text.toLowerCase();
+  return FAILURE_PHRASES.some((phrase) => lower.includes(phrase));
+}
+
+export function isAuthRelatedFile(filePath: string): boolean {
+  return /(auth|login|session|token)/i.test(filePath);
+}
+
+export function isCodeFile(filePath: string): boolean {
+  return /\.(ts|js)$/i.test(filePath);
+}
+
 // --- task relevance --------------------------------------------------------
 
 function splitWords(text: string): string[] {