dsh-pstack 0.3.0-beta.1 → 0.3.1

package/dist/subagent.mjs

@@ -1,352 +1,138 @@
-import { a as POTETO_MODE_SKILL_PATH } from "./paths-yji9SNak.mjs";
-import { a as selectRoleRoutes, i as resolveDshHome, n as readRoleConfiguration, r as requireRole, t as modelsFilePath } from "./role-config-a4JpVdc-.mjs";
+import { POTETO_MODE_SKILL_PATH } from "./paths.mjs";
+import { modelsFilePath, readRoleConfiguration, requireRole, resolveDshHome, selectRoutes } from "./role-config.mjs";
+import { isRoute } from "./route.mjs";
 import { isAbsolute, join, relative, resolve } from "node:path";
 import { existsSync, statSync } from "node:fs";
-// src/internal/effort.ts
-/** Model-facing name of the child's own control. */
-const SET_EFFORT_TOOL = "set_reasoning_effort";
-/**
-* Resolve the route a child's effort will actually apply to: an explicit route wins,
-* then the session's live request header, then the creation-time options.
-*/
-function resolveEffortTarget(agent, route) {
-	const explicit = route?.kind === "route" ? route : void 0;
-	const header = agent?.session.requestHeader()?.config;
-	const provider = explicit?.provider ?? header?.provider ?? agent?.options.provider;
-	const model = explicit?.model ?? header?.model ?? agent?.options.model;
-	return provider !== void 0 && model !== void 0 ? {
-		provider,
-		model
-	} : void 0;
-}
-async function advertisedEfforts(llm, target, signal) {
-	return (await llm.resolveModelInfo(target.provider, target.model, signal)).reasoning?.efforts.map((entry) => entry.id) ?? [];
-}
-/** Validate one effort against the route a child will actually request. */
-async function validateEffortFor(llm, route, effort, parent, signal) {
-	const target = resolveEffortTarget(parent, route);
-	if (target === void 0) throw new Error("cannot resolve the child provider/model route for reasoningEffort. Pass `role` with an exact route, or run the delegation from a session that has already made a request.");
-	let efforts;
-	try {
-		efforts = await advertisedEfforts(llm, target, signal);
-	} catch (error) {
-		throw new Error(`route ${target.provider}/${target.model} could not be resolved for reasoningEffort: ${describeError$1(error)}`, { cause: error });
-	}
-	if (efforts.length === 0) throw new Error(`${target.provider}/${target.model} advertises no reasoning efforts, so it cannot take reasoningEffort "${effort}". Omit the argument and let the adapter use its own default.`);
-	if (!efforts.includes(effort)) throw new Error(`${target.provider}/${target.model} does not support reasoning effort "${effort}". It supports: ${efforts.join(", ")}.`);
-}
-/**
-* Install the effort control on one live child.
-*
-* Registration is child-scoped, so the tool disposes with the child. A successful
-* call records the choice through `agent.model.select()`, which DSH applies when
-* it prepares the next step. `agent/request` is deliberately not used: DSH
-* deprecated it for model and effort changes and rejects a request that disagrees
-* with the selection fixed during step preparation.
-*/
-function installEffortControl(child, llm) {
-	return child.ctx.tools.register(defineSetEffortTool(child, llm));
-}
-function defineSetEffortTool(child, llm) {
-	return {
-		name: SET_EFFORT_TOOL,
-		description: [
-			"Change your reasoning effort for future model requests.",
-			"Use it when the task turns out harder or more routine than the role line assumed: raise it for subtle design, concurrency, or algorithm work, lower it once the remaining work is mechanical.",
-			"The change applies from your next step; a request already in flight is unaffected, and it lasts for the rest of this child's life."
-		].join(" "),
-		parameters: {
-			type: "object",
-			additionalProperties: false,
-			properties: { effort: {
-				type: "string",
-				description: "The reasoning effort to use from your next step onward. Use one of the values this model advertises; the call rejects an unknown one and lists the real values."
-			} },
-			required: ["effort"]
-		},
-		output: {
-			schema: { type: "string" },
-			render: (_args, value) => [{
-				type: "text",
-				text: typeof value === "string" ? value : JSON.stringify(value)
-			}]
-		},
-		isConcurrencySafe: () => false,
-		async execute(rawArgs, exec) {
-			const args = rawArgs;
-			const value = typeof args.effort === "string" ? args.effort.trim() : "";
-			if (value === "") throw new Error("set_reasoning_effort needs a non-empty `effort`. Pass the exact token this model advertises; an empty value would silently re-enable the adapter default.");
-			const agent = exec.agent ?? child;
-			const target = resolveEffortTarget(agent);
-			if (target === void 0) throw new Error("this child has no resolved provider/model route, so reasoning effort cannot be validated against the live adapter metadata");
-			let efforts;
-			try {
-				efforts = await advertisedEfforts(llm, target, exec.signal);
-			} catch (error) {
-				throw new Error(`${target.provider}/${target.model} could not be resolved for reasoning effort: ${describeError$1(error)}`, { cause: error });
-			}
-			if (efforts.length === 0) throw new Error(`${target.provider}/${target.model} advertises no reasoning efforts, so this child has none to change. Your current effort stays as it is.`);
-			if (!efforts.includes(value)) throw new Error(`${target.provider}/${target.model} does not support reasoning effort "${value}", so nothing changed. It supports: ${efforts.join(", ")}.`);
-			agent.model.select({
-				provider: target.provider,
-				model: target.model,
-				reasoningEffort: value
-			});
-			return `reasoning effort set to "${value}". It applies from your next step; the request already in flight is unaffected.`;
-		}
-	};
-}
-/** The `agentOptions` one (route, effort override) pair contributes. */
-function agentOptionsFor(route, effortOverride) {
-	const effort = effortOverride ?? (route.kind === "route" ? route.effort : void 0);
-	if (route.kind === "inherit") return effort === void 0 ? void 0 : { reasoningEffort: effort };
-	return {
-		provider: route.provider,
-		model: route.model,
-		...effort === void 0 ? {} : { reasoningEffort: effort }
-	};
-}
-/** The effort a plan entry will actually run at, override included. */
-function effectiveEffort(route, effortOverride) {
-	return effortOverride ?? (route.kind === "route" ? route.effort : void 0);
-}
-function describeError$1(error) {
-	if (error instanceof Error) {
-		const code = error.code;
-		return typeof code === "string" ? `${error.message} (${code})` : error.message;
-	}
-	return String(error);
-}
 // src/subagent.ts
 /**
 * `pstack_subagent`: pstack's delegation entry point.
 *
-* One call plans one or more fresh children, validates every route and effort before the
-* first child starts, then starts them through `ctx.subagents.startActivation` with
-* parent delivery. Results arrive as completion notices; this module never polls.
+* One call resolves the role file, validates every route and effort before the
+* first child starts, then starts one fresh child per selected route through
+* `ctx.subagents.startActivation`. Results arrive as completion notices; this
+* tool never polls.
 */
-/** Cordis row name, used by Loader diagnostics. */
 const name = "pstack-subagent";
-/**
-* Delegation needs the tool registry, the subagent registry, the LLM adapters, and the
-* session directory. `agents` is not here: it is optional and read per call, so a
-* profile without it still delegates and the child simply keeps its starting effort.
-*/
 const inject = [
 	"tools",
 	"subagents",
 	"llm",
 	"workingDirectory"
 ];
-/** Model-facing tool name. */
 const TOOL_NAME = "pstack_subagent";
-/** Backend used when the row does not name one. */
-const DEFAULT_SUBAGENT_PROVIDER = "spawn";
-/** The child kinds this tool knows how to brief. */
+/** Default backend when the row does not name one. */
+const DEFAULT_PROVIDER = "spawn";
+/** Child kinds this tool can brief. */
 const AGENT_KINDS = [
 	"poteto-agent",
 	"comment-sicko",
+	"readonly",
 	"general"
 ];
-const DEFAULT_CONFIG = { provider: DEFAULT_SUBAGENT_PROVIDER };
-/**
-* Depth a child runs at when a call omits `maxDepth` or passes an unusable value.
-*
-* A child is always at least depth 1 (it is one level below its parent), so this is the
-* smallest value that can start a child while forbidding it from delegating further.
-*/
-const DEFAULT_CHILD_MAX_DEPTH = 1;
-/** Raised for a configuration or argument problem, before any child starts. */
-var DelegationError = class extends Error {
-	constructor(message) {
-		super(message);
-		this.name = "DelegationError";
-	}
-};
-/** Raised when a child starts but a later entry in the same call fails. */
-var PartialDelegationError = class extends DelegationError {
-	startedChildren;
-	constructor(message, startedChildren) {
-		super(message);
-		this.name = "PartialDelegationError";
-		this.startedChildren = startedChildren;
-	}
+/** Preset id for each independently composed child kind. */
+const AGENT_PRESETS = {
+	"poteto-agent": "poteto-agent",
+	"comment-sicko": "comment-sicko",
+	readonly: "readonly"
 };
-function isNonEmptyString(value) {
-	return typeof value === "string" && value.trim() !== "";
-}
-function isNonNegativeSafeInteger(value) {
-	return typeof value === "number" && Number.isSafeInteger(value) && value >= 0;
-}
-/**
-* Validate the row configuration without a Schemastery dependency.
-*
-* A typo must fail at activation rather than at the first delegation, and an omitted
-* `config` must be valid. Cordis passes the raw mapping because this row exports no
-* `Config` object; callers that object as `config` are still checked here.
-*/
+const DEFAULT_CONFIG = { provider: DEFAULT_PROVIDER };
+/** Validate the row's own config; omitted config uses the default provider. */
 function normalizeConfig(value) {
 	if (value === void 0 || value === null) return DEFAULT_CONFIG;
 	if (typeof value !== "object" || Array.isArray(value)) throw new Error("pstack-subagent: config must be a mapping");
 	const record = value;
 	for (const key of Object.keys(record)) if (key !== "provider" && key !== "maxDepth") throw new Error(`pstack-subagent: unknown config field "${key}"`);
 	const provider = record["provider"];
-	if (provider !== void 0 && !isNonEmptyString(provider)) throw new Error("pstack-subagent: provider must be a non-empty string");
+	if (provider !== void 0 && (typeof provider !== "string" || provider.trim() === "")) throw new Error("pstack-subagent: provider must be a non-empty string");
 	const maxDepth = record["maxDepth"];
-	if (maxDepth !== void 0 && !isNonNegativeSafeInteger(maxDepth)) throw new Error("pstack-subagent: maxDepth must be a non-negative safe integer");
+	if (maxDepth !== void 0 && !isDepth(maxDepth)) throw new Error("pstack-subagent: maxDepth must be a positive safe integer");
 	return {
-		provider: isNonEmptyString(provider) ? provider : DEFAULT_SUBAGENT_PROVIDER,
-		...isNonNegativeSafeInteger(maxDepth) ? { maxDepth } : {}
+		provider: typeof provider === "string" ? provider : DEFAULT_PROVIDER,
+		...maxDepth === void 0 ? {} : { maxDepth }
 	};
 }
-/** The preset id behind each pstack agent kind. `general` joins the parent preset. */
-const AGENT_PRESETS = {
-	"poteto-agent": "poteto-agent",
-	"comment-sicko": "comment-sicko"
-};
-/** The preset a child of this kind must mount, or undefined to join the parent preset. */
-function agentPresetFor(agent) {
-	return agent === "general" ? void 0 : AGENT_PRESETS[agent];
-}
-/** Compose the self-contained prompt one child receives. */
-function composeChildPrompt(input) {
-	const sections = [];
-	const injected = input.injectedPaths ?? [];
-	if (injected.length > 0) sections.push([
-		"Absolute paths injected by `pstack_subagent`. Read them directly; do not search a plugin directory.",
-		"",
-		...injected.map((entry) => `- ${entry.label}: \`${entry.path}\``)
-	].join("\n"));
-	if (sections.length === 0) return input.task;
-	sections.push(`Task:\n\n${input.task}`);
-	return sections.join("\n\n---\n\n");
-}
-const DELEGATION_ARGUMENT_KEYS = [
-	"description",
-	"prompt",
-	"agent",
-	"role",
-	"roleIndex",
-	"reasoningEffort",
-	"cwd",
-	"maxDepth"
-];
-/**
-* Keys a call must pass. `maxDepth` is accepted but not required: a missing or unusable
-* value resolves to `DEFAULT_CHILD_MAX_DEPTH` instead of forcing the caller to guess an
-* integer, which is where the reflexive `0` came from.
-*/
-const REQUIRED_DELEGATION_ARGUMENT_KEYS = [
-	"description",
-	"prompt",
-	"agent",
-	"role",
-	"roleIndex",
-	"reasoningEffort",
-	"cwd"
-];
-/** Validate every required model-facing argument. Null declines an optional setting. */
+/** Validate every model-facing argument before doing any Host call. */
 function parseDelegationArgs(raw) {
-	if (typeof raw !== "object" || raw === null || Array.isArray(raw)) throw new DelegationError("arguments must be an object");
+	if (typeof raw !== "object" || raw === null || Array.isArray(raw)) throw new Error("pstack_subagent arguments must be an object");
 	const record = raw;
-	for (const key of REQUIRED_DELEGATION_ARGUMENT_KEYS) if (!Object.hasOwn(record, key)) throw new DelegationError(`missing required argument "${key}"; every argument except maxDepth must be passed explicitly`);
-	for (const key of Object.keys(record)) if (!DELEGATION_ARGUMENT_KEYS.includes(key)) throw new DelegationError(`unknown argument "${key}"; accepted arguments are ${DELEGATION_ARGUMENT_KEYS.join(", ")}`);
+	const known = /* @__PURE__ */ new Set([
+		"description",
+		"prompt",
+		"agent",
+		"role",
+		"roleIndex",
+		"reasoningEffort",
+		"cwd",
+		"maxDepth"
+	]);
+	for (const key of Object.keys(record)) if (!known.has(key)) throw new Error(`unknown pstack_subagent argument "${key}"`);
+	for (const key of [
+		"description",
+		"prompt",
+		"agent",
+		"role",
+		"roleIndex",
+		"reasoningEffort",
+		"cwd"
+	]) if (!Object.hasOwn(record, key)) throw new Error(`missing required argument "${key}"`);
 	const description = record["description"];
-	if (!isNonEmptyString(description)) throw new DelegationError("description must be a non-empty string: it becomes the child label");
 	const prompt = record["prompt"];
-	if (!isNonEmptyString(prompt)) throw new DelegationError("prompt must be a non-empty string");
 	const agent = record["agent"];
-	if (typeof agent !== "string" || !AGENT_KINDS.includes(agent)) throw new DelegationError(`agent must be one of ${AGENT_KINDS.join(", ")}; choose explicitly, and never pass a DSH agent preset id.`);
 	const role = record["role"];
-	if (role !== null && !isNonEmptyString(role)) throw new DelegationError("role must be a non-empty string or null");
-	const cwd = record["cwd"];
-	if (cwd !== null && !isNonEmptyString(cwd)) throw new DelegationError("cwd must be a non-empty string or null");
-	const rawMaxDepth = record["maxDepth"];
-	const maxDepth = rawMaxDepth === void 0 ? null : rawMaxDepth;
-	if (maxDepth !== null && !isNonNegativeSafeInteger(maxDepth)) throw new DelegationError("maxDepth must be a non-negative safe integer, or null/omitted");
 	const roleIndex = record["roleIndex"];
-	if (roleIndex !== null && !isNonNegativeSafeInteger(roleIndex)) throw new DelegationError("roleIndex must be a non-negative safe integer or null");
-	if (roleIndex !== null && role === null) throw new DelegationError("roleIndex requires a non-null role");
-	const requestedEffort = record["reasoningEffort"];
-	if (requestedEffort !== null && !isNonEmptyString(requestedEffort)) throw new DelegationError("reasoningEffort must be a non-empty string or null");
-	if (typeof requestedEffort === "string" && /\s/.test(requestedEffort.trim())) throw new DelegationError(`reasoningEffort ${JSON.stringify(requestedEffort)} contains whitespace. An effort is one token, exactly as the model advertises it.`);
+	const reasoningEffort = record["reasoningEffort"];
+	const cwd = record["cwd"];
+	const maxDepth = record["maxDepth"];
+	if (typeof description !== "string" || description.trim() === "") throw new Error("description must be a non-empty string");
+	if (typeof prompt !== "string" || prompt.trim() === "") throw new Error("prompt must be a non-empty string");
+	if (typeof agent !== "string" || !AGENT_KINDS.includes(agent)) throw new Error(`agent must be one of: ${AGENT_KINDS.join(", ")}`);
+	if (role !== null && (typeof role !== "string" || role.trim() === "")) throw new Error("role must be a non-empty string or null");
+	if (roleIndex !== null && !isNonNegativeInteger(roleIndex)) throw new Error("roleIndex must be a non-negative integer or null");
+	if (roleIndex !== null && role === null) throw new Error("roleIndex requires a non-null role");
+	if (reasoningEffort !== null) {
+		if (typeof reasoningEffort !== "string" || reasoningEffort.trim() === "") throw new Error("reasoningEffort must be a non-empty string or null");
+		if (/\s/.test(reasoningEffort.trim())) throw new Error("reasoningEffort must be one token, exactly as the model advertises it");
+	}
+	if (cwd !== null && (typeof cwd !== "string" || cwd.trim() === "")) throw new Error("cwd must be a non-empty string or null");
+	if (maxDepth !== void 0 && maxDepth !== null && !isDepth(maxDepth)) throw new Error("maxDepth must be a positive safe integer or null");
 	return {
-		description,
-		prompt,
+		description: description.trim(),
+		prompt: prompt.trim(),
 		agent,
 		role,
-		cwd,
-		maxDepth,
 		roleIndex,
-		reasoningEffort: requestedEffort?.trim() ?? null
+		reasoningEffort: typeof reasoningEffort === "string" ? reasoningEffort.trim() : null,
+		cwd: typeof cwd === "string" ? cwd.trim() : null,
+		maxDepth: maxDepth === void 0 ? null : maxDepth
 	};
 }
-/** Route text as reported to the model. */
-function routeLabel(route, effortOverride) {
-	const effort = effectiveEffort(route, effortOverride);
-	const base = route.kind === "inherit" ? "inherit-parent" : `${route.provider}/${route.model}`;
-	return effort === void 0 ? base : `${base} [${effort}]`;
-}
-/** Take the smallest of the requested depth, this row's cap, and the Host setting. */
-function resolveDelegationDepth(configured, requested, hostLimit) {
-	if (requested !== void 0 && requested !== null && !isNonNegativeSafeInteger(requested)) throw new DelegationError("maxDepth must be a non-negative safe integer or null");
-	if (configured !== void 0 && requested !== void 0 && requested !== null && requested > configured) throw new DelegationError(`maxDepth ${requested} exceeds this row's configured cap of ${configured}. Lower the request or raise \`pstack-subagent\` config.maxDepth.`);
-	const candidates = [
-		configured,
-		requested,
-		hostLimit
-	].filter((value) => value !== void 0 && value !== null);
-	return candidates.length === 0 ? void 0 : Math.min(...candidates);
-}
-/**
-* Turn the per-call `maxDepth` request into an effective depth request.
-*
-* A child is always at least depth 1, so `0` can never start one: it is reported and
-* replaced with `DEFAULT_CHILD_MAX_DEPTH` rather than failing the whole call. Omitting
-* `maxDepth` (null) also uses the default, so a caller never has to guess an integer.
-*/
-function requestedChildDepth(requested, diagnostics) {
-	if (requested === null) return 1;
-	if (requested === 0) {
-		diagnostics.push(`maxDepth 0 cannot start a child (the child itself is depth 1), so this call used 1. Omit maxDepth for that default, or pass 1 or more.`);
-		return 1;
-	}
-	return requested;
-}
-/** Resolve the role file, routes, prompt, and depth into one plan. Pure of service calls. */
+/** Resolve the role file, child routes, prompt, and depth before activation starts. */
 function planDelegation(args, runtime, config = DEFAULT_CONFIG) {
-	const rolePath = modelsFilePath(runtime.dshHome);
-	const roleFile = readRoleConfiguration(rolePath);
-	if (roleFile.status === "unreadable") throw new DelegationError(`the pstack role file at ${rolePath} exists but cannot be read (${roleFile.reason}). Fix its permissions or delete it, which makes every role inherit the parent route.`);
+	const path = modelsFilePath(runtime.dshHome);
+	const roleFile = readRoleConfiguration(path);
+	if (roleFile.status === "unreadable") throw new Error(`the pstack role file at ${path} exists but cannot be read (${roleFile.reason}). Fix its permissions or delete it to let every role inherit the parent route.`);
 	const diagnostics = [];
-	if (roleFile.status === "loaded") for (const issue of roleFile.issues) diagnostics.push(`${rolePath}${issue.line === void 0 ? "" : `:${issue.line}`}: ${issue.message}`);
+	if (roleFile.status === "loaded") for (const issue of roleFile.issues) diagnostics.push(`${path}${issue.line === void 0 ? "" : `:${issue.line}`}: ${issue.message}`);
 	let selection;
-	let roleSpec;
 	if (args.role !== null) {
-		roleSpec = requireRole(args.role);
-		const entry = roleFile.status === "loaded" ? roleFile.roles.get(roleSpec.key) : void 0;
-		selection = selectRoleRoutes(roleSpec, entry, args.roleIndex ?? void 0);
-	}
-	const routes = selection?.routes ?? [{ kind: "inherit" }];
-	const roleName = roleSpec?.key ?? "(none)";
-	if (selection !== void 0) {
-		if (!selection.configured) diagnostics.push(`role "${selection.role.key}" has no line in ${rolePath}, so it inherits the parent route. Add a line to route it explicitly.`);
-		if (selection.mode === "selected") diagnostics.push(`role "${selection.role.key}" holds ${selection.total} entries and is not an automatic panel; entry ${(selection.roleIndex ?? 0) + 1} was selected. Pass \`roleIndex\` to choose another.`);
-		if (selection.role.key === "swarm workers") diagnostics.push("role \"swarm workers\" is a list, not a panel: its first entry is the default route for every worker, and `roleIndex` gives one race arm another entry. Each worker still needs its own pstack_subagent call.");
+		const spec = requireRole(args.role);
+		const entry = roleFile.status === "loaded" ? roleFile.roles.get(spec.key) : void 0;
+		selection = selectRoutes(spec, entry, args.roleIndex ?? void 0);
+		if (!selection.configured) diagnostics.push(`role "${spec.key}" has no line in ${path}, so it inherits the parent route`);
+		if (selection.mode === "selected") diagnostics.push(`role "${spec.key}" has ${selection.total} entries; entry ${(selection.roleIndex ?? 0) + 1} was selected`);
 	}
-	const prompt = composeChildPrompt({
-		injectedPaths: args.agent === "poteto-agent" ? [{
-			label: "`poteto-mode` SKILL.md",
-			path: POTETO_MODE_SKILL_PATH
-		}] : [],
-		task: args.prompt
-	});
-	const agentPreset = agentPresetFor(args.agent);
-	const depth = resolveDelegationDepth(config.maxDepth, requestedChildDepth(args.maxDepth, diagnostics), void 0);
+	const routes = selection?.routes ?? [{
+		kind: "inherit",
+		alias: "inherit-parent"
+	}];
+	const roleName = selection?.role.key ?? "(none)";
+	const agentPreset = args.agent === "general" ? void 0 : AGENT_PRESETS[args.agent];
+	const prompt = args.agent === "poteto-agent" ? injectPotetoModePath(args.prompt, runtime.potetoModeSkillPath) : args.prompt;
+	const hostDepth = runtime.subagents.resolveMaxDepth(config.maxDepth);
+	const depth = resolveChildDepth(args.maxDepth, config, hostDepth);
 	const total = routes.length;
 	const children = routes.map((route, index) => {
-		const agentOptions = agentOptionsFor(route, args.reasoningEffort ?? void 0);
-		const effort = effectiveEffort(route, args.reasoningEffort ?? void 0);
+		const effort = args.reasoningEffort ?? (route.kind === "route" ? route.effort : void 0);
 		return {
 			index,
 			total,
@@ -354,296 +140,202 @@ function planDelegation(args, runtime, config = DEFAULT_CONFIG) {
 			agent: args.agent,
 			role: roleName,
 			route,
-			routeLabel: routeLabel(route, args.reasoningEffort ?? void 0),
-			...effort === void 0 ? {} : { effort },
-			...agentOptions === void 0 ? {} : { agentOptions },
+			routeLabel: formatRoute(route, args.reasoningEffort ?? void 0),
+			...isRoute(route) || effort !== void 0 ? { agentOptions: agentOptionsFor(route, effort) } : {},
 			...agentPreset === void 0 ? {} : { agentPreset },
 			...args.cwd === null ? {} : { cwd: args.cwd },
 			...depth === void 0 ? {} : { maxDepth: depth }
 		};
 	});
-	if (args.reasoningEffort !== null) {
-		const overwritten = routes.filter((route) => route.kind === "route" && route.effort !== void 0 && route.effort !== args.reasoningEffort);
-		if (overwritten.length > 0) diagnostics.push(`reasoningEffort "${args.reasoningEffort}" overrides the effort this call's role line names (${overwritten.map((route) => routeLabel(route)).join(", ")}). The role line is a starting level; pass no reasoningEffort to keep it.`);
+	if (selection?.mode === "panel" && children.length > 1) {
+		const distinct = new Set(children.map((child) => child.routeLabel));
+		if (distinct.size === 1) diagnostics.push(`all ${children.length} entries of panel "${roleName}" resolve to ${[...distinct][0]}, so they are not a cross-model panel`);
 	}
-	addPanelDiagnostics(children, roleName, selection, rolePath, diagnostics);
 	return {
-		agent: args.agent,
-		prompt,
 		provider: config.provider,
 		role: roleName,
-		mode: selection?.mode ?? "single",
-		...selection?.roleIndex === void 0 ? {} : { roleIndex: selection.roleIndex },
-		roleTotal: selection?.total ?? 1,
-		...args.reasoningEffort === null ? {} : { reasoningEffort: args.reasoningEffort },
+		prompt,
 		children,
 		diagnostics
 	};
 }
-function addPanelDiagnostics(children, roleName, selection, rolePath, diagnostics) {
-	if (selection === void 0 || selection.mode !== "panel") return;
-	if (children.length === 1) {
-		diagnostics.push(`panel "${roleName}" has one configured entry, so this call started one child and claims no model diversity. Add entries to ${rolePath} for a real panel.`);
-		return;
-	}
-	const distinct = new Set(children.map((child) => child.routeLabel));
-	if (distinct.size > 1) return;
-	diagnostics.push(`all ${children.length} entries of panel "${roleName}" resolve to the same route (${[...distinct][0]}), so these children are not a cross-model review.`);
-}
-/** Check the backend can enforce what this call asks for. */
-function assertProviderCapabilities(capabilities, providerName, asks) {
-	const missing = [];
-	if (asks.agentOptions && !capabilities.agentOptions) missing.push("agentOptions");
-	if (asks.agentPreset && !capabilities.agentPreset) missing.push("agentPreset");
-	if (asks.maxDepth && !capabilities.depthLimit) missing.push("maxDepth");
-	if (missing.length === 0) return;
-	throw new DelegationError(`subagent provider "${providerName}" does not support ${missing.join(", ")}. Change the route to \`inherit-parent\`, drop the unsupported argument, or configure a backend that supports it.`);
-}
-/**
-* Validate every route and effort before anything starts. An inherit-parent entry with
-* no effort carries no agentOptions and is skipped; an override on an inherit route is
-* validated against the parent's live route.
-*/
-async function preflightRoutes(llm, parent, children, roleName, effortOverride, signal) {
-	for (const child of children) {
-		if (child.agentOptions === void 0) continue;
-		if (child.route.kind === "inherit") {
-			const effort = effortOverride;
-			try {
-				await validateEffortFor(llm, void 0, effort, parent, signal);
-			} catch (error) {
-				throw new DelegationError(`role ${roleName} inherits the parent route, and reasoningEffort "${effort}" failed preflight there: ${describeError(error)} Pass null for reasoningEffort, or name a role whose line holds an exact route.`);
-			}
-			continue;
-		}
-		const route = child.route;
-		try {
-			await llm.resolveCallConfig({
-				provider: route.provider,
-				model: route.model,
-				...child.effort === void 0 ? {} : { reasoningEffort: child.effort }
-			}, signal);
-		} catch (error) {
-			throw new DelegationError(`role ${roleName} resolves to ${child.routeLabel}, which failed route preflight: ${describeError(error)}. Check the route in ${modelsFilePath("$DSH_HOME")} against the providers this deployment serves.`);
-		}
-	}
-}
-/**
-* Resolve and validate one child's `cwd`.
-*
-* A child may run inside the session directory or inside an explicitly separate Git
-* worktree. Anything else is refused: this tool does not create isolation, so it will
-* not promise one.
-*/
-function resolveChildCwd(requested, sessionRoot) {
-	if (requested === void 0) return void 0;
-	const resolved = isAbsolute(requested) ? resolve(requested) : resolve(sessionRoot, requested);
-	let isDirectory = false;
-	try {
-		isDirectory = statSync(resolved).isDirectory();
-	} catch {
-		isDirectory = false;
-	}
-	if (!isDirectory) throw new DelegationError(`cwd ${resolved} does not exist or is not a directory`);
-	if (isWithin(sessionRoot, resolved) || isGitWorktree(resolved)) return resolved;
-	throw new DelegationError(`cwd ${resolved} is outside this session's directory (${sessionRoot}) and is not a separate Git worktree. Pass a path inside the session directory, or create a worktree first and pass its checkout.`);
-}
-function isWithin(root, target) {
-	const rel = relative(root, target);
-	return rel === "" || !rel.startsWith("..") && !isAbsolute(rel);
-}
-function isGitWorktree(directory) {
-	return existsSync(join(directory, ".git"));
-}
-/**
-* Start the planned children. Everything rejectable has already been rejected; a
-* failure at entry N keeps children 1..N-1 running and reports their ids.
-*/
-async function startPlannedChildren(parent, plan, runtime, config, signal) {
+/** Start every planned child after one final capability and route check. */
+async function startPlannedChildren(parent, plan, runtime, signal) {
 	const provider = runtime.subagents.getProvider(plan.provider);
 	if (provider === void 0) {
 		const available = runtime.subagents.list();
-		throw new DelegationError(`subagent provider "${plan.provider}" is not registered; registered providers: ${available.length === 0 ? "(none)" : available.join(", ")}.`);
+		throw new Error(`subagent provider "${plan.provider}" is not registered; available providers: ${available.length === 0 ? "(none)" : available.join(", ")}`);
 	}
-	const hostLimit = runtime.subagents.resolveMaxDepth(config.maxDepth);
-	const effectiveDepth = resolveDelegationDepth(config.maxDepth, plan.children[0]?.maxDepth, hostLimit);
-	assertProviderCapabilities(provider.capabilities, plan.provider, {
-		agentOptions: plan.children.some((child) => child.agentOptions !== void 0),
-		agentPreset: plan.children.some((child) => child.agentPreset !== void 0),
-		maxDepth: effectiveDepth !== void 0
-	});
+	assertCapabilities(provider.capabilities, plan);
 	const sessionRoot = await runtime.workingDirectory.ensure(parent, signal);
 	const children = plan.children.map((child) => ({
 		...child,
-		...effectiveDepth === void 0 ? {} : { maxDepth: effectiveDepth },
-		cwd: resolveChildCwd(child.cwd, sessionRoot)
+		...child.cwd === void 0 ? {} : { cwd: resolveChildCwd(child.cwd, sessionRoot) }
 	}));
-	await preflightRoutes(runtime.llm, parent, children, plan.role, plan.reasoningEffort, signal);
-	if (runtime.subagents.getProvider(plan.provider) !== provider) throw new DelegationError(`subagent provider "${plan.provider}" changed while the child routes were being resolved; retry the delegation`);
-	const started = [];
-	const diagnostics = [...plan.diagnostics];
-	for (const child of children) {
+	await preflightRoutes(runtime.llm, parent, children, signal);
+	const records = [];
+	for (const child of children) try {
 		signal.throwIfAborted();
-		try {
-			const activation = await runtime.subagents.startActivation({
-				provider: plan.provider,
-				label: child.label,
-				request: {
-					prompt: [{
-						type: "text",
-						text: plan.prompt
-					}],
-					parent,
-					...child.agentPreset === void 0 ? {} : { agentPreset: child.agentPreset },
-					...child.cwd === void 0 ? {} : { cwd: child.cwd },
-					...child.agentOptions === void 0 ? {} : { agentOptions: child.agentOptions },
-					...child.maxDepth === void 0 ? {} : { maxDepth: child.maxDepth }
-				},
-				signal,
-				delivery: "parent"
-			});
-			const target = runtime.agents === void 0 ? void 0 : runtime.agents.get(activation.childId);
-			if (target === void 0) diagnostics.push(`child ${activation.childId} has no live local Agent handle, so it cannot change its own reasoning effort with \`${SET_EFFORT_TOOL}\`. It runs at ${child.effort === void 0 ? "the adapter default" : `"${child.effort}"`} for its whole life.`);
-			else installEffortControl(target, runtime.llm);
-			started.push({
-				childId: activation.childId,
-				label: child.label,
-				agent: child.agent,
-				role: child.role,
-				route: child.routeLabel,
-				...child.effort === void 0 ? {} : { effort: child.effort },
+		const activation = await runtime.subagents.startActivation({
+			provider: plan.provider,
+			label: child.label,
+			request: {
+				prompt: [{
+					type: "text",
+					text: plan.prompt
+				}],
+				parent,
+				...child.agentPreset === void 0 ? {} : { agentPreset: child.agentPreset },
+				...child.cwd === void 0 ? {} : { cwd: child.cwd },
+				...child.agentOptions === void 0 ? {} : { agentOptions: child.agentOptions },
 				...child.maxDepth === void 0 ? {} : { maxDepth: child.maxDepth }
-			});
-		} catch (error) {
-			if (started.length === 0) throw new DelegationError(`starting the ${plan.role === "(none)" ? "child" : `"${plan.role}" child`} "${child.label}" (${child.routeLabel}) failed: ${describeError(error)}`);
-			throw new PartialDelegationError(`starting entry ${child.index + 1} of ${child.total} for role "${plan.role}" (${child.label}, ${child.routeLabel}) failed: ${describeError(error)}. The ${started.length} already started: ${started.map((record) => `${record.childId} (${record.label})`).join(", ")}. Those children are still running and are not rolled back.`, started);
-		}
+			},
+			signal,
+			delivery: "parent"
+		});
+		records.push(recordFor(child, activation));
+	} catch (error) {
+		if (records.length === 0) throw error;
+		return {
+			kind: "partial",
+			children: records,
+			diagnostics: plan.diagnostics,
+			failure: {
+				index: child.index,
+				total: child.total,
+				label: child.label,
+				routeLabel: child.routeLabel,
+				reason: describeError(error)
+			}
+		};
 	}
 	return {
-		kind: started.length === 1 ? "activation" : "activations",
-		children: started,
-		diagnostics
+		kind: records.length === 1 ? "activation" : "activations",
+		children: records,
+		diagnostics: plan.diagnostics
+	};
+}
+/** Complete delegation: plan, check, start. */
+async function delegate(parent, args, runtime, config, signal) {
+	return startPlannedChildren(parent, planDelegation(args, runtime, config), runtime, signal);
+}
+function apply(ctx, rawConfig) {
+	const config = normalizeConfig(rawConfig);
+	const runtime = {
+		subagents: ctx.subagents,
+		llm: ctx.llm,
+		workingDirectory: ctx.workingDirectory,
+		dshHome: resolveDshHome(),
+		potetoModeSkillPath: POTETO_MODE_SKILL_PATH
 	};
+	ctx.tools.register(createPstackSubagentTool(runtime, config));
 }
-/** Full delegation: plan, validate, start. */
-async function delegate(parent, args, runtime, config = DEFAULT_CONFIG, signal = new AbortController().signal) {
-	return startPlannedChildren(parent, planDelegation(args, runtime, config), runtime, config, signal);
+/** Build the model-facing tool. Exported for tests. */
+function createPstackSubagentTool(runtime, config = DEFAULT_CONFIG) {
+	return {
+		name: TOOL_NAME,
+		description: TOOL_DESCRIPTION,
+		parameters: {
+			type: "object",
+			additionalProperties: false,
+			properties: {
+				description: {
+					type: "string",
+					description: "A short 3-5 word label for the child, shown in notices."
+				},
+				prompt: {
+					type: "string",
+					description: "The complete, self-contained brief. The child does not see this conversation."
+				},
+				agent: {
+					type: "string",
+					enum: [...AGENT_KINDS],
+					description: "Child preset: \"poteto-agent\" for pstack implementation, \"comment-sicko\" for comment review, \"readonly\" for a sandbox-enforced read-only researcher, or \"general\" to join the parent preset."
+				},
+				role: {
+					type: ["string", "null"],
+					description: "pstack role key. It selects the child's route and effort; null inherits the parent route."
+				},
+				roleIndex: {
+					type: ["integer", "null"],
+					description: "Entry index for a list role that is not an automatic panel. Null selects the first entry; omit only through null."
+				},
+				reasoningEffort: {
+					type: ["string", "null"],
+					description: "One advertised reasoning-effort token for this call, or null to keep the role's starting effort."
+				},
+				cwd: {
+					type: ["string", "null"],
+					description: "Initial child working directory inside this session or a separate Git worktree; null inherits the session directory."
+				},
+				maxDepth: {
+					type: ["integer", "null"],
+					minimum: 1,
+					description: "Optional delegation-depth cap. Omit or null to use the configured row and Host cap; explicit values must be at least 1 and cannot exceed that cap."
+				}
+			},
+			required: [
+				"description",
+				"prompt",
+				"agent",
+				"role",
+				"roleIndex",
+				"reasoningEffort",
+				"cwd"
+			]
+		},
+		output: {
+			schema: OUTPUT_SCHEMA,
+			render: (_args, value) => [{
+				type: "text",
+				text: renderDelegation(value)
+			}]
+		},
+		isConcurrencySafe: () => true,
+		async execute(rawArgs, exec) {
+			if (exec.agent === void 0) throw new Error("pstack_subagent requires a calling Agent");
+			return delegate(exec.agent, parseDelegationArgs(rawArgs), runtime, config, exec.signal);
+		}
+	};
 }
 const TOOL_DESCRIPTION = [
 	"Delegate self-contained work to one or more fresh child agents.",
 	"",
-	"Every argument is required except maxDepth. Pass null for role, roleIndex, reasoningEffort,",
-	"or cwd when that setting is not used. Choose agent explicitly. maxDepth is optional: omit it",
-	"(or pass null) for the safe default of 1, where the child exists but cannot start its own",
-	"children; pass 2 or more, within this row's configured cap, to let it delegate further. A 0 is",
-	"treated as 1, because a child is already depth 1 and 0 can never start one.",
-	"",
-	"Returns the id of every child it started plus the route each one resolved. The children run",
-	"in the background: their results arrive later as completion notices in this conversation.",
-	"Never wait or poll for them, and keep working on something useful meanwhile.",
+	"Every argument is required except maxDepth. Pass null for role, roleIndex, reasoningEffort, or cwd when unused. Choose agent explicitly.",
 	"",
-	"`agent` picks a DSH agent preset: `poteto-agent` mounts the pstack implementation delegate",
-	"and receives the absolute path of the poteto-mode instructions; `comment-sicko` mounts the",
-	"read-only comment reviewer; `general` joins the parent preset with no pstack persona.",
+	"Omit `maxDepth` to use the configured row and Host delegation cap; an explicit value cannot exceed that cap.",
 	"",
-	"`role` selects a configured pstack role and therefore the child's model route and reasoning",
-	"effort. The role names come from the pstack role manifest, not this schema. Null inherits the",
-	"parent route. Automatic panel roles start one child per configured entry.",
+	"`agent` picks a DSH agent preset: `poteto-agent` mounts the pstack implementation delegate and receives the absolute path of the poteto-mode instructions; `comment-sicko` mounts the comment reviewer; `readonly` mounts a read-only research preset with the sandbox policy set to read-only; `general` joins the parent preset.",
 	"",
-	"`reasoningEffort` raises or lowers the effort for this delegation alone, using a token the",
-	"target model advertises. An unsupported value fails the call before any child starts,",
-	"including every entry of a panel. Null keeps the role line's starting effort. A running child",
-	"can change its own effort with `set_reasoning_effort`, which applies from its next step.",
+	"`role` selects a configured pstack role and therefore the child's model route and starting effort. Null inherits the parent route. Automatic panel roles start one child per configured entry; `pstack_roles` reports which roles accept `roleIndex`.",
 	"",
-	"Every prompt is a fresh child that does not see this conversation, so include everything it",
-	"needs. A child's own summary is not evidence: review its diff or artifacts yourself."
+	"Returns every child id with its resolved route. Results arrive as completion notices; do not poll."
 ].join("\n");
-function buildParameters() {
-	return {
-		type: "object",
-		additionalProperties: false,
-		properties: {
-			description: {
-				type: "string",
-				description: "A short 3-5 word label for the child, shown in list_agents and its notices."
-			},
-			prompt: {
-				type: "string",
-				description: "The complete, self-contained brief. The child does not see this conversation, so include the task, every prior directive, and the paths or branches it needs."
-			},
-			agent: {
-				type: "string",
-				enum: [...AGENT_KINDS],
-				description: "Required child preset: \"poteto-agent\" for pstack implementation delegates, \"comment-sicko\" for the read-only comment reviewer, or \"general\" to join the parent preset."
-			},
-			role: {
-				type: ["string", "null"],
-				description: "Required pstack role key, matched exactly. It selects the child's model route and effort. Null inherits the parent route."
-			},
-			cwd: {
-				type: ["string", "null"],
-				description: "Required initial working directory, inside this session's directory or a separate Git worktree. Relative paths resolve against this session's directory. Null inherits the session directory."
-			},
-			maxDepth: {
-				type: ["integer", "null"],
-				description: "Optional delegation-depth cap for the child. Omit or pass null for the safe default of 1 (the child exists but cannot start its own children). Pass 2 or more, within this row's configured cap, to let it delegate further. 0 is treated as 1: a child is already depth 1, so 0 can never start one."
-			},
-			roleIndex: {
-				type: ["integer", "null"],
-				description: "Required entry index for a list-valued role that is not an automatic panel, 0 or greater. Requires a non-null role. Null selects the first entry. Call `pstack_roles` to see which roles accept an index."
-			},
-			reasoningEffort: {
-				type: ["string", "null"],
-				description: "Required reasoning-effort override for every child this call starts. Use one token the target model advertises, or null to keep the role line's starting effort. An unknown token fails the whole call before any child starts."
-			}
-		},
-		required: [...REQUIRED_DELEGATION_ARGUMENT_KEYS]
-	};
-}
 const OUTPUT_SCHEMA = {
 	type: "object",
 	additionalProperties: false,
 	properties: {
 		kind: {
 			type: "string",
-			enum: ["activation", "activations"],
-			description: "How many children this call started."
+			enum: [
+				"activation",
+				"activations",
+				"partial"
+			]
 		},
 		children: {
 			type: "array",
-			description: "One record per started child, in start order.",
 			items: {
 				type: "object",
 				additionalProperties: false,
 				properties: {
-					childId: {
-						type: "string",
-						description: "The child session id."
-					},
-					label: {
-						type: "string",
-						description: "The label this call gave the child."
-					},
-					agent: {
-						type: "string",
-						description: "The child brief type."
-					},
-					role: {
-						type: "string",
-						description: "The role key, or `(none)`."
-					},
-					route: {
-						type: "string",
-						description: "`inherit-parent` or `<provider>/<model> [effort]`."
-					},
-					effort: {
-						type: "string",
-						description: "The reasoning effort this child runs at, when one applies."
-					},
-					maxDepth: {
-						type: "integer",
-						description: "The delegation-depth cap the child received."
-					}
+					childId: { type: "string" },
+					label: { type: "string" },
+					agent: { type: "string" },
+					role: { type: "string" },
+					route: { type: "string" },
+					effort: { type: "string" },
+					maxDepth: { type: "integer" }
 				},
 				required: [
 					"childId",
@@ -656,8 +348,25 @@ const OUTPUT_SCHEMA = {
 		},
 		diagnostics: {
 			type: "array",
-			description: "Configuration notes, including a panel whose entries share one route.",
 			items: { type: "string" }
+		},
+		failure: {
+			type: "object",
+			additionalProperties: false,
+			properties: {
+				index: { type: "integer" },
+				total: { type: "integer" },
+				label: { type: "string" },
+				routeLabel: { type: "string" },
+				reason: { type: "string" }
+			},
+			required: [
+				"index",
+				"total",
+				"label",
+				"routeLabel",
+				"reason"
+			]
 		}
 	},
 	required: [
@@ -666,62 +375,110 @@ const OUTPUT_SCHEMA = {
 		"diagnostics"
 	]
 };
-/** Render one delegation result for the model. */
-function renderDelegationResult(result) {
+function renderDelegation(result) {
 	const lines = [
-		result.kind === "activation" ? `started 1 child: ${result.children[0]?.childId ?? "(none)"}` : `started ${result.children.length} children:`,
+		result.kind === "partial" ? `started ${result.children.length} of ${result.failure.total} children; entry ${result.failure.index + 1} failed: ${result.failure.reason}` : result.kind === "activation" ? `started 1 child: ${result.children[0]?.childId ?? "(none)"}` : `started ${result.children.length} children:`,
 		...result.children.map((child) => `- ${child.childId} (${child.label}) agent=${child.agent} role=${child.role} route=${child.route}${child.effort === void 0 ? "" : ` effort=${child.effort}`}${child.maxDepth === void 0 ? "" : ` maxDepth=${child.maxDepth}`}`),
-		"Results arrive as completion notices; do not poll for them. Each child can change its own",
-		"effort from its next step with `set_reasoning_effort`."
+		"Results from started children arrive as completion notices; do not poll for them."
 	];
-	if (result.diagnostics.length > 0) lines.push("", "Diagnostics:", ...result.diagnostics.map((diagnostic) => `- ${diagnostic}`));
+	if (result.diagnostics.length > 0) lines.push("", "Diagnostics:", ...result.diagnostics.map((line) => `- ${line}`));
 	return lines.join("\n");
 }
-/** Build the model-facing tool for one row. */
-function createPstackSubagentTool(runtime, config = DEFAULT_CONFIG) {
+function recordFor(child, activation) {
 	return {
-		name: TOOL_NAME,
-		description: TOOL_DESCRIPTION,
-		parameters: buildParameters(),
-		output: {
-			schema: OUTPUT_SCHEMA,
-			render: (_args, value) => [{
-				type: "text",
-				text: renderDelegationResult(value)
-			}]
-		},
-		isConcurrencySafe: () => true,
-		async execute(rawArgs, exec) {
-			const parent = exec.agent;
-			if (parent === void 0) throw new DelegationError("pstack_subagent needs a calling Agent, and exec.agent was undefined.");
-			return await delegate(parent, parseDelegationArgs(rawArgs), runtime, config, exec.signal);
-		}
+		childId: activation.childId,
+		label: child.label,
+		agent: child.agent,
+		role: child.role,
+		route: child.routeLabel,
+		...child.agentOptions?.reasoningEffort === void 0 ? {} : { effort: child.agentOptions.reasoningEffort },
+		...child.maxDepth === void 0 ? {} : { maxDepth: child.maxDepth }
 	};
 }
-/** Build the runtime a real Host call reads. */
-function hostRuntime(ctx, getAgentRegistry = () => void 0) {
-	return {
-		subagents: ctx.subagents,
-		llm: ctx.llm,
-		workingDirectory: ctx.workingDirectory,
-		get agents() {
-			return getAgentRegistry();
-		},
-		dshHome: resolveDshHome()
-	};
+function agentOptionsFor(route, effort) {
+	return isRoute(route) ? {
+		provider: route.provider,
+		model: route.model,
+		...effort === void 0 ? {} : { reasoningEffort: effort }
+	} : effort === void 0 ? {} : { reasoningEffort: effort };
 }
-/** Register `pstack_subagent` for this row. */
-function apply(ctx, config) {
-	const resolved = normalizeConfig(config);
-	const runtime = hostRuntime(ctx, () => ctx.get?.("agents"));
-	ctx.tools.register(createPstackSubagentTool(runtime, resolved));
+function formatRoute(route, override) {
+	const effort = override ?? (isRoute(route) ? route.effort : void 0);
+	const base = isRoute(route) ? `${route.provider}/${route.model}` : "inherit-parent";
+	return effort === void 0 ? base : `${base} [${effort}]`;
 }
-function describeError(error) {
-	if (error instanceof Error) {
-		const code = error.code;
-		const suffix = typeof code === "string" ? ` (${code})` : "";
-		return `${error.message}${suffix}`;
+function resolveChildDepth(requested, config, hostLimit) {
+	const depth = requested ?? hostLimit;
+	if (depth === void 0) return void 0;
+	if (!Number.isSafeInteger(depth) || depth < 1) throw new Error("maxDepth must be a positive safe integer or null");
+	if (config.maxDepth !== void 0 && depth > config.maxDepth) throw new Error(`maxDepth ${depth} exceeds this row's configured cap of ${config.maxDepth}`);
+	if (hostLimit !== void 0 && depth > hostLimit) throw new Error(`maxDepth ${depth} exceeds the Host delegation limit of ${hostLimit}`);
+	return depth;
+}
+function assertCapabilities(capabilities, plan) {
+	const needsAgentOptions = plan.children.some((child) => child.agentOptions !== void 0);
+	const needsPreset = plan.children.some((child) => child.agentPreset !== void 0);
+	const needsDepth = plan.children.some((child) => child.maxDepth !== void 0);
+	const missing = [
+		...needsAgentOptions && !capabilities.agentOptions ? ["agentOptions"] : [],
+		...needsPreset && !capabilities.agentPreset ? ["agentPreset"] : [],
+		...needsDepth && !capabilities.depthLimit ? ["maxDepth"] : []
+	];
+	if (missing.length > 0) throw new Error(`subagent provider does not support ${missing.join(", ")}`);
+}
+async function preflightRoutes(llm, parent, children, signal) {
+	for (const child of children) {
+		const options = child.agentOptions;
+		if (options === void 0) continue;
+		const provider = options.provider ?? parent.options.provider;
+		const model = options.model ?? parent.options.model;
+		if (provider === void 0 || model === void 0) throw new Error(`cannot resolve role "${child.role}" because the child has no provider/model and the parent has not made a request yet`);
+		try {
+			await llm.resolveCallConfig({
+				provider,
+				model,
+				...options.reasoningEffort === void 0 ? {} : { reasoningEffort: options.reasoningEffort }
+			}, signal);
+		} catch (error) {
+			throw new Error(`role "${child.role}" resolves to ${child.routeLabel}, which failed route preflight: ${describeError(error)}`);
+		}
+	}
+}
+function resolveChildCwd(requested, sessionRoot) {
+	const target = isAbsolute(requested) ? resolve(requested) : resolve(sessionRoot, requested);
+	let directory = false;
+	try {
+		directory = statSync(target).isDirectory();
+	} catch {
+		directory = false;
 	}
-	return String(error);
+	if (!directory) throw new Error(`cwd ${target} does not exist or is not a directory`);
+	if (isWithin(sessionRoot, target) || existsSync(join(target, ".git"))) return target;
+	throw new Error(`cwd ${target} is outside the session directory (${sessionRoot}) and is not a Git worktree`);
+}
+function isWithin(root, target) {
+	const rel = relative(root, target);
+	return rel === "" || !rel.startsWith("..") && !isAbsolute(rel);
+}
+function injectPotetoModePath(task, path) {
+	return [
+		"Absolute path injected by `pstack_subagent`:",
+		`- \`poteto-mode\` SKILL.md: \`${path}\``,
+		"",
+		"Read it directly before doing any work. Your skill catalog hides this skill, so a missing catalog entry does not mean pstack is absent.",
+		"",
+		"Task:",
+		"",
+		task
+	].join("\n");
+}
+function isNonNegativeInteger(value) {
+	return typeof value === "number" && Number.isSafeInteger(value) && value >= 0;
+}
+function isDepth(value) {
+	return isNonNegativeInteger(value) && value >= 1;
+}
+function describeError(error) {
+	return error instanceof Error ? error.message : String(error);
 }
-export { AGENT_KINDS, AGENT_PRESETS, DEFAULT_CHILD_MAX_DEPTH, DEFAULT_SUBAGENT_PROVIDER, DELEGATION_ARGUMENT_KEYS, DelegationError, PartialDelegationError, REQUIRED_DELEGATION_ARGUMENT_KEYS, TOOL_NAME, agentPresetFor, apply, assertProviderCapabilities, composeChildPrompt, createPstackSubagentTool, delegate, hostRuntime, inject, name, normalizeConfig, parseDelegationArgs, planDelegation, renderDelegationResult, requestedChildDepth, resolveChildCwd, resolveDelegationDepth, routeLabel, startPlannedChildren };
+export { AGENT_KINDS, DEFAULT_PROVIDER, TOOL_NAME, apply, createPstackSubagentTool, delegate, inject, name, normalizeConfig, parseDelegationArgs, planDelegation, startPlannedChildren };