@oh-my-pi/pi-coding-agent 16.4.4 → 16.4.5

package/dist/types/task/index.d.ts

@@ -20,12 +20,12 @@ export declare function isReadOnlyAgent(agent: AgentDefinition): boolean;
 export declare function formatResultOutputFallback(result: Pick<SingleResult, "output" | "stderr" | "requests">): string;
 /**
  * Advisory — never a rejection — nudging the spawner toward tailored
- * specialists when it spawns generic role-less workers and still holds spawn
- * capacity (DepthCapacity: it currently has the `task` tool). Fires when a
- * generic `task`/`sonic` spawn carries no `role`, or when one call clones
- * the same agent ≥2× all without roles. Returns undefined when no nudge applies.
+ * specific agent types when one call resolves ≥2 items to a generic
+ * `task`/`sonic` worker and the spawner still holds spawn capacity
+ * (DepthCapacity: it currently has the `task` tool). `agentNames` are the
+ * per-item resolved agent types. Returns undefined when no nudge applies.
  */
-export declare function buildSpecializationAdvisory(agentName: string | undefined, items: TaskItem[], depthCapacity: boolean): string | undefined;
+export declare function buildSpecializationAdvisory(agentNames: string[], depthCapacity: boolean): string | undefined;
 /**
  * Suggestion — never a rejection — nudging the spawner to coordinate via `irc`
  * when one call creates ≥2 live siblings and it still holds spawn capacity.
@@ -34,14 +34,15 @@ export declare function buildSpecializationAdvisory(agentName: string | undefine
 export declare function buildCoordinationAdvisory(items: TaskItem[], depthCapacity: boolean, ircEnabled: boolean): string | undefined;
 /**
  * Compose the non-blocking advisory appended to a `task` result: the
- * specialization nudge, plus — only when the siblings keep running after this
- * call (`willRunAsync`) — the coordination suggestion. Coordination is gated on
- * async because a sync fanout's siblings have already finished, so a
- * "coordinate while they run" hint would misfire. Returns undefined when
- * neither applies.
+ * specialization nudge (from the per-item resolved agent types), plus — only
+ * when some spawns keep running after this call (`willRunAsync`) — the
+ * coordination suggestion over those still-live spawns (`items`). Coordination
+ * is gated on async because a sync spawn has already finished by the time the
+ * call returns, so a "coordinate while they run" hint would misfire. Returns
+ * undefined when neither applies.
  */
 export declare function composeSpawnAdvisory(args: {
-    agentName: string | undefined;
+    agents: string[];
     items: TaskItem[];
     depthCapacity: boolean;
     ircEnabled: boolean;

package/dist/types/task/label.d.ts

@@ -0,0 +1,4 @@
+import type { ModelRegistry } from "../config/model-registry.js";
+import type { Settings } from "../config/settings.js";
+/** Compresses a delegated assignment into a one-sentence UI label via the tiny title model — fired by the executor spawn path because the task wire schema no longer carries a `description`; null on empty input or failure. */
+export declare function generateTaskLabel(assignment: string, registry: ModelRegistry, settings: Settings, sessionId?: string): Promise<string | null>;

package/dist/types/task/repair-args.d.ts

@@ -2,7 +2,7 @@
  * Repair double-encoded JSON string arguments for the task tool.
  *
  * Models occasionally JSON-escape a string value twice when emitting a
- * `task` tool call, so an `assignment` that should read
+ * `task` tool call, so a `task` field that should read
  *
  *     # Role
  *     You are a judge … "describe this" … return —
@@ -24,7 +24,8 @@
  * string.
  *
  * This is deliberately scoped to the task tool's natural-language fields
- * (`assignment`, `description`). It is NOT applied to code-bearing
+ * (`task`, shared `context`); identifier fields (`name`, `agent`)
+ * are never repaired. It is NOT applied to code-bearing
  * tools (write/edit/bash/search), where a backslash or quote is load-bearing
  * and a false-positive unescape would silently corrupt a file or command.
  */
@@ -43,11 +44,10 @@ import type { TaskParams } from "./types.js";
  */
 export declare function repairDoubleEncodedJsonString(value: string): string;
 /**
- * Repair double-encoded prose in task-tool params (`assignment`,
- * `description`, shared `context`, and each batch task item's prose fields).
- * Returns the same reference when nothing changed so callers can cheaply skip
- * work. Defensive against partially-streamed args (missing/undefined fields,
- * partial task arrays) so it is safe on the render path as well as on
- * execution.
+ * Repair double-encoded prose in task-tool params (flat `task`, shared
+ * `context`, and each batch task item's `task`). Returns the same reference
+ * when nothing changed so callers can cheaply skip work. Defensive against
+ * partially-streamed args (missing/undefined fields, partial task arrays) so
+ * it is safe on the render path as well as on execution.
  */
 export declare function repairTaskParams(params: TaskParams): TaskParams;

package/dist/types/task/types.d.ts

@@ -1,7 +1,7 @@
-import type { ThinkingLevel } from "@oh-my-pi/pi-agent-core";
 import type { Usage } from "@oh-my-pi/pi-ai";
 import { type BaseType } from "arktype";
 import type { AgentSessionEvent } from "../session/agent-session.js";
+import type { ConfiguredThinkingLevel } from "../thinking.js";
 import type { NestedRepoPatch } from "./worktree.js";
 /** Source of an agent definition */
 export type AgentSource = "bundled" | "user" | "project";
@@ -52,67 +52,52 @@ export interface SubagentLifecyclePayload {
     detached?: boolean;
 }
 /** Display cap for a normalized one-line label (roster line, registry `displayName`, prompt field). */
-export declare const ROLE_LABEL_MAX = 80;
-/** Schema bound on the raw `role` input, before it is label-normalized at every use site. */
-export declare const ROLE_INPUT_MAX = 256;
+export declare const LABEL_MAX = 80;
 export declare const taskItemSchema: import("arktype/internal/variants/object.ts").ObjectType<{
-    id?: string | undefined;
-    description?: string | undefined;
-    role?: string | undefined;
-    assignment: string;
+    name?: string | undefined;
+    agent: import("arktype/internal/attributes.ts").Default<string, "task">;
+    task: string;
 }, {}>;
 /** Single task item. Fields are optional defensively: args stream in token by token. */
 export interface TaskItem {
-    /** Stable agent id; default = generated AdjectiveNoun. */
-    id?: string;
-    /** UI label, not seen by the subagent. */
-    description?: string;
-    /** Specialist role/expertise this subagent embodies; shapes its system-prompt identity and display name. */
-    role?: string;
+    /** Stable agent name; becomes the registry/IRC id. Default = generated AdjectiveNoun. */
+    name?: string;
+    /** Agent type to run this item (e.g. "scout"). Defaults to the spawn policy's default agent. */
+    agent?: string;
     /** The work; required by the schema. */
-    assignment?: string;
+    task?: string;
     /** Run this spawn in an isolated worktree (batch form; flat form carries it top-level). */
     isolated?: boolean;
 }
 export declare const taskSchema: import("arktype/internal/variants/object.ts").ObjectType<{
+    name?: string | undefined;
     agent: import("arktype/internal/attributes.ts").Default<string, "task">;
-    id?: string | undefined;
-    description?: string | undefined;
-    role?: string | undefined;
-    assignment: string;
+    task: string;
     isolated?: boolean | undefined;
 }, {}>;
 declare const ALL_TASK_SCHEMAS: readonly [import("arktype/internal/variants/object.ts").ObjectType<{
+    name?: string | undefined;
     agent: import("arktype/internal/attributes.ts").Default<string, "task">;
-    id?: string | undefined;
-    description?: string | undefined;
-    role?: string | undefined;
-    assignment: string;
+    task: string;
     isolated?: boolean | undefined;
 }, {}>, import("arktype/internal/variants/object.ts").ObjectType<{
+    name?: string | undefined;
     agent: import("arktype/internal/attributes.ts").Default<string, "task">;
-    id?: string | undefined;
-    description?: string | undefined;
-    role?: string | undefined;
-    assignment: string;
+    task: string;
 }, {}>, import("arktype/internal/variants/object.ts").ObjectType<{
-    agent: import("arktype/internal/attributes.ts").Default<string, "task">;
     context: string;
     tasks: {
-        id?: string | undefined;
-        description?: string | undefined;
-        role?: string | undefined;
-        assignment: string;
+        name?: string | undefined;
+        agent: import("arktype/internal/attributes.ts").Default<string, "task">;
+        task: string;
         isolated?: boolean | undefined;
     }[];
 }, {}>, import("arktype/internal/variants/object.ts").ObjectType<{
-    agent: import("arktype/internal/attributes.ts").Default<string, "task">;
     context: string;
     tasks: {
-        id?: string | undefined;
-        description?: string | undefined;
-        role?: string | undefined;
-        assignment: string;
+        name?: string | undefined;
+        agent: import("arktype/internal/attributes.ts").Default<string, "task">;
+        task: string;
     }[];
 }, {}>];
 type DynamicTaskSchema = (typeof ALL_TASK_SCHEMAS)[number];
@@ -130,21 +115,17 @@ export declare function getTaskSchema(options: {
 }): TaskToolSchemaInstance;
 /**
  * Runtime params union over both wire shapes. The model sees exactly one shape
- * (`{ agent, context, tasks[] }` when `task.batch` is on, `{ agent, ...item }`
+ * (`{ context, tasks[] }` when `task.batch` is on, `{ name?, agent?, task }`
  * otherwise); runtime stays permissive so internal callers and stale
  * transcripts using the flat form keep working under either setting.
  */
 export interface TaskParams {
-    /** Agent type to spawn; omitted values resolve from the session spawn policy. */
+    /** Stable agent name (flat form). */
+    name?: string;
+    /** Agent type to spawn (flat form); omitted values resolve from the session spawn policy. */
     agent?: string;
-    /** Stable agent id (flat form); default = generated AdjectiveNoun. */
-    id?: string;
-    /** UI label (flat form), not seen by the subagent. */
-    description?: string;
-    /** Specialist role/expertise this subagent embodies; shapes its system-prompt identity and display name. */
-    role?: string;
     /** The work (flat form). */
-    assignment?: string;
+    task?: string;
     /** Batch form (`task.batch`): one subagent per item. */
     tasks?: TaskItem[];
     /** Batch form: shared background prepended to every assignment; required by the batch schema. */
@@ -157,17 +138,11 @@ export interface TaskParams {
  * `displayName`, or a system-prompt field. Collapses every run of whitespace
  * AND control/format characters — including U+0085 NEL, ESC/ANSI, and the
  * zero-width separators that `\s` misses — to a single space, then caps length.
- * So untrusted text (a spawn `role`, a peer activity gist) can neither break the
- * line, inject prompt structure, nor smuggle terminal escapes. Caps at `max`
- * characters (clamped to >= 1; default `ROLE_LABEL_MAX`), appending an ellipsis when truncated.
+ * So untrusted text (a generated task label, a peer activity gist) can neither
+ * break the line, inject prompt structure, nor smuggle terminal escapes. Caps at
+ * `max` characters (clamped to >= 1; default `LABEL_MAX`), appending an ellipsis when truncated.
  */
 export declare function oneLineLabel(text: string, max?: number): string;
-/**
- * Display name for a spawned subagent: its tailored `role` (label-normalized)
- * when one is given, else the agent type's name. Empty/whitespace roles fall
- * back to the agent name.
- */
-export declare function resolveSubagentDisplayName(role: string | undefined, agentName: string): string;
 /**
  * Whether an agent at `taskDepth` may still spawn children — i.e. it currently
  * holds the `task` tool. Mirrors the task-tool availability gate;
@@ -203,7 +178,7 @@ export interface AgentDefinition {
     tools?: string[];
     spawns?: string[] | "*";
     model?: string[];
-    thinkingLevel?: ThinkingLevel;
+    thinkingLevel?: ConfiguredThinkingLevel;
     output?: unknown;
     blocking?: boolean;
     autoloadSkills?: string[];

package/dist/types/tools/ask.d.ts

@@ -24,9 +24,11 @@ declare const askSchema: import("arktype/internal/variants/object.ts").ObjectTyp
     questions: {
         id: string;
         question: string;
+        header?: string | undefined;
         options: {
             label: string;
             description?: string | undefined;
+            preview?: string | undefined;
         }[];
         multi?: boolean | undefined;
         recommended?: number | undefined;
@@ -41,6 +43,8 @@ export interface QuestionResult {
     multi: boolean;
     selectedOptions: string[];
     customInput?: string;
+    /** Optional note attached to the selected answer in the rich ask dialog. */
+    note?: string;
     /** True when the answer was auto-selected because the dialog timed out. */
     timedOut?: boolean;
 }
@@ -50,10 +54,16 @@ export interface AskToolDetails {
     multi?: boolean;
     selectedOptions?: string[];
     customInput?: string;
+    /** Optional note attached to the selected answer in the rich ask dialog. */
+    note?: string;
     /** True when the answer was auto-selected because the dialog timed out. */
     timedOut?: boolean;
     /** Multi-part question mode */
     results?: QuestionResult[];
+    /** Chat redirect: the user chose "Chat about this" instead of answering. */
+    chatRedirect?: boolean;
+    /** Questions surfaced when chatRedirect is true. */
+    questions?: string[];
 }
 type AskParams = AskToolInput;
 /**
@@ -74,9 +84,11 @@ export declare class AskTool implements AgentTool<typeof askSchema, AskToolDetai
         questions: {
             id: string;
             question: string;
+            header?: string | undefined;
             options: {
                 label: string;
                 description?: string | undefined;
+                preview?: string | undefined;
             }[];
             multi?: boolean | undefined;
             recommended?: number | undefined;

package/dist/types/tools/conflict-detect.d.ts

@@ -117,6 +117,14 @@ export interface ParsedConflictUri {
  * `@both` shorthand) to every currently-registered conflict in one shot.
  */
 export declare function parseConflictUri(raw: string): ParsedConflictUri | null;
+/** Result of {@link spliceConflict}: the new file text plus any boundary-echo repair applied. */
+export interface ConflictSplice {
+    text: string;
+    /** Replacement lines dropped because they duplicated the context directly above the region. */
+    trimmedLeading: number;
+    /** Replacement lines dropped because they duplicated the context directly below the region. */
+    trimmedTrailing: number;
+}
 /**
  * Splice the conflict region recorded in `entry` out of `originalText`
  * and replace it with `replacement` (markers and all sides included).
@@ -126,8 +134,16 @@ export declare function parseConflictUri(raw: string): ParsedConflictUri | null;
  * match), so out-of-band edits earlier in the file that shift line
  * numbers don't break resolution. Throws clearly when the marker block
  * has actually been altered or removed.
+ *
+ * Boundary-echo repair (same philosophy as the edit tool's hashline
+ * keeper repair): models frequently paste the "whole resolved function"
+ * including the lines that live directly before/after the marker block,
+ * which the verbatim splice would duplicate. Replacement lines that
+ * exactly echo the adjacent context are dropped when the echo is
+ * unambiguous — two or more consecutive lines, or a single line whose
+ * removal fixes a delimiter-balance mismatch against the recorded sides.
  */
-export declare function spliceConflict(originalText: string, entry: ConflictEntry, replacement: string): string;
+export declare function spliceConflict(originalText: string, entry: ConflictEntry, replacement: string): ConflictSplice;
 /**
  * True when two registered blocks record the same marker-block content
  * (labels and all sides). Out-of-band edits can shift a block's line

package/dist/types/tools/job.d.ts

@@ -19,12 +19,28 @@ interface JobSnapshot {
     errorText?: string;
 }
 type CancelStatus = "cancelled" | "not_found" | "already_completed";
+/**
+ * A live subagent from the AgentRegistry that has no backing job in the
+ * AsyncJobManager — e.g. an idle agent woken (or a parked agent revived) via
+ * `irc`, or a spawn owned by another agent. Surfaced by `list` and empty-poll
+ * snapshots so the job tool's picture matches the UI's running-agent count.
+ */
+interface AgentActivitySnapshot {
+    id: string;
+    parentId?: string;
+    /** Latest activity gist recorded by the registry (display-only). */
+    activity?: string;
+    /** Time since the agent was registered. */
+    ageMs: number;
+}
 export interface JobToolDetails {
     jobs: JobSnapshot[];
     cancelled?: {
         id: string;
         status: CancelStatus;
     }[];
+    /** Running subagents not represented by a job row in this result. */
+    agents?: AgentActivitySnapshot[];
 }
 /**
  * A poll snapshot where every watched job is still running and nothing was

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "16.4.4",
+	"version": "16.4.5",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://omp.sh",
 	"author": "Can Boluk",
@@ -52,17 +52,17 @@
 		"@agentclientprotocol/sdk": "0.25.0",
 		"@babel/parser": "^7.29.7",
 		"@mozilla/readability": "^0.6.0",
-		"@oh-my-pi/hashline": "16.4.4",
-		"@oh-my-pi/omp-stats": "16.4.4",
-		"@oh-my-pi/pi-agent-core": "16.4.4",
-		"@oh-my-pi/pi-ai": "16.4.4",
-		"@oh-my-pi/pi-catalog": "16.4.4",
-		"@oh-my-pi/pi-mnemopi": "16.4.4",
-		"@oh-my-pi/pi-natives": "16.4.4",
-		"@oh-my-pi/pi-tui": "16.4.4",
-		"@oh-my-pi/pi-utils": "16.4.4",
-		"@oh-my-pi/pi-wire": "16.4.4",
-		"@oh-my-pi/snapcompact": "16.4.4",
+		"@oh-my-pi/hashline": "16.4.5",
+		"@oh-my-pi/omp-stats": "16.4.5",
+		"@oh-my-pi/pi-agent-core": "16.4.5",
+		"@oh-my-pi/pi-ai": "16.4.5",
+		"@oh-my-pi/pi-catalog": "16.4.5",
+		"@oh-my-pi/pi-mnemopi": "16.4.5",
+		"@oh-my-pi/pi-natives": "16.4.5",
+		"@oh-my-pi/pi-tui": "16.4.5",
+		"@oh-my-pi/pi-utils": "16.4.5",
+		"@oh-my-pi/pi-wire": "16.4.5",
+		"@oh-my-pi/snapcompact": "16.4.5",
 		"@opentelemetry/api": "^1.9.1",
 		"@opentelemetry/context-async-hooks": "^2.7.1",
 		"@opentelemetry/exporter-trace-otlp-proto": "^0.218.0",

package/src/async/job-manager.ts

@@ -44,6 +44,12 @@ export interface AsyncJob {
 	 * supply an id (e.g. legacy tests, SDK consumers without an agent context).
 	 */
 	ownerId?: string;
+	/**
+	 * Registry id of the subagent this job runs (task/tan/vibe jobs). Lets
+	 * job-view code link a job row to its AgentRegistry ref even when the job
+	 * id differs from the agent id (vibe turn jobs, tan clones).
+	 */
+	agentId?: string;
 	/**
 	 * Job is registered but parked behind a caller-managed gate (e.g. a task
 	 * batch semaphore). Queued jobs do not count toward the running-job limit
@@ -79,6 +85,8 @@ export interface AsyncJobRegisterOptions {
 	id?: string;
 	/** Registry id of the agent that owns this job; used to scope cancelAll. */
 	ownerId?: string;
+	/** Registry id of the subagent this job runs; see {@link AsyncJob.agentId}. */
+	agentId?: string;
 	onProgress?: (text: string, details?: Record<string, unknown>) => void | Promise<void>;
 	/** Register the job in queued state; see {@link AsyncJob.queued}. */
 	queued?: boolean;
@@ -192,6 +200,7 @@ export class AsyncJobManager {
 			abortController,
 			promise: Promise.resolve(),
 			ownerId: options?.ownerId,
+			agentId: options?.agentId,
 			queued: options?.queued === true,
 		};
 

package/src/commit/agentic/tools/analyze-file.ts

@@ -83,10 +83,9 @@ export function createAnalyzeFileTool(options: {
 						related_files: relatedFiles,
 					});
 					const taskParams: TaskParams = {
+						name: `AnalyzeFile${index + 1}`,
 						agent: "sonic",
-						id: `AnalyzeFile${index + 1}`,
-						description: `Analyze ${file}`,
-						assignment,
+						task: assignment,
 					};
 					return taskTool.execute(`${toolCallId}-${index + 1}`, taskParams, signal);
 				}),

package/src/config/settings-schema.ts

@@ -3633,6 +3633,17 @@ export const SETTINGS_SCHEMA = {
 		},
 	},
 
+	"ask.enabled": {
+		type: "boolean",
+		default: true,
+		ui: {
+			tab: "tools",
+			group: "Available Tools",
+			label: "Ask",
+			description: "Enable the ask tool for interactive user questions",
+		},
+	},
+
 	"browser.enabled": {
 		type: "boolean",
 		default: true,
@@ -4142,31 +4153,31 @@ export const SETTINGS_SCHEMA = {
 
 	"task.softRequestBudget": {
 		type: "number",
-		default: 90,
+		default: 200,
 		ui: {
 			tab: "tasks",
 			group: "Subagents",
 			label: "Soft Subagent Request Budget",
 			description:
-				"Soft per-subagent request budget (assistant requests per run). Crossing it can inject a steering notice when task.softRequestBudgetNotice is enabled; at 1.5x the budget the run is aborted gracefully, salvaging partial output. 0 disables the guard. Bundled scout/sonic agents use a lower built-in budget.",
+				"Soft per-subagent request budget (assistant requests per run). Crossing it injects a wrap-up steering notice (see task.softRequestBudgetNotice); at 1.5x the budget the run is force-stopped and the agent must yield its partial findings. 0 disables the guard. Bundled scout/sonic agents use a lower built-in budget.",
 			options: [
 				{ value: "0", label: "Disabled" },
-				{ value: "40", label: "40 requests" },
-				{ value: "90", label: "90 requests", description: "Default" },
+				{ value: "90", label: "90 requests" },
 				{ value: "150", label: "150 requests" },
+				{ value: "200", label: "200 requests", description: "Default" },
 			],
 		},
 	},
 
 	"task.softRequestBudgetNotice": {
 		type: "boolean",
-		default: false,
+		default: true,
 		ui: {
 			tab: "tasks",
 			group: "Subagents",
 			label: "Soft Request Budget Notice",
 			description:
-				"Inject one steering notice when a subagent crosses its soft request budget. Off by default; enabling it asks the child to wrap up before the 1.5x graceful abort guard.",
+				"Inject one steering notice when a subagent crosses its soft request budget, asking it to wrap up before the 1.5x forced-yield stop.",
 		},
 	},
 

package/src/config/settings.ts

@@ -604,9 +604,10 @@ export class Settings {
 	}
 
 	/**
-	 * Set a model role (helper for modelRoles record).
+	 * Set a model role (helper for modelRoles record). Passing `undefined`
+	 * clears the role from the persisted record and any runtime override.
 	 */
-	setModelRole(role: ModelRole | string, modelId: string): void {
+	setModelRole(role: ModelRole | string, modelId: string | undefined): void {
 		const current = this.#modelRolesFromLayer(this.#global);
 		const runtimeOverrides = getByPath(this.#overrides, ["modelRoles"]);
 		const updateRuntimeOverride =
@@ -615,12 +616,20 @@ export class Settings {
 			!Array.isArray(runtimeOverrides) &&
 			Object.hasOwn(runtimeOverrides, role);
 
-		current[role] = modelId;
+		if (modelId === undefined) {
+			delete current[role];
+		} else {
+			current[role] = modelId;
+		}
 		this.set("modelRoles", current);
 
 		if (updateRuntimeOverride) {
 			const nextRuntimeOverride = this.#modelRolesFromLayer(this.#overrides);
-			nextRuntimeOverride[role] = modelId;
+			if (modelId === undefined) {
+				delete nextRuntimeOverride[role];
+			} else {
+				nextRuntimeOverride[role] = modelId;
+			}
 			this.override("modelRoles", nextRuntimeOverride);
 		}
 	}

package/src/discovery/helpers.ts

@@ -1,7 +1,6 @@
 import * as fs from "node:fs";
 import * as os from "node:os";
 import * as path from "node:path";
-import type { ThinkingLevel } from "@oh-my-pi/pi-agent-core";
 import { FileType, glob } from "@oh-my-pi/pi-natives";
 import {
 	CONFIG_DIR_NAME,
@@ -17,7 +16,7 @@ import { invalidate as invalidateFsCache, readDirEntries, readFile } from "../ca
 import { parseRuleConditionAndScope, type Rule, type RuleFrontmatter } from "../capability/rule";
 import type { Skill, SkillFrontmatter } from "../capability/skill";
 import type { LoadContext, LoadResult, SourceMeta } from "../capability/types";
-import { parseThinkingLevel } from "../thinking";
+import { type ConfiguredThinkingLevel, parseConfiguredThinkingLevel } from "../thinking";
 import { normalizeToolNames } from "../tools/builtin-names";
 
 import { buildPluginDirRoot } from "./plugin-dir-roots";
@@ -229,7 +228,7 @@ export interface ParsedAgentFields {
 	spawns?: string[] | "*";
 	model?: string[];
 	output?: unknown;
-	thinkingLevel?: ThinkingLevel;
+	thinkingLevel?: ConfiguredThinkingLevel;
 	autoloadSkills?: string[];
 	readSummarize?: boolean;
 	blocking?: boolean;
@@ -283,7 +282,7 @@ export function parseAgentFields(frontmatter: Record<string, unknown>): ParsedAg
 				? frontmatter.thinking
 				: undefined;
 
-	const thinkingLevel = parseThinkingLevel(rawThinkingLevel);
+	const thinkingLevel = parseConfiguredThinkingLevel(rawThinkingLevel);
 	const model = parseModelList(frontmatter.model);
 	const blocking = parseBoolean(frontmatter.blocking);
 	const readSummarize = parseBoolean(frontmatter.readSummarize);