browser open only on manual action by user

apps/x/apps/main/src/browser/control-service.ts

@@ -1,18 +1,8 @@
-import { BrowserWindow } from 'electron';
 import type { IBrowserControlService } from '@x/core/dist/application/browser-control/service.js';
 import type { BrowserControlAction, BrowserControlInput, BrowserControlResult } from '@x/shared/dist/browser-control.js';
 import { browserViewManager } from './view.js';
 import { normalizeNavigationTarget } from './navigation.js';
 
-function emitPaneState(open: boolean): void {
-  const windows = BrowserWindow.getAllWindows();
-  for (const win of windows) {
-    if (!win.isDestroyed() && win.webContents) {
-      win.webContents.send('browser:didRequestPaneState', { open });
-    }
-  }
-}
-
 function buildSuccessResult(
   action: BrowserControlAction,
   message: string,
@@ -37,24 +27,18 @@ function buildErrorResult(action: BrowserControlAction, error: string): BrowserC
 }
 
 export class ElectronBrowserControlService implements IBrowserControlService {
-  private ensurePaneOpen(): void {
-    emitPaneState(true);
-    browserViewManager.setVisible(true);
-  }
-
   async execute(
     input: BrowserControlInput,
     ctx?: { signal?: AbortSignal },
   ): Promise<BrowserControlResult> {
     const signal = ctx?.signal;
-    this.ensurePaneOpen();
 
     try {
       switch (input.action) {
         case 'open': {
           await browserViewManager.ensureActiveTabReady(signal);
           const page = await browserViewManager.readPageSummary(signal, { waitForReady: false }) ?? undefined;
-          return buildSuccessResult('open', 'Opened the browser pane.', page);
+          return buildSuccessResult('open', 'Opened a browser session.', page);
         }
 
         case 'get-state':

apps/x/apps/renderer/src/App.tsx

@@ -789,19 +789,6 @@ function App() {
     return cleanup
   }, [refreshVoiceAvailability])
 
-  useEffect(() => {
-    const cleanup = window.ipc.on('browser:didRequestPaneState', (event) => {
-      if (event.open) {
-        setIsBrowserOpen(true)
-        setIsChatSidebarOpen(true)
-        setIsRightPaneMaximized(false)
-        return
-      }
-      setIsBrowserOpen(false)
-    })
-    return cleanup
-  }, [])
-
   const handleStartRecording = useCallback(() => {
     setIsRecording(true)
     isRecordingRef.current = true

apps/x/packages/shared/src/ipc.ts

@@ -708,12 +708,6 @@ const ipcSchemas = {
     req: BrowserStateSchema,
     res: z.null(),
   },
-  'browser:didRequestPaneState': {
-    req: z.object({
-      open: z.boolean(),
-    }),
-    res: z.null(),
-  },
   // Billing channels
   'billing:getInfo': {
     req: z.null(),