fix(gateway): resolve flaky tests from port conflicts and shared state (#75)

* fix(gateway): use OS-assigned port in app.test.ts to avoid EADDRINUSE

Fixes #72

The integration tests in app.test.ts were using the default port 3100,
which conflicts with sdk.integration.test.ts when running in parallel.

Changes:
- Set PORT=0 in vi.hoisted block to get an OS-assigned port
- Add afterAll hook to properly close the server after tests
- Import server export for cleanup

* fix(gateway): isolate tests with shared state using forks pool

Fixes #70

The concurrency module has shared state that was causing "socket hang up"
errors when concurrency.test.ts ran in parallel with integration tests.

Changes:
- Use poolMatchGlobs to run integration and concurrency tests in forks pool
- Other tests continue to run in parallel threads pool for speed
- Forks pool provides process isolation, preventing shared state conflicts

Author: matthew-petty

packages/gateway/__tests__/integration/app.test.ts

@@ -7,18 +7,28 @@
 
 import { spawn } from "node:child_process";
 import request from "supertest";
-import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+import {
+	afterAll,
+	afterEach,
+	beforeEach,
+	describe,
+	expect,
+	it,
+	vi,
+} from "vitest";
 import {
 	afterSpawnCalled,
 	createCliResultJson,
 	createMockChildProcess,
 } from "../helpers.js";
 
-// Set required environment variable BEFORE any imports
+// Set required environment variables BEFORE any imports
 // Using vi.hoisted to ensure this runs at the earliest possible time
 const TEST_API_KEY = vi.hoisted(() => {
 	const key = "test-api-key-12345";
 	process.env.CLAUDE_CODE_GATEWAY_API_KEY = key;
+	// Use port 0 to let OS assign an available port (avoids conflicts in parallel runs)
+	process.env.PORT = "0";
 	return key;
 });
 
@@ -29,12 +39,19 @@ vi.mock("node:child_process", () => ({
 
 const mockSpawn = vi.mocked(spawn);
 
-// Import app after env var is set
-import app from "../../src/index.js";
+// Import app and server after env var is set
+import app, { server } from "../../src/index.js";
 
 describe("Claude Code Wrapper App (Integration)", () => {
 	const validAuthHeader = `Bearer ${TEST_API_KEY}`;
 
+	afterAll(async () => {
+		// Close the server to prevent resource leaks and port conflicts
+		await new Promise<void>((resolve, reject) => {
+			server.close((err) => (err ? reject(err) : resolve()));
+		});
+	});
+
 	beforeEach(() => {
 		vi.clearAllMocks();
 	});

packages/gateway/vitest.config.ts

@@ -8,6 +8,16 @@ export default defineConfig({
 		setupFiles: ["./__tests__/setup.ts"],
 		include: ["__tests__/**/*.test.ts"],
 		exclude: ["__tests__/e2e/**/*.test.ts"],
+		// Use forks pool for tests with shared state (concurrency module, server ports)
+		// to isolate them from parallel execution. Other tests run in threads pool.
+		// This fixes flaky tests from #70 and #72.
+		pool: "threads",
+		poolMatchGlobs: [
+			// Integration tests start actual servers and need isolation
+			["**/__tests__/integration/**", "forks"],
+			// Concurrency tests modify shared module state
+			["**/__tests__/concurrency.test.ts", "forks"],
+		],
 		coverage: {
 			provider: "v8",
 			reporter: ["text", "lcov"],