dsh-pstack 0.2.0-beta.1 → 0.2.0-beta.3

package/dist/subagent.mjs

@@ -1,162 +1,13 @@
-import { i as POTETO_MODE_SKILL_PATH, o as agentPreamblePath, t as ASSETS_ROOT } from "./paths-D_JlVmBk.mjs";
-import { c as parseFrontmatter } from "./frontmatter-BQX3ekwP.mjs";
-import { join } from "node:path";
-import { readFileSync } from "node:fs";
+import { a as POTETO_MODE_SKILL_PATH } from "./paths-BT77khwm.mjs";
+import { isAbsolute, join, relative, resolve } from "node:path";
+import { existsSync, readFileSync, statSync } from "node:fs";
 import { homedir } from "node:os";
-//#region scripts/role-manifest.ts
-/** File name the role policy lives under inside `$DSH_HOME`. */
-const PSTACK_MODELS_FILENAME = "pstack-models.md";
-/** Values that resolve to "send no `agentOptions` and inherit the parent route". */
-const INHERIT_PARENT_ALIASES = ["inherit-parent", "auto"];
-const ROLE_MANIFEST = [
-	{
-		key: "feature, refactoring",
-		shape: "single",
-		autoPanel: false,
-		consumers: ["poteto-mode/playbooks/feature.md", "poteto-mode/playbooks/refactoring.md"],
-		summary: "One route for both code playbooks. The comma joins two playbook names into a single key."
-	},
-	{
-		key: "bug-fix",
-		shape: "single",
-		autoPanel: false,
-		consumers: ["poteto-mode/playbooks/bug-fix.md"],
-		summary: "Implementation route for the bug-fix playbook."
-	},
-	{
-		key: "perf-issue",
-		shape: "single",
-		autoPanel: false,
-		consumers: ["poteto-mode/playbooks/perf-issue.md"],
-		summary: "Implementation route for the perf-issue playbook."
-	},
-	{
-		key: "hillclimb",
-		shape: "single",
-		autoPanel: false,
-		consumers: ["poteto-mode/playbooks/hillclimb.md"],
-		summary: "Implementation route for the hillclimb playbook."
-	},
-	{
-		key: "judgment and prose",
-		shape: "single",
-		autoPanel: false,
-		consumers: [
-			"poteto-mode/SKILL.md",
-			"unslop",
-			"technical-writing"
-		],
-		summary: "Route for prose work and vague-instruction judgment calls."
-	},
-	{
-		key: "hardest tasks",
-		shape: "single",
-		autoPanel: false,
-		consumers: ["poteto-mode/SKILL.md"],
-		summary: "Route for the strongest judgment model on cross-cutting or subtle work."
-	},
-	{
-		key: "how explorer",
-		shape: "single",
-		autoPanel: false,
-		consumers: ["how/SKILL.md", "how/references/explorer-prompt.md"],
-		summary: "Route for each how explorer child."
-	},
-	{
-		key: "how explainer",
-		shape: "single",
-		autoPanel: false,
-		consumers: ["how/SKILL.md", "how/references/explainer-prompt.md"],
-		summary: "Route for the single how explainer child."
-	},
-	{
-		key: "why investigators",
-		shape: "single",
-		autoPanel: false,
-		consumers: ["why/SKILL.md", "why/references/investigator-prompt.md"],
-		summary: "Route for each why investigator child."
-	},
-	{
-		key: "why synthesizer",
-		shape: "single",
-		autoPanel: false,
-		consumers: ["why/SKILL.md", "why/references/synthesizer-prompt.md"],
-		summary: "Route for the why synthesis child."
-	},
-	{
-		key: "reflect tooling",
-		shape: "single",
-		autoPanel: false,
-		consumers: ["reflect/SKILL.md", "reflect/references/tooling-reviewer.md"],
-		summary: "Route for the reflect tooling reviewer."
-	},
-	{
-		key: "reflect judgment, divergent, synthesizer",
-		shape: "single",
-		autoPanel: false,
-		consumers: [
-			"reflect/SKILL.md",
-			"reflect/references/judgment-reviewer.md",
-			"reflect/references/divergent-reviewer.md",
-			"reflect/references/synthesizer.md"
-		],
-		summary: "One shared route for the judgment reviewer, the divergent reviewer, and the synthesizer. The commas belong to the key."
-	},
-	{
-		key: "arena runners",
-		shape: "list",
-		autoPanel: true,
-		consumers: ["arena/SKILL.md"],
-		summary: "Panel: one arena child per entry, all started together."
-	},
-	{
-		key: "arena cross-judge pool",
-		shape: "list",
-		autoPanel: false,
-		consumers: ["arena/SKILL.md"],
-		summary: "A pool arena picks one entry from, preferring a model family unlike the parent's. Not fanned out automatically."
-	},
-	{
-		key: "swarm workers",
-		shape: "list",
-		autoPanel: false,
-		consumers: ["swarm/SKILL.md", "poteto-mode/playbooks/multi-phase-plan.md"],
-		summary: "One default route for every worker. A race gives each arm another entry through `roleIndex`, and the caller still starts one worker per call."
-	},
-	{
-		key: "architect runners",
-		shape: "list",
-		autoPanel: true,
-		consumers: ["architect/SKILL.md"],
-		summary: "Panel: one architect child per entry, all started together."
-	},
-	{
-		key: "interrogate reviewers",
-		shape: "list",
-		autoPanel: true,
-		consumers: ["interrogate/SKILL.md"],
-		summary: "Panel: one interrogate reviewer per entry, all started together."
-	}
-];
-ROLE_MANIFEST.map((role) => role.key);
-ROLE_MANIFEST.filter((role) => role.autoPanel).map((role) => role.key);
-/** Exact manifest lookup. Callers normalize the key first. */
-function findRoleSpec(key) {
-	return ROLE_MANIFEST.find((role) => role.key === key);
-}
-//#endregion
-//#region src/effort.ts
+// 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.
-*
-* The session's latest request header is the live route after a model switch, and
-* `Agent.options` is the creation-time fallback. DSH's own delegation code resolves
-* `inherit-parent` the same way, so reading `options` alone would validate a
-* different model than the child really inherits.
-*
-* An explicit route wins over both, because it is what the delegation will request.
+* 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;
@@ -168,18 +19,10 @@ function resolveEffortTarget(agent, route) {
 		model
 	} : void 0;
 }
-/** The efforts one route advertises, in the adapter's own order. */
 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.
-*
-* Rejects with the target's real effort list rather than a generic message, because
-* "max is not supported" leaves the caller guessing while "supports xhigh, high,
-* medium" is actionable. A model that advertises no efforts at all is reported
-* separately: that is a route incapable of the control, not a wrong token.
-*/
+/** 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.");
@@ -195,60 +38,37 @@ async function validateEffortFor(llm, route, effort, parent, signal) {
 /**
 * Install the effort control on one live child.
 *
-* Registration is child-scoped, so the waterfall and the tool dispose with the
-* child: a sibling delegation never sees them, and a later child cannot inherit a
-* stale value.
-*
-* The waterfall re-states the starting effort on every request and applies a
-* change from the next request onward. It never rewrites a request already in
-* flight, because `next()` has already been called and its result is the request
-* the machine built.
-*
-* A missing route is not fatal here. The child still runs on whatever the adapter
-* chose; the tool reports the missing route when the child tries to change effort.
-*
-* @param child - the live child agent, already started.
-* @param llm - the adapter host used to validate a requested change.
-* @param initialEffort - the effort the delegation started the child at, if any.
-* @returns a disposer that unwinds this contribution early, or does nothing when
-* the child context is already gone.
+* 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, initialEffort) {
-	let current = initialEffort;
-	const disposeListener = child.ctx.on("agent/request", async (_payload, next) => {
-		const config = await next();
-		return current === void 0 ? config : {
-			...config,
-			reasoningEffort: current
-		};
-	});
-	const disposeTool = child.ctx.tools.register(defineSetEffortTool(child, llm, (effort) => {
-		current = effort;
-	}));
-	return () => {
-		disposeTool();
-		disposeListener();
-	};
+function installEffortControl(child, llm) {
+	return child.ctx.tools.register(defineSetEffortTool(child, llm));
 }
-/** The child's own view of its effort, for the tool body. */
-function defineSetEffortTool(child, llm, apply) {
+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.",
-			"This tool may not be visible until your second step. The effort you were started with applies to the first step, so a slow first step is expected when that level is high."
+			"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: { 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."
-		} },
+		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: String(value)
+				text: typeof value === "string" ? value : JSON.stringify(value)
 			}]
 		},
 		isConcurrencySafe: () => false,
@@ -256,7 +76,8 @@ function defineSetEffortTool(child, llm, apply) {
 			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 target = resolveEffortTarget(exec.agent ?? child);
+			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 {
@@ -266,22 +87,16 @@ function defineSetEffortTool(child, llm, apply) {
 			}
 			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(", ")}.`);
-			apply(value);
+			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.
-*
-* The two independent halves are easy to get wrong in opposite directions:
-*
-* - an inherit-parent route with an override must still send `reasoningEffort`,
-*   which means `{ reasoningEffort }` on its own is a real `agentOptions` value and
-*   the backend still has to support the capability;
-* - an override always wins over the role line, because the caller asked for this
-*   delegation specifically.
-*/
+/** 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 };
@@ -302,128 +117,133 @@ function describeError$2(error) {
 	}
 	return String(error);
 }
-//#endregion
-//#region src/roles.ts
-/**
-* Role lookup over the single manifest in `scripts/role-manifest.ts`.
-*
-* A role name is matched exactly after normalization. A near miss is reported as
-* a hint and never guessed, because silently running a role on the wrong route is
-* worse than refusing the delegation.
-*
-* Commas belong to some keys (`feature, refactoring`), so nothing here splits a
-* key on a comma. Only a role file *value* is a comma-separated list.
-*/
-/** Raised when a role name does not match the manifest. */
-var RoleNameError = class extends Error {
-	constructor(message) {
-		super(message);
-		this.name = "RoleNameError";
-	}
-};
+// src/internal/role-manifest.ts
 /**
-* Normalize a role name for lookup.
+* File name the role configuration lives under inside `$DSH_HOME`.
 *
-* Case, surrounding whitespace, runs of whitespace, spacing around a comma, and a
-* trailing colon are formatting, not naming. Nothing else is normalized: a
-* different word is a different role.
+* It keeps the upstream name `pstack-models.mdc`, so a reader moving from the Cursor docs
+* recognizes it, but stores it directly under `$DSH_HOME`; DSH does not load `.mdc` files
+* on its own, and `pstack_subagent` is the only reader.
 */
-function normalizeRoleName(raw) {
-	return raw.normalize("NFC").trim().toLowerCase().replace(/\s+/g, " ").replace(/\s*,\s*/g, ", ").replace(/:+$/, "").trim();
-}
-/** Look a role up by its exact normalized key. */
-function lookupRole(raw) {
-	return findRoleSpec(normalizeRoleName(raw));
-}
-/**
-* Near-miss hints for a role name that did not match.
-*
-* Only used to build a diagnostic. A suggestion is never turned into a selection.
-*/
-function suggestRoles(raw, limit = 3) {
-	const target = normalizeRoleName(raw);
-	if (target === "") return [];
-	return ROLE_MANIFEST.map((role) => {
-		const shared = sharedTokenCount(target, role.key);
-		const distance = editDistance(target, role.key);
-		const contains = role.key.includes(target) || target.includes(role.key);
-		const score = (contains ? 1e3 : 0) + shared * 100 - distance + Math.min(role.key.length, target.length);
-		return {
-			key: role.key,
-			score,
-			distance,
-			shared,
-			contains
-		};
-	}).filter((candidate) => candidate.contains || candidate.shared > 0 || candidate.distance <= 4).sort((left, right) => right.score - left.score).slice(0, limit).map((candidate) => candidate.key);
-}
-/**
-* Resolve a role name, or throw a diagnostic that names the near misses.
-*/
-function requireRole(raw) {
-	const spec = lookupRole(raw);
-	if (spec !== void 0) return spec;
-	const suggestions = suggestRoles(raw);
-	const hint = suggestions.length === 0 ? "No role key is close to it." : `Closest role keys: ${suggestions.map((key) => `"${key}"`).join(", ")}.`;
-	throw new RoleNameError(`unknown pstack role ${JSON.stringify(raw)}. ${hint} Role names come from the role manifest, not from free text: pass the exact key, or omit \`role\` to inherit the parent route.`);
-}
-function sharedTokenCount(left, right) {
-	const rightTokens = new Set(tokenize(right));
-	return [...new Set(tokenize(left))].filter((token) => rightTokens.has(token)).length;
-}
-function tokenize(value) {
-	return value.split(/[^a-z0-9]+/).filter((token) => token !== "");
-}
-/** Levenshtein distance, iterative with one row of state. */
-function editDistance(left, right) {
-	if (left === right) return 0;
-	if (left === "") return right.length;
-	if (right === "") return left.length;
-	let previous = Array.from({ length: right.length + 1 }, (_, index) => index);
-	for (let row = 1; row <= left.length; row += 1) {
-		const current = [row];
-		for (let column = 1; column <= right.length; column += 1) {
-			const substitution = previous[column - 1] + (left[row - 1] === right[column - 1] ? 0 : 1);
-			const deletion = previous[column] + 1;
-			const insertion = current[column - 1] + 1;
-			current.push(Math.min(substitution, deletion, insertion));
-		}
-		previous = current;
+const PSTACK_MODELS_FILENAME = "pstack-models.mdc";
+/** Values that resolve to "send no `agentOptions` and inherit the parent route". */
+const INHERIT_PARENT_ALIASES = ["inherit-parent", "auto"];
+/** The 17 roles, in manifest order. */
+const ROLE_MANIFEST = [
+	{
+		key: "feature, refactoring",
+		shape: "single",
+		autoPanel: false
+	},
+	{
+		key: "bug-fix",
+		shape: "single",
+		autoPanel: false
+	},
+	{
+		key: "perf-issue",
+		shape: "single",
+		autoPanel: false
+	},
+	{
+		key: "hillclimb",
+		shape: "single",
+		autoPanel: false
+	},
+	{
+		key: "judgment and prose",
+		shape: "single",
+		autoPanel: false
+	},
+	{
+		key: "hardest tasks",
+		shape: "single",
+		autoPanel: false
+	},
+	{
+		key: "how explorer",
+		shape: "single",
+		autoPanel: false
+	},
+	{
+		key: "how explainer",
+		shape: "single",
+		autoPanel: false
+	},
+	{
+		key: "why investigators",
+		shape: "single",
+		autoPanel: false
+	},
+	{
+		key: "why synthesizer",
+		shape: "single",
+		autoPanel: false
+	},
+	{
+		key: "reflect tooling",
+		shape: "single",
+		autoPanel: false
+	},
+	{
+		key: "reflect judgment, divergent, synthesizer",
+		shape: "single",
+		autoPanel: false
+	},
+	{
+		key: "arena runners",
+		shape: "list",
+		autoPanel: true
+	},
+	{
+		key: "arena cross-judge pool",
+		shape: "list",
+		autoPanel: false
+	},
+	{
+		key: "swarm workers",
+		shape: "list",
+		autoPanel: false
+	},
+	{
+		key: "architect runners",
+		shape: "list",
+		autoPanel: true
+	},
+	{
+		key: "interrogate reviewers",
+		shape: "list",
+		autoPanel: true
 	}
-	return previous[right.length];
+];
+/** Exact manifest lookup. Callers normalize the key first. */
+function findRoleSpec(key) {
+	return ROLE_MANIFEST.find((role) => role.key === key);
 }
-//#endregion
-//#region src/models.ts
+// src/internal/role-config.ts
 /**
-* The pstack role file: `$DSH_HOME/pstack-models.md`.
+* The pstack role configuration: `$DSH_HOME/pstack-models.mdc`.
 *
-* `pstack_subagent` reads this file again on every delegation, so a change reaches
-* the next child and never a child that already started.
+* `pstack_subagent` reads this file again on every delegation, so a change reaches the
+* next child and never a child that already started.
 *
-* Three states are kept apart on purpose, because they need different fixes:
-*
-* - the file does not exist: no policy is configured, so every role inherits the
-*   parent route;
-* - the file exists but cannot be read: a broken installation or permission
-*   problem, reported as an error and never treated as "no configuration";
-* - the file loads but a line is wrong: a typo in a key, an unknown retired role,
-*   or a malformed value is reported with the line number and the near misses.
+* Three states stay apart because each needs a different fix: the file is missing (no
+* policy, so every role inherits the parent route), unreadable (a broken install, never
+* treated as "no configuration"), or loaded with per-line issues reported by line.
 */
 /** Absolute path of the role file for one DSH home. */
 function modelsFilePath(dshHome) {
 	return join(dshHome, PSTACK_MODELS_FILENAME);
 }
-/**
-* Resolve `$DSH_HOME` the way the Host does: the environment when set, else
-* `~/.dsh`.
-*/
+/** Resolve `$DSH_HOME` the way the Host does: the environment when set, else `~/.dsh`. */
 function resolveDshHome(env = process.env, home = homedir()) {
 	const configured = env["DSH_HOME"];
-	if (configured !== void 0 && configured.trim() !== "") return configured;
-	return join(home, ".dsh");
+	const path = configured !== void 0 && configured.trim() !== "" ? configured : join(home, ".dsh");
+	const expanded = path === "~" ? home : path.startsWith("~/") || path.startsWith("~\\") ? join(home, path.slice(2)) : path;
+	return resolve(expanded);
 }
 /** Read the role file. Never throws for a missing or unreadable file. */
-function readRoleFile(path) {
+function readRoleConfiguration(path) {
 	let text;
 	try {
 		text = readFileSync(path, "utf8");
@@ -438,15 +258,14 @@ function readRoleFile(path) {
 			reason: describeError$1(error)
 		};
 	}
-	return parseRoleFile(text, path);
+	return parseRoleConfiguration(text, path);
 }
 /** Parse role-file text. Pure: no file system access. */
-function parseRoleFile(text, path) {
+function parseRoleConfiguration(text, path) {
 	const issues = [];
 	const roles = /* @__PURE__ */ new Map();
-	const lines = stripFrontmatter(text).split("\n");
 	let budget;
-	lines.forEach((rawLine, index) => {
+	stripFrontmatter(text).split("\n").forEach((rawLine, index) => {
 		const line = index + 1;
 		const trimmed = rawLine.trim();
 		if (trimmed === "") return;
@@ -473,7 +292,7 @@ function parseRoleFile(text, path) {
 			});
 			return;
 		}
-		const spec = findSpec(key);
+		const spec = findRoleSpec(key);
 		if (spec === void 0) {
 			issues.push({
 				line,
@@ -482,7 +301,9 @@ function parseRoleFile(text, path) {
 			return;
 		}
 		const parsed = parseRouteValue(rawValue);
-		for (const issue of parsed.issues) issues.push({
+		const entryIssues = [...parsed.issues];
+		if (spec.shape === "single" && parsed.routes.length > 1) entryIssues.push("a single-route role cannot hold a comma-separated list");
+		for (const issue of entryIssues) issues.push({
 			line,
 			message: `${rawKey}: ${issue}`
 		});
@@ -497,8 +318,8 @@ function parseRoleFile(text, path) {
 			rawKey,
 			key: spec.key,
 			spec,
-			rawValue,
-			routes: parsed.routes
+			routes: parsed.routes,
+			issues: entryIssues
 		});
 	});
 	return budget === void 0 ? {
@@ -573,28 +394,27 @@ var RouteSelectionError = class extends Error {
 /**
 * Expand one role into the routes this call starts.
 *
-* The three auto-panel roles start one child per entry. Another list role selects
-* one entry, the first by default, and `roleIndex` chooses which. A single role
-* holds exactly one route and rejects `roleIndex`.
+* Automatic panels start one child per entry. Another list role selects one entry, the
+* first by default; `roleIndex` chooses which. A single role holds exactly one route and
+* rejects `roleIndex`.
 */
 function selectRoleRoutes(spec, entry, roleIndex) {
 	if (spec.shape === "single" && roleIndex !== void 0) throw new RouteSelectionError(`role "${spec.key}" holds a single route, so \`roleIndex\` does not apply; drop \`roleIndex\` or use a list-valued role (${listRoleKeys().join(", ")})`);
-	if (spec.autoPanel && roleIndex !== void 0) throw new RouteSelectionError(`role "${spec.key}" is an automatic panel: it starts one child per entry, so \`roleIndex\` does not apply; configure the entries you want in ${PSTACK_MODELS_FILENAME}`);
+	if (spec.autoPanel && roleIndex !== void 0) throw new RouteSelectionError(`role "${spec.key}" is an automatic panel: it starts one child per entry, so \`roleIndex\` does not apply`);
 	if (entry === void 0) return {
 		role: spec,
 		mode: spec.autoPanel ? "panel" : "single",
 		configured: false,
 		routes: [{ kind: "inherit" }],
-		total: 1,
-		issues: []
+		total: 1
 	};
+	if (entry.issues.length > 0) throw new RouteSelectionError(`role "${spec.key}" has invalid configuration on line ${entry.line}: ${entry.issues.join("; ")}. Fix the role file before delegating.`);
 	if (spec.autoPanel) return {
 		role: spec,
 		mode: "panel",
 		configured: true,
 		routes: entry.routes,
-		total: entry.routes.length,
-		issues: []
+		total: entry.routes.length
 	};
 	if (spec.shape === "list") {
 		const index = roleIndex ?? 0;
@@ -605,8 +425,7 @@ function selectRoleRoutes(spec, entry, roleIndex) {
 			configured: true,
 			routes: [entry.routes[index]],
 			total: entry.routes.length,
-			roleIndex: index,
-			issues: []
+			roleIndex: index
 		};
 	}
 	return {
@@ -614,21 +433,78 @@ function selectRoleRoutes(spec, entry, roleIndex) {
 		mode: "single",
 		configured: true,
 		routes: [entry.routes[0]],
-		total: entry.routes.length,
-		issues: []
+		total: entry.routes.length
 	};
 }
 /** Keys of the list-valued roles, including the automatic panels. */
 function listRoleKeys() {
 	return ROLE_MANIFEST.filter((role) => role.shape === "list").map((role) => role.key);
 }
-function findSpec(key) {
-	return ROLE_MANIFEST.find((role) => role.key === key);
+/** Raised when a role name does not match the manifest. */
+var RoleNameError = class extends Error {
+	constructor(message) {
+		super(message);
+		this.name = "RoleNameError";
+	}
+};
+/** Normalize a role name for lookup. Case, spacing, spacing around a comma, and a trailing colon are formatting. */
+function normalizeRoleName(raw) {
+	return raw.normalize("NFC").trim().toLowerCase().replace(/\s+/g, " ").replace(/\s*,\s*/g, ", ").replace(/:+$/, "").trim();
+}
+/** Resolve a role name, or throw a diagnostic that names the near misses. */
+function requireRole(raw) {
+	const spec = findRoleSpec(normalizeRoleName(raw));
+	if (spec !== void 0) return spec;
+	const suggestions = suggestRoles(raw);
+	const hint = suggestions.length === 0 ? "No role key is close to it." : `Closest role keys: ${suggestions.map((key) => `"${key}"`).join(", ")}.`;
+	throw new RoleNameError(`unknown pstack role ${JSON.stringify(raw)}. ${hint} Role names come from the role manifest, not from free text: pass the exact key, or omit \`role\` to inherit the parent route.`);
+}
+function suggestRoles(raw, limit = 3) {
+	const target = normalizeRoleName(raw);
+	if (target === "") return [];
+	return ROLE_MANIFEST.map((role) => {
+		const shared = sharedTokenCount(target, role.key);
+		const distance = editDistance(target, role.key);
+		const contains = role.key.includes(target) || target.includes(role.key);
+		const score = (contains ? 1e3 : 0) + shared * 100 - distance + Math.min(role.key.length, target.length);
+		return {
+			key: role.key,
+			score,
+			distance,
+			shared,
+			contains
+		};
+	}).filter((candidate) => candidate.contains || candidate.shared > 0 || candidate.distance <= 4).sort((left, right) => right.score - left.score).slice(0, limit).map((candidate) => candidate.key);
 }
 function unknownRoleMessage(rawKey, key) {
 	const suggestions = suggestRoles(key);
 	const hint = suggestions.length === 0 ? "no role key is close to it" : `closest role keys: ${suggestions.map((candidate) => `"${candidate}"`).join(", ")}`;
-	return `unknown role ${JSON.stringify(rawKey)} (${hint}). A line for a retired or misspelled role would otherwise be ignored, so it is reported instead.`;
+	return `unknown role ${JSON.stringify(rawKey)} (${hint}). A line for a retired or misspelled role is reported rather than ignored.`;
+}
+function sharedTokenCount(left, right) {
+	const rightTokens = new Set(tokenize(right));
+	return [...new Set(tokenize(left))].filter((token) => rightTokens.has(token)).length;
+}
+function tokenize(value) {
+	return value.split(/[^a-z0-9]+/).filter((token) => token !== "");
+}
+/** Levenshtein distance, iterative with one row of state. */
+function editDistance(left, right) {
+	if (left === right) return 0;
+	if (left === "") return right.length;
+	if (right === "") return left.length;
+	let previous = Array.from({ length: right.length + 1 }, (_, index) => index);
+	for (let row = 1; row <= left.length; row += 1) {
+		const current = [row];
+		for (let column = 1; column <= right.length; column += 1) {
+			const substitution = previous[column - 1] + (left[row - 1] === right[column - 1] ? 0 : 1);
+			const deletion = previous[column] + 1;
+			const insertion = current[column - 1] + 1;
+			current.push(Math.min(substitution, deletion, insertion));
+		}
+		previous = current;
+	}
+	return previous[right.length];
 }
 function stripFrontmatter(text) {
 	const normalized = text.replaceAll("\r\n", "\n");
@@ -648,54 +524,38 @@ function describeError$1(error) {
 	}
 	return String(error);
 }
-//#endregion
-//#region src/subagent.ts
+// src/subagent.ts
 /**
 * `pstack_subagent`: pstack's delegation entry point.
 *
-* One call plans one or more fresh children, validates everything before the first
-* start, then starts them through `ctx.subagents.startActivation` with parent
-* delivery. Results arrive as completion notices; this module never polls.
-*
-* Validation order follows the design's delegation flow, and every check that can
-* reject a call happens before the first child starts, except a start failure in
-* the middle of a panel, which keeps the children already running and names them.
+* 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.
 */
 /** Cordis row name, used by Loader diagnostics. */
 const name = "pstack-subagent";
 /**
-* Delegation needs the tool registry, the subagent registry, and the LLM adapters.
-*
-* `agents` is deliberately **not** here. It is used only to reach a child this tool
-* has just started, and a composition without it still delegates: the children keep
-* the effort their request carried and cannot change it afterwards. Declaring it as
-* a hard dependency would make the whole row inert in such a profile, so the tool
-* would not exist at all — the failure `pstack-history` already paid for once with
-* `sessionQuery`. It is read per call through `ctx.get("agents")` instead, which is
-* why a controller mounted after this row is still seen.
+* 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"
+	"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";
-/** Agent type used when the call does not name one. */
-const DEFAULT_AGENT_KIND = "poteto-agent";
 /** The child kinds this tool knows how to brief. */
 const AGENT_KINDS = [
 	"poteto-agent",
 	"comment-sicko",
 	"general"
 ];
-const DEFAULT_CONFIG = {
-	provider: DEFAULT_SUBAGENT_PROVIDER,
-	allowedModels: [],
-	allowedAgentPresets: []
-};
+const DEFAULT_CONFIG = { provider: DEFAULT_SUBAGENT_PROVIDER };
 /** Raised for a configuration or argument problem, before any child starts. */
 var DelegationError = class extends Error {
 	constructor(message) {
@@ -712,13 +572,6 @@ var PartialDelegationError = class extends DelegationError {
 		this.startedChildren = startedChildren;
 	}
 };
-/** Raised when a required agent preamble is missing or unusable. */
-var PreambleError = class extends DelegationError {
-	constructor(message) {
-		super(message);
-		this.name = "PreambleError";
-	}
-};
 function isNonEmptyString(value) {
 	return typeof value === "string" && value.trim() !== "";
 }
@@ -726,139 +579,38 @@ function isNonNegativeSafeInteger(value) {
 	return typeof value === "number" && Number.isSafeInteger(value) && value >= 0;
 }
 /**
-* Minimal Standard Schema for this row, so an omitted `config` is valid and a
-* typo fails at load instead of at the first delegation.
-*/
-const Config = { "~standard": {
-	version: 1,
-	vendor: "dsh-pstack",
-	validate(value) {
-		if (value === void 0 || value === null) return { value: DEFAULT_CONFIG };
-		if (typeof value !== "object" || Array.isArray(value)) return { issues: [{ message: "config must be a mapping" }] };
-		const record = value;
-		const issues = [];
-		const allowedKeys = /* @__PURE__ */ new Set([
-			"provider",
-			"maxDepth",
-			"allowedModels",
-			"allowedAgentPresets"
-		]);
-		for (const key of Object.keys(record)) if (!allowedKeys.has(key)) issues.push({
-			message: `unknown field "${key}"`,
-			path: [key]
-		});
-		const provider = record["provider"];
-		if (provider !== void 0 && !isNonEmptyString(provider)) issues.push({
-			message: "provider must be a non-empty string",
-			path: ["provider"]
-		});
-		const maxDepth = record["maxDepth"];
-		if (maxDepth !== void 0 && !isNonNegativeSafeInteger(maxDepth)) issues.push({
-			message: "maxDepth must be a non-negative safe integer",
-			path: ["maxDepth"]
-		});
-		const allowedModels = record["allowedModels"];
-		const models = [];
-		if (allowedModels !== void 0) {
-			if (!Array.isArray(allowedModels)) issues.push({
-				message: "allowedModels must be an array",
-				path: ["allowedModels"]
-			});
-			else allowedModels.forEach((entry, index) => {
-				if (typeof entry !== "object" || entry === null || Array.isArray(entry)) {
-					issues.push({
-						message: "each allowedModels entry must be a { provider, model } object",
-						path: ["allowedModels", index]
-					});
-					return;
-				}
-				const route = entry;
-				const routeProvider = route["provider"];
-				const routeModel = route["model"];
-				if (!isNonEmptyString(routeProvider) || !isNonEmptyString(routeModel)) {
-					issues.push({
-						message: "each allowedModels entry needs a non-empty provider and model",
-						path: ["allowedModels", index]
-					});
-					return;
-				}
-				if (models.some((known) => known.provider === routeProvider && known.model === routeModel)) {
-					issues.push({
-						message: `duplicate allowedModels route "${routeProvider}/${routeModel}"`,
-						path: ["allowedModels", index]
-					});
-					return;
-				}
-				models.push({
-					provider: routeProvider,
-					model: routeModel
-				});
-			});
-		}
-		const allowedPresets = record["allowedAgentPresets"];
-		const presets = [];
-		if (allowedPresets !== void 0) {
-			if (!Array.isArray(allowedPresets)) issues.push({
-				message: "allowedAgentPresets must be an array",
-				path: ["allowedAgentPresets"]
-			});
-			else allowedPresets.forEach((entry, index) => {
-				if (!isNonEmptyString(entry)) {
-					issues.push({
-						message: "each allowedAgentPresets entry must be a non-empty string",
-						path: ["allowedAgentPresets", index]
-					});
-					return;
-				}
-				if (presets.includes(entry)) {
-					issues.push({
-						message: `duplicate allowedAgentPresets entry "${entry}"`,
-						path: ["allowedAgentPresets", index]
-					});
-					return;
-				}
-				presets.push(entry);
-			});
-		}
-		if (issues.length > 0) return { issues };
-		return { value: {
-			provider: isNonEmptyString(provider) ? provider : DEFAULT_SUBAGENT_PROVIDER,
-			...isNonNegativeSafeInteger(maxDepth) ? { maxDepth } : {},
-			allowedModels: models,
-			allowedAgentPresets: presets
-		} };
-	}
-} };
-/**
-* Read one agent preamble from the package assets by absolute path.
+* Validate the row configuration without a Schemastery dependency.
 *
-* A missing or empty preamble blocks that agent type. It never degrades into a
-* plain child, because a `comment-sicko` or `poteto-agent` child without its
-* instructions is a different agent wearing the name.
+* 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.
 */
-function loadAgentPreamble(agent, assetsRoot) {
-	if (agentPreamblePath(agent) === void 0) return void 0;
-	const absolute = join(assetsRoot, "agents", `${agent}.md`);
-	let text;
-	try {
-		text = readFileSync(absolute, "utf8");
-	} catch (error) {
-		const code = error.code;
-		throw new PreambleError(`the ${agent} preamble cannot be read at ${absolute} (${typeof code === "string" ? code : String(error)}). This agent type does not start without its preamble; reinstall dsh-pstack so the packaged assets are present, or pass agent: "general" for a plain child that gets no preamble.`);
-	}
-	const body = parseFrontmatter(text).body.trim();
-	if (body === "") throw new PreambleError(`the ${agent} preamble at ${absolute} is empty. This agent type does not start without its instructions; repair the packaged asset or pass agent: "general".`);
-	return body;
+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");
+	const maxDepth = record["maxDepth"];
+	if (maxDepth !== void 0 && !isNonNegativeSafeInteger(maxDepth)) throw new Error("pstack-subagent: maxDepth must be a non-negative safe integer");
+	return {
+		provider: isNonEmptyString(provider) ? provider : DEFAULT_SUBAGENT_PROVIDER,
+		...isNonNegativeSafeInteger(maxDepth) ? { maxDepth } : {}
+	};
 }
-/**
-* Compose the self-contained prompt one child receives.
-*
-* The preambles used to locate things themselves; the Host injects the absolute
-* paths instead, so a child never scans a plugin directory or guesses a version.
-*/
+/** 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 = [];
-	if (input.preamble !== void 0) sections.push(input.preamble);
 	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.",
@@ -869,141 +621,102 @@ function composeChildPrompt(input) {
 	sections.push(`Task:\n\n${input.task}`);
 	return sections.join("\n\n---\n\n");
 }
-/** The text form of one route, as reported back to the model. */
-function routeLabel(route) {
-	if (route.kind === "inherit") return "inherit-parent";
-	return route.effort === void 0 ? `${route.provider}/${route.model}` : `${route.provider}/${route.model} [${route.effort}]`;
-}
-/**
-* The route text a caller sees when an effort override wins.
-*
-* `routeLabel` describes the role file, which stops being the whole truth once the
-* caller overrode the effort. Reporting the file's value there would tell the model
-* its override had been ignored.
-*/
-function plannedRouteLabel(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.
-*
-* A request above the row cap is an error rather than a silent clamp, so a caller
-* cannot believe it asked for two levels when the row allows one.
-*/
-function resolveDelegationDepth(configured, requested, hostLimit) {
-	if (requested !== void 0 && !isNonNegativeSafeInteger(requested)) throw new DelegationError("maxDepth must be a non-negative safe integer");
-	if (configured !== void 0 && requested !== void 0 && requested > configured) throw new DelegationError(`maxDepth ${requested} exceeds this row's configured cap of ${configured}. Lower the request or raise \`pstack-subagent\` config.maxDepth; this tool never forwards a depth above its own row.`);
-	const candidates = [
-		configured,
-		requested,
-		hostLimit
-	].filter((value) => value !== void 0);
-	return candidates.length === 0 ? void 0 : Math.min(...candidates);
-}
-/**
-* Validate the model-facing arguments.
-*
-* The tool schema is advisory at the model boundary, so every constraint is
-* re-checked here and reported with the reason.
-*/
-function parseDelegationArgs(raw, config = DEFAULT_CONFIG) {
+/** Validate every required model-facing argument. Null declines an optional setting. */
+function parseDelegationArgs(raw) {
 	if (typeof raw !== "object" || raw === null || Array.isArray(raw)) throw new DelegationError("arguments must be an object");
 	const record = raw;
-	const known = /* @__PURE__ */ new Set([
+	const known = [
 		"description",
 		"prompt",
 		"agent",
 		"role",
-		"cwd",
-		...config.allowedAgentPresets.length > 0 ? ["agentPreset"] : [],
-		"maxDepth",
 		"roleIndex",
-		"reasoningEffort"
-	]);
-	for (const key of Object.keys(record)) if (!known.has(key)) throw new DelegationError(`unknown argument "${key}": ${key === "agentPreset" ? "agentPreset is only available when this row configures allowedAgentPresets" : `accepted arguments are ${[...known].join(", ")}`}`);
+		"reasoningEffort",
+		"cwd",
+		"maxDepth"
+	];
+	for (const key of known) if (!Object.hasOwn(record, key)) throw new DelegationError(`missing required argument "${key}"; every argument must be passed explicitly`);
+	for (const key of Object.keys(record)) if (!known.includes(key)) throw new DelegationError(`unknown argument "${key}"; accepted arguments are ${known.join(", ")}`);
 	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 agentRaw = record["agent"];
-	if (agentRaw !== void 0 && typeof agentRaw !== "string") throw new DelegationError(`agent must be a string: ${AGENT_KINDS.join(", ")}`);
-	const agent = agentRaw ?? "poteto-agent";
-	if (!AGENT_KINDS.includes(agent)) throw new DelegationError(`unknown agent "${agent}". Accepted kinds are ${AGENT_KINDS.join(", ")}; a pstack agent type is not a DSH agent preset, so \`agent\` never accepts a preset id.`);
+	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 !== void 0 && !isNonEmptyString(role)) throw new DelegationError("role must be a non-empty string when present");
+	if (role !== null && !isNonEmptyString(role)) throw new DelegationError("role must be a non-empty string or null");
 	const cwd = record["cwd"];
-	if (cwd !== void 0 && !isNonEmptyString(cwd)) throw new DelegationError("cwd must be a non-empty string when present");
-	const agentPreset = record["agentPreset"];
-	if (agentPreset !== void 0) {
-		if (config.allowedAgentPresets.length === 0) throw new DelegationError("agentPreset is not available: this pstack-subagent row configures no allowedAgentPresets");
-		if (!isNonEmptyString(agentPreset) || !config.allowedAgentPresets.includes(agentPreset)) throw new DelegationError(`agentPreset ${JSON.stringify(agentPreset)} is not in this row's allowlist (${config.allowedAgentPresets.join(", ")})`);
-	}
+	if (cwd !== null && !isNonEmptyString(cwd)) throw new DelegationError("cwd must be a non-empty string or null");
 	const maxDepth = record["maxDepth"];
-	if (maxDepth !== void 0 && !isNonNegativeSafeInteger(maxDepth)) throw new DelegationError("maxDepth must be a non-negative safe integer");
+	if (!isNonNegativeSafeInteger(maxDepth)) throw new DelegationError("maxDepth must be a non-negative safe integer");
 	const roleIndex = record["roleIndex"];
-	if (roleIndex !== void 0) {
-		if (!isNonNegativeSafeInteger(roleIndex)) throw new DelegationError("roleIndex must be a non-negative safe integer");
-		if (role === void 0) throw new DelegationError("roleIndex requires role: it selects one entry of a list-valued role, and without a role there is no list");
-	}
+	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 !== void 0 && !isNonEmptyString(requestedEffort)) throw new DelegationError("reasoningEffort must be a non-empty string when present. It is one adapter-owned token, validated against the model this call will actually request; an empty value would silently re-enable the adapter default.");
+	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.`);
 	return {
 		description,
 		prompt,
 		agent,
-		...role === void 0 ? {} : { role },
-		...cwd === void 0 ? {} : { cwd },
-		...agentPreset === void 0 ? {} : { agentPreset },
-		...maxDepth === void 0 ? {} : { maxDepth },
-		...roleIndex === void 0 ? {} : { roleIndex },
-		...requestedEffort === void 0 ? {} : { reasoningEffort: requestedEffort.trim() }
+		role,
+		cwd,
+		maxDepth,
+		roleIndex,
+		reasoningEffort: requestedEffort?.trim() ?? null
 	};
 }
-/**
-* Resolve the role file, the routes, the prompt, and the depth into one plan.
-*
-* Pure with respect to the Host: it reads the role file and the packaged preamble,
-* and it makes no service call.
-*/
+/** 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 && !isNonNegativeSafeInteger(requested)) throw new DelegationError("maxDepth must be a non-negative safe integer");
+	if (configured !== void 0 && requested !== void 0 && 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);
+	return candidates.length === 0 ? void 0 : Math.min(...candidates);
+}
+/** Resolve the role file, routes, prompt, and depth into one plan. Pure of service calls. */
 function planDelegation(args, runtime, config = DEFAULT_CONFIG) {
-	const preamble = loadAgentPreamble(args.agent, runtime.assetsRoot);
 	const rolePath = modelsFilePath(runtime.dshHome);
-	const roleFile = readRoleFile(rolePath);
-	if (roleFile.status === "unreadable") throw new DelegationError(`the pstack role file at ${rolePath} exists but cannot be read (${roleFile.reason}). This is a broken configuration, not "no configuration": fix its permissions or delete the file, which makes every role inherit the parent route.`);
+	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 diagnostics = [];
 	if (roleFile.status === "loaded") for (const issue of roleFile.issues) diagnostics.push(`${rolePath}${issue.line === void 0 ? "" : `:${issue.line}`}: ${issue.message}`);
 	let selection;
 	let roleSpec;
-	if (args.role !== void 0) {
+	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);
+		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, because this tool cannot compare separate calls.");
+		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.");
 	}
-	assertAllowedModels(routes, config, roleName);
-	const injected = args.agent === "poteto-agent" ? [{
-		label: "`poteto-mode` SKILL.md",
-		path: POTETO_MODE_SKILL_PATH
-	}] : [];
 	const prompt = composeChildPrompt({
-		...preamble === void 0 ? {} : { preamble },
-		injectedPaths: injected,
+		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, args.maxDepth, void 0);
 	const total = routes.length;
 	const children = routes.map((route, index) => {
-		const agentOptions = agentOptionsFor(route, args.reasoningEffort);
-		const effort = effectiveEffort(route, args.reasoningEffort);
+		const agentOptions = agentOptionsFor(route, args.reasoningEffort ?? void 0);
+		const effort = effectiveEffort(route, args.reasoningEffort ?? void 0);
 		return {
 			index,
 			total,
@@ -1011,17 +724,17 @@ function planDelegation(args, runtime, config = DEFAULT_CONFIG) {
 			agent: args.agent,
 			role: roleName,
 			route,
-			routeLabel: plannedRouteLabel(route, args.reasoningEffort),
+			routeLabel: routeLabel(route, args.reasoningEffort ?? void 0),
 			...effort === void 0 ? {} : { effort },
 			...agentOptions === void 0 ? {} : { agentOptions },
-			...args.agentPreset === void 0 ? {} : { agentPreset: args.agentPreset },
-			...args.cwd === void 0 ? {} : { cwd: args.cwd },
+			...agentPreset === void 0 ? {} : { agentPreset },
+			...args.cwd === null ? {} : { cwd: args.cwd },
 			...depth === void 0 ? {} : { maxDepth: depth }
 		};
 	});
-	if (args.reasoningEffort !== void 0) {
+	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 baseline, so this is allowed; pass no reasoningEffort to keep it.`);
+		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.`);
 	}
 	addPanelDiagnostics(children, roleName, selection, rolePath, diagnostics);
 	return {
@@ -1032,18 +745,11 @@ function planDelegation(args, runtime, config = DEFAULT_CONFIG) {
 		mode: selection?.mode ?? "single",
 		...selection?.roleIndex === void 0 ? {} : { roleIndex: selection.roleIndex },
 		roleTotal: selection?.total ?? 1,
-		...args.reasoningEffort === void 0 ? {} : { reasoningEffort: args.reasoningEffort },
+		...args.reasoningEffort === null ? {} : { reasoningEffort: args.reasoningEffort },
 		children,
 		diagnostics
 	};
 }
-function assertAllowedModels(routes, config, roleName) {
-	if (config.allowedModels.length === 0) return;
-	const unauthorized = routes.filter((route) => route.kind === "route" && !config.allowedModels.some((allowed) => allowed.provider === route.provider && allowed.model === route.model));
-	if (unauthorized.length === 0) return;
-	const allowed = config.allowedModels.map((route) => `${route.provider}/${route.model}`).join(", ");
-	throw new DelegationError(`role ${roleName} resolves to ${unauthorized.map(routeLabel).join(", ")}, which this row's allowedModels does not authorize. Authorized routes: ${allowed}. This is pstack's own row limit and is unrelated to the native session allowedModels policy.`);
-}
 function addPanelDiagnostics(children, roleName, selection, rolePath, diagnostics) {
 	if (selection === void 0 || selection.mode !== "panel") return;
 	if (children.length === 1) {
@@ -1052,20 +758,21 @@ function addPanelDiagnostics(children, roleName, selection, rolePath, diagnostic
 	}
 	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. Configure distinct routes in ${rolePath} for a real panel.`);
+	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 every requested effort, before anything starts.
-*
-* `resolveCallConfig` rejects an unsupported effort and any route the owning
-* adapter does not serve. An inherit-parent entry has no provider/model of its own,
-* so its effort is validated against the parent's live route instead. This does not
-* prove credentials, quota, or a successful model call; only a real child request
-* does.
-*
-* Every entry is checked before the first child starts. A panel whose second entry
-* cannot take the effort therefore starts none of them, which is the only outcome
-* that leaves no half-applied override behind.
+* 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) {
@@ -1075,54 +782,58 @@ async function preflightRoutes(llm, parent, children, roleName, 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)} Omit \`reasoningEffort\`, or name a role whose line holds an exact route.`);
+				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;
-		const callConfig = {
-			provider: route.provider,
-			model: route.model,
-			...child.effort === void 0 ? {} : { reasoningEffort: child.effort }
-		};
 		try {
-			await llm.resolveCallConfig(callConfig, signal);
+			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)}. ${await effortHint(llm, route, child.effort, signal)}`);
+			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.`);
 		}
 	}
 }
-async function effortHint(llm, route, effort, signal) {
-	if (effort !== void 0) try {
-		const efforts = (await llm.resolveModelInfo(route.provider, route.model, signal)).reasoning?.efforts.map((entry) => entry.id) ?? [];
-		if (efforts.length > 0) return `Model ${route.provider}/${route.model} supports these efforts: ${efforts.join(", ")}.`;
-		return `Model ${route.provider}/${route.model} advertises no reasoning efforts.`;
+/**
+* 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 {
-		return `Fix the route or the effort in ${PSTACK_MODELS_FILENAME}; the configured provider/model pair may not be served by this deployment.`;
+		isDirectory = false;
 	}
-	return `Check the route in ${PSTACK_MODELS_FILENAME} against the providers this deployment registers.`;
+	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.`);
 }
-/** 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(", ")}, so this call cannot be honored. Change the route to \`inherit-parent\`, drop the unsupported argument, or configure a backend that supports it; this tool never silently ignores it.`);
+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 that can be rejected has already been rejected. A failure at entry N
-* keeps children 1..N-1 running and reports their ids, because the runtime owns
-* those activations and this tool cannot roll them back.
+* 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) {
 	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(", ")}. This pstack-subagent row delegates to "${plan.provider}"; compose that backend or change the row's provider.`);
+		throw new DelegationError(`subagent provider "${plan.provider}" is not registered; registered providers: ${available.length === 0 ? "(none)" : available.join(", ")}.`);
 	}
 	const hostLimit = runtime.subagents.resolveMaxDepth(config.maxDepth);
 	const effectiveDepth = resolveDelegationDepth(config.maxDepth, plan.children[0]?.maxDepth, hostLimit);
@@ -1131,9 +842,11 @@ async function startPlannedChildren(parent, plan, runtime, config, signal) {
 		agentPreset: plan.children.some((child) => child.agentPreset !== void 0),
 		maxDepth: effectiveDepth !== void 0
 	});
+	const sessionRoot = await runtime.workingDirectory.ensure(parent, signal);
 	const children = plan.children.map((child) => ({
 		...child,
-		...effectiveDepth === void 0 ? {} : { maxDepth: effectiveDepth }
+		...effectiveDepth === void 0 ? {} : { maxDepth: effectiveDepth },
+		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`);
@@ -1151,8 +864,8 @@ async function startPlannedChildren(parent, plan, runtime, config, signal) {
 						text: plan.prompt
 					}],
 					parent,
-					...child.cwd === void 0 ? {} : { cwd: child.cwd },
 					...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 }
 				},
@@ -1161,7 +874,7 @@ async function startPlannedChildren(parent, plan, runtime, config, signal) {
 			});
 			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, child.effort);
+			else installEffortControl(target, runtime.llm);
 			started.push({
 				childId: activation.childId,
 				label: child.label,
@@ -1173,7 +886,7 @@ async function startPlannedChildren(parent, plan, runtime, config, signal) {
 			});
 		} 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; do not start them again, and use list_agents or send_message to reach them.`, started);
+			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);
 		}
 	}
 	return {
@@ -1189,82 +902,79 @@ async function delegate(parent, args, runtime, config = DEFAULT_CONFIG, signal =
 const TOOL_DESCRIPTION = [
 	"Delegate self-contained work to one or more fresh child agents.",
 	"",
+	"Every argument is required. Pass null for role, roleIndex, reasoningEffort, or cwd when that",
+	"setting is not used. Choose agent explicitly. maxDepth is a non-negative integer. No argument",
+	"has an implicit default.",
+	"",
 	"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.",
 	"",
-	"`agent` picks the brief: `poteto-agent` (default) injects the poteto subagent preamble and",
-	"the absolute path of the poteto-mode instructions; `comment-sicko` injects the comment",
-	"reviewer persona and must be briefed read-only; `general` is a plain child with no preamble.",
+	"`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.",
 	"",
 	"`role` selects a configured pstack role and therefore the child's model route and reasoning",
-	"effort. The role names and the meaning of each are in the pstack role manifest, not here.",
-	"Omit `role` to inherit the parent route. Panel roles start one child per configured entry.",
+	"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.",
 	"",
-	"`reasoningEffort` raises or lowers the effort for this delegation alone. The role line is a",
-	"starting baseline, not a verdict: start from it, raise it for hard work and lower it for",
-	"routine work. The value must be one the target model advertises, and this package does not",
-	"assume a ladder, so read the effort list the role's model reports rather than guessing. An",
-	"unsupported value fails the call before any child starts, including every entry of a panel.",
-	"",
-	"A running child can also change its own effort with `set_reasoning_effort`, which applies",
-	"from its next step. You do not have to predict the level up front.",
+	"`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.",
 	"",
 	"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."
 ].join("\n");
-function buildParameters(config) {
-	const 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: "Child brief type: \"poteto-agent\" (default) for pstack implementation delegates, \"comment-sicko\" for the read-only comment reviewer, \"general\" for a plain child with no preamble."
-		},
-		role: {
-			type: "string",
-			description: "A pstack role key, matched exactly. It selects the child's model route and effort. Omit it to inherit the parent route."
-		},
-		cwd: {
-			type: "string",
-			description: "The child's initial working directory. Relative paths resolve against this session's directory. Omitted inherits it."
-		},
-		maxDepth: {
-			type: "integer",
-			description: "How many further delegation levels the child may use; 0 or greater, and negative values are rejected. It can only lower this row's configured cap, never raise it."
-		},
-		roleIndex: {
-			type: "integer",
-			description: "Selects one entry of a list-valued role that is not an automatic panel; 0 or greater, and negative values are rejected. Requires `role`."
-		},
-		reasoningEffort: {
-			type: "string",
-			description: [
-				"Reasoning effort for every child this call starts, overriding the role line's value for this delegation.",
-				"Start from the role's level, raise it for hard work, lower it for routine work.",
-				"The value must be one the target model advertises: this package assumes no ladder and rejects an unknown token with that model's real effort list.",
-				"It is validated for every panel entry before any child starts, so one incompatible entry fails the whole call instead of half-starting it.",
-				"Omit it to use the role line's effort."
-			].join(" ")
-		}
-	};
-	if (config.allowedAgentPresets.length > 0) properties["agentPreset"] = {
-		type: "string",
-		enum: [...config.allowedAgentPresets],
-		description: "An independent child preset from this row's allowlist."
-	};
+function buildParameters() {
 	return {
 		type: "object",
 		additionalProperties: false,
-		properties,
-		required: ["description", "prompt"]
+		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",
+				description: "Required non-negative delegation-depth limit. It can only lower this row's configured cap."
+			},
+			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."
+			},
+			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: [
+			"description",
+			"prompt",
+			"agent",
+			"role",
+			"roleIndex",
+			"reasoningEffort",
+			"cwd",
+			"maxDepth"
+		]
 	};
 }
 const OUTPUT_SCHEMA = {
@@ -1305,7 +1015,7 @@ const OUTPUT_SCHEMA = {
 					},
 					effort: {
 						type: "string",
-						description: "The reasoning effort this child runs at, when one applies. Equal to the call's `reasoningEffort` when that was passed."
+						description: "The reasoning effort this child runs at, when one applies."
 					},
 					maxDepth: {
 						type: "integer",
@@ -1323,7 +1033,7 @@ const OUTPUT_SCHEMA = {
 		},
 		diagnostics: {
 			type: "array",
-			description: "Configuration notes, including a panel whose entries share one route. Empty when nothing needs reporting.",
+			description: "Configuration notes, including a panel whose entries share one route.",
 			items: { type: "string" }
 		}
 	},
@@ -1339,7 +1049,7 @@ function renderDelegationResult(result) {
 		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`; you do not need to predict it up front."
+		"effort from its next step with `set_reasoning_effort`."
 	];
 	if (result.diagnostics.length > 0) lines.push("", "Diagnostics:", ...result.diagnostics.map((diagnostic) => `- ${diagnostic}`));
 	return lines.join("\n");
@@ -1349,7 +1059,7 @@ function createPstackSubagentTool(runtime, config = DEFAULT_CONFIG) {
 	return {
 		name: TOOL_NAME,
 		description: TOOL_DESCRIPTION,
-		parameters: buildParameters(config),
+		parameters: buildParameters(),
 		output: {
 			schema: OUTPUT_SCHEMA,
 			render: (_args, value) => [{
@@ -1360,8 +1070,8 @@ function createPstackSubagentTool(runtime, config = DEFAULT_CONFIG) {
 		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. This tool runs inside a conversation; it cannot start a child from a context that has no parent Agent.");
-			return await delegate(parent, parseDelegationArgs(rawArgs, config), runtime, config, exec.signal);
+			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);
 		}
 	};
 }
@@ -1370,23 +1080,18 @@ function hostRuntime(ctx, getAgentRegistry = () => void 0) {
 	return {
 		subagents: ctx.subagents,
 		llm: ctx.llm,
+		workingDirectory: ctx.workingDirectory,
 		get agents() {
 			return getAgentRegistry();
 		},
-		dshHome: resolveDshHome(),
-		assetsRoot: ASSETS_ROOT
+		dshHome: resolveDshHome()
 	};
 }
-/**
-* Register `pstack_subagent` for this row.
-*
-* @param ctx - context providing the tool registry, the subagent registry, the LLM
-*   adapters, and optionally `ctx.get` for the agent registry.
-* @param config - validated row config.
-*/
-function apply(ctx, config = DEFAULT_CONFIG) {
-	const tool = createPstackSubagentTool(hostRuntime(ctx, () => ctx.get?.("agents")), config);
-	ctx.tools.register(tool);
+/** 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 describeError(error) {
 	if (error instanceof Error) {
@@ -1396,7 +1101,4 @@ function describeError(error) {
 	}
 	return String(error);
 }
-//#endregion
-export { AGENT_KINDS, Config, DEFAULT_AGENT_KIND, DEFAULT_SUBAGENT_PROVIDER, DelegationError, OUTPUT_SCHEMA, PartialDelegationError, PreambleError, TOOL_NAME, agentOptionsFor, apply, assertProviderCapabilities, composeChildPrompt, createPstackSubagentTool, delegate, hostRuntime, inject, loadAgentPreamble, name, parseDelegationArgs, planDelegation, plannedRouteLabel, renderDelegationResult, resolveDelegationDepth, routeLabel, startPlannedChildren };
-
-//# sourceMappingURL=subagent.mjs.map
+export { AGENT_KINDS, AGENT_PRESETS, DEFAULT_SUBAGENT_PROVIDER, DelegationError, PartialDelegationError, TOOL_NAME, agentPresetFor, apply, assertProviderCapabilities, composeChildPrompt, createPstackSubagentTool, delegate, hostRuntime, inject, name, normalizeConfig, parseDelegationArgs, planDelegation, renderDelegationResult, resolveChildCwd, resolveDelegationDepth, routeLabel, startPlannedChildren };