@caupulican/pi-adaptative 0.96.3 → 0.96.5

package/dist/core/agent-session.js

@@ -8,7 +8,7 @@ import { NATIVE_TOOL_PROTOCOL_RESIDUE_ERROR } from "@caupulican/pi-agent-core/to
 import { modelsAreEqual } from "@caupulican/pi-ai/models";
 import { cleanupSessionResources } from "@caupulican/pi-ai/session-resources";
 import { streamSimple } from "@caupulican/pi-ai/stream";
-import { getAgentDir } from "../config.js";
+import { getAgentDir, VERSION, VERSION_SOURCE_AVAILABLE } from "../config.js";
 import { resourceDir, stateFile } from "./agent-paths.js";
 import { formatNoApiKeyFoundMessage, formatNoModelSelectedMessage } from "./auth-guidance.js";
 import { buildForegroundEnvelope, formatForegroundEnvelopeObservation } from "./autonomy/foreground-envelope.js";
@@ -35,6 +35,7 @@ import { hasGoalContinuationControl } from "./goals/goal-tool-names.js";
 import { parseExplicitGoalStartAuthority } from "./goals/natural-language-goal.js";
 import { constrainStreamIdleToHttpTimeout } from "./http-dispatcher.js";
 import { HumanInputController } from "./human-input-controller.js";
+import { DURABLE_LEARNING_MEMORY_POLICY_VERSION, DurableLearningState } from "./learning/durable-learning-state.js";
 import { appendLearningDecisionSnapshot, getLearningDecisionSnapshots } from "./learning/session-learning-decision.js";
 import { SkillCurator } from "./learning/skill-curator.js";
 import { isWarmableLocalModel, LocalPrefixWarmController } from "./local-prefix-warm-controller.js";
@@ -228,14 +229,19 @@ export class AgentSession {
                     this._foregroundRecovery.observeAssistant(assistantMsg);
                 }
             }
-            if (event.type === "agent_end" && !this._willRetryAfterAgentEnd(event)) {
-                const reflectionTurn = this._reflection.analyzeCompletedTurn(event.messages);
-                // The shared projection excludes raw tool-result payloads and bounds both roles before any
-                // deterministic memory sync. No reflection provider request is scheduled at this boundary.
-                this._memory.scheduleTurnSync(reflectionTurn.userText, reflectionTurn.assistantText);
-                if (reflectionTurn.trigger !== "none")
-                    this._reflection.queueCurrentTurnCue(reflectionTurn.trigger);
-                this._goals.endQueuedOwnerChatGoalExecution();
+            if (event.type === "agent_end") {
+                const willRetry = this._willRetryAfterAgentEnd(event);
+                // Settle the exact cue-bearing run before completed-turn analysis can queue a later cue.
+                this._reflection.finishCurrentTurnCue(event.messages, { willRetry });
+                if (!willRetry) {
+                    const reflectionTurn = this._reflection.analyzeCompletedTurn(event.messages);
+                    // The shared projection excludes raw tool-result payloads and bounds both roles before any
+                    // deterministic memory sync. No reflection provider request is scheduled at this boundary.
+                    this._memory.scheduleTurnSync(reflectionTurn.userText, reflectionTurn.assistantText);
+                    if (reflectionTurn.trigger !== "none")
+                        this._reflection.queueCurrentTurnCue(reflectionTurn.trigger);
+                    this._goals.endQueuedOwnerChatGoalExecution();
+                }
             }
             // Error/abort turns bypass the normal steering boundary, and a completion can arrive after the
             // final normal poll. agent_end is the fallback before foreground recovery decides whether queued
@@ -315,6 +321,8 @@ export class AgentSession {
         this._customTools = config.customTools ?? [];
         this._cwd = config.cwd;
         this._agentDir = agentDir;
+        this._isChildSession = (config.isChildSession ?? process.env.PI_CHILD_SESSION === "1") || isWorkerSession();
+        this._durableLearningState = this._isChildSession ? undefined : DurableLearningState.forAgentDir(agentDir);
         this._skillVault = new SkillVaultController({
             getSkills: () => this._resourceLoader.getActiveSkills(),
             getMaxBodyBytes: () => resolveActiveSkillBodyByteLimit(this.model?.contextWindow),
@@ -668,6 +676,10 @@ export class AgentSession {
             getSettingsManager: () => this.settingsManager,
             getSessionManager: () => this.sessionManager,
             getAgentDir: () => this._agentDir,
+            getDurableLearningState: () => this._durableLearningState,
+            getRuntimeVersion: () => (VERSION_SOURCE_AVAILABLE ? VERSION : undefined),
+            getMemoryPolicyVersion: () => DURABLE_LEARNING_MEMORY_POLICY_VERSION,
+            warn: (message) => this._emit({ type: "warning", message }),
             getCwd: () => this._cwd,
             getSkillsForAudit: () => this._resourceLoader.getActiveSkills(),
             isChildSession: () => this._isChildSession,
@@ -700,7 +712,6 @@ export class AgentSession {
             : undefined;
         this._isExplicitModel = config.isExplicitModel ?? false;
         this._isExplicitThinking = config.isExplicitThinking ?? false;
-        this._isChildSession = (config.isChildSession ?? process.env.PI_CHILD_SESSION === "1") || isWorkerSession();
         this._baseToolsOverride = config.baseToolsOverride;
         this._sessionStartEvent = config.sessionStartEvent ?? { type: "session_start", reason: "startup" };
         this._runtimeBuilder = new RuntimeBuilder({
@@ -1388,14 +1399,18 @@ export class AgentSession {
         if (goalRecovered) {
             void this.sendCustomMessage({
                 customType: "runaway-recovery",
-                content: "A bounded harness guard ended the previous agent loop, but the durable goal remains active and must continue automatically. Do not repeat the same failed operation unchanged. Inspect the recorded failure, change tool or approach, and keep working unless evidence proves a true owner/approval boundary.",
+                content: info.reason === "stagnant_tool_cycle"
+                    ? "A bounded harness guard ended an unchanged tool-result cycle, but the durable goal remains active and must continue automatically. Reuse the latest returned state; do not call the same status/read cycle again. Execute an available state-changing or finalization action, wait once on the true dependency, or record a concrete blocker."
+                    : "A bounded harness guard ended the previous agent loop, but the durable goal remains active and must continue automatically. Do not repeat the same failed operation unchanged. Inspect the recorded failure, change tool or approach, and keep working unless evidence proves a true owner/approval boundary.",
                 display: false,
                 details: info,
             }, { deliverAs: "nextTurn" });
         }
         const cause = info.reason === "provider_turn_limit"
             ? `the configured provider-turn limit of ${info.repeats} requests was reached`
-            : `the model repeated the same tool call ${info.repeats} times in a row without making progress`;
+            : info.reason === "stagnant_tool_cycle"
+                ? `the same tool-call cycle returned identical results ${info.repeats} times`
+                : `the model repeated the same tool call ${info.repeats} times in a row without making progress`;
         this._emit({
             type: "warning",
             message: `Bounded guard ended this run: ${cause}.${goalRecovered ? " The active goal remains scheduled; the next pass must use a different approach." : ""}`,
@@ -2217,7 +2232,7 @@ export class AgentSession {
             // Queue one durable logical cue. ProviderRequestContextController owns its transient preview
             // and accepted-plan commit, so it never enters or needs cleanup from agent history.
             if (!options?.internalContextType)
-                this._reflection.queueCurrentTurnCue("root-turn");
+                this._reflection.queueExternalRootTurnCue();
             // Add user message (built earlier, before the router judge, so it could be painted
             // immediately — see the early message_start emit above).
             messages.push(promptMessage);
@@ -2874,7 +2889,7 @@ export class AgentSession {
     // =========================================================================
     async navigateTree(targetId, options = {}) {
         const result = await this._treeNavigator.navigateTree(targetId, options);
-        this._reflection.invalidateCurrentTurnCueStateCache();
+        this._reflection.invalidateCurrentTurnCueStateCache({ releaseActiveClaim: !result.cancelled });
         return result;
     }
     getUserMessagesForForking() {