@gramatr/client 0.6.5 → 0.6.6

package/bin/gmtr-login.ts

@@ -148,10 +148,11 @@ function isHeadless(forceFlag: boolean = false): boolean {
   if (forceFlag) return true;
   if (process.env.GRAMATR_LOGIN_HEADLESS === '1') return true;
 
-  // SSH session without display forwarding
-  if (process.env.SSH_CONNECTION || process.env.SSH_TTY) {
-    if (!process.env.DISPLAY && process.platform !== 'darwin') return true;
-  }
+  // SSH session — always go headless. Even on macOS (which has a local
+  // display), `open` would launch Safari on the Mac's *physical* screen,
+  // not on the remote terminal where the user actually is. Device flow
+  // lets them paste the code on whichever machine they're sitting at.
+  if (process.env.SSH_CONNECTION || process.env.SSH_TTY) return true;
   // Docker / CI / no TTY
   if (process.env.CI || process.env.DOCKER) return true;
   // Linux without display
@@ -493,23 +494,35 @@ async function loginBrowser(opts: { forceHeadless?: boolean } = {}): Promise<voi
       const code = url.searchParams.get('code');
       const returnedState = url.searchParams.get('state');
       const error = url.searchParams.get('error');
+      // v0.6.6: `server.close()` alone does not terminate keep-alive
+      // sockets — the browser holds the connection open and the Node
+      // event loop never exits, so the CLI prints "Authenticated" and
+      // then hangs until Ctrl+C. Send `Connection: close` on the
+      // response and call `closeAllConnections()` to forcibly drop
+      // lingering sockets after we respond.
+      const shutdown = () => {
+        callbackServer.close();
+        if (typeof (callbackServer as any).closeAllConnections === 'function') {
+          (callbackServer as any).closeAllConnections();
+        }
+      };
       if (error) {
-        res.writeHead(200, { 'Content-Type': 'text/html' });
+        res.writeHead(200, { 'Content-Type': 'text/html', Connection: 'close' });
         res.end(errorPage('Authentication Failed', error));
-        callbackServer.close();
+        shutdown();
         reject(new Error(`OAuth error: ${error}`));
         return;
       }
       if (!code || returnedState !== state) {
-        res.writeHead(400, { 'Content-Type': 'text/html' });
+        res.writeHead(400, { 'Content-Type': 'text/html', Connection: 'close' });
         res.end(errorPage('Invalid Callback', 'Missing code or state mismatch. Please try again.'));
-        callbackServer.close();
+        shutdown();
         reject(new Error('Invalid callback'));
         return;
       }
-      res.writeHead(200, { 'Content-Type': 'text/html' });
+      res.writeHead(200, { 'Content-Type': 'text/html', Connection: 'close' });
       res.end(successPage());
-      callbackServer.close();
+      shutdown();
       resolve(code);
     });
     codeTimeoutHandle = setTimeout(() => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gramatr/client",
-  "version": "0.6.5",
+  "version": "0.6.6",
   "publishConfig": {
     "access": "public"
   },