fix: resolve lint errors from merge

- Properly type proxy dispatcher with explicit return types
- Extract port validation to separate function to fix max-lines-per-function
- Improve type safety in proxy.ts without using eslint-disable

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>

Author: Louis454545

src/lib/proxy.ts

@@ -6,31 +6,24 @@ export function initProxyFromEnv(): void {
   if (typeof Bun !== "undefined") return
 
   try {
-    const direct = new Agent()
+    const direct = new Agent() as Dispatcher
     const proxies = new Map<string, ProxyAgent>()
 
-    // We only need a minimal dispatcher that implements `dispatch` at runtime.
-    // Typing the object as `Dispatcher` forces TypeScript to require many
-    // additional methods. Instead, keep a plain object and cast when passing
-    // to `setGlobalDispatcher`.
-    const dispatcher = {
+    const dispatcher: Dispatcher = {
       dispatch(
         options: Dispatcher.DispatchOptions,
         handler: Dispatcher.DispatchHandler,
-      ) {
+      ): boolean {
         try {
           const origin =
             typeof options.origin === "string" ?
               new URL(options.origin)
             : (options.origin as URL)
-          const get = getProxyForUrl as unknown as (
-            u: string,
-          ) => string | undefined
-          const raw = get(origin.toString())
+          const raw = getProxyForUrl(origin.toString())
           const proxyUrl = raw && raw.length > 0 ? raw : undefined
           if (!proxyUrl) {
             consola.debug(`HTTP proxy bypass: ${origin.hostname}`)
-            return (direct as unknown as Dispatcher).dispatch(options, handler)
+            return direct.dispatch(options, handler)
           }
           let agent = proxies.get(proxyUrl)
           if (!agent) {
@@ -45,20 +38,20 @@ export function initProxyFromEnv(): void {
             /* noop */
           }
           consola.debug(`HTTP proxy route: ${origin.hostname} via ${label}`)
-          return (agent as unknown as Dispatcher).dispatch(options, handler)
+          return (agent as Dispatcher).dispatch(options, handler)
         } catch {
-          return (direct as unknown as Dispatcher).dispatch(options, handler)
+          return direct.dispatch(options, handler)
         }
       },
-      close() {
+      close(): Promise<void> {
         return direct.close()
       },
-      destroy() {
+      destroy(): Promise<void> {
         return direct.destroy()
       },
     }
 
-    setGlobalDispatcher(dispatcher as unknown as Dispatcher)
+    setGlobalDispatcher(dispatcher)
     consola.debug("HTTP proxy configured from environment (per-URL)")
   } catch (err) {
     consola.debug("Proxy setup skipped:", err)

src/start.ts

@@ -29,6 +29,31 @@ interface RunServerOptions {
   proxyEnv: boolean
 }
 
+async function validatePort(port: number): Promise<void> {
+  try {
+    const testServer = await import("node:net").then((net) =>
+      net.createServer(),
+    )
+    await new Promise<void>((resolve, reject) => {
+      testServer.listen(port, () => {
+        testServer.close(() => resolve())
+      })
+      testServer.on("error", (err: NodeJS.ErrnoException) => {
+        if (err.code === "EADDRINUSE") {
+          reject(new Error(`Port ${port} is already in use`))
+        } else {
+          reject(err)
+        }
+      })
+    })
+  } catch (error: unknown) {
+    const errorMessage =
+      error instanceof Error ? error.message : "Unknown error occurred"
+    consola.error(errorMessage)
+    process.exit(1)
+  }
+}
+
 export async function runServer(options: RunServerOptions): Promise<void> {
   if (options.proxyEnv) {
     initProxyFromEnv()
@@ -118,30 +143,7 @@ export async function runServer(options: RunServerOptions): Promise<void> {
     `🌐 Usage Viewer: https://ericc-ch.github.io/copilot-api?endpoint=${serverUrl}/usage`,
   )
 
-  // Check if port is already in use
-  try {
-    // Try to create a temporary server to test the port
-    const testServer = await import("node:net").then((net) =>
-      net.createServer(),
-    )
-    await new Promise<void>((resolve, reject) => {
-      testServer.listen(options.port, () => {
-        testServer.close(() => resolve())
-      })
-      testServer.on("error", (err: NodeJS.ErrnoException) => {
-        if (err.code === "EADDRINUSE") {
-          reject(new Error(`Port ${options.port} is already in use`))
-        } else {
-          reject(err)
-        }
-      })
-    })
-  } catch (error: unknown) {
-    const errorMessage =
-      error instanceof Error ? error.message : "Unknown error occurred"
-    consola.error(errorMessage)
-    process.exit(1)
-  }
+  await validatePort(options.port)
 
   serve({
     fetch: server.fetch as ServerHandler,