fix(test): allocates dynamic port for E2E to avoid collisions

Author: wgordon17

package.json

@@ -12,7 +12,7 @@
     "test:watch": "vitest --config vitest.config.ts",
     "deploy": "wrangler deploy",
     "typecheck": "tsc --noEmit",
-    "test:e2e": "playwright test"
+    "test:e2e": "E2E_PORT=$(node -e \"const s=require('net').createServer();s.listen(0,()=>{console.log(s.address().port);s.close()})\") playwright test"
   },
   "dependencies": {
     "@octokit/core": "^7.0.6",

playwright.config.ts

@@ -1,10 +1,12 @@
 import { defineConfig, devices } from "@playwright/test";
 
+const port = Number(process.env.E2E_PORT) || 5173;
+
 export default defineConfig({
   testDir: "./e2e",
   reporter: [["html", { open: "never" }]],
   use: {
-    baseURL: "http://localhost:5173",
+    baseURL: `http://localhost:${port}`,
     trace: "on-first-retry",
   },
   projects: [
@@ -14,8 +16,8 @@ export default defineConfig({
     },
   ],
   webServer: {
-    command: "pnpm dev",
-    port: 5173,
+    command: `pnpm exec vite dev --port ${port}`,
+    url: `http://localhost:${port}`,
     timeout: 120_000,
     reuseExistingServer: !process.env.CI,
   },