address reviewer feedback

Author: Eric Oliver

src/cli/cli-entry.ts

@@ -299,7 +299,7 @@ program
 					})
 					getCLILogger().debug("[cli-entry] GlobalCLIMcpService initialized successfully")
 				} catch (error) {
-					getCLILogger().debug("[cli-entry] Failed to initialize GlobalCLIMcpService:", error)
+					getCLILogger().warn("[cli-entry] Failed to initialize GlobalCLIMcpService:", error)
 					if (options.verbose) {
 						console.warn(
 							chalk.yellow("Warning: MCP initialization failed:"),

src/cli/commands/__tests__/BatchProcessor.test.ts

@@ -91,7 +91,12 @@ describe("BatchProcessor", () => {
 			jest.advanceTimersByTime(59000)
 
 			// Task should still be running, no timeout yet
-			expect(runPromise).not.toBe(undefined)
+			await expect(
+				Promise.race([
+					runPromise,
+					new Promise((resolve) => setTimeout(resolve, 1000)), // Ensure promise does not resolve within 1 second
+				]),
+			).resolves.not.toBe(runPromise)
 
 			// Now wait the full 60 seconds from the last activity
 			jest.advanceTimersByTime(1000)
@@ -121,7 +126,12 @@ describe("BatchProcessor", () => {
 				mockTask.emit(event[0], ...event.slice(1))
 
 				// Should not timeout yet
-				expect(runPromise).not.toBe(undefined)
+				await expect(
+					Promise.race([
+						runPromise,
+						new Promise((resolve) => setTimeout(resolve, 1000)), // Ensure promise does not resolve within 1 second
+					]),
+				).resolves.not.toBe(runPromise)
 			}
 
 			// Wait full timeout after last activity
@@ -192,7 +202,12 @@ describe("BatchProcessor", () => {
 			jest.advanceTimersByTime(59000)
 
 			// Should still be running
-			expect(runPromise).not.toBe(undefined)
+			await expect(
+				Promise.race([
+					runPromise,
+					new Promise((resolve) => setTimeout(resolve, 1000)), // Ensure promise does not resolve within 1 second
+				]),
+			).resolves.not.toBe(runPromise)
 
 			// Wait the final second to trigger timeout
 			jest.advanceTimersByTime(1000)