fix(exec): 子进程输出按 UTF-8 解码——toString() 平台默认字符集在非 UTF-8 locale 下损坏中文；附 printf 八进制转义回归测试

Author: loong10k

src/main/java/io/github/easy4j/comfy/cli/ComfyCliExecutor.java

@@ -180,8 +180,8 @@ private ComfyCliResult runProcess(String stdin, String... args) {
         long startNanos = System.nanoTime();
         try {
             int exitCode = childEnv == null ? executor.execute(cmd) : executor.execute(cmd, childEnv);
-            String out = stdout.toString().trim();
-            String err = stderr.toString().trim();
+            String out = stdout.toString(StandardCharsets.UTF_8).trim();
+            String err = stderr.toString(StandardCharsets.UTF_8).trim();
             log.debug("comfy CLI executed: exitCode={}, stdout.len={}", exitCode, out.length());
             if (watchdog.killedProcess()) {
                 return new ComfyCliResult(-1, out, "comfy CLI timed out after " + timeoutMs + " ms\n" + err);
@@ -194,8 +194,8 @@ private ComfyCliResult runProcess(String stdin, String... args) {
             // with the real exit code instead of discarding the output. The
             // deadline check makes the timeout verdict race-free even when
             // {@code watchdog.killedProcess()} has not observed the kill yet.
-            String out = stdout.toString().trim();
-            String err = stderr.toString().trim();
+            String out = stdout.toString(StandardCharsets.UTF_8).trim();
+            String err = stderr.toString(StandardCharsets.UTF_8).trim();
             boolean timedOut = watchdog.killedProcess()
                     || System.nanoTime() - startNanos >= timeoutMs * 1_000_000L;
             if (timedOut) {

src/test/java/io/github/easy4j/comfy/cli/ComfyCliExecutorTest.java

@@ -88,6 +88,20 @@ void shouldReturnIoExceptionMessageWhenExecutableMissing() {
         assertTrue(result.getStderr() != null && !result.getStderr().isEmpty());
     }
 
+    @Test
+    void shouldDecodeUtf8OutputRegardlessOfPlatformCharset() {
+        // POSIX printf octal escapes emit 你好 as raw UTF-8 bytes; with a
+        // platform-default-charset decode this corrupts on C-locale JVMs.
+        // NOTE: the backslashes are doubled in Java source so the shell
+        // receives single ones — an octal escape like \344 must be written
+        // \\344 here or the compiler eats it at compile time.
+        ComfyCliExecutor executor = new ComfyCliExecutor(configFor("/bin/sh"));
+
+        ComfyCliResult result = executor.execute("-c", "printf '\\344\\275\\240\\345\\245\\275'");
+
+        assertEquals("你好", result.getStdout());
+    }
+
     @Test
     void shouldIgnoreNullArguments() {
         ComfyCliExecutor executor = new ComfyCliExecutor(configFor(ECHO));