pi-lazy-extensions 0.1.2 → 0.1.4

package/config.ts

@@ -8,6 +8,7 @@
 import { existsSync, readFileSync } from "node:fs";
 import { homedir } from "node:os";
 import { dirname, isAbsolute, join, resolve } from "node:path";
+import { CONFIG_DIR_NAME } from "@earendil-works/pi-coding-agent";
 import type { LazyExtensionConfig, LazyExtensionsManifest, LazyExtensionsState, LoadedExtensionState } from "./types.js";
 
 /**
@@ -26,8 +27,9 @@ export function loadManifest(cwd: string, agentDir: string): { manifest: LazyExt
     if (manifest) return { manifest, path: resolve(envPath) };
   }
 
-  // Project .pi dir
-  const projectPi = join(cwd, ".pi", "lazy-extensions.json");
+  // Project "dot-pi" dir (uses CONFIG_DIR_NAME so rebranded pi distributions
+  // use their configured config dir name instead of hardcoded .pi)
+  const projectPi = join(cwd, CONFIG_DIR_NAME, "lazy-extensions.json");
   if (existsSync(projectPi)) {
     const manifest = parseManifest(projectPi);
     if (manifest) return { manifest, path: projectPi };

package/index.ts

@@ -27,24 +27,48 @@ export default function lazyExtensions(pi: ExtensionAPI) {
   let initPromise: Promise<void> | null = null;
   let lifecycleGeneration = 0;
 
-  const getPiTools = (): ToolInfo[] => pi.getAllTools();
+  // pi >= 0.81.1 throws from action methods while the extension runtime is
+  // still uninitialized during extension loading, so degrade to an empty list.
+  const getPiTools = (): ToolInfo[] => {
+    try {
+      return pi.getAllTools();
+    } catch {
+      return [];
+    }
+  };
   const agentDir = getAgentDir();
 
   // Check for tool name collision before registering the proxy tool.
   // Pi uses "first registration wins" — if another extension already
   // registered a tool named "ext", our proxy would be silently skipped.
-  const existingTools = pi.getAllTools();
-  if (existingTools.some(t => t.name === "ext")) {
-    console.error(
-      "pi-lazy-extensions: another extension already registered a tool named 'ext'. " +
-      "The proxy tool will not be available. Use the /ext command instead, " +
-      "or remove the conflicting extension."
-    );
-  }
+  // The runtime may not be bound yet at factory time (pi >= 0.81.1), in which
+  // case the check is retried on session_start once the runtime is ready.
+  let collisionChecked = false;
+  const checkExtCollision = () => {
+    if (collisionChecked) return;
+    let existingTools: ToolInfo[];
+    try {
+      existingTools = pi.getAllTools();
+    } catch {
+      return;
+    }
+    collisionChecked = true;
+    if (existingTools.some(t => t.name === "ext")) {
+      console.error(
+        "pi-lazy-extensions: another extension already registered a tool named 'ext'. " +
+        "The proxy tool will not be available. Use the /ext command instead, " +
+        "or remove the conflicting extension."
+      );
+    }
+  };
+  checkExtCollision();
 
   pi.on("session_start", async (_event, ctx) => {
     const generation = ++lifecycleGeneration;
 
+    // Retry the collision check now that the extension runtime is bound.
+    checkExtCollision();
+
     // Clear previous state
     if (state) {
       clearAllTimers(state);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-lazy-extensions",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "description": "Lazy-load pi extensions on demand via a ToolSearch-style proxy tool",
   "type": "module",
   "license": "MIT",
@@ -34,6 +34,13 @@
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": ">=0.74.0"
   },
+  "scripts": {
+    "test": "vitest run",
+    "test:watch": "vitest",
+    "test:coverage": "vitest run --coverage",
+    "typecheck": "tsc --noEmit",
+    "lint:dead": "knip --no-gitignore"
+  },
   "devDependencies": {
     "@commitlint/cli": "21.0.1",
     "@commitlint/config-conventional": "21.0.1",
@@ -49,11 +56,5 @@
     "protobufjs": "8.4.0",
     "ws": "8.20.1"
   },
-  "scripts": {
-    "test": "vitest run",
-    "test:watch": "vitest",
-    "test:coverage": "vitest run --coverage",
-    "typecheck": "tsc --noEmit",
-    "lint:dead": "knip --no-gitignore"
-  }
-}
+  "packageManager": "pnpm@11.20.0"
+}