@pentoshi/clai 3.8.21 → 3.8.22

package/dist/agent/runner.js

@@ -37,6 +37,7 @@ import { fsWrite, isOutsideWorkingDirectory, resolveFsToolPath, } from "../tools
 import { stripSentinelTokens, parseToolCall, recognizeBareToolJson, looksLikeTruncatedToolCall, salvageTruncatedWrite, salvageTruncatedWriteFromNative, countToolFences, parseAllToolCalls, groupToolCallsForExecution, buildTurnHistory, collapseRepeatedText, textBeforeToolCall, formatToolArgs, looksLikePentestTask, looksLikeBuildTask, looksLikeInformationalQuery, looksLikeIdleOrSocialPrompt, looksLikeActionNarration, looksLikeWebActionNarration, looksLikePlanNarration, looksLikeErrorDiagnosisWithFixIntent, localHttpProbeIsFailure, localHttpProbeIsSuccess, requiresFreshWebSearch, freshnessGuardMessage, buildWorkflowDirective, narrowNmapOperationDirective, pentestWorkflowDirective, pentestNoLocalServerDirective, shouldDimToolChatter, looksLikePromptLeak, } from "./tool-call-parser.js";
 import { createSessionPolicy, isPreApprovalAllowedTool, isPlanModeAllowedShellCommand, isPlanModeAllowedTool, isPlanApprovedByStatus, planHasOpenWork, isAbortError, shouldEnableImageOcr, } from "./session-policy.js";
 import { saveToolOutput, summarizeOutput, formatToolContext, } from "./tool-output-formatting.js";
+import { codingSessionFromContext, isProtocolPlaceholderOutput, progressPauseMode, } from "./progress-pause-policy.js";
 import { renderPlanForTerminal, planContextMessage, handlePlanTool, resolvePlanTaskId, } from "./plan-tool.js";
 import { absorbLooseWorkIntoLedger, applyDestinationCwd, canMarkTaskDone, hasLocalRuntimeProof, hasRemoteWorkProof, isDevServerCall, isEvidenceWorkTool, isFeatureImplementationCall, isPackageInstallCommand, isPlanPreflightTool, isPortListeningOutput, isReadOnlyReconTool, isRemoteActiveTestCall, isRemoteObservationTask, isRemoteReconToolCall, isRuntimeObservationTask, isScaffoldCreateCommand, isServerReadyOutput, ledgerFromTaskEvidence, pickPendingTaskForToolCall, recordTaskWorkSuccess, resolveUserDestinationHint, taskEvidenceFromLedger, TOOL_ABORT_GRACE_MS, toolHardBudgetMs, toolStallBudgetMs, userAskedForFeatureApp, } from "./task-evidence.js";
 import { buildSessionStateBlock, inferNextHint, upsertSessionStateMessage, } from "./session-state.js";
@@ -793,6 +794,15 @@ export async function runAgentTurn(prompt, options = {}) {
         const hasHistory = (options.history?.length ?? 0) > 0;
         const buildLike = buildLikeTurn;
         const pentestLike = looksLikePentestTask(prompt, options.history);
+        /** Coding/build sessions never hard-pause mid-turn for the progress governor. */
+        let codingSession = codingSessionFromContext({
+            buildLike,
+            planKind: activePlan?.kind,
+        });
+        let pauseMode = progressPauseMode({
+            codingSession,
+            autoConfirm: Boolean(options.autoConfirm),
+        });
         const continueExistingOutcome = /^(?:continue|resume|proceed|keep\s+going|finish|next)\b/i.test(prompt.trim()) ||
             Boolean(activePlan && !isPlanTerminal(activePlan));
         const outcomeState = await openOutcomeState({
@@ -1767,27 +1777,52 @@ export async function runAgentTurn(prompt, options = {}) {
                         ((call.name === "shell.exec" || call.name === "shell.start") &&
                             isPackageInstallCommand(String(call.args.command ?? "")));
             }
-            const governed = governProgress(governorState, "activity", {
-                evidenceDelta: newEvidence.length,
-                hypothesisDelta,
-                repetitionScore: loopGuard.getAttemptCount(call.name, call.args) > 1 ? 1 : 0,
-                policy: {
-                    resourceEnvelope: Math.max(12, maxSteps),
-                    emergencyCeiling: Math.max(70, maxSteps * 3),
-                    reflectionAfterNoDelta: 3,
-                    pauseAfterNoDelta: 6,
-                    repetitionThreshold: 0.8,
-                },
-            });
-            governorState = governed.state;
-            if (governed.recommendation === "reflect") {
-                deferredPostToolMessages.push({
-                    role: "system",
-                    content: `PROGRESS GOVERNOR: ${governed.reason}. Reassess the current premise and choose the next action that can produce criterion-linked evidence.`,
+            // Protocol-repair placeholders are not live work — never let them
+            // accumulate into a mid-turn pause (they used to look like failed tools).
+            if (!isProtocolPlaceholderOutput(result.output)) {
+                const governed = governProgress(governorState, "activity", {
+                    evidenceDelta: newEvidence.length,
+                    hypothesisDelta,
+                    repetitionScore: loopGuard.getAttemptCount(call.name, call.args) > 1 ? 1 : 0,
+                    policy: {
+                        resourceEnvelope: Math.max(12, maxSteps),
+                        // Coding builds get a much higher ceiling; never use the tight
+                        // default that stopped multi-file scaffolds after a handful of steps.
+                        emergencyCeiling: codingSession
+                            ? Math.max(200, maxSteps * 5)
+                            : Math.max(70, maxSteps * 3),
+                        reflectionAfterNoDelta: codingSession ? 5 : 3,
+                        pauseAfterNoDelta: codingSession ? 24 : 6,
+                        repetitionThreshold: 0.8,
+                    },
                 });
-            }
-            else if (governed.recommendation === "paused_budget") {
-                governorPauseReason = governed.reason;
+                governorState = governed.state;
+                if (governed.recommendation === "reflect") {
+                    deferredPostToolMessages.push({
+                        role: "system",
+                        content: `PROGRESS GOVERNOR: ${governed.reason}. Reassess the current premise and choose the next action that can produce criterion-linked evidence.` +
+                            (codingSession
+                                ? " Keep working — coding builds do not stop for a continue prompt."
+                                : ""),
+                    });
+                }
+                else if (governed.recommendation === "paused_budget") {
+                    if (pauseMode === "never") {
+                        // Soft reset so we do not re-trip every subsequent tool.
+                        governorState = {
+                            ...governed.state,
+                            consecutiveNoDelta: 0,
+                        };
+                        deferredPostToolMessages.push({
+                            role: "system",
+                            content: `PROGRESS GOVERNOR (soft, coding build): ${governed.reason}. ` +
+                                "Change approach if stuck, but keep implementing — do not stop for user confirmation.",
+                        });
+                    }
+                    else {
+                        governorPauseReason = governed.reason;
+                    }
+                }
             }
             await saveOutcomeState(outcomeState);
             loopGuard.recordAttempt(step, call.name, call.args, result.ok, result.exitCode);
@@ -2033,14 +2068,53 @@ export async function runAgentTurn(prompt, options = {}) {
             // advances when the previous iteration actually executed a tool.
             step = productiveSteps;
             if (governorPauseReason) {
-                const richSummary = await buildRichStopSummary(messages, session, productiveSteps);
-                outcomeState.outcome.status = "paused_budget";
-                await saveOutcomeState(outcomeState);
-                moveTurn("paused_budget", governorPauseReason);
-                lastAnswer = richSummary;
-                return finishTurn(lastAnswer, productiveSteps, "paused_budget", outcomeState.outcome.criteria
-                    .filter((criterion) => criterion.required && criterion.status !== "proven")
-                    .map((criterion) => criterion.statement), governorPauseReason);
+                // Non-coding: always ask continue/stop. Coding never sets this reason
+                // (pauseMode === "never"), but guard anyway.
+                if (pauseMode === "never") {
+                    governorPauseReason = undefined;
+                    governorState = {
+                        ...governorState,
+                        consecutiveNoDelta: 0,
+                    };
+                }
+                else {
+                    const confirmPort = options.confirm;
+                    let keepGoing = false;
+                    if (confirmPort?.confirmContinue) {
+                        try {
+                            keepGoing = await confirmPort.confirmContinue(productiveSteps, governorPauseReason);
+                        }
+                        catch {
+                            keepGoing = false;
+                        }
+                        finally {
+                            restoreInteractiveStdin();
+                        }
+                    }
+                    if (keepGoing) {
+                        writeNotice("info", "continuing after progress pause", chalk.dim(`  ℹ continuing after pause (${governorPauseReason}) — change approach if stuck\n`));
+                        deferredPostToolMessages.push({
+                            role: "system",
+                            content: `User chose CONTINUE after progress pause (${governorPauseReason}). ` +
+                                "Do not repeat the same failing step; change approach and produce new evidence.",
+                        });
+                        governorPauseReason = undefined;
+                        governorState = {
+                            ...governorState,
+                            consecutiveNoDelta: 0,
+                        };
+                    }
+                    else {
+                        const richSummary = await buildRichStopSummary(messages, session, productiveSteps);
+                        outcomeState.outcome.status = "paused_budget";
+                        await saveOutcomeState(outcomeState);
+                        moveTurn("paused_budget", governorPauseReason);
+                        lastAnswer = richSummary;
+                        return finishTurn(lastAnswer, productiveSteps, "paused_budget", outcomeState.outcome.criteria
+                            .filter((criterion) => criterion.required && criterion.status !== "proven")
+                            .map((criterion) => criterion.statement), governorPauseReason);
+                    }
+                }
             }
             options.signal?.throwIfAborted();
             let call;
@@ -3216,6 +3290,19 @@ export async function runAgentTurn(prompt, options = {}) {
                         else {
                             session.planApproved.value = true;
                         }
+                        // Re-derive pause policy from the new plan kind (coding builds
+                        // must not hard-pause even if the free-text prompt was generic).
+                        const kindArg = typeof res.call.args.kind === "string"
+                            ? res.call.args.kind
+                            : undefined;
+                        codingSession = codingSessionFromContext({
+                            buildLike,
+                            planKind: kindArg,
+                        });
+                        pauseMode = progressPauseMode({
+                            codingSession,
+                            autoConfirm: Boolean(options.autoConfirm),
+                        });
                     }
                     // User Esc/Ctrl+C only — never cancel siblings because a delete failed
                     // or a confirm was declined; the model must see every tool result.
@@ -3335,6 +3422,8 @@ export async function runAgentTurn(prompt, options = {}) {
                 }
             }
         }
+        // Hard iteration ceiling (hundreds of steps) — rare. Mid-turn governor
+        // pauses already confirm for non-coding; coding never hard-pauses there.
         const richSummary = await buildRichStopSummary(messages, session, productiveSteps);
         lastAnswer = richSummary;
         outcomeState.outcome.status = "paused_budget";