@suseona/runner 1.0.1 → 1.0.2

package/api.mjs

@@ -37,10 +37,11 @@ export class ApiError extends Error {
   }
 }
 
-/** POST 래퍼 — 네트워크 오류·5xx 만 2회 재시도(4xx 는 즉시 throw). */
+/** POST 래퍼 — 네트워크 오류·5xx 만 2회 재시도(4xx 는 즉시 throw). 재시도 전 백오프. */
 export async function api(path, body = {}, { retries = 2 } = {}) {
   let lastErr;
   for (let attempt = 0; attempt <= retries; attempt++) {
+    let retryable = false;
     try {
       const res = await fetch(`${base()}${path}`, {
         method: 'POST',
@@ -52,17 +53,20 @@ export async function api(path, body = {}, { retries = 2 } = {}) {
       if (!res.ok) {
         const err = new ApiError(res.status, data.detail ?? data.error, path);
         if (res.status >= 500 && attempt < retries) {
-          lastErr = err;
-          continue;
+          lastErr = err; // 5xx — 백오프 후 재시도
+          retryable = true;
+        } else {
+          throw err;
         }
-        throw err;
+      } else {
+        return data;
       }
-      return data;
     } catch (e) {
       if (e instanceof ApiError) throw e;
-      lastErr = e; // 네트워크 오류 — 재시도
+      lastErr = e; // 네트워크 오류 — 백오프 후 재시도
+      retryable = true;
     }
-    await sleep(500 * (attempt + 1));
+    if (retryable && attempt < retries) await sleep(500 * (attempt + 1));
   }
   throw lastErr;
 }

package/claude.mjs

@@ -58,7 +58,11 @@ export function runClaude(opts) {
       '--max-turns', String(maxTurns),
       ...(resumeSessionId ? ['--resume', resumeSessionId] : []),
     ];
-    const child = spawn(CLAUDE_BIN, args, { cwd, env: process.env });
+    // detached(POSIX)=신규 프로세스 그룹 — 취소·타임아웃이 claude 가 띄운 도구 자식(bash 등)까지
+    // 종료시킨다. detached 로 파이프(stdio)는 그대로 동작한다.
+    const child = spawn(CLAUDE_BIN, args, {
+      cwd, env: process.env, detached: process.platform !== 'win32',
+    });
     let settled = false;
     let cancelled = false;
     let timedOut = false;
@@ -170,14 +174,19 @@ export function runClaude(opts) {
   });
 }
 
-/** 프로세스 그룹 종료(SIGTERM) — 자식 셸까지 남지 않게 detached 옵션은 쓰지 않고 프로세스만. */
+/** 프로세스 그룹 종료(SIGTERM → 5s → SIGKILL) — claude 세션이 띄운 자식 프로세스까지. */
 function killTree(child) {
+  const killGroup = (signal) => {
+    try {
+      if (process.platform === 'win32') {
+        if (child.pid) spawn('taskkill', ['/pid', String(child.pid), '/T', '/F']);
+      } else if (child.pid) {
+        process.kill(-child.pid, signal); // detached 그룹 전체
+      }
+    } catch { /* already dead */ }
+  };
   try {
-    child.kill('SIGTERM');
-    setTimeout(() => {
-      try {
-        child.kill('SIGKILL');
-      } catch { /* already dead */ }
-    }, 5000).unref?.();
+    killGroup('SIGTERM');
+    setTimeout(() => killGroup('SIGKILL'), 5000).unref?.();
   } catch { /* already dead */ }
 }

package/git.mjs

@@ -54,6 +54,25 @@ export async function currentBranch(cwd) {
   return run(cwd, 'rev-parse', '--abbrev-ref', 'HEAD');
 }
 
+/** 이전 작업이 남긴 e2e-task/* 브랜치 위에서 시작했다면 저장소 기본 브랜치로 복귀한 뒤 그 이름 반환.
+ *  실패·취소된 작업은 HEAD 를 작업 브랜치에 남기는데, 그대로 두면 다음 작업의 base 가
+ *  검토되지 않은 이전 작업 코드 위에 쌓인다 — implement 전 반드시 탈출시킨다.
+ *  기본 브랜치는 origin/HEAD(symbolic-ref)에서 유도, 실패 시 'main' 폴백. */
+export async function escapeTaskBranch(cwd) {
+  const branch = await currentBranch(cwd);
+  if (!branch.startsWith('e2e-task/')) return branch;
+  let base = 'main';
+  try {
+    const ref = await run(cwd, 'symbolic-ref', 'refs/remotes/origin/HEAD');
+    const derived = ref.replace(/^refs\/remotes\/origin\//, '');
+    if (derived && !derived.startsWith('e2e-task/')) base = derived;
+  } catch {
+    /* origin/HEAD 미설정 — main 폴백 */
+  }
+  await run(cwd, 'checkout', base);
+  return base;
+}
+
 export function taskBranchName(taskId) {
   return `e2e-task/${String(taskId).slice(0, 8)}`;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@suseona/runner",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "suseona e2e 작업 에이전트 로컬 러너 — 내 컴퓨터의 Claude Code 로 구현→테스트→리뷰→배포를 자동 수행",
   "type": "module",
   "license": "UNLICENSED",

package/runner.mjs

@@ -18,7 +18,7 @@ import { dirname, join } from 'node:path';
 import { api, runnerIdFromToken, sleep } from './api.mjs';
 import { parsePhaseMarker, probeClaude, runClaude } from './claude.mjs';
 import {
-  currentBranch, ensureTaskBranch, GitError, mergeToBase, preflight, probeFolder, taskBranchName,
+  ensureTaskBranch, escapeTaskBranch, GitError, mergeToBase, preflight, probeFolder, taskBranchName,
 } from './git.mjs';
 
 // runner/.env.local 자동 로드 — 이미 설정된 env 는 덮지 않는다(셀 export 우선).
@@ -108,7 +108,14 @@ class EventBuffer {
 
   async stop() {
     clearInterval(this.timer);
-    await this.flush();
+    // 종결 보고 직전 최종 플러시 — 실패해도 버퍼가 남으므로 짧은 백오프로 몇 번 더 시도.
+    for (let i = 0; i < 3 && (this.events.length || this.taskMeta); i++) {
+      await this.flush();
+      if (this.events.length || this.taskMeta) await sleep(1000 * (i + 1));
+    }
+    if (this.events.length || this.taskMeta) {
+      log(`최종 flush 실패 — ${this.events.length}건·meta 유실(단계는 계속 진행)`);
+    }
   }
 
   push(stepId, kind, data) {
@@ -127,7 +134,12 @@ class EventBuffer {
       });
       mergeCancels(res);
     } catch (e) {
-      log(`event flush 실패(${e.message}) — ${events.length}건 유실`);
+      // 전송 실패 — 버퍼 선두로 되돌려 다음 플러시(1s)가 재전송한다(재시도 백오프는 api.mjs).
+      // base_branch 등 meta 유실은 잘못된 브랜치 병합으로 이어지므로 절대 버리지 않는다.
+      this.events.unshift(...events);
+      if (taskMeta) this.taskMeta = taskMeta;
+      if (this.events.length > 2000) this.events.splice(0, this.events.length - 2000); // 무한 증식 방지
+      log(`event flush 실패(${e.message}) — ${this.events.length}건 재시도 대기`);
     }
   }
 }
@@ -181,10 +193,11 @@ async function handleStep({ step, task, project }) {
   }
 }
 
-/** implement — git 프리플라이트 → 작업 브랜치 → claude(신규 세션). base_branch 를 서버에 보고. */
+/** implement — git 프리플라이트 → (이전 작업 브랜치 탈출) → 작업 브랜치 → claude(신규 세션).
+ *  base_branch 를 서버에 보고(유실 방지: EventBuffer 가 재전송한다). */
 async function runImplement({ step, task, project, buf, cancelled }) {
   await preflight(project.folder_path);
-  const base = await currentBranch(project.folder_path);
+  const base = await escapeTaskBranch(project.folder_path);
   const branch = await ensureTaskBranch(project.folder_path, task.id);
   buf.taskMeta = { base_branch: base, branch };
   log(`브랜치 준비: ${branch} (base ${base})`);
@@ -219,8 +232,13 @@ async function runDeploy({ step, task, project, buf, cancelled }) {
 
   if (project.deploy_command) {
     const output = await new Promise((resolve) => {
+      // detached(POSIX)=신규 프로세스 그룹 — 타임아웃·취소가 셸뿐 아니라 그 자식(wrangler 등)
+      // 까지 종료시킨다. 취소한 작업의 배포가 고아 프로세스로 완료되는 것을 막는다.
       const child = spawn(project.deploy_command, {
-        cwd: project.folder_path, shell: true, env: process.env,
+        cwd: project.folder_path,
+        shell: true,
+        env: process.env,
+        detached: process.platform !== 'win32',
       });
       let out = '';
       const onData = (c) => {
@@ -229,9 +247,9 @@ async function runDeploy({ step, task, project, buf, cancelled }) {
       child.stdout.on('data', onData);
       child.stderr.on('data', onData);
       const cancelTimer = setInterval(() => {
-        if (cancelled()) child.kill('SIGTERM');
+        if (cancelled()) killGroup(child, 'SIGTERM');
       }, 1000);
-      const killer = setTimeout(() => child.kill('SIGKILL'), DEPLOY_TIMEOUT_S * 1000);
+      const killer = setTimeout(() => killGroup(child, 'SIGKILL'), DEPLOY_TIMEOUT_S * 1000);
       child.on('close', (code) => {
         clearInterval(cancelTimer);
         clearTimeout(killer);
@@ -256,6 +274,20 @@ async function runDeploy({ step, task, project, buf, cancelled }) {
 
 // ─── 메인 루프 ────────────────────────────────────────────────────────────────
 
+/** 프로세스 그룹 종료 — detached 스폰한 자식의 그룹(pid 음수) 전체에 시그널.
+ *  셸(spawn shell:true)을 죽여도 실제 커맨드가 살아남는 것을 방지한다. */
+function killGroup(child, signal) {
+  try {
+    if (process.platform === 'win32') {
+      if (child.pid) spawn('taskkill', ['/pid', String(child.pid), '/T', '/F']);
+    } else if (child.pid) {
+      process.kill(-child.pid, signal);
+    }
+  } catch {
+    /* already dead */
+  }
+}
+
 async function heartbeat() {
   try {
     const res = await api('/ai/e2e/agent/runner/heartbeat', {