pi-goosedump 0.12.53 → 0.12.55

package/README.md

@@ -11,9 +11,12 @@ pi install npm:pi-goosedump
 
 Bundled native binaries: Linux x64 and arm64, macOS arm64, and Windows x64.
 
-Unsupported platforms are rejected when the extension loads. A missing packaged
-binary is treated as a broken installation instead of falling back to a
-`goosedump` executable on `PATH`. Local models download on first use.
+Unsupported platforms are rejected when the extension loads. The extension uses
+that packaged binary by default and does not fall back to a `goosedump`
+executable on `PATH`. Set the shared `GOOSEDUMP_BINARY` environment variable to
+an absolute executable path to override it. An invalid override or missing
+packaged binary is treated as a broken installation. Local models download on
+first use.
 
 ## Tools
 
@@ -99,6 +102,27 @@ The footer shows the remaining turn count in hexadecimal.
   compacts at that turn boundary, and resumes automatically.
 - Goosedump failures fall back to Pi's compactor.
 
+## Development
+
+Install dependencies and run the test suite:
+
+```sh
+npm install
+npm test
+npm run check
+```
+
+To exercise the extension against the binary built in this repository:
+
+```sh
+cargo build
+GOOSEDUMP_BINARY="$(cd ../.. && pwd)/target/debug/goosedump" \
+  pi --no-extensions -e ./index.ts
+```
+
+The explicit override takes precedence over the packaged platform binary. It must
+be an absolute path to an executable regular file.
+
 ## License
 
 Apache-2.0. See [LICENSE](LICENSE).

package/dist/index.ts

@@ -5,9 +5,9 @@
 import { Type as Type2 } from "@sinclair/typebox";
 import { Value as Value2 } from "@sinclair/typebox/value";
 import { execFile } from "node:child_process";
-import { existsSync, readFileSync } from "node:fs";
+import { accessSync, constants as fsConstants, existsSync, readFileSync as readFileSync2, statSync } from "node:fs";
 import { createRequire } from "node:module";
-import { dirname, join } from "node:path";
+import { dirname, isAbsolute, join } from "node:path";
 import { fileURLToPath } from "node:url";
 
 // src/format.ts
@@ -160,7 +160,7 @@ var GOOSE_PROVIDERS = [
 var SUMMARIZABLE_ENTRY_TYPES = new Set(["message", "custom_message", "branch_summary"]);
 
 // src/util.ts
-import { closeSync, openSync, readSync } from "node:fs";
+import { readFileSync } from "node:fs";
 function isRecord(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
@@ -181,21 +181,13 @@ function compactStartedMessage(keep, queued = false) {
   return keep !== null ? `${action}; keeping the last ${keep} user turn${keep === 1 ? "" : "s"}.` : `${action} up to Pi's token-based cut.`;
 }
 function readFirstLine(path) {
-  let fd;
   try {
-    fd = openSync(path, "r");
-    const buf = Buffer.alloc(8192);
-    const bytesRead = readSync(fd, buf, 0, buf.length, 0);
-    if (bytesRead < 0)
-      return null;
-    const slice = buf.subarray(0, bytesRead);
-    const newline = slice.indexOf(10);
-    return (newline < 0 ? slice : slice.subarray(0, newline)).toString("utf8");
+    const content = readFileSync(path, "utf8");
+    const newline = content.indexOf(`
+`);
+    return newline < 0 ? content : content.slice(0, newline);
   } catch {
     return null;
-  } finally {
-    if (fd !== undefined)
-      closeSync(fd);
   }
 }
 function formatTarget(provider, id) {
@@ -206,16 +198,17 @@ function appendScope(args, scope) {
     args.push("--all");
 }
 function utf8Prefix(text, maxBytes) {
-  const prefix = [];
   let bytes = 0;
+  let index = 0;
   for (const char of text) {
-    const charBytes = Buffer.byteLength(char);
+    const cp = char.codePointAt(0);
+    const charBytes = cp <= 127 ? 1 : cp <= 2047 ? 2 : cp <= 65535 ? 3 : 4;
     if (bytes + charBytes > maxBytes)
       break;
-    prefix.push(char);
     bytes += charBytes;
+    index += char.length;
   }
-  return prefix.join("");
+  return text.slice(0, index);
 }
 function isControlChar(char) {
   const code = char.charCodeAt(0);
@@ -417,7 +410,7 @@ function resolveNativePackageTable() {
   throw new Error("broken pi-goosedump installation: cannot resolve the native package table from @jarkkojs/goosedump");
 }
 function loadNativeGoosedumpPackages() {
-  const table = JSON.parse(readFileSync(resolveNativePackageTable(), "utf8"));
+  const table = JSON.parse(readFileSync2(resolveNativePackageTable(), "utf8"));
   if (!Value2.Check(NATIVE_GOOSEDUMP_PACKAGES_SCHEMA, table)) {
     throw new Error("broken pi-goosedump installation: native package table is invalid");
   }
@@ -442,7 +435,29 @@ function resolveGoosedumpBinary() {
     throw new Error("broken pi-goosedump installation: cannot resolve the @jarkkojs/goosedump dependency", { cause });
   }
 }
-function resolveNativeGoosedumpBinary() {
+var GOOSEDUMP_BINARY_ENV = "GOOSEDUMP_BINARY";
+function resolveGoosedumpBinaryOverride(environment = process.env) {
+  const binaryPath = environment[GOOSEDUMP_BINARY_ENV];
+  if (binaryPath === undefined)
+    return null;
+  if (!isAbsolute(binaryPath)) {
+    throw new Error(`${GOOSEDUMP_BINARY_ENV} must be an absolute path to the goosedump executable`);
+  }
+  try {
+    if (!statSync(binaryPath).isFile())
+      throw new Error("not a regular file");
+    accessSync(binaryPath, fsConstants.X_OK);
+  } catch (cause) {
+    throw new Error(`${GOOSEDUMP_BINARY_ENV} does not point to an executable file: ${binaryPath}`, {
+      cause
+    });
+  }
+  return binaryPath;
+}
+function resolveNativeGoosedumpBinary(environment = process.env) {
+  const override = resolveGoosedumpBinaryOverride(environment);
+  if (override)
+    return override;
   const target = `${process.platform}-${process.arch}`;
   const { packageName, binaryName } = nativeGoosedumpPackage(target);
   try {
@@ -610,7 +625,7 @@ import { randomUUID } from "node:crypto";
 import {
   existsSync as existsSync2,
   mkdirSync,
-  readFileSync as readFileSync2,
+  readFileSync as readFileSync3,
   renameSync,
   unlinkSync,
   writeFileSync
@@ -663,7 +678,7 @@ function readJsonObject(path) {
   if (!existsSync2(path))
     return {};
   try {
-    const parsed = JSON.parse(readFileSync2(path, "utf8"));
+    const parsed = JSON.parse(readFileSync3(path, "utf8"));
     return isRecord(parsed) ? parsed : {};
   } catch {
     return {};
@@ -674,7 +689,7 @@ function readJsonObjectStrict(path) {
     return {};
   let parsed;
   try {
-    parsed = JSON.parse(readFileSync2(path, "utf8"));
+    parsed = JSON.parse(readFileSync3(path, "utf8"));
   } catch (error) {
     const message = error instanceof Error ? error.message : String(error);
     throw new Error(`${path}: ${message}`);
@@ -2827,7 +2842,11 @@ function pi_goosedump_default(pi) {
   createGoosedumpIntegration().register(pi);
 }
 export {
+  resolveNativeGoosedumpBinary,
+  resolveGoosedumpBinaryOverride,
+  resolveGoosedumpBinary,
   parsePiMessages,
   pi_goosedump_default as default,
-  createGoosedumpIntegration
+  createGoosedumpIntegration,
+  GOOSEDUMP_BINARY_ENV
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-goosedump",
-  "version": "0.12.53",
+  "version": "0.12.55",
   "description": "Pi extension for goosedump session search, durable memory, session management, and compaction",
   "keywords": [
     "compaction",
@@ -38,7 +38,7 @@
   },
   "dependencies": {
     "@earendil-works/pi-tui": "^0.82.1",
-    "@jarkkojs/goosedump": "^0.12.53",
+    "@jarkkojs/goosedump": "^0.12.55",
     "@sinclair/typebox": "^0.34.49"
   },
   "devDependencies": {