@kenkaiiii/ggcoder 5.74.2 → 5.74.3

package/dist/core/agent-session.d.ts

@@ -188,6 +188,10 @@ export interface AgentSessionOptions {
     additionalTools?: AgentTool[];
     /** Mode-owned completion policy; absent in Coder/chat/worker sessions. */
     completionReview?: CompletionReview;
+    /** Remind the agent to submit its plan (exit_plan) when a run is about to
+     *  stop still in plan mode. Desktop only: an unsubmitted plan there leaves
+     *  nothing to review. See plan-submission-gate.ts. */
+    planSubmissionGate?: boolean;
 }
 /** Resolve the per-result cap passed to the agent loop for the active transport. */
 export declare function resolveSessionToolResultCharLimit(model: string, provider: Provider, accountId?: string): number;
@@ -269,6 +273,8 @@ export declare class AgentSession {
     private runStartedAt;
     /** Gate injections spent this run, capped by MAX_PROCESS_GATE_INJECTIONS. */
     private processGateInjected;
+    /** Plan-submission reminders spent this run (plan-submission-gate.ts). */
+    private planSubmissionNudges;
     /** Verification evidence: code edited this run, and what has proved it since.
      *  Passive tracking only — it feeds run status and autopilot, never a turn. */
     private readonly verificationGate;

package/dist/core/agent-session.js

@@ -26,6 +26,7 @@ import { createTools, createWebSearchTool, } from "../tools/index.js";
 import { partitionToolsByTier } from "../tools/tool-tiers.js";
 import { autoBackgroundedId, REVIEW_REJECTED_BEFORE_START } from "../tools/bash.js";
 import { buildProcessCompletionFollowUp } from "./process-gate.js";
+import { PLAN_SUBMISSION_NUDGE, shouldNudgePlanSubmission } from "./plan-submission-gate.js";
 import { buildSubAgentCompletionFollowUp } from "./subagent-manager.js";
 import { applyAsyncSubagentPolicy } from "./subagent-policy.js";
 import { resolveResponsesLite, resolveStrictTools, } from "./codex-request-shape.js";
@@ -178,6 +179,8 @@ export class AgentSession {
     runStartedAt = 0;
     /** Gate injections spent this run, capped by MAX_PROCESS_GATE_INJECTIONS. */
     processGateInjected = 0;
+    /** Plan-submission reminders spent this run (plan-submission-gate.ts). */
+    planSubmissionNudges = 0;
     /** Verification evidence: code edited this run, and what has proved it since.
      *  Passive tracking only — it feeds run status and autopilot, never a turn. */
     verificationGate = new VerificationGate();
@@ -1108,6 +1111,7 @@ export class AgentSession {
         };
         this.runStartedAt = Date.now();
         this.processGateInjected = 0;
+        this.planSubmissionNudges = 0;
         this.verificationGate.beginRun();
         const processes = new Set(this.processManager?.list().map((p) => p.id) ?? []);
         for (const id of this.backgroundVerification.keys()) {
@@ -1212,7 +1216,10 @@ export class AgentSession {
                 // Only host-observed successful mutations and trustworthy check results
                 // affect approval. The model's text is never evidence.
                 let verificationChanged = false;
-                if (!event.isError && args) {
+                // In plan mode write/edit can only touch `.gg/plans/*.md`; a code-file
+                // write there was refused (refusals return text, not isError). Counting
+                // it as a change left "unverified" work that blocked plan approval.
+                if (!event.isError && args && !this.planModeRef.current) {
                     if (name === "edit" || name === "write") {
                         // Check-owning files (tsconfig.json, pytest.ini, vitest.config.ts …)
                         // are tracked even when they are not source code: editing one
@@ -1770,6 +1777,22 @@ export class AgentSession {
         }
         if (diagnosticMessages.length > 0)
             return diagnosticMessages;
+        if (shouldNudgePlanSubmission({
+            enabled: this.opts.planSubmissionGate === true,
+            planMode: this.planModeRef.current,
+            aborted: this.opts.signal?.aborted === true,
+            nudgesThisRun: this.planSubmissionNudges,
+        })) {
+            this.planSubmissionNudges += 1;
+            log("INFO", "plan-gate", "Run stopping in plan mode without a submitted plan; reminding");
+            return [
+                {
+                    role: "user",
+                    content: buildNotificationSteeringText([PLAN_SUBMISSION_NUDGE]),
+                    provenance: { source: "runtime", kind: "notification", visibility: "hidden" },
+                },
+            ];
+        }
         if (this.opts.completionReview) {
             const followUp = await this.opts.completionReview.followUp((request, signal) => this.callCompletionReviewer(request, signal), this.opts.signal);
             await this.persistCompletionReviewState();
@@ -2788,6 +2811,11 @@ export class AgentSession {
         });
     }
     async newSession(preserveConversation = false) {
+        // A background post-turn compaction replaces `this.messages` when it
+        // lands. Letting it finish after the reset would swap the old history back
+        // into the fresh session (e.g. an accepted plan's build would inherit the
+        // whole planning conversation and lose the approved-plan prompt).
+        await this.settlePostTurnCompaction();
         this.cacheDiagnostics.reset();
         // Approved-plan execution is a clean checkpoint of the same conversation;
         // explicit new sessions reset the conversation identity.