open-agents-ai 0.187.586 → 0.187.588

package/dist/index.js

@@ -514656,7 +514656,8 @@ function evaluate(inputs) {
     return {
       decision: "observer_block",
       reason: "Littleman observer flagged this fingerprint as redundant",
-      cachedResult: cached ? cached.result : null
+      cachedResult: cached ? `[CACHED RESULT — you already have this information from a prior call. Do NOT call this tool again with the same arguments.]
+${cached.result}` : null
     };
   }
   if (isReadLike) {
@@ -514672,10 +514673,12 @@ function evaluate(inputs) {
           blockMessage: buildForceProgressBlockMessage(proposedCall, hits)
         };
       }
+      const cachedEnvelope = `[CACHED RESULT — you already have this information from a prior call. Do NOT call this tool again with the same arguments.]
+${cached.result}`;
       return {
         decision: "serve_cached",
         reason: cached.compacted ? "post-compaction cache re-serve" : `duplicate call #${hits} (still under ${threshold}-hit gate)`,
-        cachedResult: cached.result,
+        cachedResult: cachedEnvelope,
         compacted: cached.compacted,
         hitNumber: hits
       };
@@ -529241,14 +529244,15 @@ ${body}`;
       proactivePrune(messages2, currentTurn) {
         if (process.env["OA_DISABLE_PROACTIVE_PRUNE"] === "1")
           return;
-        const AGED_FILE_READ_TURNS = 10;
-        const AGED_SHELL_TURNS = 5;
+        const AGED_FILE_READ_TURNS = 20;
+        const AGED_SHELL_TURNS = 12;
         const ERROR_MARKERS = /(?:error|fail|exception|traceback|enoent|enotfound|exit code [^0]|status[: ]+1\d?\d?)/i;
         const PRUNE_PREFIX = "[Tool result cleared";
         const DEDUPE_PREFIX = "[deduped — same call as turn";
         const FILE_AGED_PREFIX = "[file_read aged out, summary:";
         const SHELL_AGED_PREFIX = "[shell succeeded, output pruned —";
         const seen = /* @__PURE__ */ new Map();
+        const seenResource = /* @__PURE__ */ new Map();
         const pending = [];
         let scanTurn = 0;
         for (let i2 = 0; i2 < messages2.length; i2++) {
@@ -529293,7 +529297,26 @@ ${body}`;
               }
               seen.set(fp, { turn: scanTurn, idx: resultIdx });
               const ageTurns = currentTurn - scanTurn;
-              if (name10 === "file_read" && ageTurns > AGED_FILE_READ_TURNS && content.length > 200) {
+              const wasRecentlyWritten = (() => {
+                try {
+                  const o2 = JSON.parse(tc.function.arguments || "{}");
+                  const p2 = String(o2.path ?? o2.file ?? "");
+                  for (let k = Math.max(0, i2 - 20); k < i2; k++) {
+                    const m2 = messages2[k];
+                    if (m2.role === "assistant" && m2.tool_calls) {
+                      for (const tc2 of m2.tool_calls) {
+                        if ((tc2.function.name === "file_edit" || tc2.function.name === "file_write" || tc2.function.name === "file_patch") && tc2.function.arguments?.includes(p2)) {
+                          return true;
+                        }
+                      }
+                    }
+                  }
+                  return false;
+                } catch {
+                  return false;
+                }
+              })();
+              if (name10 === "file_read" && ageTurns > AGED_FILE_READ_TURNS && content.length > 200 && !wasRecentlyWritten) {
                 const pathArg = (() => {
                   try {
                     const o2 = JSON.parse(tc.function.arguments || "{}");
@@ -529302,11 +529325,15 @@ ${body}`;
                     return "?";
                   }
                 })();
-                const firstLine = content.split("\n")[0]?.slice(0, 80) ?? "";
+                const lines = content.split("\n");
+                const lineCount = lines.length;
+                const contentPreview = content.slice(0, 500);
                 pending.push({
                   idx: resultIdx,
                   reason: "aged_file",
-                  replacement: `${FILE_AGED_PREFIX} path=${pathArg}, size=${content.length} chars, first-line="${firstLine}"]`
+                  replacement: `${FILE_AGED_PREFIX} path=${pathArg}, ${lineCount} lines, ${content.length} chars]
+${contentPreview}
+[End of aged file_read preview — do NOT re-read this file, use the content above]`
                 });
                 continue;
               }
@@ -530113,6 +530140,61 @@ ${latest.output || ""}`.trim();
         sections.push(`(turn ${turn} — this block is regenerated every turn from your tool history)`);
         return sections.join("\n");
       }
+      /**
+       * REG-62: Pre-call knowledge injection. Builds a concise system message
+       * summarizing what the model has already read/searched/listed so it
+       * doesn't re-call those tools. This is the PRIMARY fix for duplicate
+       * tool calls — the model re-reads because it can't find prior results
+       * in the long transcript. By injecting a summary right before the LLM
+       * call, the model knows what it already has.
+       */
+      _renderKnowledgeBlock(recentToolResults) {
+        if (recentToolResults.size === 0)
+          return null;
+        const filesRead = [];
+        const dirsListed = [];
+        const searches = [];
+        const shells = [];
+        for (const [fingerprint, entry] of recentToolResults) {
+          if (entry.compacted)
+            continue;
+          const colonIdx = fingerprint.indexOf(":");
+          const toolName = colonIdx > 0 ? fingerprint.slice(0, colonIdx) : fingerprint;
+          if (toolName === "file_read") {
+            const pathMatch = fingerprint.match(/path=([^,\s]+)/);
+            if (pathMatch?.[1])
+              filesRead.push(pathMatch[1]);
+          } else if (toolName === "list_directory") {
+            const pathMatch = fingerprint.match(/path=([^,\s]+)/);
+            if (pathMatch?.[1])
+              dirsListed.push(pathMatch[1]);
+          } else if (toolName === "grep_search" || toolName === "find_files") {
+            searches.push(toolName);
+          } else if (toolName === "shell" || toolName === "shell_async") {
+            const cmdMatch = fingerprint.match(/cmd=([^,\s]+)/);
+            shells.push(cmdMatch?.[1] ?? toolName);
+          }
+        }
+        const sections = ["[KNOWLEDGE — you already have these results in context above. Do NOT re-call these tools for the same targets:]"];
+        if (filesRead.length > 0) {
+          const unique = [...new Set(filesRead)].slice(0, 30);
+          sections.push(`Files already read (${unique.length}): ${unique.join(", ")}`);
+        }
+        if (dirsListed.length > 0) {
+          const unique = [...new Set(dirsListed)].slice(0, 15);
+          sections.push(`Directories already listed (${unique.length}): ${unique.join(", ")}`);
+        }
+        if (searches.length > 0) {
+          sections.push(`Searches already run: ${searches.length}`);
+        }
+        if (shells.length > 0) {
+          const unique = [...new Set(shells)].slice(0, 15);
+          sections.push(`Shell commands already executed (${unique.length}): ${unique.join(", ")}`);
+        }
+        if (sections.length <= 1)
+          return null;
+        return sections.join("\n");
+      }
       makePhaseSummarizer() {
         if (process.env["OA_DISABLE_PHASE_SUMMARIZER"] === "1")
           return null;
@@ -530341,6 +530423,46 @@ ${blob}
       _buildToolFingerprint(name10, args) {
         return `${name10}:${this._buildExactArgsKey(args)}`;
       }
+      /**
+       * REG-62: Build a resource-level key for semantic dedup.
+       *
+       * Unlike _buildToolFingerprint (exact args match), this normalizes
+       * different calls that target the SAME resource:
+       *   file_read("foo.ts") == file_read("foo.ts", offset=10)
+       *   file_read("foo.ts") == shell("cat foo.ts")
+       *   grep_search("x", path="src") == grep_search("x", path="src", include="*.ts")
+       *
+       * Returns empty string if no resource key can be extracted (non-dedupable).
+       */
+      _buildResourceKey(name10, args) {
+        const a2 = args ?? {};
+        if (name10 === "file_read") {
+          const p2 = String(a2.path ?? a2.file ?? "");
+          return p2 ? `resource:file:${p2}` : "";
+        }
+        if (name10 === "shell" || name10 === "shell_async") {
+          const cmd = String(a2.command ?? a2.cmd ?? "");
+          const catMatch = cmd.match(/(?:cat|head|tail|type|less|more)\s+["']?([^\s"';&|>]+)/);
+          if (catMatch?.[1])
+            return `resource:file:${catMatch[1]}`;
+          return "";
+        }
+        if (name10 === "grep_search") {
+          const p2 = String(a2.path ?? ".");
+          const pat = String(a2.pattern ?? "");
+          return `resource:grep:${p2}:${pat}`;
+        }
+        if (name10 === "find_files") {
+          const p2 = String(a2.path ?? ".");
+          const pat = String(a2.pattern ?? "");
+          return `resource:find:${p2}:${pat}`;
+        }
+        if (name10 === "list_directory") {
+          const p2 = String(a2.path ?? ".");
+          return `resource:ls:${p2}`;
+        }
+        return "";
+      }
       _formatExactArgValue(value2) {
         const canonical = this._canonicalArgValue(value2);
         const oneLine = canonical.replace(/\r/g, "\\r").replace(/\n/g, "\\n");
@@ -532566,6 +532688,9 @@ ${memoryLines.join("\n")}`
           const progressBlock = this._renderProgressNudgeBlock(turn);
           if (progressBlock)
             _injections.push(progressBlock);
+          const knowledgeBlock = this._renderKnowledgeBlock(recentToolResults);
+          if (knowledgeBlock)
+            _injections.push(knowledgeBlock);
           if (_injections.length > 0) {
             const reqMsgs = chatRequest.messages;
             if (Array.isArray(reqMsgs)) {
@@ -579023,12 +579148,9 @@ sleep 1
       if (ctx3.isBlessed?.()) {
         ctx3.blessStop?.();
       }
-      if (ctx3.isTelegramActive?.()) {
-        ctx3.telegramStop?.();
-      }
       const clearedQueue = ctx3.clearQueuedPrompts?.() ?? 0;
       if (!ctx3.hasActiveTask?.()) {
-        if (!ctx3.isBlessed?.() && !ctx3.isTelegramActive?.()) {
+        if (!ctx3.isBlessed?.()) {
           renderWarning(clearedQueue > 0 ? `No active task to stop. Cleared ${clearedQueue} queued prompt(s).` : "No active task to stop.");
         }
         return "handled";
@@ -591030,6 +591152,60 @@ function renderTelegramLiveProgressHTML(progressLines, accumulated) {
   }
   return progressLines.slice(-6).map((line) => line.trim()).filter(Boolean).join("\n");
 }
+function telegramSyntheticHelpSignatures() {
+  return [
+    { signature: "/help", description: "Show Telegram command help" },
+    { signature: "/start", description: "Show Telegram bridge status and authentication instructions" },
+    { signature: "/auth <code>", description: "Authenticate this Telegram user as bot admin using the TUI code" }
+  ];
+}
+function telegramHelpCommandAllowed(cmd, scope) {
+  if (scope === "admin") return cmd.implementationStatus === "implemented";
+  if (TELEGRAM_PUBLIC_HELP_COMMANDS.has(cmd.name)) return true;
+  return cmd.surfaces.agentTool && !cmd.safety.secretBearing && !cmd.safety.destructive && !cmd.safety.profileGated;
+}
+function buildTelegramHelpHTML(scope, maxPublicCommands = 24) {
+  const commands = listCommandRegistry({ includePlanned: false }).filter((cmd) => telegramHelpCommandAllowed(cmd, scope));
+  const signatures = [
+    ...telegramSyntheticHelpSignatures(),
+    ...commands.flatMap((cmd) => cmd.signatures)
+  ];
+  const seen = /* @__PURE__ */ new Set();
+  const unique = signatures.filter((sig) => {
+    if (seen.has(sig.signature)) return false;
+    seen.add(sig.signature);
+    return true;
+  });
+  const visible = scope === "public" ? unique.slice(0, maxPublicCommands) : unique;
+  const lines = [
+    `<b>Commands (${scope === "admin" ? "admin full scope" : "public secure scope"})</b>`,
+    "",
+    ...visible.map(
+      (sig) => `<code>${escapeTelegramHTML(sig.signature)}</code> - ${escapeTelegramHTML(sig.description)}`
+    )
+  ];
+  if (scope === "public" && unique.length > visible.length) {
+    lines.push("");
+    lines.push(`Public scope truncated to ${visible.length} safe commands. Authenticate as admin for full command help.`);
+  }
+  return lines.join("\n");
+}
+function splitTelegramHTMLMessage(html, maxLength = 3600) {
+  const chunks = [];
+  let current = "";
+  for (const line of html.split("\n")) {
+    const candidate = current ? `${current}
+${line}` : line;
+    if (candidate.length > maxLength && current) {
+      chunks.push(current);
+      current = line;
+    } else {
+      current = candidate;
+    }
+  }
+  if (current.trim()) chunks.push(current);
+  return chunks.length > 0 ? chunks : [html];
+}
 function normalizeTelegramChatId(value2) {
   if (typeof value2 === "number" || typeof value2 === "string") return value2;
   return 0;
@@ -591374,7 +591550,7 @@ function renderTelegramSubAgentError(username, error) {
   process.stdout.write(`    ${c3.dim("⎿")} ${c3.red("✘")} @${username}: ${c3.dim(preview)}
 `);
 }
-var TELEGRAM_SAFETY_PROMPT, ADMIN_DM_PROMPT, ADMIN_GROUP_PROMPT, GROUP_REPLY_DISCRETION_PROMPT, TELEGRAM_CHAT_MODE_PROMPT, ADMIN_CHAT_PROFILE_PROMPT, TELEGRAM_ACTION_RESPONSE_CONTRACT, TELEGRAM_ACTION_INTENT_RE, TELEGRAM_CODEBASE_CONTEXT_RE, TELEGRAM_COMMANDISH_RE, TELEGRAM_CHAT_INTENT_RE, MEDIA_CACHE_TTL_MS, TelegramBridge;
+var TELEGRAM_SAFETY_PROMPT, ADMIN_DM_PROMPT, ADMIN_GROUP_PROMPT, GROUP_REPLY_DISCRETION_PROMPT, TELEGRAM_CHAT_MODE_PROMPT, ADMIN_CHAT_PROFILE_PROMPT, TELEGRAM_ACTION_RESPONSE_CONTRACT, TELEGRAM_ACTION_INTENT_RE, TELEGRAM_CODEBASE_CONTEXT_RE, TELEGRAM_COMMANDISH_RE, TELEGRAM_CHAT_INTENT_RE, TELEGRAM_PUBLIC_HELP_COMMANDS, MEDIA_CACHE_TTL_MS, TelegramBridge;
 var init_telegram_bridge = __esm({
   "packages/cli/src/tui/telegram-bridge.ts"() {
     "use strict";
@@ -591384,6 +591560,7 @@ var init_telegram_bridge = __esm({
     init_render();
     init_tool_policy();
     init_media_routing();
+    init_command_registry();
     TELEGRAM_SAFETY_PROMPT = `
 CRITICAL SAFETY NOTICE — PUBLIC TELEGRAM CHANNEL
 
@@ -591465,6 +591642,7 @@ Telegram response contract:
     TELEGRAM_CODEBASE_CONTEXT_RE = /\b(repo|repository|codebase|workspace|working directory|file|files|folder|directory|src|source|test|tests|package|pnpm|npm|node|git|branch|commit|pr|pull request|issue|shell|terminal|cli|command|function|class|component|endpoint|api)\b/i;
     TELEGRAM_COMMANDISH_RE = /(^|\s)(pnpm|npm|node|git|rg|grep|sed|cat|ls|cd|mkdir|rm|mv|cp|curl|docker|pytest|vitest|tsc)\b/i;
     TELEGRAM_CHAT_INTENT_RE = /\b(hi|hello|hey|thanks|thank you|lol|haha|joke|how are you|what's up|whats up|can you hear|are you there|explain|what is|what are|why|how does|tell me|opinion|quick question)\b/i;
+    TELEGRAM_PUBLIC_HELP_COMMANDS = /* @__PURE__ */ new Set(["help", "start", "auth", "call"]);
     MEDIA_CACHE_TTL_MS = 30 * 60 * 1e3;
     TelegramBridge = class {
       constructor(botToken, onMessage, agentConfig, repoRoot, toolPolicyConfig) {
@@ -591632,6 +591810,15 @@ Telegram response contract:
         if (!canonical || canonical === botless) return input;
         return `/${canonical}${trimmed.slice(first2.length)}`;
       }
+      telegramSlashName(input) {
+        const first2 = input.trim().split(/\s+/)[0] ?? "";
+        if (!first2.startsWith("/")) return "";
+        return first2.slice(1).split("@")[0]?.toLowerCase() ?? "";
+      }
+      isTelegramHelpCommand(input) {
+        const name10 = this.telegramSlashName(input);
+        return name10 === "help" || name10 === "h" || name10 === "commands" || name10 === "cmds";
+      }
       isKnownTelegramSlash(input) {
         const first2 = input.trim().split(/\s+/)[0] ?? "";
         const botless = first2.startsWith("/") ? first2.slice(1).split("@")[0]?.toLowerCase() : "";
@@ -591658,6 +591845,25 @@ Telegram response contract:
       canUseChatActions(msg) {
         return !msg.guestQueryId && (typeof msg.chatId === "number" || String(msg.chatId).startsWith("@"));
       }
+      async replyWithTelegramHelp(msg, isAdmin) {
+        const scope = isAdmin ? "admin" : "public";
+        const chunks = splitTelegramHTMLMessage(buildTelegramHelpHTML(scope));
+        if (msg.guestQueryId) {
+          await this.answerGuestQuery(msg.guestQueryId, chunks[0] ?? "", { parseMode: "HTML" });
+          return;
+        }
+        for (let i2 = 0; i2 < chunks.length; i2++) {
+          const chunk = chunks[i2];
+          if (i2 === 0) {
+            await this.replyToTelegramMessage(msg, chunk, {
+              html: true,
+              replyToMessageId: msg.chatType !== "private" ? msg.messageId : void 0
+            });
+          } else {
+            await this.sendMessageHTML(msg.chatId, chunk);
+          }
+        }
+      }
       recordChatHistory(sessionKey, entry) {
         const existing = this.chatHistory.get(sessionKey) ?? [];
         existing.push({ ...entry, ts: entry.ts ?? Date.now() });
@@ -591938,11 +592144,15 @@ ${TELEGRAM_SAFETY_PROMPT}`);
         if (msg.text.trim().startsWith("/") && await this.handleAdminAuthCommand({ ...msg, text: normalizedCommandText })) {
           return;
         }
+        const isAdmin = this.isAdminUser(msg);
+        if (msg.text.trim().startsWith("/") && this.isTelegramHelpCommand(normalizedCommandText)) {
+          await this.replyWithTelegramHelp(msg, isAdmin);
+          return;
+        }
         if (!this.agentConfig || !this.repoRoot) {
           this.onMessage(msg);
           return;
         }
-        const isAdmin = this.isAdminUser(msg);
         const toolContext = this.resolveToolContext(msg, isAdmin);
         const isAdminDM = toolContext === "telegram-admin-dm";
         const sessionKey = this.sessionKeyForMessage(msg);

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "open-agents-ai",
-  "version": "0.187.586",
+  "version": "0.187.588",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "open-agents-ai",
-      "version": "0.187.586",
+      "version": "0.187.588",
       "hasInstallScript": true,
       "license": "CC-BY-NC-4.0",
       "dependencies": {
@@ -1914,12 +1914,12 @@
       }
     },
     "node_modules/@types/node": {
-      "version": "25.7.0",
-      "resolved": "https://registry.npmjs.org/@types/node/-/node-25.7.0.tgz",
-      "integrity": "sha512-z+pdZyxE+RTQE9AcboAZCb4otwcrvgHD+GlBpPgn0emDVt0ohrTMhAwlr2Wd9nZ+nihhYFxO2pThz3C5qSu2Eg==",
+      "version": "25.8.0",
+      "resolved": "https://registry.npmjs.org/@types/node/-/node-25.8.0.tgz",
+      "integrity": "sha512-TCFSk8IZh+iLX1xtksoBVtdmgL+1IX0fC9BeU4QqFSuNdN/K+HUlhqOzEmSYYpZUVsLYcPqc9KX+60iDuninSQ==",
       "license": "MIT",
       "dependencies": {
-        "undici-types": "~7.21.0"
+        "undici-types": ">=7.24.0 <7.24.7"
       }
     },
     "node_modules/@types/sinon": {
@@ -6994,9 +6994,9 @@
       }
     },
     "node_modules/undici-types": {
-      "version": "7.21.0",
-      "resolved": "https://registry.npmjs.org/undici-types/-/undici-types-7.21.0.tgz",
-      "integrity": "sha512-w9IMgQrz4O0YN1LtB7K5P63vhlIOvC7opSmouCJ+ZywlPAlO9gIkJ+otk6LvGpAs2wg4econaCz3TvQ9xPoyuQ==",
+      "version": "7.24.6",
+      "resolved": "https://registry.npmjs.org/undici-types/-/undici-types-7.24.6.tgz",
+      "integrity": "sha512-WRNW+sJgj5OBN4/0JpHFqtqzhpbnV0GuB+OozA9gCL7a993SmU+1JBZCzLNxYsbMfIeDL+lTsphD5jN5N+n0zg==",
       "license": "MIT"
     },
     "node_modules/unlimited-timeout": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "open-agents-ai",
-  "version": "0.187.586",
+  "version": "0.187.588",
   "description": "AI coding agent powered by open-source models (Ollama/vLLM) — interactive TUI with agentic tool-calling loop",
   "type": "module",
   "main": "./dist/index.js",