@arhen/pi-core-subagent 1.3.56 → 1.3.57

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@arhen/pi-core-subagent",
-	"version": "1.3.56",
+	"version": "1.3.57",
 	"type": "module",
 	"description": "pi extension: fast in-process subagents with a dependency-graph scheduler (needs edges gate tasks and carry upstream output into dependent prompts), plus background runs, intercom and agent-to-agent mailbox. Leader defines agents inline.",
 	"license": "MIT",

package/src/manager.ts

@@ -157,16 +157,38 @@ export function resolveChildModel(ctx: ExtensionContext, explicit: string | unde
 	throw new Error(`Model not found: ${ref}`);
 }
 
+const PROBE_THINKING_LEVELS: ThinkingLevel[] = ["low", "minimal", "medium", "high", "xhigh", "max"];
+
+/**
+ * Thinking level for the usability probe: the one the task asked for, else the cheapest
+ * the model accepts. Providers that require adaptive thinking reject a probe without one
+ * (9router claude models answer `thinking.type.disabled is not supported`), which used to
+ * make every preflight fail and fall back to the session model.
+ */
+function probeThinking(model: Model<Api>, thinking?: string): ThinkingLevel | undefined {
+	if (thinking && thinking !== "off") return thinking as ThinkingLevel;
+	if (!model.reasoning) return undefined;
+	const map = model.thinkingLevelMap;
+	if (!map) return "low";
+	for (const level of PROBE_THINKING_LEVELS) {
+		const mapped = map[level as keyof typeof map];
+		if (mapped !== null && mapped !== undefined) return level;
+	}
+	return undefined;
+}
+
 async function probeModel(
 	ctx: ExtensionContext,
 	model: Model<Api>,
 	signal: AbortSignal | undefined,
+	thinking?: string,
 ): Promise<string | undefined> {
 	try {
+		const reasoningEffort = probeThinking(model, thinking);
 		const reply = await ctx.modelRegistry.complete(
 			model,
 			{ messages: [{ role: "user", content: "ping", timestamp: Date.now() }] },
-			{ maxTokens: 16, signal },
+			{ maxTokens: 16, signal, ...(reasoningEffort ? { reasoningEffort } : {}) },
 		);
 		return reply.stopReason === "error" ? (reply.errorMessage ?? "provider returned an error") : undefined;
 	} catch (err) {
@@ -178,11 +200,12 @@ export async function ensureUsableModel(
 	ctx: ExtensionContext,
 	model: Model<Api> | undefined,
 	signal: AbortSignal | undefined,
+	thinking?: string,
 ): Promise<{ model: Model<Api> | undefined; note?: string }> {
 	const session = ctx.model;
 	if (!model || !ctx.modelRegistry) return { model };
 	if (session && model.provider === session.provider && model.id === session.id) return { model };
-	const error = await probeModel(ctx, model, signal);
+	const error = await probeModel(ctx, model, signal, thinking);
 	if (!error) return { model };
 	if (model.provider === "opencode-go" && /MissingSessionID|x-opencode-session/i.test(error)) {
 		// ponytail: opencode-go rejects stateless probes but accepts AgentSession requests, which add the session header.
@@ -747,7 +770,7 @@ export class SubagentManager {
 			model = resolveChildModel(ctx, file?.model ?? input.model);
 			validateThinking(model, thinking);
 
-			const checked = await ensureUsableModel(ctx, model, signal);
+			const checked = await ensureUsableModel(ctx, model, signal, thinking);
 			model = checked.model;
 			if (checked.note) {
 				task.modelNote = checked.note;