🐛 Bugfix: in speed mode, shouldn't skipped authorization

🐛 Bugfix: in speed mode, shouldn't skipped authorization

Author: WMC001

frontend/hooks/auth/useAuthenticationState.ts

@@ -29,10 +29,14 @@ export function useAuthenticationState(): AuthenticationStateReturn {
   const [authServiceUnavailable, setAuthServiceUnavailable] =
     useState<boolean>(false);
 
-  if (isSpeedMode) {
-    log.info("Speed mode: valid session found, emitting login success event");
-    authEventUtils.emitLoginSuccess();
-  }
+  // Speed mode: skip authentication checks, consider user as authenticated
+  useEffect(() => {
+    if (isSpeedMode) {
+      // In speed mode, user is considered authenticated without session
+      setIsAuthenticated(true);
+      setIsAuthChecking(false);
+    }
+  }, [isSpeedMode]);
 
   // Initialize authentication state based on session
   useEffect(() => {

frontend/hooks/auth/useAuthorization.ts

@@ -10,6 +10,7 @@ import { authEvents, authzEvents, authzEventUtils } from "@/lib/authEvents";
 import { AUTH_EVENTS, AUTHZ_EVENTS } from "@/const/auth";
 import { getEffectiveRoutePath } from "@/lib/auth";
 import log from "@/lib/logger";
+import { useDeployment } from "@/components/providers/deploymentProvider";
 
 /**
  * Custom hook for authorization management
@@ -19,6 +20,8 @@ export function useAuthorization(): AuthorizationContextType {
   const router = useRouter();
   const pathname = usePathname();
 
+  const { isSpeedMode } = useDeployment();
+
   const [user, setUser] = useState<User | null>(null);
   const [groupIds, setGroupIds] = useState<number[]>([]);
   const [permissions, setPermissions] = useState<string[]>([]);
@@ -101,6 +104,7 @@ export function useAuthorization(): AuthorizationContextType {
 
   // Listen for authentication events
   useEffect(() => {
+    if (isSpeedMode) return;
     // Handle login success - set user info immediately, then fetch full permissions
     const handleLoginSuccess = () => {
       refetch().then((result) => {
@@ -176,26 +180,35 @@ export function useAuthorization(): AuthorizationContextType {
   // Initialize authorization data on mount if user is already authenticated
   useEffect(() => {
     const initializeAuthz = () => {
-      const session = getSessionFromStorage();
-      if (session?.access_token) {
+      // In speed mode, always fetch authorization info
+      // In full mode, only fetch if session exists
+      if (!isSpeedMode) {
+        const session = getSessionFromStorage();
+        if (!session?.access_token) {
+          return;
+        }
         const now = Date.now();
         const expiresAt = session.expires_at * 1000;
 
-        if (expiresAt > now) {
-          log.info(
-            "Valid session found on initialization, fetching authorization info..."
-          );
-          refetch().catch((error) => {
-            log.error("Initial refetch error:", error);
-          });
+        if (expiresAt <= now) {
+          return;
         }
       }
+
+      log.info(
+        isSpeedMode
+          ? "Speed mode: fetching authorization info..."
+          : "Valid session found on initialization, fetching authorization info..."
+      );
+      refetch().catch((error) => {
+        log.error("Initial refetch error:", error);
+      });
     };
 
     // Small delay to ensure authentication state is initialized
     const timeoutId = setTimeout(initializeAuthz, 100);
     return () => clearTimeout(timeoutId);
-  }, [refetch]);
+  }, [isSpeedMode, refetch]);
 
   // Authz prompt modal control functions (defined before useLayoutEffect)
   const openAuthzPromptModal = useCallback(() => setIsAuthzPromptModalOpen(true), []);

frontend/hooks/auth/useSessionManager.ts

@@ -60,13 +60,15 @@ export const refreshSession = async (): Promise<boolean> => {
   // ============================================================================
 
 export function useSessionManager() {
-  const { isSpeedMode } = useDeployment();
+  const { isSpeedMode, isDeploymentReady } = useDeployment();
   const reconcileExpiryRef = useRef<() => void>(() => {});
 
   // Initialize session management when hook is used
   useEffect(() => {
-    // In speed mode, skip session validation
-    if (isSpeedMode) return;
+    // In speed mode or before deployment is ready, skip session validation
+    // This prevents the modal from showing when isSpeedMode is initially false
+    // and then becomes true after API returns deployment_version
+    if (isSpeedMode || !isDeploymentReady) return;
 
     const session = getSessionFromStorage();
     if (!session) {
@@ -80,14 +82,15 @@ export function useSessionManager() {
 
     // Session is expired or invalid
     handleSessionExpired();
-  }, [isSpeedMode]);
+  }, [isSpeedMode, isDeploymentReady]);
 
   /**
    * Proactive session expiry watcher
    * Triggers session-expired even if user does not make any API request
    */
   useEffect(() => {
-    if (isSpeedMode) return;
+    // In speed mode or before deployment is ready, skip session watching
+    if (isSpeedMode || !isDeploymentReady) return;
 
     let timeoutId: number | null = null;
     let intervalId: number | null = null;