Fix EffectivelyPrivate warnings causing ./gradlew compileJava to fail (#4393)

Author: Copilot

agent/agent-tooling/src/main/java/com/microsoft/applicationinsights/agent/internal/configuration/ConfigurationLogger.java

@@ -67,6 +67,6 @@ public void log(Logger logger, String message, Object... args) {
       }
     };
 
-    public abstract void log(Logger logger, String message, Object... args);
+    abstract void log(Logger logger, String message, Object... args);
   }
 }

agent/agent-tooling/src/main/java/com/microsoft/applicationinsights/agent/internal/processors/AttributeProcessor.java

@@ -254,10 +254,10 @@ private static HttpUrlKeyValuePair processHttpUrl(Attributes attributes) {
   }
 
   private static class HttpUrlKeyValuePair {
-    public final String key;
-    public final String value;
+    final String key;
+    final String value;
 
-    public HttpUrlKeyValuePair(String key, String value) {
+    HttpUrlKeyValuePair(String key, String value) {
       this.key = key;
       this.value = value;
     }

agent/agent-tooling/src/main/java/com/microsoft/applicationinsights/agent/internal/telemetry/BatchItemProcessor.java

@@ -187,7 +187,7 @@ public void run() {
       }
     }
 
-    public void internalRun() {
+    void internalRun() {
       updateNextExportTime();
 
       while (continueWork) {

agent/agent-tooling/src/test/java/com/microsoft/applicationinsights/agent/internal/profiler/ProfilingInitializerTest.java

@@ -29,27 +29,27 @@
 
 public class ProfilingInitializerTest {
   private static class ProfilingInitializerTestCaseBuilder {
-    public final String name;
-    public final List<ProfilerConfiguration> configurations = new ArrayList<>();
+    final String name;
+    final List<ProfilerConfiguration> configurations = new ArrayList<>();
 
     private ProfilingInitializerTestCaseBuilder(String name) {
       this.name = name;
     }
 
-    public ProfilingInitializerTestCaseBuilder then(ProfilerConfiguration configuration) {
+    ProfilingInitializerTestCaseBuilder then(ProfilerConfiguration configuration) {
       configurations.add(configuration);
       return this;
     }
 
-    public ProfilingInitializerTestCase assertThat(Consumer<ProfilingInitializer> assertion) {
+    ProfilingInitializerTestCase assertThat(Consumer<ProfilingInitializer> assertion) {
       return new ProfilingInitializerTestCase(name, configurations, assertion);
     }
   }
 
   private static class ProfilingInitializerTestCase {
-    public final String name;
-    public final List<ProfilerConfiguration> configurations;
-    public final Consumer<ProfilingInitializer> assertion;
+    final String name;
+    final List<ProfilerConfiguration> configurations;
+    final Consumer<ProfilingInitializer> assertion;
 
     private ProfilingInitializerTestCase(
         String name,

smoke-tests/framework/src/main/java/com/microsoft/applicationinsights/smoketest/fakeingestion/JsonHelper.java

@@ -41,7 +41,7 @@ private static class BaseDataContractDeserializer implements JsonDeserializer<Ba
 
     private final Map<String, Class<? extends Domain>> classMap;
 
-    public BaseDataContractDeserializer() {
+    BaseDataContractDeserializer() {
       classMap = new HashMap<>();
       @SuppressWarnings("unchecked")
       Class<? extends Domain>[] classes =