memtrace 0.3.42 → 0.3.43

package/bin/memtrace.js

@@ -138,12 +138,17 @@ if (args[0] === "install" || args[0] === "update" || args[0] === "upgrade") {
   const memtraceCmd = platformBinary("memtrace", process.platform);
 
   process.stdout.write("memtrace: fetching latest from npm registry…\n");
+  // Tell the postinstall RTK opt-in to stay silent — *we* own the
+  // prompt and have a clean TTY (npm's progress spinner shares the
+  // user's terminal with the postinstall, which makes its prompt
+  // unreadable and readline picks up a phantom EOF, default-yes
+  // installing RTK without the user seeing the question).
   const installResult = spawnSync(
     npmCmd,
     ["install", "-g", "memtrace@latest"],
     spawnOptionsForPlatform(process.platform, {
       stdio: "inherit",
-      env: process.env,
+      env: { ...process.env, MEMTRACE_INSTALL_PARENT: "1" },
     })
   );
 

package/install.js

@@ -197,6 +197,14 @@ if (require.main === module) {
     const readline = require("readline");
     const rtk = require("./lib/rtk-integration");
 
+    // Nested under `memtrace install` shim? The parent owns the prompt
+    // and has a clean TTY. Two prompt paths racing over the same
+    // terminal means npm's progress spinner overwrites the question
+    // line and readline observes a phantom EOF — user gets RTK
+    // installed without ever seeing or answering Y/n. Bail out here;
+    // bin/memtrace.js will run the prompt cleanly after npm exits.
+    if (rtk.isPostinstallNestedUnderShim({ env: process.env })) return;
+
     let alreadyInstalled = false;
     try { alreadyInstalled = rtk.detectRtk(); } catch (_) { /* skip */ }
 

package/lib/rtk-integration.js

@@ -53,6 +53,23 @@ function isRtkAutoInstall(input = {}) {
     return isTruthyEnv(env.MEMTRACE_AUTO_INSTALL_RTK);
 }
 
+/**
+ * True when this postinstall is running nested under the `memtrace
+ * install` shim — i.e. the parent process is bin/memtrace.js spawning
+ * `npm install -g memtrace@latest`. The parent owns user interaction
+ * (it has a clean TTY, and it runs its own RTK opt-in flow after npm
+ * exits). The nested postinstall must therefore stay completely silent
+ * for RTK — otherwise both prompt paths race over the same terminal,
+ * the spinner overwrites the prompt, and readline picks up a phantom
+ * EOF.
+ *
+ * Coordinated via `MEMTRACE_INSTALL_PARENT=1` in the spawn env.
+ */
+function isPostinstallNestedUnderShim(input = {}) {
+    const env = input.env || {};
+    return isTruthyEnv(env.MEMTRACE_INSTALL_PARENT);
+}
+
 // ── Decision helpers ────────────────────────────────────────────────
 
 /**
@@ -275,19 +292,27 @@ function openTtyStreams(opts = {}) {
     const fs = opts.fs || require("fs");
     const tty = opts.tty || require("tty");
 
-    let fd;
+    // Open TWO fds — one for read, one for write. Sharing a single
+    // r+ fd between ReadStream and WriteStream causes readline to
+    // immediately observe EOF on some setups (notably when a parent
+    // process is also writing progress to the same controlling tty,
+    // e.g. npm's spinner during `memtrace install`).
+    let inFd, outFd;
     try {
-        fd = fs.openSync("/dev/tty", "r+");
+        inFd = fs.openSync("/dev/tty", "r");
+        outFd = fs.openSync("/dev/tty", "w");
     } catch (_) {
+        if (inFd != null) { try { fs.closeSync(inFd); } catch (_) {} }
         return null; // no controlling tty (Docker, CI, etc.)
     }
 
     let input, output;
     try {
-        input = new tty.ReadStream(fd);
-        output = new tty.WriteStream(fd);
+        input = new tty.ReadStream(inFd);
+        output = new tty.WriteStream(outFd);
     } catch (e) {
-        try { fs.closeSync(fd); } catch (_) { /* best-effort */ }
+        try { fs.closeSync(inFd); } catch (_) {}
+        try { fs.closeSync(outFd); } catch (_) {}
         return null;
     }
 
@@ -341,6 +366,7 @@ module.exports = {
     // Pure helpers
     isRtkPromptDisabled,
     isRtkAutoInstall,
+    isPostinstallNestedUnderShim,
     shouldPromptForRtk,
     effectiveRtkAction,
     chooseInstallStrategy,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "memtrace",
-  "version": "0.3.42",
+  "version": "0.3.43",
   "description": "Code intelligence graph — MCP server + AI agent skills + visualization UI",
   "keywords": [
     "mcp",