From cd40ad41cad0b631ba83d9e2e6b7da4766ffdbe3 Mon Sep 17 00:00:00 2001
From: Sebastian Westemeyer <sebastian.westemeyer@gebit.de>
Date: Fri, 16 Aug 2024 10:57:21 +0200
Subject: [PATCH] Added new config option for java debugger instrumentation

---
 .idea/codeStyles/Project.xml                  |  3 +-
 .../autoconfig/handlers/java/JavaHandler.java | 35 ++++++++++++++-----
 src/main/resources/schema/java.schema.json    | 16 +++++++++
 3 files changed, 45 insertions(+), 9 deletions(-)

diff --git a/.idea/codeStyles/Project.xml b/.idea/codeStyles/Project.xml
index 6f54c1b..d502fa7 100644
--- a/.idea/codeStyles/Project.xml
+++ b/.idea/codeStyles/Project.xml
@@ -1,6 +1,7 @@
 <component name="ProjectCodeStyleConfiguration">
   <code_scheme name="Project" version="173">
-    <option name="RIGHT_MARGIN" value="2048" />
+    <option name="RIGHT_MARGIN" value="120" />
+    <option name="SOFT_MARGINS" value="120" />
     <codeStyleSettings language="JAVA">
       <option name="RIGHT_MARGIN" value="120" />
       <option name="KEEP_LINE_BREAKS" value="false" />
diff --git a/src/main/java/de/gebit/plugins/autoconfig/handlers/java/JavaHandler.java b/src/main/java/de/gebit/plugins/autoconfig/handlers/java/JavaHandler.java
index 730e05c..487a939 100644
--- a/src/main/java/de/gebit/plugins/autoconfig/handlers/java/JavaHandler.java
+++ b/src/main/java/de/gebit/plugins/autoconfig/handlers/java/JavaHandler.java
@@ -11,12 +11,12 @@
 import com.intellij.compiler.CompilerConfiguration;
 import com.intellij.compiler.CompilerConfigurationImpl;
 import com.intellij.compiler.impl.javaCompiler.BackendCompiler;
+import com.intellij.debugger.settings.DebuggerSettings;
 import com.intellij.openapi.project.Project;
 import de.gebit.plugins.autoconfig.UpdateHandler;
 import de.gebit.plugins.autoconfig.handlers.AbstractHandler;
-import de.gebit.plugins.autoconfig.model.AnnotationProcessor;
 import de.gebit.plugins.autoconfig.model.Compiler;
-import de.gebit.plugins.autoconfig.model.JavaConfiguration;
+import de.gebit.plugins.autoconfig.model.*;
 import org.jetbrains.annotations.NonNls;
 import org.jetbrains.jps.model.java.compiler.ProcessorConfigProfile;
 
@@ -55,12 +55,16 @@ public Class<JavaConfiguration> getConfigurationClass() {
 	public List<String> updateConfiguration(JavaConfiguration configuration, Project project) {
 		List<String> changedConfigs = new ArrayList<>();
 		Compiler compiler = configuration.getCompiler();
-		if (compiler != null && CompilerConfiguration.getInstance(project) instanceof CompilerConfigurationImpl compilerConfiguration) {
+		if (compiler != null && CompilerConfiguration.getInstance(
+				project) instanceof CompilerConfigurationImpl compilerConfiguration) {
 			Compiler.JavaCompiler javaCompiler = compiler.getJavaCompiler();
-			if (javaCompiler != null && !compilerConfiguration.getDefaultCompiler().getId().equals(javaCompiler.value())) {
+			if (javaCompiler != null && !compilerConfiguration.getDefaultCompiler()
+					.getId()
+					.equals(javaCompiler.value())) {
 				for (BackendCompiler registeredCompiler : compilerConfiguration.getRegisteredJavaCompilers()) {
 					if (registeredCompiler.getId().equals(javaCompiler.value())) {
-						applySetting(registeredCompiler, compilerConfiguration.getDefaultCompiler(), compilerConfiguration::setDefaultCompiler, changedConfigs, "Java compiler");
+						applySetting(registeredCompiler, compilerConfiguration.getDefaultCompiler(),
+								compilerConfiguration::setDefaultCompiler, changedConfigs, "Java compiler");
 						break;
 					}
 				}
@@ -68,17 +72,32 @@ public List<String> updateConfiguration(JavaConfiguration configuration, Project
 			AnnotationProcessor annotationProcessor = compiler.getAnnotationProcessor();
 			if (annotationProcessor != null) {
 				ProcessorConfigProfile defaultProcessorProfile = compilerConfiguration.getDefaultProcessorProfile();
-				applySetting(annotationProcessor.getEnable(), defaultProcessorProfile.isEnabled(), defaultProcessorProfile::setEnabled, changedConfigs, "Enable annotation processor");
+				applySetting(annotationProcessor.getEnable(), defaultProcessorProfile.isEnabled(),
+						defaultProcessorProfile::setEnabled, changedConfigs, "Enable annotation processor");
 			}
 
 			Boolean parallelCompilation = compiler.getParallelCompilation();
 			if (parallelCompilation != null) {
-				applySetting(parallelCompilation, compilerConfiguration.isParallelCompilationEnabled(), compilerConfiguration::setParallelCompilationEnabled, changedConfigs, "Enable parallel compilation");
+				applySetting(parallelCompilation, compilerConfiguration.isParallelCompilationEnabled(),
+						compilerConfiguration::setParallelCompilationEnabled, changedConfigs,
+						"Enable parallel compilation");
 			}
 
 			Integer buildProcessHeapSize = compiler.getBuildProcessHeapSize();
 			if (buildProcessHeapSize != null) {
-				applySetting(buildProcessHeapSize, compilerConfiguration.getBuildProcessHeapSize(0), compilerConfiguration::setBuildProcessHeapSize, changedConfigs, "Build process heap size");
+				applySetting(buildProcessHeapSize, compilerConfiguration.getBuildProcessHeapSize(0),
+						compilerConfiguration::setBuildProcessHeapSize, changedConfigs, "Build process heap size");
+			}
+		}
+
+		Debugger debugger = configuration.getDebugger();
+		if (debugger != null) {
+			AsyncStackTraces asyncStackTraces = debugger.getAsyncStackTraces();
+			if (asyncStackTraces != null) {
+				applySetting(asyncStackTraces.getUseInstrumentingAgent(),
+						DebuggerSettings.getInstance().INSTRUMENTING_AGENT,
+						v -> DebuggerSettings.getInstance().INSTRUMENTING_AGENT = v, changedConfigs,
+						"Use instrumenting agent (application setting)");
 			}
 		}
 		return changedConfigs;
diff --git a/src/main/resources/schema/java.schema.json b/src/main/resources/schema/java.schema.json
index 4160afe..430fc71 100644
--- a/src/main/resources/schema/java.schema.json
+++ b/src/main/resources/schema/java.schema.json
@@ -4,6 +4,22 @@
   "title": "Java configuration",
   "type": "object",
   "properties": {
+    "debugger": {
+      "type": "object",
+      "description": "Debugger settings",
+      "properties": {
+        "asyncStackTraces": {
+          "description": "Settings for async stack traces",
+          "type": "object",
+          "properties": {
+            "useInstrumentingAgent": {
+              "type": "boolean",
+              "description": "Whether instrumenting agent should be used (application wide setting!)"
+            }
+          }
+        }
+      }
+    },
     "compiler": {
       "type": "object",
       "description": "Compiler settings",