@goondan/openharness-base 0.1.7 → 0.1.9

package/dist/utils.d.ts

@@ -1,17 +0,0 @@
-import type { JsonArray, JsonObject, JsonValue, Message } from './types.js';
-export declare function createId(prefix: string): string;
-export declare function isRecord(value: unknown): value is Record<string, unknown>;
-export declare function isJsonValue(value: unknown): value is JsonValue;
-export declare function isJsonObject(value: unknown): value is JsonObject;
-export declare function isJsonArray(value: unknown): value is JsonArray;
-export declare function parseJsonObject(text: string): JsonObject | null;
-export declare function requireString(input: JsonObject, key: string): string;
-export declare function optionalString(input: JsonObject, key: string): string | undefined;
-export declare function optionalNumber(input: JsonObject, key: string, fallback?: number): number | undefined;
-export declare function optionalBoolean(input: JsonObject, key: string, fallback?: boolean): boolean | undefined;
-export declare function optionalStringArray(input: JsonObject, key: string): string[] | undefined;
-export declare function optionalJsonObject(input: JsonObject, key: string): JsonObject | undefined;
-export declare function resolveFromWorkdir(workdir: string, targetPath: string): string;
-export declare function estimateMessageLength(message: Message): number;
-export declare function truncateString(text: string, maxLength: number): string;
-//# sourceMappingURL=utils.d.ts.map

package/dist/utils.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../src/utils.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,SAAS,EAAE,UAAU,EAAE,SAAS,EAAE,OAAO,EAAE,MAAM,YAAY,CAAC;AAE5E,wBAAgB,QAAQ,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,CAE/C;AAED,wBAAgB,QAAQ,CAAC,KAAK,EAAE,OAAO,GAAG,KAAK,IAAI,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAEzE;AAED,wBAAgB,WAAW,CAAC,KAAK,EAAE,OAAO,GAAG,KAAK,IAAI,SAAS,CAyB9D;AAED,wBAAgB,YAAY,CAAC,KAAK,EAAE,OAAO,GAAG,KAAK,IAAI,UAAU,CAYhE;AAED,wBAAgB,WAAW,CAAC,KAAK,EAAE,OAAO,GAAG,KAAK,IAAI,SAAS,CAE9D;AAED,wBAAgB,eAAe,CAAC,IAAI,EAAE,MAAM,GAAG,UAAU,GAAG,IAAI,CAU/D;AAED,wBAAgB,aAAa,CAAC,KAAK,EAAE,UAAU,EAAE,GAAG,EAAE,MAAM,GAAG,MAAM,CAMpE;AAED,wBAAgB,cAAc,CAAC,KAAK,EAAE,UAAU,EAAE,GAAG,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CASjF;AAED,wBAAgB,cAAc,CAC5B,KAAK,EAAE,UAAU,EACjB,GAAG,EAAE,MAAM,EACX,QAAQ,CAAC,EAAE,MAAM,GAChB,MAAM,GAAG,SAAS,CASpB;AAED,wBAAgB,eAAe,CAC7B,KAAK,EAAE,UAAU,EACjB,GAAG,EAAE,MAAM,EACX,QAAQ,CAAC,EAAE,OAAO,GACjB,OAAO,GAAG,SAAS,CASrB;AAED,wBAAgB,mBAAmB,CAAC,KAAK,EAAE,UAAU,EAAE,GAAG,EAAE,MAAM,GAAG,MAAM,EAAE,GAAG,SAAS,CAiBxF;AAED,wBAAgB,kBAAkB,CAAC,KAAK,EAAE,UAAU,EAAE,GAAG,EAAE,MAAM,GAAG,UAAU,GAAG,SAAS,CASzF;AAED,wBAAgB,kBAAkB,CAAC,OAAO,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,MAAM,CAK9E;AAaD,wBAAgB,qBAAqB,CAAC,OAAO,EAAE,OAAO,GAAG,MAAM,CAgB9D;AAED,wBAAgB,cAAc,CAAC,IAAI,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,GAAG,MAAM,CAQtE"}

package/dist/utils.js

@@ -1,159 +0,0 @@
-import { randomUUID } from 'node:crypto';
-import { join } from 'node:path';
-export function createId(prefix) {
-    return `${prefix}_${randomUUID()}`;
-}
-export function isRecord(value) {
-    return typeof value === 'object' && value !== null && !Array.isArray(value);
-}
-export function isJsonValue(value) {
-    if (value === null) {
-        return true;
-    }
-    const kind = typeof value;
-    if (kind === 'string' || kind === 'number' || kind === 'boolean') {
-        return true;
-    }
-    if (Array.isArray(value)) {
-        return value.every((item) => isJsonValue(item));
-    }
-    if (!isRecord(value)) {
-        return false;
-    }
-    for (const candidate of Object.values(value)) {
-        if (!isJsonValue(candidate)) {
-            return false;
-        }
-    }
-    return true;
-}
-export function isJsonObject(value) {
-    if (!isRecord(value)) {
-        return false;
-    }
-    for (const candidate of Object.values(value)) {
-        if (!isJsonValue(candidate)) {
-            return false;
-        }
-    }
-    return true;
-}
-export function isJsonArray(value) {
-    return Array.isArray(value) && value.every((item) => isJsonValue(item));
-}
-export function parseJsonObject(text) {
-    try {
-        const parsed = JSON.parse(text);
-        if (isJsonObject(parsed)) {
-            return parsed;
-        }
-        return null;
-    }
-    catch {
-        return null;
-    }
-}
-export function requireString(input, key) {
-    const value = input[key];
-    if (typeof value !== 'string' || value.length === 0) {
-        throw new Error(`'${key}' must be a non-empty string`);
-    }
-    return value;
-}
-export function optionalString(input, key) {
-    const value = input[key];
-    if (value === undefined) {
-        return undefined;
-    }
-    if (typeof value !== 'string') {
-        throw new Error(`'${key}' must be a string`);
-    }
-    return value;
-}
-export function optionalNumber(input, key, fallback) {
-    const value = input[key];
-    if (value === undefined) {
-        return fallback;
-    }
-    if (typeof value !== 'number' || Number.isNaN(value)) {
-        throw new Error(`'${key}' must be a valid number`);
-    }
-    return value;
-}
-export function optionalBoolean(input, key, fallback) {
-    const value = input[key];
-    if (value === undefined) {
-        return fallback;
-    }
-    if (typeof value !== 'boolean') {
-        throw new Error(`'${key}' must be a boolean`);
-    }
-    return value;
-}
-export function optionalStringArray(input, key) {
-    const value = input[key];
-    if (value === undefined) {
-        return undefined;
-    }
-    if (!Array.isArray(value)) {
-        throw new Error(`'${key}' must be an array of strings`);
-    }
-    const output = [];
-    for (const item of value) {
-        if (typeof item !== 'string') {
-            throw new Error(`'${key}' must contain only strings`);
-        }
-        output.push(item);
-    }
-    return output;
-}
-export function optionalJsonObject(input, key) {
-    const value = input[key];
-    if (value === undefined) {
-        return undefined;
-    }
-    if (!isJsonObject(value)) {
-        throw new Error(`'${key}' must be a JSON object`);
-    }
-    return value;
-}
-export function resolveFromWorkdir(workdir, targetPath) {
-    if (targetPath.startsWith('/')) {
-        return targetPath;
-    }
-    return join(workdir, targetPath);
-}
-function isTextPart(part) {
-    return (typeof part === 'object' &&
-        part !== null &&
-        'type' in part &&
-        part.type === 'text' &&
-        'text' in part &&
-        typeof part.text === 'string');
-}
-export function estimateMessageLength(message) {
-    const content = message.data.content;
-    if (typeof content === 'string') {
-        return content.length;
-    }
-    if (!Array.isArray(content)) {
-        return 0;
-    }
-    let total = 0;
-    for (const part of content) {
-        if (isTextPart(part)) {
-            total += part.text.length;
-        }
-    }
-    return total;
-}
-export function truncateString(text, maxLength) {
-    if (maxLength < 1) {
-        return '';
-    }
-    if (text.length <= maxLength) {
-        return text;
-    }
-    return text.slice(0, maxLength);
-}
-//# sourceMappingURL=utils.js.map

package/dist/utils.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"utils.js","sourceRoot":"","sources":["../src/utils.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAC;AACzC,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AAGjC,MAAM,UAAU,QAAQ,CAAC,MAAc;IACrC,OAAO,GAAG,MAAM,IAAI,UAAU,EAAE,EAAE,CAAC;AACrC,CAAC;AAED,MAAM,UAAU,QAAQ,CAAC,KAAc;IACrC,OAAO,OAAO,KAAK,KAAK,QAAQ,IAAI,KAAK,KAAK,IAAI,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC;AAC9E,CAAC;AAED,MAAM,UAAU,WAAW,CAAC,KAAc;IACxC,IAAI,KAAK,KAAK,IAAI,EAAE,CAAC;QACnB,OAAO,IAAI,CAAC;IACd,CAAC;IAED,MAAM,IAAI,GAAG,OAAO,KAAK,CAAC;IAC1B,IAAI,IAAI,KAAK,QAAQ,IAAI,IAAI,KAAK,QAAQ,IAAI,IAAI,KAAK,SAAS,EAAE,CAAC;QACjE,OAAO,IAAI,CAAC;IACd,CAAC;IAED,IAAI,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,EAAE,CAAC;QACzB,OAAO,KAAK,CAAC,KAAK,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC,CAAC;IAClD,CAAC;IAED,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC;QACrB,OAAO,KAAK,CAAC;IACf,CAAC;IAED,KAAK,MAAM,SAAS,IAAI,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC;QAC7C,IAAI,CAAC,WAAW,CAAC,SAAS,CAAC,EAAE,CAAC;YAC5B,OAAO,KAAK,CAAC;QACf,CAAC;IACH,CAAC;IAED,OAAO,IAAI,CAAC;AACd,CAAC;AAED,MAAM,UAAU,YAAY,CAAC,KAAc;IACzC,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC;QACrB,OAAO,KAAK,CAAC;IACf,CAAC;IAED,KAAK,MAAM,SAAS,IAAI,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC;QAC7C,IAAI,CAAC,WAAW,CAAC,SAAS,CAAC,EAAE,CAAC;YAC5B,OAAO,KAAK,CAAC;QACf,CAAC;IACH,CAAC;IAED,OAAO,IAAI,CAAC;AACd,CAAC;AAED,MAAM,UAAU,WAAW,CAAC,KAAc;IACxC,OAAO,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,IAAI,KAAK,CAAC,KAAK,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC,CAAC;AAC1E,CAAC;AAED,MAAM,UAAU,eAAe,CAAC,IAAY;IAC1C,IAAI,CAAC;QACH,MAAM,MAAM,GAAY,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;QACzC,IAAI,YAAY,CAAC,MAAM,CAAC,EAAE,CAAC;YACzB,OAAO,MAAM,CAAC;QAChB,CAAC;QACD,OAAO,IAAI,CAAC;IACd,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAC;IACd,CAAC;AACH,CAAC;AAED,MAAM,UAAU,aAAa,CAAC,KAAiB,EAAE,GAAW;IAC1D,MAAM,KAAK,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC;IACzB,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACpD,MAAM,IAAI,KAAK,CAAC,IAAI,GAAG,8BAA8B,CAAC,CAAC;IACzD,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,MAAM,UAAU,cAAc,CAAC,KAAiB,EAAE,GAAW;IAC3D,MAAM,KAAK,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC;IACzB,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;QACxB,OAAO,SAAS,CAAC;IACnB,CAAC;IACD,IAAI,OAAO,KAAK,KAAK,QAAQ,EAAE,CAAC;QAC9B,MAAM,IAAI,KAAK,CAAC,IAAI,GAAG,oBAAoB,CAAC,CAAC;IAC/C,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,MAAM,UAAU,cAAc,CAC5B,KAAiB,EACjB,GAAW,EACX,QAAiB;IAEjB,MAAM,KAAK,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC;IACzB,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;QACxB,OAAO,QAAQ,CAAC;IAClB,CAAC;IACD,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,EAAE,CAAC;QACrD,MAAM,IAAI,KAAK,CAAC,IAAI,GAAG,0BAA0B,CAAC,CAAC;IACrD,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,MAAM,UAAU,eAAe,CAC7B,KAAiB,EACjB,GAAW,EACX,QAAkB;IAElB,MAAM,KAAK,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC;IACzB,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;QACxB,OAAO,QAAQ,CAAC;IAClB,CAAC;IACD,IAAI,OAAO,KAAK,KAAK,SAAS,EAAE,CAAC;QAC/B,MAAM,IAAI,KAAK,CAAC,IAAI,GAAG,qBAAqB,CAAC,CAAC;IAChD,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,MAAM,UAAU,mBAAmB,CAAC,KAAiB,EAAE,GAAW;IAChE,MAAM,KAAK,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC;IACzB,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;QACxB,OAAO,SAAS,CAAC;IACnB,CAAC;IACD,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,EAAE,CAAC;QAC1B,MAAM,IAAI,KAAK,CAAC,IAAI,GAAG,+BAA+B,CAAC,CAAC;IAC1D,CAAC;IAED,MAAM,MAAM,GAAa,EAAE,CAAC;IAC5B,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,OAAO,IAAI,KAAK,QAAQ,EAAE,CAAC;YAC7B,MAAM,IAAI,KAAK,CAAC,IAAI,GAAG,6BAA6B,CAAC,CAAC;QACxD,CAAC;QACD,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACpB,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC;AAED,MAAM,UAAU,kBAAkB,CAAC,KAAiB,EAAE,GAAW;IAC/D,MAAM,KAAK,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC;IACzB,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;QACxB,OAAO,SAAS,CAAC;IACnB,CAAC;IACD,IAAI,CAAC,YAAY,CAAC,KAAK,CAAC,EAAE,CAAC;QACzB,MAAM,IAAI,KAAK,CAAC,IAAI,GAAG,yBAAyB,CAAC,CAAC;IACpD,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,MAAM,UAAU,kBAAkB,CAAC,OAAe,EAAE,UAAkB;IACpE,IAAI,UAAU,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;QAC/B,OAAO,UAAU,CAAC;IACpB,CAAC;IACD,OAAO,IAAI,CAAC,OAAO,EAAE,UAAU,CAAC,CAAC;AACnC,CAAC;AAED,SAAS,UAAU,CAAC,IAAa;IAC/B,OAAO,CACL,OAAO,IAAI,KAAK,QAAQ;QACxB,IAAI,KAAK,IAAI;QACb,MAAM,IAAI,IAAI;QACd,IAAI,CAAC,IAAI,KAAK,MAAM;QACpB,MAAM,IAAI,IAAI;QACd,OAAO,IAAI,CAAC,IAAI,KAAK,QAAQ,CAC9B,CAAC;AACJ,CAAC;AAED,MAAM,UAAU,qBAAqB,CAAC,OAAgB;IACpD,MAAM,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC,OAAO,CAAC;IACrC,IAAI,OAAO,OAAO,KAAK,QAAQ,EAAE,CAAC;QAChC,OAAO,OAAO,CAAC,MAAM,CAAC;IACxB,CAAC;IACD,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC;QAC5B,OAAO,CAAC,CAAC;IACX,CAAC;IAED,IAAI,KAAK,GAAG,CAAC,CAAC;IACd,KAAK,MAAM,IAAI,IAAI,OAAO,EAAE,CAAC;QAC3B,IAAI,UAAU,CAAC,IAAI,CAAC,EAAE,CAAC;YACrB,KAAK,IAAI,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC;QAC5B,CAAC;IACH,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,MAAM,UAAU,cAAc,CAAC,IAAY,EAAE,SAAiB;IAC5D,IAAI,SAAS,GAAG,CAAC,EAAE,CAAC;QAClB,OAAO,EAAE,CAAC;IACZ,CAAC;IACD,IAAI,IAAI,CAAC,MAAM,IAAI,SAAS,EAAE,CAAC;QAC7B,OAAO,IAAI,CAAC;IACd,CAAC;IACD,OAAO,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,SAAS,CAAC,CAAC;AAClC,CAAC"}

package/src/__tests__/basic-system-prompt.test.ts

@@ -1,186 +0,0 @@
-import { describe, it, expect, vi } from "vitest";
-import { BasicSystemPrompt } from "../extensions/basic-system-prompt.js";
-import type {
-  ExtensionApi,
-  TurnMiddleware,
-  TurnContext,
-  TurnResult,
-  ConversationState,
-  Message,
-  MessageEvent,
-} from "@goondan/openharness-types";
-
-// ---------------------------------------------------------------------------
-// Helpers
-// ---------------------------------------------------------------------------
-
-function makeMockConversationState(): ConversationState & { emitted: MessageEvent[] } {
-  const emitted: MessageEvent[] = [];
-  const messages: Message[] = [];
-  return {
-    messages,
-    events: [],
-    emitted,
-    emit: vi.fn((event: MessageEvent) => {
-      emitted.push(event);
-      if (event.type === "append") {
-        messages.push(event.message);
-      }
-    }),
-    restore: vi.fn(),
-  };
-}
-
-function makeMockApi(conversation: ConversationState): {
-  api: ExtensionApi;
-  registeredMiddleware: Array<{ level: string; handler: TurnMiddleware; options?: { priority?: number } }>;
-} {
-  const registeredMiddleware: Array<{ level: string; handler: TurnMiddleware; options?: { priority?: number } }> = [];
-
-  const api: ExtensionApi = {
-    pipeline: {
-      register: vi.fn((level: string, handler: TurnMiddleware, options?: { priority?: number }) => {
-        registeredMiddleware.push({ level, handler, options });
-      }) as unknown as ExtensionApi["pipeline"]["register"],
-    },
-    tools: {
-      register: vi.fn(),
-      remove: vi.fn(),
-      list: vi.fn(() => []),
-    },
-    on: vi.fn(),
-    conversation,
-    runtime: {
-      agent: {
-        name: "test-agent",
-        model: { provider: "openai", model: "gpt-4o" },
-        extensions: [],
-        tools: [],
-      },
-      agents: {},
-      connections: {},
-    },
-  };
-
-  return { api, registeredMiddleware };
-}
-
-function makeTurnContext(conversation: ConversationState): TurnContext {
-  return {
-    turnId: "turn-1",
-    agentName: "test-agent",
-    conversationId: "conv-1",
-    conversation,
-    abortSignal: new AbortController().signal,
-    input: {
-      name: "test-event",
-      content: [{ type: "text", text: "hello" }],
-      properties: {},
-      source: {
-        connector: "test-connector",
-        connectionName: "test",
-        receivedAt: new Date().toISOString(),
-      },
-    },
-    llm: { chat: vi.fn().mockResolvedValue({ text: "mock" }) },
-  };
-}
-
-const stubTurnResult: TurnResult = {
-  turnId: "turn-1",
-  agentName: "test-agent",
-  conversationId: "conv-1",
-  status: "completed",
-  steps: [],
-};
-
-// ---------------------------------------------------------------------------
-// Tests
-// ---------------------------------------------------------------------------
-
-describe("BasicSystemPrompt", () => {
-  it("creates an Extension with name 'basic-system-prompt'", () => {
-    const ext = BasicSystemPrompt("You are helpful.");
-    expect(ext.name).toBe("basic-system-prompt");
-  });
-
-  it("register() calls api.pipeline.register with level 'turn'", () => {
-    const conversation = makeMockConversationState();
-    const { api, registeredMiddleware } = makeMockApi(conversation);
-
-    const ext = BasicSystemPrompt("You are helpful.");
-    ext.register(api);
-
-    expect(api.pipeline.register).toHaveBeenCalledOnce();
-    expect(registeredMiddleware).toHaveLength(1);
-    expect(registeredMiddleware[0].level).toBe("turn");
-    expect(typeof registeredMiddleware[0].handler).toBe("function");
-  });
-
-  it("registers turn middleware with priority 10 (high priority)", () => {
-    const conversation = makeMockConversationState();
-    const { api, registeredMiddleware } = makeMockApi(conversation);
-
-    const ext = BasicSystemPrompt("You are helpful.");
-    ext.register(api);
-
-    expect(registeredMiddleware[0].options?.priority).toBe(10);
-  });
-
-  it("middleware appends a system message and calls next()", async () => {
-    const conversation = makeMockConversationState();
-    const { api, registeredMiddleware } = makeMockApi(conversation);
-
-    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);
-
-    // 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.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");
-    }
-  });
-
-  it("does not duplicate the system message on subsequent turns", async () => {
-    const conversation = makeMockConversationState();
-    const { api, registeredMiddleware } = makeMockApi(conversation);
-
-    const ext = BasicSystemPrompt("You are helpful.");
-    ext.register(api);
-
-    const middleware = registeredMiddleware[0].handler;
-    const next = vi.fn(async () => stubTurnResult);
-
-    // First turn — should append
-    const ctx1 = makeTurnContext(conversation);
-    await middleware(ctx1, next);
-    expect(conversation.emitted).toHaveLength(1);
-
-    // 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
-
-    // Third turn — still no duplication
-    const ctx3 = makeTurnContext(conversation);
-    await middleware(ctx3, next);
-    expect(conversation.emitted).toHaveLength(1);
-
-    // next() should have been called every turn
-    expect(next).toHaveBeenCalledTimes(3);
-  });
-});

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

@@ -1,282 +0,0 @@
-import { describe, it, expect, vi } from "vitest";
-import { CompactionSummarize } from "../extensions/compaction-summarize.js";
-import type {
-  ExtensionApi,
-  StepMiddleware,
-  StepContext,
-  StepResult,
-  ConversationState,
-  Message,
-  MessageEvent,
-} from "@goondan/openharness-types";
-
-// ---------------------------------------------------------------------------
-// Helpers
-// ---------------------------------------------------------------------------
-
-function makeMessages(count: number): Message[] {
-  return Array.from({ length: count }, (_, i) => ({
-    id: `msg-${i}`,
-    data: {
-      role: "user" as const,
-      content: `Message ${i}`,
-    },
-  }));
-}
-
-function makeMockConversationState(
-  messages: Message[],
-): ConversationState & { emitted: MessageEvent[] } {
-  const emitted: MessageEvent[] = [];
-  return {
-    messages,
-    events: [],
-    emitted,
-    emit: vi.fn((event: MessageEvent) => {
-      emitted.push(event);
-    }),
-    restore: vi.fn(),
-  };
-}
-
-function makeMockApi(conversation: ConversationState): {
-  api: ExtensionApi;
-  registeredMiddleware: Array<{
-    level: string;
-    handler: StepMiddleware;
-    options?: { priority?: number };
-  }>;
-} {
-  const registeredMiddleware: Array<{
-    level: string;
-    handler: StepMiddleware;
-    options?: { priority?: number };
-  }> = [];
-
-  const api: ExtensionApi = {
-    pipeline: {
-      register: vi.fn(
-        (level: string, handler: StepMiddleware, options?: { priority?: number }) => {
-          registeredMiddleware.push({ level, handler, options });
-        },
-      ) as unknown as ExtensionApi["pipeline"]["register"],
-    },
-    tools: {
-      register: vi.fn(),
-      remove: vi.fn(),
-      list: vi.fn(() => []),
-    },
-    on: vi.fn(),
-    conversation,
-    runtime: {
-      agent: {
-        name: "test-agent",
-        model: { provider: "openai", model: "gpt-4o" },
-        extensions: [],
-        tools: [],
-      },
-      agents: {},
-      connections: {},
-    },
-  };
-
-  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",
-    agentName: "test-agent",
-    conversationId: "conv-1",
-    conversation,
-    stepNumber: 1,
-    abortSignal: new AbortController().signal,
-    input: {
-      name: "test-event",
-      content: [{ type: "text", text: "hello" }],
-      properties: {},
-      source: {
-        connector: "test-connector",
-        connectionName: "test",
-        receivedAt: new Date().toISOString(),
-      },
-    },
-    llm: makeMockLlmClient(),
-  };
-}
-
-const stubStepResult: StepResult = {
-  toolCalls: [],
-};
-
-// ---------------------------------------------------------------------------
-// Tests
-// ---------------------------------------------------------------------------
-
-describe("CompactionSummarize", () => {
-  it("creates an Extension with name 'compaction-summarize'", () => {
-    const ext = CompactionSummarize({ threshold: 10 });
-    expect(ext.name).toBe("compaction-summarize");
-  });
-
-  it("registers step middleware via api.pipeline.register", () => {
-    const conversation = makeMockConversationState([]);
-    const { api, registeredMiddleware } = makeMockApi(conversation);
-
-    const ext = CompactionSummarize({ threshold: 10 });
-    ext.register(api);
-
-    expect(api.pipeline.register).toHaveBeenCalledOnce();
-    expect(registeredMiddleware[0].level).toBe("step");
-  });
-
-  it("does NOT compact when messages are below threshold", async () => {
-    const messages = makeMessages(5);
-    const conversation = makeMockConversationState(messages);
-    const { api, registeredMiddleware } = makeMockApi(conversation);
-
-    const ext = CompactionSummarize({ threshold: 10 });
-    ext.register(api);
-
-    const middleware = registeredMiddleware[0].handler;
-    const ctx = makeStepContext(conversation);
-    const next = vi.fn(async () => stubStepResult);
-
-    await middleware(ctx, next);
-
-    expect(conversation.emit).not.toHaveBeenCalled();
-    expect(next).toHaveBeenCalledOnce();
-  });
-
-  it("compacts when messages exceed threshold", async () => {
-    const messages = makeMessages(12);
-    const conversation = makeMockConversationState(messages);
-    const { api, registeredMiddleware } = makeMockApi(conversation);
-
-    const ext = CompactionSummarize({ threshold: 10 });
-    ext.register(api);
-
-    const middleware = registeredMiddleware[0].handler;
-    const ctx = makeStepContext(conversation);
-    const next = vi.fn(async () => stubStepResult);
-
-    await middleware(ctx, next);
-
-    const emitted = (conversation as ReturnType<typeof makeMockConversationState>).emitted;
-
-    // keepCount = floor(10/2) = 5, removeCount = 12 - 5 = 7
-    // Should have 7 remove events + 1 append event
-    const removeEvents = emitted.filter((e) => e.type === "remove");
-    const appendEvents = emitted.filter((e) => e.type === "append");
-
-    expect(removeEvents).toHaveLength(7);
-    expect(appendEvents).toHaveLength(1);
-
-    // The appended message should be a system summary
-    const appendEvent = appendEvents[0];
-    if (appendEvent.type === "append") {
-      expect(appendEvent.message.data.role).toBe("system");
-      expect(appendEvent.message.data.content).toContain("[Summary of earlier conversation]:");
-      expect(appendEvent.message.metadata?.__createdBy).toBe("compaction-summarize");
-    }
-
-    expect(next).toHaveBeenCalledOnce();
-  });
-
-  it("removes the oldest messages (not the newest)", async () => {
-    const messages = makeMessages(12);
-    const conversation = makeMockConversationState(messages);
-    const { api, registeredMiddleware } = makeMockApi(conversation);
-
-    const ext = CompactionSummarize({ threshold: 10 });
-    ext.register(api);
-
-    const middleware = registeredMiddleware[0].handler;
-    const ctx = makeStepContext(conversation);
-    const next = vi.fn(async () => stubStepResult);
-
-    await middleware(ctx, next);
-
-    const emitted = (conversation as ReturnType<typeof makeMockConversationState>).emitted;
-    const removeEvents = emitted.filter(
-      (e): e is Extract<typeof e, { type: "remove" }> => e.type === "remove",
-    );
-
-    // Should remove msg-0 through msg-6 (the oldest 7)
-    const removedIds = removeEvents.map((e) => e.messageId);
-    expect(removedIds).toContain("msg-0");
-    expect(removedIds).toContain("msg-6");
-    expect(removedIds).not.toContain("msg-7");
-    expect(removedIds).not.toContain("msg-11");
-  });
-
-  it("summary content includes text from removed messages", async () => {
-    const messages = makeMessages(12);
-    const conversation = makeMockConversationState(messages);
-    const { api, registeredMiddleware } = makeMockApi(conversation);
-
-    const ext = CompactionSummarize({ threshold: 10 });
-    ext.register(api);
-
-    const middleware = registeredMiddleware[0].handler;
-    const ctx = makeStepContext(conversation);
-    const next = vi.fn(async () => stubStepResult);
-
-    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 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();
-  });
-});