mcp-agents-memory 0.6.2 → 0.8.0

package/README.md

@@ -102,6 +102,8 @@ Claude Desktop / Claude Code / any MCP-aware client:
 
 `AGENT_PLATFORM` is recorded as the Curator's harness identity on every memory_add call. The Curator's model is captured per-call (defaulting to the Producer's author_model) — set explicitly via the curator_model argument when an orchestrator saves memories on behalf of a different model (e.g. delegating to a subagent). This avoids the staleness that env-static model values would introduce when /model swaps mid-session.
 
+`agent_key` (optional): Agent persona key for multi-persona harnesses (OpenClaw, Hermes, Opencode). Single-persona setups can ignore — `AGENT_KEY` env is the default. Applies to `memory_add`, `memory_save_skill`, and `memory_curator_run`.
+
 ### Cross-machine memory
 
 On a second computer, run `npm i -g mcp-agents-memory` and `mcp-agents-memory setup` pointing to the **same** `DATABASE_URL`. Memory shares automatically — the database is the source of truth and the MCP server is stateless.

package/build/index.js

@@ -116026,22 +116026,21 @@ ${JSON.stringify(toAudit)}`,
 }
 var MODEL_CACHE = /* @__PURE__ */ new Map();
 var PLATFORM_CACHE = /* @__PURE__ */ new Map();
-var DEFAULT_TRUST_WEIGHT = 0.8;
 async function resolvePlatform(name) {
   if (!name) return null;
   const key = name.toLowerCase();
   if (PLATFORM_CACHE.has(key)) return PLATFORM_CACHE.get(key);
-  const sel = await db.query("SELECT id, name, trust_weight FROM platforms WHERE LOWER(name) = $1 LIMIT 1", [key]);
+  const sel = await db.query("SELECT id, name FROM platforms WHERE LOWER(name) = $1 LIMIT 1", [key]);
   if (sel.rows.length > 0) {
-    const r2 = { id: sel.rows[0].id, name: sel.rows[0].name, trust_weight: parseFloat(sel.rows[0].trust_weight) };
+    const r2 = { id: sel.rows[0].id, name: sel.rows[0].name };
     PLATFORM_CACHE.set(key, r2);
     return r2;
   }
   const ins = await db.query(
-    "INSERT INTO platforms (name, trust_weight) VALUES ($1, 1.00) ON CONFLICT (name) DO UPDATE SET name = EXCLUDED.name RETURNING id, name, trust_weight",
+    "INSERT INTO platforms (name) VALUES ($1) ON CONFLICT (name) DO UPDATE SET name = EXCLUDED.name RETURNING id, name",
     [name]
   );
-  const r = { id: ins.rows[0].id, name: ins.rows[0].name, trust_weight: parseFloat(ins.rows[0].trust_weight) };
+  const r = { id: ins.rows[0].id, name: ins.rows[0].name };
   PLATFORM_CACHE.set(key, r);
   console.error(`\u2728 [Librarian] Auto-registered platform "${name}"`);
   return r;
@@ -116052,34 +116051,46 @@ async function resolveModel(name) {
   if (MODEL_CACHE.has(key)) return MODEL_CACHE.get(key);
   try {
     const res = await db.query(
-      `SELECT id, model_name, trust_weight
+      `SELECT id, model_name
        FROM models
        WHERE LOWER(model_name) = $1
           OR LOWER(metadata->>'alias') = $1
        LIMIT 1`,
       [key]
     );
-    if (res.rows.length === 0) {
-      console.error(`\u26A0\uFE0F [Librarian] Unknown model "${name}" \u2014 defaulting trust_weight ${DEFAULT_TRUST_WEIGHT}, author_model_id=NULL`);
+    if (res.rows.length > 0) {
+      const resolved2 = {
+        id: res.rows[0].id,
+        model_name: res.rows[0].model_name
+      };
+      MODEL_CACHE.set(key, resolved2);
+      return resolved2;
+    }
+    const provider = inferProvider(name);
+    if (!provider) {
+      console.error(`\u26A0\uFE0F [Librarian] Unknown model "${name}" \u2014 provider not inferable, author_model_id=NULL`);
       MODEL_CACHE.set(key, null);
       return null;
     }
+    const ins = await db.query(
+      `INSERT INTO models (provider, model_name, metadata)
+       VALUES ($1, $2, '{}'::jsonb)
+       ON CONFLICT (model_name) DO UPDATE SET model_name = EXCLUDED.model_name
+       RETURNING id, model_name`,
+      [provider, name]
+    );
     const resolved = {
-      id: res.rows[0].id,
-      trust_weight: parseFloat(res.rows[0].trust_weight),
-      model_name: res.rows[0].model_name
+      id: ins.rows[0].id,
+      model_name: ins.rows[0].model_name
     };
     MODEL_CACHE.set(key, resolved);
+    console.error(`\u2728 [Librarian] Auto-registered model "${name}" (provider=${provider})`);
     return resolved;
   } catch (err) {
     console.error(`\u274C [Librarian] resolveModel failed for "${name}":`, err);
     return null;
   }
 }
-function computeEffectiveConfidence(confidence, trustWeight) {
-  const raw = confidence / 10 * trustWeight;
-  return Math.round(raw * 100) / 100;
-}
 async function resolveContradiction(newFact, subjectId, existingEmbedding) {
   try {
     const embedding = existingEmbedding || await generateEmbedding(newFact.content);
@@ -116220,8 +116231,6 @@ async function processBatch(text, subjectId, projectId, rawSource, provenance =
       }
       const resolvedModel = await resolveModel(provenance.author_model);
       const resolvedPlatform = await resolvePlatform(provenance.platform);
-      const trustWeight = resolvedModel?.trust_weight ?? DEFAULT_TRUST_WEIGHT;
-      const effectiveConfidence = computeEffectiveConfidence(fact.confidence, trustWeight);
       const auditedStatus = auditResult ? tierToStatus(auditResult.validation_tier) : null;
       const validationStatus = auditedStatus ?? (fact.importance > 7 ? "pending" : "valid");
       const insertRes = await db.query(
@@ -116229,8 +116238,8 @@ async function processBatch(text, subjectId, projectId, rawSource, provenance =
            subject_id, project_subject_id, content, fact_type,
            confidence, importance, tags, embedding, validation_status,
            author_model, platform, session_id,
-           agent_platform, agent_model,
-           author_model_id, platform_id, effective_confidence, source
+           agent_platform, agent_model, agent_curator_id,
+           author_model_id, platform_id, source
          )
          VALUES ($1, $2, $3, $4, $5, $6, $7, $8, $9, $10, $11, $12, $13, $14, $15, $16, $17, $18)
          RETURNING id`,
@@ -116249,9 +116258,9 @@ async function processBatch(text, subjectId, projectId, rawSource, provenance =
           provenance.session_id,
           provenance.agent_platform,
           provenance.agent_model,
+          provenance.agent_curator_id ?? null,
           resolvedModel?.id ?? null,
           resolvedPlatform?.id ?? null,
-          effectiveConfidence,
           provenance.source ?? "librarian"
         ]
       );
@@ -116317,26 +116326,33 @@ var BRANCH_THRESHOLD = 0.7;
 function singletonArray(value) {
   return typeof value === "number" ? [value] : [];
 }
+function mergeProjectIntoApplicableTo(base, projectKey) {
+  if (!projectKey) return base;
+  if (Array.isArray(base.projects)) return base;
+  return { ...base, projects: [projectKey] };
+}
 function getPersistedSkillFields(candidate, audit) {
   if (!audit) {
+    const applicable = mergeProjectIntoApplicableTo({}, candidate.project_key);
     return {
       content: candidate.content,
       sources: JSON.stringify([]),
       validationTier: "unvalidated",
-      applicableTo: JSON.stringify({}),
+      applicableTo: JSON.stringify(applicable),
       auditMetadata: {}
     };
   }
+  const merged = mergeProjectIntoApplicableTo(audit.applicable_to ?? {}, candidate.project_key);
   return {
     content: audit.reconciled_content,
     sources: JSON.stringify(audit.sources),
     validationTier: audit.validation_tier,
-    applicableTo: JSON.stringify(audit.applicable_to ?? {}),
+    applicableTo: JSON.stringify(merged),
     auditMetadata: {
       validation_tier: audit.validation_tier,
       audit_reasoning: audit.audit_reasoning,
       sources: audit.sources,
-      applicable_to: audit.applicable_to ?? {}
+      applicable_to: merged
     }
   };
 }
@@ -116374,11 +116390,14 @@ async function getInjectableSkills(ctx) {
            AND
            ($2::text IS NULL OR NOT (applicable_to ? 'platforms')
              OR applicable_to->'platforms' @> to_jsonb($2::text))
+           AND
+           ($3::text IS NULL OR NOT (applicable_to ? 'projects')
+             OR applicable_to->'projects' @> to_jsonb($3::text))
          )
        )
      ORDER BY use_count DESC, last_used_at DESC NULLS LAST, created_at DESC
-     LIMIT $3`,
-    [ctx.author_model ?? null, ctx.platform ?? null, limit2]
+     LIMIT $4`,
+    [ctx.author_model ?? null, ctx.platform ?? null, ctx.project_key ?? null, limit2]
   );
   return res.rows.map((row) => ({
     id: Number(row.id),
@@ -116398,7 +116417,7 @@ async function recordSkillExposure(skillIds) {
     [skillIds]
   );
 }
-async function updateOrCreateSkill(candidate, audit) {
+async function updateOrCreateSkill(candidate, audit, agentCuratorId) {
   const persisted = getPersistedSkillFields(candidate, audit);
   const embedding = await generateEmbedding(`${candidate.title}
 
@@ -116418,16 +116437,23 @@ ${persisted.content}`);
       await client2.query(
         `UPDATE skills
          SET use_count = use_count + 1,
-             last_used_at = NOW()
+             last_used_at = NOW(),
+             applicable_to = CASE
+               WHEN $2::text IS NULL THEN applicable_to
+               WHEN NOT (applicable_to ? 'projects') THEN applicable_to
+               WHEN applicable_to->'projects' @> to_jsonb($2::text) THEN applicable_to
+               ELSE jsonb_set(applicable_to, '{projects}',
+                              applicable_to->'projects' || to_jsonb($2::text))
+             END
          WHERE id = $1`,
-        [match.id]
+        [match.id, candidate.project_key ?? null]
       );
       await client2.query(
         `INSERT INTO skill_changelog (
            skill_id, change_type, content_diff, source_memory_ids,
-           author_model_id, platform_id, metadata
+           author_model_id, platform_id, metadata, agent_curator_id
          )
-         VALUES ($1, 'append', $2, $3, $4, $5, $6)`,
+         VALUES ($1, 'append', $2, $3, $4, $5, $6, $7)`,
         [
           match.id,
           persisted.content,
@@ -116439,7 +116465,8 @@ ${persisted.content}`);
             matched_skill_id: match.id,
             similarity,
             audit: persisted.auditMetadata
-          })
+          }),
+          agentCuratorId ?? null
         ]
       );
       await client2.query("COMMIT");
@@ -116481,9 +116508,9 @@ ${persisted.content}`);
       await client2.query(
         `INSERT INTO skill_changelog (
            skill_id, change_type, content_diff, source_memory_ids,
-           author_model_id, platform_id, metadata
+           author_model_id, platform_id, metadata, agent_curator_id
          )
-         VALUES ($1, 'branched', $2, $3, $4, $5, $6)`,
+         VALUES ($1, 'branched', $2, $3, $4, $5, $6, $7)`,
         [
           skillId2,
           persisted.content,
@@ -116494,7 +116521,8 @@ ${persisted.content}`);
             branched_from_skill_id: match.id,
             similarity,
             audit: persisted.auditMetadata
-          })
+          }),
+          agentCuratorId ?? null
         ]
       );
       await client2.query("COMMIT");
@@ -116528,9 +116556,9 @@ ${persisted.content}`);
     await client2.query(
       `INSERT INTO skill_changelog (
          skill_id, change_type, content_diff, source_memory_ids,
-         author_model_id, platform_id, metadata
+         author_model_id, platform_id, metadata, agent_curator_id
        )
-       VALUES ($1, 'created', $2, $3, $4, $5, $6)`,
+       VALUES ($1, 'created', $2, $3, $4, $5, $6, $7)`,
       [
         skillId,
         persisted.content,
@@ -116541,7 +116569,8 @@ ${persisted.content}`);
           matched_skill_id: match?.id ?? null,
           similarity,
           audit: persisted.auditMetadata
-        })
+        }),
+        agentCuratorId ?? null
       ]
     );
     await client2.query("COMMIT");
@@ -116902,7 +116931,10 @@ async function runCurator(options = {}) {
         content: parsed.content,
         source_memory_ids: cluster.member_ids,
         author_model: "curator",
-        platform: "system"
+        platform: "system",
+        // v0.8: propagate project scope from cluster context to the skill.
+        // Cross-project clusters (projectKey unset) leave applicable_to.projects unset → match-all.
+        project_key: options.projectKey
       };
       let audit;
       let audited;
@@ -116924,7 +116956,7 @@ async function runCurator(options = {}) {
       }
       let skillResult = null;
       if (!normalized.dryRun) {
-        skillResult = await updateOrCreateSkill(candidate, audited);
+        skillResult = await updateOrCreateSkill(candidate, audited, options.agentCuratorId ?? null);
         result.skills_saved++;
       }
       result.candidates.push({
@@ -117355,7 +117387,8 @@ function registerTools(server) {
       inputSchema: {
         user_key: external_exports.string().optional().default(process.env.MEMORY_DEFAULT_SUBJECT || "default_user").describe("User subject key."),
         author_model: external_exports.string().optional().describe("Caller's author model (for skill filtering)."),
-        platform: external_exports.string().optional().describe("Caller's platform (for skill filtering).")
+        platform: external_exports.string().optional().describe("Caller's platform (for skill filtering)."),
+        project_key: external_exports.string().optional().describe("Active project key. Skills scoped to specific projects only inject when this matches; unscoped skills (applicable_to.projects unset) inject regardless.")
       }
     },
     async (args) => {
@@ -117456,6 +117489,7 @@ function registerTools(server) {
         const skills = await getInjectableSkills({
           author_model: args.author_model,
           platform: args.platform,
+          project_key: args.project_key,
           limit: 5
         });
         if (skills.length > 0) {
@@ -117511,19 +117545,22 @@ function registerTools(server) {
         project_key: external_exports.string().optional().describe("Project key if relevant."),
         author_model: external_exports.string().optional().describe("Model name or alias (e.g., 'sonnet', 'opus', 'gemini')."),
         curator_model: external_exports.string().optional().describe("Curator model override \u2014 the model running memory_add. Defaults to author_model (Producer == Curator solo case)."),
+        agent_key: external_exports.string().optional().describe("Agent persona key (e.g., 'agent_openclaw_reviewer'). Multi-persona harnesses pass this per-call to differentiate personas in calibration data. Defaults to env AGENT_KEY."),
         platform: external_exports.string().optional().describe("Platform (e.g., 'antigravity', 'claude-code')."),
         session_id: external_exports.string().optional().describe("Unique session identifier.")
       }
     },
     async (args) => {
       const agentPlatform = process.env.AGENT_PLATFORM;
+      const agentKeyRaw = args.agent_key ?? process.env.AGENT_KEY ?? null;
+      const agentCuratorId = agentKeyRaw ? await getOrCreateSubject(agentKeyRaw, "agent") : null;
       const subjectId = await getOrCreateSubject(args.subject_key, "person");
       let projectId = null;
       if (args.project_key) {
         projectId = await getOrCreateSubject(args.project_key, "project");
       }
-      const authorModel = args.author_model ?? null;
-      const curatorModel = args.curator_model ?? args.author_model ?? null;
+      const authorModel = args.author_model ?? void 0;
+      const curatorModel = args.curator_model ?? args.author_model ?? void 0;
       const platform = args.platform ?? agentPlatform;
       const result = await processBatch(
         args.text,
@@ -117535,6 +117572,7 @@ function registerTools(server) {
           platform,
           agent_platform: agentPlatform,
           agent_model: curatorModel,
+          agent_curator_id: agentCuratorId,
           session_id: args.session_id
         }
       );
@@ -117568,6 +117606,8 @@ function registerTools(server) {
         source_memory_ids: external_exports.array(external_exports.number()).optional().describe("Memory IDs that produced this skill."),
         author_model: external_exports.string().optional().describe("Model name or alias that authored the skill."),
         platform: external_exports.string().optional().describe("Platform where the skill was authored."),
+        project_key: external_exports.string().optional().describe("Project subject key. If set, skill is scoped to this project (only injects when memory_startup gets matching project_key). Omit for cross-project skills."),
+        agent_key: external_exports.string().optional().describe("Agent persona key. See memory_add for details."),
         audit: external_exports.boolean().optional().default(true).describe("Run Skill Auditor before saving. Default: true.")
       }
     },
@@ -117577,10 +117617,13 @@ function registerTools(server) {
         content: args.content,
         source_memory_ids: args.source_memory_ids,
         author_model: args.author_model,
-        platform: args.platform
+        platform: args.platform,
+        project_key: args.project_key
       };
+      const agentKeyRaw = args.agent_key ?? process.env.AGENT_KEY ?? null;
+      const agentCuratorId = agentKeyRaw ? await getOrCreateSubject(agentKeyRaw, "agent") : null;
       const audited = args.audit !== false ? await auditSkill(candidate) : void 0;
-      const result = await updateOrCreateSkill(candidate, audited);
+      const result = await updateOrCreateSkill(candidate, audited, agentCuratorId);
       const validationTier = audited?.validation_tier ?? "unvalidated";
       const auditReasoning = audited?.audit_reasoning ?? "audit disabled";
       const sourcesCount = audited?.sources.length ?? 0;
@@ -117613,10 +117656,13 @@ function registerTools(server) {
         min_cluster_size: external_exports.number().optional().describe("Minimum memory count per cluster."),
         similarity_threshold: external_exports.number().optional().describe("Minimum cosine similarity for cluster membership."),
         min_importance: external_exports.number().optional().describe("Minimum average importance for accepted clusters."),
-        max_clusters: external_exports.number().optional().describe("Maximum clusters to analyze in one run.")
+        max_clusters: external_exports.number().optional().describe("Maximum clusters to analyze in one run."),
+        agent_key: external_exports.string().optional().describe("Agent persona key for the curator caller. Auto-promotion loop passes none (NULL).")
       }
     },
     async (args) => {
+      const agentKeyRaw = args.agent_key ?? process.env.AGENT_KEY ?? null;
+      const agentCuratorId = agentKeyRaw ? await getOrCreateSubject(agentKeyRaw, "agent") : null;
       const result = await runCurator({
         subjectKey: args.subject_key,
         projectKey: args.project_key,
@@ -117624,7 +117670,8 @@ function registerTools(server) {
         minClusterSize: args.min_cluster_size,
         similarityThreshold: args.similarity_threshold,
         minImportance: args.min_importance,
-        maxClusters: args.max_clusters
+        maxClusters: args.max_clusters,
+        agentCuratorId
       });
       const lines = [];
       lines.push(`SKILL CURATOR ${result.dry_run ? "DRY RUN" : "RUN"}`);
@@ -117726,7 +117773,7 @@ function registerTools(server) {
       const whereClause = conditions.join(" AND ");
       const results = await db.query(
         `SELECT f.id, f.content, f.fact_type, f.confidence, f.importance, f.tags, f.created_at,
-                f.effective_confidence, m.model_name as author_model,
+                m.model_name as author_model,
                 subj.display_name as subject_name,
                 proj.display_name as project_name,
                 CASE WHEN f.embedding IS NOT NULL
@@ -117753,9 +117800,8 @@ function registerTools(server) {
       let formatted = baseContext + "\u{1F9E0} Recalled Facts:\n\n";
       results.rows.forEach((r) => {
         const sim = (r.similarity * 100).toFixed(0);
-        const conf = r.effective_confidence ? `${r.effective_confidence} (eff)` : r.confidence;
         const author = r.author_model ? ` | via ${r.author_model}` : "";
-        formatted += `[#${r.id}] [${r.fact_type}] (imp: ${r.importance}, conf: ${conf}, sim: ${sim}%${author})
+        formatted += `[#${r.id}] [${r.fact_type}] (imp: ${r.importance}, conf: ${r.confidence}, sim: ${sim}%${author})
 `;
         if (r.subject_name) formatted += `Subject: ${r.subject_name}`;
         if (r.project_name) formatted += ` | Project: ${r.project_name}`;