Fix noisy WARN log for expected SIGTERM exit codes (143, 137)

Mark Pollack · Mark Pollack · commit ba3709d20b09 · 2026-01-13T10:29:40.000-05:00
diff --git a/acp-core/src/main/java/com/agentclientprotocol/sdk/client/transport/StdioAcpClientTransport.java b/acp-core/src/main/java/com/agentclientprotocol/sdk/client/transport/StdioAcpClientTransport.java
@@ -383,11 +383,13 @@ public Mono<Void> closeGracefully() {
 				return Mono.<Process>empty();
 			}
 		})).doOnNext(process -> {
-			if (process.exitValue() != 0) {
-				logger.warn("Process terminated with code {}", process.exitValue());
+			int exitCode = process.exitValue();
+			// 143 = SIGTERM (128+15), 137 = SIGKILL (128+9) - expected when we destroy
+			if (exitCode == 0 || exitCode == 143 || exitCode == 137) {
+				logger.info("ACP agent process stopped (exit code {})", exitCode);
 			}
 			else {
-				logger.info("ACP agent process stopped");
+				logger.warn("Process terminated unexpectedly with code {}", exitCode);
 			}
 		}).then(Mono.fromRunnable(() -> {
 			try {

Original file line number	Diff line number	Diff line change
`@@ -383,11 +383,13 @@ public Mono<Void> closeGracefully() {`
`383`	`383`	`return Mono.<Process>empty();`
`384`	`384`	`}`
`385`	`385`	`})).doOnNext(process -> {`
`386`		`- if (process.exitValue() != 0) {`
`387`		`- logger.warn("Process terminated with code {}", process.exitValue());`
	`386`	`+ int exitCode = process.exitValue();`
	`387`	`+ // 143 = SIGTERM (128+15), 137 = SIGKILL (128+9) - expected when we destroy`
	`388`	`+ if (exitCode == 0 \|\| exitCode == 143 \|\| exitCode == 137) {`
	`389`	`+ logger.info("ACP agent process stopped (exit code {})", exitCode);`
`388`	`390`	`}`
`389`	`391`	`else {`
`390`		`- logger.info("ACP agent process stopped");`
	`392`	`+ logger.warn("Process terminated unexpectedly with code {}", exitCode);`
`391`	`393`	`}`
`392`	`394`	`}).then(Mono.fromRunnable(() -> {`
`393`	`395`	`try {`