Fix ApiDevTools isDev duplicate

Resolve TS2451 redeclare by removing duplicate isDev, fix fetch patch scope, and restore proper closure. Also improve login error logging to show dev details without breaking prod.

X-Lovable-Edit-ID: edt-ee281f89-34b9-41ce-ae0a-1ed1dcece8ab

src/components/dev/ApiDevTools.tsx

@@ -150,9 +150,7 @@ if (isDev && !globalAny.__gatehouseFetchPatched) {
   };
 }
 
-// Check if we're in development mode
-const isDev = import.meta.env.DEV;
-
+// isDev is already declared at module level for the fetch patch
 export default function ApiDevTools() {
   const [isOpen, setIsOpen] = useState(false);
   const [logs, setLogs] = useState<ApiLog[]>([...apiLogs]);

src/pages/auth/LoginPage.tsx

@@ -25,7 +25,14 @@ export default function LoginPage() {
     try {
       await login(email, password, rememberMe);
     } catch (error) {
+      // Log to console in dev mode for easier debugging
+      if (import.meta.env.DEV) {
+        console.error("[Gatehouse] Login failed:", error);
+      }
+
       const message = error instanceof ApiError 
+        ? error.message 
+        : import.meta.env.DEV && error instanceof Error
           ? error.message
           : "An unexpected error occurred";