From 1b30551c54d6479b470e3c2c32138bd16b8dc7ed Mon Sep 17 00:00:00 2001 From: Ruben Fiszel Date: Wed, 28 Oct 2026 23:10:52 +0100 Subject: [PATCH] skip the '*' file in the review prompt test on Windows (#12090) Windows forbids '*' in a file name, so creating it failed the test on the Windows CLI runner. The :(literal) pathspec guard it covers is platform-independent and stays exercised on Linux and macOS. Co-authored-by: Claude Opus 5.5 --- cli/test/ai_review_bundle_unit.test.ts | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/cli/test/ai_review_bundle_unit.test.ts b/cli/test/ai_review_bundle_unit.test.ts index 03ba770d6d..a1508050ff 100644 --- a/cli/test/ai_review_bundle_unit.test.ts +++ b/cli/test/ai_review_bundle_unit.test.ts @@ -140,8 +140,9 @@ describe("wmill review prompt", () => { "encryption_key.yaml": "HEAD-ENCRYPTION-KEY\n", "f/a/notes.prod.resource.yaml": "resource_type: postgresql\nvalue:\n password: PROD-PASSWORD\n", "f/a/files.fileset/x.flow/flow.yaml": "password: FILESET-PASSWORD-2\n", - // A pathspec matching every changed file, if git were let read it as one. - "*": "star\n", + // A pathspec matching every changed file, if git were let read it as one. Windows + // forbids `*` in a file name. + ...(process.platform === "win32" ? {} : { "*": "star\n" }), }, (base, head) => { const { system, user, instructions } = buildReviewPrompt(base, head);