harnesstrim 0.0.5 → 0.0.7

package/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2026 HarnessTrim contributors
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/assets/adapter-hermes/plugin/__init__.py

@@ -225,14 +225,19 @@ def _write_metric(tool: str, reducer: str | None, before: int, after: int) -> No
     directory lazily and swallows any error — telemetry must never crash the plugin.
     """
     import datetime as _dt
+    import uuid as _uuid
 
     event = {
+        "schemaVersion": 1,
+        "eventId": str(_uuid.uuid4()),
         "ts": _dt.datetime.now(_dt.timezone.utc).isoformat(),
         "harness": "hermes",
         "tool": tool,
         "reducer": reducer,
         "beforeChars": before,
         "afterChars": after,
+        "beforeTokens": None,
+        "afterTokens": None,
     }
     try:
         METRICS_PATH.parent.mkdir(parents=True, exist_ok=True)

package/assets/adapter-pi/extension/{harnesstrim.ts → index.ts}

@@ -2,7 +2,7 @@
 //
 // Pi fires `tool_result` after a tool finishes and before the result reaches the model;
 // handlers chain like middleware and may return a patch ({ content, details, isError }).
-// This extension reduces string `content` for noisy output (test runners, git diffs, ...)
+// This extension reduces text chunks in structured tool results (test runners, git diffs, ...)
 // by shelling out to `harnesstrim reduce`, so it is self-contained (no workspace imports)
 // and loads from `~/.pi/agent/extensions/` or `<project>/.pi/extensions/`.
 //
@@ -12,19 +12,28 @@
 //   HARNESSTRIM_MINLENGTH=<chars>        (default 400)
 import { spawnSync } from "node:child_process";
 
+type TextContent = { type: "text"; text: string };
+type ToolContent = TextContent | { type: string; [key: string]: unknown };
+
 interface ToolResultEvent {
-  content?: unknown;
+  content?: ToolContent[];
   isError?: boolean;
 }
 interface ExtensionAPI {
   on(event: string, handler: (event: ToolResultEvent, ctx: unknown) => unknown): void;
 }
 
-const env = globalThis.process?.env ?? {};
+const runtime = globalThis as typeof globalThis & { process?: NodeJS.Process };
+const env = runtime.process?.env ?? {};
 const MODE = env.HARNESSTRIM_MODE ?? "dryrun";
 const MIN_LENGTH = Number(env.HARNESSTRIM_MINLENGTH ?? "400") || 400;
 const MARKER = "[harnesstrim";
 
+/** True when a text chunk should not be reduced: too short, or already reduced. */
+export function shouldSkip(text: string, minLength: number): boolean {
+  return text.length < minLength || text.includes(MARKER);
+}
+
 function reduceViaCli(text: string): string | null {
   try {
     const r = spawnSync("harnesstrim", ["reduce", "--min-length", String(MIN_LENGTH)], {
@@ -44,19 +53,28 @@ function reduceViaCli(text: string): string | null {
 export default function harnesstrim(pi: ExtensionAPI): void {
   if (MODE === "off") return;
   pi.on("tool_result", async (event) => {
-    const content = event.content;
-    if (typeof content !== "string" || content.length < MIN_LENGTH) return;
-    if (content.includes(MARKER)) return; // already reduced — avoid double work
-
-    const reduced = reduceViaCli(content);
-    if (!reduced || reduced.length >= content.length) return;
-
-    if (MODE === "dryrun") {
-      globalThis.process?.stderr?.write(
-        `[harnesstrim] dryrun tool_result: ${content.length} -> ${reduced.length} chars\n`
-      );
-      return; // dryrun: observe only
-    }
-    return { content: reduced }; // active: patch the tool result the model sees
+    if (!Array.isArray(event.content)) return;
+
+    let changed = false;
+    const content = event.content.map((chunk) => {
+      if (chunk.type !== "text" || typeof chunk.text !== "string") return chunk;
+      const text = chunk.text;
+      if (shouldSkip(text, MIN_LENGTH)) return chunk;
+
+      const reduced = reduceViaCli(text);
+      if (!reduced || reduced.length >= text.length) return chunk;
+
+      if (MODE === "dryrun") {
+        runtime.process?.stderr?.write(
+          `[harnesstrim] dryrun tool_result: ${text.length} -> ${reduced.length} chars\n`
+        );
+        return chunk;
+      }
+
+      changed = true;
+      return { ...chunk, text: reduced };
+    });
+
+    return changed ? { content } : undefined;
   });
 }