Address PR feedback: simplify test setup and remove Node 22 from matrix

Co-authored-by: dgreif <[email protected]>

Author: Copilot

.github/workflows/nodejs.yml

@@ -5,15 +5,12 @@ permissions:
 jobs:
   build:
     runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        node-version: [22, 24]
     steps:
       - uses: actions/checkout@v4
-      - name: Use Node.js ${{ matrix.node-version }}
+      - name: Use Node.js
         uses: actions/setup-node@v4
         with:
-          node-version: ${{ matrix.node-version }}
+          node-version: 24
           cache: npm
       - run: npm install
       - run: npx playwright install chromium

package.json

@@ -18,7 +18,7 @@
     "prebuild": "npm run clean && npm run compile",
     "compile": "tsc",
     "build": "rollup -c",
-    "test": "npm run lint && node test-runner.js",
+    "test": "npm run lint && ./scripts/test-with-server.sh",
     "prepublishOnly": "npm run build",
     "postpublish": "npm publish --ignore-scripts --@github:registry='https://npm.pkg.github.com'"
   },

scripts/test-with-server.sh

@@ -0,0 +1,33 @@
+#!/bin/bash
+
+# Start the WebSocket server in the background
+node -e "
+const ws = require('nodejs-websocket');
+ws.createServer(function (conn) {
+  conn.on('text', function (msg) {
+    if (msg.startsWith('echo:')) {
+      conn.sendText(msg.replace('echo:', ''));
+    } else if (msg.startsWith('close:')) {
+      const code = msg.replace('close:', '');
+      conn.close(parseInt(code), 'reason');
+    }
+  });
+  conn.on('error', function (error) {
+    if (error.code !== 'ECONNRESET') throw error;
+  });
+}).listen(7999, () => console.log('WebSocket server ready'));
+" &
+
+SERVER_PID=$!
+
+# Wait for server to be ready
+sleep 1
+
+# Run vitest
+npx vitest run
+TEST_EXIT=$?
+
+# Kill the server
+kill $SERVER_PID 2>/dev/null
+
+exit $TEST_EXIT

test-runner.js

@@ -7,11 +7,6 @@
   "env": {
     "browser": true
   },
-  "globals": {
-    "describe": "readonly",
-    "it": "readonly",
-    "expect": "readonly"
-  },
   "rules": {
     "import/named": "off"
   }

test/.eslintrc.json

@@ -12,7 +12,7 @@ export default defineConfig({
         },
       ],
     },
-    include: ['test/**/test.js'],
+    include: ['test/test*.js'],
     testTimeout: 10000,
   },
 })