lsd-pi 1.1.10 → 1.2.1

package/packages/pi-coding-agent/dist/core/agent-session.js

@@ -735,10 +735,38 @@ export class AgentSession {
     _formatSkillInvocation(skillName, args) {
         return this._expandSkillByName(skillName, args);
     }
+    _isPremiumReconModel() {
+        const model = this.model;
+        if (!model)
+            return false;
+        const provider = model.provider.toLowerCase();
+        const id = model.id.toLowerCase();
+        return ((provider === "anthropic" && (id.includes("opus") || id.includes("sonnet"))) ||
+            (provider === "openai" && /^gpt-5(\.|-|$)/.test(id)) ||
+            (provider === "google" && id.includes("gemini-2.5-pro")));
+    }
+    _buildScoutPolicyGuidelines(toolNames) {
+        if (!toolNames.includes("subagent"))
+            return [];
+        const guidelines = [
+            "Recon planning policy: use 0 scouts for narrow known-file work, 1 scout for one broad unfamiliar subsystem, and 2-4 parallel scouts only when the work spans multiple loosely-coupled subsystems.",
+            "Scout-first reconnaissance policy: when you need architecture context across multiple files or folders, do not map the subsystem by reading file-after-file yourself. Launch the scout subagent first, then continue with lsp and targeted reads.",
+            "If your next step would be broad exploration rather than a targeted lookup, prefer scout before doing more read/find/grep sweeps yourself.",
+            "When you choose scout, call subagent directly with valid parameters: { agent, task } for one scout, or { tasks: [{ agent, task }, ...] } for parallel scouts.",
+            "Scout is reconnaissance-only. Do not use it as the reviewer, auditor, or final issue-ranker; use it to map files, ownership, and likely hotspots for later evaluation.",
+            "If the work spans multiple loosely-coupled subsystems, prefer parallel scout subagents so each scout maps one area and the parent model reads the summaries instead of the raw files.",
+            "For broad review or audit requests, use scout only as a prep step to map architecture and hotspots; the parent model or a reviewer should make the final judgments.",
+        ];
+        if (this._isPremiumReconModel()) {
+            guidelines.push("You are on a premium model. Spend tokens on synthesis, planning, and edits — not broad reconnaissance. Delegate exploratory reading to scout whenever the scope is not already narrow.");
+        }
+        return guidelines;
+    }
     _rebuildSystemPrompt(toolNames) {
         const validToolNames = toolNames.filter((name) => this._toolRegistry.has(name));
         const toolSnippets = {};
         const promptGuidelines = [];
+        promptGuidelines.push(...this._buildScoutPolicyGuidelines(validToolNames));
         for (const name of validToolNames) {
             const snippet = this._toolPromptSnippets.get(name);
             if (snippet) {
@@ -1388,7 +1416,7 @@ export class AgentSession {
         if (!this._modelRegistry.isProviderRequestReady(model.provider)) {
             throw new Error(`No API key for ${model.provider}/${model.id}`);
         }
-        const thinkingLevel = this._getThinkingLevelForModelSwitch();
+        const thinkingLevel = this._getThinkingLevelForModelSwitch(model);
         await this._applyModelChange(model, thinkingLevel, "set", options);
     }
     /**
@@ -1419,7 +1447,7 @@ export class AgentSession {
         const next = scopedModels[nextIndex];
         // Explicit scoped model thinking level overrides current session level;
         // undefined scoped model thinking level inherits the current session preference.
-        const thinkingLevel = this._getThinkingLevelForModelSwitch(next.thinkingLevel);
+        const thinkingLevel = this._getThinkingLevelForModelSwitch(next.model, next.thinkingLevel);
         await this._applyModelChange(next.model, thinkingLevel, "cycle", options);
         return { model: next.model, thinkingLevel: this.thinkingLevel, isScoped: true };
     }
@@ -1434,7 +1462,7 @@ export class AgentSession {
         const len = availableModels.length;
         const nextIndex = direction === "forward" ? (currentIndex + 1) % len : (currentIndex - 1 + len) % len;
         const nextModel = availableModels[nextIndex];
-        const thinkingLevel = this._getThinkingLevelForModelSwitch();
+        const thinkingLevel = this._getThinkingLevelForModelSwitch(nextModel);
         await this._applyModelChange(nextModel, thinkingLevel, "cycle", options);
         return { model: nextModel, thinkingLevel: this.thinkingLevel, isScoped: false };
     }
@@ -1507,11 +1535,17 @@ export class AgentSession {
     supportsThinking() {
         return !!this.model?.reasoning;
     }
-    _getThinkingLevelForModelSwitch(explicitLevel) {
+    _getThinkingLevelForModelSwitch(targetModel, explicitLevel) {
         if (explicitLevel !== undefined) {
             return explicitLevel;
         }
-        if (!this.supportsThinking()) {
+        if (targetModel.provider === "anthropic" &&
+            targetModel.reasoning === true &&
+            this.settingsManager.getAnthropicAdaptiveByDefault() &&
+            supportsAdaptiveThinking(targetModel.id)) {
+            return "adaptive";
+        }
+        if (!targetModel.reasoning) {
             return this.settingsManager.getDefaultThinkingLevel() ?? DEFAULT_THINKING_LEVEL;
         }
         return this.thinkingLevel;