prjct-cli 3.88.0 → 3.91.0

package/dist/bin/prjct.mjs

@@ -1,9 +1,17 @@
 #!/usr/bin/env node
-import{connect}from"node:net";import{existsSync}from"node:fs";import{createHash,randomUUID}from"node:crypto";import{homedir}from"node:os";import{join,resolve}from"node:path";import{spawn}from"node:child_process";
+import{connect}from"node:net";import{existsSync,readSync}from"node:fs";import{isatty}from"node:tty";import{createHash,randomUUID}from"node:crypto";import{homedir}from"node:os";import{join,resolve}from"node:path";import{spawn}from"node:child_process";
 const cliHome=process.env.PRJCT_CLI_HOME?resolve(process.env.PRJCT_CLI_HOME):join(homedir(),".prjct-cli");
 const namedPipe=process.platform==="win32";
 const sockPath=namedPipe?"\\.\pipe\prjct-"+createHash("sha1").update(resolve(cliHome)).digest("hex").slice(0,16)+"-daemon":join(cliHome,"run","daemon.sock");
 const hasEndpoint=()=>namedPipe||existsSync(sockPath);
+// Sync-read stdin via fs.readSync on fd 0 directly — never READS through
+// process.stdin, whose stream wrapper costs event-loop turnaround per hook.
+// Touching process.stdin.fd once up front flips fd 0 non-blocking (this
+// shim always runs on node), so an open-but-empty pipe yields EAGAIN and
+// the deadline is REAL: a host that opens stdin but never closes degrades
+// to whatever arrived instead of hanging until the host's hook-timeout
+// kill. EAGAIN retries sleep ~1ms via Atomics.wait to avoid a busy-spin.
+function readStdinSync(ms){if(isatty(0))return"";try{void process.stdin.fd}catch{}const dl=Date.now()+ms;const slp=new Int32Array(new SharedArrayBuffer(4));const buf=Buffer.alloc(65536);const parts=[];for(;;){try{const n=readSync(0,buf,0,buf.length,null);if(n===0)break;parts.push(Buffer.from(buf.subarray(0,n)))}catch(e){if(e&&e.code==="EAGAIN"&&Date.now()<dl){Atomics.wait(slp,0,0,1);continue}break}}return Buffer.concat(parts).toString("utf8")}
 const args=process.argv.slice(2);
 const cmd=args.find(a=>!a.startsWith("-"));
 const skip=new Set(["seed","mcp","install","help","start","setup","login","logout","auth","context","update","uninstall","daemon","stop","restart","upgrade","hook","hooks","claude","crew","harness","doctor","watch","version","prefs","retro","health","skill-adherence","review-risk","context-save","context-restore","-h","--help","-v","--version","intent","spec","audit-spec","dev","web","serve","__internal-auto-update","__post-upgrade","__internal-ensure-daemon"]);
@@ -28,7 +36,14 @@ function sendHook(sub,data){
 if(cmd==="hook"){
   if(process.env.PRJCT_NO_DAEMON!=="1"&&hasEndpoint()){
     const sub=args[1];
-    if(process.stdin.isTTY){sendHook(sub,"")}else{const chunks=[];process.stdin.on("data",c=>chunks.push(c));const send=()=>sendHook(sub,Buffer.concat(chunks).toString("utf8"));process.stdin.on("end",send);process.stdin.on("error",send);setTimeout(send,1000)}
+    // Sync stdin read: event-accumulating a pipe costs ~15-20ms of event-loop
+    // turnaround per hook in bun/node; hosts write the event JSON and close
+    // stdin immediately, so a sync read returns at once. Fail-soft: an
+    // unrecoverable read error yields whatever was read so far (the hook
+    // treats a blank result as {}). The host-side hook timeout
+    // (settings-installer HOOK_TIMEOUT_SECONDS) remains the backstop for a
+    // pathological host that never closes stdin.
+    sendHook(sub,readStdinSync(1000));
   }else{
     // Cold path (daemon disabled/unreachable): run the hook from the dedicated
     // hooks bundle, NOT the full core. cold-entry emits host JSON then detaches