playwriter 0.2.0 → 0.3.1

package/src/start-relay-server.ts

@@ -1,5 +1,6 @@
 import { startPlayWriterCDPRelayServer } from './cdp-relay.js'
 import { createFileLogger } from './create-logger.js'
+import { waitForRelayVersion } from './relay-client.js'
 import { LOG_CDP_FILE_PATH } from './utils.js'
 
 process.title = 'playwriter-ws-server'
@@ -25,7 +26,27 @@ export async function startServer({
   host = '127.0.0.1',
   token,
 }: { port?: number; host?: string; token?: string } = {}) {
-  const server = await startPlayWriterCDPRelayServer({ port, host, token, logger })
+  let server
+  try {
+    server = await startPlayWriterCDPRelayServer({ port, host, token, logger })
+  } catch (err: unknown) {
+    // When two relay processes race to start (issue #75), the loser gets
+    // EADDRINUSE. Check if the winner is a valid relay and exit cleanly
+    // instead of crashing with a scary error in the logs.
+    const errWithCode = err as NodeJS.ErrnoException
+    if (errWithCode?.code === 'EADDRINUSE') {
+      // The winner may have bound the port but not be ready to answer /version
+      // yet, so poll for up to 2 seconds before giving up.
+      const version = await waitForRelayVersion({ port })
+      if (version) {
+        await logger.log(`Another relay (v${version}) already bound to port ${port}, exiting gracefully`)
+        process.exit(0)
+      }
+      await logger.error(`Port ${port} is in use by a non-relay process`)
+      process.exit(1)
+    }
+    throw err
+  }
 
   console.log('CDP Relay Server running. Press Ctrl+C to stop.')
   console.log('Logs are being written to:', logger.logFilePath)

package/src/utils.ts

@@ -56,6 +56,10 @@ export function getCdpUrl({
   return `${wsBaseUrl}/cdp/${id}${suffix}`
 }
 
+export function shouldAutoEnablePlaywriter(): boolean {
+  return process.env.PLAYWRITER_AUTO_ENABLE?.toLowerCase() !== 'false'
+}
+
 // Use ~/.playwriter for logs so each OS user gets their own dir (avoids permission errors on shared machines, see #44)
 const LOG_BASE_DIR = path.join(os.homedir(), '.playwriter')
 export const LOG_FILE_PATH = process.env.PLAYWRITER_LOG_FILE_PATH || path.join(LOG_BASE_DIR, 'relay-server.log')