@jfrog-boost/boost 0.11.0 → 0.11.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jfrog-boost/boost",
-  "version": "0.11.0",
+  "version": "0.11.2",
   "description": "Boost CLI — compress agent/CI tool output (prebuilt binary wrapper)",
   "license": "UNLICENSED",
   "homepage": "https://boost.jfrog.com",
@@ -26,11 +26,11 @@
     "node": ">=18"
   },
   "optionalDependencies": {
-    "@jfrog-boost/boost-darwin-arm64": "0.11.0",
-    "@jfrog-boost/boost-darwin-x64": "0.11.0",
-    "@jfrog-boost/boost-linux-arm64": "0.11.0",
-    "@jfrog-boost/boost-linux-x64": "0.11.0",
-    "@jfrog-boost/boost-win32-x64": "0.11.0"
+    "@jfrog-boost/boost-darwin-arm64": "0.11.2",
+    "@jfrog-boost/boost-darwin-x64": "0.11.2",
+    "@jfrog-boost/boost-linux-arm64": "0.11.2",
+    "@jfrog-boost/boost-linux-x64": "0.11.2",
+    "@jfrog-boost/boost-win32-x64": "0.11.2"
   },
   "keywords": [
     "boost",

package/scripts/postinstall.js

@@ -5,15 +5,23 @@
  * After a global `npm i -g @jfrog-boost/boost`, run `boost init` so the terms
  * prompt and agent integrations match the shell installer path.
  *
+ * npm lifecycle scripts never get a TTY on stdin/stdout/stderr (even when the
+ * parent `npm` is interactive). Skipping on `process.stderr.isTTY` therefore
+ * skipped init for every real `npm i -g`. Instead we attach stdio to the
+ * controlling terminal (`/dev/tty`, or CONIN$/CONOUT$ on Windows) — same idea
+ * as `install.sh`'s `boost init </dev/tty` — so Go's `isTTY()` and prompts work.
+ *
  * Skips (exit 0) when:
  *   - not a global install (local dependency)
- *   - CI / non-TTY (cannot prompt)
+ *   - CI
  *   - BOOST_SKIP_INIT is set
+ *   - no controlling terminal
  *   - the platform binary is missing
  *
  * Never fails the npm install: init errors print a hint and exit 0.
  */
 
+const fs = require('fs');
 const { spawnSync } = require('child_process');
 const { resolveBinaryPath } = require('../lib/platform');
 
@@ -28,35 +36,59 @@ function skipReason() {
   if (process.env.CI === 'true' || process.env.CI === '1') {
     return 'CI';
   }
-  // Match boost's isTTY(): prompts need a real terminal on stderr.
-  if (!process.stderr.isTTY) {
-    return 'non-TTY';
-  }
   return null;
 }
 
-const reason = skipReason();
-if (reason) {
-  console.log(`→ Skipping boost init (${reason}). Run \`boost init\` when ready.`);
-  process.exit(0);
+/** Open the controlling terminal for child stdio, or null if none. */
+function openControllingStdio() {
+  try {
+    if (process.platform === 'win32') {
+      return [fs.openSync('\\\\.\\CONIN$', 'r'), fs.openSync('\\\\.\\CONOUT$', 'w'), fs.openSync('\\\\.\\CONOUT$', 'w')];
+    }
+    // Separate in/out fds so the child sees a real char device on stderr
+    // (Go's isTTY checks stderr) while still reading prompts from the TTY.
+    return [fs.openSync('/dev/tty', 'r'), fs.openSync('/dev/tty', 'w'), fs.openSync('/dev/tty', 'w')];
+  } catch {
+    return null;
+  }
 }
 
-let bin;
-try {
-  bin = resolveBinaryPath();
-} catch (err) {
-  console.warn(`→ Skipping boost init: ${err.message || err}`);
-  process.exit(0);
-}
+function run() {
+  const reason = skipReason();
+  if (reason) {
+    console.log(`→ Skipping boost init (${reason}). Run \`boost init\` when ready.`);
+    process.exit(0);
+  }
 
-console.log('→ Running boost init…');
-const result = spawnSync(bin, ['init'], { stdio: 'inherit' });
-if (result.error) {
-  console.warn(`→ boost init failed to start: ${result.error.message || result.error}`);
-  console.warn('  Run `boost init` manually when ready.');
+  const stdio = openControllingStdio();
+  if (!stdio) {
+    console.log('→ Skipping boost init (no controlling TTY). Run `boost init` when ready.');
+    process.exit(0);
+  }
+
+  let bin;
+  try {
+    bin = resolveBinaryPath();
+  } catch (err) {
+    console.warn(`→ Skipping boost init: ${err.message || err}`);
+    process.exit(0);
+  }
+
+  console.log('→ Running boost init…');
+  const result = spawnSync(bin, ['init'], { stdio });
+  if (result.error) {
+    console.warn(`→ boost init failed to start: ${result.error.message || result.error}`);
+    console.warn('  Run `boost init` manually when ready.');
+    process.exit(0);
+  }
+  if (result.status !== 0) {
+    console.warn('→ boost init exited non-zero; run `boost init` manually when ready.');
+  }
   process.exit(0);
 }
-if (result.status !== 0) {
-  console.warn('→ boost init exited non-zero; run `boost init` manually when ready.');
+
+if (require.main === module) {
+  run();
 }
-process.exit(0);
+
+module.exports = { skipReason, openControllingStdio };