vrr/qwen-code
2
0
Fork 0
mirror of https://github.com/QwenLM/qwen-code.git synced 2026-05-02 13:40:46 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 10

Merge pull request #2472 from QwenLM/fix/acp-connection-state-cleanup-1780

Browse source 
fix: clean up ACP connection state when child process exits
This commit is contained in:
tanzhenxin 2026-03-20 11:51:14 +08:00 • committed by GitHub
commit 572fa81f38
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 199 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

26
packages/cli/src/acp-integration/acpAgent.ts
View file

@ -87,7 +87,33 @@ export async function runAcpAgent(
stream,
);
// Handle SIGTERM/SIGINT for graceful shutdown.
// Without this, signal handlers registered elsewhere in the CLI
// (e.g., stdin raw mode restoration) override the default exit behavior,
// causing the ACP process to ignore termination signals.
let shuttingDown = false;
const shutdownHandler = () => {
if (shuttingDown) return;
shuttingDown = true;
debugLogger.debug('[ACP] Shutdown signal received, closing streams');
try {
process.stdin.destroy();
} catch {
// stdin may already be closed
}
try {
process.stdout.destroy();
} catch {
// stdout may already be closed
}
};
process.on('SIGTERM', shutdownHandler);
process.on('SIGINT', shutdownHandler);
await connection.closed;
process.off('SIGTERM', shutdownHandler);
process.off('SIGINT', shutdownHandler);
}
function toStdioServer(server: McpServer): McpServerStdio | undefined {