@goondan/openharness-base 0.1.1 → 0.1.3

package/dist/index.d.ts

@@ -1,12 +1,12 @@
 import { Extension, 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.
  *
  * 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
@@ -69,4 +69,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,8 +1,8 @@
-// src/extensions/context-message.ts
+// src/extensions/basic-system-prompt.ts
 import { randomUUID } from "crypto";
-function ContextMessage(text) {
+function BasicSystemPrompt(text) {
   return {
-    name: "context-message",
+    name: "basic-system-prompt",
     register(api) {
       api.pipeline.register(
         "turn",
@@ -10,13 +10,13 @@ function ContextMessage(text) {
           ctx.conversation.emit({
             type: "append",
             message: {
-              id: `ctx-msg-${randomUUID()}`,
+              id: `sys-${randomUUID()}`,
               data: {
                 role: "system",
                 content: text
               },
               metadata: {
-                __createdBy: "context-message"
+                __createdBy: "basic-system-prompt"
               }
             }
           });
@@ -490,8 +490,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.1",
+  "version": "0.1.3",
   "type": "module",
   "exports": {
     ".": {
@@ -9,7 +9,7 @@
     }
   },
   "dependencies": {
-    "@goondan/openharness-types": "0.1.1"
+    "@goondan/openharness-types": "0.1.3"
   },
   "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,
@@ -97,19 +97,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,49 +116,21 @@ 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);
-    ext.register(api);
-
-    const middleware = registeredMiddleware[0].handler;
-    const ctx = makeTurnContext(conversation);
-    const next = vi.fn(async () => stubTurnResult);
-
-    await middleware(ctx, next);
-
-    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");
-    }
-  });
-
-  // Test 5: middleware calls next() after emitting
-  it("middleware calls next() after emitting the system message", 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;
@@ -171,33 +141,15 @@ describe("ContextMessage", () => {
 
     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);
-
-    expect(conversation.emitted).toHaveLength(2);
-
-    const texts = conversation.emitted
-      .filter((e): e is Extract<typeof e, { type: "append" }> => e.type === "append")
-      .map((e) => e.message.data.content);
-
-    expect(texts).toContain("First context");
-    expect(texts).toContain("Second context");
+    // Verify system message was emitted
+    expect(conversation.emit).toHaveBeenCalledOnce();
+    const emittedEvent = conversation.emitted[0];
+    expect(emittedEvent.type).toBe("append");
+    if (emittedEvent.type === "append") {
+      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");
+    }
   });
 });

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

@@ -2,14 +2,14 @@ import type { Extension, ExtensionApi } from "@goondan/openharness-types";
 import { randomUUID } from "node:crypto";
 
 /**
- * ContextMessage extension — prepends a system message to the conversation
+ * BasicSystemPrompt 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 {
+export function BasicSystemPrompt(text: string): Extension {
   return {
-    name: "context-message",
+    name: "basic-system-prompt",
 
     register(api: ExtensionApi): void {
       api.pipeline.register(
@@ -18,13 +18,13 @@ export function ContextMessage(text: string): Extension {
           ctx.conversation.emit({
             type: "append",
             message: {
-              id: `ctx-msg-${randomUUID()}`,
+              id: `sys-${randomUUID()}`,
               data: {
                 role: "system",
                 content: text,
               },
               metadata: {
-                __createdBy: "context-message",
+                __createdBy: "basic-system-prompt",
               },
             },
           });

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"}