refactor: limit executable search to PATH and add error handling for missing executable

README.md

@@ -14,7 +14,7 @@ Welcome to KCL developing! We hope this plugin enhances your development experie
 
 - **Step 1.** [Install KCL](https://kcl-lang.io/docs/user_docs/getting-started/install) on your system.
 
-- **Step 2.** Install the KCL plugin for IntelliJ IDEA in Marketplace. Preference -> plugins -> Marketplace -> search `KCL` -> install -> restart IDE. This plugin requires the IntelliJ IDEA Ultimate 2023.2+
+- **Step 2.** Install the KCL plugin for IntelliJ IDEA in Marketplace. Preference -> plugins -> Marketplace -> search `KCL` -> install -> restart IDE.
 
 - **Step 3.** Open or create a KCL file and begin your KCL tour!
 

src/main/java/io/kusionstack/kcl/KCLLanguageServer.java

@@ -17,13 +17,14 @@
 public class KCLLanguageServer extends ProcessStreamConnectionProvider {
 
     public KCLLanguageServer(Project project) {
-        String kclLSPPath = findExecutableInPATH("kcl-language-server");
-        if(kclLSPPath!=null && !kclLSPPath.isEmpty()) {
+        try {
+            String kclLSPPath = findExecutableInPATH("kcl-language-server");
             super.setCommands(List.of(kclLSPPath));
-        } else {
+        } catch (Exception e) {
             NotificationGroupManager.getInstance().getNotificationGroup("KCL LSP").createNotification(
                     "KCL LSP",
-                    "KCL Language Server not found. Make sure it is installed properly (and is available in PATH), and restart the IDE.",
+                    "KCL Language Server not found. Ensure it is installed properly (and is available in PATH), and restart the IDE. "
+                            + "Note: A path containing a colon ':' may cause issues on some systems.",
                     NotificationType.ERROR
             ).notify(project);
             LanguageServerManager.getInstance(project).stop("KCLLanguageServerId");
@@ -38,12 +39,12 @@ private String findExecutableInPATH(String executable) {
         // Append '.exe' to the executable name if on Windows, otherwise leave it as is
         String executableName = isWindows ? executable + ".exe" : executable;
 
-        Optional<File> foundExecutable =  envVariablesMap.values().stream()
-                                                         // Split the PATH by the platform-specific separator (':' for Unix, ';' for Windows)
-                                                         .flatMap(path -> Stream.of(path.split(File.pathSeparator)))
-                                                         .map(path -> Path.of(path,executableName).toFile())
-                                                         .filter(file -> file.exists() && file.canExecute())
-                                                         .findFirst();
+        Optional<File> foundExecutable = Stream.of(envVariablesMap.get("PATH"))
+                                               // Split the PATH by the platform-specific separator (':' for Unix, ';' for Windows)
+                                               .flatMap(path -> Stream.of(path.split(File.pathSeparator)))
+                                               .map(path -> Path.of(path,executableName).toFile())
+                                               .filter(file -> file.exists() && file.canExecute())
+                                               .findFirst();
 
         return foundExecutable.map(File::getPath).orElse(null);
     }