From a215b2471a95f56cefc899b012fd479e7b430dd6 Mon Sep 17 00:00:00 2001
From: Eyejoker <phj1081@gmail.com>
Date: Mon, 30 Mar 2026 00:57:03 +0900
Subject: [PATCH] chore: formatter cleanup and test updates from bot
 cherry-pick

---
 src/container-runner.ts              | 19 ++++++++--------
 src/message-turn-controller.ts       |  3 +--
 src/paired-execution-context.test.ts | 34 +++++++++++++++++++---------
 3 files changed, 33 insertions(+), 23 deletions(-)

diff --git a/src/container-runner.ts b/src/container-runner.ts
index 160e33b..e8666a5 100644
--- a/src/container-runner.ts
+++ b/src/container-runner.ts
@@ -72,10 +72,7 @@ function detectPnpmStorePath(workspaceDir: string): string | null {
   if (!fs.existsSync(path.join(workspaceDir, 'pnpm-lock.yaml'))) {
     return null;
   }
-  if (
-    process.env.PNPM_STORE_DIR &&
-    fs.existsSync(process.env.PNPM_STORE_DIR)
-  ) {
+  if (process.env.PNPM_STORE_DIR && fs.existsSync(process.env.PNPM_STORE_DIR)) {
     return process.env.PNPM_STORE_DIR;
   }
   try {
@@ -108,11 +105,10 @@ function ensureContainerReady(): void {
   if (containerRuntimeChecked) return;
   ensureContainerRuntimeRunning();
   try {
-    execFileSync(
-      CONTAINER_RUNTIME_BIN,
-      ['image', 'inspect', CONTAINER_IMAGE],
-      { stdio: 'pipe', timeout: 10000 },
-    );
+    execFileSync(CONTAINER_RUNTIME_BIN, ['image', 'inspect', CONTAINER_IMAGE], {
+      stdio: 'pipe',
+      timeout: 10000,
+    });
   } catch {
     throw new Error(
       `Container image '${CONTAINER_IMAGE}' not found. Build it with: ./container/build.sh`,
@@ -191,7 +187,10 @@ export function stopReviewerContainer(groupFolder: string): void {
       stdio: 'pipe',
       timeout: 10000,
     });
-    logger.info({ containerName: name }, 'Stopped persistent reviewer container');
+    logger.info(
+      { containerName: name },
+      'Stopped persistent reviewer container',
+    );
   } catch {
     /* already gone */
   }
diff --git a/src/message-turn-controller.ts b/src/message-turn-controller.ts
index e2b1428..83aee35 100644
--- a/src/message-turn-controller.ts
+++ b/src/message-turn-controller.ts
@@ -267,8 +267,7 @@ export class MessageTurnController {
       // If the progress message already shows the same text as the final
       // result, finalize it in-place instead of sending a duplicate message.
       const alreadyVisible =
-        this.progressMessageId &&
-        this.latestProgressText === text;
+        this.progressMessageId && this.latestProgressText === text;
       if (alreadyVisible) {
         await this.finalizeProgressMessage();
         this.visiblePhase = toVisiblePhase(phase);
diff --git a/src/paired-execution-context.test.ts b/src/paired-execution-context.test.ts
index 03e5b25..3d24804 100644
--- a/src/paired-execution-context.test.ts
+++ b/src/paired-execution-context.test.ts
@@ -126,10 +126,12 @@ function buildWorkspace(
     task_id: 'task-1',
     role,
     workspace_dir: workspaceDir,
-    snapshot_source_dir: role === 'reviewer' ? '/tmp/paired/task-1/owner' : null,
+    snapshot_source_dir:
+      role === 'reviewer' ? '/tmp/paired/task-1/owner' : null,
     snapshot_ref: role === 'reviewer' ? 'fingerprint-1' : null,
     status: 'ready',
-    snapshot_refreshed_at: role === 'reviewer' ? '2026-03-28T00:00:00.000Z' : null,
+    snapshot_refreshed_at:
+      role === 'reviewer' ? '2026-03-28T00:00:00.000Z' : null,
     created_at: '2026-03-28T00:00:00.000Z',
     updated_at: '2026-03-28T00:00:00.000Z',
   };
@@ -442,14 +444,19 @@ describe('paired execution context', () => {
       'task-1',
       expect.objectContaining({ status: 'completed' }),
     );
-    expect(pairedWorkspaceManager.markPairedTaskReviewReady).not.toHaveBeenCalled();
+    expect(
+      pairedWorkspaceManager.markPairedTaskReviewReady,
+    ).not.toHaveBeenCalled();
   });
 
   it('re-triggers review when owner changed code after approval', () => {
     const repoDir = createCanonicalRepoWithCommit('reviewed');
     const approvedSourceRef = resolveTreeRef(repoDir);
     fs.writeFileSync(path.join(repoDir, 'README.md'), 'changed\n');
-    execFileSync('git', ['add', 'README.md'], { cwd: repoDir, stdio: 'ignore' });
+    execFileSync('git', ['add', 'README.md'], {
+      cwd: repoDir,
+      stdio: 'ignore',
+    });
     execFileSync('git', ['commit', '-m', 'code change'], {
       cwd: repoDir,
       stdio: 'ignore',
@@ -465,10 +472,15 @@ describe('paired execution context', () => {
     vi.mocked(db.getPairedWorkspace).mockImplementation((_taskId, role) =>
       role === 'owner' ? buildWorkspace('owner', repoDir) : undefined,
     );
-    vi.mocked(pairedWorkspaceManager.markPairedTaskReviewReady).mockReturnValue({
-      ownerWorkspace: buildWorkspace('owner', repoDir),
-      reviewerWorkspace: buildWorkspace('reviewer', '/tmp/paired/task-1/reviewer'),
-    });
+    vi.mocked(pairedWorkspaceManager.markPairedTaskReviewReady).mockReturnValue(
+      {
+        ownerWorkspace: buildWorkspace('owner', repoDir),
+        reviewerWorkspace: buildWorkspace(
+          'reviewer',
+          '/tmp/paired/task-1/reviewer',
+        ),
+      },
+    );
 
     completePairedExecutionContext({
       taskId: 'task-1',
@@ -477,9 +489,9 @@ describe('paired execution context', () => {
       summary: 'DONE',
     });
 
-    expect(pairedWorkspaceManager.markPairedTaskReviewReady).toHaveBeenCalledWith(
-      'task-1',
-    );
+    expect(
+      pairedWorkspaceManager.markPairedTaskReviewReady,
+    ).toHaveBeenCalledWith('task-1');
     expect(db.updatePairedTask).toHaveBeenCalledWith(
       'task-1',
       expect.objectContaining({