@animalabs/agent-framework 0.23.1 → 0.24.0

package/src/modules/history/semantic.ts

@@ -17,8 +17,12 @@
  * What gets embedded per message: text blocks verbatim, plus the string
  * arguments of the agent's private-prose tools (think / journal / skip_reply
  * / private_note) labelled `[think] …` — that is the resident's own diary and
- * is exactly what "what did I think about X" should find. tool_result
- * payloads, other tool arguments, thinking blocks and media are skipped.
+ * is exactly what "what did I think about X" should find. The message
+ * argument of the send tools and edit_message, labelled by tool, such as
+ * `[send_message] …`, since with prose routing disabled the resident speaks
+ * through them.
+ * tool_result payloads, other tool arguments, thinking blocks and media are
+ * skipped.
  *
  * Failure posture: the service being down never breaks the module. Sync ticks
  * log and back off; `semantic_search` returns a clean tool error.
@@ -27,6 +31,7 @@
 import type { ContextManager, StoredMessage } from '@animalabs/context-manager';
 import type { ContentBlock } from '@animalabs/membrane';
 import { safeSlice } from '../../safe-slice.js';
+import { SEND_TOOLS, bareToolName } from '../../send-tools.js';
 
 export interface SemanticIndexConfig {
   /** Base URL of the embed-service, e.g. `http://100.90.161.34:8804`. */
@@ -80,8 +85,10 @@ export interface SearchHit {
   kind: string;
   level: number | null;
   meta: Record<string, unknown>;
+  /** The text the service sends for the hit, where it keeps text; a service that keeps none leaves it out. */
   text?: string;
-  chars: number;
+  /** The length of the text the service holds for the hit; a service that keeps no text may leave it out too. */
+  chars?: number;
 }
 
 export interface SearchRequest {
@@ -104,6 +111,12 @@ export interface SearchResponse {
 }
 
 const PRIVATE_PROSE_TOOLS = new Set(['think', 'journal', 'skip_reply', 'private_note']);
+/**
+ * Tools whose message argument is what the resident said: the send tools,
+ * and edit_message, whose argument is the new text of a message it sent.
+ * With prose routing disabled the resident speaks through these.
+ */
+const SPEECH_TOOLS = new Set([...SEND_TOOLS, 'edit_message']);
 
 /**
  * The semantic index service's failure, said of the service alone: it didn't
@@ -269,6 +282,12 @@ export function messageIndexText(msg: StoredMessage, includePrivateTools = true)
       for (const v of Object.values(input)) {
         if (typeof v === 'string' && v.trim().length > 0) parts.push(`[${block.name}] ${v}`);
       }
+    } else if (block.type === 'tool_use' && SPEECH_TOOLS.has(bareToolName(block.name))) {
+      // Only the message: the other arguments are ids, topics and names.
+      // channel_publish still takes `text` from older callers.
+      const input = (block as { input?: { content?: unknown; text?: unknown } }).input ?? {};
+      const v = input.content ?? input.text;
+      if (typeof v === 'string' && v.trim().length > 0) parts.push(`[${bareToolName(block.name)}] ${v}`);
     }
   }
   return parts.join('\n').trim();
@@ -315,12 +334,16 @@ export interface SummaryLike {
   parentId?: string;
 }
 
+/** The text a summary is indexed by: its content trimmed, and cut at `maxChars` at a whole character. */
+export function summaryIndexText(s: { content: string }, cfg: { maxChars?: number }): string {
+  return safeSlice(s.content.trim(), 0, cfg.maxChars ?? 32_000);
+}
+
 export function summaryToItem(s: SummaryLike, cfg: { maxChars?: number }): IndexItem | null {
-  const text = s.content.trim();
-  if (!text) return null;
+  if (!s.content.trim()) return null;
   return {
     id: `sum:${s.id}`,
-    text: safeSlice(text, 0, cfg.maxChars ?? 32_000),
+    text: summaryIndexText(s, cfg),
     ts: s.startMs / 1000,
     kind: 'summary',
     level: s.level,

package/src/modules/index.ts

@@ -21,6 +21,7 @@ export type {
   ReadInput,
   WriteInput,
   EditInput,
+  EditManyInput,
   DeleteInput,
   LsInput,
   GlobInput,

package/src/modules/unknown-tool.ts

@@ -5,7 +5,10 @@ import type { ToolDefinition } from '../types/events.js';
  * tool, so nothing ran, and which tools the module has, read from its own
  * getTools() so the list can't drift. The registry hands every `module--name`
  * to its module, even one the module doesn't declare, since some modules route
- * undeclared names on purpose, so this answer is the module's to give.
+ * undeclared names on purpose, so this answer is the module's to give. On the
+ * dispatcher's path it comes after the call has opened (RFC-007), and an
+ * in-process route opens none, so either way its result is a failure whose
+ * text says nothing ran, not a refusal (ToolResult.refused).
  */
 export function unknownToolMessage(module: string, name: string, tools: readonly Pick<ToolDefinition, 'name'>[]): string {
   const missing = `There's no tool "${module}--${name}", so nothing ran.`;

package/src/modules/workspace/branch-intent.ts

@@ -6,7 +6,9 @@
  *
  *  - tombstone: the workspace deleted the path while disk kept a copy (no
  *    autoMaterialize, or a refused unlink). The hash is the deleted entry's,
- *    so even a legacy entry with no disk evidence keeps a durable intent.
+ *    so even a legacy entry with no disk evidence keeps a durable intent. A
+ *    tombstone beside an entry that's still there, left by a change that
+ *    stopped partway (commitToolChanges), is no deletion, and guards nothing.
  *  - origin 'store': the entry was created through workspace tools and disk
  *    has never agreed with it.
  *  - conflict: disk and workspace diverged in a way no rule may resolve