From 45096acdcbb63b72ea4b30d92c5c06b50adafbfd Mon Sep 17 00:00:00 2001
From: Mohamed Boudra <boudra.moha@gmail.com>
Date: Sat, 17 Jan 2026 13:09:42 +0700
Subject: [PATCH] fix(server): increase hookTimeout and fix ClaudeAgentClient
 constructor

- Increase vitest hookTimeout from 30s to 60s to prevent hook timeouts
  in e2e tests that need time to start daemon instances
- Add `**/dist/**` to excludes to avoid running compiled test files
- Fix ClaudeAgentClient constructor call in claude-agent-commands.test.ts
  to pass required logger option
---
 .../src/server/agent/providers/claude-agent-commands.test.ts  | 3 ++-
 packages/server/vitest.config.ts                              | 4 ++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/packages/server/src/server/agent/providers/claude-agent-commands.test.ts b/packages/server/src/server/agent/providers/claude-agent-commands.test.ts
index eb7442927..cfac70697 100644
--- a/packages/server/src/server/agent/providers/claude-agent-commands.test.ts
+++ b/packages/server/src/server/agent/providers/claude-agent-commands.test.ts
@@ -12,6 +12,7 @@
 import { describe, it, expect, beforeAll, afterAll } from "vitest";
 import { ClaudeAgentClient } from "./claude-agent.js";
 import type { AgentSession, AgentSessionConfig, AgentSlashCommand } from "../agent-sdk-types.js";
+import { createTestLogger } from "../../../test-utils/test-logger.js";
 
 describe("ClaudeAgentSession Commands", () => {
   let client: ClaudeAgentClient;
@@ -28,7 +29,7 @@ describe("ClaudeAgentSession Commands", () => {
   };
 
   beforeAll(async () => {
-    client = new ClaudeAgentClient();
+    client = new ClaudeAgentClient({ logger: createTestLogger() });
   });
 
   afterAll(async () => {
diff --git a/packages/server/vitest.config.ts b/packages/server/vitest.config.ts
index 11341e7d3..5a4432cc8 100644
--- a/packages/server/vitest.config.ts
+++ b/packages/server/vitest.config.ts
@@ -9,7 +9,7 @@ export default defineConfig({
   },
   test: {
     testTimeout: 30000,
-    hookTimeout: 30000,
+    hookTimeout: 60000,
     globals: true,
     environment: "node",
     setupFiles: [path.resolve(__dirname, "./src/test-utils/vitest-setup.ts")],
@@ -20,6 +20,6 @@ export default defineConfig({
         maxForks: 4,
       },
     },
-    exclude: ["**/node_modules/**"],
+    exclude: ["**/node_modules/**", "**/dist/**"],
   },
 });