@goondan/openharness-base 0.1.2 → 0.1.4

package/dist/index.d.ts

@@ -1,12 +1,15 @@
-import { Extension, ToolDefinition } from '@goondan/openharness-types';
+import { Extension, LlmChatOptions, Message, ToolDefinition } from '@goondan/openharness-types';
 
 /**
- * ContextMessage extension — prepends a system message to the conversation
+ * BasicSystemPrompt extension — prepends a system message to the conversation
  * at the start of every turn.
  *
+ * Uses a fixed message ID so the system prompt is only appended once;
+ * subsequent turns detect the existing message and skip the append.
+ *
  * Priority 10 (HIGH) ensures it runs before other turn middleware.
  */
-declare function ContextMessage(text: string): Extension;
+declare function BasicSystemPrompt(text: string): Extension;
 
 /**
  * MessageWindow extension — truncates conversation history to keep only
@@ -18,13 +21,22 @@ declare function MessageWindow(config: {
 
 /**
  * CompactionSummarize extension — when message count exceeds `threshold`,
- * removes the oldest messages and prepends a summary system message.
+ * removes the oldest messages and replaces them with an LLM-generated summary.
+ *
+ * By default, uses the agent's own LLM (`ctx.llm`) to produce the summary.
+ * A custom `summarizer` callback can override this for advanced use cases
+ * (e.g. using a cheaper model, external API, or deterministic logic).
  *
- * For MVP the "summary" is just the concatenation of removed message text.
+ * @param config.threshold - Trigger compaction when messages exceed this count.
+ * @param config.summaryPrompt - Custom system prompt for the LLM summarizer.
+ * @param config.summarizer - Optional override: produce summary text from messages.
  */
 declare function CompactionSummarize(config: {
     threshold: number;
     summaryPrompt?: string;
+    /** LLM options for the summarization call (e.g. model override for cheaper summarization). */
+    llmOptions?: LlmChatOptions;
+    summarizer?: (messages: Message[]) => Promise<string>;
 }): Extension;
 
 /**
@@ -69,4 +81,4 @@ interface WaitToolConfig {
 }
 declare function WaitTool(config?: WaitToolConfig): ToolDefinition;
 
-export { BashTool, type BashToolConfig, CompactionSummarize, ContextMessage, FileListTool, FileReadTool, FileWriteTool, HttpFetchTool, JsonQueryTool, Logging, MessageWindow, RequiredToolsGuard, TextTransformTool, ToolSearch, WaitTool, type WaitToolConfig };
+export { BashTool, type BashToolConfig, BasicSystemPrompt, CompactionSummarize, FileListTool, FileReadTool, FileWriteTool, HttpFetchTool, JsonQueryTool, Logging, MessageWindow, RequiredToolsGuard, TextTransformTool, ToolSearch, WaitTool, type WaitToolConfig };

package/dist/index.js

@@ -1,25 +1,30 @@
-// src/extensions/context-message.ts
-import { randomUUID } from "crypto";
-function ContextMessage(text) {
+// src/extensions/basic-system-prompt.ts
+var SYSTEM_MESSAGE_ID = "sys-basic-system-prompt";
+function BasicSystemPrompt(text) {
   return {
-    name: "context-message",
+    name: "basic-system-prompt",
     register(api) {
       api.pipeline.register(
         "turn",
         async (ctx, next) => {
-          ctx.conversation.emit({
-            type: "append",
-            message: {
-              id: `ctx-msg-${randomUUID()}`,
-              data: {
-                role: "system",
-                content: text
-              },
-              metadata: {
-                __createdBy: "context-message"
+          const alreadyExists = ctx.conversation.messages.some(
+            (m) => m.id === SYSTEM_MESSAGE_ID
+          );
+          if (!alreadyExists) {
+            ctx.conversation.emit({
+              type: "append",
+              message: {
+                id: SYSTEM_MESSAGE_ID,
+                data: {
+                  role: "system",
+                  content: text
+                },
+                metadata: {
+                  __createdBy: "basic-system-prompt"
+                }
               }
-            }
-          });
+            });
+          }
           return next();
         },
         { priority: 10 }
@@ -47,7 +52,13 @@ function MessageWindow(config) {
 }
 
 // src/extensions/compaction-summarize.ts
-import { randomUUID as randomUUID2 } from "crypto";
+import { randomUUID } from "crypto";
+var DEFAULT_SUMMARY_PROMPT = "You are a conversation compactor. Summarize the following messages into a concise summary that preserves all important context, decisions, facts, and action items. Be thorough but brief. Output only the summary text, nothing else.";
+function messageToText(m) {
+  const role = m.data.role;
+  const content = typeof m.data.content === "string" ? m.data.content : JSON.stringify(m.data.content);
+  return `[${role}]: ${content}`;
+}
 function CompactionSummarize(config) {
   return {
     name: "compaction-summarize",
@@ -58,16 +69,36 @@ function CompactionSummarize(config) {
           const keepCount = Math.floor(config.threshold / 2);
           const removeCount = messages.length - keepCount;
           const toRemove = messages.slice(0, removeCount);
-          const summaryText = toRemove.map(
-            (m) => typeof m.data.content === "string" ? m.data.content : JSON.stringify(m.data.content)
-          ).join(" ");
-          for (const msg of toRemove) {
+          let summaryText;
+          if (config.summarizer) {
+            summaryText = await config.summarizer([...toRemove]);
+          } else {
+            const transcript = toRemove.map(messageToText).join("\n");
+            const prompt = config.summaryPrompt ?? DEFAULT_SUMMARY_PROMPT;
+            const llmResponse = await ctx.llm.chat(
+              [
+                { id: `compaction-sys-${randomUUID()}`, data: { role: "system", content: prompt }, metadata: {} },
+                { id: `compaction-usr-${randomUUID()}`, data: { role: "user", content: transcript }, metadata: {} }
+              ],
+              [],
+              // no tools needed for summarization
+              ctx.abortSignal,
+              config.llmOptions
+            );
+            summaryText = llmResponse.text ?? transcript;
+          }
+          const [firstToRemove, ...restToRemove] = toRemove;
+          ctx.conversation.emit({
+            type: "remove",
+            messageId: firstToRemove.id
+          });
+          for (const msg of restToRemove) {
             ctx.conversation.emit({ type: "remove", messageId: msg.id });
           }
           ctx.conversation.emit({
             type: "append",
             message: {
-              id: `summary-${randomUUID2()}`,
+              id: `summary-${randomUUID()}`,
               data: {
                 role: "system",
                 content: `[Summary of earlier conversation]: ${summaryText}`
@@ -490,8 +521,8 @@ function WaitTool(config = {}) {
 }
 export {
   BashTool,
+  BasicSystemPrompt,
   CompactionSummarize,
-  ContextMessage,
   FileListTool,
   FileReadTool,
   FileWriteTool,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goondan/openharness-base",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "type": "module",
   "exports": {
     ".": {
@@ -9,7 +9,7 @@
     }
   },
   "dependencies": {
-    "@goondan/openharness-types": "0.1.2"
+    "@goondan/openharness-types": "0.1.4"
   },
   "devDependencies": {
     "@types/node": "^25.5.0",

package/src/__tests__/{context-message.test.ts → basic-system-prompt.test.ts}

@@ -1,5 +1,5 @@
 import { describe, it, expect, vi } from "vitest";
-import { ContextMessage } from "../extensions/context-message.js";
+import { BasicSystemPrompt } from "../extensions/basic-system-prompt.js";
 import type {
   ExtensionApi,
   TurnMiddleware,
@@ -82,6 +82,7 @@ function makeTurnContext(conversation: ConversationState): TurnContext {
         receivedAt: new Date().toISOString(),
       },
     },
+    llm: { chat: vi.fn().mockResolvedValue({ text: "mock" }) },
   };
 }
 
@@ -97,19 +98,17 @@ const stubTurnResult: TurnResult = {
 // Tests
 // ---------------------------------------------------------------------------
 
-describe("ContextMessage", () => {
-  // Test 1: creates Extension with correct name
-  it("creates an Extension with name 'context-message'", () => {
-    const ext = ContextMessage("You are a helpful assistant.");
-    expect(ext.name).toBe("context-message");
+describe("BasicSystemPrompt", () => {
+  it("creates an Extension with name 'basic-system-prompt'", () => {
+    const ext = BasicSystemPrompt("You are helpful.");
+    expect(ext.name).toBe("basic-system-prompt");
   });
 
-  // Test 2: register() adds turn middleware via api.pipeline.register
   it("register() calls api.pipeline.register with level 'turn'", () => {
     const conversation = makeMockConversationState();
     const { api, registeredMiddleware } = makeMockApi(conversation);
 
-    const ext = ContextMessage("Hello");
+    const ext = BasicSystemPrompt("You are helpful.");
     ext.register(api);
 
     expect(api.pipeline.register).toHaveBeenCalledOnce();
@@ -118,86 +117,70 @@ describe("ContextMessage", () => {
     expect(typeof registeredMiddleware[0].handler).toBe("function");
   });
 
-  // Test 3: middleware is registered with HIGH priority (low number)
   it("registers turn middleware with priority 10 (high priority)", () => {
     const conversation = makeMockConversationState();
     const { api, registeredMiddleware } = makeMockApi(conversation);
 
-    const ext = ContextMessage("Hello");
+    const ext = BasicSystemPrompt("You are helpful.");
     ext.register(api);
 
     expect(registeredMiddleware[0].options?.priority).toBe(10);
   });
 
-  // Test 4: middleware prepends system message to conversation and text matches
-  it("middleware emits append event with system message matching the text argument", async () => {
+  it("middleware appends a system message and calls next()", async () => {
     const conversation = makeMockConversationState();
     const { api, registeredMiddleware } = makeMockApi(conversation);
-    const systemText = "You are a helpful assistant.";
 
-    const ext = ContextMessage(systemText);
+    const ext = BasicSystemPrompt("You are helpful.");
     ext.register(api);
 
     const middleware = registeredMiddleware[0].handler;
     const ctx = makeTurnContext(conversation);
     const next = vi.fn(async () => stubTurnResult);
 
-    await middleware(ctx, next);
+    const result = await middleware(ctx, next);
 
+    expect(next).toHaveBeenCalledOnce();
+    expect(result).toBe(stubTurnResult);
+
+    // Verify system message was emitted
     expect(conversation.emit).toHaveBeenCalledOnce();
-    const emitted = conversation.emitted[0];
-    expect(emitted.type).toBe("append");
-    if (emitted.type === "append") {
-      expect(emitted.message.data.role).toBe("system");
-      expect(emitted.message.data.content).toBe(systemText);
-      expect(emitted.message.metadata?.__createdBy).toBe("context-message");
-      expect(typeof emitted.message.id).toBe("string");
+    const emittedEvent = conversation.emitted[0];
+    expect(emittedEvent.type).toBe("append");
+    if (emittedEvent.type === "append") {
+      expect(emittedEvent.message.id).toBe("sys-basic-system-prompt");
+      expect(emittedEvent.message.data.role).toBe("system");
+      expect(emittedEvent.message.data.content).toBe("You are helpful.");
+      expect(emittedEvent.message.metadata?.__createdBy).toBe("basic-system-prompt");
     }
   });
 
-  // Test 5: middleware calls next() after emitting
-  it("middleware calls next() after emitting the system message", async () => {
+  it("does not duplicate the system message on subsequent turns", async () => {
     const conversation = makeMockConversationState();
     const { api, registeredMiddleware } = makeMockApi(conversation);
 
-    const ext = ContextMessage("Some context");
+    const ext = BasicSystemPrompt("You are helpful.");
     ext.register(api);
 
     const middleware = registeredMiddleware[0].handler;
-    const ctx = makeTurnContext(conversation);
-    const next = vi.fn(async () => stubTurnResult);
-
-    const result = await middleware(ctx, next);
-
-    expect(next).toHaveBeenCalledOnce();
-    expect(result).toBe(stubTurnResult);
-  });
-
-  // Test 6: multiple ContextMessage extensions → multiple system messages
-  it("multiple ContextMessage extensions each prepend their own system message", async () => {
-    const conversation = makeMockConversationState();
-    const { api: api1, registeredMiddleware: rm1 } = makeMockApi(conversation);
-    const { api: api2, registeredMiddleware: rm2 } = makeMockApi(conversation);
-
-    const ext1 = ContextMessage("First context");
-    const ext2 = ContextMessage("Second context");
-
-    ext1.register(api1);
-    ext2.register(api2);
-
-    const ctx = makeTurnContext(conversation);
     const next = vi.fn(async () => stubTurnResult);
 
-    await rm1[0].handler(ctx, next);
-    await rm2[0].handler(ctx, next);
+    // First turn — should append
+    const ctx1 = makeTurnContext(conversation);
+    await middleware(ctx1, next);
+    expect(conversation.emitted).toHaveLength(1);
 
-    expect(conversation.emitted).toHaveLength(2);
+    // Second turn — system message already in conversation.messages, should skip
+    const ctx2 = makeTurnContext(conversation);
+    await middleware(ctx2, next);
+    expect(conversation.emitted).toHaveLength(1); // still 1, no new append
 
-    const texts = conversation.emitted
-      .filter((e): e is Extract<typeof e, { type: "append" }> => e.type === "append")
-      .map((e) => e.message.data.content);
+    // Third turn — still no duplication
+    const ctx3 = makeTurnContext(conversation);
+    await middleware(ctx3, next);
+    expect(conversation.emitted).toHaveLength(1);
 
-    expect(texts).toContain("First context");
-    expect(texts).toContain("Second context");
+    // next() should have been called every turn
+    expect(next).toHaveBeenCalledTimes(3);
   });
 });

package/src/__tests__/compaction-summarize.test.ts

@@ -83,6 +83,12 @@ function makeMockApi(conversation: ConversationState): {
   return { api, registeredMiddleware };
 }
 
+function makeMockLlmClient() {
+  return {
+    chat: vi.fn().mockResolvedValue({ text: "LLM-generated summary of the conversation." }),
+  };
+}
+
 function makeStepContext(conversation: ConversationState): StepContext {
   return {
     turnId: "turn-1",
@@ -101,6 +107,7 @@ function makeStepContext(conversation: ConversationState): StepContext {
         receivedAt: new Date().toISOString(),
       },
     },
+    llm: makeMockLlmClient(),
   };
 }
 
@@ -223,12 +230,53 @@ describe("CompactionSummarize", () => {
 
     await middleware(ctx, next);
 
+    // Verify LLM was called for summarization
+    expect(ctx.llm.chat).toHaveBeenCalledOnce();
+
     const emitted = (conversation as ReturnType<typeof makeMockConversationState>).emitted;
     const appendEvent = emitted.find((e) => e.type === "append");
     if (appendEvent && appendEvent.type === "append") {
       const content = appendEvent.message.data.content as string;
-      // Should include text from the removed messages
-      expect(content).toContain("Message 0");
+      // Should include the LLM-generated summary
+      expect(content).toContain("LLM-generated summary of the conversation.");
     }
   });
+
+  it("uses custom summarizer when provided", async () => {
+    const messages = makeMessages(12);
+    const conversation = makeMockConversationState(messages);
+    const { api, registeredMiddleware } = makeMockApi(conversation);
+
+    const summarizer = vi.fn(async (msgs: Message[]) => {
+      return `Custom summary of ${msgs.length} messages`;
+    });
+
+    const ext = CompactionSummarize({ threshold: 10, summarizer });
+    ext.register(api);
+
+    const middleware = registeredMiddleware[0].handler;
+    const ctx = makeStepContext(conversation);
+    const next = vi.fn(async () => stubStepResult);
+
+    await middleware(ctx, next);
+
+    // summarizer should have been called with the 7 removed messages
+    expect(summarizer).toHaveBeenCalledOnce();
+    expect(summarizer).toHaveBeenCalledWith(
+      expect.arrayContaining([
+        expect.objectContaining({ id: "msg-0" }),
+        expect.objectContaining({ id: "msg-6" }),
+      ]),
+    );
+    expect(summarizer.mock.calls[0][0]).toHaveLength(7);
+
+    const emitted = (conversation as ReturnType<typeof makeMockConversationState>).emitted;
+    const appendEvent = emitted.find((e) => e.type === "append");
+    if (appendEvent && appendEvent.type === "append") {
+      const content = appendEvent.message.data.content as string;
+      expect(content).toContain("Custom summary of 7 messages");
+    }
+
+    expect(next).toHaveBeenCalledOnce();
+  });
 });

package/src/__tests__/message-window.test.ts

@@ -101,6 +101,7 @@ function makeStepContext(conversation: ConversationState): StepContext {
         receivedAt: new Date().toISOString(),
       },
     },
+    llm: { chat: vi.fn().mockResolvedValue({ text: "mock" }) },
   };
 }
 

package/src/__tests__/required-tools-guard.test.ts

@@ -86,6 +86,7 @@ function makeTurnContext(conversation: ConversationState): TurnContext {
         receivedAt: new Date().toISOString(),
       },
     },
+    llm: { chat: vi.fn().mockResolvedValue({ text: "mock" }) },
   };
 }
 

package/src/extensions/basic-system-prompt.ts

@@ -0,0 +1,48 @@
+import type { Extension, ExtensionApi } from "@goondan/openharness-types";
+
+const SYSTEM_MESSAGE_ID = "sys-basic-system-prompt";
+
+/**
+ * BasicSystemPrompt extension — prepends a system message to the conversation
+ * at the start of every turn.
+ *
+ * Uses a fixed message ID so the system prompt is only appended once;
+ * subsequent turns detect the existing message and skip the append.
+ *
+ * Priority 10 (HIGH) ensures it runs before other turn middleware.
+ */
+export function BasicSystemPrompt(text: string): Extension {
+  return {
+    name: "basic-system-prompt",
+
+    register(api: ExtensionApi): void {
+      api.pipeline.register(
+        "turn",
+        async (ctx, next) => {
+          const alreadyExists = ctx.conversation.messages.some(
+            (m) => m.id === SYSTEM_MESSAGE_ID,
+          );
+
+          if (!alreadyExists) {
+            ctx.conversation.emit({
+              type: "append",
+              message: {
+                id: SYSTEM_MESSAGE_ID,
+                data: {
+                  role: "system",
+                  content: text,
+                },
+                metadata: {
+                  __createdBy: "basic-system-prompt",
+                },
+              },
+            });
+          }
+
+          return next();
+        },
+        { priority: 10 },
+      );
+    },
+  };
+}

package/src/extensions/compaction-summarize.ts

@@ -1,15 +1,41 @@
-import type { Extension, ExtensionApi } from "@goondan/openharness-types";
+import type { Extension, ExtensionApi, Message, LlmChatOptions } from "@goondan/openharness-types";
 import { randomUUID } from "node:crypto";
 
+const DEFAULT_SUMMARY_PROMPT =
+  "You are a conversation compactor. Summarize the following messages into a concise summary " +
+  "that preserves all important context, decisions, facts, and action items. " +
+  "Be thorough but brief. Output only the summary text, nothing else.";
+
+/**
+ * Extract plain-text representation of a Message for summarization.
+ */
+function messageToText(m: Message): string {
+  const role = m.data.role;
+  const content =
+    typeof m.data.content === "string"
+      ? m.data.content
+      : JSON.stringify(m.data.content);
+  return `[${role}]: ${content}`;
+}
+
 /**
  * CompactionSummarize extension — when message count exceeds `threshold`,
- * removes the oldest messages and prepends a summary system message.
+ * removes the oldest messages and replaces them with an LLM-generated summary.
+ *
+ * By default, uses the agent's own LLM (`ctx.llm`) to produce the summary.
+ * A custom `summarizer` callback can override this for advanced use cases
+ * (e.g. using a cheaper model, external API, or deterministic logic).
  *
- * For MVP the "summary" is just the concatenation of removed message text.
+ * @param config.threshold - Trigger compaction when messages exceed this count.
+ * @param config.summaryPrompt - Custom system prompt for the LLM summarizer.
+ * @param config.summarizer - Optional override: produce summary text from messages.
  */
 export function CompactionSummarize(config: {
   threshold: number;
   summaryPrompt?: string;
+  /** LLM options for the summarization call (e.g. model override for cheaper summarization). */
+  llmOptions?: LlmChatOptions;
+  summarizer?: (messages: Message[]) => Promise<string>;
 }): Extension {
   return {
     name: "compaction-summarize",
@@ -22,21 +48,41 @@ export function CompactionSummarize(config: {
           const removeCount = messages.length - keepCount;
           const toRemove = messages.slice(0, removeCount);
 
-          // Build a naive summary from removed messages
-          const summaryText = toRemove
-            .map((m) =>
-              typeof m.data.content === "string"
-                ? m.data.content
-                : JSON.stringify(m.data.content),
-            )
-            .join(" ");
-
-          // Remove old messages
-          for (const msg of toRemove) {
+          let summaryText: string;
+
+          if (config.summarizer) {
+            // User-provided summarizer takes precedence
+            summaryText = await config.summarizer([...toRemove]);
+          } else {
+            // Default: LLM-based summarization via ctx.llm
+            const transcript = toRemove.map(messageToText).join("\n");
+            const prompt = config.summaryPrompt ?? DEFAULT_SUMMARY_PROMPT;
+
+            const llmResponse = await ctx.llm.chat(
+              [
+                { id: `compaction-sys-${randomUUID()}`, data: { role: "system", content: prompt }, metadata: {} },
+                { id: `compaction-usr-${randomUUID()}`, data: { role: "user", content: transcript }, metadata: {} },
+              ],
+              [], // no tools needed for summarization
+              ctx.abortSignal,
+              config.llmOptions,
+            );
+
+            summaryText = llmResponse.text ?? transcript;
+          }
+
+          // Remove old messages, replace first with summary for stable ordering
+          const [firstToRemove, ...restToRemove] = toRemove;
+          // Replace the first message with the summary
+          ctx.conversation.emit({
+            type: "remove",
+            messageId: firstToRemove.id,
+          });
+          for (const msg of restToRemove) {
             ctx.conversation.emit({ type: "remove", messageId: msg.id });
           }
 
-          // Prepend summary
+          // Prepend summary as a system message
           ctx.conversation.emit({
             type: "append",
             message: {

package/src/index.ts

@@ -1,4 +1,4 @@
-export { ContextMessage } from "./extensions/context-message.js";
+export { BasicSystemPrompt } from "./extensions/basic-system-prompt.js";
 export { MessageWindow } from "./extensions/message-window.js";
 export { CompactionSummarize } from "./extensions/compaction-summarize.js";
 export { Logging } from "./extensions/logging.js";

package/dist/extensions/message-integrity.d.ts

@@ -1,8 +0,0 @@
-import type { Message } from '../types.js';
-/**
- * Removes messages that would leave invalid tool_result references after trimming.
- * Anthropic requires every tool_result block to map to a tool_use/tool-call block
- * in the immediately previous assistant message.
- */
-export declare function normalizeRemovalTargets(messages: Message[], initialRemovedIds: ReadonlySet<string>): Set<string>;
-//# sourceMappingURL=message-integrity.d.ts.map

package/dist/extensions/message-integrity.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"message-integrity.d.ts","sourceRoot":"","sources":["../../src/extensions/message-integrity.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,aAAa,CAAC;AAiF3C;;;;GAIG;AACH,wBAAgB,uBAAuB,CACrC,QAAQ,EAAE,OAAO,EAAE,EACnB,iBAAiB,EAAE,WAAW,CAAC,MAAM,CAAC,GACrC,GAAG,CAAC,MAAM,CAAC,CAwBb"}

package/dist/extensions/message-integrity.js

@@ -1,88 +0,0 @@
-function readToolCallId(part) {
-    const value = part.toolCallId;
-    if (typeof value !== 'string' || value.length === 0) {
-        return null;
-    }
-    return value;
-}
-function collectToolCallIds(message) {
-    const ids = new Set();
-    if (!message) {
-        return ids;
-    }
-    const content = message.data.content;
-    if (!Array.isArray(content)) {
-        return ids;
-    }
-    for (const part of content) {
-        if (part.type !== 'tool-call') {
-            continue;
-        }
-        const toolCallId = readToolCallId(part);
-        if (!toolCallId) {
-            continue;
-        }
-        ids.add(toolCallId);
-    }
-    return ids;
-}
-function collectToolResultIds(message) {
-    const content = message.data.content;
-    if (!Array.isArray(content)) {
-        return [];
-    }
-    const ids = [];
-    for (const part of content) {
-        if (part.type !== 'tool-result') {
-            continue;
-        }
-        const toolCallId = readToolCallId(part);
-        if (!toolCallId) {
-            continue;
-        }
-        ids.push(toolCallId);
-    }
-    return ids;
-}
-function hasDanglingToolResult(message, previousMessage) {
-    const toolResultIds = collectToolResultIds(message);
-    if (toolResultIds.length === 0) {
-        return false;
-    }
-    const previousToolCallIds = collectToolCallIds(previousMessage);
-    if (previousToolCallIds.size === 0) {
-        return true;
-    }
-    for (const toolResultId of toolResultIds) {
-        if (!previousToolCallIds.has(toolResultId)) {
-            return true;
-        }
-    }
-    return false;
-}
-/**
- * Removes messages that would leave invalid tool_result references after trimming.
- * Anthropic requires every tool_result block to map to a tool_use/tool-call block
- * in the immediately previous assistant message.
- */
-export function normalizeRemovalTargets(messages, initialRemovedIds) {
-    const removedIds = new Set(initialRemovedIds);
-    let changed = true;
-    while (changed) {
-        changed = false;
-        let previousRemainingMessage;
-        for (const message of messages) {
-            if (removedIds.has(message.id)) {
-                continue;
-            }
-            if (hasDanglingToolResult(message, previousRemainingMessage)) {
-                removedIds.add(message.id);
-                changed = true;
-                continue;
-            }
-            previousRemainingMessage = message;
-        }
-    }
-    return removedIds;
-}
-//# sourceMappingURL=message-integrity.js.map

package/dist/extensions/message-integrity.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"message-integrity.js","sourceRoot":"","sources":["../../src/extensions/message-integrity.ts"],"names":[],"mappings":"AAOA,SAAS,cAAc,CAAC,IAAwB;IAC9C,MAAM,KAAK,GAAG,IAAI,CAAC,UAAU,CAAC;IAC9B,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACpD,OAAO,IAAI,CAAC;IACd,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,SAAS,kBAAkB,CAAC,OAA4B;IACtD,MAAM,GAAG,GAAG,IAAI,GAAG,EAAU,CAAC;IAC9B,IAAI,CAAC,OAAO,EAAE,CAAC;QACb,OAAO,GAAG,CAAC;IACb,CAAC;IAED,MAAM,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC,OAAO,CAAC;IACrC,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC;QAC5B,OAAO,GAAG,CAAC;IACb,CAAC;IAED,KAAK,MAAM,IAAI,IAAI,OAAO,EAAE,CAAC;QAC3B,IAAI,IAAI,CAAC,IAAI,KAAK,WAAW,EAAE,CAAC;YAC9B,SAAS;QACX,CAAC;QACD,MAAM,UAAU,GAAG,cAAc,CAAC,IAA0B,CAAC,CAAC;QAC9D,IAAI,CAAC,UAAU,EAAE,CAAC;YAChB,SAAS;QACX,CAAC;QACD,GAAG,CAAC,GAAG,CAAC,UAAU,CAAC,CAAC;IACtB,CAAC;IAED,OAAO,GAAG,CAAC;AACb,CAAC;AAED,SAAS,oBAAoB,CAAC,OAAgB;IAC5C,MAAM,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC,OAAO,CAAC;IACrC,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC;QAC5B,OAAO,EAAE,CAAC;IACZ,CAAC;IAED,MAAM,GAAG,GAAa,EAAE,CAAC;IACzB,KAAK,MAAM,IAAI,IAAI,OAAO,EAAE,CAAC;QAC3B,IAAI,IAAI,CAAC,IAAI,KAAK,aAAa,EAAE,CAAC;YAChC,SAAS;QACX,CAAC;QACD,MAAM,UAAU,GAAG,cAAc,CAAC,IAA0B,CAAC,CAAC;QAC9D,IAAI,CAAC,UAAU,EAAE,CAAC;YAChB,SAAS;QACX,CAAC;QACD,GAAG,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;IACvB,CAAC;IAED,OAAO,GAAG,CAAC;AACb,CAAC;AAED,SAAS,qBAAqB,CAAC,OAAgB,EAAE,eAAoC;IACnF,MAAM,aAAa,GAAG,oBAAoB,CAAC,OAAO,CAAC,CAAC;IACpD,IAAI,aAAa,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC/B,OAAO,KAAK,CAAC;IACf,CAAC;IAED,MAAM,mBAAmB,GAAG,kBAAkB,CAAC,eAAe,CAAC,CAAC;IAChE,IAAI,mBAAmB,CAAC,IAAI,KAAK,CAAC,EAAE,CAAC;QACnC,OAAO,IAAI,CAAC;IACd,CAAC;IAED,KAAK,MAAM,YAAY,IAAI,aAAa,EAAE,CAAC;QACzC,IAAI,CAAC,mBAAmB,CAAC,GAAG,CAAC,YAAY,CAAC,EAAE,CAAC;YAC3C,OAAO,IAAI,CAAC;QACd,CAAC;IACH,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,uBAAuB,CACrC,QAAmB,EACnB,iBAAsC;IAEtC,MAAM,UAAU,GAAG,IAAI,GAAG,CAAC,iBAAiB,CAAC,CAAC;IAE9C,IAAI,OAAO,GAAG,IAAI,CAAC;IACnB,OAAO,OAAO,EAAE,CAAC;QACf,OAAO,GAAG,KAAK,CAAC;QAChB,IAAI,wBAA6C,CAAC;QAElD,KAAK,MAAM,OAAO,IAAI,QAAQ,EAAE,CAAC;YAC/B,IAAI,UAAU,CAAC,GAAG,CAAC,OAAO,CAAC,EAAE,CAAC,EAAE,CAAC;gBAC/B,SAAS;YACX,CAAC;YAED,IAAI,qBAAqB,CAAC,OAAO,EAAE,wBAAwB,CAAC,EAAE,CAAC;gBAC7D,UAAU,CAAC,GAAG,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;gBAC3B,OAAO,GAAG,IAAI,CAAC;gBACf,SAAS;YACX,CAAC;YAED,wBAAwB,GAAG,OAAO,CAAC;QACrC,CAAC;IACH,CAAC;IAED,OAAO,UAAU,CAAC;AACpB,CAAC"}

package/src/extensions/context-message.ts

@@ -1,37 +0,0 @@
-import type { Extension, ExtensionApi } from "@goondan/openharness-types";
-import { randomUUID } from "node:crypto";
-
-/**
- * ContextMessage extension — prepends a system message to the conversation
- * at the start of every turn.
- *
- * Priority 10 (HIGH) ensures it runs before other turn middleware.
- */
-export function ContextMessage(text: string): Extension {
-  return {
-    name: "context-message",
-
-    register(api: ExtensionApi): void {
-      api.pipeline.register(
-        "turn",
-        async (ctx, next) => {
-          ctx.conversation.emit({
-            type: "append",
-            message: {
-              id: `ctx-msg-${randomUUID()}`,
-              data: {
-                role: "system",
-                content: text,
-              },
-              metadata: {
-                __createdBy: "context-message",
-              },
-            },
-          });
-          return next();
-        },
-        { priority: 10 },
-      );
-    },
-  };
-}