@xdevops/issue-auto-finish 1.0.73 → 1.0.75

package/dist/{chunk-RFMARJ5D.js → chunk-MC7YPVJG.js}

@@ -48,7 +48,7 @@ import {
   createAIRunner,
   getRunnerCapabilities,
   isShuttingDown
-} from "./chunk-XAMJNIXE.js";
+} from "./chunk-66OK4LG3.js";
 import {
   logger,
   runWithIssueContext
@@ -656,16 +656,17 @@ var IssueTracker = class extends BaseTracker {
     logger5.info("Issue state updated", { issueIid, state });
     eventBus.emitTyped("issue:stateChanged", { issueIid, state, record });
   }
-  markFailed(issueIid, error, failedAtState) {
+  markFailed(issueIid, error, failedAtState, isRetryable) {
     const record = this.collection[this.key(issueIid)];
     if (!record) return;
     record.state = "failed" /* Failed */;
     record.lastError = error;
     record.failedAtState = failedAtState;
+    record.lastErrorRetryable = isRetryable;
     record.attempts += 1;
     record.updatedAt = (/* @__PURE__ */ new Date()).toISOString();
     this.save();
-    logger5.warn("Issue marked as failed", { issueIid, error, failedAtState, attempts: record.attempts });
+    logger5.warn("Issue marked as failed", { issueIid, error, failedAtState, attempts: record.attempts, isRetryable });
     eventBus.emitTyped("issue:failed", { issueIid, error, failedAtState, record });
   }
   isProcessing(issueIid) {
@@ -697,7 +698,7 @@ var IssueTracker = class extends BaseTracker {
   getDrivableIssues(maxRetries, stalledThresholdMs) {
     return this.getAllRecords().filter((record) => {
       const lm = this.lifecycleFor(record);
-      if (lm.isDrivable(record.state, record.attempts, maxRetries)) return true;
+      if (lm.isDrivable(record.state, record.attempts, maxRetries, record.lastErrorRetryable)) return true;
       if (this.isStalled(getIid(record), stalledThresholdMs)) return true;
       return false;
     });
@@ -1249,7 +1250,8 @@ ${t("basePhase.rulesSection", { rules: matchedRulesText })}` : basePrompt;
       );
       throw new AIExecutionError(this.phaseName, `Phase ${this.phaseName} failed: ${shortReason}`, {
         output: result.output,
-        exitCode: result.exitCode
+        exitCode: result.exitCode,
+        isRetryable: this.classifyRetryable(result)
       });
     }
     this.persistSessionId(result.sessionId);
@@ -1341,6 +1343,22 @@ ${t("basePhase.rulesSection", { rules: matchedRulesText })}` : basePrompt;
     }
     return result;
   }
+  /**
+   * 判断 AI 执行失败是否为永久性错误（不可重试）。
+   * 模型不存在、API key 无效等配置问题重试不会成功。
+   */
+  classifyRetryable(result) {
+    const msg = (result.errorMessage ?? result.output ?? "").toLowerCase();
+    const permanentPatterns = [
+      /model\b.*\b(?:not found|not supported|unavailable|service info not found)/,
+      /invalid.?api.?key/,
+      /authentication.*(?:failed|denied|error)/,
+      /permission.?denied/,
+      /billing/,
+      /quota.*exceeded/
+    ];
+    return !permanentPatterns.some((p) => p.test(msg));
+  }
   /**
    * Heuristic: a resume failure is typically an immediate process exit
    * (exit code != 0, empty output) caused by an invalid/expired session ID.
@@ -1348,7 +1366,14 @@ ${t("basePhase.rulesSection", { rules: matchedRulesText })}` : basePrompt;
   isResumeFailure(result) {
     if (result.success) return false;
     const msg = (result.errorMessage ?? "").toLowerCase();
-    return msg.includes("session") || msg.includes("resume") || msg.includes("not found") || result.output.length === 0 && result.exitCode !== null && result.exitCode !== 0;
+    if (msg.includes("session") || msg.includes("resume") || msg.includes("session_id")) {
+      return true;
+    }
+    if (result.output.length === 0 && result.exitCode !== null && result.exitCode !== 0) {
+      const isConfigError = msg.includes("model") || msg.includes("api key") || msg.includes("authentication");
+      return !isConfigError;
+    }
+    return false;
   }
   persistSessionId(sessionId) {
     if (sessionId) {
@@ -2250,7 +2275,7 @@ var ActionLifecycleManager = class {
    * - running → 不可驱动（stalled 由外部叠加）
    * - done/skipped → 不可驱动
    */
-  isDrivable(state, attempts, maxRetries) {
+  isDrivable(state, attempts, maxRetries, lastErrorRetryable) {
     if (state === "phase_done" /* PhaseDone */) return true;
     if (state === "phase_approved" /* PhaseApproved */) return true;
     if (state === "phase_running" /* PhaseRunning */) return false;
@@ -2261,6 +2286,7 @@ var ActionLifecycleManager = class {
       case "ready":
         return true;
       case "failed":
+        if (lastErrorRetryable === false) return false;
         return attempts < maxRetries;
       case "waiting":
       case "running":
@@ -2384,6 +2410,19 @@ var ActionLifecycleManager = class {
   derivePhaseStatuses(currentState, currentPhase) {
     const result = {};
     let passedCurrent = false;
+    if (currentState === "failed" /* Failed */ && currentPhase) {
+      for (const phase of this.def.phases) {
+        if (passedCurrent) {
+          result[phase.name] = "pending";
+        } else if (phase.name === currentPhase) {
+          result[phase.name] = "failed";
+          passedCurrent = true;
+        } else {
+          result[phase.name] = "completed";
+        }
+      }
+      return result;
+    }
     if ((currentState === "phase_running" /* PhaseRunning */ || currentState === "phase_done" /* PhaseDone */) && currentPhase) {
       for (const phase of this.def.phases) {
         if (passedCurrent) {
@@ -3990,13 +4029,14 @@ async function executeCompletion(ctx, deps, phaseResult, _wtGitMap) {
 var logger18 = logger.child("FailureHandler");
 async function handleFailure(err, issue, wtCtx, deps) {
   const errorMsg = err.message;
-  logger18.error("Issue processing failed", { iid: issue.iid, error: errorMsg });
+  const isRetryable = err instanceof AIExecutionError ? err.isRetryable : true;
+  logger18.error("Issue processing failed", { iid: issue.iid, error: errorMsg, isRetryable });
   metrics.incCounter("iaf_issues_failed_total");
   const currentRecord = deps.tracker.get(issue.iid);
   const failedAtState = currentRecord?.state || "pending" /* Pending */;
   const wasReset = failedAtState === "pending" /* Pending */ && currentRecord?.attempts === 0;
   if (failedAtState !== "failed" /* Failed */ && !wasReset) {
-    deps.tracker.markFailed(issue.iid, errorMsg.slice(0, 500), failedAtState);
+    deps.tracker.markFailed(issue.iid, errorMsg.slice(0, 500), failedAtState, isRetryable);
   }
   if (wasReset) {
     logger18.info("Issue was reset during processing, skipping failure marking", { iid: issue.iid });
@@ -4347,7 +4387,7 @@ var PipelineOrchestrator = class {
       } catch {
       }
     });
-    this.tracker.delete(issueIid);
+    this.tracker.updateState(issueIid, "skipped" /* Skipped */);
     logger19.info("Issue cancelled", { issueIid });
   }
   retryFromPhase(issueIid, phase) {
@@ -4661,9 +4701,6 @@ E2E \u6D4B\u8BD5\u5C06\u5C1D\u8BD5\u4F7F\u7528 config.json \u4E2D\u7684\u9ED8\u8
   async restartPreview(issueIid) {
     const record = this.tracker.get(issueIid);
     if (!record) throw new IssueNotFoundError(issueIid);
-    if (record.state !== "completed" /* Completed */) {
-      throw new InvalidStateError(record.state, "Issue not in completed state");
-    }
     const wtCtx = this.computeWorktreeContext(issueIid, record.branchName);
     if (!fsSync.existsSync(wtCtx.workDir)) {
       throw new InvalidStateError(record.state, "Worktree no longer exists");
@@ -5082,4 +5119,4 @@ export {
   PipelineOrchestrator,
   BrainstormService
 };
-//# sourceMappingURL=chunk-RFMARJ5D.js.map
+//# sourceMappingURL=chunk-MC7YPVJG.js.map