test(cli): expose probe timeout semantics

Author: loong10k

src/test/java/io/github/easy4j/codex/cli/CodexCliExecutorTest.java

@@ -23,6 +23,9 @@
 import static org.junit.jupiter.api.Assertions.assertNotNull;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
+import java.nio.file.Paths;
+import java.util.concurrent.TimeUnit;
+
 /**
  * Unit tests for {@link CodexCliExecutor}.
  *
@@ -36,6 +39,9 @@
  */
 class CodexCliExecutorTest {
 
+    private static final String SLOW_CODEX_SCRIPT =
+            Paths.get("src", "test", "resources", "slow-codex.sh").toAbsolutePath().toString();
+
     private CodexClientConfig configFor(String executable) {
         CodexClientConfig config = new CodexClientConfig();
         config.setLocalExecutable(executable);
@@ -158,14 +164,31 @@ void shouldReportFailureFromProbeWhenExecutableMissing() {
         assertFalse(executor.probe());
     }
 
+    @Test
+    void shouldUseDedicatedProbeTimeoutWithoutChangingNormalTimeout() {
+        CodexClientConfig config = configFor("/bin/sh " + SLOW_CODEX_SCRIPT);
+        config.setLocalProbeTimeoutSeconds(1);
+        config.setLocalTimeoutSeconds(8);
+        CodexCliExecutor executor = new CodexCliExecutor(config);
+
+        long started = System.nanoTime();
+        boolean available = executor.probe();
+        long elapsedMs = TimeUnit.NANOSECONDS.toMillis(System.nanoTime() - started);
+
+        assertFalse(available, "probe must time out using localProbeTimeoutSeconds");
+        assertTrue(elapsedMs < 3_000,
+                "probe must not wait for the normal localTimeoutSeconds window; elapsed=" + elapsedMs);
+    }
+
     @Test
     void shouldTimeoutOnHangingProcess() {
-        // Use a short timeout and a command that sleeps for a long time.
-        CodexClientConfig config = configFor("/bin/sh");
+        // Keep the blocking loop in the shell process itself so killing the
+        // shell closes stdout/stderr immediately.
+        CodexClientConfig config = configFor("/bin/sh " + SLOW_CODEX_SCRIPT);
         config.setLocalTimeoutSeconds(1);
         CodexCliExecutor executor = new CodexCliExecutor(config);
 
-        CodexCliResult result = executor.execute("-c", "sleep 60");
+        CodexCliResult result = executor.execute();
 
         // On macOS/Linux the watchdog kills the process; the exit code is -1
         // and stderr contains the timeout notice.