@gachlab/devup 0.9.0 → 0.9.2

package/CHANGELOG.md

@@ -5,6 +5,25 @@ All notable changes to `@gachlab/devup` are documented here.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.9.2] — 2026-05-22
+
+Critical hotfix. **All 0.9.x users should upgrade immediately.**
+
+### Fixed
+- **Bundle's top-level `main()` would fire when devup was imported as a library.** When a user's `devup.config.ts` did `import { defineConfig } from '@gachlab/devup'`, Node loaded our compiled `dist/index.js` to satisfy the import — and the bundle's top-level `main()` invocation ran, starting a SECOND, concurrent devup process. Symptoms: every line of output duplicated, three or more racing instances of the same service spawning in milliseconds, ports clobbered, the daemon's socket created but never bound, `devup ctl ping` ECONNREFUSED. Fix: guard `main()` with a `realpathSync(process.argv[1]) === fileURLToPath(import.meta.url)` check so it only fires when the script is invoked directly (as the `devup` binary), not when imported.
+- **`devup ctl` crashed with an unhandled `error` event on ECONNREFUSED.** The socket-error handler was attached to the connection but Node's `readline.createInterface` re-forwarded the error through its own emitter, which had no listener, so the process crashed instead of reporting cleanly. Now we attach the handler on both the socket and the readline interface.
+
+### Why this matters
+Every devup invocation that loaded a config file (i.e. everything except `--version` / `--help`) was silently running two instances of itself. That is the root cause of every weird behaviour reported against 0.9.0 / 0.9.1: duplicated prompts, daemons that "die" right after starting, sockets that exist but refuse connections, port-takeover prompts firing for ports the user expected devup to own.
+
+## [0.9.1] — 2026-05-22
+
+Hotfix for two issues reported moments after 0.9.0 hit:
+
+### Fixed
+- **The y/N prompt no longer no-ops silently.** The conflict list would print, then the process would just continue without waiting for input on some terminals (IDE integrated terminals, multiplexers, custom shells where `process.stdin.isTTY` is misreported). Replaced `readline.question` with direct stdin handling. TTY detection now also accepts stderr / stdout being a TTY when stdin isn't — covers more real environments.
+- **Daemon-already-running guard moved before the port scan.** Running `devup` (TUI) or `devup up -d` while a daemon was already up for the same project caused the scan to list the daemon's own services as conflicts, prompt the user to kill them, restart them (because the daemon's auto-restarter kicked in), then bail with "daemon already running". Now the daemon check runs first and short-circuits cleanly — no churn, single clear error.
+
 ## [0.9.0] — 2026-05-22
 
 Pre-boot port-conflict resolution. When devup detects another process already holding a port it needs, it now offers to take it over instead of silently marking the service as crashed.

package/dist/control-plane/client.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"client.d.ts","sourceRoot":"","sources":["../../src/control-plane/client.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,iBAAiB,EAAE,MAAM,oBAAoB,CAAC;AAEvD,OAAO,EAAE,iBAAiB,EAAE,CAAC;AAE7B,gEAAgE;AAChE,wBAAgB,aAAa,CAAC,WAAW,EAAE,MAAM,EAAE,YAAY,CAAC,EAAE,MAAM,GAAG,MAAM,CAEhF;AAED,8EAA8E;AAC9E,wBAAgB,kBAAkB,CAAC,UAAU,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,GAAG,IAAI,CAMhF;AAED,0EAA0E;AAC1E,wBAAgB,OAAO,CACrB,UAAU,EAAE,MAAM,EAClB,MAAM,EAAE,MAAM,EACd,MAAM,GAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAM,GACnC,OAAO,CAAC,OAAO,CAAC,CAiBlB;AAED,MAAM,WAAW,WAAW;IAC1B,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,EAAE,OAAO,CAAC;IACd,GAAG,CAAC,EAAE,MAAM,CAAC;CACd;AAED;+FAC+F;AAC/F,wBAAgB,UAAU,CACxB,UAAU,EAAE,MAAM,EAClB,MAAM,EAAE,MAAM,EACd,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC/B,OAAO,EAAE,CAAC,KAAK,EAAE,WAAW,KAAK,IAAI,EACrC,OAAO,CAAC,EAAE,CAAC,GAAG,EAAE,KAAK,KAAK,IAAI,GAC7B,MAAM,IAAI,CAqBZ"}
+{"version":3,"file":"client.d.ts","sourceRoot":"","sources":["../../src/control-plane/client.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,iBAAiB,EAAE,MAAM,oBAAoB,CAAC;AAEvD,OAAO,EAAE,iBAAiB,EAAE,CAAC;AAE7B,gEAAgE;AAChE,wBAAgB,aAAa,CAAC,WAAW,EAAE,MAAM,EAAE,YAAY,CAAC,EAAE,MAAM,GAAG,MAAM,CAEhF;AAED,8EAA8E;AAC9E,wBAAgB,kBAAkB,CAAC,UAAU,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,GAAG,IAAI,CAMhF;AAED,0EAA0E;AAC1E,wBAAgB,OAAO,CACrB,UAAU,EAAE,MAAM,EAClB,MAAM,EAAE,MAAM,EACd,MAAM,GAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAM,GACnC,OAAO,CAAC,OAAO,CAAC,CAyBlB;AAED,MAAM,WAAW,WAAW;IAC1B,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,EAAE,OAAO,CAAC;IACd,GAAG,CAAC,EAAE,MAAM,CAAC;CACd;AAED;+FAC+F;AAC/F,wBAAgB,UAAU,CACxB,UAAU,EAAE,MAAM,EAClB,MAAM,EAAE,MAAM,EACd,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC/B,OAAO,EAAE,CAAC,KAAK,EAAE,WAAW,KAAK,IAAI,EACrC,OAAO,CAAC,EAAE,CAAC,GAAG,EAAE,KAAK,KAAK,IAAI,GAC7B,MAAM,IAAI,CA0BZ"}

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAwBA,OAAO,EAAE,YAAY,EAAE,MAAM,mBAAmB,CAAC;AACjD,YAAY,EAAE,cAAc,EAAE,aAAa,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAChG,YAAY,EAAE,QAAQ,EAAE,YAAY,EAAE,MAAM,qBAAqB,CAAC;AAClE,YAAY,EAAE,mBAAmB,EAAE,SAAS,EAAE,MAAM,yBAAyB,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAuBA,OAAO,EAAE,YAAY,EAAE,MAAM,mBAAmB,CAAC;AACjD,YAAY,EAAE,cAAc,EAAE,aAAa,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAChG,YAAY,EAAE,QAAQ,EAAE,YAAY,EAAE,MAAM,qBAAqB,CAAC;AAClE,YAAY,EAAE,mBAAmB,EAAE,SAAS,EAAE,MAAM,yBAAyB,CAAC"}

package/dist/index.js

@@ -3,7 +3,7 @@
 // src/index.ts
 import React7 from "react";
 import { render } from "ink";
-import { readFileSync as readFileSync4 } from "fs";
+import { readFileSync as readFileSync4, realpathSync } from "fs";
 import { dirname as dirname7, join as join10 } from "path";
 import { fileURLToPath as fileURLToPath2 } from "url";
 import { homedir as homedir5 } from "os";
@@ -884,17 +884,31 @@ Start it with \`devup\` first.`
 }
 function sendRpc(socketPath, method, params = {}) {
   return new Promise((resolve4, reject) => {
+    let settled = false;
+    const fail = (err) => {
+      if (!settled) {
+        settled = true;
+        reject(err);
+      }
+    };
+    const ok = (v) => {
+      if (!settled) {
+        settled = true;
+        resolve4(v);
+      }
+    };
     const c = createConnection(socketPath);
-    c.on("error", reject);
     const rl = createInterface2({ input: c });
+    c.on("error", fail);
+    rl.on("error", fail);
     rl.once("line", (l) => {
       c.end();
       try {
         const msg = JSON.parse(l);
-        if (msg.error) reject(new Error(msg.error.message ?? String(msg.error)));
-        else resolve4(msg.result);
+        if (msg.error) fail(new Error(msg.error.message ?? String(msg.error)));
+        else ok(msg.result);
       } catch (e) {
-        reject(e);
+        fail(e);
       }
     });
     c.write(JSON.stringify({ id: 1, method, params }) + "\n");
@@ -904,7 +918,9 @@ function openStream(socketPath, method, params, onFrame, onError) {
   const c = createConnection(socketPath);
   const rl = createInterface2({ input: c });
   let ackDone = false;
-  c.on("error", (err) => onError?.(err));
+  const onErr = (err) => onError?.(err);
+  c.on("error", onErr);
+  rl.on("error", onErr);
   c.write(JSON.stringify({ id: 1, method, params }) + "\n");
   rl.on("line", (l) => {
     try {
@@ -2647,9 +2663,6 @@ async function killAll(conflicts, out) {
   return allOk;
 }
 
-// src/index.ts
-import { createInterface as createInterface6 } from "readline";
-
 // src/platform/detect.ts
 async function detectPlatform() {
   switch (process.platform) {
@@ -4106,6 +4119,19 @@ ${formatValidationWarnings(warnings)}`);
   if (cliArgs.logFile) {
     logSink = new LogSink({ projectName: config.name, rootDir: cliArgs.logDir });
   }
+  if (process.env.DEVUP_DAEMON_CHILD !== "1") {
+    const daemonStatus = isDaemonRunning(config.name);
+    if (daemonStatus.pid && !daemonStatus.stale) {
+      console.error(`\u274C A devup daemon is already running for "${config.name}" (pid=${daemonStatus.pid}).`);
+      console.error("");
+      console.error("Stop it first with `devup down`, or interact via the control plane:");
+      console.error("  devup ctl status");
+      console.error("  devup ctl logs <svc> --follow");
+      console.error("  devup ctl restart <svc>");
+      await logSink?.close();
+      process.exit(1);
+    }
+  }
   if (process.env.DEVUP_DAEMON_CHILD !== "1") {
     const conflicts = await scanPortConflicts(services);
     if (conflicts.length) {
@@ -4153,16 +4179,6 @@ ${formatValidationWarnings(warnings)}`);
       proxyOpts
     }));
   }
-  const daemonStatus = isDaemonRunning(config.name);
-  if (daemonStatus.pid && !daemonStatus.stale) {
-    console.error(`\u274C A devup daemon is already running for "${config.name}" (pid=${daemonStatus.pid}).`);
-    console.error("");
-    console.error("Stop it first with `devup down`, or interact via the control plane:");
-    console.error("  devup ctl status");
-    console.error("  devup ctl logs <svc> --follow");
-    console.error("  devup ctl restart <svc>");
-    process.exit(1);
-  }
   const isInteractive = process.stdin.isTTY ?? false;
   const { waitUntilExit } = render(
     React7.createElement(App, {
@@ -4182,21 +4198,47 @@ ${formatValidationWarnings(warnings)}`);
 }
 function askYesNo(question) {
   return new Promise((resolve4) => {
-    if (!process.stdin.isTTY) {
+    const isTTY = Boolean(process.stdin.isTTY || process.stderr.isTTY || process.stdout.isTTY);
+    if (!isTTY) {
       resolve4(false);
       return;
     }
-    const rl = createInterface6({ input: process.stdin, output: process.stderr });
-    rl.question(question, (answer) => {
-      rl.close();
-      resolve4(/^y(es)?$/i.test(answer.trim()));
-    });
+    process.stderr.write(question);
+    process.stdin.resume();
+    process.stdin.setEncoding("utf8");
+    const cleanup = () => {
+      process.stdin.removeListener("data", onData);
+      process.stdin.removeListener("end", onEnd);
+      process.stdin.pause();
+    };
+    const onData = (data) => {
+      cleanup();
+      resolve4(/^y(es)?$/i.test(String(data).trim()));
+    };
+    const onEnd = () => {
+      cleanup();
+      resolve4(false);
+    };
+    process.stdin.once("data", onData);
+    process.stdin.once("end", onEnd);
+  });
+}
+function isInvokedDirectly() {
+  const argvPath = process.argv[1];
+  if (!argvPath) return false;
+  const moduleFile = fileURLToPath2(import.meta.url);
+  try {
+    return realpathSync(argvPath) === moduleFile;
+  } catch {
+    return argvPath === moduleFile;
+  }
+}
+if (isInvokedDirectly()) {
+  main().catch((e) => {
+    console.error(e);
+    process.exit(1);
   });
 }
-main().catch((e) => {
-  console.error(e);
-  process.exit(1);
-});
 export {
   defineConfig
 };