refactor: rename use_popup_auth to use_redirect_auth and negate logic

Signed-off-by: Patrick Chin <8509935+thepatrickchin@users.noreply.github.com>

Author: thepatrickchin

components/Chat/Chat.tsx

@@ -571,7 +571,7 @@ export const Chat = () => {
           return false;
         }
 
-        const shouldUsePopup = message.content?.use_popup !== false;
+        const shouldUsePopup = !message.content?.use_redirect;
         if (shouldUsePopup) {
           const popup = window.open(
             oauthUrl,
@@ -822,7 +822,7 @@ export const Chat = () => {
         if (oauthUrl) {
           // Validate URL before opening to prevent Open Redirect attacks
           if (isValidConsentPromptURL(oauthUrl)) {
-            const shouldUsePopup = message?.content?.use_popup !== false;
+            const shouldUsePopup = !message?.content?.use_redirect;
             if (shouldUsePopup) {
               if (oauthPopupCancelledRef.current) return;
               // Open the validated OAuth URL in a new tab

types/websocket.ts

@@ -44,7 +44,7 @@ export interface SystemInteractionMessage extends WebSocketMessageBase {
     text?: string;
     timeout?: number | null;
     error?: string | null;
-    use_popup?: boolean;
+    use_redirect?: boolean;
   };
   thread_id?: string;
 }