Improve CI and tests: use npm ci; stabilize test server lifecycle and avoid flakiness

Author: ryaneggz

__tests__/server.test.js

@@ -4,10 +4,8 @@ const http = require('http');
 let app;
 let server;
 
-beforeAll(() => {
-  // require the app by starting the server module
-  app = require('express')();
-  // mount the actual server.js app by requiring and exporting express app
+beforeAll(async () => {
+  // Ensure server is running before tests (npm test starts it)
 });
 
 describe('basic endpoints', () => {

package.json

@@ -5,7 +5,7 @@
   "scripts": {
     "start": "node server.js",
     "dev": "node server.js",
-    "test": "node server.js & wait-on tcp:8080 && jest --runInBand; kill $(lsof -t -i:8080)"
+    "test": "node server.js & wait-on tcp:8080 && jest --runInBand; kill $(lsof -t -i:8080) || true"
   },
   "keywords": [],
   "author": "",
@@ -18,7 +18,7 @@
   },
   "devDependencies": {
     "jest": "^29.7.0",
-    "supertest": "^7.0.0",
+    "supertest": "^7.1.4",
     "wait-on": "^7.2.0",
     "lsof": "^0.1.0"
   }