killeros 1.5.5 → 1.5.6

package/CHANGELOG.md

@@ -4,6 +4,14 @@ All notable changes to KillerOS are documented here.
 
 ## [Unreleased]
 
+## [1.5.6] - 2026-08-05
+
+### Fixed
+
+- Rejected unknown named subagent roles before child launch and returned the available roles instead of falling back to the write-capable `worker`.
+- Kept the real child-boundary test isolated from parent Pi session variables and added direct coverage for inline parallel and chain spawns.
+- Clarified and tested the shared `message` field limits: 20,000 characters for a spawn task alias and 4,000 characters for steering.
+
 ## [1.5.5] - 2026-08-05
 
 ### Added

package/README.md

@@ -116,7 +116,7 @@ The tool supports a single `agent` plus `task` or its `message` alias, parallel
 | `resume` | `threadId` | `task` |
 | `close` | `threadId` | none |
 
-The three spawn shapes cannot be mixed. On a single spawn, `message` aliases `task`; supplying both is invalid. With `action: "steer"`, `message` remains required, and other lifecycle actions reject it. An unknown role name falls back to `worker` on a new spawn and reports that choice in the tool output. The `wait` action defaults to all queued or active children, waits up to 30 seconds by default, and never stops a child when it times out. The `resume` action keeps the same thread ID, name, session ID, and session directory and increments `attempt`; it requires the original named role and rejects inline roles. KillerOS rejects malformed requests before role discovery, project confirmation, thread creation, or child launch. The TUI shows a parallel or shared-pool schedule only after shape validation; malformed calls show `invalid request` instead of queued work. For example:
+The three spawn shapes cannot be mixed. On a single spawn, `message` aliases `task` with the same 20,000-character limit; supplying both is invalid. With `action: "steer"`, `message` remains required and has a 4,000-character limit, while other lifecycle actions reject it. An unknown named role fails before child launch and reports the available roles. The `wait` action defaults to all queued or active children, waits up to 30 seconds by default, and never stops a child when it times out. The `resume` action keeps the same thread ID, name, session ID, and session directory and increments `attempt`; it requires the original named role and rejects inline roles. KillerOS rejects malformed requests before role discovery, project confirmation, thread creation, or child launch. The TUI shows a parallel or shared-pool schedule only after shape validation; malformed calls show `invalid request` instead of queued work. For example:
 
 ```json
 {"agent":"reviewer","task":"Review the change","name":"auth-audit","model":"provider/model","thinking":"high"}

package/killeros/subagents.ts

@@ -1166,7 +1166,7 @@ function createSubagentParams(limits: Pick<SubagentLimits, "maxTasks" | "maxRead
     action: Type.Optional(StringEnum(SUBAGENT_ACTIONS, { default: SUBAGENT_ACTION.spawn, description: "Spawn, list, inspect, steer, interrupt, collect, wait, resume, or close. Omit threadId when spawning" })),
     threadId: Type.Optional(threadId),
     name: Type.Optional(Type.String({ minLength: 1, maxLength: 48, pattern: THREAD_NAME_PATTERN, description: "Parent-scoped child display name" })),
-    message: Type.Optional(Type.String({ minLength: 1, maxLength: Math.max(4_000, limits.taskCharacters), description: "Task alias when spawning; steering message with action steer" })),
+    message: Type.Optional(Type.String({ minLength: 1, maxLength: Math.max(4_000, limits.taskCharacters), description: `Spawn task alias up to ${limits.taskCharacters.toLocaleString("en-US")} characters; steer message up to 4,000 characters` })),
     all: Type.Optional(Type.Literal(true, { description: "Target every active or queued child thread for interrupt or wait" })),
     timeoutMs: Type.Optional(Type.Integer({ minimum: 1, maximum: MAX_WAIT_TIMEOUT_MS, description: "Wait timeout in milliseconds; defaults to 30 seconds" })),
     agent: Type.Optional(agentSchema),
@@ -2292,26 +2292,18 @@ export function registerSubagentTool(pi: ExtensionAPI, options: SubagentRuntimeO
       const rawInputs: TaskInput[] = spawnRequest.kind === "spawn-single"
         ? [{ agent: spawnRequest.input.agent, task: spawnRequest.input.task, ...(spawnRequest.input.name ? { name: spawnRequest.input.name } : {}) }]
         : spawnRequest.kind === "spawn-parallel" ? spawnRequest.input.tasks : spawnRequest.input.chain;
-      const fallbackNotices: string[] = [];
-      const rolesForInputs = rawInputs.map((input) => {
+      const rolesForInputs = rawInputs.map((input) => {
         if (typeof input.agent !== "string") {
           const parentTools = new Set(pi.getActiveTools());
           for (const tool of input.agent.tools) {
             if (!parentTools.has(tool)) throw new Error(`Inline role ${JSON.stringify(input.agent.name)} tool ${JSON.stringify(tool)} is not active for the parent`);
           }
           return inlineAgentRole(input.agent);
-        }
+        }
         const selected = roles.get(input.agent);
         if (selected) return selected;
-        if (isResume) {
-          const available = discovery.agents.map((agent) => `${agent.name} (${agent.source})`).join(", ") || "none";
-          throw new Error(`Unknown subagent ${JSON.stringify(input.agent)}. Available: ${available}`);
-        }
-        const worker = roles.get("worker");
-        if (!worker) throw new Error(`Unknown subagent ${JSON.stringify(input.agent)} and fallback role "worker" is unavailable`);
-        const notice = `Unknown role ${JSON.stringify(input.agent)} - used worker`;
-        if (!fallbackNotices.includes(notice)) fallbackNotices.push(notice);
-        return worker;
+        const available = discovery.agents.map((agent) => `${agent.name} (${agent.source})`).join(", ") || "none";
+        throw new Error(`Unknown subagent ${JSON.stringify(input.agent)}. Available: ${available}`);
       });
       const inputs: Array<Omit<TaskInput, "agent"> & { agent: string }> = rawInputs.map((input, index) => ({
         ...input,
@@ -2351,7 +2343,7 @@ export function registerSubagentTool(pi: ExtensionAPI, options: SubagentRuntimeO
           ? `Parallel schedule: all tasks run through a shared pool of up to ${writerConcurrency}${writerConcurrencyOverride === undefined ? " (safe default)" : " (explicit)"}. Write-capable tasks are serialized in the shared parent worktree.`
           : `Parallel schedule: read-only tasks run concurrently up to ${limits.maxReadConcurrency}.`
         : undefined;
-      const executionNote = [...fallbackNotices, ...(scheduleNote ? [scheduleNote] : [])].join("\n") || undefined;
+      const executionNote = scheduleNote;
 
       const inFlight = threads.listAll().filter((thread) => ["queued", "active"].includes(thread.state)).length;
       if (inFlight + inputs.length > limits.maxTasks) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "killeros",
-  "version": "1.5.5",
+  "version": "1.5.6",
   "description": "A production-hardened TUI and workflow extension for the Pi coding agent.",
   "type": "module",
   "keywords": [