fix(shell): augment PATH with common tool directories on JVM

Ensure the shell executor prepends common Homebrew and Linuxbrew paths to PATH if not set in config. This improves tool discovery in production builds where the app does not inherit the user's full environment.

Author: phodal

mpp-core/src/jvmMain/kotlin/cc/unitmesh/agent/tool/shell/DefaultShellExecutor.jvm.kt

@@ -69,6 +69,11 @@ actual class DefaultShellExecutor : ShellExecutor {
                     environment().putAll(config.environment)
                 }
 
+                // Augment PATH to include common tool installation directories
+                // This is critical for production builds where the app doesn't inherit
+                // the user's full shell environment (e.g., Homebrew paths)
+                augmentEnvironmentPath(environment(), config.environment)
+                
                 // Redirect error stream if not inheriting IO
                 if (!config.inheritIO) {
                     redirectErrorStream(false)
@@ -191,6 +196,68 @@ actual class DefaultShellExecutor : ShellExecutor {
         }
     }
 
+    /**
+     * Augment the PATH environment variable to include common tool installation directories.
+     * 
+     * This is essential for production builds where the app runs as a standalone bundle
+     * and doesn't inherit the user's full shell environment. Common package managers like
+     * Homebrew install tools in non-standard paths that need to be explicitly added.
+     * 
+     * @param processEnv The process environment map (will be modified)
+     * @param configEnv The user-provided environment from config (should not override if PATH is set)
+     */
+    private fun augmentEnvironmentPath(
+        processEnv: MutableMap<String, String>,
+        configEnv: Map<String, String>
+    ) {
+        // Don't augment if user explicitly set PATH in config
+        if (configEnv.containsKey("PATH")) {
+            return
+        }
+        
+        val os = System.getProperty("os.name").lowercase()
+        val currentPath = processEnv["PATH"] ?: System.getenv("PATH") ?: ""
+        
+        // Determine additional paths based on OS
+        val additionalPaths = when {
+            os.contains("mac") || os.contains("darwin") -> listOf(
+                "/opt/homebrew/bin",        // Homebrew (Apple Silicon)
+                "/opt/homebrew/sbin",
+                "/usr/local/bin",           // Homebrew (Intel)
+                "/usr/local/sbin"
+            )
+            os.contains("linux") -> listOf(
+                "/usr/local/bin",
+                "/home/linuxbrew/.linuxbrew/bin",  // Linuxbrew
+                "/home/linuxbrew/.linuxbrew/sbin"
+            )
+            else -> emptyList()  // Windows - use default PATH
+        }
+        
+        if (additionalPaths.isEmpty()) {
+            return
+        }
+        
+        // Filter to only include paths that actually exist
+        val existingAdditionalPaths = additionalPaths.filter { path ->
+            File(path).exists()
+        }
+        
+        if (existingAdditionalPaths.isEmpty()) {
+            return
+        }
+        
+        // Build the augmented PATH by prepending additional paths to current PATH
+        // This ensures tools in these directories are found first
+        val pathSeparator = if (os.contains("windows")) ";" else ":"
+        val augmentedPath = (existingAdditionalPaths + currentPath.split(pathSeparator))
+            .filter { it.isNotBlank() }
+            .distinct()
+            .joinToString(pathSeparator)
+        
+        processEnv["PATH"] = augmentedPath
+    }
+    
     private fun prepareCommand(command: String, shell: String?): List<String> {
         val effectiveShell = shell ?: getDefaultShell()
 

mpp-ui/src/wasmJsMain/kotlin/cc/unitmesh/devins/ui/compose/sketch/MarkdownSketchRenderer.wasmJs.kt

@@ -111,10 +111,6 @@ actual object MarkdownSketchRenderer {
                                     mermaidCode = code,
                                     modifier = Modifier.fillMaxWidth().heightIn(min = 200.dp)
                                 )
-//                                RemoteMermaidRenderer(
-//                                    mermaidCode = code,
-//                                    modifier = Modifier.fillMaxWidth().heightIn(min = 200.dp)
-//                                )
                             }
                         }
                     },

mpp-viewer-web/src/commonMain/kotlin/cc/unitmesh/viewer/web/MermaidRenderer.kt

@@ -47,12 +47,10 @@ fun MermaidRenderer(
     modifier: Modifier = Modifier,
     onRenderComplete: ((success: Boolean, message: String) -> Unit)? = null
 ) {
-    // Track dynamic height from rendered content
     var webViewHeight by remember { mutableStateOf(200.dp) }
     var showFullscreen by remember { mutableStateOf(false) }
 
     Box(modifier = modifier) {
-        // Main WebView display
         MermaidWebView(
             mermaidCode = mermaidCode,
             isDarkTheme = isDarkTheme,
@@ -66,7 +64,6 @@ fun MermaidRenderer(
             onRenderComplete = onRenderComplete
         )
 
-        // Expand button (top-right corner)
         IconButton(
             onClick = { showFullscreen = true },
             modifier = Modifier
@@ -82,7 +79,6 @@ fun MermaidRenderer(
         }
     }
 
-    // Fullscreen dialog
     if (showFullscreen) {
         MermaidFullscreenDialog(
             mermaidCode = mermaidCode,
@@ -94,9 +90,7 @@ fun MermaidRenderer(
     }
 }
 
-/**
- * Fullscreen Mermaid viewer with zoom controls
- */
+
 @Composable
 fun MermaidFullscreenDialog(
     mermaidCode: String,
@@ -120,7 +114,6 @@ fun MermaidFullscreenDialog(
             color = MaterialTheme.colorScheme.background
         ) {
             Column(modifier = Modifier.fillMaxSize()) {
-                // Top toolbar
                 Row(
                     modifier = Modifier
                         .fillMaxWidth()
@@ -129,7 +122,6 @@ fun MermaidFullscreenDialog(
                     horizontalArrangement = Arrangement.SpaceBetween,
                     verticalAlignment = Alignment.CenterVertically
                 ) {
-                    // Zoom controls
                     Row(
                         horizontalArrangement = Arrangement.spacedBy(8.dp),
                         verticalAlignment = Alignment.CenterVertically
@@ -168,7 +160,6 @@ fun MermaidFullscreenDialog(
                         }
                     }
 
-                    // Close button
                     IconButton(onClick = onDismiss) {
                         Icon(
                             imageVector = Icons.Default.Close,
@@ -194,9 +185,6 @@ fun MermaidFullscreenDialog(
     }
 }
 
-/**
- * Internal WebView component for Mermaid rendering
- */
 @Composable
 private fun MermaidWebView(
     mermaidCode: String,
@@ -213,7 +201,6 @@ private fun MermaidWebView(
     val jsBridge = rememberWebViewJsBridge()
 
     LaunchedEffect(Unit) {
-        // Handler for render callback
         jsBridge.register(object : IJsMessageHandler {
             override fun methodName(): String = "mermaidRenderCallback"
 
@@ -227,7 +214,6 @@ private fun MermaidWebView(
             }
         })
 
-        // Handler for height updates
         jsBridge.register(object : IJsMessageHandler {
             override fun methodName(): String = "mermaidHeightCallback"