fix: detect dev server URLs with ANSI color codes

- Add stripAnsiCodes() to remove color escape codes from output
- Reorganize URL patterns by server type for better maintainability
- Process output line-by-line to handle chunked stdout
- Support Vite Unicode arrow (➜) with proper regex flag

Fixes issue where Vite and other dev servers output ANSI color codes
even with FORCE_COLOR=0, preventing URL detection. Now strips color
codes before pattern matching.

URL patterns are now organized into separate constants:
- VITE_PATTERNS (Unicode and ASCII arrows)
- CRA_PATTERNS (Create React App)
- NEXTJS_PATTERNS (Next.js)
- GENERIC_PATTERNS (fallback for custom servers)

This makes it easy to add support for new dev servers.

Work Item: W-20274055

Author: deepu-mungamuri94

src/server/DevServerManager.ts

@@ -24,25 +24,38 @@ import { Logger } from '../utils/Logger.js';
  * URL detection patterns for various dev servers
  * These patterns extract URLs from different dev server outputs
  */
-const URL_PATTERNS = [
-  // Vite with Unicode arrow: "  ➜  Local:   http://localhost:5173/"
-  /\s*[➜>]\s*Local:\s+(https?:\/\/[^\s]+)/i,
-  // Generic Local pattern with optional prefix: "Local:   http://localhost:5173"
-  /\s*Local:\s+(https?:\/\/[^\s]+)/i,
-  // Create React App: "On Your Network:  http://192.168.1.1:3000"
-  /On Your Network:\s+(https?:\/\/[^\s]+)/i,
-  // Next.js: "ready - started server on 0.0.0.0:3000, url: http://localhost:3000"
-  /url:\s+(https?:\/\/[^\s,]+)/i,
-  // Next.js alternative: "- Local:        http://localhost:3000"
-  /-\s*Local:\s+(https?:\/\/[^\s]+)/i,
-  // Generic: "Server running at http://localhost:8080"
+/**
+ * URL detection patterns organized by dev server type
+ * Add new server patterns here for easy maintenance
+ */
+
+// Vite dev server patterns
+// Example: "  ➜  Local:   http://localhost:5173/"
+const VITE_PATTERNS = [
+  /➜\s*Local:\s+(https?:\/\/[^\s]+)/iu, // Unicode arrow (with colors)
+  />\s*Local:\s+(https?:\/\/[^\s]+)/i, // ASCII arrow fallback
+];
+
+// Create React App (Webpack) patterns
+// Example: "On Your Network:  http://192.168.1.1:3000"
+const CRA_PATTERNS = [/On Your Network:\s+(https?:\/\/[^\s]+)/i, /Local:\s+(https?:\/\/[^\s]+)/i];
+
+// Next.js dev server patterns
+// Example: "ready - started server on 0.0.0.0:3000, url: http://localhost:3000"
+const NEXTJS_PATTERNS = [/url:\s+(https?:\/\/[^\s,]+)/i, /-\s*Local:\s+(https?:\/\/[^\s]+)/i];
+
+// Generic patterns for custom/unknown servers
+// Example: "Server running at http://localhost:8080"
+const GENERIC_PATTERNS = [
   /(?:Server|server|Running|running|started|Started).*?(https?:\/\/[^\s]+)/i,
-  // Port-based: "localhost:3000" or "http://localhost:3000"
   /(https?:\/\/localhost:[0-9]+)/i,
-  // Generic URL anywhere in the output
   /(https?:\/\/(?:localhost|127\.0\.0\.1|0\.0\.0\.0):[0-9]+)/i,
 ];
 
+// Combined patterns in priority order
+// Specific patterns first, generic fallbacks last
+const URL_PATTERNS = [...VITE_PATTERNS, ...CRA_PATTERNS, ...NEXTJS_PATTERNS, ...GENERIC_PATTERNS];
+
 /**
  * Default configuration values for DevServerManager
  */
@@ -134,23 +147,47 @@ export class DevServerManager extends EventEmitter {
   }
 
   /**
-   * Detects a dev server URL from process output
+   * Strips ANSI color codes from a string
    *
-   * Tries multiple regex patterns to match various dev server formats
-   * (Vite, Create React App, Next.js, etc.)
+   * @param text - Text with potential ANSI codes
+   * @returns Clean text without ANSI codes
+   */
+  private static stripAnsiCodes(text: string): string {
+    // eslint-disable-next-line no-control-regex
+    return text.replace(/\x1B\[[0-9;]*[a-zA-Z]/g, '');
+  }
+
+  /**
+   * Detects dev server URL from process output
    *
-   * @param output The output string to analyze
-   * @returns Detected URL or null if not found
+   * Attempts to match common dev server URL patterns like Vite,
+   * Create React App, Next.js, etc. Processes line-by-line for robustness.
+   *
+   * @param output - The output string to search for URLs
+   * @returns Detected URL or null if none found
    */
   private static detectUrlFromOutput(output: string): string | null {
-    for (const pattern of URL_PATTERNS) {
-      const match = output.match(pattern);
-      if (match?.[1]) {
-        const url = match[1].trim();
-        // Normalize 0.0.0.0 to localhost for better usability
-        return url.replace('0.0.0.0', 'localhost');
+    // Split by newlines and check each line separately
+    // This is more robust against chunked output
+    const lines = output.split('\n');
+
+    for (const line of lines) {
+      // Strip ANSI color codes first (some tools ignore FORCE_COLOR=0)
+      const cleanLine = DevServerManager.stripAnsiCodes(line);
+      const trimmedLine = cleanLine.trim();
+      if (!trimmedLine) continue;
+
+      for (const pattern of URL_PATTERNS) {
+        const match = trimmedLine.match(pattern);
+
+        if (match?.[1]) {
+          const url = match[1].trim();
+          // Normalize 0.0.0.0 to localhost for better usability
+          return url.replace('0.0.0.0', 'localhost');
+        }
       }
     }
+
     return null;
   }