open-ready 0.2.21 → 0.2.23

package/open-when-ready.mjs

@@ -219,8 +219,12 @@ function resolvePortless() {
  * file, and polls the log for ready/error signals to open the browser or AI helper.
  */
 async function run() {
+  // Pin the log to an absolute path: the poll below runs for as long as the
+  // server does, and must keep reading this run's log even if the process's
+  // working directory changes in the meantime.
+  const logFile = path.resolve(logPath);
   try {
-    await fsPromises.rm(logPath, { force: true });
+    await fsPromises.rm(logFile, { force: true });
   } catch {}
 
   const { portlessBin, appName } = resolvePortless();
@@ -230,7 +234,12 @@ async function run() {
     detached: true,
   });
 
-  const logStream = fs.createWriteStream(logPath);
+  const logStream = fs.createWriteStream(logFile);
+  // A log that cannot be written (e.g. its directory was removed) must not
+  // crash the wrapper; the server output still reaches the terminal.
+  logStream.on("error", (err) => {
+    console.error(`[open-ready] Could not write ${logFile}: ${err.message}`);
+  });
   proc.stdout?.pipe(process.stdout);
   proc.stdout?.pipe(logStream);
   proc.stderr?.pipe(process.stderr);
@@ -244,8 +253,8 @@ async function run() {
 
   const poll = setInterval(async () => {
     try {
-      const stats = fs.statSync(logPath);
-      const currentLog = await fsPromises.readFile(logPath, "utf8");
+      const stats = fs.statSync(logFile);
+      const currentLog = await fsPromises.readFile(logFile, "utf8");
 
       // Stability check LAST - check ready first!
       const logChanged = stats.size !== lastLogSize;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "open-ready",
-  "version": "0.2.21",
+  "version": "0.2.23",
   "description": "Smart dev server launcher: error→AI search, success→auto-open browser. Any CLI tool.",
   "type": "module",
   "bin": {

package/test/run.test.mjs

@@ -20,6 +20,7 @@ let origArgv;
 let sigintBefore;
 let origTTY;
 let origCI;
+let intervals;
 
 const printfArgs = (text) => ["printf", `'${text}'`];
 
@@ -46,6 +47,15 @@ beforeEach(() => {
   sigintBefore = process.listeners("SIGINT");
   origTTY = process.stdin.isTTY;
   origCI = process.env.CI;
+  // Each run() starts a poll that only stops once the server is ready or its
+  // log goes quiet; stop any left over so one case's poll cannot outlive it.
+  intervals = [];
+  const realSetInterval = globalThis.setInterval;
+  vi.spyOn(globalThis, "setInterval").mockImplementation((...args) => {
+    const id = realSetInterval(...args);
+    intervals.push(id);
+    return id;
+  });
   opener.mockReset();
   waitOn.mockReset().mockResolvedValue(undefined);
   vi.spyOn(console, "error").mockImplementation(() => {});
@@ -54,6 +64,7 @@ beforeEach(() => {
 });
 
 afterEach(() => {
+  for (const id of intervals) clearInterval(id);
   process.chdir(cwd);
   process.argv = origArgv;
   for (const l of process.listeners("SIGINT")) {