From 4b776435136581091b2cf2f2b6c3016cbf657dab Mon Sep 17 00:00:00 2001
From: James Barney <Jamesbarney71@gmail.com>
Date: Fri, 7 Nov 2025 20:04:20 -0500
Subject: [PATCH] =?UTF-8?q?Fix=20arcade=20configure=20claude=20to=20use=20?=
 =?UTF-8?q?correct=20path=20on=20Windows=20when=20runni=E2=80=A6=20(#682)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

…ng in WSL

When running `arcade configure claude` in WSL, the configuration file
was being written to the WSL filesystem
(~/.config/Claude/claude_desktop_config.json) instead of the Windows
AppData directory where Claude Desktop actually reads it.

This commit adds:
- WSL detection via WSL_DISTRO_NAME env var and /proc/version
- Windows username retrieval when running in WSL
- Updated config path functions to use Windows paths when in WSL
- Applied the same fix to Cursor and VS Code config paths for
consistency

The fix ensures that when running in WSL, the config file is written to:
/mnt/c/Users/{username}/AppData/Roaming/Claude/claude_desktop_config.json

This allows Claude Desktop on Windows to properly detect and use the MCP
server configuration.

Fixes #681

Co-authored-by: Claude <noreply@anthropic.com>
---
 libs/arcade-cli/arcade_cli/configure.py | 77 +++++++++++++++++++++++++
 1 file changed, 77 insertions(+)

diff --git a/libs/arcade-cli/arcade_cli/configure.py b/libs/arcade-cli/arcade_cli/configure.py
index 093cbbeb..5d195411 100644
--- a/libs/arcade-cli/arcade_cli/configure.py
+++ b/libs/arcade-cli/arcade_cli/configure.py
@@ -5,6 +5,7 @@ import os
 import platform
 import re
 import shutil
+import subprocess
 from pathlib import Path
 
 import typer
@@ -14,6 +15,44 @@ from rich.console import Console
 console = Console()
 
 
+def is_wsl() -> bool:
+    """Check if running in Windows Subsystem for Linux."""
+    # Check for WSL environment variable
+    if os.environ.get("WSL_DISTRO_NAME"):
+        return True
+
+    # Check /proc/version for WSL indicators
+    try:
+        with open("/proc/version") as f:
+            version_info = f.read().lower()
+            return "microsoft" in version_info or "wsl" in version_info
+    except (FileNotFoundError, PermissionError):
+        return False
+
+
+def get_windows_username() -> str | None:
+    """Get the Windows username when running in WSL."""
+    try:
+        # Try to get username from Windows environment via cmd.exe
+        # Note: cmd.exe is safe to use here as it's a Windows system binary available in WSL
+        result = subprocess.run(
+            ["cmd.exe", "/c", "echo", "%USERNAME%"],  # noqa: S607
+            capture_output=True,
+            text=True,
+            timeout=5,
+        )
+        if result.returncode == 0:
+            username = result.stdout.strip()
+            # Remove any carriage returns
+            username = username.replace("\r", "")
+            if username and username != "%USERNAME%":
+                return username
+    except (subprocess.SubprocessError, FileNotFoundError, subprocess.TimeoutExpired):
+        pass
+
+    return None
+
+
 def get_claude_config_path() -> Path:
     """Get the Claude Desktop configuration file path."""
     system = platform.system()
@@ -28,6 +67,20 @@ def get_claude_config_path() -> Path:
     elif system == "Windows":
         return Path(os.environ["APPDATA"]) / "Claude" / "claude_desktop_config.json"
     else:  # Linux
+        # Check if we're in WSL - if so, use Windows path
+        if is_wsl():
+            username = get_windows_username()
+            if username:
+                # Use the Windows AppData path accessible via WSL mount
+                return Path(
+                    f"/mnt/c/Users/{username}/AppData/Roaming/Claude/claude_desktop_config.json"
+                )
+            else:
+                console.print(
+                    "[yellow]Warning: Running in WSL but couldn't determine Windows username. "
+                    "Using Linux path instead. Claude Desktop may not detect this configuration.[/yellow]"
+                )
+
         return Path.home() / ".config" / "Claude" / "claude_desktop_config.json"
 
 
@@ -39,6 +92,18 @@ def get_cursor_config_path() -> Path:
     elif system == "Windows":
         return Path(os.environ["APPDATA"]) / "Cursor" / "mcp.json"
     else:  # Linux
+        # Check if we're in WSL - if so, use Windows path
+        if is_wsl():
+            username = get_windows_username()
+            if username:
+                # Use the Windows AppData path accessible via WSL mount
+                return Path(f"/mnt/c/Users/{username}/AppData/Roaming/Cursor/mcp.json")
+            else:
+                console.print(
+                    "[yellow]Warning: Running in WSL but couldn't determine Windows username. "
+                    "Using Linux path instead. Cursor may not detect this configuration.[/yellow]"
+                )
+
         return Path.home() / ".config" / "Cursor" / "mcp.json"
 
 
@@ -51,6 +116,18 @@ def get_vscode_config_path() -> Path:
     elif system == "Windows":
         return Path(os.environ["APPDATA"]) / "Code" / "User" / "mcp.json"
     else:  # Linux
+        # Check if we're in WSL - if so, use Windows path
+        if is_wsl():
+            username = get_windows_username()
+            if username:
+                # Use the Windows AppData path accessible via WSL mount
+                return Path(f"/mnt/c/Users/{username}/AppData/Roaming/Code/User/mcp.json")
+            else:
+                console.print(
+                    "[yellow]Warning: Running in WSL but couldn't determine Windows username. "
+                    "Using Linux path instead. VS Code may not detect this configuration.[/yellow]"
+                )
+
         return Path.home() / ".config" / "Code" / "User" / "mcp.json"