@kenkaiiii/ggcoder 5.55.1 → 5.56.1

package/dist/app-sidecar.js

@@ -2196,7 +2196,12 @@ async function createSession(deps, opts) {
         }
         finally {
             const cancelled = runLifecycle.isCancellationRequested(generation);
+            const verificationProblem = cancelled ? null : session.getVerificationProblem();
+            if (runSucceeded && verificationProblem && ownsGeneration) {
+                broadcastError("error", "verification incomplete", new Error(verificationProblem));
+            }
             if (runSucceeded &&
+                !verificationProblem &&
                 !cancelled &&
                 cancelGeneration === cancelGenAtStart &&
                 countAssistantMessages(session.getMessages()) > assistantsBeforeRun) {
@@ -2219,6 +2224,7 @@ async function createSession(deps, opts) {
             // clear the widget before run_end paints the idle activity bar.
             if (runSucceeded &&
                 !cancelled &&
+                !verificationProblem &&
                 approvedPlanPath !== null &&
                 (await queueApprovedPlanProgressSync())) {
                 try {
@@ -2235,7 +2241,7 @@ async function createSession(deps, opts) {
                 }
             }
             if (ownsGeneration) {
-                finishOwnedGeneration(generation, false, runSucceeded ? "completed" : "failed");
+                finishOwnedGeneration(generation, false, verificationProblem ? "unverified" : runSucceeded ? "completed" : "failed");
             }
             // A cancelled injected run is still owned by the surrounding autopilot
             // cycle; its outer finalizer emits the one terminal cancelled run_end.
@@ -2244,6 +2250,7 @@ async function createSession(deps, opts) {
                     cancelledRunEndGenerations.add(generation);
                 broadcast("run_end", {
                     ...(cancelled ? { cancelled: true } : {}),
+                    ...(verificationProblem ? { unverified: true } : {}),
                     runState: runLifecycle.state,
                 });
             }
@@ -2355,7 +2362,7 @@ async function createSession(deps, opts) {
     // control flow lives in driveAutopilotCycle (core/autopilot-cycle.ts) so
     // every exit path is unit-tested; this only wires the real dependencies.
     async function runAutopilotCycle(originalRequest) {
-        if (!autopilot || autopilotCancelled)
+        if (!autopilot || autopilotCancelled || session.getVerificationProblem())
             return;
         const generation = runLifecycle.begin(abortOwnedWork).generation;
         pendingCancelDrain = null;
@@ -2371,6 +2378,7 @@ async function createSession(deps, opts) {
                 // round available.
                 maxRounds: pendingPlanPath !== null ? MAX_AUTOPILOT_ROUNDS + 2 : MAX_AUTOPILOT_ROUNDS,
                 isCancelled: () => autopilotCancelled,
+                verificationProblem: () => session.getVerificationProblem(),
                 // An injected run entering plan mode WITHOUT submitting (enter_plan,
                 // no exit_plan) halts the cycle — Ken never prompts into a read-only
                 // plan-mode session. A submitted plan takes the planPending branch.
@@ -2383,7 +2391,7 @@ async function createSession(deps, opts) {
                 // Auto-accept: the inlined POST /plan/accept body. Returns false when
                 // the plan generation moved since the review (user acted) — the cycle
                 // exits silently and the user's action stands.
-                acceptPlan: async () => {
+                acceptPlan: async (reason) => {
                     if (pendingPlanPath === null || planGeneration !== planGenAtReview)
                         return false;
                     const planPath = pendingPlanPath;
@@ -2400,11 +2408,11 @@ async function createSession(deps, opts) {
                     clearPendingPlan();
                     // Keep the approval marker ahead of the reset, then seed the reset
                     // with the sidecar's canonical count from the actual plan file.
-                    broadcast("autopilot_plan_accepted", {});
+                    broadcast("autopilot_plan_accepted", reason ? { reason } : {});
                     broadcast("session_reset", { planTotal });
                     broadcast("plan_progress", planProgressPayload());
                     // Persisted into the NEW session so a resume shows the marker.
-                    void session.persistAutopilotMarker("plan_approved");
+                    void session.persistAutopilotMarker("plan_approved", { reason });
                     return true;
                 },
                 runImplement: () => {
@@ -2458,9 +2466,10 @@ async function createSession(deps, opts) {
                             version: 1,
                             phase: "done",
                             afterMessageCount: session.getPersistedTranscriptCount(),
+                            ...event.data,
                         });
                         broadcast(event.type, { ...event.data, copySeed: seed });
-                        void session.persistAutopilotMarker("done");
+                        void session.persistAutopilotMarker("done", event.data);
                         return;
                     }
                     broadcast(event.type, event.data);
@@ -2477,7 +2486,7 @@ async function createSession(deps, opts) {
         finally {
             autopilotActive = false;
             session.setIdealReviewSuppressed(autopilot);
-            finishOwnedGeneration(generation, true);
+            finishOwnedGeneration(generation, true, session.getVerificationProblem() ? "unverified" : "completed");
             queueMicrotask(() => void runStrandedQueue());
         }
     }