Fix maven report background thread exception and update logs

Author: srnagar

PluginsAndFeatures/azure-toolkit-for-intellij/azure-intellij-plugin-azuremcp/src/main/java/com/microsoft/azure/toolkit/intellij/azuremcp/AzureMcpPackageManager.java

@@ -77,7 +77,8 @@ public synchronized File getAzureMcpExecutable() {
                 }
             }
         } catch (final IOException e) {
-            System.err.println("Error reading Azure MCP Server version: " + e.getMessage());
+            log.error("Error getting Azure MCP executable: " + e.getMessage());
+            AzureMcpUtils.logErrorTelemetryEvent("azmcp-get-executable", e);
         }
         return null;
     }

PluginsAndFeatures/azure-toolkit-for-intellij/azure-intellij-plugin-azuremcp/src/main/java/com/microsoft/azure/toolkit/intellij/azuremcp/AzureMcpUtils.java

@@ -0,0 +1,41 @@
+package com.microsoft.azure.toolkit.intellij.azuremcp;
+
+import com.microsoft.azure.toolkit.lib.common.task.AzureTaskManager;
+import com.microsoft.azure.toolkit.lib.common.telemetry.AzureTelemeter;
+import com.microsoft.azure.toolkit.lib.common.telemetry.AzureTelemetry;
+import org.apache.commons.lang3.exception.ExceptionUtils;
+
+import java.util.Map;
+
+public final class AzureMcpUtils {
+
+    private static final String GHCP_MCP_INITIALIZER = "GHCP_MCP_INITIALIZER";
+
+    private AzureMcpUtils() {
+    }
+
+    public static void logTelemetryEvent(String eventName) {
+        Map<String, String> properties = Map.of(
+                AzureTelemeter.OP_NAME, eventName,
+                AzureTelemeter.OP_PARENT_ID, GHCP_MCP_INITIALIZER,
+                AzureTelemeter.OPERATION_NAME, eventName, // what's the difference between OP_NAME and OPERATION_NAME?
+                AzureTelemeter.SERVICE_NAME, GHCP_MCP_INITIALIZER
+        );
+        AzureTaskManager.getInstance().runLater(() -> {
+            AzureTelemeter.log(AzureTelemetry.Type.INFO, properties);
+        });
+    }
+
+    public static void logErrorTelemetryEvent(String eventName, Exception ex) {
+        Map<String, String> properties = Map.of(
+                AzureTelemeter.OP_NAME, eventName,
+                AzureTelemeter.OP_PARENT_ID, GHCP_MCP_INITIALIZER,
+                AzureTelemeter.OPERATION_NAME, eventName, // what's the difference between OP_NAME and OPERATION_NAME?
+                AzureTelemeter.SERVICE_NAME, GHCP_MCP_INITIALIZER,
+                AzureTelemeter.ERROR_STACKTRACE, ExceptionUtils.getStackTrace(ex)
+        );
+        AzureTaskManager.getInstance().runLater(() -> {
+            AzureTelemeter.log(AzureTelemetry.Type.ERROR, properties);
+        });
+    }
+}

PluginsAndFeatures/azure-toolkit-for-intellij/azure-intellij-plugin-azuremcp/src/main/java/com/microsoft/azure/toolkit/intellij/azuremcp/GithubCopilotMcpInitializer.java

@@ -13,13 +13,9 @@
 import com.intellij.openapi.startup.ProjectActivity;
 import com.intellij.openapi.util.SystemInfo;
 import com.intellij.openapi.util.registry.Registry;
-import com.microsoft.azure.toolkit.lib.common.task.AzureTaskManager;
-import com.microsoft.azure.toolkit.lib.common.telemetry.AzureTelemeter;
-import com.microsoft.azure.toolkit.lib.common.telemetry.AzureTelemetry;
 import kotlin.Unit;
 import kotlin.coroutines.Continuation;
 import lombok.extern.slf4j.Slf4j;
-import org.apache.commons.lang3.exception.ExceptionUtils;
 import org.apache.maven.artifact.versioning.ComparableVersion;
 import org.jetbrains.annotations.NotNull;
 
@@ -30,6 +26,9 @@
 import java.util.HashMap;
 import java.util.Map;
 
+import static com.microsoft.azure.toolkit.intellij.azuremcp.AzureMcpUtils.logErrorTelemetryEvent;
+import static com.microsoft.azure.toolkit.intellij.azuremcp.AzureMcpUtils.logTelemetryEvent;
+
 @Slf4j
 public class GithubCopilotMcpInitializer implements ProjectActivity, DumbAware, ProjectManagerListener {
     private static final ObjectMapper OBJECT_MAPPER = new ObjectMapper()
@@ -39,7 +38,6 @@ public class GithubCopilotMcpInitializer implements ProjectActivity, DumbAware,
             .enable(SerializationFeature.INDENT_OUTPUT);
 
     private static final ComparableVersion LOWEST_SUPPORTED_COPILOT_VERSION = new ComparableVersion("1.5.50");
-    private static final String GHCP_MCP_INITIALIZER = "GitHubCopilotMcpInitializer";
     private static final String COPILOT_PLUGIN_ID = "com.github.copilot";
     private final AzureMcpPackageManager azureMcpPackageManager;
 
@@ -71,7 +69,7 @@ public Object execute(@NotNull Project project, @NotNull Continuation<? super Un
     private boolean isCopilotMcpSupported() {
         // Get all installed plugins
         final IdeaPluginDescriptor[] installedPlugins = PluginManagerCore.getPlugins();
-        boolean copilotMcpSupported = Arrays.stream(installedPlugins)
+        final boolean copilotMcpSupported = Arrays.stream(installedPlugins)
                 .anyMatch(plugin -> {
                     final boolean copilotPluginInstalled = COPILOT_PLUGIN_ID.equals(plugin.getPluginId().getIdString());
                     return copilotPluginInstalled && isMcpSupported(plugin.getVersion());
@@ -156,29 +154,4 @@ private File getConfigPath() {
         }
         return configPath;
     }
-
-    public static void logTelemetryEvent(String eventName) {
-        Map<String, String> properties = Map.of(
-                AzureTelemeter.OP_NAME, eventName,
-                AzureTelemeter.OP_PARENT_ID, GHCP_MCP_INITIALIZER,
-                AzureTelemeter.OPERATION_NAME, eventName, // what's the difference between OP_NAME and OPERATION_NAME?
-                AzureTelemeter.SERVICE_NAME, GHCP_MCP_INITIALIZER
-        );
-        AzureTaskManager.getInstance().runLater(() -> {
-            AzureTelemeter.log(AzureTelemetry.Type.INFO, properties);
-        });
-    }
-
-    public static void logErrorTelemetryEvent(String eventName, Exception ex) {
-        Map<String, String> properties = Map.of(
-                AzureTelemeter.OP_NAME, eventName,
-                AzureTelemeter.OP_PARENT_ID, GHCP_MCP_INITIALIZER,
-                AzureTelemeter.OPERATION_NAME, eventName, // what's the difference between OP_NAME and OPERATION_NAME?
-                AzureTelemeter.SERVICE_NAME, GHCP_MCP_INITIALIZER,
-                AzureTelemeter.ERROR_STACKTRACE, ExceptionUtils.getStackTrace(ex)
-        );
-        AzureTaskManager.getInstance().runLater(() -> {
-            AzureTelemeter.log(AzureTelemetry.Type.ERROR, properties);
-        });
-    }
 }

PluginsAndFeatures/azure-toolkit-for-intellij/azure-intellij-plugin-java-sdk/src/main/java/com/microsoft/azure/toolkit/intellij/java/sdk/MavenProjectReportGenerator.java

@@ -83,7 +83,8 @@ public MavenProjectReportGenerator() {
     @Nullable
     @Override
     public Object execute(@Nonnull Project project, @Nonnull Continuation<? super Unit> continuation) {
-        scheduledExecutor.schedule(() -> generateReport(project), INITIAL_DELAY_IN_MINUTES, TimeUnit.MINUTES);
+        scheduledExecutor.schedule(() -> ApplicationManager.getApplication().runReadAction(() -> generateReport(project)),
+                INITIAL_DELAY_IN_MINUTES, TimeUnit.MINUTES);
         return null;
     }