feat(cli): add e2e testing support for login flow

- Add auto-login mechanism via CODEBUFF_E2E_NO_BROWSER flag
- Add file-based IPC for login URL coordination (CODEBUFF_E2E_URL_FILE)
- Add getWebsiteUrl() for dynamic URL resolution in tests
- Support process.env override for CB_ENVIRONMENT in getConfigDir()

Author: brandonkachen

cli/src/components/login-modal.tsx

@@ -198,6 +198,29 @@ export const LoginModal = ({
     }
   }, [hasOpenedBrowser, loginUrl, copyToClipboard])
 
+  // E2E auto-login: automatically trigger login URL fetch without waiting for Enter key
+  // This is needed because OpenTUI keyboard events don't work reliably in PTY testing
+  // The auto-login only activates when CODEBUFF_E2E_NO_BROWSER=true
+  const hasTriggeredAutoLogin = useRef(false)
+  
+  useEffect(() => {
+    const isE2EMode = process.env.CODEBUFF_E2E_NO_BROWSER === 'true'
+    
+    if (!isE2EMode) return
+    if (hasTriggeredAutoLogin.current) return
+    if (hasOpenedBrowser || loading) return
+    
+    // Mark as triggered immediately to prevent double-triggering
+    hasTriggeredAutoLogin.current = true
+    
+    // Small delay to ensure component is fully mounted
+    const timer = setTimeout(() => {
+      fetchLoginUrlAndOpenBrowser()
+    }, 1000)
+    
+    return () => clearTimeout(timer)
+  }, [hasOpenedBrowser, loading, fetchLoginUrlAndOpenBrowser])
+
   // Calculate terminal width and height for responsive display
   const terminalWidth = renderer?.width || 80
   const terminalHeight = renderer?.height || 24

cli/src/hooks/use-fetch-login-url.ts

@@ -1,9 +1,41 @@
+import fs from 'fs'
+
 import { useMutation } from '@tanstack/react-query'
 import open from 'open'
 
-import { WEBSITE_URL } from '../login/constants'
 import { generateLoginUrl } from '../login/login-flow'
 import { logger } from '../utils/logger'
+import { getWebsiteUrl } from '../login/constants'
+
+/**
+ * Check if we should skip browser opening for e2e tests.
+ * When CODEBUFF_E2E_NO_BROWSER=true, we print the URL instead of opening browser.
+ */
+function shouldSkipBrowserOpen(): boolean {
+  return process.env.CODEBUFF_E2E_NO_BROWSER === 'true'
+}
+
+/**
+ * Write login URL status to coordination file for e2e tests.
+ * This provides reliable IPC between CLI and test runner.
+ */
+function writeE2ELoginStatus(status: 'pending' | 'ready' | 'error', data: { loginUrl?: string; error?: string }): void {
+  const e2eUrlFile = process.env.CODEBUFF_E2E_URL_FILE
+  if (!e2eUrlFile) return
+  
+  try {
+    const payload = {
+      status,
+      loginUrl: data.loginUrl,
+      error: data.error,
+      timestamp: Date.now(),
+    }
+    fs.writeFileSync(e2eUrlFile, JSON.stringify(payload, null, 2))
+  } catch (err) {
+    // Don't fail the login flow if we can't write the coordination file
+    logger.debug({ err, e2eUrlFile }, 'Failed to write e2e login status file')
+  }
+}
 
 interface UseFetchLoginUrlParams {
   setLoginUrl: (url: string | null) => void
@@ -27,12 +59,25 @@ export function useFetchLoginUrl({
 }: UseFetchLoginUrlParams) {
   const fetchLoginUrlMutation = useMutation({
     mutationFn: async (fingerprintId: string) => {
+      // Get website URL dynamically to support e2e tests with custom server URLs
+      const baseUrl = getWebsiteUrl()
+      
+      // Debug logging for e2e tests
+      if (process.env.CODEBUFF_E2E_NO_BROWSER === 'true') {
+        process.stderr.write(`[E2E_FETCH] Starting mutation, baseUrl=${baseUrl}\n`)
+      }
+      
+      logger.debug({ baseUrl }, 'Fetching login URL')
+      
+      // Write 'pending' status for e2e tests to confirm mutation was triggered
+      writeE2ELoginStatus('pending', {})
+      
       return generateLoginUrl(
         {
           logger,
         },
         {
-          baseUrl: WEBSITE_URL,
+          baseUrl,
           fingerprintId,
         },
       )
@@ -44,6 +89,12 @@ export function useFetchLoginUrl({
       setIsWaitingForEnter(true)
       setHasOpenedBrowser(true)
 
+      // In e2e test mode, write URL to coordination file for reliable IPC
+      if (shouldSkipBrowserOpen()) {
+        writeE2ELoginStatus('ready', { loginUrl: data.loginUrl })
+        return
+      }
+
       // Open browser after fetching URL
       try {
         await open(data.loginUrl)
@@ -53,7 +104,14 @@ export function useFetchLoginUrl({
       }
     },
     onError: (err) => {
-      setError(err instanceof Error ? err.message : 'Failed to get login URL')
+      const errorMessage = err instanceof Error ? err.message : 'Failed to get login URL'
+      setError(errorMessage)
+      
+      // In e2e test mode, write error to coordination file
+      if (shouldSkipBrowserOpen()) {
+        writeE2ELoginStatus('error', { error: errorMessage })
+      }
+      
       logger.error(
         {
           error: err instanceof Error ? err.message : String(err),

cli/src/hooks/use-login-keyboard-handlers.ts

@@ -27,6 +27,11 @@ export function useLoginKeyboardHandlers({
   useKeyboard(
     useCallback(
       (key: KeyEvent) => {
+        // Debug: log ALL key events in e2e mode
+        if (process.env.CODEBUFF_E2E_NO_BROWSER === 'true') {
+          process.stderr.write(`[E2E_KEY] Received key: ${key.name}, loading=${loading}, hasOpenedBrowser=${hasOpenedBrowser}\n`)
+        }
+        
         const isEnter =
           (key.name === 'return' || key.name === 'enter') &&
           !key.ctrl &&

cli/src/login/constants.ts

@@ -1,8 +1,23 @@
 import { env } from '@codebuff/common/env'
 
 // Get the website URL from environment or use default
+// This is the static version - prefer getWebsiteUrl() for dynamic access
 export const WEBSITE_URL = env.NEXT_PUBLIC_CODEBUFF_APP_URL
 
+/**
+ * Get website URL dynamically from process.env.
+ * This is needed for e2e tests where the URL is set per-process
+ * and the static WEBSITE_URL constant is evaluated at module load time.
+ */
+export function getWebsiteUrl(): string {
+  // Check process.env first (for e2e tests with custom server)
+  if (process.env.NEXT_PUBLIC_CODEBUFF_APP_URL) {
+    return process.env.NEXT_PUBLIC_CODEBUFF_APP_URL
+  }
+  // Fall back to the statically parsed env
+  return WEBSITE_URL
+}
+
 // Codebuff ASCII Logo - compact version for 80-width terminals
 export const LOGO = `
   ██████╗ ██████╗ ██████╗ ███████╗██████╗ ██╗   ██╗███████╗███████╗

cli/src/utils/auth.ts

@@ -30,13 +30,17 @@ const credentialsSchema = z
 
 // Get the config directory path
 export const getConfigDir = (): string => {
+  // Use process.env directly for e2e tests where environment is set per-process
+  // Fall back to parsed env for normal operation
+  const cbEnvironment = process.env.NEXT_PUBLIC_CB_ENVIRONMENT || env.NEXT_PUBLIC_CB_ENVIRONMENT
+  
   return path.join(
     os.homedir(),
     '.config',
     'manicode' +
       // on a development stack?
-      (env.NEXT_PUBLIC_CB_ENVIRONMENT !== 'prod'
-        ? `-${env.NEXT_PUBLIC_CB_ENVIRONMENT}`
+      (cbEnvironment !== 'prod'
+        ? `-${cbEnvironment}`
         : ''),
   )
 }