From 10100d6365e21f7d477c8c873a79f2f2efd6ec65 Mon Sep 17 00:00:00 2001 From: Garry Tan Date: Fri, 4 Sep 2026 23:06:30 +0000 Subject: [PATCH] docs(test-free-shards): the rerun-refresh harness spawns bash via test/helpers/bash-script.ts, not bash -c Co-Authored-By: Claude Fable 5.1 --- scripts/test-free-shards.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/test-free-shards.ts b/scripts/test-free-shards.ts index 06aa2c215..9f5daeb0f 100755 --- a/scripts/test-free-shards.ts +++ b/scripts/test-free-shards.ts @@ -262,8 +262,8 @@ const KNOWN_WINDOWS_SAFE: Array<{ file: string; reason: string }> = [ { file: 'test/setup-windows-rerun-refresh.test.ts', // Trips the "spawns bin/ shebang script" pattern via path.join(..., 'bin', - // 'tool.sh') fixture paths, but every spawn goes through spawnSync('bash', - // ['-c', ...]) — Git Bash executes it fine on windows-latest. This file IS + // 'tool.sh') fixture paths, but every spawn goes through test/helpers/bash-script.ts + // (bash ) — Git Bash executes it fine on windows-latest, with no argv-length ceiling. This file IS // the #2444 Windows regression coverage (IS_WINDOWS=1 copy-refresh path); // excluding it here would keep the bug class unexercised on the one // platform it bites.