fix: resolve CI issues with missing generated files and non-null assertions

- Add schema generation step before typecheck in CI workflow
- Fix non-null assertions in api-server.test.ts by using optional checks
- Ensure generated files are available before TypeScript compilation

Author: Mearman

.github/workflows/main.yml

@@ -100,6 +100,10 @@ jobs:
             git push
           fi
         
+      # Generate required files before quality checks
+      - name: Generate schemas (required for TypeScript compilation)
+        run: npm run generate:schemas
+        
       # Quality checks (must pass for all further steps)
       - name: Run lint (fast fail)
         run: npm run lint

src/api-server.test.ts

@@ -62,9 +62,11 @@ describe('API Server', () => {
       server = createApiServer(port);
 
       await new Promise<void>((resolve) => {
-        server!.on('listening', () => {
-          resolve();
-        });
+        if (server) {
+          server.on('listening', () => {
+            resolve();
+          });
+        }
       });
 
       expect(server.listening).toBe(true);
@@ -95,7 +97,9 @@ describe('API Server', () => {
     beforeEach(async () => {
       server = createApiServer(port);
       await new Promise<void>((resolve) => {
-        server!.on('listening', () => resolve());
+        if (server) {
+          server.on('listening', () => resolve());
+        }
       });
     });
 
@@ -140,7 +144,9 @@ describe('API Server', () => {
     beforeEach(async () => {
       server = createApiServer(port);
       await new Promise<void>((resolve) => {
-        server!.on('listening', () => resolve());
+        if (server) {
+          server.on('listening', () => resolve());
+        }
       });
     });
 
@@ -194,7 +200,9 @@ describe('API Server', () => {
     beforeEach(async () => {
       server = createApiServer(port);
       await new Promise<void>((resolve) => {
-        server!.on('listening', () => resolve());
+        if (server) {
+          server.on('listening', () => resolve());
+        }
       });
     });
 
@@ -241,7 +249,9 @@ describe('API Server', () => {
     beforeEach(async () => {
       server = createApiServer(port);
       await new Promise<void>((resolve) => {
-        server!.on('listening', () => resolve());
+        if (server) {
+          server.on('listening', () => resolve());
+        }
       });
     });
 
@@ -321,7 +331,9 @@ describe('API Server', () => {
     beforeEach(async () => {
       server = createApiServer(port);
       await new Promise<void>((resolve) => {
-        server!.on('listening', () => resolve());
+        if (server) {
+          server.on('listening', () => resolve());
+        }
       });
     });
 
@@ -404,7 +416,9 @@ describe('API Server', () => {
     beforeEach(async () => {
       server = createApiServer(port);
       await new Promise<void>((resolve) => {
-        server!.on('listening', () => resolve());
+        if (server) {
+          server.on('listening', () => resolve());
+        }
       });
     });