fix(extension): clear stale authentication when returning to Chat

Author: waleedlatif1

apps/sim/app/extension/sign-in.tsx

@@ -52,7 +52,7 @@ function PanelSignIn({ expectedUserId }: ExtensionSignInProps) {
     let active = true
     let checking = false
     let checkAgain = false
-    const resume = async () => {
+    const resume = async (returning = false) => {
       if (document.visibilityState === 'hidden') return
       if (checking) {
         checkAgain = true
@@ -64,8 +64,8 @@ function PanelSignIn({ expectedUserId }: ExtensionSignInProps) {
         if (active) {
           setFailed(false)
           if (session?.user) {
-            if (expectedUserId && session.user.id !== expectedUserId) window.location.reload()
-            else router.refresh()
+            if (!expectedUserId) router.refresh()
+            else if (returning || session.user.id !== expectedUserId) window.location.reload()
           }
         }
       } catch {
@@ -74,14 +74,14 @@ function PanelSignIn({ expectedUserId }: ExtensionSignInProps) {
         checking = false
         if (active && checkAgain) {
           checkAgain = false
-          void resume()
+          void resume(true)
         }
       }
     }
-    const check = () => void resume()
+    const check = () => void resume(true)
     window.addEventListener('focus', check)
     document.addEventListener('visibilitychange', check)
-    check()
+    void resume()
     return () => {
       active = false
       window.removeEventListener('focus', check)

apps/sim/scripts/test-browser-extension-e2e.ts

@@ -1011,7 +1011,7 @@ try {
       await capture()
       await capture()
       await expect(attachments()).toHaveCount(3)
-      await expect(frame().getByRole('alert').filter({ hasText: 'Remove one' })).toBeVisible()
+      await expect(frame().getByRole('alert').filter({ hasText: 'Remove a page' })).toBeVisible()
       await screenshot('page-limit')
       await attachments().first().getByRole('button', { name: 'Remove page context' }).click()
       await expect(attachments()).toHaveCount(2)
@@ -1483,8 +1483,31 @@ try {
         await settingsPage.waitForURL('chrome://settings/content/microphone')
         await settingsPage.close()
         await shell.bringToFront()
+        const viewport = shell.viewportSize()
+        assert(viewport)
+        const { windowId } = await microphone.send('Browser.getWindowForTarget')
+        const before = await microphone.send('Browser.getWindowBounds', { windowId })
+        await setPanelViewport(viewport.width, viewport.height)
+        const after = await microphone.send('Browser.getWindowBounds', { windowId })
+        voiceDiagnostics.push({ event: 'settings-return', viewport, before, after })
         await setMicrophonePermission('granted')
+        const retryClick = await mic.evaluateHandle((button) => {
+          let clicked = false
+          button.addEventListener(
+            'click',
+            (event) => {
+              clicked = event.isTrusted
+            },
+            { once: true }
+          )
+          return () => clicked
+        })
         await mic.click()
+        voiceDiagnostics.push({
+          event: 'retry-click',
+          trusted: await retryClick.evaluate((read) => read()),
+        })
+        await retryClick.dispose()
         await expect(stop).toBeVisible()
         await expect.poll(() => sessions[0]?.chunks ?? 0).toBeGreaterThan(0)
         sessions[0].socket.send(