🤖 ci: fix unit tests after compaction changes

Author: ammar-agent

src/browser/utils/messages/compactionOptions.test.ts

@@ -74,7 +74,7 @@ describe("applyCompactionOverrides", () => {
     const result = applyCompactionOverrides(baseWithTools, compactData);
 
     expect(result.mode).toBe("compact");
-    expect(result.toolPolicy).toEqual([{ regex_match: ".*", action: "disable" }]); // Tools always disabled for compaction
+    expect(result.toolPolicy).toEqual([{ regex_match: ".*", action: "disable" }]);
   });
 
   it("applies all overrides together", () => {

src/node/services/serverService.test.ts

@@ -77,22 +77,18 @@ describe("ServerService.startServer", () => {
   }
 
   test("cleans up server when lockfile acquisition fails", async () => {
-    // Skip on Windows where chmod doesn't work the same way
-    if (process.platform === "win32") {
-      return;
-    }
-
     const service = new ServerService();
 
-    // Make muxHome read-only so lockfile.acquire() will fail
-    await fs.chmod(tempDir, 0o444);
+    // Make muxHome a file (not a directory) so lockfile.acquire fails deterministically.
+    const muxHomeFile = path.join(tempDir, "not-a-dir");
+    await fs.writeFile(muxHomeFile, "not a directory");
 
     let thrownError: Error | null = null;
 
     try {
       // Start server - this should fail when trying to write lockfile
       await service.startServer({
-        muxHome: tempDir,
+        muxHome: muxHomeFile,
         context: stubContext as ORPCContext,
         authToken: "test-token",
         port: 0, // random port
@@ -103,7 +99,7 @@ describe("ServerService.startServer", () => {
 
     // Verify that an error was thrown
     expect(thrownError).not.toBeNull();
-    expect(thrownError!.message).toMatch(/EACCES|permission denied/i);
+    expect(thrownError!.message).toMatch(/ENOTDIR|not a directory/i);
 
     // Verify the server is NOT left running
     expect(service.isServerRunning()).toBe(false);