@henryqw/pi-subagent 2.3.5 → 2.5.0

package/CONTEXT.md

@@ -9,7 +9,7 @@ Provide validated user Roles, shared task-model Pi launch policy, generic manage
 - **Main**: Pi session delegating work.
 - **Subagent**: isolated Pi child process handling one task.
 - **Role**: user-owned Markdown profile defining name, description, system instructions, optional exact tool allowlist, extensions, and Skill names.
-- **Model Class**: `fast`, `balanced`, or `frontier`, assigned in shared task-model settings or overridden by Main from task complexity.
+- **Model Class**: `fast`, `balanced`, `frontier`, or `fav`, assigned in shared task-model settings or overridden by Main from task complexity.
 - **Route**: configured model and thinking-level pair selected from a shared Model Class profile; the primary route precedes its optional fallback.
 - **Delegated Task**: one bounded work request sent from Main to one Role.
 - **Pi Launch**: reusable `{env,args}` policy for one Role, resolved model route, explicit caller resources, and project trust.

package/README.md

@@ -14,16 +14,18 @@ pi install npm:@henryqw/pi-subagent
 
 | Package | Why |
 | --- | --- |
-| `@henryqw/pi-task-models` | Required. Shared `fast` / `balanced` / `frontier` routes. |
+| `@henryqw/pi-task-models` | Required. Shared `fast` / `balanced` / `frontier` / `fav` routes. |
 | `@henryqw/pi-multi-codex` | Required. Child uses Main's active Codex slot. |
 
 ## Use
 
 | Surface | Type | Purpose |
 | --- | --- | --- |
-| `delegate_task` | tool | Start one isolated child for `role`, `task`, and optional `modelClass`. |
+| `delegate_task` | tool | Start one isolated child for `role`, `task`, and optional `model` or `modelClass`. |
 
-`modelClass` is `fast`, `balanced`, or `frontier`. Omitted class uses the shared `pi-subagent/delegateTask` assignment, which defaults to `balanced`. Primary route is resolved against current scoped text models; fallback is tried only before launch. If no route is usable, delegation rejects with `Run /task-models`. A started child is never retried.
+An explicit `model` (`provider/modelId`) overrides `modelClass` and resolves against the currently available text models; an unknown reference rejects with the list of available models. Thinking level defaults to `medium` when supported, otherwise the highest supported level.
+
+`modelClass` is `fast`, `balanced`, `frontier`, or `fav`. Omitted class uses the shared `pi-subagent/delegateTask` assignment, which defaults to `balanced`. Primary route is resolved against current scoped text models; fallback is tried only before launch. If no route is usable, delegation rejects with `Run /task-models`. A started child is never retried.
 
 Main splits broad work into independent bounded tasks and keeps integration and cross-cutting decisions. Each `task` states its objective, exact scope and exclusions, relevant context and constraints, expected deliverable, and validation. Each call uses the least capable `modelClass` that can reliably complete its task. Independent sibling calls can run concurrently; concurrent edit tasks must own non-overlapping files. Up to four active ephemeral `delegate_task` children run per Main; excess calls wait FIFO. Queued calls do not start a child or consume child timeout. Managed Herdr workers are unaffected.
 

package/extensions/subagent.ts

@@ -4,7 +4,15 @@ import { basename } from "node:path";
 import { StringEnum } from "@earendil-works/pi-ai";
 import { type ExtensionAPI, type ExtensionContext, type Theme } from "@earendil-works/pi-coding-agent";
 import { type Component, truncateToWidth, type TUI, visibleWidth } from "@earendil-works/pi-tui";
-import { modelReference, PROFILE_NAMES, type ProfileName } from "@henryqw/pi-task-models";
+import {
+	availableTaskModels,
+	modelReference,
+	PROFILE_NAMES,
+	type ProfileName,
+	resolveAvailableModel,
+	type ResolvedTaskRoute,
+	taskThinkingLevels,
+} from "@henryqw/pi-task-models";
 import { Type } from "typebox";
 import { createRoleLaunch, isProfileName, loadRoles, resolveRoleLaunch, resolveTaskRoute } from "@henryqw/pi-subagent";
 
@@ -415,11 +423,24 @@ const Parameters = Type.Object({
 	task: Type.String({
 		description: "Bounded task packet: objective; exact scope and exclusions; relevant context and constraints; expected deliverable; validation. Never the whole parent request.",
 	}),
+	model: Type.Optional(Type.String({
+		description: "Designated model as provider/modelId; overrides modelClass. Unknown references reject with the list of available models.",
+	})),
 	modelClass: Type.Optional(StringEnum(MODEL_CLASSES, {
-		description: "Classify task complexity: fast for narrow lookups or mechanical edits; balanced for normal bounded work; frontier for ambiguous, cross-cutting, or high-risk reasoning. Defaults to the shared pi-subagent/delegateTask assignment.",
+		description: "Classify task complexity: fast for narrow lookups or mechanical edits; balanced for normal bounded work; frontier for ambiguous, cross-cutting, or high-risk reasoning; fav for the user's favorite model when they ask for it. Defaults to the shared pi-subagent/delegateTask assignment.",
 	})),
 });
 
+function resolveDesignatedRoute(ctx: ExtensionContext, reference: string): ResolvedTaskRoute {
+	const models = availableTaskModels(ctx);
+	const model = resolveAvailableModel(models, reference, ctx.model?.provider);
+	if (!model) {
+		throw new Error(`Unknown delegate_task model: ${reference}. Available models: ${models.map((candidate) => modelReference(candidate)).join(", ") || "none"}.`);
+	}
+	const levels = taskThinkingLevels(ctx, model);
+	return { model, thinkingLevel: levels.includes("medium") ? "medium" : levels.at(-1)! };
+}
+
 const roleSummary = (): string => {
 	try {
 		const roles = loadRoles();
@@ -553,12 +574,12 @@ export default function subagentExtension(
 	pi.registerTool({
 		name: "delegate_task",
 		label: "Subagent",
-		description: `Delegate one bounded, independently executable task to one isolated Pi Subagent. Roles: ${roleSummary()}. Choose fast for narrow work, balanced for normal work, or frontier for ambiguous and high-risk work; omit modelClass to use shared task-model settings.`,
+		description: `Delegate one bounded, independently executable task to one isolated Pi Subagent. Roles: ${roleSummary()}. Choose fast for narrow work, balanced for normal work, frontier for ambiguous and high-risk work, or fav when the user asks for their favorite model; omit modelClass to use shared task-model settings. When the user designates a specific model, pass it as provider/modelId in model.`,
 		promptSnippet: "Delegate one bounded, independently executable task to an isolated role",
 		promptGuidelines: [
 			"Before calling delegate_task, split broad work into the smallest independent bounded tasks; keep integration and cross-cutting decisions in Main.",
 			"Each delegate_task task must state its objective, exact scope and exclusions, relevant context and constraints, expected deliverable, and validation; never pass the parent request unchanged.",
-			"For each delegate_task call, choose the least capable modelClass that can reliably complete the task: fast for narrow work, balanced for normal work, and frontier only for ambiguous, cross-cutting, or high-risk work.",
+			"Use fav when the user explicitly asks for their favorite model. Otherwise, choose the least capable modelClass that can reliably complete the task: fast for narrow work, balanced for normal work, and frontier only for ambiguous, cross-cutting, or high-risk work.",
 			"Submit independent delegate_task calls together for parallel execution. Parallel edits must own non-overlapping files; otherwise sequence them. Use the minimum number of Subagents needed.",
 		],
 		parameters: Parameters,
@@ -571,11 +592,13 @@ export default function subagentExtension(
 			}
 
 			if (params.modelClass !== undefined && !isModelClass(params.modelClass)) {
-				throw new Error("delegate_task modelClass must be fast, balanced, or frontier.");
+				throw new Error("delegate_task modelClass must be fast, balanced, frontier, or fav.");
 			}
-			const launch = params.modelClass === undefined
-				? resolveRoleLaunch(pi, ctx, { role, taskId: SUBAGENT_TASK })
-				: createRoleLaunch(pi, ctx, { role, route: resolveTaskRoute(ctx, params.modelClass) });
+			const launch = params.model !== undefined
+				? createRoleLaunch(pi, ctx, { role, route: resolveDesignatedRoute(ctx, cleanText(params.model, "model", "delegate_task")) })
+				: params.modelClass === undefined
+					? resolveRoleLaunch(pi, ctx, { role, taskId: SUBAGENT_TASK })
+					: createRoleLaunch(pi, ctx, { role, route: resolveTaskRoute(ctx, params.modelClass) });
 			const modelReferenceValue = modelReference(launch.model);
 			const thinkingLevel = launch.thinkingLevel;
 			if (launch.missingSkills.length) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@henryqw/pi-subagent",
-  "version": "2.3.5",
+  "version": "2.5.0",
   "description": "Delegate one task to an isolated Pi role with explicit extensions and skills.",
   "keywords": [
     "pi-package",
@@ -59,6 +59,6 @@
   "dependencies": {
     "@henryqw/pi-herdr": "^0.1.1",
     "@henryqw/pi-multi-codex": "^0.3.8",
-    "@henryqw/pi-task-models": "^0.3.0"
+    "@henryqw/pi-task-models": "^0.4.0"
   }
 }