fix: open handles

Author: Junyi-99

jest.config.js

@@ -29,5 +29,9 @@ module.exports = {
             lines: 70,
             statements: 70
         }
-    }
+    },
+    // Help detect open handles that prevent Jest from exiting
+    detectOpenHandles: true,
+    // Force exit after tests complete (as a safety net)
+    forceExit: true
 };

src/app.test.ts

@@ -2,9 +2,20 @@
 
 import request from 'supertest';
 // import express from 'express';
-import { app } from './app';
+import { app, server } from './app';
 
 describe('Express App Tests', () => {
+  // Clean up after all tests
+  afterAll(async () => {
+    if (server) {
+      await new Promise<void>((resolve) => {
+        server.close(() => {
+          resolve();
+        });
+      });
+    }
+  });
+
   // Root endpoint test
   describe('GET /', () => {
     it('should return Hello, TypeScript Express!', async () => {

src/app.ts

@@ -10,6 +10,7 @@ import { loadPaperScoreCriteria } from './agents/paper-score.1.criteria.schema';
 import { joinSuggestions, PaperScoreResultSchema } from './agents/paper-score.schema';
 import { z } from 'zod';
 import { asyncHandler } from './middleware/error-handler';
+import { Server } from 'http';
 
 export const app = express();
 const port = process.env['PORT'] || 8000;
@@ -145,10 +146,16 @@ app.use((err: Error, _: Request, res: Response, __: express.NextFunction) => {
   res.status(500).json(response);
 });
 
-app
-  .listen(port, () => {
-    console.log(`app listening on port ${port}`);
-  })
-  .on('error', (err: any) => {
-    console.error('Failed to start server:', err);
-  });
+// Export server instance for testing
+export let server: Server;
+
+// Only start the server if this file is run directly (not imported)
+if (require.main === module) {
+  server = app
+    .listen(port, () => {
+      console.log(`app listening on port ${port}`);
+    })
+    .on('error', (err: any) => {
+      console.error('Failed to start server:', err);
+    });
+}