From d2833cfdbfdb11b663433dcbbd16ff66124a4f50 Mon Sep 17 00:00:00 2001 From: panandicoding Date: Sat, 25 Jul 2026 23:56:32 +0800 Subject: [PATCH] fix(test): remove shell-escaped newline from hooks runner test The test appended a literal backslash-n via shell escaping which is misinterpreted on Windows (cmd.exe produces the two characters backslash+n instead of a newline). Since the assertion already uses .trim(), the trailing newline is unnecessary. --- .../test/app/externalHooksRunner/externalHooksRunner.test.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/agent-core-v2/test/app/externalHooksRunner/externalHooksRunner.test.ts b/packages/agent-core-v2/test/app/externalHooksRunner/externalHooksRunner.test.ts index 38ec1ff7a2..f425ae7d90 100644 --- a/packages/agent-core-v2/test/app/externalHooksRunner/externalHooksRunner.test.ts +++ b/packages/agent-core-v2/test/app/externalHooksRunner/externalHooksRunner.test.ts @@ -204,7 +204,7 @@ describe('ExternalHooksRunnerService', () => { }); it('does not dedupe hooks that share a command but have different cwd', async () => { - const command = nodeCommand('process.stdout.write(process.cwd() + "\\n");'); + const command = nodeCommand('process.stdout.write(process.cwd());'); const runner = makeHookRunner([ { event: 'Stop', command, timeout: 5, cwd: process.cwd() }, { event: 'Stop', command, timeout: 5, cwd: tmpdir() },