pi-goosedump 0.9.0 → 0.9.2

package/README.md

@@ -11,10 +11,9 @@ summaries.
 pi install npm:pi-goosedump
 ```
 
-This installs `pi-goosedump` and its `@jarkkojs/goosedump` dependency (`^0.9.0`),
+This installs `pi-goosedump` and its main dependency `@jarkkojs/goosedump`,
 which includes platform-specific native binaries for Linux, macOS, and Windows.
-If the goosedump binary cannot be resolved, the extension disables itself and
-the footer status goes dark.
+If the goosedump binary cannot be resolved, the extension disables itself.
 
 ## Tools
 
@@ -72,8 +71,8 @@ section and are optional (defaults shown):
 
 `turnsBeforeCompact` sets how many LLM turns elapse before goosedump triggers
 compaction, as an integer 0–255. `0` disables the counter; when enabled, the
-footer status shows `GD: XX` counting down in hex (e.g. `GD: 0A`), or just
-`GD` when disabled.
+footer status shows `GD: XX` counting down in hex (e.g. `GD: 0A`), or
+`GD: 00` when the counter is disabled.
 
 ### Compaction flow
 

package/index.ts

@@ -12,7 +12,7 @@ import type {
 
 import { defineTool, getSettingsListTheme } from '@earendil-works/pi-coding-agent';
 
-import { execFileSync } from 'node:child_process';
+import { execFile, execFileSync } from 'node:child_process';
 import {
   closeSync,
   existsSync,
@@ -262,6 +262,30 @@ function runGoosedump(args: string[]): string {
   }
 }
 
+// Non-blocking variant for the compact path: goosedump compact runs an ML
+// model and can take many seconds, so execFileSync would freeze Pi's event
+// loop (and its compaction spinner). execFile keeps the loop free to render.
+function runGoosedumpAsync(args: string[]): Promise<string> {
+  return new Promise((resolve, reject) => {
+    execFile(
+      'node',
+      [resolveGoosedumpBinary(), ...args],
+      {
+        encoding: 'utf-8',
+        maxBuffer: 16 * 1024 * 1024,
+      },
+      (err, stdout, stderr) => {
+        if (err) {
+          const message = stderr.trim() || err.message;
+          reject(new Error(message));
+        } else {
+          resolve(stdout);
+        }
+      },
+    );
+  });
+}
+
 // Read only the first line of a file without loading the whole transcript.
 function readFirstLine(path: string): string | null {
   const fd = openSync(path, 'r');
@@ -329,16 +353,16 @@ function resolveTarget(
   return null;
 }
 
-function goosedumpCompact(
+async function goosedumpCompact(
   target: string,
   options: { scope?: string; from?: string; until: string },
-): string {
+): Promise<string> {
   const scope = options.scope ?? 'lineage';
   const args = ['compact', target, '--scope', scope];
   if (options.from) args.push('--from', options.from);
   args.push('--until', options.until);
 
-  const output = runGoosedump(args);
+  const output = await runGoosedumpAsync(args);
   for (const line of output.split('\n')) {
     if (!line) continue;
     try {
@@ -475,7 +499,7 @@ function buildCompactRange(
     : { scope: 'lineage', until: firstKeptEntryId };
 }
 
-function buildGoosedumpCompaction(
+async function buildGoosedumpCompaction(
   target: string,
   preparation: {
     firstKeptEntryId: string;
@@ -485,7 +509,9 @@ function buildGoosedumpCompaction(
   },
   branchEntries: SessionEntry[],
   keep?: number,
-): CompactionResult<GoosedumpCompactionDetails> | { cancel: true; reason: string } | undefined {
+): Promise<
+  CompactionResult<GoosedumpCompactionDetails> | { cancel: true; reason: string } | undefined
+> {
   // A `keep:N` override replaces Pi's token-based cut with a turn-based one:
   // summarize everything before the Nth-from-last user entry and keep the tail.
   let firstKeptEntryId = preparation.firstKeptEntryId;
@@ -500,7 +526,7 @@ function buildGoosedumpCompaction(
   const range = buildCompactRange(branchEntries, firstKeptEntryId);
   if (!range) return undefined;
 
-  const summary = goosedumpCompact(target, range);
+  const summary = await goosedumpCompact(target, range);
   if (!summary) return undefined;
 
   return {
@@ -934,11 +960,12 @@ export function createGoosedumpIntegration(
 
   function updateStatus(ctx: ExtensionContext): void {
     if (!goosedumpAvailable) return;
-    const showCount = shouldOverrideDefaultCompaction && compactSettings.turnsBeforeCompact > 0;
-    const suffix = showCount
-      ? `: ${Math.max(0, turnsRemaining).toString(16).toUpperCase().padStart(2, '0')}`
-      : '';
-    ctx.ui.setStatus('goosedump', ctx.ui.theme.fg('text', `GD${suffix}`));
+    const active = shouldOverrideDefaultCompaction && compactSettings.turnsBeforeCompact > 0;
+    const count = active ? Math.max(0, turnsRemaining) : 0;
+    ctx.ui.setStatus(
+      'goosedump',
+      ctx.ui.theme.fg('text', `GD: ${count.toString(16).toUpperCase().padStart(2, '0')}`),
+    );
   }
 
   function noteCompletedTurn(ctx: ExtensionContext): void {
@@ -1312,7 +1339,7 @@ export function createGoosedumpIntegration(
 
       try {
         const keep = pendingKeep ?? undefined;
-        const compaction = buildGoosedumpCompaction(
+        const compaction = await buildGoosedumpCompaction(
           target,
           event.preparation,
           event.branchEntries,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-goosedump",
-  "version": "0.9.0",
+  "version": "0.9.2",
   "description": "Pi extension for goosedump-backed session-history search, entry expansion, and compaction",
   "keywords": [
     "compaction",
@@ -31,7 +31,7 @@
   },
   "dependencies": {
     "@earendil-works/pi-tui": "^0.78.0",
-    "@jarkkojs/goosedump": "^0.9.2",
+    "@jarkkojs/goosedump": "^0.9.3",
     "@sinclair/typebox": "^0.34.49"
   },
   "devDependencies": {