threadnote 1.4.0 → 1.4.2

package/dist/mcp_server.cjs

@@ -35689,6 +35689,9 @@ function commandResultFromExecFileCallback(params) {
 function defaultCommandTimeoutMs() {
   return positiveIntegerFromEnv("THREADNOTE_COMMAND_TIMEOUT_MS") ?? 10 * 60 * 1e3;
 }
+function reindexWaitTimeoutMs() {
+  return positiveIntegerFromEnv("THREADNOTE_REINDEX_TIMEOUT_MS") ?? 12e4;
+}
 function defaultCommandMaxOutputBytes() {
   return positiveIntegerFromEnv("THREADNOTE_COMMAND_MAX_OUTPUT_BYTES") ?? 5 * 1024 * 1024;
 }
@@ -35712,6 +35715,57 @@ async function sleep(ms) {
     setTimeout(resolvePromise, ms);
   });
 }
+function compareVersions(a, b) {
+  const left = parseVersion(a);
+  const right = parseVersion(b);
+  for (let index = 0; index < 3; index += 1) {
+    const difference = left.numbers[index] - right.numbers[index];
+    if (difference !== 0) {
+      return difference;
+    }
+  }
+  const rankDelta = suffixRank(left.suffix) - suffixRank(right.suffix);
+  if (rankDelta !== 0) {
+    return rankDelta;
+  }
+  if (left.suffix === right.suffix) {
+    return 0;
+  }
+  return (left.suffix ?? "").localeCompare(right.suffix ?? "");
+}
+function suffixRank(suffix) {
+  if (suffix === void 0) {
+    return 0;
+  }
+  return /^post/i.test(suffix) ? 1 : -1;
+}
+function parseVersion(version2) {
+  const normalized = version2.trim().replace(/^v/, "").split("+", 1)[0];
+  const core = normalized.match(/^\d+(?:\.\d+){0,2}/)?.[0] ?? "";
+  const rawSuffix = normalized.slice(core.length).replace(/^[-_.]/, "");
+  const suffix = core.length > 0 && rawSuffix.length > 0 ? rawSuffix : void 0;
+  const parts = core.split(".");
+  return {
+    numbers: [
+      safeVersionNumber(Number.parseInt(parts[0] ?? "", 10)),
+      safeVersionNumber(Number.parseInt(parts[1] ?? "", 10)),
+      safeVersionNumber(Number.parseInt(parts[2] ?? "", 10))
+    ],
+    suffix
+  };
+}
+function safeVersionNumber(value) {
+  return typeof value === "number" && Number.isInteger(value) && value >= 0 ? value : 0;
+}
+function formatStaleVersionNotice(runningVersion, diskVersion) {
+  if (runningVersion === void 0 || diskVersion === void 0) {
+    return void 0;
+  }
+  if (compareVersions(diskVersion, runningVersion) <= 0) {
+    return void 0;
+  }
+  return `threadnote ${diskVersion} is installed but this MCP server is still running ${runningVersion}. Reconnect the threadnote MCP server (e.g. /mcp) to load the update.`;
+}
 async function ensureDirectory(path, dryRun) {
   if (dryRun) {
     console.log(`Would create directory: ${path}`);
@@ -36215,6 +36269,20 @@ function shellQuote(value) {
   }
   return `'${value.replaceAll("'", `'"'"'`)}'`;
 }
+function toolRoot() {
+  if ((0, import_node_fs.existsSync)((0, import_node_path.join)(__dirname, "package.json"))) {
+    return __dirname;
+  }
+  return (0, import_node_path.resolve)(__dirname, "..");
+}
+async function currentPackageVersion() {
+  const rawPackage = await (0, import_promises.readFile)((0, import_node_path.join)(toolRoot(), "package.json"), "utf8");
+  const parsed = JSON.parse(rawPackage);
+  if (!isJsonObject(parsed) || typeof parsed.version !== "string") {
+    throw new Error("Could not read current threadnote package version.");
+  }
+  return parsed.version;
+}
 function errorMessage(err) {
   return err instanceof Error ? err.message : String(err);
 }
@@ -36290,7 +36358,7 @@ function readStringArray(object3, key) {
 
 // src/runtime.ts
 function withIdentity(config2, args) {
-  return [...args, "--account", config2.account, "--user", config2.user, "--agent-id", config2.agentId];
+  return [...args, "--account", config2.account, "--user", config2.user];
 }
 
 // src/index_repair.ts
@@ -36334,7 +36402,11 @@ async function repairStaleRecallIndex(config2, ov, options = {}) {
     const result = await runCommand(
       ov,
       withIdentity(config2, ["reindex", target.uri, "--mode", "semantic_and_vectors", "--wait", "true"]),
-      { allowFailure: true }
+      // Bound the wait: `ov reindex` has no --timeout, so a stuck/poisoned
+      // semantic queue would block each target for the full 10-min command
+      // timeout. A timed-out target counts as a failure and trips the
+      // consecutive-failure stop instead of hanging the whole repair.
+      { allowFailure: true, timeoutMs: reindexWaitTimeoutMs() }
     );
     if (result.exitCode === 0) {
       consecutiveFailures = 0;
@@ -38580,7 +38652,7 @@ async function refreshMemoryIndex(config2, ov, uri, options = {}) {
   const result = await runCommand(
     ov,
     withIdentity(config2, ["reindex", uri, "--mode", "vectors_only", "--wait", "true"]),
-    { allowFailure: true }
+    { allowFailure: true, timeoutMs: reindexWaitTimeoutMs() }
   );
   if (result.exitCode === 0) {
     if (options.quiet !== true && result.stdout.trim()) {
@@ -39260,8 +39332,36 @@ function formatPlanSection(title, lines) {
 }
 
 // src/mcp_server.ts
+var mcpStartupVersion;
+var staleNoticeCache;
+var STALE_NOTICE_TTL_MS = 6e4;
+async function staleVersionNotice() {
+  if (mcpStartupVersion === void 0) {
+    return void 0;
+  }
+  const nowMs = Date.now();
+  if (staleNoticeCache && nowMs - staleNoticeCache.checkedAtMs < STALE_NOTICE_TTL_MS) {
+    return staleNoticeCache.notice;
+  }
+  let notice;
+  try {
+    notice = formatStaleVersionNotice(mcpStartupVersion, await currentPackageVersion());
+  } catch {
+    notice = void 0;
+  }
+  staleNoticeCache = { checkedAtMs: nowMs, notice };
+  return notice;
+}
+async function withStaleVersionNotice(result) {
+  const notice = await staleVersionNotice();
+  if (notice === void 0) {
+    return result;
+  }
+  return { ...result, content: [...result.content ?? [], { type: "text", text: `\u26A0 ${notice}` }] };
+}
 async function main() {
   const config2 = getRuntimeConfig();
+  mcpStartupVersion = await currentPackageVersion().catch(() => void 0);
   const server = new McpServer(
     { name: "threadnote-local-adapter", version: "0.2.0" },
     {
@@ -39459,7 +39559,7 @@ function registerTools(server, config2) {
       description: "Check OpenViking server health through the CLI.",
       inputSchema: {}
     },
-    async () => runOpenVikingMcpTool(config2, "health", {})
+    async () => withStaleVersionNotice(await runOpenVikingMcpTool(config2, "health", {}))
   );
   registerOpenVikingParityTools(server, config2);
   server.registerTool(
@@ -39952,14 +40052,16 @@ function registerSearchTool(server, config2, name, description) {
       if (!checkedUri.ok) {
         return checkedUri.error;
       }
-      return runRecallTool(config2, {
-        callerCwd,
-        query: checkedQuery.value,
-        pinnedUri: checkedUri.value,
-        nodeLimit,
-        includeArchived: includeArchived === true,
-        threshold: threshold === void 0 ? void 0 : String(threshold)
-      });
+      return withStaleVersionNotice(
+        await runRecallTool(config2, {
+          callerCwd,
+          query: checkedQuery.value,
+          pinnedUri: checkedUri.value,
+          nodeLimit,
+          includeArchived: includeArchived === true,
+          threshold: threshold === void 0 ? void 0 : String(threshold)
+        })
+      );
     }
   );
 }
@@ -40187,11 +40289,13 @@ function registerStoreTool(server, config2, name, description) {
         timestamp: (/* @__PURE__ */ new Date()).toISOString(),
         topic: normalizeOptionalMetadata2(topic)
       };
-      return writeDurableMemory(config2, {
-        bodyText: checkedText.value,
-        metadata,
-        replaceUri: checkedReplaceUri.value
-      });
+      return withStaleVersionNotice(
+        await writeDurableMemory(config2, {
+          bodyText: checkedText.value,
+          metadata,
+          replaceUri: checkedReplaceUri.value
+        })
+      );
     }
   );
 }
@@ -40814,8 +40918,9 @@ async function runOpenVikingMcpTool(config2, toolName, args) {
     const transport = new StreamableHTTPClientTransport(new URL(config2.openVikingMcpUrl), {
       requestInit: {
         headers: {
+          // OpenViking 0.4.x dropped agent_id as an identity input; only
+          // account + user are honored (mirrors withIdentity in runtime.ts).
           "X-OpenViking-Account": config2.account,
-          "X-OpenViking-Agent": config2.agentId,
           "X-OpenViking-User": config2.user
         }
       }

package/dist/threadnote.cjs

@@ -3835,6 +3835,9 @@ function commandResultFromExecFileCallback(params) {
 function defaultCommandTimeoutMs() {
   return positiveIntegerFromEnv("THREADNOTE_COMMAND_TIMEOUT_MS") ?? 10 * 60 * 1e3;
 }
+function reindexWaitTimeoutMs() {
+  return positiveIntegerFromEnv("THREADNOTE_REINDEX_TIMEOUT_MS") ?? 12e4;
+}
 function defaultCommandMaxOutputBytes() {
   return positiveIntegerFromEnv("THREADNOTE_COMMAND_MAX_OUTPUT_BYTES") ?? 5 * 1024 * 1024;
 }
@@ -4580,6 +4583,14 @@ function toolRoot() {
   }
   return (0, import_node_path2.resolve)(__dirname, "..");
 }
+async function currentPackageVersion() {
+  const rawPackage = await (0, import_promises.readFile)((0, import_node_path2.join)(toolRoot(), "package.json"), "utf8");
+  const parsed = JSON.parse(rawPackage);
+  if (!isJsonObject(parsed) || typeof parsed.version !== "string") {
+    throw new Error("Could not read current threadnote package version.");
+  }
+  return parsed.version;
+}
 function errorMessage(err) {
   return err instanceof Error ? err.message : String(err);
 }
@@ -7548,7 +7559,7 @@ function openVikingServerArgs(config) {
   return ["--config", (0, import_node_path4.join)(config.agentContextHome, "ov.conf"), "--host", config.host, "--port", String(config.port)];
 }
 function withIdentity(config, args) {
-  return [...args, "--account", config.account, "--user", config.user, "--agent-id", config.agentId];
+  return [...args, "--account", config.account, "--user", config.user];
 }
 function renderTemplate(template, config, extras = {}) {
   let rendered = template.replaceAll("{{THREADNOTE_HOME}}", config.agentContextHome).replaceAll("{{OPENVIKING_ACCOUNT}}", config.account).replaceAll("{{OPENVIKING_AGENT_ID}}", config.agentId).replaceAll("{{OPENVIKING_HOST}}", config.host).replaceAll("{{OPENVIKING_PORT}}", String(config.port)).replaceAll("{{OPENVIKING_USER}}", config.user);
@@ -7602,7 +7613,11 @@ async function repairStaleRecallIndex(config, ov, options = {}) {
     const result = await runCommand(
       ov,
       withIdentity(config, ["reindex", target.uri, "--mode", "semantic_and_vectors", "--wait", "true"]),
-      { allowFailure: true }
+      // Bound the wait: `ov reindex` has no --timeout, so a stuck/poisoned
+      // semantic queue would block each target for the full 10-min command
+      // timeout. A timed-out target counts as a failure and trips the
+      // consecutive-failure stop instead of hanging the whole repair.
+      { allowFailure: true, timeoutMs: reindexWaitTimeoutMs() }
     );
     if (result.exitCode === 0) {
       consecutiveFailures = 0;
@@ -10905,7 +10920,7 @@ async function refreshMemoryIndex(config, ov, uri, options = {}) {
   const result = await runCommand(
     ov,
     withIdentity(config, ["reindex", uri, "--mode", "vectors_only", "--wait", "true"]),
-    { allowFailure: true }
+    { allowFailure: true, timeoutMs: reindexWaitTimeoutMs() }
   );
   if (result.exitCode === 0) {
     if (options.quiet !== true && result.stdout.trim()) {
@@ -13423,14 +13438,6 @@ async function getUpdateInfo(config, options) {
     registry: options.registry
   };
 }
-async function currentPackageVersion() {
-  const rawPackage = await (0, import_promises10.readFile)((0, import_node_path13.join)(toolRoot(), "package.json"), "utf8");
-  const parsed = JSON.parse(rawPackage);
-  if (!isJsonObject(parsed) || typeof parsed.version !== "string") {
-    throw new Error("Could not read current threadnote package version.");
-  }
-  return parsed.version;
-}
 async function fetchLatestVersion2(registry) {
   const url = new URL(`${NPM_PACKAGE_NAME}/latest`, normalizeRegistry(registry));
   const controller = new AbortController();
@@ -14256,22 +14263,20 @@ async function recallShapeCheck(config) {
   if (result.exitCode !== 0) {
     return { name: "recall shape", status: "warn", detail: "search failed; run threadnote repair" };
   }
-  let parsed;
+  const start = result.stdout.search(/^\{/m);
+  let envelope;
   try {
-    parsed = JSON.parse(result.stdout.trim());
+    const parsed = start >= 0 ? JSON.parse(result.stdout.slice(start)) : void 0;
+    envelope = isJsonObject(parsed) ? parsed.result : void 0;
   } catch {
-    return {
-      name: "recall shape",
-      status: "warn",
-      detail: "search output is not JSON; recall may silently return nothing"
-    };
+    envelope = void 0;
   }
   const buckets = ["memories", "resources", "skills"];
-  if (!isJsonObject(parsed) || !buckets.some((key) => Array.isArray(parsed[key]))) {
+  if (!isJsonObject(envelope) || !buckets.some((key) => Array.isArray(envelope[key]))) {
     return {
       name: "recall shape",
       status: "warn",
-      detail: `search JSON missing ${buckets.join("/")} buckets; recall parsing is out of sync with this OpenViking`
+      detail: `search JSON missing the result.{${buckets.join(",")}} buckets recall parsing depends on`
     };
   }
   return { name: "recall shape", status: "ok", detail: "memories/resources/skills buckets present" };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "threadnote",
-  "version": "1.4.0",
+  "version": "1.4.2",
   "type": "commonjs",
   "main": "dist/threadnote.cjs",
   "description": "Shared local context and handoffs for development agents",