@code-yeongyu/senpi 2026.5.13 → 2026.5.15-2

package/dist/core/agent-session.js

@@ -15,7 +15,8 @@
 import { randomUUID } from "node:crypto";
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
 import { basename, dirname, resolve } from "node:path";
-import { cleanupSessionResources, getSupportedThinkingLevels, isContextOverflow, modelsAreEqual, resetApiProviders, supportsMax, supportsXhigh, } from "@earendil-works/pi-ai";
+import { cleanupSessionResources, isContextOverflow, modelsAreEqual, resetApiProviders } from "@earendil-works/pi-ai";
+import { expandTildePath } from "../config.js";
 import { theme } from "../modes/interactive/theme/theme.js";
 import { stripFrontmatter } from "../utils/frontmatter.js";
 import { sleep } from "../utils/sleep.js";
@@ -33,6 +34,7 @@ import { getModelNarrowingPatterns, resolveModelScope } from "./model-resolver.j
 import { expandPromptTemplate } from "./prompt-templates.js";
 import { buildSessionContext, CURRENT_SESSION_VERSION, getLatestCompactionEntry, } from "./session-manager.js";
 import { createSyntheticSourceInfo } from "./source-info.js";
+import { getSupportedThinkingLevels, supportsMax, supportsXhigh } from "./thinking-levels.js";
 import { createLocalBashOperations } from "./tools/bash.js";
 import { createAllToolDefinitions } from "./tools/index.js";
 import { createToolDefinitionFromAgentTool } from "./tools/tool-definition-wrapper.js";
@@ -1376,8 +1378,11 @@ export class AgentSession {
         if (options.expectedRevision !== undefined && options.expectedRevision !== this._messageRevision) {
             return { applied: false, reason: "stale" };
         }
-        this._compactionAbortController = new AbortController();
-        this._emit({ type: "compaction_start", reason: options.reason });
+        const ownsController = this._compactionAbortController === undefined;
+        if (ownsController) {
+            this._compactionAbortController = new AbortController();
+            this._emit({ type: "compaction_start", reason: options.reason });
+        }
         try {
             const execution = await this._executeCompaction({
                 reason: options.reason,
@@ -1406,6 +1411,38 @@ export class AgentSession {
             this._compactionAbortController = undefined;
         }
     }
+    _beginExtensionCompactionFeedback(reason) {
+        if (!this._compactionAbortController) {
+            this._compactionAbortController = new AbortController();
+            this._emit({ type: "compaction_start", reason });
+        }
+        return this._compactionAbortController.signal;
+    }
+    _updateExtensionCompactionFeedback(options) {
+        if (!this._compactionAbortController && !this._autoCompactionAbortController)
+            return;
+        this._emit({
+            type: "compaction_progress",
+            reason: options.reason,
+            ...(options.delta !== undefined ? { delta: options.delta } : {}),
+            ...(options.text !== undefined ? { text: options.text } : {}),
+        });
+    }
+    _endExtensionCompactionFeedback(options) {
+        const controller = this._compactionAbortController;
+        if (!controller)
+            return;
+        const aborted = options.aborted ?? controller.signal.aborted;
+        this._emit({
+            type: "compaction_end",
+            reason: options.reason,
+            result: undefined,
+            aborted,
+            willRetry: false,
+            errorMessage: aborted ? undefined : options.errorMessage,
+        });
+        this._compactionAbortController = undefined;
+    }
     async _executeCompaction(request) {
         if (!this.model) {
             throw new Error(formatNoModelSelectedMessage());
@@ -1563,8 +1600,14 @@ export class AgentSession {
         if (assistantIsFromBeforeCompaction) {
             return;
         }
-        // Case 1: Overflow - LLM returned context overflow error
-        if (sameModel && isContextOverflow(assistantMessage, contextWindow)) {
+        // Case 1: Overflow - LLM returned context overflow error.
+        // If the saved assistant provider differs from the currently selected provider alias,
+        // still recover as overflow when the current context is also at the compaction limit.
+        const contextUsage = this.getContextUsage();
+        const currentContextNeedsCompaction = contextUsage !== undefined &&
+            contextUsage.tokens !== null &&
+            shouldCompact(contextUsage.tokens, contextUsage.contextWindow, settings);
+        if (isContextOverflow(assistantMessage, contextWindow) && (sameModel || currentContextNeedsCompaction)) {
             if (this._overflowRecoveryAttempted) {
                 this._emit({
                     type: "compaction_end",
@@ -1628,14 +1671,20 @@ export class AgentSession {
         this._emit({ type: "compaction_start", reason: "pre_prompt" });
         this._compactionAbortController = new AbortController();
         try {
-            await this._executeCompaction({
+            const execution = await this._executeCompaction({
                 reason: "pre_prompt",
                 willRetry: false,
                 lastAssistantMessage,
                 skipAbortedCheck,
             });
+            if (!execution.accepted && isContextOverflow(lastAssistantMessage, this.model?.contextWindow ?? 0)) {
+                this._overflowRecoveryAttempted = false;
+            }
         }
         catch (error) {
+            if (isContextOverflow(lastAssistantMessage, this.model?.contextWindow ?? 0)) {
+                this._overflowRecoveryAttempted = false;
+            }
             const errorMessage = error instanceof Error ? error.message : "compaction failed";
             const aborted = errorMessage === "Compaction cancelled" || (error instanceof Error && error.name === "AbortError");
             this._emit({
@@ -1659,6 +1708,8 @@ export class AgentSession {
         this._autoCompactionAbortController = new AbortController();
         try {
             if (!this.model) {
+                if (reason === "overflow")
+                    this._overflowRecoveryAttempted = false;
                 this._emit({
                     type: "compaction_end",
                     reason,
@@ -1670,6 +1721,8 @@ export class AgentSession {
             }
             const authResult = await this._modelRegistry.getApiKeyAndHeaders(this.model);
             if (!authResult.ok || !authResult.apiKey) {
+                if (reason === "overflow")
+                    this._overflowRecoveryAttempted = false;
                 this._emit({
                     type: "compaction_end",
                     reason,
@@ -1681,6 +1734,8 @@ export class AgentSession {
             }
             const preparation = prepareCompaction(this.sessionManager.getBranch(), this.settingsManager.getCompactionSettings());
             if (!preparation) {
+                if (reason === "overflow")
+                    this._overflowRecoveryAttempted = false;
                 this._emit({
                     type: "compaction_end",
                     reason,
@@ -1692,6 +1747,8 @@ export class AgentSession {
             }
             const execution = await this._executeCompaction({ reason, willRetry });
             if (!execution.accepted) {
+                if (reason === "overflow")
+                    this._overflowRecoveryAttempted = false;
                 return;
             }
             if (willRetry) {
@@ -1714,6 +1771,8 @@ export class AgentSession {
             }
         }
         catch (error) {
+            if (reason === "overflow")
+                this._overflowRecoveryAttempted = false;
             const errorMessage = error instanceof Error ? error.message : "compaction failed";
             const aborted = errorMessage === "Compaction cancelled" || (error instanceof Error && error.name === "AbortError");
             this._emit({
@@ -1936,6 +1995,9 @@ export class AgentSession {
                     }
                 })();
             },
+            beginCompaction: (options) => this._beginExtensionCompactionFeedback(options.reason),
+            updateCompaction: (options) => this._updateExtensionCompactionFeedback(options),
+            endCompaction: (options) => this._endExtensionCompactionFeedback(options),
             getMessageRevision: () => this.getMessageRevision(),
             applyCompaction: (precomputed, options) => this.applyCompaction(precomputed, options),
             getSystemPrompt: () => this.systemPrompt,
@@ -2094,15 +2156,48 @@ export class AgentSession {
      * Context overflow errors are NOT retryable (handled by compaction instead).
      */
     _isRetryableError(message) {
-        if (message.stopReason !== "error" || !message.errorMessage)
+        if (!message.errorMessage)
             return false;
         // Context overflow is handled by compaction, not retry
         const contextWindow = this.model?.contextWindow ?? 0;
         if (isContextOverflow(message, contextWindow))
             return false;
         const err = message.errorMessage;
-        // Match: overloaded_error, provider returned error, rate limit, 429, 500, 502, 503, 504, service unavailable, network/connection errors (including connection lost), WebSocket transport closes/errors, fetch failed, request ended without sending chunks, HTTP/2 closed before response, terminated, retry delay exceeded
-        return /overloaded|provider.?returned.?error|rate.?limit|too many requests|429|500|502|503|504|service.?unavailable|server.?error|internal.?error|network.?error|connection.?error|connection.?refused|connection.?lost|websocket.?closed|websocket.?error|other side closed|fetch failed|upstream.?connect|reset before headers|socket hang up|ended without|http2 request did not get a response|timed? out|timeout|terminated|retry delay/i.test(err);
+        if (message.stopReason === "aborted") {
+            return /timed? out|timeout/i.test(err);
+        }
+        if (message.stopReason !== "error")
+            return false;
+        // Match: overloaded_error, provider returned error, rate limit, 429, 500, 502, 503, 504, service unavailable, network/connection errors (including connection lost), WebSocket transport closes/errors, fetch failed, premature stream endings, HTTP/2 closed before response, terminated, retry delay exceeded
+        return /overloaded|provider.?returned.?error|rate.?limit|too many requests|429|500|502|503|504|service.?unavailable|server.?error|internal.?error|network.?error|connection.?error|connection.?refused|connection.?lost|websocket.?closed|websocket.?error|other side closed|fetch failed|upstream.?connect|reset before headers|socket hang up|ended without|stream ended before message_stop|http2 request did not get a response|timed? out|timeout|terminated|retry delay/i.test(err);
+    }
+    _getProviderRetryDelayMs(errorMessage) {
+        const retryAfterMsMatch = errorMessage.match(/\bretry[-_ ]?after[-_ ]?ms\s*[:=]\s*(\d+(?:\.\d+)?)/i);
+        if (retryAfterMsMatch) {
+            const delayMs = Math.ceil(Number(retryAfterMsMatch[1]));
+            return Number.isFinite(delayMs) && delayMs > 0 ? delayMs : undefined;
+        }
+        const retryAfterSecondsMatch = errorMessage.match(/\bretry[-_ ]?after\s*[:=]\s*(\d+(?:\.\d+)?)/i);
+        if (retryAfterSecondsMatch) {
+            const delayMs = Math.ceil(Number(retryAfterSecondsMatch[1]) * 1000);
+            return Number.isFinite(delayMs) && delayMs > 0 ? delayMs : undefined;
+        }
+        const retryInMatch = errorMessage.match(/\b(?:retry|try again|wait)\s+(?:after|in)\s*(\d+(?:\.\d+)?)\s*(milliseconds?|msecs?|ms|seconds?|secs?|s|minutes?|mins?|m)\b/i);
+        if (!retryInMatch) {
+            return undefined;
+        }
+        const value = Number(retryInMatch[1]);
+        if (!Number.isFinite(value) || value <= 0) {
+            return undefined;
+        }
+        const unit = retryInMatch[2].toLowerCase();
+        if (unit === "m" || unit.startsWith("min")) {
+            return Math.ceil(value * 60_000);
+        }
+        if (unit.startsWith("s")) {
+            return Math.ceil(value * 1000);
+        }
+        return Math.ceil(value);
     }
     /**
      * Handle retryable errors with exponential backoff.
@@ -2134,13 +2229,27 @@ export class AgentSession {
             this._resolveRetry(); // Resolve so waitForRetry() completes
             return false;
         }
-        const delayMs = settings.baseDelayMs * 2 ** (this._retryAttempt - 1);
+        const errorMessage = message.errorMessage || "Unknown error";
+        const providerDelayMs = this._getProviderRetryDelayMs(errorMessage);
+        const maxRetryDelayMs = this.settingsManager.getProviderRetrySettings().maxRetryDelayMs;
+        if (providerDelayMs !== undefined && providerDelayMs > maxRetryDelayMs) {
+            this._emit({
+                type: "auto_retry_end",
+                success: false,
+                attempt: this._retryAttempt,
+                finalError: `Provider requested retry delay ${providerDelayMs}ms, exceeding configured maximum ${maxRetryDelayMs}ms`,
+            });
+            this._retryAttempt = 0;
+            this._resolveRetry();
+            return false;
+        }
+        const delayMs = providerDelayMs ?? settings.baseDelayMs * 2 ** (this._retryAttempt - 1);
         this._emit({
             type: "auto_retry_start",
             attempt: this._retryAttempt,
             maxAttempts: settings.maxRetries,
             delayMs,
-            errorMessage: message.errorMessage || "Unknown error",
+            errorMessage,
         });
         // Remove error message from agent state (keep in session for history)
         const messages = this.agent.state.messages;
@@ -2622,7 +2731,7 @@ export class AgentSession {
      * @returns The resolved output file path.
      */
     exportToJsonl(outputPath) {
-        const filePath = resolve(outputPath ?? `session-${new Date().toISOString().replace(/[:.]/g, "-")}.jsonl`);
+        const filePath = resolve(expandTildePath(outputPath ?? `session-${new Date().toISOString().replace(/[:.]/g, "-")}.jsonl`));
         const dir = dirname(filePath);
         if (!existsSync(dir)) {
             mkdirSync(dir, { recursive: true });