@stfade/pi-read-delegator 1.0.8 → 1.0.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stfade/pi-read-delegator",
-  "version": "1.0.8",
+  "version": "1.0.10",
   "description": "Pi extension that delegates all read operations to a Reader subagent, blocking read tools from the main model",
   "main": "index.js",
   "types": "index.d.ts",

package/reader-manager.d.ts

@@ -21,18 +21,9 @@ export declare class ReaderError extends Error {
     readonly originalError?: unknown | undefined;
     constructor(message: string, originalError?: unknown | undefined);
 }
-/**
- * Verify that pi-subagents is installed as a Pi extension.
- *
- * - Uses `require.resolve('pi-subagents')` to check installation.
- * - If not installed, prompts the user to install via `pi install pi-subagents`.
- * - If the user declines or installation fails, returns false (caller disables the extension).
- *
- * @returns true if installed or successfully installed; false otherwise
- */
 export declare function checkDependencies(prompt: (message: string) => Promise<string>): Promise<boolean>;
 /**
- * Quick synchronous check: can we resolve pi-subagents?
+ * Quick synchronous check: does pi-subagents exist on disk?
  */
 export declare function isSubagentsInstalled(): boolean;
 /**

package/reader-manager.js

@@ -77,21 +77,24 @@ const READER_TEMPLATE_PATH = expandTilde("~/.pi/agent/agents/reader.md");
 /**
  * Verify that pi-subagents is installed as a Pi extension.
  *
- * - Uses `require.resolve('pi-subagents')` to check installation.
- * - If not installed, prompts the user to install via `pi install pi-subagents`.
+ * - Checks fs.existsSync for the pi-subagents directory (bypasses require.resolve
+ *   caching issues after npm install within the same process).
+ * - If not installed, prompts the user to install via `npm install --prefix`.
  * - If the user declines or installation fails, returns false (caller disables the extension).
  *
  * @returns true if installed or successfully installed; false otherwise
  */
+function piSubagentsDir() {
+    return path.join(os.homedir(), ".pi", "agent", "npm", "node_modules", "pi-subagents");
+}
+function piSubagentsPackageJson() {
+    return path.join(piSubagentsDir(), "package.json");
+}
 async function checkDependencies(prompt) {
     // Already installed — nothing to do
-    try {
-        require.resolve("pi-subagents");
+    if (fs.existsSync(piSubagentsPackageJson())) {
         return true;
     }
-    catch {
-        // MODULE_NOT_FOUND — prompt the user
-    }
     console.warn("[pi-read-delegator] ⚠️ pi-subagents is not installed.");
     const answer = await prompt("pi-subagents is not installed. Install it now? [Y/n]");
     const normalized = answer.trim().toLowerCase();
@@ -100,51 +103,35 @@ async function checkDependencies(prompt) {
         return false;
     }
     // Attempt installation
-    console.log("[pi-read-delegator] 📦 Installing pi-subagents…");
+    const piNpmDir = path.join(os.homedir(), ".pi", "agent", "npm");
+    console.log("[pi-read-delegator] 📦 Installing pi-subagents to " + piNpmDir + "…");
     try {
-        (0, child_process_1.execSync)("pi install npm:pi-subagents", {
+        (0, child_process_1.execSync)(`npm install --prefix "${piNpmDir}" pi-subagents`, {
             stdio: "pipe",
             timeout: 60_000,
             encoding: "utf-8",
         });
-        console.log("[pi-read-delegator] ✅ pi-subagents installed via 'pi install npm:pi-subagents'.");
+        console.log("[pi-read-delegator] ✅ pi-subagents installed via npm.");
     }
     catch (firstErr) {
-        // Fallback to global npm install
-        try {
-            (0, child_process_1.execSync)("npm install -g pi-subagents", {
-                stdio: "pipe",
-                timeout: 60_000,
-                encoding: "utf-8",
-            });
-            console.log("[pi-read-delegator] ✅ pi-subagents installed via npm.");
-        }
-        catch (secondErr) {
-            console.error("[pi-read-delegator] ❌ Cannot proceed without pi-subagents. Extension disabled.");
-            return false;
-        }
+        console.error("[pi-read-delegator] ⚠️ npm install failed:", firstErr instanceof Error ? firstErr.message : String(firstErr));
+        console.error("[pi-read-delegator] ❌ Cannot proceed without pi-subagents. Extension disabled.");
+        return false;
     }
     // Verify installation took effect
-    try {
-        require.resolve("pi-subagents");
+    if (fs.existsSync(piSubagentsPackageJson())) {
         return true;
     }
-    catch {
-        console.error("[pi-read-delegator] ❌ pi-subagents installed but cannot be found. Restart Pi and try again.");
-        return false;
-    }
+    console.error("[pi-read-delegator] ❌ pi-subagents installed but cannot be found at " +
+        piSubagentsDir() +
+        ". Restart Pi and try again.");
+    return false;
 }
 /**
- * Quick synchronous check: can we resolve pi-subagents?
+ * Quick synchronous check: does pi-subagents exist on disk?
  */
 function isSubagentsInstalled() {
-    try {
-        require.resolve("pi-subagents");
-        return true;
-    }
-    catch {
-        return false;
-    }
+    return fs.existsSync(piSubagentsPackageJson());
 }
 // ---------------------------------------------------------------------------
 // 2. Reader template