@inetafrica/open-claudia 2.6.2 → 2.6.4

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## v2.6.4
+- **Recall matcher: kill false-positive pack/entity injections.** Three compounding fixes after Sharon/Euvencia/higgsfield got recalled on turns that had nothing to do with them:
+  - **Match only the user's own words.** The FTS routers used the full composed prompt — so a Telegram reply-quote fed the entire quoted message (~40 terms) into matching, and any entity named in the quote scored straight past the threshold. The reply-quote block and any 📖 recall-announcement echoes are now stripped before matching (`recallMatchText`); the verbatim prompt still goes to the model untouched.
+  - **Haiku relevance gate (`core/recall-filter.js`).** When keywords produce candidates, a cheap model judges which the message is genuinely about — "zoom" no longer drags in the video-gen pack via word collision. Fail-open: error/timeout/garbage keeps the keyword baseline. Costs one haiku call (~3-6s) only on turns that matched something; `RECALL_FILTER=off` to disable, `RECALL_FILTER_MODEL`/`RECALL_FILTER_TIMEOUT_MS` to tune.
+  - Quoting a 📖 announcement no longer re-injects every entity it names (the self-reinforcing feedback loop).
+
+## v2.6.3
+- Inject local time + off-hours awareness into the runtime state block, so suggestions respect working hours.
+
+## v2.6.2
+- Announce freshly recalled packs/entities in chat (📖 line), mirroring the write-side notes.
+
 ## v2.6.1
 - **Fix: replying to a bot message now carries the quoted context.** Since v2.0.0 the router skipped reply context when the replied-to message came from the bot itself (assuming it was already in conversation context) — which silently dropped the link whenever that message was old or compacted away. Reply context is now always injected, labelled with provenance ("your earlier assistant message" vs "this message").
 

package/core/recall-filter.js

@@ -0,0 +1,61 @@
+// LLM relevance gate over keyword recall candidates. The FTS matchers in
+// packs.js/entities.js are high-recall but dumb — "zoom" pulls in the
+// video-gen pack, "kazee" pulls in every colleague. When keywords produce
+// candidates, a cheap model judges which are genuinely about the message.
+// Fail-open: any error, timeout, or unparseable reply keeps all candidates,
+// so recall never gets worse than the keyword baseline.
+
+const { spawnSubagent } = require("./subagent");
+
+const FILTER_MODEL = process.env.RECALL_FILTER_MODEL || "haiku";
+const FILTER_TIMEOUT_MS = Number(process.env.RECALL_FILTER_TIMEOUT_MS || 20000);
+const MESSAGE_CLIP_CHARS = 1500;
+
+function enabled() {
+  return (process.env.RECALL_FILTER || "on").toLowerCase() !== "off";
+}
+
+const FILTER_SYSTEM_PROMPT = [
+  "You are a relevance judge for an assistant's memory recall.",
+  "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) {
+  if (!enabled() || candidates.length === 0) return null;
+  const msg = String(message || "").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,
+    ">>>",
+    "",
+    "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.",
+    "",
+    "Candidates:",
+    ...lines,
+    "",
+    `Reply with ONLY a JSON array of the relevant ids, e.g. ["${candidates[0].id}"]. Use [] if none are relevant.`,
+  ].join("\n");
+  try {
+    const { text } = await spawnSubagent(prompt, {
+      model: FILTER_MODEL,
+      systemPrompt: FILTER_SYSTEM_PROMPT,
+      timeoutMs: FILTER_TIMEOUT_MS,
+    });
+    const match = String(text || "").match(/\[[\s\S]*?\]/);
+    if (!match) return null;
+    const ids = JSON.parse(match[0]);
+    if (!Array.isArray(ids)) return null;
+    const known = new Set(candidates.map((c) => c.id));
+    return new Set(ids.filter((id) => typeof id === "string" && known.has(id)));
+  } catch (e) {
+    return null;
+  }
+}
+
+module.exports = { judgeRelevance };

package/core/runner.js

@@ -112,7 +112,7 @@ function shouldAutoCompact(state = currentState(), opts = {}) {
   return true;
 }
 
-function buildClaudeArgs(prompt, opts = {}) {
+async function buildClaudeArgs(prompt, opts = {}) {
   const state = currentState();
   const { settings } = state;
   if (settings.backend === "cursor") return buildCursorArgs(prompt, opts);
@@ -139,7 +139,7 @@ function buildClaudeArgs(prompt, opts = {}) {
   if (settings.worktree) args.push("--worktree");
   // Dynamic state rides in the user prompt so the appended system prompt
   // stays byte-stable across turns and the prompt-cache prefix survives.
-  args.push(promptWithDynamicContext(prompt));
+  args.push(await promptWithDynamicContext(prompt));
   return args;
 }
 
@@ -439,6 +439,7 @@ async function runClaudeCapture(prompt, cwd, opts = {}) {
   const authPreflight = preflightClaudeAuthMessage();
   if (authPreflight) throw new Error(authPreflight);
 
+  const args = await buildClaudeArgs(prompt, { ...opts, skipAutoCompact: true });
   return new Promise((resolve, reject) => {
     let assistantText = "";
     let stderrBuffer = "";
@@ -451,7 +452,6 @@ async function runClaudeCapture(prompt, cwd, opts = {}) {
       continueSession: !!opts.continueSession,
       resumeSessionId: opts.resumeSessionId || null,
     }, state, getActiveSessionId);
-    const args = buildClaudeArgs(prompt, { ...opts, skipAutoCompact: true });
     const proc = spawn(getActiveBinary(), args, {
       cwd,
       env: subprocessEnv(),
@@ -646,7 +646,7 @@ async function runClaude(prompt, cwd, replyToMsgId, opts = {}) {
     } catch (e) { /* announcements are best-effort */ }
   };
 
-  const args = buildClaudeArgs(prompt, opts);
+  const args = await buildClaudeArgs(prompt, opts);
   // Recall announcements: mirror the write-side "🧠 jotted down" notes with
   // a read-side line whenever packs/entities freshly enter context this turn.
   try {
@@ -966,10 +966,11 @@ async function runClaude(prompt, cwd, replyToMsgId, opts = {}) {
 }
 
 async function runClaudeSilent(prompt, cwd, label) {
+  const dynamicPrompt = await promptWithDynamicContext(prompt);
   return new Promise((resolve) => {
     const args = ["-p", "--output-format", "text", "--verbose",
       "--append-system-prompt", buildSystemPrompt(),
-      "--dangerously-skip-permissions", promptWithDynamicContext(prompt)];
+      "--dangerously-skip-permissions", dynamicPrompt];
     const proc = spawn(CLAUDE_PATH, args, {
       cwd, env: claudeSubprocessEnv(),
       stdio: ["ignore", "pipe", "pipe"],

package/core/system-prompt.js

@@ -214,6 +214,7 @@ If you tell the user "I'll check back in N minutes" or "I'll run this every morn
 
 ## Guidelines
 - Keep responses concise — many users are on mobile.
+- Be time-aware: the Runtime state block carries the current local time. Before suggesting that the user contact, test with, or wait on a colleague, check it against that person's working hours (entity notes often record their timezone). Late evening / night / early morning → schedule a wakeup for their next working morning instead of suggesting it now.
 - Markdown: use simple chat-safe formatting only. On Telegram follow the Telegram formatting section exactly; on Kazee use short bullets/paragraphs and avoid tables unless requested.
 - For long output (logs, diffs, large code), save to a file and send it as an artifact instead of pasting walls of text.
 - Act on screenshots (fix bugs, implement designs) — don't just describe what you see.
@@ -245,8 +246,13 @@ function buildDynamicContextBlock() {
   const state = currentState();
   const adapter = currentAdapter();
   const channelId = currentChannelId();
+  const now = new Date();
+  const localTime = now.toLocaleString("en-GB", { weekday: "long", hour: "2-digit", minute: "2-digit", hour12: false, timeZoneName: "short" });
+  const hour = now.getHours();
+  const offHours = hour >= 19 || hour < 7;
   const lines = [
     "## Runtime state (current turn)",
+    `- Local time: ${localTime}${offHours ? " — outside normal working hours: don't suggest contacting, testing with, or waiting on colleagues right now; schedule it for their next working morning instead" : ""}`,
     `- Vault: ${vault.isUnlocked() ? "unlocked" : "locked"}`,
     `- Session: ${state.lastSessionId ? "resuming existing conversation" : "new conversation"}`,
   ];
@@ -310,10 +316,9 @@ function formatPackForContext(pack, packsLib) {
   return clip(parts.join("\n\n"), PACK_INJECT_MAX_CHARS);
 }
 
-function buildPackBlock(prompt) {
+function buildPackBlock(matches) {
   try {
     const packsLib = require("./packs");
-    const matches = packsLib.matchPacks(prompt, { limit: 3 });
     if (matches.length === 0) return "";
     const state = currentState();
     const adapter = currentAdapter();
@@ -357,10 +362,9 @@ function formatEntityForContext(ent) {
   return clip(parts.join("\n\n"), ENTITY_INJECT_MAX_CHARS);
 }
 
-function buildEntityBlock(prompt) {
+function buildEntityBlock(matches) {
   try {
     const entitiesLib = require("./entities");
-    const matches = entitiesLib.matchEntities(prompt, { limit: 4 });
     if (matches.length === 0) return "";
     const state = currentState();
     const adapter = currentAdapter();
@@ -387,10 +391,55 @@ function buildEntityBlock(prompt) {
   }
 }
 
-function promptWithDynamicContext(prompt) {
+// The composed prompt can carry text the user didn't write this turn:
+// 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();
+}
+
+async function filterMatches(matchText, packMatches, entityMatches) {
+  if (packMatches.length === 0 && entityMatches.length === 0) {
+    return { packMatches, entityMatches };
+  }
+  try {
+    const packsLib = require("./packs");
+    const entitiesLib = require("./entities");
+    const candidates = [];
+    for (const m of packMatches) {
+      const p = packsLib.readPack(m.dir);
+      candidates.push({ id: `pack:${m.dir}`, name: m.name, description: p?.description || "" });
+    }
+    for (const m of 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
+    return {
+      packMatches: packMatches.filter((m) => kept.has(`pack:${m.dir}`)),
+      entityMatches: entityMatches.filter((m) => kept.has(`entity:${m.slug}`)),
+    };
+  } catch (e) {
+    return { packMatches, entityMatches };
+  }
+}
+
+async function promptWithDynamicContext(prompt) {
   lastInjected = { packs: [], entities: [] };
   try {
-    return `${buildDynamicContextBlock()}${buildPackBlock(prompt)}${buildEntityBlock(prompt)}\n\nCurrent user request:\n${prompt}`;
+    const matchText = recallMatchText(prompt);
+    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));
+    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.2",
+  "version": "2.6.4",
   "description": "Your always-on AI coding assistant — Claude Code, Cursor Agent, and OpenAI Codex via Telegram or Kazee Chat",
   "main": "bot.js",
   "bin": {