@parix/cli 0.1.7 → 0.1.8

package/dist/cli.cjs

@@ -25,8 +25,14 @@ async function main() {
         .addCommand((0, tb_1.createTbCommand)());
     await program.parseAsync(process.argv);
 }
-main().catch((error) => {
+main()
+    .then(() => {
+    // CLIs that open HTTP servers / use undici fetch can leave sockets that keep
+    // Node alive after the command is done. Exit explicitly once work is finished.
+    process.exit(process.exitCode ?? 0);
+})
+    .catch((error) => {
     const message = error instanceof Error ? error.message : String(error);
     prompts_1.log.error(message);
-    process.exitCode = 1;
+    process.exit(1);
 });

package/dist/cli.js

@@ -23,8 +23,14 @@ async function main() {
         .addCommand(createTbCommand());
     await program.parseAsync(process.argv);
 }
-main().catch((error) => {
+main()
+    .then(() => {
+    // CLIs that open HTTP servers / use undici fetch can leave sockets that keep
+    // Node alive after the command is done. Exit explicitly once work is finished.
+    process.exit(process.exitCode ?? 0);
+})
+    .catch((error) => {
     const message = error instanceof Error ? error.message : String(error);
     log.error(message);
-    process.exitCode = 1;
+    process.exit(1);
 });

package/dist/commands/auth.cjs

@@ -69,6 +69,9 @@ async function handleLogin(options) {
         waitSpinner.start('Waiting for browser callback');
         const callbackResult = await loopbackServer.waitForResult();
         waitSpinner.stop('Received browser callback');
+        // Shut down the loopback server before token exchange so browser keep-alive
+        // sockets cannot keep the process open after sign-in completes.
+        await loopbackServer.close().catch(() => { });
         const tokenSet = await (0, oauth_api_1.exchangeAuthorizationCode)({
             baseUrl,
             code: callbackResult.code,

package/dist/commands/auth.js

@@ -66,6 +66,9 @@ async function handleLogin(options) {
         waitSpinner.start('Waiting for browser callback');
         const callbackResult = await loopbackServer.waitForResult();
         waitSpinner.stop('Received browser callback');
+        // Shut down the loopback server before token exchange so browser keep-alive
+        // sockets cannot keep the process open after sign-in completes.
+        await loopbackServer.close().catch(() => { });
         const tokenSet = await exchangeAuthorizationCode({
             baseUrl,
             code: callbackResult.code,

package/dist/lib/loopback-server.cjs

@@ -37,6 +37,8 @@ async function startLoopbackServer(options) {
         rejectResult = reject;
     });
     server.on('request', (request, response) => {
+        // One-shot OAuth loopback: do not keep browser sockets alive after the response.
+        response.setHeader('connection', 'close');
         const requestUrl = new URL(request.url ?? '/', 'http://127.0.0.1');
         if (requestUrl.pathname === '/favicon.ico') {
             response.writeHead(204);
@@ -120,17 +122,37 @@ function listen(server, port) {
 }
 function closeServer(server) {
     return new Promise((resolve, reject) => {
+        let settled = false;
+        const finish = (error) => {
+            if (settled) {
+                return;
+            }
+            settled = true;
+            if (error) {
+                reject(error);
+                return;
+            }
+            resolve();
+        };
         server.close((error) => {
             if (error) {
                 if ('code' in error && error.code === 'ERR_SERVER_NOT_RUNNING') {
-                    resolve();
+                    finish();
                     return;
                 }
-                reject(error);
+                finish(error);
                 return;
             }
-            resolve();
+            finish();
         });
+        // `server.close()` waits for existing connections. Browsers often keep the
+        // loopback socket open via HTTP keep-alive, which would hang login forever
+        // after the session is already stored. Force those sockets closed (Node >= 18.2).
+        server.closeAllConnections();
+        // Last resort: never let close block the CLI indefinitely.
+        setTimeout(() => {
+            finish();
+        }, 250).unref();
     });
 }
 function buildHtml(title, message) {

package/dist/lib/loopback-server.js

@@ -34,6 +34,8 @@ export async function startLoopbackServer(options) {
         rejectResult = reject;
     });
     server.on('request', (request, response) => {
+        // One-shot OAuth loopback: do not keep browser sockets alive after the response.
+        response.setHeader('connection', 'close');
         const requestUrl = new URL(request.url ?? '/', 'http://127.0.0.1');
         if (requestUrl.pathname === '/favicon.ico') {
             response.writeHead(204);
@@ -117,17 +119,37 @@ function listen(server, port) {
 }
 function closeServer(server) {
     return new Promise((resolve, reject) => {
+        let settled = false;
+        const finish = (error) => {
+            if (settled) {
+                return;
+            }
+            settled = true;
+            if (error) {
+                reject(error);
+                return;
+            }
+            resolve();
+        };
         server.close((error) => {
             if (error) {
                 if ('code' in error && error.code === 'ERR_SERVER_NOT_RUNNING') {
-                    resolve();
+                    finish();
                     return;
                 }
-                reject(error);
+                finish(error);
                 return;
             }
-            resolve();
+            finish();
         });
+        // `server.close()` waits for existing connections. Browsers often keep the
+        // loopback socket open via HTTP keep-alive, which would hang login forever
+        // after the session is already stored. Force those sockets closed (Node >= 18.2).
+        server.closeAllConnections();
+        // Last resort: never let close block the CLI indefinitely.
+        setTimeout(() => {
+            finish();
+        }, 250).unref();
     });
 }
 function buildHtml(title, message) {

package/package.json

@@ -2,7 +2,7 @@
   "name": "@parix/cli",
   "description": "Parix command line interface",
   "type": "module",
-  "version": "0.1.7",
+  "version": "0.1.8",
   "bin": {
     "parix": "dist/cli.cjs"
   },