dsh-pstack 0.2.0 → 0.2.1

package/dist/role-manifest-79cnFfDM.mjs

@@ -0,0 +1,112 @@
+// src/internal/role-manifest.ts
+/**
+* File name the role configuration lives under inside `$DSH_HOME`.
+*
+* 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.
+*/
+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
+	}
+];
+function isRoleKey(value) {
+	return ROLE_MANIFEST.some((role) => role.key === value);
+}
+/** Normalize formatting while keeping commas that belong to a role name. */
+function normalizeRoleName(raw) {
+	return raw.normalize("NFC").trim().toLowerCase().replace(/\s+/g, " ").replace(/\s*,\s*/g, ", ").replace(/:+$/, "").trim();
+}
+/** Exact manifest lookup. Callers normalize the key first. */
+function findRoleSpec(key) {
+	if (!isRoleKey(key)) return void 0;
+	return ROLE_MANIFEST.find((role) => role.key === key);
+}
+export { isRoleKey as a, findRoleSpec as i, PSTACK_MODELS_FILENAME as n, normalizeRoleName as o, ROLE_MANIFEST as r, INHERIT_PARENT_ALIASES as t };

package/dist/role-manifest-8OERX9-s.d.mts

@@ -0,0 +1,90 @@
+/**
+ * The one pstack role manifest.
+ *
+ * Keys are matched exactly after normalization, and commas belong to some keys
+ * (`feature, refactoring`, `reflect judgment, divergent, synthesizer`), so nothing here
+ * may split a key on a comma. A second literal role list anywhere is a bug.
+ *
+ * This module has no Node imports: the bundle imports it directly.
+ */
+/** Whether a role holds one route or a comma-separated list of routes. */
+type RoleValueShape = "single" | "list";
+interface RoleSpec {
+  /** Exact key as written in the role file. Commas belong to the key. */
+  readonly key: string;
+  readonly shape: RoleValueShape;
+  /** A list role that starts one child per entry automatically. */
+  readonly autoPanel: boolean;
+}
+/** The 17 roles, in manifest order. */
+declare const ROLE_MANIFEST: readonly [{
+  readonly key: "feature, refactoring";
+  readonly shape: "single";
+  readonly autoPanel: false;
+}, {
+  readonly key: "bug-fix";
+  readonly shape: "single";
+  readonly autoPanel: false;
+}, {
+  readonly key: "perf-issue";
+  readonly shape: "single";
+  readonly autoPanel: false;
+}, {
+  readonly key: "hillclimb";
+  readonly shape: "single";
+  readonly autoPanel: false;
+}, {
+  readonly key: "judgment and prose";
+  readonly shape: "single";
+  readonly autoPanel: false;
+}, {
+  readonly key: "hardest tasks";
+  readonly shape: "single";
+  readonly autoPanel: false;
+}, {
+  readonly key: "how explorer";
+  readonly shape: "single";
+  readonly autoPanel: false;
+}, {
+  readonly key: "how explainer";
+  readonly shape: "single";
+  readonly autoPanel: false;
+}, {
+  readonly key: "why investigators";
+  readonly shape: "single";
+  readonly autoPanel: false;
+}, {
+  readonly key: "why synthesizer";
+  readonly shape: "single";
+  readonly autoPanel: false;
+}, {
+  readonly key: "reflect tooling";
+  readonly shape: "single";
+  readonly autoPanel: false;
+}, {
+  readonly key: "reflect judgment, divergent, synthesizer";
+  readonly shape: "single";
+  readonly autoPanel: false;
+}, {
+  readonly key: "arena runners";
+  readonly shape: "list";
+  readonly autoPanel: true;
+}, {
+  readonly key: "arena cross-judge pool";
+  readonly shape: "list";
+  readonly autoPanel: false;
+}, {
+  readonly key: "swarm workers";
+  readonly shape: "list";
+  readonly autoPanel: false;
+}, {
+  readonly key: "architect runners";
+  readonly shape: "list";
+  readonly autoPanel: true;
+}, {
+  readonly key: "interrogate reviewers";
+  readonly shape: "list";
+  readonly autoPanel: true;
+}];
+type RoleKey = (typeof ROLE_MANIFEST)[number]["key"];
+export { RoleSpec as n, RoleValueShape as r, RoleKey as t };

package/dist/{role-wire-DZwTn2wF.d.mts → role-wire-CAJsVA6e.d.mts}

@@ -1,5 +1,4 @@
-import "./models.mjs";
-import { n as RoleValueShape } from "./role-manifest-C3fUzjLI.mjs";
+import { r as RoleValueShape, t as RoleKey } from "./role-manifest-8OERX9-s.mjs";
 /** Which spelling of "inherit the parent route" a role line uses. */
 type InheritAlias = "inherit-parent" | "auto";
 /** One role value as the configuration page sees it. */
@@ -14,7 +13,7 @@ type RouteView = {
 };
 /** One of the 17 roles in manifest order. */
 interface RoleView {
-  readonly key: string;
+  readonly key: RoleKey;
   readonly shape: RoleValueShape;
   /** Whether the role file configures this role. */
   readonly configured: boolean;

package/dist/roles.d.mts

@@ -1,5 +1,5 @@
 import { CatalogueResult } from "./models.mjs";
-import { n as SaveRoleRequest, r as SaveRoleResult, t as RoleFileSnapshot } from "./role-wire-DZwTn2wF.mjs";
+import { n as SaveRoleRequest, r as SaveRoleResult, t as RoleFileSnapshot } from "./role-wire-CAJsVA6e.mjs";
 import { Context, Service } from "@deepseek-ai/cordis";
 import { TypertRemoteService } from "@deepseek-ai/dsh-typert-protocol";
 /** Read and write the role file for the Plugins configuration page. */

package/dist/roles.mjs

@@ -1,6 +1,7 @@
 import { collectCatalogue } from "./models.mjs";
+import { i as findRoleSpec, o as normalizeRoleName, r as ROLE_MANIFEST, t as INHERIT_PARENT_ALIASES } from "./role-manifest-79cnFfDM.mjs";
 import { REMOTE_PACKAGE, ROLES_SERVICE, pstackRolesRemote } from "./remote.mjs";
-import { a as resolveDshHome, c as ROLE_MANIFEST, l as findRoleSpec, n as normalizeRoleName, r as readRoleConfiguration, s as INHERIT_PARENT_ALIASES, t as modelsFilePath } from "./role-config-NNn5r1JX.mjs";
+import { i as resolveDshHome, n as readRoleConfiguration, o as updateRoleDocument, t as modelsFilePath } from "./role-config-a4JpVdc-.mjs";
 import { dirname } from "node:path";
 import { mkdirSync, readFileSync, renameSync, rmSync, writeFileSync } from "node:fs";
 import { Service } from "@deepseek-ai/cordis";
@@ -178,40 +179,9 @@ function validateSaveRequest(request, catalogue) {
 function renderRoleValue(routes) {
 	return routes.map((route) => route.kind === "inherit" ? route.alias ?? "inherit-parent" : route.effort === void 0 ? `${route.provider}/${route.model}` : `${route.provider}/${route.model} [${route.effort}]`).join(", ");
 }
-function bodyStart(lines) {
-	if (lines[0]?.trim() !== "---") return 0;
-	for (let index = 1; index < lines.length; index += 1) if (lines[index]?.trim() === "---") return index + 1;
-	return 0;
-}
-/**
-* Replace the winning line for one role, or append one when absent.
-*
-* Every other line is returned verbatim. A role configured twice keeps its
-* earlier line and replaces the last one, which is the line the parser treats
-* as winning; the duplicate remains visible as a per-line issue.
-*/
+/** Update the winning role through the same line grammar used by the parser. */
 function updateRoleText(text, spec, value) {
-	const lines = text.split("\n");
-	const crlf = text.includes("\r\n");
-	if (lines.length > 0 && lines[lines.length - 1] === "") lines.pop();
-	let match = -1;
-	for (let index = bodyStart(lines); index < lines.length; index += 1) {
-		const line = lines[index];
-		const trimmed = line.endsWith("\r") ? line.slice(0, -1).trim() : line.trim();
-		if (trimmed === "" || trimmed.startsWith("#")) continue;
-		const separator = trimmed.indexOf(":");
-		if (separator === -1) continue;
-		if (normalizeRoleName(trimmed.slice(0, separator)) !== spec.key) continue;
-		match = index;
-	}
-	const rendered = `${spec.key}: ${value}`;
-	if (match >= 0) {
-		const original = lines[match];
-		const indent = /^\s*/.exec(original)?.[0] ?? "";
-		const ending = original.endsWith("\r") ? "\r" : "";
-		lines[match] = `${indent}${rendered}${ending}`;
-	} else lines.push(`${rendered}${crlf ? "\r" : ""}`);
-	return `${lines.join("\n")}\n`;
+	return updateRoleDocument(text, spec, value);
 }
 function writeAtomically(path, text) {
 	mkdirSync(dirname(path), { recursive: true });

package/dist/script.mjs

@@ -1,4 +1,4 @@
-import { i as PORT_ROOT } from "./paths-BT77khwm.mjs";
+import { i as PORT_ROOT } from "./paths-yji9SNak.mjs";
 import { isAbsolute, join, resolve } from "node:path";
 import { spawn } from "node:child_process";
 // src/internal/script-registry.ts

package/dist/subagent.d.mts

@@ -1,5 +1,5 @@
 import { _ as ToolsHost, g as ToolDefinition, h as SubagentsHost, m as SubagentCapabilities, n as AgentOptions, r as AgentsHost, s as LlmHost, t as Agent, v as WorkingDirectoryHost } from "./host-contract-CUPSpeNk.mjs";
-import { t as RoleSpec } from "./role-manifest-C3fUzjLI.mjs";
+import { n as RoleSpec } from "./role-manifest-8OERX9-s.mjs";
 /** One resolved child route. */
 type Route = {
   readonly kind: "inherit";
@@ -43,6 +43,13 @@ export interface PstackSubagentConfig {
   readonly provider: string;
   readonly maxDepth?: number;
 }
+/**
+ * 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.
+ */
+export declare const DEFAULT_CHILD_MAX_DEPTH = 1;
 /** What a call asked for, after every required argument has been validated. */
 export interface DelegationArgs {
   readonly description: string;
@@ -50,7 +57,7 @@ export interface DelegationArgs {
   readonly agent: AgentKind;
   readonly role: string | null;
   readonly cwd: string | null;
-  readonly maxDepth: number;
+  readonly maxDepth: number | null;
   readonly roleIndex: number | null;
   readonly reasoningEffort: string | null;
 }
@@ -107,6 +114,13 @@ export declare function composeChildPrompt(input: {
   }[];
   readonly task: string;
 }): string;
+export declare const DELEGATION_ARGUMENT_KEYS: readonly ["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.
+ */
+export declare const REQUIRED_DELEGATION_ARGUMENT_KEYS: readonly ["description", "prompt", "agent", "role", "roleIndex", "reasoningEffort", "cwd"];
 /** Validate every required model-facing argument. Null declines an optional setting. */
 export declare function parseDelegationArgs(raw: unknown): DelegationArgs;
 /** One planned child, before any Host call. */
@@ -140,7 +154,15 @@ export interface DelegationPlan {
 /** Route text as reported to the model. */
 export declare function routeLabel(route: Route, effortOverride?: string): string;
 /** Take the smallest of the requested depth, this row's cap, and the Host setting. */
-export declare function resolveDelegationDepth(configured: number | undefined, requested: number | undefined, hostLimit: number | undefined): number | undefined;
+export declare function resolveDelegationDepth(configured: number | undefined, requested: number | null | undefined, hostLimit: number | undefined): number | undefined;
+/**
+ * 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.
+ */
+export declare function requestedChildDepth(requested: number | null, diagnostics: string[]): number;
 /** Resolve the role file, routes, prompt, and depth into one plan. Pure of service calls. */
 export declare function planDelegation(args: DelegationArgs, runtime: SubagentRuntime, config?: PstackSubagentConfig): DelegationPlan;
 /** Check the backend can enforce what this call asks for. */

package/dist/subagent.mjs

@@ -1,5 +1,5 @@
-import { a as POTETO_MODE_SKILL_PATH } from "./paths-BT77khwm.mjs";
-import { a as resolveDshHome, i as requireRole, o as selectRoleRoutes, r as readRoleConfiguration, t as modelsFilePath } from "./role-config-NNn5r1JX.mjs";
+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 { isAbsolute, join, relative, resolve } from "node:path";
 import { existsSync, statSync } from "node:fs";
 // src/internal/effort.ts
@@ -149,6 +149,13 @@ const AGENT_KINDS = [
 	"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) {
@@ -214,22 +221,36 @@ function composeChildPrompt(input) {
 	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. */
 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 = [
-		"description",
-		"prompt",
-		"agent",
-		"role",
-		"roleIndex",
-		"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(", ")}`);
+	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 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"];
@@ -240,8 +261,9 @@ function parseDelegationArgs(raw) {
 	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 maxDepth = record["maxDepth"];
-	if (!isNonNegativeSafeInteger(maxDepth)) throw new DelegationError("maxDepth must be a non-negative safe integer");
+	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");
@@ -267,15 +289,30 @@ function routeLabel(route, effortOverride) {
 }
 /** 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.`);
+	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);
+	].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. */
 function planDelegation(args, runtime, config = DEFAULT_CONFIG) {
 	const rolePath = modelsFilePath(runtime.dshHome);
@@ -305,7 +342,7 @@ function planDelegation(args, runtime, config = DEFAULT_CONFIG) {
 		task: args.prompt
 	});
 	const agentPreset = agentPresetFor(args.agent);
-	const depth = resolveDelegationDepth(config.maxDepth, args.maxDepth, void 0);
+	const depth = resolveDelegationDepth(config.maxDepth, requestedChildDepth(args.maxDepth, diagnostics), void 0);
 	const total = routes.length;
 	const children = routes.map((route, index) => {
 		const agentOptions = agentOptionsFor(route, args.reasoningEffort ?? void 0);
@@ -495,9 +532,11 @@ 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.",
+	"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.",
@@ -546,28 +585,19 @@ function buildParameters() {
 				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."
+				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."
+				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: [
-			"description",
-			"prompt",
-			"agent",
-			"role",
-			"roleIndex",
-			"reasoningEffort",
-			"cwd",
-			"maxDepth"
-		]
+		required: [...REQUIRED_DELEGATION_ARGUMENT_KEYS]
 	};
 }
 const OUTPUT_SCHEMA = {
@@ -694,4 +724,4 @@ function describeError(error) {
 	}
 	return String(error);
 }
-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 };
+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 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-pstack",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "pstack for DeepSeek Harness: rigorous agent workflows ported from cursor/plugins.",
   "keywords": [
     "agent-skills",
@@ -38,6 +38,7 @@
     ".": "./dist/index.mjs",
     "./models": "./dist/models.mjs",
     "./remote": "./dist/remote.mjs",
+    "./role-catalog": "./dist/role-catalog.mjs",
     "./roles": "./dist/roles.mjs",
     "./script": "./dist/script.mjs",
     "./skill": "./dist/skill.mjs",
@@ -130,7 +131,7 @@
     }
   ],
   "scripts": {
-    "prerelease": "vp run -r check && vp run -r test",
+    "prerelease": "vp run build && vp run -r check && vp run typecheck && vp run -r test",
     "release": "bumpp"
   }
 }

package/dist/role-manifest-C3fUzjLI.d.mts

@@ -1,19 +0,0 @@
-/**
- * The one pstack role manifest.
- *
- * Keys are matched exactly after normalization, and commas belong to some keys
- * (`feature, refactoring`, `reflect judgment, divergent, synthesizer`), so nothing here
- * may split a key on a comma. A second literal role list anywhere is a bug.
- *
- * This module has no Node imports: the bundle imports it directly.
- */
-/** Whether a role holds one route or a comma-separated list of routes. */
-type RoleValueShape = "single" | "list";
-interface RoleSpec {
-  /** Exact key as written in the role file. Commas belong to the key. */
-  readonly key: string;
-  readonly shape: RoleValueShape;
-  /** A list role that starts one child per entry automatically. */
-  readonly autoPanel: boolean;
-}
-export { RoleValueShape as n, RoleSpec as t };