Fix CI-aware options to respect --force-interactive flag

- Modified MavenInvoker to skip CI optimizations when --force-interactive is specified
- Updated CIOptimizationsIntegrationTest to reflect correct behavior where --force-interactive completely disables CI optimizations
- This ensures users can override CI detection and get normal interactive behavior even in CI environments

Author: gnodet

impl/maven-cli/src/main/java/org/apache/maven/cling/invoker/mvn/MavenInvoker.java

@@ -89,7 +89,10 @@ protected MavenContext createContext(InvokerRequest invokerRequest) {
         MavenOptions options = (MavenOptions) invokerRequest.options().orElse(null);
 
         // Apply CI-specific defaults if CI is detected and not overridden by user
-        if (invokerRequest.ciInfo().isPresent() && options != null) {
+        // Skip CI optimizations if --force-interactive is specified
+        if (invokerRequest.ciInfo().isPresent()
+                && options != null
+                && !options.forceInteractive().orElse(false)) {
             options = new CIAwareMavenOptions(options, invokerRequest.ciInfo().get());
         }
 

impl/maven-cli/src/test/java/org/apache/maven/cling/invoker/mvn/CIOptimizationsIntegrationTest.java

@@ -196,12 +196,22 @@ void testForceInteractiveOverridesCIDefaults() throws Exception {
         MavenInvoker invoker = new MavenInvoker(ProtoLookup.builder().build(), null);
         MavenContext context = invoker.createContext(mockRequest);
 
-        // Then: CI optimizations should still be applied (force-interactive affects interactive mode, not these
-        // options)
-        CIAwareMavenOptions ciOptions = (CIAwareMavenOptions) context.options();
-        assertTrue(ciOptions.showVersion().orElse(false), "showVersion should still be enabled in CI");
-        assertTrue(ciOptions.showErrors().orElse(false), "showErrors should still be enabled in CI");
-        assertTrue(ciOptions.nonInteractive().orElse(false), "nonInteractive should still be enabled in CI");
-        assertTrue(ciOptions.forceInteractive().orElse(false), "forceInteractive should be preserved");
+        // Then: CI optimizations should NOT be applied when force-interactive is specified
+        // The context should contain the original options, not the CI-aware wrapper
+        assertFalse(
+                context.options() instanceof CIAwareMavenOptions,
+                "CI optimizations should not be applied when --force-interactive is specified");
+
+        // The original options should be preserved
+        MavenOptions options = context.options();
+        assertFalse(
+                options.showVersion().orElse(false),
+                "showVersion should not be enabled when force-interactive is used");
+        assertFalse(
+                options.showErrors().orElse(false), "showErrors should not be enabled when force-interactive is used");
+        assertFalse(
+                options.nonInteractive().orElse(false),
+                "nonInteractive should not be enabled when force-interactive is used");
+        assertTrue(options.forceInteractive().orElse(false), "forceInteractive should be preserved");
     }
 }