ci: add unit test step to workflow

.github/workflows/test.yml

@@ -19,12 +19,6 @@ jobs:
       - name: Install Node
         uses: actions/setup-node@949feb2413d6458794dcd2491c4babbbce0c15c1 # v7.1.0
         with:
-          # Pin to the version the app actually ships on (Dockerfile /
-          # Dockerfile.dev both use 24.18.1; .nvmrc pins the 24 major).
-          # A bare '24' floats to the latest 24.x; Node >= 24.19.0 added
-          # node::ObjectWrap cleanup hooks that crash better-sqlite3 on teardown
-          # ("Assertion failed: (env) != nullptr" in RemoveEnvironmentCleanupHook),
-          # which intermittently kills `npm run dev` in the step below.
           node-version: '24.18.1'
 
       - name: Copy config file
@@ -48,6 +42,9 @@ jobs:
       - name: Test with tsc
         run: npx tsc --noEmit
 
+      - name: Run unit tests
+        run: npm test
+
       - name: Start app in background
         run: |
           nohup npm run dev > app.log 2>&1 &
@@ -63,9 +60,6 @@ jobs:
               echo "App is up"
               exit 0
             fi
-            # Fail fast (and show why) if the process is gone. Read the PID
-            # safely: if it's missing/empty (e.g. nohup never started), treat
-            # that as "not running" instead of passing a bad arg to kill.
             pid="$(cat app.pid 2>/dev/null || true)"
             if [ -z "$pid" ] || ! kill -0 "$pid" 2>/dev/null; then
               echo "App process exited before becoming available. Output:"