fix

Author: sorryhyun

backend/launcher.py

@@ -23,6 +23,9 @@
 import webbrowser
 from pathlib import Path
 
+# Global flag to prevent duplicate browser opens
+_browser_opened = False
+
 
 def get_base_path() -> Path:
     """Get the base path for resources (handles both dev and bundled modes)."""
@@ -214,9 +217,21 @@ def is_tauri_sidecar() -> bool:
 
 
 def open_browser_delayed(url: str, delay: float = 1.5):
-    """Open browser after a delay to allow server to start."""
+    """Open browser after a delay to allow server to start.
+
+    Uses a global flag to prevent duplicate browser opens which can happen
+    when Claude SDK or Codex subprocess triggers browser behavior on Windows.
+    """
+    global _browser_opened
+
+    if _browser_opened:
+        return
+
+    _browser_opened = True
+
     def _open():
         time.sleep(delay)
+        # Double-check flag in case of race condition
         webbrowser.open(url)
 
     thread = threading.Thread(target=_open, daemon=True)
@@ -230,7 +245,7 @@ def run_mcp_server(server_type: str) -> None:
     allowing Codex to spawn this exe as an MCP server subprocess.
 
     Args:
-        server_type: Either "action" or "guidelines"
+        server_type: Either "action", "guidelines", or "etc"
     """
     import asyncio
 
@@ -241,9 +256,11 @@ def run_mcp_server(server_type: str) -> None:
         from mcp_servers.action_server import main as server_main
     elif server_type == "guidelines":
         from mcp_servers.guidelines_server import main as server_main
+    elif server_type == "etc":
+        from mcp_servers.etc_server import main as server_main
     else:
         print(f"Unknown MCP server type: {server_type}", file=sys.stderr)
-        print("Valid types: action, guidelines", file=sys.stderr)
+        print("Valid types: action, guidelines, etc", file=sys.stderr)
         sys.exit(1)
 
     # Run the MCP server (async main)

backend/providers/codex/transport.py

@@ -98,12 +98,15 @@ async def start(self) -> None:
 
         logger.info(f"[Transport {self._instance_id}] Starting: {' '.join(self._command)}")
 
+        # Build environment with BROWSER="" to prevent subprocess from opening browser
+        subprocess_env = {**os.environ, "BROWSER": ""}
+
         self._process = await asyncio.create_subprocess_exec(
             *self._command,
             stdin=asyncio.subprocess.PIPE,
             stdout=asyncio.subprocess.PIPE,
             stderr=asyncio.subprocess.PIPE,
-            env={**os.environ},
+            env=subprocess_env,
         )
 
         self._reader_task = asyncio.create_task(self._read_stdout())

backend/providers/configs.py

@@ -40,6 +40,8 @@ class ClaudeStaticConfig:
             "DISABLE_TELEMETRY": "true",
             "DISABLE_ERROR_REPORTING": "true",
             "CLAUDE_CODE_DISABLE_FEEDBACK_SURVEY": "true",
+            # Prevent subprocess from opening browser (Windows fix)
+            "BROWSER": "",
         }
     )
 

backend/providers/mcp_config.py

@@ -82,16 +82,30 @@ def build_all_servers(
         # Build guidelines server config
         guidelines_env = MCPConfigBuilder._build_guidelines_env(base_env, env_config)
 
+        # When running as PyInstaller bundle, use --mcp-server flag instead of -m
+        is_frozen = getattr(sys, "frozen", False)
+
+        if is_frozen:
+            # PyInstaller bundle: use --mcp-server flag that launcher.py understands
+            action_args = ["--mcp-server", "action"]
+            guidelines_args = ["--mcp-server", "guidelines"]
+            etc_args = ["--mcp-server", "etc"]
+        else:
+            # Development: use Python module invocation
+            action_args = ["-m", "mcp_servers.action_server"]
+            guidelines_args = ["-m", "mcp_servers.guidelines_server"]
+            etc_args = ["-m", "mcp_servers.etc_server"]
+
         servers: Dict[str, MCPServerConfig] = {
             "action": {
                 "command": python_exe,
-                "args": ["-m", "mcp_servers.action_server"],
+                "args": action_args,
                 "env": action_env,
                 "cwd": backend_dir,
             },
             "guidelines": {
                 "command": python_exe,
-                "args": ["-m", "mcp_servers.guidelines_server"],
+                "args": guidelines_args,
                 "env": guidelines_env,
                 "cwd": backend_dir,
             },
@@ -101,7 +115,7 @@ def build_all_servers(
         if include_etc:
             servers["etc"] = {
                 "command": python_exe,
-                "args": ["-m", "mcp_servers.etc_server"],
+                "args": etc_args,
                 "env": base_env.copy(),
                 "cwd": backend_dir,
             }