@sting8k/pi-vcc 0.1.0 → 0.1.3

package/package.json

@@ -1,33 +1,23 @@
 {
   "name": "@sting8k/pi-vcc",
-  "version": "0.1.0",
-  "description": "Structured conversation compactor for pi - replaces default compact summary with an agent-optimized structured format.",
+  "version": "0.1.3",
+  "description": "Algorithmic conversation compactor for pi - transcript-preserving structured summaries, no LLM calls",
+  "main": "index.ts",
   "keywords": [
-    "pi-package"
+    "pi-package",
+    "pi-extension",
+    "vcc",
+    "compact",
+    "compaction"
   ],
-  "homepage": "https://github.com/sting8k/pi-vcc#readme",
-  "bugs": {
-    "url": "https://github.com/sting8k/pi-vcc/issues"
-  },
   "repository": {
     "type": "git",
     "url": "git+https://github.com/sting8k/pi-vcc.git"
   },
-  "license": "ISC",
-  "author": "sting8k",
-  "type": "commonjs",
-  "main": "index.ts",
-  "directories": {
-    "test": "tests"
-  },
-  "scripts": {
-    "test": "bun test",
-    "bench": "bun run ./scripts/benchmark-real-sessions.ts"
-  },
-  "dependencies": {
-    "@mariozechner/pi-coding-agent": "*"
+  "peerDependencies": {
+    "@mariozechner/pi-coding-agent": "*",
+    "@sinclair/typebox": "*"
   },
-  "devDependencies": {},
   "pi": {
     "extensions": [
       "./index.ts"

package/src/core/content.ts

@@ -10,6 +10,7 @@ export const firstLine = (text: string, max = 200): string =>
   clip(text.split("\n")[0] ?? "", max);
 
 export const textParts = (content: Message["content"]): string[] => {
+  if (!content) return [];
   if (typeof content === "string") return [content];
   return content
     .filter((part) => part.type === "text")

package/src/core/normalize.ts

@@ -8,7 +8,7 @@ const normalizeOne = (msg: Message): NormalizedBlock[] => {
     const blocks: NormalizedBlock[] = [];
     const text = sanitize(textOf(msg.content));
     if (text) blocks.push({ kind: "user", text });
-    if (typeof msg.content !== "string") {
+    if (msg.content && typeof msg.content !== "string") {
       for (const part of msg.content) {
         if (part.type === "image") {
           blocks.push({ kind: "user", text: `[image: ${part.mimeType}]` });
@@ -28,6 +28,7 @@ const normalizeOne = (msg: Message): NormalizedBlock[] => {
   }
 
   if (msg.role === "assistant") {
+    if (!msg.content) return [];
     if (typeof msg.content === "string") {
       return [{ kind: "assistant", text: sanitize(msg.content) }];
     }

package/src/core/render-entries.ts

@@ -11,7 +11,7 @@ export interface RenderedEntry {
 }
 
 const toolCalls = (content: Message["content"]): string => {
-  if (typeof content === "string") return "";
+  if (!content || typeof content === "string") return "";
   return content
     .filter((c) => c.type === "toolCall")
     .map((c) => `${c.name}(${summarizeToolArgs(c.arguments)})`)
@@ -19,7 +19,7 @@ const toolCalls = (content: Message["content"]): string => {
 };
 
 const extractFilesFromContent = (content: Message["content"]): string[] => {
-  if (typeof content === "string") return [];
+  if (!content || typeof content === "string") return [];
   return content
     .filter((c) => c.type === "toolCall")
     .map((c) => extractPath(c.arguments))
@@ -38,6 +38,13 @@ export const renderMessage = (msg: Message, index: number, full = false): Render
       summary: `${prefix}[${msg.toolName}] ${text}`,
     };
   }
+  // bashExecution has command+output instead of content
+  if ((msg as any).role === "bashExecution") {
+    const cmd = (msg as any).command ?? "";
+    const out = (msg as any).output ?? "";
+    const text = full ? `$ ${cmd}\n${out}` : clip(`$ ${cmd}\n${out}`, 300);
+    return { index, role: "bash", summary: text };
+  }
   const text = full ? textOf(msg.content) : clip(textOf(msg.content), 300);
   const tools = toolCalls(msg.content);
   const files = extractFilesFromContent(msg.content);

package/tests/content.test.ts

@@ -0,0 +1,31 @@
+import { describe, it, expect } from "bun:test";
+import { textParts, textOf, clip, firstLine } from "../src/core/content";
+
+describe("textParts", () => {
+  it("returns [] for undefined content", () => {
+    expect(textParts(undefined as any)).toEqual([]);
+  });
+
+  it("returns [] for null content", () => {
+    expect(textParts(null as any)).toEqual([]);
+  });
+
+  it("wraps string content", () => {
+    expect(textParts("hello")).toEqual(["hello"]);
+  });
+
+  it("extracts text parts from array content", () => {
+    const content = [
+      { type: "text" as const, text: "first" },
+      { type: "toolCall" as const, name: "x", id: "1", arguments: {} },
+      { type: "text" as const, text: "second" },
+    ];
+    expect(textParts(content)).toEqual(["first", "second"]);
+  });
+});
+
+describe("textOf", () => {
+  it("returns empty string for undefined content", () => {
+    expect(textOf(undefined as any)).toBe("");
+  });
+});

package/tests/render-entries.test.ts

@@ -36,5 +36,27 @@ describe("renderMessage", () => {
     const r = renderMessage(userMsg(long), 0);
     expect(r.summary.length).toBeLessThanOrEqual(300);
   });
+
+  it("renders bashExecution message", () => {
+    const msg = { role: "bashExecution", command: "ls -la", output: "total 0\n" } as any;
+    const r = renderMessage(msg, 5);
+    expect(r.role).toBe("bash");
+    expect(r.summary).toContain("$ ls -la");
+    expect(r.summary).toContain("total 0");
+  });
+
+  it("renders bashExecution with missing output", () => {
+    const msg = { role: "bashExecution", command: "exit 1" } as any;
+    const r = renderMessage(msg, 6);
+    expect(r.role).toBe("bash");
+    expect(r.summary).toContain("$ exit 1");
+  });
+
+  it("handles message with undefined content", () => {
+    const msg = { role: "assistant", content: undefined } as any;
+    const r = renderMessage(msg, 3);
+    expect(r.role).toBe("assistant");
+    expect(r.summary).toBe("");
+  });
 });