Fix issues

Author: nquinquenel

src/main/java/org/sonarsource/sonarqube/mcp/transport/StdioServerTransportProvider.java

@@ -91,8 +91,12 @@ public StdioServerTransportProvider(ObjectMapper objectMapper) {
    * Returns false when running tests to avoid terminating the test runner.
    */
   private static boolean shouldExitOnStdinEof() {
+    return shouldExitOnStdinEof(Thread.currentThread().getStackTrace());
+  }
+
+  static boolean shouldExitOnStdinEof(StackTraceElement[] stackTrace) {
     // Check if we're running in test mode
-    var stackTrace = Thread.currentThread().getStackTrace();
+    // Common indicators: JUnit, Gradle test runner, Maven Surefire
     for (var element : stackTrace) {
       if (element.getClassName().startsWith("org.junit.")) {
         return false;

src/test/java/org/sonarsource/sonarqube/mcp/transport/StdioServerTransportProviderTest.java

@@ -23,6 +23,7 @@
 import org.mockito.Mockito;
 import reactor.core.publisher.Mono;
 
+import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatCode;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.never;
@@ -123,5 +124,41 @@ void closeGracefully_should_complete_immediately_if_session_closes_in_5_seconds(
     verify(mockSession, never()).close();
   }
 
+  @Test
+  void shouldExitOnStdinEof_should_return_true_for_production_stack_trace() {
+    // Simulate a production stack trace with no test frameworks
+    var stackTrace = new StackTraceElement[] {
+      new StackTraceElement("com.example.MyApp", "main", "MyApp.java", 10),
+      new StackTraceElement("com.example.MyService", "start", "MyService.java", 20),
+      new StackTraceElement("java.lang.Thread", "run", "Thread.java", 100)
+    };
+
+    boolean result = StdioServerTransportProvider.shouldExitOnStdinEof(stackTrace);
+
+    assertThat(result).isTrue();
+  }
+
+  @Test
+  void shouldExitOnStdinEof_should_return_false_when_junit_in_stack_trace() {
+    // Simulate a test stack trace with JUnit
+    var stackTrace = new StackTraceElement[] {
+      new StackTraceElement("org.junit.jupiter.engine.execution.ExecutableInvoker", "invoke", "ExecutableInvoker.java", 20)
+    };
+
+    boolean result = StdioServerTransportProvider.shouldExitOnStdinEof(stackTrace);
+
+    assertThat(result).isFalse();
+  }
+
+  @Test
+  void shouldExitOnStdinEof_should_return_true_for_empty_stack_trace() {
+    // Edge case: empty stack trace
+    var stackTrace = new StackTraceElement[] {};
+
+    boolean result = StdioServerTransportProvider.shouldExitOnStdinEof(stackTrace);
+
+    assertThat(result).isTrue();
+  }
+
 }