From 493e37bb7dc7c3805a2ed5a55f065d15a45cd2f4 Mon Sep 17 00:00:00 2001
From: Sisyphus <sisyphus@omo.ai>
Date: Sat, 18 Apr 2026 14:09:38 +0900
Subject: [PATCH] test(comment-checker): cover lazy CLI init and cleanup
 startup

---
 .../comment-checker/hook.lazy-init.test.ts    | 51 +++++++++++++++++++
 1 file changed, 51 insertions(+)
 create mode 100644 src/hooks/comment-checker/hook.lazy-init.test.ts

diff --git a/src/hooks/comment-checker/hook.lazy-init.test.ts b/src/hooks/comment-checker/hook.lazy-init.test.ts
new file mode 100644
index 000000000..2598aa3c6
--- /dev/null
+++ b/src/hooks/comment-checker/hook.lazy-init.test.ts
@@ -0,0 +1,51 @@
+import { describe, expect, it, mock, afterAll } from "bun:test"
+
+const startPendingCallCleanup = mock(() => {})
+const initializeCommentCheckerCli = mock(() => {})
+
+mock.module("./cli-runner", () => ({
+  initializeCommentCheckerCli,
+  getCommentCheckerCliPathPromise: () => Promise.resolve("/tmp/fake-comment-checker"),
+  isCliPathUsable: () => true,
+  processWithCli: async () => {},
+  processApplyPatchEditsWithCli: async () => {},
+}))
+
+mock.module("./pending-calls", () => ({
+  registerPendingCall: () => {},
+  startPendingCallCleanup,
+  stopPendingCallCleanup: () => {},
+  takePendingCall: () => undefined,
+}))
+
+afterAll(() => {
+  mock.restore()
+})
+
+const { createCommentCheckerHooks } = await import("./hook")
+
+describe("comment-checker lazy initialization", () => {
+  it("initializes CLI and cleanup on first tool hook call only", async () => {
+    // given
+    const hooks = createCommentCheckerHooks()
+    const beforeHook = hooks["tool.execute.before"]
+    const input = { tool: "write", sessionID: "ses_test", callID: "call_test" }
+    const output = { args: { filePath: "src/a.ts" } }
+
+    // when
+    expect(startPendingCallCleanup).toHaveBeenCalledTimes(0)
+    expect(initializeCommentCheckerCli).toHaveBeenCalledTimes(0)
+
+    // then
+    await beforeHook(input, output)
+    expect(startPendingCallCleanup).toHaveBeenCalledTimes(1)
+    expect(initializeCommentCheckerCli).toHaveBeenCalledTimes(1)
+
+    // when
+    await beforeHook(input, output)
+
+    // then
+    expect(startPendingCallCleanup).toHaveBeenCalledTimes(1)
+    expect(initializeCommentCheckerCli).toHaveBeenCalledTimes(1)
+  })
+})