@prestyj/cli 5.21.0 → 5.22.1

package/dist/app-sidecar.js

@@ -35,6 +35,7 @@ import { buildNolanDigest, buildNolanAutopilotContext, buildNolanAutopilotPlanCo
 import { parseAutopilotVerdict } from "./core/autopilot-verdict.js";
 import { isWorkflowCommandText, countAssistantMessages, shouldStartAutopilotCycle, extractTurnToolCalls, isMechanicalOnlyTurn, } from "./core/autopilot-gate.js";
 import { driveAutopilotCycle, frameAutopilotInjection } from "./core/autopilot-cycle.js";
+import { describeRunVerification, describeTurnVerification } from "./core/run-status.js";
 import { validateNolanModelPref, effectiveNolanModel, } from "./core/nolan-model.js";
 import { normalizeAutopilotMarkersForHistory, normalizeAppMarkersForHistory, normalizeNolanTurnsForHistory, getHistoryMessageVisibility, replayMessagesInOrder, restoreUserRow, restoreAssistantTexts, resolveRestoredCommand, autopilotMarkerCopySeed, } from "./core/session-history.js";
 import { sessionToMarkdown, defaultExportFilename, } from "./core/session-export.js";
@@ -1830,6 +1831,11 @@ async function createSession(deps, opts) {
         recordApprovedPlanMarkers(d.text);
     });
     session.eventBus.on("thinking_delta", (d) => broadcast("thinking_delta", d));
+    session.eventBus.on("retry", (d) => {
+        if (!d.silent)
+            broadcast("retry", { reason: d.reason, attempt: d.attempt, delayMs: d.delayMs });
+    });
+    session.eventBus.on("max_turns", (d) => broadcast("max_turns", d));
     // The agent consumed queued steering at a turn boundary. Re-broadcast as the
     // usual `queued` depth update so the webview drops the pending affordance the
     // moment the message lands in the loop, not at run_end.
@@ -2139,8 +2145,10 @@ async function createSession(deps, opts) {
         // his own Ideal self-review adds latency and can corrupt the verdict shape.
         nolanAgent.setIdealReviewSuppressed(true);
         await nolanAgent.initialize();
-        // Deliberately no bus bridge: the review is silent. Errors surface via the
-        // runAutopilotReview try/catch as autopilot_error frames.
+        // Keep review text/tools silent; report usage only for whole-task accounting.
+        nolanAgent.eventBus.on("turn_end", (d) => {
+            broadcast("autopilot_usage", { outputTokens: d.usage.outputTokens });
+        });
         nolanAutoSession = nolanAgent;
         log("INFO", "app-sidecar", "nolan autopilot session ready", {
             provider: target.provider,
@@ -2182,7 +2190,7 @@ async function createSession(deps, opts) {
     }
     // Core provider-run bracket. Standalone runs own a lifecycle generation;
     // injected autopilot runs share the cycle's outer generation.
-    async function runAgent(label, run) {
+    async function runAgent(label, run, reviewPending = () => false) {
         const ownsGeneration = !runLifecycle.running;
         const generation = ownsGeneration
             ? runLifecycle.begin(abortOwnedWork).generation
@@ -2195,7 +2203,11 @@ async function createSession(deps, opts) {
         const cancelGenAtStart = cancelGeneration;
         const assistantsBeforeRun = countAssistantMessages(session.getMessages());
         let runSucceeded = false;
-        broadcast("run_start", { text: label, runState: runLifecycle.state });
+        broadcast("run_start", {
+            text: label,
+            runState: runLifecycle.state,
+            continued: !ownsGeneration,
+        });
         try {
             if (!runLifecycle.isCancellationRequested(generation))
                 await run();
@@ -2266,6 +2278,16 @@ async function createSession(deps, opts) {
                 broadcast("run_end", {
                     ...(cancelled ? { cancelled: true } : {}),
                     ...(verificationProblem ? { unverified: true } : {}),
+                    failed: !cancelled && !runSucceeded,
+                    // The cycle refuses an unresolved verification gate. Do not advertise
+                    // a review handoff that will exit before emitting any review events.
+                    reviewPending: !cancelled &&
+                        runSucceeded &&
+                        !verificationProblem &&
+                        (reviewPending() || !ownsGeneration),
+                    ...describeRunVerification(session.getVerificationEvidence(), verificationProblem),
+                    turnVerification: describeTurnVerification(session.getRunVerificationActivity(), verificationProblem),
+                    ...(verificationProblem ? { verificationReason: verificationProblem } : {}),
                     runState: runLifecycle.state,
                 });
             }
@@ -2554,7 +2576,7 @@ async function createSession(deps, opts) {
                     else {
                         await session.prompt(next.text);
                     }
-                });
+                }, () => autopilot);
                 const decision = shouldStartAutopilotCycle({
                     enabled: autopilot,
                     cancelled: autopilotCancelled,
@@ -2576,7 +2598,8 @@ async function createSession(deps, opts) {
                     });
                     await runAutopilotCycle(next.text);
                 }
-                else if (autopilot) {
+                else {
+                    broadcast("autopilot_ignored", { reason: decision.reason });
                     log("INFO", "app-sidecar", "autopilot skipped (queued turn)", {
                         reason: decision.reason,
                     });
@@ -2880,6 +2903,7 @@ async function createSession(deps, opts) {
                     mode,
                     chatAgent,
                     running,
+                    reviewPending: autopilotActive,
                     runState: runLifecycle.state,
                     thinkingLevel: session.getThinkingLevel() ?? null,
                     supportedThinkingLevels: getSupportedThinkingLevels(st.provider, st.model),
@@ -3746,7 +3770,7 @@ async function createSession(deps, opts) {
                         // while the webview keeps showing the short `/name`.
                         await session.prompt(text);
                     }
-                });
+                }, () => autopilot);
                 // After the user's run settles, kick off Nolan's auto-review loop — but
                 // only when the turn is actually reviewable (shouldStartAutopilotCycle):
                 // workflow commands (/compare, /expand, …) end with reports or
@@ -3775,7 +3799,8 @@ async function createSession(deps, opts) {
                     log("INFO", "app-sidecar", "autopilot cycle starting", { kind: decision.kind });
                     await runAutopilotCycle(text);
                 }
-                else if (autopilot) {
+                else {
+                    broadcast("autopilot_ignored", { reason: decision.reason });
                     log("INFO", "app-sidecar", "autopilot skipped", { reason: decision.reason });
                 }
                 // A prompt sent while Nolan was reviewing (build idle) queued but had no