test: reduce Windows-only skips

Author: benvinegar

src/core/loaders.test.ts

@@ -491,22 +491,19 @@ describe("loadAppBootstrap", () => {
   });
 
   test("loads untracked files whose names need parser-safe diff headers", async () => {
-    if (platform() === "win32") {
-      return;
-    }
-
     const dir = createTempRepo("hunk-git-quoted-untracked-");
 
     writeFileSync(join(dir, "tracked.ts"), "export const tracked = 1;\n");
     git(dir, "add", "tracked.ts");
     git(dir, "commit", "-m", "initial");
 
-    const quoteFile = 'quote"name.txt';
-    const tabFile = "tab\tname.txt";
-    const backslashFile = "back\\slash.txt";
-    writeFileSync(join(dir, quoteFile), "quote\n");
-    writeFileSync(join(dir, tabFile), "tab\n");
-    writeFileSync(join(dir, backslashFile), "backslash\n");
+    const portableFiles = ["space name.txt"];
+    const unixOnlyFiles = ['quote"name.txt', "tab\tname.txt", "back\\slash.txt"];
+    const fixtureFiles =
+      platform() === "win32" ? portableFiles : [...portableFiles, ...unixOnlyFiles];
+    for (const file of fixtureFiles) {
+      writeFileSync(join(dir, file), `${file}\n`);
+    }
 
     const bootstrap = await loadFromRepo(dir, {
       kind: "vcs",
@@ -515,10 +512,10 @@ describe("loadAppBootstrap", () => {
     });
     const paths = bootstrap.changeset.files.map((file) => file.path);
 
-    expect(paths).toContain(quoteFile);
-    expect(paths).toContain(tabFile);
-    expect(paths).toContain(backslashFile);
-    expect(paths).toHaveLength(3);
+    for (const file of fixtureFiles) {
+      expect(paths).toContain(file);
+    }
+    expect(paths).toHaveLength(fixtureFiles.length);
   });
 
   test("still shows an untracked agent sidecar when it lives inside the repo", async () => {
@@ -1115,21 +1112,16 @@ describe("loadAppBootstrap", () => {
   });
 
   test("loads quoted noprefix patch text emitted for escaped git paths", async () => {
-    if (platform() === "win32") {
-      return;
-    }
-
-    const dir = createTempRepo("hunk-patch-quoted-noprefix-");
-    const fileName = "src\tfile.txt";
-
-    writeFileSync(join(dir, fileName), "one\n");
-    git(dir, "add", ".");
-    git(dir, "commit", "-m", "initial");
-
-    writeFileSync(join(dir, fileName), "two\n");
-    const patchText = git(dir, "-c", "diff.noprefix=true", "diff", "--", fileName);
-
-    expect(patchText).toContain('diff --git "src\\tfile.txt" "src\\tfile.txt"');
+    const patchText = [
+      'diff --git "src\\tfile.txt" "src\\tfile.txt"',
+      "index 5626abf..f719efd 100644",
+      '--- "src\\tfile.txt"',
+      '+++ "src\\tfile.txt"',
+      "@@ -1 +1 @@",
+      "-one",
+      "+two",
+      "",
+    ].join("\n");
 
     const bootstrap = await loadAppBootstrap({
       kind: "patch",

src/session-broker/brokerServer.test.ts

@@ -94,7 +94,6 @@ async function openSessionSocket(port: number) {
       () => reject(new Error("Timed out waiting for websocket open.")),
       500,
     );
-    timeout.unref?.();
 
     socket.addEventListener(
       "open",
@@ -142,7 +141,6 @@ async function waitForSocketClose(socket: WebSocket) {
       () => reject(new Error("Timed out waiting for websocket close.")),
       1_000,
     );
-    timeout.unref?.();
 
     socket.addEventListener(
       "close",
@@ -248,6 +246,8 @@ describe("Hunk session daemon server", () => {
   });
 
   test("closes snapshots for missing sessions with a specific not-registered reason", async () => {
+    // Bun's Windows WebSocket client does not reliably surface this immediate server close.
+    // The daemon-core test covers the close code/reason without the flaky transport layer.
     if (platform() === "win32") {
       return;
     }