@inetafrica/open-claudia 2.6.4 → 2.6.5

package/core/recall-filter.js

@@ -20,21 +20,35 @@ const FILTER_SYSTEM_PROMPT = [
   "Reply with ONLY a JSON array of candidate ids — no prose, no markdown fences.",
 ].join("\n");
 
-// candidates: [{ id, name, description }]. Returns a Set of kept ids,
-// or null to signal fail-open (keep everything).
-async function judgeRelevance(message, candidates) {
+// candidates: [{ id, name, description }]. opts.context: surrounding
+// conversation text (e.g. the message the user replied to) used to
+// disambiguate pronouns like "the app" — context only, not a topic
+// source in its own right. Returns a Set of kept ids, or null to
+// signal fail-open (caller decides what that keeps).
+async function judgeRelevance(message, candidates, opts = {}) {
   if (!enabled() || candidates.length === 0) return null;
   const msg = String(message || "").slice(0, MESSAGE_CLIP_CHARS);
+  const context = String(opts.context || "").slice(0, MESSAGE_CLIP_CHARS);
   const lines = candidates.map((c) => `- ${c.id}: ${c.name}${c.description ? ` — ${c.description}` : ""}`);
   const prompt = [
     "A user sent this message to their assistant:",
     "<<<",
     msg,
     ">>>",
+    ...(context
+      ? [
+          "",
+          "It was a reply to this earlier message (context for resolving references like \"the app\" or \"it\" — judge relevance against what the user's message is about in light of it):",
+          "<<<",
+          context,
+          ">>>",
+        ]
+      : []),
     "",
     "These stored memory notes were keyword-matched as possibly relevant.",
     "Keep only the ones the message is GENUINELY about — the topic, project, or person itself.",
     "Sharing an incidental word with the note (e.g. \"zoom\" matching a video tool, a person's employer matching a project name) is NOT relevance.",
+    "If the conversation context shows the user means a different project than a candidate (e.g. they are discussing app A, and a candidate is the unrelated app B that merely shares words like \"mobile\" or \"gradient\"), drop that candidate.",
     "",
     "Candidates:",
     ...lines,

package/core/system-prompt.js

@@ -395,15 +395,32 @@ function buildEntityBlock(matches) {
 // router.js wraps Telegram reply-quotes in "Replying to …:\n---\n…\n---",
 // and quoting a 📖 recall announcement echoes every entity named in it.
 // Matching recall over that text re-injects whatever was quoted (names
-// score 2 — instant threshold), so recall only sees the user's own words.
-function recallMatchText(prompt) {
-  let text = String(prompt || "");
-  text = text.replace(/^Replying to [^\n]*:\n---\n[\s\S]*?\n---\n+/, "");
-  text = text.split("\n").filter((l) => !l.includes("📖")).join("\n");
-  return text.trim();
+// score 2 — instant threshold). So we split: the user's own words drive
+// recall as before, and the quoted text is kept separately as context.
+// Context-derived candidates only survive if the LLM relevance judge
+// confirms them — without a working judge they are dropped, so a quote
+// can never force-inject on keywords alone.
+function stripRecallAnnouncements(text) {
+  return String(text || "").split("\n").filter((l) => !l.includes("📖")).join("\n").trim();
 }
 
-async function filterMatches(matchText, packMatches, entityMatches) {
+function recallMatchParts(prompt) {
+  const text = String(prompt || "");
+  const m = text.match(/^Replying to [^\n]*:\n---\n([\s\S]*?)\n---\n+/);
+  return {
+    userText: stripRecallAnnouncements(m ? text.slice(m[0].length) : text),
+    contextText: stripRecallAnnouncements(m ? m[1] : ""),
+  };
+}
+
+// packMatches/entityMatches carry an `origin` of "user" or "context".
+// Fail-open keeps user-text candidates (the keyword baseline) but drops
+// context-only ones — those are only trustworthy with a working judge.
+async function filterMatches(userText, contextText, packMatches, entityMatches) {
+  const failOpen = () => ({
+    packMatches: packMatches.filter((m) => m.origin === "user"),
+    entityMatches: entityMatches.filter((m) => m.origin === "user"),
+  });
   if (packMatches.length === 0 && entityMatches.length === 0) {
     return { packMatches, entityMatches };
   }
@@ -419,26 +436,52 @@ async function filterMatches(matchText, packMatches, entityMatches) {
       const e = entitiesLib.readEntity(m.slug);
       candidates.push({ id: `entity:${m.slug}`, name: m.name, description: e?.description || "" });
     }
-    const kept = await require("./recall-filter").judgeRelevance(matchText, candidates);
-    if (!kept) return { packMatches, entityMatches }; // fail-open
+    const kept = await require("./recall-filter").judgeRelevance(userText, candidates, { context: contextText });
+    if (!kept) return failOpen();
     return {
       packMatches: packMatches.filter((m) => kept.has(`pack:${m.dir}`)),
       entityMatches: entityMatches.filter((m) => kept.has(`entity:${m.slug}`)),
     };
   } catch (e) {
-    return { packMatches, entityMatches };
+    return failOpen();
   }
 }
 
+function mergeMatches(primary, secondary, keyOf) {
+  const out = primary.map((m) => ({ ...m, origin: "user" }));
+  const seen = new Set(primary.map(keyOf));
+  for (const m of secondary) {
+    if (seen.has(keyOf(m))) continue;
+    out.push({ ...m, origin: "context" });
+  }
+  return out;
+}
+
 async function promptWithDynamicContext(prompt) {
   lastInjected = { packs: [], entities: [] };
   try {
-    const matchText = recallMatchText(prompt);
+    const { userText, contextText } = recallMatchParts(prompt);
+    const packsLib = require("./packs");
+    const entitiesLib = require("./entities");
     let packMatches = [];
     let entityMatches = [];
-    try { packMatches = require("./packs").matchPacks(matchText, { limit: 3 }); } catch (e) {}
-    try { entityMatches = require("./entities").matchEntities(matchText, { limit: 4 }); } catch (e) {}
-    ({ packMatches, entityMatches } = await filterMatches(matchText, packMatches, entityMatches));
+    try {
+      packMatches = mergeMatches(
+        packsLib.matchPacks(userText, { limit: 3 }),
+        contextText ? packsLib.matchPacks(contextText, { limit: 3 }) : [],
+        (m) => m.dir,
+      );
+    } catch (e) {}
+    try {
+      entityMatches = mergeMatches(
+        entitiesLib.matchEntities(userText, { limit: 4 }),
+        contextText ? entitiesLib.matchEntities(contextText, { limit: 4 }) : [],
+        (m) => m.slug,
+      );
+    } catch (e) {}
+    ({ packMatches, entityMatches } = await filterMatches(userText, contextText, packMatches, entityMatches));
+    packMatches = packMatches.slice(0, 3);
+    entityMatches = entityMatches.slice(0, 4);
     return `${buildDynamicContextBlock()}${buildPackBlock(packMatches)}${buildEntityBlock(entityMatches)}\n\nCurrent user request:\n${prompt}`;
   } catch (e) {
     return prompt;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@inetafrica/open-claudia",
-  "version": "2.6.4",
+  "version": "2.6.5",
   "description": "Your always-on AI coding assistant — Claude Code, Cursor Agent, and OpenAI Codex via Telegram or Kazee Chat",
   "main": "bot.js",
   "bin": {