fix: use kill-port npm package for reliable cross-platform port cleanup

- Replace manual port killing (lsof/fuser) with kill-port npm package
- Add kill-port as workspace dependency
- Update all startup scripts in federated-css and federated-css-react-ssr
- Ensures consistent port cleanup across CI and local environments

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: ScriptedAlchemy

federated-css-react-ssr/scripts/start-exposes.cjs

@@ -1,27 +1,19 @@
-const { spawn, execSync } = require('node:child_process');
+const { spawn } = require('node:child_process');
 const path = require('node:path');
 const waitOn = require('wait-on');
+const kill = require('kill-port');
 
 const root = path.resolve(__dirname, '..');
 
 function run(cmd, args, opts = {}) {
   return spawn(cmd, args, { stdio: 'inherit', cwd: root, shell: true, ...opts });
 }
 
-function killPort(port) {
+async function killPort(port) {
   try {
-    // Try fuser first (more reliable in CI)
-    execSync(`fuser -k ${port}/tcp 2>/dev/null || true`, { stdio: 'ignore' });
+    await kill(port, 'tcp');
   } catch (e) {
-    // Fallback to lsof for macOS/local development
-    try {
-      const pids = execSync(`lsof -ti tcp:${port} 2>/dev/null || true`, { encoding: 'utf8' }).trim();
-      if (pids) {
-        execSync(`kill -9 ${pids} 2>/dev/null || true`, { stdio: 'ignore' });
-      }
-    } catch (e2) {
-      // Ignore errors, port might not be in use
-    }
+    // Port might not be in use, ignore
   }
 }
 
@@ -46,7 +38,7 @@ async function main() {
 
   // Clean up all ports first
   console.log('[exposes] cleaning up ports...');
-  exposes.forEach(({ port }) => killPort(port));
+  await Promise.all(exposes.map(({ port }) => killPort(port)));
   await new Promise(resolve => setTimeout(resolve, 1000));
 
   const procs = [];

federated-css-react-ssr/scripts/start-shells.cjs

@@ -1,27 +1,19 @@
-const { spawn, execSync } = require('node:child_process');
+const { spawn } = require('node:child_process');
 const path = require('node:path');
 const waitOn = require('wait-on');
+const kill = require('kill-port');
 
 const root = path.resolve(__dirname, '..');
 
 function run(cmd, args, opts = {}) {
   return spawn(cmd, args, { stdio: 'inherit', cwd: root, shell: true, ...opts });
 }
 
-function killPort(port) {
+async function killPort(port) {
   try {
-    // Try fuser first (more reliable in CI)
-    execSync(`fuser -k ${port}/tcp 2>/dev/null || true`, { stdio: 'ignore' });
+    await kill(port, 'tcp');
   } catch (e) {
-    // Fallback to lsof for macOS/local development
-    try {
-      const pids = execSync(`lsof -ti tcp:${port} 2>/dev/null || true`, { encoding: 'utf8' }).trim();
-      if (pids) {
-        execSync(`kill -9 ${pids} 2>/dev/null || true`, { stdio: 'ignore' });
-      }
-    } catch (e2) {
-      // Ignore errors, port might not be in use
-    }
+    // Port might not be in use, ignore
   }
 }
 
@@ -45,7 +37,7 @@ async function main() {
 
   // Clean up all ports first
   console.log('[shells] cleaning up ports...');
-  shells.forEach(({ port }) => killPort(port));
+  await Promise.all(shells.map(({ port }) => killPort(port)));
   await new Promise(resolve => setTimeout(resolve, 1000));
 
   const procs = [];

federated-css/package.json

@@ -27,6 +27,7 @@
   },
   "devDependencies": {
     "@playwright/test": "^1.54.2",
+    "kill-port": "2.0.1",
     "wait-on": "7.2.0"
   }
 }

federated-css/scripts/start-all.cjs

@@ -1,7 +1,8 @@
-const { spawn, execSync } = require('node:child_process');
+const { spawn } = require('node:child_process');
 const waitOn = require('wait-on');
 const path = require('node:path');
 const fs = require('node:fs');
+const kill = require('kill-port');
 
 const root = path.resolve(__dirname, '..');
 
@@ -15,20 +16,11 @@ function run(cmd, args) {
   return spawn(cmd, args, { stdio: 'inherit', cwd: root, shell: true });
 }
 
-function killPort(port) {
+async function killPort(port) {
   try {
-    // Try fuser first (more reliable in CI)
-    execSync(`fuser -k ${port}/tcp 2>/dev/null || true`, { stdio: 'ignore' });
+    await kill(port, 'tcp');
   } catch (e) {
-    // Fallback to lsof for macOS/local development
-    try {
-      const pids = execSync(`lsof -ti tcp:${port} 2>/dev/null || true`, { encoding: 'utf8' }).trim();
-      if (pids) {
-        execSync(`kill -9 ${pids} 2>/dev/null || true`, { stdio: 'ignore' });
-      }
-    } catch (e2) {
-      // Ignore errors, port might not be in use
-    }
+    // Port might not be in use, ignore
   }
 }
 
@@ -56,7 +48,8 @@ async function main() {
 
   // Kill all potentially conflicting ports first
   console.log('[federated-css] cleaning up ports...');
-  [...reactConsumers.map(c => c.port), ...exposes, ...nextConsumers.map(c => c.port)].forEach(killPort);
+  const allPorts = [...reactConsumers.map(c => c.port), ...exposes, ...nextConsumers.map(c => c.port)];
+  await Promise.all(allPorts.map(port => killPort(port)));
 
   // Give OS time to fully release ports
   await new Promise(resolve => setTimeout(resolve, 1000));

package.json

@@ -48,6 +48,7 @@
     "husky": "9.0.11",
     "jest": "29.7.0",
     "js-yaml": "4.1.0",
+    "kill-port": "2.0.1",
     "lerna": "8.1.8",
     "lint-staged": "^15.2.10",
     "mocha": "10.6.0",