crewx-pi-kit 0.1.12 → 0.1.13

package/README.md

@@ -10,7 +10,7 @@ work, safety boundaries, and agent handoffs.
 Install it with Pi:
 
 ```sh
-pi install npm:crewx-pi-kit@0.1.12
+pi install npm:crewx-pi-kit@0.1.13
 ```
 
 CrewX injects short-lived `CREWX_URL` and `CREWX_TOKEN` capabilities only while

package/extensions/crewx-tools.ts

@@ -151,8 +151,7 @@ function wait(milliseconds: number, signal?: AbortSignal): Promise<void> {
 export default function crewxTools(pi: ExtensionAPI) {
   const protectedValues = new Set<string>();
   const deliveredSecretRequests = new Set<string>();
-  const computerReleases = new Map<string, () => void>();
-  let computerTail = Promise.resolve();
+  const computerCalls = new Set<string>();
   let computerLease: string | undefined;
   let computerTimer: ReturnType<typeof setTimeout> | undefined;
   let computerOperations = Promise.resolve();
@@ -179,8 +178,8 @@ export default function crewxTools(pi: ExtensionAPI) {
     if (computerTimer) clearTimeout(computerTimer);
     computerTimer = setTimeout(async () => {
       if (!computerLease || computerClosed) return;
-      try { await updateComputer(computerReleases.size > 0); renewComputer(); }
-      catch { computerLease = undefined; if (computerReleases.size > 0) abortComputerAction?.(); }
+      try { await updateComputer(computerCalls.size > 0); renewComputer(); }
+      catch { computerLease = undefined; if (computerCalls.size > 0) abortComputerAction?.(); }
     }, 20_000);
     computerTimer.unref();
   };
@@ -188,8 +187,7 @@ export default function crewxTools(pi: ExtensionAPI) {
     computerClosed = true;
     if (computerTimer) clearTimeout(computerTimer);
     if (computerLease) await updateComputer(false, true).catch(() => {});
-    for (const release of computerReleases.values()) release();
-    computerReleases.clear();
+    computerCalls.clear();
   };
   pi.on('agent_start', () => { computerClosed = false; });
   pi.on('agent_end', closeComputer);
@@ -226,26 +224,30 @@ export default function crewxTools(pi: ExtensionAPI) {
     }
 
     if (!isComputerTool(event.toolName, event.input)) return;
-    const previous = computerTail;
-    let release!: () => void;
-    computerTail = new Promise<void>((resolve) => { release = resolve; });
-    await previous;
+    // Pi prepares every parallel call before executing any of them. Waiting
+    // here for an earlier result deadlocks the whole batch. Admit one computer
+    // action and return a retryable tool error for the others instead.
+    if (computerCalls.size > 0) {
+      return { block: true, terminate: false, reason: 'Another computer action is already prepared in this batch. Wait for its result, then retry this action in the next turn. Computer actions must be sequential.' };
+    }
+    computerCalls.add(event.toolCallId);
     try { await updateComputer(true); }
     catch (error) {
-      computerLease = undefined; release();
+      computerLease = undefined; computerCalls.delete(event.toolCallId);
       return { block: true, terminate: false, reason: error instanceof Error ? error.message : 'Computer ownership is unavailable. No browser action was performed.' };
     }
     abortComputerAction = () => ctx.abort();
-    computerReleases.set(event.toolCallId, release);
     renewComputer();
   });
-  pi.on("tool_result", async (event) => {
-    const release = computerReleases.get(event.toolCallId);
-    if (release) {
-      computerReleases.delete(event.toolCallId);
+  const finishComputerCall = async (toolCallId: string) => {
+    if (computerCalls.delete(toolCallId)) {
       if (computerLease) await updateComputer(false).catch(() => { computerLease = undefined; });
-      release();
     }
+  };
+  // Also covers a later extension blocking a call after our preflight passed.
+  pi.on('tool_execution_end', async (event) => { await finishComputerCall(event.toolCallId); });
+  pi.on("tool_result", async (event) => {
+    await finishComputerCall(event.toolCallId);
     return {
       content: event.content.map((item) =>
         item.type === "text" ? { ...item, text: scrub(item.text) } : item,

package/package.json

@@ -1,19 +1,16 @@
 {
   "name": "crewx-pi-kit",
-  "version": "0.1.12",
+  "version": "0.1.13",
   "description": "Typed CrewX tools and operating skills for managed Pi agents.",
   "type": "module",
-  "files": [
-    "extensions",
-    "skills"
-  ],
+  "files": ["extensions", "skills"],
   "pi": {
-    "extensions": [
-      "extensions"
-    ],
-    "skills": [
-      "skills"
-    ]
+    "extensions": ["extensions"],
+    "skills": ["skills"]
+  },
+  "scripts": {
+    "types:check": "tsc --noEmit",
+    "test": "tsc --noEmit && node --experimental-strip-types --test test/*.test.mjs"
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "*",
@@ -25,15 +22,7 @@
     "typebox": "^1.0.55",
     "typescript": "^5.9.3"
   },
-  "engines": {
-    "node": ">=22"
-  },
-  "publishConfig": {
-    "access": "public"
-  },
-  "license": "MIT",
-  "scripts": {
-    "types:check": "tsc --noEmit",
-    "test": "tsc --noEmit && node --experimental-strip-types --test test/*.test.mjs"
-  }
-}
+  "engines": {"node": ">=22"},
+  "publishConfig": {"access": "public"},
+  "license": "MIT"
+}

package/LICENSE

@@ -1,21 +0,0 @@
-MIT License
-
-Copyright (c) 2026 CrewX contributors
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-SOFTWARE.