@eleboucher/pi-memini 0.7.1 → 0.7.3

package/dist/index.js

@@ -28,9 +28,72 @@ function validateNamespace(ns) {
   return null;
 }
 
+// ../../../packages/memini-client/src/capture.ts
+var TRUNCATION_MARKER = "\n[...truncated]";
+function truncateForCapture(s, max) {
+  if (typeof max !== "number" || !Number.isFinite(max) || max <= 0) return s;
+  const cap = Math.floor(max);
+  if (s.length <= cap) return s;
+  let i = 0;
+  for (let n = 0; i < s.length && n < cap; n++) {
+    i += s.codePointAt(i) > 65535 ? 2 : 1;
+  }
+  if (i >= s.length) return s;
+  return s.slice(0, i) + TRUNCATION_MARKER;
+}
+function buildTurnCapture(userText, assistantText, userMax, assistantMax) {
+  return `${truncateForCapture(userText, userMax)}
+
+${truncateForCapture(assistantText, assistantMax)}`;
+}
+
 // ../../../packages/memini-client/src/session.ts
 var SESSION_CWD_TTL_MS = 6 * 60 * 60 * 1e3;
 
+// ../../../packages/memini-client/src/bootstrap.ts
+var DEFAULT_TIMEOUT_MS = 3e4;
+var MIN_TIMEOUT_MS = 100;
+var LOOPBACK_HOSTS = /* @__PURE__ */ new Set(["localhost", "127.0.0.1", "::1"]);
+function envTimeoutMs(raw, fallback = DEFAULT_TIMEOUT_MS) {
+  if (raw == null || raw.trim() === "") return fallback;
+  const n = Number(raw.trim());
+  if (!Number.isFinite(n)) return fallback;
+  return Math.max(MIN_TIMEOUT_MS, Math.trunc(n));
+}
+function envEnabled(raw, defaultOn) {
+  if (raw == null || raw === "") return defaultOn;
+  return !/^(0|false|no|off)$/i.test(raw.trim());
+}
+function readBootstrap(env = process.env) {
+  return {
+    baseUrl: env["MEMINI_BASE_URL"] || "http://localhost:8080",
+    apiKey: env["MEMINI_API_KEY"] || "",
+    requireHttps: envEnabled(env["MEMINI_REQUIRE_HTTPS"], false),
+    debug: envEnabled(env["MEMINI_DEBUG"], false),
+    agent: env["MEMINI_AGENT"] || "",
+    namespaceEnv: (env["MEMINI_NAMESPACE"] || "").trim(),
+    namespacePrefixEnv: (env["MEMINI_NAMESPACE_PREFIX"] || "").trim(),
+    homeEnv: (env["MEMINI_HOME"] || "").trim(),
+    timeoutMs: envTimeoutMs(env["MEMINI_TIMEOUT_MS"])
+  };
+}
+function isPlaintextBearerUnsafe(baseUrl, secret) {
+  if (!secret) return false;
+  try {
+    const u = new URL(baseUrl);
+    return u.protocol === "http:" && !LOOPBACK_HOSTS.has(u.hostname.replace(/^\[|\]$/g, "").toLowerCase());
+  } catch {
+    return false;
+  }
+}
+function assertBearerTransportSafe(baseUrl, secret, env = process.env) {
+  if (!isPlaintextBearerUnsafe(baseUrl, secret)) return;
+  if (!envEnabled(env["MEMINI_REQUIRE_HTTPS"], false)) return;
+  throw new Error(
+    `memini: a bearer token is configured for plaintext HTTP to ${baseUrl}. The token and memory payloads can be observed on the network; use HTTPS or an SSH tunnel.`
+  );
+}
+
 // ../../../packages/memini-client/src/settings.ts
 var BEHAVIOR_KNOBS = [
   { envName: "MEMINI_CAPTURE_TURNS", wireKey: "capture_turns", kind: "bool", default: true },
@@ -38,6 +101,7 @@ var BEHAVIOR_KNOBS = [
   { envName: "MEMINI_INLINE_EXTRACT", wireKey: "inline_extract", kind: "bool", default: true },
   { envName: "MEMINI_AUTO_SAVE", wireKey: "auto_save", kind: "bool", default: true },
   { envName: "MEMINI_AUTO_SAVE_INTERVAL", wireKey: "auto_save_interval", kind: "int", default: 10 },
+  { envName: "MEMINI_AUTO_SAVE_MIN_EVENTS", wireKey: "auto_save_min_events", kind: "int", default: 3 },
   { envName: "MEMINI_INJECT_BRIEFING_PINNED", wireKey: "inject_briefing_pinned", kind: "int", default: 5 },
   { envName: "MEMINI_INJECT_BRIEFING_FACTS", wireKey: "inject_briefing_facts", kind: "int", default: 5 },
   { envName: "MEMINI_INJECT_BRIEFING_PROCEDURES", wireKey: "inject_briefing_procedures", kind: "int", default: 5 },
@@ -59,11 +123,30 @@ var BEHAVIOR_KNOBS = [
   { envName: "MEMINI_RECALL_LIMIT", wireKey: "recall_limit", kind: "int", default: 3 },
   { envName: "MEMINI_INJECT_RECALL_MAX_TOK", wireKey: "inject_recall_max_tok", kind: "int", default: 0 },
   { envName: "MEMINI_INJECT_RECALL_MIN_SCORE", wireKey: "inject_recall_min_score", kind: "float", default: 0 },
-  { envName: "MEMINI_MIN_CAPTURE_CHARS", wireKey: "min_capture_chars", kind: "int", default: 0 }
+  { envName: "MEMINI_MIN_CAPTURE_CHARS", wireKey: "min_capture_chars", kind: "int", default: 0 },
+  { envName: "MEMINI_CAPTURE_USER_MAX_CHARS", wireKey: "capture_user_max_chars", kind: "int", default: 1e3 },
+  {
+    envName: "MEMINI_CAPTURE_ASSISTANT_MAX_CHARS",
+    wireKey: "capture_assistant_max_chars",
+    kind: "int",
+    default: 3e3
+  },
+  // Also read at the transport layer (bootstrap.ts's timeoutMs), because a
+  // request timeout has to exist before the handshake that fetches settings.
+  // Both spellings share DEFAULT_TIMEOUT_MS/MIN_TIMEOUT_MS, so there is exactly
+  // one number: raising it here (server-side) or in the env both work.
+  {
+    envName: "MEMINI_TIMEOUT_MS",
+    wireKey: "request_timeout_ms",
+    kind: "int",
+    default: DEFAULT_TIMEOUT_MS,
+    min: MIN_TIMEOUT_MS
+  }
 ];
-function parseIntKnob(raw, fallback) {
+function parseIntKnob(raw, fallback, min = 0) {
   const n = Number.parseInt(raw, 10);
-  return Number.isFinite(n) && n >= 0 ? n : fallback;
+  if (!Number.isFinite(n) || n < 0) return fallback;
+  return Math.max(min, n);
 }
 function parseFloatKnob(raw, fallback) {
   const n = Number.parseFloat(raw);
@@ -81,7 +164,7 @@ function effectiveSetting(knob2, server, env = process.env) {
         value = !/^(0|false|no|off)$/i.test(raw.trim());
         break;
       case "int":
-        value = parseIntKnob(raw, knob2.default);
+        value = parseIntKnob(raw, knob2.default, knob2.min);
         break;
       case "float":
         value = parseFloatKnob(raw, knob2.default);
@@ -98,41 +181,6 @@ function effectiveSetting(knob2, server, env = process.env) {
   return { value: knob2.default, source: "default" };
 }
 
-// ../../../packages/memini-client/src/bootstrap.ts
-var LOOPBACK_HOSTS = /* @__PURE__ */ new Set(["localhost", "127.0.0.1", "::1"]);
-function envEnabled(raw, defaultOn) {
-  if (raw == null || raw === "") return defaultOn;
-  return !/^(0|false|no|off)$/i.test(raw.trim());
-}
-function readBootstrap(env = process.env) {
-  return {
-    baseUrl: env["MEMINI_BASE_URL"] || "http://localhost:8080",
-    apiKey: env["MEMINI_API_KEY"] || "",
-    requireHttps: envEnabled(env["MEMINI_REQUIRE_HTTPS"], false),
-    debug: envEnabled(env["MEMINI_DEBUG"], false),
-    agent: env["MEMINI_AGENT"] || "",
-    namespaceEnv: (env["MEMINI_NAMESPACE"] || "").trim(),
-    namespacePrefixEnv: (env["MEMINI_NAMESPACE_PREFIX"] || "").trim(),
-    homeEnv: (env["MEMINI_HOME"] || "").trim()
-  };
-}
-function isPlaintextBearerUnsafe(baseUrl, secret) {
-  if (!secret) return false;
-  try {
-    const u = new URL(baseUrl);
-    return u.protocol === "http:" && !LOOPBACK_HOSTS.has(u.hostname.replace(/^\[|\]$/g, "").toLowerCase());
-  } catch {
-    return false;
-  }
-}
-function assertBearerTransportSafe(baseUrl, secret, env = process.env) {
-  if (!isPlaintextBearerUnsafe(baseUrl, secret)) return;
-  if (!envEnabled(env["MEMINI_REQUIRE_HTTPS"], false)) return;
-  throw new Error(
-    `memini: a bearer token is configured for plaintext HTTP to ${baseUrl}. The token and memory payloads can be observed on the network; use HTTPS or an SSH tunnel.`
-  );
-}
-
 // ../../../packages/memini-client/src/facts.ts
 import { execFileSync } from "node:child_process";
 import path from "node:path";
@@ -269,7 +317,7 @@ async function performHandshake(boot, facts, opts = {}) {
 }
 
 // src/index.ts
-var DEFAULT_TIMEOUT_MS = 3e4;
+var DEFAULT_TIMEOUT_MS2 = 3e4;
 var DEFAULT_RECALL_LIMIT = 3;
 var STATUS_TIMEOUT_MS = 4e3;
 var HANDSHAKE_TIMEOUT_MS = 2500;
@@ -349,7 +397,7 @@ function resolveStaticConfig(env = process.env) {
   const homeEnv = (e.MEMINI_HOME || "").trim();
   return {
     home: homeEnv || void 0,
-    timeout_ms: Number(e.MEMINI_TIMEOUT_MS || DEFAULT_TIMEOUT_MS),
+    timeout_ms: Number(e.MEMINI_TIMEOUT_MS || DEFAULT_TIMEOUT_MS2),
     fallback_on_error: envBool(e.MEMINI_FALLBACK, true)
   };
 }
@@ -369,7 +417,9 @@ function resolveLiveConfig(boot, facts, hs, env = process.env) {
     capture: effectiveSetting(knob("capture"), server, env).value,
     recall_limit: effectiveSetting(knob("recall_limit"), server, env).value,
     recall_max_tokens: effectiveSetting(knob("inject_recall_max_tok"), server, env).value,
-    recall_min_score: effectiveSetting(knob("inject_recall_min_score"), server, env).value
+    recall_min_score: effectiveSetting(knob("inject_recall_min_score"), server, env).value,
+    capture_user_max_chars: effectiveSetting(knob("capture_user_max_chars"), server, env).value,
+    capture_assistant_max_chars: effectiveSetting(knob("capture_assistant_max_chars"), server, env).value
   };
 }
 async function sessionLive(ctx, env = process.env) {
@@ -537,10 +587,8 @@ function extractLastAssistantText(messages) {
   }
   return "";
 }
-function buildTurnContent(userText, assistantText) {
-  return `${String(userText).slice(0, 1e3)}
-
-${String(assistantText).slice(0, 3e3)}`;
+function buildTurnContent(userText, assistantText, userMax, assistantMax) {
+  return buildTurnCapture(String(userText), String(assistantText), userMax, assistantMax);
 }
 function pinKeyFacts(facts) {
   const out = {};
@@ -1003,7 +1051,7 @@ function meminiExtension(pi) {
     const stored = await client.postJson(
       "/v1/memories",
       {
-        content: buildTurnContent(userText, assistantText),
+        content: buildTurnContent(userText, assistantText, live.capture_user_max_chars, live.capture_assistant_max_chars),
         tags: ["pi"],
         metadata
       },
@@ -1119,7 +1167,8 @@ function meminiExtension(pi) {
   pi.registerTool({
     name: "memory_remember",
     label: "Remember",
-    description: "Store a durable fact, decision, or preference in long-term memory (memini). Call proactively when the user says 'remember this', after an architectural decision (capture the why), or after discovering a non-obvious bug or convention. Keep memories atomic \u2014 one self-contained fact per call. Don't store what's already in project docs or trivially recoverable from code. To correct an existing memory, pass its id \u2014 the write updates it in place. visibility decides who should know: 'project' (default) keeps it here; 'personal' follows the user everywhere; or name an ancestor from the memory_briefing Scope line to share it up that chain. reinforced=true in the result means the fact was ALREADY KNOWN: no new memory was created, the existing one was strengthened, and `id` names that pre-existing memory rather than anything you just wrote \u2014 do not report it to the user as a new save.",
+    // Save-policy invariants are canonical in internal/api/mcp/mcp.go serverInstructions.
+    description: "Store a fact, decision, preference, or event for later recall. Do not wait to be asked \u2014 call this the moment you learn: a decision and why it was made, a bug's root cause, a project convention, a stated user preference, a correction from the user (a correction IS a durable preference), an environment or tool quirk, or a non-obvious command/workflow. When the user says 'remember this', 'note that', 'don't forget', 'going forward...', or corrects you, call this tool FIRST, then acknowledge \u2014 and on an explicit request save unconditionally, even if it seems trivial or already stored; secrets and credentials are the one exception. Keep memories atomic \u2014 one self-contained fact per call; search works better on small records. Do NOT store secrets or credentials, transient session state, task progress, or facts already in project docs/CLAUDE.md or trivially recoverable from code. To correct an existing memory, pass its id \u2014 the write updates it in place. If a stored memory proves wrong or outdated, fix it immediately: re-save the corrected fact with the existing id, or delete it with memory_forget if it should not exist \u2014 never leave a known-incorrect memory in place. visibility decides who should know: 'project' (default) keeps it here; 'personal' follows the user everywhere; or name an ancestor from the memory_briefing Scope line to share it up that chain. reinforced=true in the result means the fact was ALREADY KNOWN: no new memory was created, the existing one was strengthened, and `id` names that pre-existing memory rather than anything you just wrote \u2014 do not report it to the user as a new save.",
     parameters: Type.Object({
       content: Type.String({ description: "The fact to remember \u2014 atomic and self-contained." }),
       id: Type.Optional(

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eleboucher/pi-memini",
-  "version": "0.7.1",
+  "version": "0.7.3",
   "description": "Shared cross-session memory for the Pi coding agent, backed by a memini service.",
   "files": [
     "dist"