@nklisch/pi-enhanced 0.1.7 → 0.1.9

package/node_modules/@nklisch/pi-plugins/node_modules/@nklisch/pi-subagents/src/session/thinking-level.ts

@@ -0,0 +1,42 @@
+import { clampThinkingLevel, getSupportedThinkingLevels } from "@earendil-works/pi-ai/compat";
+import type { Model } from "@earendil-works/pi-ai";
+import type { ThinkingLevel } from "#src/types";
+
+/** Pi's default when no session setting is available. */
+export const DEFAULT_THINKING_LEVEL: ThinkingLevel = "medium";
+
+/**
+ * Resolve the level Pi will use for a model before its AgentSession exists.
+ *
+ * The session factory passes this resolved value to Pi, while the Subagent
+ * record replaces it with AgentSession.thinkingLevel after construction. That
+ * two-step flow keeps queued/launch status useful without allowing a local
+ * approximation to outrank the SDK's authoritative value.
+ */
+export function resolveEffectiveThinkingLevel(
+  model: unknown,
+  requested?: ThinkingLevel,
+  inherited?: ThinkingLevel,
+): ThinkingLevel {
+  const level = requested ?? inherited ?? DEFAULT_THINKING_LEVEL;
+  // With no model yet, preserve the level Pi would receive rather than forcing
+  // `off`: createAgentSession may still resolve a configured default model.
+  // The created child session remains authoritative and supplies the clamp.
+  if (model == null) return level;
+  if (!isThinkingModel(model)) return "off";
+
+  const supported = getSupportedThinkingLevels(model);
+  return supported.includes(level)
+    ? level
+    : clampThinkingLevel(model, level);
+}
+
+function isThinkingModel(value: unknown): value is Model<any> {
+  if (typeof value !== "object" || value === null) return false;
+  const candidate = value as { provider?: unknown; id?: unknown; reasoning?: unknown };
+  return typeof candidate.provider === "string"
+    && candidate.provider.length > 0
+    && typeof candidate.id === "string"
+    && candidate.id.length > 0
+    && typeof candidate.reasoning === "boolean";
+}

package/node_modules/@nklisch/pi-plugins/node_modules/@nklisch/pi-subagents/src/tools/agent-tool.ts

@@ -13,7 +13,7 @@ import { buildAgentGuidelines, buildDetails, buildTypeListText, textResult } fro
 import { renderAgentResult } from "#src/tools/result-renderer";
 import { type ModelInfo, resolveSpawnConfig } from "#src/tools/spawn-config";
 import type { ParentSessionInfo, Subagent } from "#src/types";
-import { type AgentDetails, formatDuration, getDisplayName } from "#src/ui/display";
+import { type AgentDetails, formatDuration, formatModelThinking, getDisplayName } from "#src/ui/display";
 
 // ---- Deps interfaces ----
 
@@ -113,9 +113,9 @@ export class AgentTool {
 					: `Error: ${record.error}`
 				: partial ?? "No output.";
 			return textResult(
-				`Model: ${record.modelLabel}\nRuntime: ${formatDuration(record.startedAt, record.completedAt)}\n\n${resultText}`,
+				`Model: ${formatModelThinking(record.modelLabel, record.effectiveThinkingLevel)}\nRuntime: ${formatDuration(record.startedAt, record.completedAt)}\n\n${resultText}`,
 				buildDetails(
-					{ ...config.presentation.detailBase, modelName: record.modelLabel },
+					{ ...config.presentation.detailBase, modelName: record.modelLabel, thinkingLevel: record.effectiveThinkingLevel },
 					record,
 				),
 			);

package/node_modules/@nklisch/pi-plugins/node_modules/@nklisch/pi-subagents/src/tools/background-spawner.ts

@@ -1,6 +1,7 @@
 import type { ParentSnapshot } from "#src/lifecycle/parent-snapshot";
 import type { AgentSpawnConfig } from "#src/lifecycle/subagent-manager";
 import { textResult } from "#src/tools/helpers";
+import { formatModelThinking } from "#src/ui/display";
 import type { ResolvedSpawnConfig } from "#src/tools/spawn-config";
 import type { ParentSessionInfo, Subagent } from "#src/types";
 
@@ -36,7 +37,7 @@ export function spawnBackground(
       model: execution.model,
       maxTurns: execution.effectiveMaxTurns,
       inheritContext: execution.inheritContext,
-      thinkingLevel: execution.thinking,
+      thinkingLevel: execution.effectiveThinkingLevel,
       isBackground: true,
       origin: "tool",
       invocation: execution.agentInvocation,
@@ -52,7 +53,10 @@ export function spawnBackground(
     `Agent ${isQueued ? "queued" : "started"} in background.\n` +
       `Agent ID: ${id}\n` +
       `Type: ${identity.displayName}\n` +
-      `Model: ${presentation.modelName}\n` +
+      `Model: ${formatModelThinking(
+        record?.modelLabel ?? presentation.modelName,
+        record?.effectiveThinkingLevel ?? execution.effectiveThinkingLevel,
+      )}\n` +
       `Runtime: 0.0s\n` +
       `Description: ${execution.description}\n` +
       (identity.fellBack ? `Fallback: requested ${identity.rawType}; using ${identity.subagentType}\n` : "") +
@@ -66,6 +70,7 @@ export function spawnBackground(
       `Do not duplicate this agent's work.`,
     {
       ...presentation.detailBase,
+      thinkingLevel: record?.effectiveThinkingLevel ?? execution.effectiveThinkingLevel,
       toolUses: 0,
       tokens: "",
       durationMs: 0,

package/node_modules/@nklisch/pi-plugins/node_modules/@nklisch/pi-subagents/src/tools/foreground-runner.ts

@@ -12,6 +12,7 @@ import type { ParentSessionInfo, Subagent } from "#src/types";
 import {
   type AgentDetails,
   describeActivity,
+  formatModelThinking,
   formatMs,
   SPINNER,
 } from "#src/ui/display";
@@ -73,11 +74,12 @@ export async function runForeground(
     });
   };
 
-  // Animate spinner at ~80ms (smooth rotation through 10 braille frames)
+  // Each update traverses the parent TUI tree; avoid doing that at animation
+  // speed while the foreground tool already owns the progress surface.
   const spinnerInterval = setInterval(() => {
     spinnerFrame++;
     streamUpdate();
-  }, 80);
+  }, 500);
 
   streamUpdate();
 
@@ -92,7 +94,7 @@ export async function runForeground(
         model: execution.model,
         maxTurns: execution.effectiveMaxTurns,
         inheritContext: execution.inheritContext,
-        thinkingLevel: execution.thinking,
+        thinkingLevel: execution.effectiveThinkingLevel,
         origin: "tool",
         invocation: execution.agentInvocation,
         signal,
@@ -125,7 +127,7 @@ export async function runForeground(
   if (record.status === "error") {
     const partial = record.result?.trim();
     return textResult(
-      `${fallbackNote}Model: ${record.modelLabel}\nRuntime: ${formatMs(details.durationMs)}\nAgent failed: ${record.error}` +
+      `${fallbackNote}Model: ${formatModelThinking(record.modelLabel, record.effectiveThinkingLevel)}\nRuntime: ${formatMs(details.durationMs)}\nAgent failed: ${record.error}` +
         (partial ? `\n\nPartial output before the failure:\n${partial}` : ""),
       details,
     );
@@ -135,7 +137,7 @@ export async function runForeground(
   const statsParts = [`${record.toolUses} tool uses`];
   if (tokenText) statsParts.push(tokenText);
   return textResult(
-    `${fallbackNote}Model: ${record.modelLabel}\nRuntime: ${formatMs(durationMs)}\nAgent completed (${statsParts.join(", ")})${getStatusNote(record.status)}.\n\n` +
+    `${fallbackNote}Model: ${formatModelThinking(record.modelLabel, record.effectiveThinkingLevel)}\nRuntime: ${formatMs(durationMs)}\nAgent completed (${statsParts.join(", ")})${getStatusNote(record.status)}.\n\n` +
       (record.result?.trim() ?? "No output."),
     details,
   );

package/node_modules/@nklisch/pi-plugins/node_modules/@nklisch/pi-subagents/src/tools/get-result-report.ts

@@ -8,12 +8,15 @@
  */
 
 import type { SubagentStatus } from "#src/lifecycle/subagent";
+import type { ThinkingLevel } from "#src/types";
+import { formatModelThinking } from "#src/ui/display";
 
 /** The data a get_subagent_result report renders from — only what the formatter reads. */
 export interface AgentReport {
 	id: string;
 	displayName: string;
 	modelLabel: string;
+	thinkingLevel: ThinkingLevel;
 	status: SubagentStatus;
 	toolUses: number;
 	/** Pre-formatted lifetime token total; "" when zero. */
@@ -63,7 +66,7 @@ export function renderReportBody(report: AgentReport): string {
 export function formatAgentReport(report: AgentReport): string {
 	let output =
 		`Agent: ${report.id}\n` +
-		`Type: ${report.displayName} | Model: ${report.modelLabel} | Status: ${report.status} | ${renderStatsParts(report).join(" | ")}\n` +
+		`Type: ${report.displayName} | Model: ${formatModelThinking(report.modelLabel, report.thinkingLevel)} | Status: ${report.status} | ${renderStatsParts(report).join(" | ")}\n` +
 		`Description: ${report.description}\n\n`;
 	output += renderReportBody(report);
 	if (report.conversation) {

package/node_modules/@nklisch/pi-plugins/node_modules/@nklisch/pi-subagents/src/tools/get-result-tool.ts

@@ -47,6 +47,7 @@ export class GetResultTool {
 			id: record.id,
 			displayName: getDisplayName(record.type, this.registry),
 			modelLabel: record.modelLabel,
+			thinkingLevel: record.effectiveThinkingLevel,
 			status: record.status,
 			toolUses: record.toolUses,
 			tokens: formatLifetimeTokens(record),

package/node_modules/@nklisch/pi-plugins/node_modules/@nklisch/pi-subagents/src/tools/helpers.ts

@@ -18,7 +18,7 @@ export function getStatusNote(status: string): string {
 
 /** Build AgentDetails from a base + record-specific fields. */
 export function buildDetails(
-  base: Pick<AgentDetails, "displayName" | "description" | "subagentType" | "modelName" | "tags">,
+  base: Pick<AgentDetails, "displayName" | "description" | "subagentType" | "modelName" | "thinkingLevel" | "tags">,
   record: {
     toolUses: number;
     startedAt: number;

package/node_modules/@nklisch/pi-plugins/node_modules/@nklisch/pi-subagents/src/tools/result-renderer.ts

@@ -7,7 +7,7 @@
  */
 
 import type { AgentDetails, Theme } from "#src/ui/display";
-import { formatMs, formatTurns, SPINNER } from "#src/ui/display";
+import { formatModelThinking, formatMs, formatTurns, SPINNER } from "#src/ui/display";
 
 // ---- Dispatcher ----
 
@@ -42,7 +42,10 @@ export function renderRunning(details: AgentDetails, theme: Theme): string {
 
 /** Render background launch status. */
 export function renderBackground(details: AgentDetails, theme: Theme): string {
-	const identity = [details.modelName, formatMs(details.durationMs)].filter(Boolean).join(" · ");
+	const identity = [
+		formatModelThinking(details.modelName ?? "unknown model", details.thinkingLevel),
+		formatMs(details.durationMs),
+	].join(" · ");
 	return theme.fg("dim", `  \u23BF  Running in background (ID: ${details.agentId}) · ${identity}`);
 }
 
@@ -113,8 +116,8 @@ export function renderFailed(details: AgentDetails, theme: Theme): string {
  */
 export function renderStats(details: AgentDetails, theme: Theme): string {
 	const parts: string[] = [];
-	if (details.modelName) parts.push(details.modelName);
-	if (details.tags) parts.push(...details.tags);
+	parts.push(formatModelThinking(details.modelName ?? "unknown model", details.thinkingLevel));
+	if (details.tags) parts.push(...details.tags.filter((tag) => !tag.startsWith("thinking: ")));
 	if (details.turnCount != null && details.turnCount > 0) {
 		parts.push(formatTurns(details.turnCount, details.maxTurns));
 	}

package/node_modules/@nklisch/pi-plugins/node_modules/@nklisch/pi-subagents/src/tools/spawn-config.ts

@@ -14,6 +14,7 @@ import { normalizeMaxTurns } from "#src/lifecycle/turn-limits";
 import type { ModelRegistry } from "#src/session/model-resolver";
 import { formatModelLabel } from "#src/session/model-label";
 import { resolveInvocationModel } from "#src/session/model-resolver";
+import { resolveEffectiveThinkingLevel } from "#src/session/thinking-level";
 import type { AgentInvocation, SubagentType, ThinkingLevel } from "#src/types";
 import {
   type AgentDetails,
@@ -25,6 +26,8 @@ import {
 /** Model info extracted from the parent session context. */
 export interface ModelInfo {
   parentModel: Model<any> | undefined;
+  /** Parent's current level for children that inherit thinking settings. */
+  parentThinkingLevel?: ThinkingLevel;
   modelRegistry: ModelRegistry | undefined;
 }
 
@@ -42,7 +45,10 @@ export interface SpawnExecution {
   description: string;
   model: Model<any> | undefined;
   effectiveMaxTurns: number | undefined;
+  /** Requested/configured level before model-capability clamping. */
   thinking: ThinkingLevel | undefined;
+  /** Exact level passed to the child session after model-capability clamping. */
+  effectiveThinkingLevel: ThinkingLevel;
   inheritContext: boolean;
   runInBackground: boolean;
   agentInvocation: AgentInvocation;
@@ -50,9 +56,9 @@ export interface SpawnExecution {
 
 /** Presentation: display/UI values derived from identity and execution. */
 export interface SpawnPresentation {
-  modelName: string | undefined;
+  modelName: string;
   agentTags: string[];
-  detailBase: Pick<AgentDetails, "displayName" | "description" | "subagentType" | "modelName" | "tags">;
+  detailBase: Pick<AgentDetails, "displayName" | "description" | "subagentType" | "modelName" | "thinkingLevel" | "tags">;
 }
 
 /** Fully resolved config for spawning an agent — composed of domain-aligned sub-interfaces. */
@@ -110,6 +116,11 @@ export function resolveSpawnConfig(
   const model = resolution.model;
 
   const thinking = resolvedConfig.thinking;
+  const effectiveThinkingLevel = resolveEffectiveThinkingLevel(
+    model,
+    thinking,
+    modelInfo.parentThinkingLevel,
+  );
   const inheritContext = resolvedConfig.inheritContext;
   const runInBackground = resolvedConfig.runInBackground;
 
@@ -137,6 +148,7 @@ export function resolveSpawnConfig(
     description: params.description as string,
     subagentType,
     modelName,
+    thinkingLevel: effectiveThinkingLevel,
     tags: agentTags.length > 0 ? agentTags : undefined,
   };
 
@@ -148,6 +160,7 @@ export function resolveSpawnConfig(
       model,
       effectiveMaxTurns,
       thinking,
+      effectiveThinkingLevel,
       inheritContext,
       runInBackground,
       agentInvocation,

package/node_modules/@nklisch/pi-plugins/node_modules/@nklisch/pi-subagents/src/tools/steer-tool.ts

@@ -2,7 +2,7 @@ import { defineTool } from "@earendil-works/pi-coding-agent";
 import { Type } from "@sinclair/typebox";
 import { formatLifetimeTokens, textResult } from "#src/tools/helpers";
 import type { SteerOutcome, Subagent } from "#src/types";
-import { formatDuration } from "#src/ui/display";
+import { formatDuration, formatModelThinking } from "#src/ui/display";
 
 // ---- Deps interfaces ----
 
@@ -68,7 +68,7 @@ export class SteerTool {
 	private renderDelivered(record: Subagent) {
 		const tokens = formatLifetimeTokens(record);
 		const contextPercent = record.getContextPercent();
-		const stateParts: string[] = [record.modelLabel, formatDuration(record.startedAt, record.completedAt)];
+		const stateParts: string[] = [formatModelThinking(record.modelLabel, record.effectiveThinkingLevel), formatDuration(record.startedAt, record.completedAt)];
 		if (tokens) stateParts.push(tokens);
 		stateParts.push(`${record.toolUses} tool ${record.toolUses === 1 ? "use" : "uses"}`);
 		if (contextPercent !== null)
@@ -84,7 +84,7 @@ export class SteerTool {
 	}
 
 	private renderRunIdentity(record: Subagent): string {
-		return `Model: ${record.modelLabel}\nRuntime: ${formatDuration(record.startedAt, record.completedAt)}`;
+		return `Model: ${formatModelThinking(record.modelLabel, record.effectiveThinkingLevel)}\nRuntime: ${formatDuration(record.startedAt, record.completedAt)}`;
 	}
 
 	toToolDefinition() {

package/node_modules/@nklisch/pi-plugins/node_modules/@nklisch/pi-subagents/src/types.ts

@@ -2,14 +2,16 @@
  * types.ts — Type definitions for the subagent system.
  */
 
-import type { ThinkingLevel } from "@earendil-works/pi-ai";
+import type { ModelThinkingLevel } from "@earendil-works/pi-ai";
 import type { AgentSessionEvent, SessionContext as SdkSessionContext } from "@earendil-works/pi-coding-agent";
 import type { ModelRegistry } from "#src/session/model-resolver";
 
 
 export type { SteerOutcome } from "#src/lifecycle/subagent";
 export { Subagent } from "#src/lifecycle/subagent";
-export type { AgentSessionEvent, ThinkingLevel };
+export type { AgentSessionEvent };
+/** Pi's model-capability thinking levels, including the explicit `off` value. */
+export type ThinkingLevel = ModelThinkingLevel;
 
 /**
  * One message in a child session's history, typed from Pi's `SessionContext`.
@@ -86,6 +88,8 @@ export interface AgentInvocation {
 export interface SessionContext {
   readonly cwd: string;
   readonly model: unknown;
+  /** Parent session's current effective thinking level for inherited children. */
+  readonly thinkingLevel?: ThinkingLevel;
   readonly modelRegistry: ModelRegistry | undefined;
   getSystemPrompt(): string;
   readonly sessionManager: {