feat: enable eslint in ci

.github/workflows/ci.yml

@@ -29,10 +29,10 @@ jobs:
         run: |
           echo testing > internal/assets/version
 
-      - name: Build frontend
+      - name: Lint frontend
         run: |
           cd frontend
-          bun run build
+          bun run lint
 
       - name: Copy frontend
         run: |

frontend/src/lib/utils.ts

@@ -9,7 +9,7 @@ export const isValidUrl = (url: string) => {
   try {
     new URL(url);
     return true;
-  } catch (e) {
+  } catch {
     return false;
   }
 };

frontend/src/pages/continue-page.tsx

@@ -76,7 +76,14 @@ export const ContinuePage = () => {
       clearTimeout(auto);
       clearTimeout(reveal);
     };
-  }, []);
+  }, [
+    handleRedirect,
+    isAllowedRedirectProto,
+    isHttpsDowngrade,
+    isLoggedIn,
+    isTrustedRedirectUri,
+    isValidRedirectUri,
+  ]);
 
   if (!isLoggedIn) {
     return (

frontend/src/pages/login-page.tsx

@@ -119,6 +119,8 @@ export const LoginPage = () => {
         !isLoggedIn &&
         redirectUri
       ) {
+        // Not sure of a better way to do this
+        // eslint-disable-next-line react-hooks/set-state-in-effect
         setOauthAutoRedirectHandover(true);
         oauthMutation.mutate(oauthAutoRedirect);
         redirectButtonTimer.current = window.setTimeout(() => {
@@ -126,7 +128,15 @@ export const LoginPage = () => {
         }, 5000);
       }
     }
-  }, []);
+  }, [
+    isMounted,
+    oauthProviders.length,
+    providers,
+    isLoggedIn,
+    redirectUri,
+    oauthAutoRedirect,
+    oauthMutation,
+  ]);
 
   useEffect(
     () => () => {