@inetafrica/open-claudia 2.6.31 → 2.6.33

package/core/config.js

@@ -90,7 +90,7 @@ const WORKSPACE = config.WORKSPACE;
 const CLAUDE_PATH = resolveExecutablePath(config.CLAUDE_PATH, null, "Claude CLI", { required: true });
 const CURSOR_PATH = config.CURSOR_PATH || null;
 const CODEX_PATH = config.CODEX_PATH || null;
-const DEFAULT_CLAUDE_MODEL = config.CLAUDE_MODEL || process.env.CLAUDE_MODEL || "claude-fable-5";
+const DEFAULT_CLAUDE_MODEL = config.CLAUDE_MODEL || process.env.CLAUDE_MODEL || "claude-opus-4-8";
 const AUTO_COMPACT_TOKENS = parseInt(config.AUTO_COMPACT_TOKENS || process.env.AUTO_COMPACT_TOKENS || "380000", 10);
 const MIN_COMPACT_INTERVAL_MS = parseInt(config.MIN_COMPACT_INTERVAL_MS || process.env.MIN_COMPACT_INTERVAL_MS || "1800000", 10);
 const PROJECT_TRANSCRIPTS = configTruthy(config.PROJECT_TRANSCRIPTS || process.env.PROJECT_TRANSCRIPTS, true);

package/core/pack-review.js

@@ -216,7 +216,7 @@ function reviewTurn({ userText, assistantText, channelId, announce }) {
         } else if (r) {
           lines.push(r.kind === "create"
             ? `📦 New pack: ${r.name}\n${clipWords(r.note, 180)}\n↳ open-claudia pack show ${r.dir}`
-            : `✏️ ${r.name} — ${clipWords(r.note, 180)}`);
+            : `✏️ ${r.name} — ${clipWords(r.note, 180)}\n↳ open-claudia pack show ${r.dir}`);
         }
       } catch (e) {
         console.warn(`[pack-review] apply failed: ${e.message}`);

package/core/runner.js

@@ -842,16 +842,41 @@ async function runClaude(prompt, cwd, replyToMsgId, opts = {}) {
     } catch (e) { /* announcements are best-effort */ }
   };
 
+  // Read-side recall: fire "📖 Recalled my notes on …" only when the agent
+  // actually pulls a full pack/entity via the CLI (it decided it was worth
+  // reading), not when a headline was auto-injected. Deduped per turn through
+  // notifySkill's Set so repeated `pack show` calls don't spam the chat.
+  const noteRecallFromShell = (command) => {
+    try {
+      const cmd = String(command || "");
+      if (!cmd.includes("open-claudia")) return;
+      let m;
+      const packRe = /\bpack\s+show\s+["']?([a-z0-9][\w.-]*)/gi;
+      while ((m = packRe.exec(cmd))) {
+        const dir = m[1];
+        const pack = packsLib.readPack(dir);
+        const name = (pack && (pack.name || pack.dir)) || dir;
+        notifySkill(`recall:pack:${dir}`, `📖 Recalled my notes on: ${name}`);
+      }
+      const entRe = /\bentity\s+show\s+["']?([a-z0-9][\w.-]*)/gi;
+      while ((m = entRe.exec(cmd))) {
+        const slug = m[1];
+        const ent = entitiesLib.readEntity(slug);
+        const name = (ent && (ent.name || ent.slug)) || slug;
+        notifySkill(`recall:entity:${slug}`, `📖 Recalled my notes on: ${name}`);
+      }
+    } catch (e) { /* announcements are best-effort */ }
+  };
+
   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 {
-    const { packs, entities } = require("./system-prompt").consumeLastInjected();
-    const recalled = [...packs, ...entities];
-    if (recalled.length > 0) {
-      chatContext.run(store, () => send(`📖 Recalled my notes on: ${recalled.join(", ")}`).catch(() => {}));
-    }
-  } catch (e) { /* announcements are best-effort */ }
+  // Recall announcements are now fired at READ time, not injection time:
+  // matched packs/entities enter context only as small headlines (see
+  // system-prompt.js recallHeadline). The "📖 Recalled my notes on …" line is
+  // emitted by noteRecallFromShell below when the agent actually runs
+  // `open-claudia pack show <dir>` / `entity show <slug>` — so the banner
+  // reflects what was read, not what was pushed. (consumeLastInjected is
+  // drained here to keep the per-turn buffer from leaking into the next turn.)
+  try { require("./system-prompt").consumeLastInjected(); } catch (e) { /* best-effort */ }
   const binaryPath = getActiveBinary();
   const proc = spawn(binaryPath, args, {
     cwd,
@@ -928,6 +953,7 @@ async function runClaude(prompt, cwd, replyToMsgId, opts = {}) {
             else if (block.name === "Skill" && input.skill) currentToolDetail = input.skill;
             else currentToolDetail = "";
             noteSkillToolUse(block.name, input);
+            if (block.name === "Bash" && input.command) noteRecallFromShell(input.command);
           }
         }
       }
@@ -937,6 +963,7 @@ async function runClaude(prompt, cwd, replyToMsgId, opts = {}) {
           const a = tc.shellToolCall.args || {};
           currentTool = "Shell"; toolUses.push("Shell");
           currentToolDetail = (a.description || a.command || "").slice(0, 80);
+          if (a.command) noteRecallFromShell(a.command);
         } else if (tc.readToolCall) {
           currentTool = "Read"; toolUses.push("Read");
           currentToolDetail = (tc.readToolCall.args?.path || "").split("/").slice(-2).join("/");
@@ -983,6 +1010,7 @@ async function runClaude(prompt, cwd, replyToMsgId, opts = {}) {
         if (it.type === "command_execution" && it.command) {
           currentTool = "Shell"; toolUses.push("Shell");
           currentToolDetail = String(it.command).slice(0, 80);
+          noteRecallFromShell(it.command);
         } else if (it.type === "file_change" && (it.path || it.file_path)) {
           currentTool = "Edit"; toolUses.push("Edit");
           const p = it.path || it.file_path;

package/core/system-prompt.js

@@ -304,6 +304,17 @@ function packMatchLimit() {
   return intSetting("PACK_MATCH_LIMIT", packProgressive() ? 6 : 3);
 }
 
+// Pull model: when on, a matched pack/entity is injected as a *headline*
+// (description + Stance for packs, description for entities) plus a pointer.
+// The heavier sections (State/Procedure/Journal, entity Notes/Log) are NOT
+// pushed — the agent decides which packs are worth a real read and pulls them
+// on demand via `open-claudia pack show <dir>` / `entity show <slug>`. This is
+// what makes the "📖 Recalled my notes on …" banner mean "what I actually
+// read" (fired at read time in the runner) instead of "what got pushed at me".
+function recallHeadline() {
+  return String(config.RECALL_HEADLINE ?? process.env.RECALL_HEADLINE ?? "on").toLowerCase() === "on";
+}
+
 function intSetting(key, fallback) {
   const raw = config[key] ?? process.env[key];
   if (raw === undefined || raw === null || raw === "") return fallback;
@@ -346,6 +357,7 @@ function consumeLastInjected() {
 }
 
 function formatPackForContext(pack, packsLib, opts = {}) {
+  const headline = !!opts.headline;
   const progressive = !!opts.progressive;
   const clip = (s, n) => (s.length > n ? s.slice(0, n) + "\n…[truncated — read the full pack file]" : s);
   const parts = [`### Pack: ${pack.name} (${pack.dir})`];
@@ -362,12 +374,22 @@ function formatPackForContext(pack, packsLib, opts = {}) {
   // Progressive mode injects only the live sections (Stance = how to think,
   // State = where we are) plus a pointer; full mode also inlines Procedure
   // and the recent Journal.
-  const sections = progressive ? ["Stance", "State"] : ["Stance", "Procedure", "State"];
+  // Headline = description + Stance only (Stance carries hard rules/prefs that
+  // must always apply when the topic comes up). Everything else is pulled on
+  // demand. Progressive = Stance + State. Full = Stance + Procedure + State.
+  const sections = headline
+    ? ["Stance"]
+    : progressive
+    ? ["Stance", "State"]
+    : ["Stance", "Procedure", "State"];
   for (const section of sections) {
     const body = (pack.sections[section] || "").trim();
     if (body) parts.push(`#### ${section}\n${body}`);
   }
-  if (progressive) {
+  if (headline) {
+    const hidden = ["State", "Procedure", "Journal"].filter((s) => (pack.sections[s] || "").trim());
+    if (hidden.length) parts.push(`#### More\n${hidden.join(" + ")} not shown — run \`open-claudia pack show ${pack.dir}\` to read them.`);
+  } else if (progressive) {
     const hasMore = (pack.sections.Procedure || "").trim() || (pack.sections.Journal || "").trim();
     if (hasMore) parts.push(`#### More\nProcedure + Journal not shown — run \`open-claudia pack show ${pack.dir}\` to read them.`);
   } else {
@@ -386,6 +408,7 @@ function buildPackBlock(matches, budget) {
     const channelId = currentChannelId();
     const sess = state.lastSessionId || "new";
     const progressive = packProgressive();
+    const headline = recallHeadline();
     const blocks = [];
     const used = [];
     for (const m of matches) {
@@ -400,7 +423,7 @@ function buildPackBlock(matches, budget) {
       // as the task tree), so the pack survives compaction without paying
       // to re-stamp an anchor on every intervening turn.
       if (packsInjectedFor.get(key) === stamp) continue;
-      const block = formatPackForContext(pack, packsLib, { progressive });
+      const block = formatPackForContext(pack, packsLib, { progressive, headline });
       if (!tryUseRecallBudget(budget, block)) continue;
       packsInjectedFor.set(key, stamp);
       lastInjected.packs.push(pack.name || m.dir);
@@ -408,7 +431,10 @@ function buildPackBlock(matches, budget) {
     }
     if (used.length > 0) setImmediate(() => { try { packsLib.touchUsed(used); } catch (e) {} });
     if (blocks.length === 0) return "";
-    return `\n\n## Active Open Claudia skills / context packs\nLong-term topic context auto-matched to this message. If the user asked for a skill by name, treat the matching pack's Procedure section as that Open Claudia skill. Source files live under ${packsLib.PACKS_DIR}/<dir>/PACK.md — read or edit them directly when deeper context or a correction is needed.\n\n${blocks.join("\n\n---\n\n")}`;
+    const intro = headline
+      ? `Long-term topic context auto-matched to this message — these are *headlines* (description + Stance only), not full packs. Decide which are actually relevant and pull the full State/Procedure/Journal on demand with \`open-claudia pack show <dir>\`. If the user asked for a skill by name, \`pack show\` the matching pack to load its Procedure. Source files live under ${packsLib.PACKS_DIR}/<dir>/PACK.md — read or edit them directly when deeper context or a correction is needed.`
+      : `Long-term topic context auto-matched to this message. If the user asked for a skill by name, treat the matching pack's Procedure section as that Open Claudia skill. Source files live under ${packsLib.PACKS_DIR}/<dir>/PACK.md — read or edit them directly when deeper context or a correction is needed.`;
+    return `\n\n## Active Open Claudia skills / context packs\n${intro}\n\n${blocks.join("\n\n---\n\n")}`;
   } catch (e) {
     return "";
   }
@@ -420,14 +446,23 @@ function buildPackBlock(matches, budget) {
 const entitiesInjectedFor = new Map(); // `${adapterId}:${channelId}:${slug}` -> `${sessionId}:${updated}`
 const ENTITY_INJECT_MAX_CHARS = 1200;
 
-function formatEntityForContext(ent) {
+function formatEntityForContext(ent, opts = {}) {
+  const headline = !!opts.headline;
+  const slug = opts.slug || ent.slug;
   const clip = (s, n) => (s.length > n ? s.slice(0, n) + "\n…[truncated — read the full entity file]" : s);
   const head = `### ${ent.name} (${ent.type}${ent.aliases.length ? `, aka ${ent.aliases.join(", ")}` : ""})`;
   const parts = [head];
   if (ent.description) parts.push(ent.description);
-  if (ent.sections.Notes) parts.push(ent.sections.Notes);
-  const log = ent.sections.Log.split("\n").filter(Boolean).slice(-4).join("\n");
-  if (log) parts.push(`Recent:\n${log}`);
+  if (headline) {
+    const hidden = [];
+    if ((ent.sections.Notes || "").trim()) hidden.push("Notes");
+    if ((ent.sections.Log || "").trim()) hidden.push("Log");
+    if (hidden.length) parts.push(`${hidden.join(" + ")} not shown — run \`open-claudia entity show ${slug}\` to read them.`);
+  } else {
+    if (ent.sections.Notes) parts.push(ent.sections.Notes);
+    const log = ent.sections.Log.split("\n").filter(Boolean).slice(-4).join("\n");
+    if (log) parts.push(`Recent:\n${log}`);
+  }
   return clip(parts.join("\n\n"), ENTITY_INJECT_MAX_CHARS);
 }
 
@@ -448,7 +483,7 @@ function buildEntityBlock(matches, budget) {
       const key = `${adapter?.id || "?"}:${channelId || "?"}:${m.slug}`;
       const stamp = `${sess}:${ent.updated}`;
       if (entitiesInjectedFor.get(key) === stamp) continue;
-      const block = formatEntityForContext(ent);
+      const block = formatEntityForContext(ent, { headline: recallHeadline(), slug: m.slug });
       if (!tryUseRecallBudget(budget, block)) continue;
       entitiesInjectedFor.set(key, stamp);
       lastInjected.entities.push(ent.name || m.slug);
@@ -456,7 +491,10 @@ function buildEntityBlock(matches, budget) {
     }
     if (seen.length > 0) setImmediate(() => { try { entitiesLib.touchSeen(seen); } catch (e) {} });
     if (blocks.length === 0) return "";
-    return `\n\n## Known entities\nMemory notes on people/places/projects auto-matched to this message. Source files live under ${entitiesLib.ENTITIES_DIR}/<slug>.md — read or edit them directly to correct or deepen.\n\n${blocks.join("\n\n---\n\n")}`;
+    const intro = recallHeadline()
+      ? `Memory notes on people/places/projects auto-matched to this message — headlines only (description + a pointer). Pull a full note's Notes/Log on demand with \`open-claudia entity show <slug>\`. Source files live under ${entitiesLib.ENTITIES_DIR}/<slug>.md — read or edit them directly to correct or deepen.`
+      : `Memory notes on people/places/projects auto-matched to this message. Source files live under ${entitiesLib.ENTITIES_DIR}/<slug>.md — read or edit them directly to correct or deepen.`;
+    return `\n\n## Known entities\n${intro}\n\n${blocks.join("\n\n---\n\n")}`;
   } catch (e) {
     return "";
   }

package/package.json

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