@hivelore/mcp 0.38.0 → 0.39.1

package/dist/index.js

@@ -1140,6 +1140,8 @@ import { existsSync as existsSync15 } from "fs";
 import path5 from "path";
 import {
   extractSensorExamples,
+  extractTestFilePathsFromCommand,
+  hasPendingTestMarker,
   judgeProposedSensor,
   loadMemoriesFromDir as loadMemoriesFromDir13,
   serializeMemory as serializeMemory7
@@ -1261,7 +1263,26 @@ async function proposeSensor(input, ctx) {
   if (!found) {
     throw new Error(`No memory found with id ${input.memory_id}`);
   }
+  const personalScopeNudge = found.memory.frontmatter.scope === "personal" ? ` Note: this lesson is personal-scoped, so the sensor guards only YOUR machine (personal memories are gitignored). Promote it so the gate travels with the repo: hivelore memory promote ${input.memory_id}.` : "";
   if (kind !== "regex") {
+    const referencedTests = extractTestFilePathsFromCommand(input.command.trim()).filter((rel) => existsSync15(path5.resolve(ctx.paths.root, rel)));
+    const pendingTests = [];
+    for (const rel of referencedTests) {
+      try {
+        if (hasPendingTestMarker(await readFile3(path5.resolve(ctx.paths.root, rel), "utf8"))) pendingTests.push(rel);
+      } catch {
+      }
+    }
+    if (pendingTests.length > 0 && input.severity === "block") {
+      return {
+        accepted: false,
+        memory_id: input.memory_id,
+        severity: input.severity,
+        reason: "oracle-pending",
+        guidance: `The routed test is still a PENDING stub (${pendingTests.join(", ")}) \u2014 it passes on anything, so the sensor would enforce nothing while reporting protection. Write the assertion (RED on the incident, GREEN once fixed), run it, then re-propose.`,
+        self_check: { silent_on_current: false, fires_on_bad: null, fired_on: pendingTests }
+      };
+    }
     const verdictCmd = runCommandForValidation(input.command.trim(), ctx.paths.root, input.timeout_ms);
     const anchorPathsCmd = input.paths.length > 0 ? input.paths : found.memory.frontmatter.anchor.paths;
     if (verdictCmd.status !== "passed" && input.severity === "block") {
@@ -1295,7 +1316,7 @@ ${verdictCmd.detail}`,
       accepted: true,
       memory_id: input.memory_id,
       severity: input.severity,
-      guidance: verdictCmd.status === "passed" ? "Command oracle passes on the current tree; the gate now runs it when the diff touches the sensor's paths (requires enforcement.runCommandSensors=true)." : `Accepted at warn severity, but note: ${verdictCmd.status} on the current tree (${verdictCmd.detail}).`,
+      guidance: (verdictCmd.status === "passed" ? "Command oracle passes on the current tree; the gate now runs it when the diff touches the sensor's paths (requires enforcement.runCommandSensors=true)." : `Accepted at warn severity, but note: ${verdictCmd.status} on the current tree (${verdictCmd.detail}).`) + (pendingTests.length > 0 ? ` Note: the routed test is still a PENDING stub (${pendingTests.join(", ")}) \u2014 it passes on anything; write the assertion to make this oracle real.` : "") + personalScopeNudge,
       self_check: { silent_on_current: verdictCmd.status === "passed", fires_on_bad: null, fired_on: [] }
     };
   }
@@ -1344,6 +1365,7 @@ ${verdictCmd.detail}`,
     accepted: true,
     memory_id: input.memory_id,
     severity: input.severity,
+    ...personalScopeNudge ? { guidance: personalScopeNudge.trim() } : {},
     self_check,
     file_path: found.filePath
   };
@@ -1354,7 +1376,9 @@ var MemTriedInputSchema = {
   what: z16.string().min(1).describe("Brief description of the approach that was tried"),
   why_failed: z16.string().min(1).describe("Why it failed or why it should NOT be used"),
   instead: z16.string().optional().describe("What to use or do instead (recommended alternative)"),
-  scope: z16.enum(["personal", "team", "module"]).default("personal").describe("Visibility scope"),
+  scope: z16.enum(["personal", "team", "module"]).optional().describe(
+    "Visibility scope. Defaults to personal \u2014 EXCEPT when a one-shot `sensor` is attached: an enforced lesson is team truth (the sensor must travel to every machine and CI), so it defaults to team. Pass scope explicitly to override."
+  ),
   module: z16.string().optional().describe("Module name (required when scope=module)"),
   tags: z16.array(z16.string()).default([]).describe("Tags for filtering"),
   paths: z16.array(z16.string()).default([]).describe("Anchor file paths this applies to"),
@@ -1377,11 +1401,15 @@ async function memTried(input, ctx) {
   if (!existsSync16(ctx.paths.haiveDir)) {
     throw new Error(`No .ai/ directory at ${ctx.paths.root}. Run 'hivelore init' first.`);
   }
-  const slug = input.what.toLowerCase().replace(/[^a-z0-9\s]/g, "").trim().split(/\s+/).slice(0, 5).join("-");
+  const SLUG_STOPWORDS = /* @__PURE__ */ new Set(["and", "or", "the", "a", "an", "of", "to", "in", "for", "with", "on", "at", "by"]);
+  const words = input.what.toLowerCase().replace(/[^a-z0-9\s]/g, "").trim().split(/\s+/).slice(0, 5);
+  while (words.length > 2 && SLUG_STOPWORDS.has(words[words.length - 1])) words.pop();
+  const slug = words.join("-");
+  const scope = input.scope ?? (input.sensor ? "team" : "personal");
   const baseFm = buildFrontmatter2({
     type: "attempt",
     slug,
-    scope: input.scope,
+    scope,
     module: input.module,
     tags: input.tags,
     paths: input.paths,
@@ -1429,7 +1457,7 @@ async function memTried(input, ctx) {
         ...verdict.reason ? { reason: verdict.reason } : {},
         ...verdict.guidance ? { guidance: verdict.guidance } : {}
       },
-      hint: verdict.accepted ? "Loop closed: the attempt is saved AND enforced \u2014 the gate now refuses a repeat deterministically." : `Attempt saved, but the sensor was rejected (${verdict.reason}). Revise per the guidance and re-propose with propose_sensor.`
+      hint: (verdict.accepted ? "Loop closed: the attempt is saved AND enforced \u2014 the gate now refuses a repeat deterministically." : `Attempt saved, but the sensor was rejected (${verdict.reason}). Revise per the guidance and re-propose with propose_sensor.`) + (input.scope === void 0 ? " Saved team-scoped (an enforced lesson must travel with the repo) \u2014 pass scope:'personal' to keep it private." : "")
     };
   }
   const seed = input.paths.length > 0 ? suggestSensorSeed2(body, input.paths) : null;
@@ -1456,6 +1484,7 @@ import { mkdir as mkdir4, readFile as readFile4, writeFile as writeFile10 } from
 import path7 from "path";
 import { z as z17 } from "zod";
 import {
+  buildProposeCommand,
   loadMemoriesFromDir as loadMemoriesFromDir14,
   normalizeFramework,
   parseLessonFields,
@@ -1463,38 +1492,46 @@ import {
   scaffoldPostIncidentTest
 } from "@hivelore/core";
 var PY_SIGNALS = ["pyproject.toml", "setup.py", "pytest.ini", "requirements.txt", "tox.ini"];
-async function detectTestFrameworkForPaths(root, anchorPaths) {
-  const starts = anchorPaths.length > 0 ? anchorPaths : ["."];
-  for (const rel of starts) {
-    let dir = path7.resolve(root, rel);
-    try {
-      if (!statSync(dir).isDirectory()) dir = path7.dirname(dir);
-    } catch {
-      if (path7.extname(dir)) dir = path7.dirname(dir);
-    }
-    while (dir.startsWith(root)) {
-      const pkgJson = path7.join(dir, "package.json");
-      const hasPkg = existsSync17(pkgJson);
-      const goMod = existsSync17(path7.join(dir, "go.mod"));
-      const pySignal = PY_SIGNALS.some((s) => existsSync17(path7.join(dir, s)));
-      if (hasPkg || goMod || pySignal) {
-        let pkg = null;
-        if (hasPkg) {
-          try {
-            pkg = JSON.parse(await readFile4(pkgJson, "utf8"));
-          } catch {
-            pkg = null;
-          }
+async function detectForAnchor(root, rel) {
+  let dir = path7.resolve(root, rel);
+  try {
+    if (!statSync(dir).isDirectory()) dir = path7.dirname(dir);
+  } catch {
+    if (path7.extname(dir)) dir = path7.dirname(dir);
+  }
+  while (dir.startsWith(root)) {
+    const pkgJson = path7.join(dir, "package.json");
+    const hasPkg = existsSync17(pkgJson);
+    const goMod = existsSync17(path7.join(dir, "go.mod"));
+    const pySignal = PY_SIGNALS.some((s) => existsSync17(path7.join(dir, s)));
+    if (hasPkg || goMod || pySignal) {
+      let pkg = null;
+      if (hasPkg) {
+        try {
+          pkg = JSON.parse(await readFile4(pkgJson, "utf8"));
+        } catch {
+          pkg = null;
         }
-        const baseDir = path7.relative(root, dir).split(path7.sep).join("/");
-        return { framework: pickTestFramework(pkg, { goMod, pySignal }), baseDir };
       }
-      const parent = path7.dirname(dir);
-      if (parent === dir || dir === root) break;
-      dir = parent;
+      const baseDir = path7.relative(root, dir).split(path7.sep).join("/");
+      return { framework: pickTestFramework(pkg, { goMod, pySignal }), baseDir };
     }
+    const parent = path7.dirname(dir);
+    if (parent === dir || dir === root) break;
+    dir = parent;
+  }
+  return null;
+}
+async function detectTestFrameworksForAnchors(root, anchorPaths) {
+  const starts = anchorPaths.length > 0 ? anchorPaths : ["."];
+  const groups = /* @__PURE__ */ new Map();
+  for (const rel of starts) {
+    const found = await detectForAnchor(root, rel) ?? { framework: "vitest", baseDir: "" };
+    const existing = groups.get(found.baseDir);
+    if (existing) existing.anchors.push(rel);
+    else groups.set(found.baseDir, { ...found, anchors: [rel] });
   }
-  return { framework: "vitest", baseDir: "" };
+  return [...groups.values()];
 }
 var ScaffoldTestInputSchema = {
   memory_id: z17.string().min(1).describe("Id of the attempt/gotcha lesson to scaffold a post-incident test from."),
@@ -1509,43 +1546,69 @@ async function scaffoldTest(input, ctx) {
     return { ok: false, error: `No memory found with id ${input.memory_id}`, memory_id: input.memory_id };
   }
   const anchorPaths = found.memory.frontmatter.anchor.paths ?? [];
-  const detected = await detectTestFrameworkForPaths(ctx.paths.root, anchorPaths);
-  const framework = input.framework ? normalizeFramework(input.framework) ?? detected.framework : detected.framework;
+  const allGroups = await detectTestFrameworksForAnchors(ctx.paths.root, anchorPaths);
+  const groups = input.out_path ? allGroups.slice(0, 1) : allGroups;
+  const frameworkFor = (detected) => input.framework ? normalizeFramework(input.framework) ?? detected : detected;
   const fields = parseLessonFields(found.memory.body);
-  const scaffold = scaffoldPostIncidentTest(
-    {
-      memoryId: input.memory_id,
-      title: fields.title || input.memory_id,
-      whyFailed: fields.whyFailed,
-      instead: fields.instead,
-      incident: found.memory.frontmatter.sensor?.incident,
-      paths: anchorPaths
-    },
-    { framework, outPath: input.out_path, baseDir: detected.baseDir }
+  const lesson = {
+    memoryId: input.memory_id,
+    title: fields.title || input.memory_id,
+    whyFailed: fields.whyFailed,
+    instead: fields.instead,
+    incident: found.memory.frontmatter.sensor?.incident,
+    paths: anchorPaths
+  };
+  let scaffolds = groups.map(
+    (g) => scaffoldPostIncidentTest(lesson, { framework: frameworkFor(g.framework), outPath: input.out_path, baseDir: g.baseDir })
   );
-  const abs = path7.isAbsolute(scaffold.relPath) ? scaffold.relPath : path7.resolve(ctx.paths.root, scaffold.relPath);
-  let written = false;
-  let alreadyExists = false;
-  if (input.write) {
-    if (existsSync17(abs)) {
-      alreadyExists = true;
-    } else {
-      await mkdir4(path7.dirname(abs), { recursive: true });
-      await writeFile10(abs, scaffold.content, "utf8");
-      written = true;
+  let proposeCommand = scaffolds[0].proposeCommand;
+  if (scaffolds.length > 1) {
+    proposeCommand = buildProposeCommand(lesson, scaffolds.map((s) => s.runCommand).join(" && "));
+    scaffolds = groups.map(
+      (g) => scaffoldPostIncidentTest(lesson, {
+        framework: frameworkFor(g.framework),
+        baseDir: g.baseDir,
+        proposeCommandOverride: proposeCommand
+      })
+    );
+  }
+  const results = [];
+  for (const scaffold of scaffolds) {
+    const abs = path7.isAbsolute(scaffold.relPath) ? scaffold.relPath : path7.resolve(ctx.paths.root, scaffold.relPath);
+    let written = false;
+    let alreadyExists = false;
+    if (input.write) {
+      if (existsSync17(abs)) {
+        alreadyExists = true;
+      } else {
+        await mkdir4(path7.dirname(abs), { recursive: true });
+        await writeFile10(abs, scaffold.content, "utf8");
+        written = true;
+      }
     }
+    results.push({
+      framework: scaffold.framework,
+      path: scaffold.relPath,
+      run_command: scaffold.runCommand,
+      content: scaffold.content,
+      written,
+      already_exists: alreadyExists
+    });
   }
+  const first = results[0];
+  const anyExisting = results.some((r) => r.already_exists);
   return {
     ok: true,
     memory_id: input.memory_id,
-    framework,
-    path: scaffold.relPath,
-    run_command: scaffold.runCommand,
-    propose_command: scaffold.proposeCommand,
-    content: scaffold.content,
-    written,
-    already_exists: alreadyExists,
-    notice: alreadyExists ? "File already exists \u2014 not overwritten. Delete it or pass out_path to write elsewhere." : "PENDING test scaffolded. Fill in the assertion (RED on the incident, GREEN once fixed), run it, then arm it with propose_command \u2014 propose_sensor stays the sole validated writer."
+    framework: first.framework,
+    path: first.path,
+    run_command: first.run_command,
+    propose_command: proposeCommand,
+    content: first.content,
+    written: first.written,
+    already_exists: first.already_exists,
+    ...results.length > 1 ? { scaffolds: results } : {},
+    notice: (results.length > 1 ? `Lesson spans ${results.length} packages \u2014 one pending test per owning package; ONE propose_command arms them all (chained oracle). ` : "") + (anyExisting ? "Some file(s) already exist \u2014 not overwritten. Delete them or pass out_path to write elsewhere." : "PENDING test scaffolded. Fill in the assertion (RED on the incident, GREEN once fixed), run it, then arm it with propose_command \u2014 propose_sensor stays the sole validated writer.")
   };
 }
 
@@ -2812,7 +2875,7 @@ function oneLine(value) {
   return value.replace(/\s+/g, " ").replace(/"/g, '\\"').trim().slice(0, 120);
 }
 function serverVersion() {
-  return true ? "0.38.0" : "dev";
+  return true ? "0.39.1" : "dev";
 }
 
 // src/tools/code-map.ts
@@ -4240,7 +4303,7 @@ When done, respond with: "Imported N memories: [list of IDs]" or "Nothing action
 // src/server.ts
 import { hasRecentBriefingMarker, loadConfigSync } from "@hivelore/core";
 var SERVER_NAME = "hivelore";
-var SERVER_VERSION = "0.38.0";
+var SERVER_VERSION = "0.39.1";
 function jsonResult(data) {
   return {
     content: [