@kenkaiiii/ggcoder 5.69.0 → 5.71.0

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

@@ -158,7 +158,7 @@ export interface AgentSessionOptions {
     sessionRootDir?: string;
     /** Register GG Coder built-in/prompt/custom slash commands. Defaults to true. */
     coderSlashCommands?: boolean;
-    /** Enable loop-break, re-grounding, and Ideal review hooks. Defaults to true. */
+    /** Enable loop-break and re-grounding hooks. Defaults to true. */
     selfCorrectionHooks?: boolean;
     /** Override the semantic-loop judge LLM call (tests). Receives the finished
      *  prompt, returns the model's raw reply. Default: one-shot `stream()` call
@@ -237,6 +237,7 @@ export declare class AgentSession {
     /** Forgets every file read; called whenever the conversation is replaced or
      *  rewound, so the model must re-read a file before changing it. */
     private clearReadTracker;
+    private recordBashReads;
     private skills;
     private cacheKeyLogged;
     private hookStats;
@@ -249,20 +250,6 @@ export declare class AgentSession {
     private hookFileEditCounts;
     private hookToolCalls;
     private backgroundVerification;
-    private idealReviewPhase;
-    /** Runtime-only suppression while Ken owns verification in autopilot mode. */
-    private idealReviewSuppressed;
-    /** Mirror of the last `hook_armed` value broadcast this run, so the event
-     *  fires only on a real edge. */
-    private idealReviewArmed;
-    /** Cached test-drift probe, keyed by the size of the edited-file set. Drift
-     *  depends only on WHICH files were edited and that set only grows, so this
-     *  keeps the arming check off the filesystem on most tool results — the probe
-     *  is several sync existsSync calls per edited file. */
-    private idealDriftProbe;
-    private readonly reviewCoverage;
-    /** Coverage follow-ups spent this run, capped by MAX_REVIEW_COVERAGE_INJECTIONS. */
-    private reviewCoverageInjected;
     /** 0 = none; 1 = first nudge sent; 2 = final stop-and-report injected. */
     private loopBreakInjected;
     private regroundingInjected;
@@ -272,8 +259,6 @@ export declare class AgentSession {
      *  injection at the next steering poll; judge failures fail open (no
      *  injection) and still consume budget + cooldown. */
     private semanticLoop;
-    /** Independent Ideal reviewer spawned once per run (score-gated). */
-    private independentReviewStarted;
     /**
      * The environment as the cached system prompt currently describes it.
      * Re-recorded on every prompt build, so a rebuild (e.g. `/add-dir`) needs no
@@ -284,11 +269,11 @@ export declare class AgentSession {
     private runStartedAt;
     /** Gate injections spent this run, capped by MAX_PROCESS_GATE_INJECTIONS. */
     private processGateInjected;
-    /** Verification gate: code edited this run, nothing proved it since. */
+    /** 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;
-    /** Mirror of the last verification `hook_armed` value, so the event fires
-     *  only on a real edge. */
-    private verificationArmed;
+    /** Mirror of the last `hook_armed` value, so the event fires only on an edge. */
+    private preFinalArmed;
     private compactionOccurred;
     /**
      * Re-grounding carry-over for post-turn compaction. `resetHookState` clears
@@ -318,6 +303,7 @@ export declare class AgentSession {
     private steeringListeners;
     private processManager?;
     private lspManager?;
+    private debugManager?;
     private subAgentManager?;
     /**
      * Out-of-band push notifications (finished children, background-process
@@ -531,10 +517,17 @@ export declare class AgentSession {
     private resetHookState;
     /**
      * Fold one agent event into the hook stat accumulators. Pure bookkeeping —
-     * the same signals the TUI's useAgentLoop collects, so the loop-break and
-     * ideal-review decisions match across the CLI and the app.
+     * the same signals the TUI's useAgentLoop collects, so loop-break decisions
+     * match across the CLI and the app.
      */
     private trackHookEvent;
+    /**
+     * Count full-file `cat` output from the step that just finished as reads, so
+     * an edit after `cat` does not cost a second read. Uses the step's results as
+     * stored in the transcript (after per-turn trimming): only bytes the model
+     * actually received count.
+     */
+    private creditBashReads;
     /**
      * Append every message added since the last flush to the session file.
      *
@@ -566,15 +559,6 @@ export declare class AgentSession {
      *  cheaper routing: judging a model's own failure patterns with a weaker
      *  model swaps false negatives for false positives. */
     private callSemanticLoopJudge;
-    /** Independent fresh-context review of the finished work (Codex Guardian
-     *  pattern). Spawns a READ-ONLY child on the ACTIVE model, waits bounded,
-     * and returns findings for the acting agent to address — or nothing when
-     *  the review passes, is unavailable, or fails (in-thread review remains the
-     *  fallback; the feature degrades, never blocks).
-     *
-     *  Runs inside the pre-stop poll, so the candidate final answer is already
-     *  held by arming and this wait cannot race a streamed answer. */
-    private runIndependentReview;
     /**
      * Turn-budget extension gate. The loop consults this instead of stopping
      * mid-task when it exhausts `maxTurns`. Grant ONLY on evidence of progress —
@@ -582,43 +566,20 @@ export declare class AgentSession {
      * with — so reuse the same stuck signals that drive the loop-breaker.
      */
     private shouldExtendTurnBudget;
-    /**
-     * Would the stop AFTER the current turn inject the Ideal review? Same inputs
-     * as the pre-stop gate below, evaluated early so clients know a candidate
-     * final answer is a review draft BEFORE it streams.
-     *
-     * The turn count is looked ahead by one on purpose. `hookStats.turns` only
-     * advances at `turn_end`, so while the model is writing the draft the counter
-     * still reads the PREVIOUS turn; the real gate sees one more. Without the
-     * lookahead a run sitting on score 3 crosses to 4 on the draft's own
-     * `turn_end` — after the text already streamed — which is precisely the
-     * appear-then-vanish flash. Over-arming by one turn point costs only live
-     * token streaming on a final answer that then shows whole; under-arming costs
-     * the flash, so this errs toward arming.
-     */
-    private wouldInjectIdealReview;
-    /** Would a stop right now inject the verification gate? Same conditions as
-     *  the pre-stop branch below, so arming and injection cannot disagree. */
-    private wouldInjectVerification;
+    /** Would a stop right now inject a pre-final follow-up? Only queued LSP
+     *  diagnostics (real errors injected below) or a mode-owned completion
+     *  review can, so clients hold the candidate answer only then. */
+    private wouldInjectBeforeFinal;
     /** Broadcast pre-final hook arming on change. Both edges matter: armed=false
-     *  after the hook fires is what lets a client stream the REVIEWED final
-     *  answer live again.
-     *
-     *  Callable before `initialize()`: the sidecar sets Ken's review suppression
-     *  on a freshly constructed session, and every arming predicate below reads
-     *  settings that `initialize()` has not loaded yet. Nothing can be armed
-     *  before the session can run a turn, and the first `tool_result`/`turn_end`
-     *  recomputes both edges — so skipping is the correct answer, not a patch. */
+     *  after the hook fires is what lets a client stream the final answer live
+     *  again. Callable before `initialize()`, when no manager exists yet. */
     private refreshHookArming;
-    private refreshVerificationArmed;
-    private refreshIdealReviewArmed;
     /**
-     * Pre-stop Ideal review phase machine. Once review starts, completion is
-     * blocked until harness-owned post-injection reads cover every changed file.
+     * Pre-stop follow-ups: LSP errors, unread child agents and background
+     * processes, and a mode-owned completion review. Verification evidence is
+     * tracked passively for run status; it never forces another turn.
      */
     private getHookFollowUpMessages;
-    private reviewLspEvidence;
-    private withReviewLspEvidence;
     /** Wraps the real run: cancels an in-flight cache prewarm and tracks run
      *  activity / last real request time for {@link prewarm}. */
     private runLoop;
@@ -727,12 +688,6 @@ export declare class AgentSession {
      */
     getCacheExpiryStatus(now?: number): CacheExpiryStatus | null;
     getPlanMode(): boolean;
-    /**
-     * Suppress only the pre-final Ideal self-review for this live session.
-     * Autopilot uses this while Ken independently owns verification; loop-break
-     * and post-compaction re-grounding remain active.
-     */
-    setIdealReviewSuppressed(suppressed: boolean): void;
     /** Queue a user message (optionally with attachments) to be injected mid-run
      *  as steering. Returns the new queue length. No-op semantics are the caller's
      *  concern. */
@@ -764,6 +719,13 @@ export declare class AgentSession {
     listBackgroundProcesses(): BackgroundProcess[];
     /** Stop a background process by id. Returns a human-readable status string. */
     killBackgroundProcess(id: string): Promise<string>;
+    /**
+     * Force-stop every background process tree, synchronously. Background
+     * commands run in their own process group, so the daemon's group kill on
+     * quit never reaches them: this is the only thing that does. Callers on a
+     * shutdown deadline run it before awaiting anything that can hang.
+     */
+    stopBackgroundProcesses(): void;
     /** Replace a host-owned system prompt in place without resetting conversation history. */
     setCustomSystemPrompt(systemPrompt: string, promptCacheKeyPrefix?: string): void;
     /**