@caupulican/pi-adaptative 0.80.68 → 0.80.70

package/dist/modes/interactive/interactive-mode.js

@@ -4814,6 +4814,23 @@ export class InteractiveMode {
      * warranted, run the in-process {@link AgentSession.runReflectionPass} as a fire-and-forget
      * background microtask. No subprocess, no blocking of the UI.
      */
+    /**
+     * Resolve the model + thinking level the native reflection pass should use, from auto-learn
+     * settings (`model`, `thinkingLevel`). The configured model is honored only when its provider is
+     * AVAILABLE (api key / logged in) — otherwise we fall back to the session model (undefined). This
+     * lets the user pick a balanced/cheaper reflection model without risking an unusable one.
+     */
+    _resolveReflectionModel(settings) {
+        let model;
+        if (settings.model && settings.model !== "active") {
+            const resolved = resolveCliModel({ cliModel: settings.model, modelRegistry: this.session.modelRegistry });
+            if (resolved.model && this.session.modelRegistry.hasConfiguredAuth(resolved.model)) {
+                model = resolved.model;
+            }
+        }
+        const thinkingLevel = settings.thinkingLevel ?? "low";
+        return { model, thinkingLevel };
+    }
     maybeRunNativeReflection(messages) {
         if (!this.isNativeReflectionEnabled())
             return;
@@ -4839,13 +4856,22 @@ export class InteractiveMode {
             .filter(Boolean)
             .join("\n");
         // Stable per-turn id so a duplicate scheduling/retry can't double-count the reflection cost.
-        const lastId = messages[messages.length - 1]?.id;
-        const reportId = lastId ? `reflection:${lastId}` : undefined;
+        // Messages carry no `id` on the real path (only timestamps), so derive the key from the last
+        // message's timestamp + the turn size — present on every real turn, stable across a retry of the
+        // same agent_end, and distinct between turns. Falls back to a content signature if needed.
+        const last = messages[messages.length - 1];
+        const turnKey = last?.id ?? (last?.timestamp != null ? `${last.timestamp}:${recentTurnText.length}` : undefined);
+        const reportId = turnKey ? `reflection:${turnKey}` : undefined;
+        // User-configurable reflection model + thinking (auto-learn settings), restricted to AVAILABLE
+        // (authed) models — falls back to the session model when unset or unavailable.
+        const { model, thinkingLevel } = this._resolveReflectionModel(settings);
         void this.session
             .runReflectionPass({
             signals: { trigger, toolCallCount, hadCorrection, contextHeadroomPct, usefulLately: 0 },
             recentTurnText,
             reportId,
+            model,
+            thinkingLevel,
         })
             .catch(() => {
             // best-effort background learning; never disrupt the session