From 4ff8d77cdbb315300e387740c710e238befc8e3e Mon Sep 17 00:00:00 2001
From: Helin Shiah <helinx@google.com>
Date: Fri, 6 Jun 2025 13:40:20 -0700
Subject: [PATCH] Add a separate file for plugin logs

---
 .../flutter_intellij__runIde_.xml             |  1 +
 .../src/io/flutter/FlutterInitializer.java    |  8 ++-
 .../io/flutter/logging/PluginLogHandler.java  | 52 +++++++++++++++++++
 .../src/io/flutter/logging/PluginLogger.java  | 16 ++++++
 .../run/daemon/DevToolsServerTask.java        |  5 +-
 5 files changed, 79 insertions(+), 3 deletions(-)
 create mode 100644 flutter-idea/src/io/flutter/logging/PluginLogHandler.java
 create mode 100644 flutter-idea/src/io/flutter/logging/PluginLogger.java

diff --git a/.idea/runConfigurations/flutter_intellij__runIde_.xml b/.idea/runConfigurations/flutter_intellij__runIde_.xml
index 506924c3d..4dbda177c 100644
--- a/.idea/runConfigurations/flutter_intellij__runIde_.xml
+++ b/.idea/runConfigurations/flutter_intellij__runIde_.xml
@@ -1,5 +1,6 @@
 <component name="ProjectRunConfigurationManager">
   <configuration default="false" name="flutter-intellij [runIde]" type="GradleRunConfiguration" factoryName="Gradle">
+    <log_file alias="flutter" path="$PROJECT_DIR$/build/idea-sandbox/AI-AI-251.23774.16.2511.13338727/log/flutter.log" />
     <ExternalSystemSettings>
       <option name="executionName" />
       <option name="externalProjectPath" value="$PROJECT_DIR$" />
diff --git a/flutter-idea/src/io/flutter/FlutterInitializer.java b/flutter-idea/src/io/flutter/FlutterInitializer.java
index 1e9b83b72..918b41e51 100644
--- a/flutter-idea/src/io/flutter/FlutterInitializer.java
+++ b/flutter-idea/src/io/flutter/FlutterInitializer.java
@@ -41,6 +41,7 @@
 import io.flutter.devtools.RemainingDevToolsViewFactory;
 import io.flutter.editor.FlutterSaveActionsManager;
 import io.flutter.logging.FlutterConsoleLogManager;
+import io.flutter.logging.PluginLogger;
 import io.flutter.module.FlutterModuleBuilder;
 import io.flutter.pub.PubRoot;
 import io.flutter.pub.PubRoots;
@@ -74,8 +75,7 @@
  * may run when a project is being imported.
  */
 public class FlutterInitializer extends FlutterProjectActivity {
-  private static final @NotNull Logger LOG = Logger.getInstance(FlutterInitializer.class);
-
+  private static final @NotNull Logger LOG = PluginLogger.createLogger(FlutterInitializer.class);
   private boolean toolWindowsInitialized = false;
 
   private boolean busSubscribed = false;
@@ -84,12 +84,15 @@ public class FlutterInitializer extends FlutterProjectActivity {
 
   @Override
   public void executeProjectStartup(@NotNull Project project) {
+    LOG.info("starting executeProjectStartup");
     // Disable the 'Migrate Project to Gradle' notification.
     FlutterUtils.disableGradleProjectMigrationNotification(project);
 
+    LOG.info("Starting watching for devices");
     // Start watching for devices.
     DeviceService.getInstance(project);
 
+    LOG.info("Starting a DevTools server");
     // Start a DevTools server
     DevToolsService.getInstance(project);
 
@@ -108,6 +111,7 @@ public void executeProjectStartup(@NotNull Project project) {
         continue;
       }
 
+      LOG.info("This is a Flutter module");
       // Ensure SDKs are configured; needed for clean module import.
       FlutterModuleUtils.enableDartSDK(module);
 
diff --git a/flutter-idea/src/io/flutter/logging/PluginLogHandler.java b/flutter-idea/src/io/flutter/logging/PluginLogHandler.java
new file mode 100644
index 000000000..75725c607
--- /dev/null
+++ b/flutter-idea/src/io/flutter/logging/PluginLogHandler.java
@@ -0,0 +1,52 @@
+/*
+ * Copyright 2025 The Chromium Authors. All rights reserved.
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file.
+ */
+package io.flutter.logging;
+
+import com.intellij.openapi.application.PathManager;
+
+import java.io.File;
+import java.io.FileWriter;
+import java.io.IOException;
+import java.util.logging.Handler;
+import java.util.logging.LogRecord;
+
+public class PluginLogHandler extends Handler {
+  private static final String LOG_FILE_NAME = "flutter.log";
+  public PluginLogHandler() {
+  }
+
+  @Override
+  public void publish(LogRecord record) {
+    // Write to a file
+    final String logPath = PathManager.getLogPath();
+    System.out.println("Log path: " + logPath);
+    File logFile = new File(logPath + File.separatorChar + LOG_FILE_NAME);
+
+    try {
+      if (!logFile.exists()) {
+        logFile.createNewFile(); // Create the file if it doesn't exist
+        System.out.println("Log file created: " + logFile.getAbsolutePath());
+      }
+      FileWriter writer = new FileWriter(logFile, true); // Append to the file
+
+      String message = record.getInstant().toString() + " " + record.getLevel() + ": " + record.getMessage() + System.lineSeparator();
+      writer.write(message); // Write the log message
+      writer.close(); // Close the writer
+    } catch (IOException e) {
+      System.err.println("Error creating or writing to the log file: " + e.getMessage());
+    }
+  }
+
+  @Override
+  public void flush() {
+    System.out.println("LOG FLUSH: ");
+  }
+
+  @Override
+  public void close() throws SecurityException {
+    System.out.println("in close of logging");
+  }
+}
diff --git a/flutter-idea/src/io/flutter/logging/PluginLogger.java b/flutter-idea/src/io/flutter/logging/PluginLogger.java
new file mode 100644
index 000000000..bea31741b
--- /dev/null
+++ b/flutter-idea/src/io/flutter/logging/PluginLogger.java
@@ -0,0 +1,16 @@
+/*
+ * Copyright 2025 The Chromium Authors. All rights reserved.
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file.
+ */
+package io.flutter.logging;
+
+import com.intellij.openapi.diagnostic.Logger;
+import org.jetbrains.annotations.NotNull;
+
+public class PluginLogger {
+  public static Logger createLogger(@NotNull Class logClass) {
+    java.util.logging.Logger.getLogger(logClass.getName()).addHandler(new PluginLogHandler());
+    return Logger.getInstance(logClass.getName());
+  }
+}
diff --git a/flutter-idea/src/io/flutter/run/daemon/DevToolsServerTask.java b/flutter-idea/src/io/flutter/run/daemon/DevToolsServerTask.java
index 07ad93771..4e6c02992 100644
--- a/flutter-idea/src/io/flutter/run/daemon/DevToolsServerTask.java
+++ b/flutter-idea/src/io/flutter/run/daemon/DevToolsServerTask.java
@@ -38,6 +38,8 @@
 import io.flutter.bazel.Workspace;
 import io.flutter.bazel.WorkspaceCache;
 import io.flutter.dart.DtdUtils;
+import io.flutter.logging.PluginLogHandler;
+import io.flutter.logging.PluginLogger;
 import io.flutter.sdk.FlutterSdk;
 import io.flutter.sdk.FlutterSdkUtil;
 import io.flutter.utils.JsonUtils;
@@ -57,7 +59,7 @@
 import java.util.concurrent.atomic.AtomicReference;
 
 class DevToolsServerTask extends Task.Backgroundable {
-  private @NotNull static final Logger LOG = Logger.getInstance(DevToolsServerTask.class);
+  private @NotNull static final Logger LOG = PluginLogger.createLogger(DevToolsServerTask.class);
   public @NotNull static final String LOCAL_DEVTOOLS_DIR = "flutter.local.devtools.dir";
   public @NotNull static final String LOCAL_DEVTOOLS_ARGS = "flutter.local.devtools.args";
   private @NotNull final Project project;
@@ -79,6 +81,7 @@ public void run(@NotNull ProgressIndicator progressIndicator) {
     try {
       progressIndicator.setFraction(30);
       progressIndicator.setText2("Init");
+      LOG.info("Here we are in DevTools run");
 
       // If DevTools is not supported, start the daemon instead.
       final boolean dartDevToolsSupported = dartSdkSupportsDartDevTools();