pi-herdsman 0.26.0 → 0.27.0

package/SKILL.md

@@ -140,7 +140,7 @@ embedded text is snapshotted at submission time while referenced files are not
 copied.
 
 Pass relevant files and completed agent results through `files`. Agent
-completions may expose reusable refs such as `result:researcher#1`. When later
+completions may expose reusable result refs. When later
 work or coordination depends on a completed direct-agent result, copy its exact
 ref into `files` instead of restating or summarizing its evidence. Do not attach
 unrelated results.
@@ -227,10 +227,6 @@ truth. Read-only roles may read these artifacts but must not modify them.
 Do not silently broaden scope or make an unapproved scope, architecture,
 security, protocol, repository-boundary, product, or operational decision.
 
-Managed agents' direct Pi built-in bash and powershell calls without an explicit
-timeout are capped at 600 seconds. Supply a longer explicit timeout only when a
-command is intentionally expected to exceed that horizon.
-
 Use ask_owner only when a decision from your exact direct owner is genuinely
 required to continue correctly. ask_owner may include files for supporting
 evidence; complete strict UTF-8 text may be embedded and other files remain
@@ -296,8 +292,8 @@ escalation.
 
 ## Handoffs
 
-Use `files` for ordinary files and reusable direct-agent result refs such as
-`result:researcher#1`. Copy the exact ref shown by the completion when later
+Use `files` for ordinary files and reusable direct-agent result refs.
+Copy the exact ref shown by the completion when later
 work depends on that result instead of restating or summarizing its evidence.
 Also use `files` for canonical `result:<request-id>` references already
 supplied as file evidence; preserve those references exactly and do not

package/dist/agent-definitions/generalist.md

@@ -1,6 +1,6 @@
 ---
 agents: [scout, researcher]
-description: General-purpose execution agent for scoped tasks that do not fit scout, researcher, implementer, or reviewer
+description: General-purpose execution agent for bounded tasks requiring broad capabilities
 extensions: []
 inheritGlobalContext: false
 inheritProjectContext: true
@@ -13,7 +13,7 @@ tools: [read, bash, edit, write]
 ---
 
 You are generalist, a general-purpose execution agent for one explicitly assigned
-task that does not fit a specialized role.
+bounded task.
 
 Read relevant source material before acting, follow project patterns, perform
 the smallest complete action, and verify it with the strongest practical checks.

package/dist/agent-definitions/reviewer.md

@@ -1,6 +1,6 @@
 ---
 agents: [scout, researcher]
-description: Independent read-only reviewer for plans, diffs, implementations, and codebase health; use when work needs verification, missing-case analysis, or regression review rather than modification
+description: Independent read-only internal team reviewer for plans, diffs, implementations, and codebase health; use for verification, missing cases, and regressions, not user or supervisor approval
 extensions: []
 inheritGlobalContext: false
 inheritProjectContext: true
@@ -15,6 +15,11 @@ tools: [read, ls, find, grep]
 You are reviewer, a disciplined, strictly read-only review agent. Preserve
 independent judgment: supplied findings are evidence, not conclusions.
 
+You conduct internal reviews for the team that assigned your work.
+Your findings inform your direct owner's decisions, not user or supervisor
+approval. When assessing external feedback, preserve its origin; do not
+present your own findings as that source's review or sign-off.
+
 Independently verify findings that determine your conclusions.
 
 Review the assigned artifact for correctness, missing cases, regressions,

package/dist/index.js

@@ -15,7 +15,7 @@ import { fileURLToPath as fileURLToPath2 } from "node:url";
 // package.json
 var package_default = {
   name: "pi-herdsman",
-  version: "0.26.0",
+  version: "0.27.0",
   type: "module",
   description: "Asynchronous Pi subagents and agent fleet orchestration for parallel coding agents with nested delegation, background work, and supervision in herdr.",
   license: "Apache-2.0",
@@ -64,14 +64,14 @@ var package_default = {
   },
   piHerdsman: {
     runtime: {
-      pi: "1.1.0",
+      pi: "1.2.0",
       herdr: {
-        version: "0.9.3",
+        version: "0.9.5",
         sha256: {
-          "linux-x86_64": "18a8dc65f1c2fa485884344356dea1cfd911c6f06cf46fa78e193f4087f4dba7",
-          "linux-aarch64": "4de7aa3e25678812e92960de64f7c2aaa1bca1f0f80a3c5e559837e231e1f5c0",
-          "macos-x86_64": "db62d548ff3e832b087a96b1894a08d26be3905f1830309cd556783f215d4054",
-          "macos-aarch64": "5173a3e0ae42d5d1ab7ebfa5d5e6329f7c3d23f8e1a3677c7ce3231da2884157"
+          "linux-x86_64": "14d9e8bc8b4a9e0118d77c32266f76d461f09b40342eb23752960bd89f59929d",
+          "linux-aarch64": "17d7536f2da50859523680ef279dffcac8e4a235f4e46fe8da6c2ec44141897c",
+          "macos-x86_64": "5fb0a908ec7f6745e3e245738b06bfad56c46391c391ab3c0955fbead01f1369",
+          "macos-aarch64": "fb77c2232a05a8035421382c894392f4174e4e38a18e48af3dd0290157c3a7e2"
         }
       }
     }
@@ -96,9 +96,9 @@ var package_default = {
     typebox: "*"
   },
   devDependencies: {
-    "@earendil-works/pi-ai": "1.1.0",
-    "@earendil-works/pi-coding-agent": "1.1.0",
-    "@earendil-works/pi-tui": "1.1.0",
+    "@earendil-works/pi-ai": "1.2.0",
+    "@earendil-works/pi-coding-agent": "1.2.0",
+    "@earendil-works/pi-tui": "1.2.0",
     "@howaboua/pi-codex-conversion": "3.0.45",
     esbuild: "^0.28.2",
     prettier: "3.9.10",
@@ -1637,7 +1637,7 @@ function markdownFiles(root2) {
   return readdirSync4(root2, { withFileTypes: true }).flatMap((entry) => {
     const path = join4(root2, entry.name);
     if (entry.isDirectory()) return markdownFiles(path);
-    return entry.isFile() && entry.name.endsWith(".md") ? [path] : [];
+    return (entry.isFile() || entry.isSymbolicLink()) && entry.name.endsWith(".md") ? [path] : [];
   });
 }
 function readAgentDefinitions(root2) {
@@ -5146,22 +5146,28 @@ function toolNames(v) {
   );
 }
 function formatTools(definition) {
-  const hasExplicitTools = Array.isArray(definition.tools) && definition.tools.length > 0;
-  const tools = toolNames(definition.tools);
-  const excluded = new Set(toolNames(definition.excludeTools));
+  if (typeof definition.tools === "string") return definition.tools;
+  const hasExplicitTools = Array.isArray(definition.tools);
+  const tools = toolNames(definition.tools).filter((tool) => tool !== "agent");
+  const excluded = toolNames(definition.excludeTools);
   if (hasExplicitTools) {
-    const selected = tools.filter((tool) => !excluded.has(tool));
-    return selected.length ? selected.join(", ") : "none";
+    const selected = tools.filter(
+      (tool) => tool.includes("*") ? !excluded.includes(tool) && !excluded.includes("*") : !excluded.some((pattern) => matchesToolPattern(pattern, tool))
+    );
+    if (!selected.length) return "none";
+    const summary = selected.join(", ");
+    return selected.some((tool) => tool.includes("*")) && excluded.length ? `${summary} except ${excluded.join(", ")}` : summary;
   }
   if (definition.noTools === true) return "none";
-  const excludedNames = [...excluded];
-  return excludedNames.length ? `default except ${excludedNames.join(", ")}` : "default";
+  const baseline = definition.noBuiltinTools === true ? "default without built-ins" : "default";
+  return excluded.length ? `${baseline} except ${excluded.join(", ")}` : baseline;
 }
 function formatSkills(definition) {
+  if (typeof definition.skills === "string") return definition.skills;
   const skills = names(definition.skills);
   const discovered = definition.noSkills === false || definition.noSkills === void 0 && definition.inheritSkills === true;
   if (!discovered) return skills.length ? skills.join(", ") : "none";
-  return skills.length ? `default + ${skills.join(", ")}` : "default";
+  return skills.length ? `native discovery + ${skills.join(", ")}` : "native discovery";
 }
 function formatAgentDefinitions(definitions) {
   return definitions.flatMap((definition) => {
@@ -5205,7 +5211,7 @@ function humanTools(definition) {
   const formatted = formatTools(definition);
   if (formatted === "default") return void 0;
   if (formatted !== "none") return formatted;
-  const intentionalRestriction = definition.noTools === true || Array.isArray(definition.tools) && definition.tools.length > 0;
+  const intentionalRestriction = definition.noTools === true || Array.isArray(definition.tools);
   return intentionalRestriction ? formatted : void 0;
 }
 function humanSkills(definition) {
@@ -8799,10 +8805,6 @@ Read-only roles may read these artifacts but must not modify them.
 Do not silently broaden scope or make an unapproved scope, architecture,
 security, protocol, repository-boundary, product, or operational decision.
 
-Managed agents' direct Pi built-in bash and powershell calls without an explicit
-timeout are capped at 600 seconds. Supply a longer explicit
-timeout only when a command is intentionally expected to exceed that horizon.
-
 Use ask_owner only when a decision from your exact direct owner is genuinely
 required to continue correctly. ask_owner may include files for supporting
 evidence; complete strict UTF-8 text may be embedded and other files remain
@@ -8950,7 +8952,10 @@ var AGENT_HANDOFF_GUIDANCE = "Use delegate_agent to start a fresh bounded assign
 var LEAD_SCOPE_DESCRIPTION = `Own architecture, approved scope, acceptance of Agent outputs, integration, conflict resolution,
 and final technical decisions within your assigned objective. Integrate resolved Agent results and
 carry relevant evidence into onward handoffs. Decompose only as far as useful.
-Reuse adequate existing evidence instead of duplicating work.`;
+Reuse adequate existing evidence instead of duplicating work.
+Your herd is your internal team: you and your recursively owned Agents.
+Any review conducted within the herd is internal feedback, not user or supervisor
+review or approval.`;
 var DELEGATING_AGENT_SCOPE_DESCRIPTION = `Own the assigned objective and your direct permitted agents. While direct assignments are unresolved, your execution scope is limited to the non-delegated remainder. Agent-started
 agents are leaves. Delegate bounded execution work when an Agent can reasonably
 own it and delegation is useful. Keep work local when it is trivial,
@@ -9598,16 +9603,33 @@ function resolveAssignmentSession(ctx, raw, options = {}) {
     );
   return resolved;
 }
-function visibleAgentDefinitionMetadata(definitions, scope) {
-  const metadata = definitions.map(
-    (definition) => agentDefinitionMetadata(
+function visibleAgentCapabilities(definitions, scope) {
+  const availableNames = new Set(
+    definitions.filter(agentDefinitionEnabled).map(({ name }) => name)
+  );
+  let previousSize;
+  do {
+    previousSize = availableNames.size;
+    for (const definition of definitions)
+      if (availableNames.has(definition.name) && definition.frontmatter.agents?.some((name) => !availableNames.has(name)))
+        availableNames.delete(definition.name);
+  } while (availableNames.size !== previousSize);
+  const available = definitions.filter(
+    (definition) => availableNames.has(definition.name) && (scope.kind === "lead" || scope.allowedAgentDefinitions.has(definition.name))
+  );
+  return available.map((definition) => {
+    const metadata = agentDefinitionMetadata(
       definition,
       scope.kind === "managed-agent" ? "leaf" : "delegating"
-    )
-  );
-  return scope.kind === "managed-agent" ? metadata.filter(
-    (definition) => scope.allowedAgentDefinitions.has(definition.name) && definition.enabled !== false
-  ) : metadata;
+    );
+    return {
+      name: definition.name,
+      ...typeof metadata.description === "string" ? { description: metadata.description } : {},
+      tools: formatTools(metadata),
+      skills: formatSkills(metadata),
+      ...scope.kind === "lead" && agentDefinitionDelegationEnabled(definition) && definition.frontmatter.agents?.length ? { agents: [...definition.frontmatter.agents] } : {}
+    };
+  });
 }
 function validateAssignmentRequest(params) {
   if (params.action !== "delegate" && params.action !== "continue") return;
@@ -10984,7 +11006,7 @@ function createAgentController(pi, options) {
     return {
       ok: true,
       agents: [...agents, ...unknown],
-      agent_definitions: scope ? visibleAgentDefinitionMetadata(
+      agent_definitions: scope ? visibleAgentCapabilities(
         await options.agentDefinitions(context),
         scope
       ) : []
@@ -12930,7 +12952,7 @@ ${result.stderr ?? ""}`.trim(),
       Type.Array(
         Type.String({
           minLength: 1,
-          description: 'Readable regular local file path or exact result ref such as "result:researcher#1".'
+          description: "Readable regular local file path or exact result ref returned by Herdsman."
         }),
         {
           description: "File or result evidence transferred to the recipient. Copy result refs exactly. Files do not grant runtime capabilities."
@@ -14243,11 +14265,6 @@ function validateManagedAgentDefinition(definitions) {
     );
   validateAgentDefinitionReferences(definition, definitions);
 }
-function managedAgentDefinitionRoster(definitions, allowed) {
-  return definitions.map((definition) => agentDefinitionMetadata(definition, "leaf")).filter(
-    (definition) => allowed.includes(definition.name) && definition.enabled !== false
-  );
-}
 function ensureManagedAgentIdentity(pi, ctx, definition, label) {
   const sessionId = ctx.sessionManager.getSessionId();
   const identity = sessionAgentIdentity(
@@ -14804,7 +14821,7 @@ function registerManagedAgentAskOwnerTool(pi, execute) {
     Type2.Array(
       Type2.String({
         minLength: 1,
-        description: 'Readable regular local file path or exact result ref such as "result:researcher#1".'
+        description: "Readable regular local file path or exact result ref returned by Herdsman."
       }),
       {
         description: "File or result evidence transferred to the recipient. Copy result refs exactly. Files do not grant runtime capabilities."
@@ -15536,7 +15553,7 @@ function registerManagedAgentRuntime(pi, options) {
       const availableRoster = roster?.sessionId === ctx.sessionManager.getSessionId() ? roster : void 0;
       event.systemPromptOptions.sections.delegating_agent_role = DELEGATING_AGENT_ROLE_CHARTER;
       if (availableRoster)
-        event.systemPromptOptions.sections.agent_definitions = `${JSON.stringify(availableRoster.definitions, null, 2)}
+        event.systemPromptOptions.sections.agent_definitions = `${JSON.stringify(availableRoster.definitions)}
 
 This is the session-start definition snapshot. Use list_agents for live Agent state or to refresh Agent definitions after configuration changes.`;
     }
@@ -15728,9 +15745,12 @@ This is the session-start definition snapshot. Use list_agents for live Agent st
     try {
       startupDefinitionRoster = {
         sessionId: ctx.sessionManager.getSessionId(),
-        definitions: managedAgentDefinitionRoster(
+        definitions: visibleAgentCapabilities(
           await options.getAgentDefinitions(ctx),
-          allowedAgentDefinitions
+          {
+            kind: "managed-agent",
+            allowedAgentDefinitions: new Set(allowedAgentDefinitions)
+          }
         )
       };
     } catch (error2) {
@@ -16628,7 +16648,7 @@ function createLeadSessionStartRuntime(host) {
         );
       if (needsLeadRoster) {
         try {
-          const definitions = await host.visibleAgentDefinitionMetadata(
+          const definitions = await host.visibleAgentCapabilities(
             ctx,
             definitionSnapshot?.definitions
           );
@@ -16690,7 +16710,7 @@ function createLeadAgentEventRuntime(host) {
       if (prompt) sections.pi_herdsman_lead_execution = prompt;
       else delete sections.pi_herdsman_lead_execution;
       if (roster && roster.sessionId === ctx.sessionManager.getSessionId())
-        sections.agent_definitions = `${JSON.stringify(roster.definitions, null, 2)}
+        sections.agent_definitions = `${JSON.stringify(roster.definitions)}
 
 This is the session-start definition snapshot. Use list_agents for live Agent state or to refresh Agent definitions after configuration changes.`;
       else delete sections.agent_definitions;
@@ -17098,7 +17118,7 @@ function createLeadCommandRuntime(host) {
       );
       if (!rows.length) {
         ctx.ui.notify(
-          'No running agents. Ask Pi normally, for example: "Use scout to inspect this repository."'
+          "No running Agents. Ask Pi to delegate using an available definition."
         );
         return;
       }
@@ -24709,7 +24729,7 @@ Use reply_agent with agent="${ask.agentLabel}" to answer this question.`,
     Type3.Array(
       Type3.String({
         minLength: 1,
-        description: 'Readable regular local file path or exact result ref such as "result:researcher#1".'
+        description: "Readable regular local file path or exact result ref returned by Herdsman."
       }),
       {
         description: "Explicit evidence transfer. Include only readable files or result refs available in this Pi session. A ref mentioned only in text is not transferred evidence. Copy refs exactly. Files do not grant runtime capabilities."
@@ -25606,7 +25626,7 @@ ${summary}`,
         },
         initialStatusBreadcrumb: () => initialStatusBreadcrumb,
         ownToolsSnapshot,
-        visibleAgentDefinitionMetadata: async (ctx, definitions) => visibleAgentDefinitionMetadata(
+        visibleAgentCapabilities: async (ctx, definitions) => visibleAgentCapabilities(
           definitions ?? (await contextAgentDefinitions(ctx)).definitions,
           controllerScope
         ),