@pentoshi/clai 3.8.1 → 3.8.3

package/dist/agent/runner.js

@@ -779,6 +779,8 @@ export async function runAgentTurn(prompt, options = {}) {
         // normal continuation is governed by evidence and resource deltas above.
         const maxIterations = Math.max(210, computeMaxIterations(stepBudget));
         let productiveSteps = 0;
+        /** Successful file mutation this turn — kills false "error diagnosed but not fixed". */
+        let sawSuccessfulMutation = false;
         let step = -1;
         let nextToolEventId = 0;
         const alreadyPrintedIds = new Set();
@@ -2575,12 +2577,17 @@ export async function runAgentTurn(prompt, options = {}) {
                     const planNarrated = (buildLikeTurn || pentestLikeTurn) &&
                         !activePlan &&
                         looksLikePlanNarration(cleaned);
-                    const errorFixNarration = looksLikeErrorDiagnosisWithFixIntent(cleaned);
+                    // Only force "diagnosed but not fixed" when the model is still
+                    // narrating a fix without having applied one this turn. Post-fix
+                    // summaries ("I've fixed…", build passed) must never re-enter.
+                    const errorFixNarration = !sawSuccessfulMutation &&
+                        looksLikeErrorDiagnosisWithFixIntent(cleaned);
                     const shouldRetryBeforeFinalizing = productiveSteps === 0 ||
                         planNarrated ||
                         (session.planApproved.value &&
                             planHasOpenWorkNow &&
                             (narratedAction || errorFixNarration)) ||
+                        // errorFix only when no mutation yet (gate is in errorFixNarration)
                         (session.planApproved.value && errorFixNarration) ||
                         (buildLikeTurn && errorFixNarration);
                     if (wantsAction &&
@@ -2933,6 +2940,22 @@ export async function runAgentTurn(prompt, options = {}) {
                     truncatedToolRetries = 0;
                     malformedFenceRetries = 0;
                     bareToolJsonRetries = 0;
+                    if (res.ok &&
+                        (res.call.name === "fs.edit" ||
+                            res.call.name === "fs.write" ||
+                            res.call.name === "fs.writeMany" ||
+                            res.call.name === "fs.replaceLines" ||
+                            res.call.name === "fs.append")) {
+                        // Mutation landed — post-fix summaries must not re-force tools.
+                        sawSuccessfulMutation = true;
+                    }
+                    // A fresh failed localhost probe re-opens the diagnosis→fix gate.
+                    if ((res.call.name === "http.fetch" ||
+                        res.call.name === "web.fetch" ||
+                        res.call.name === "shell.exec") &&
+                        localHttpProbeIsFailure(res.result.output ?? res.contextOutput ?? "")) {
+                        sawSuccessfulMutation = false;
+                    }
                     if (res.ok && isEvidenceWorkTool(res.call.name)) {
                         recovery.prematureComplete = 0;
                         recovery.actionIntent = 0;