@vincemakes/kiso-subagent-ext 0.39.2 → 0.40.0

package/dist/kiso-subagent.mjs

@@ -98,7 +98,10 @@ export default async function createSubagentExtension() {
 				execute: async (input, ctx) => {
 					const tasks = ((input ?? {}).tasks ?? []).slice(0, 8);
 					if (tasks.length === 0) return { content: "delegate: no tasks", isError: true, errorKind: "precondition" };
-					const sessionsDir = join(process.env.KISO_HOME ?? join(homedir(), ".kiso"), "sessions");
+					// 0.40.0: the parent's own folder (one per project), handed over
+					// by the CLI; a pinned KISO_SESSIONS_DIR, or the legacy folder,
+					// when no CLI said (a direct load)
+					const sessionsDir = delegationConfig().sessionsDir ?? (process.env.KISO_SESSIONS_DIR || join(process.env.KISO_HOME ?? join(homedir(), ".kiso"), "sessions"));
 					// P3: the loop now threads the session id through
 					// ToolContext.sessionId — the discovery heuristic below is
 					// kept ONLY as a fallback for direct tool use / tests.
@@ -149,14 +152,10 @@ export default async function createSubagentExtension() {
 					// Partial success is not overall failure — only ALL failed
 					// makes the whole result an error.
 					// W12: the blob opens with a machine-readable summary line —
-					// the ONE-LINE shape the TUI's settled row renders (└ N
-					// tool calls · R roles · F failed · /last for the report).
+					// the ONE-LINE shape the TUI's settled row renders verbatim.
 					// The per-section text below is unchanged — the model's
 					// view is preserved, the summary is additive.
-					const toolCalls = sections.reduce((n, s) => n + (s.toolCalls ?? 0), 0);
-					const roles = new Set(tasks.map((t) => t.role)).size;
-					const failed = sections.filter((s) => s.failed).length;
-					const summary = `summary: ${toolCalls} tool calls · ${roles} role${roles === 1 ? "" : "s"} · ${failed} failed`;
+					const summary = delegateSummary(sections, new Set(tasks.map((t) => t.role)).size);
 					return { content: `${summary}\n${sections.map((s) => s.text).join("\n")}`, isError: sections.every((s) => s.failed) };
 				},
 			},
@@ -253,7 +252,7 @@ async function runChild({ childId, role, task, scope, acceptance, model, after,
 		// the fixed UNRESOLVED instruction the result parser reads back.
 		const taskPath = join(manifestDir ?? policyDir, `${childId}.task`);
 		writeFileSync(taskPath, `${task}\n\n${UNRESOLVED_INSTRUCTION}\n`, "utf8");
-		const { code, stdout, killed } = await runProcess(childId, bin, childCwd, policyDir, taskPath, timeout, signal, model);
+		const { code, stdout, killed } = await runProcess(childId, bin, childCwd, policyDir, taskPath, timeout, signal, model, sessionsDir);
 		const extraction = await extractChildResult(sessionsDir, childId, `exit ${code}\n${stdout}`);
 		const status = killed === "timeout" ? "timeout" : killed === "abort" ? "killed" : code !== 0 && extraction.outcome === "missing" ? "spawn-failed" : extraction.outcome;
 		let failed = code !== 0 || killed !== null || extraction.failed;
@@ -347,13 +346,43 @@ async function runChild({ childId, role, task, scope, acceptance, model, after,
 				text += `\n  FAILED: collecting the worktree's changes: ${collection.reason}${collection.partialPath !== undefined ? ` (partial patch at ${collection.partialPath})` : ""}\n  worktree kept at: ${worktree}`;
 			}
 		}
-		return { failed, text, toolCalls: extraction.toolCalls };
+		return { failed, ...(failed ? { failKind: failKindOf(status, verification) } : {}), text, toolCalls: extraction.toolCalls };
 	} finally {
 		rmSync(policyDir, { recursive: true, force: true });
 		if (worktree !== null && ownsWorktree && !keepWorktree) removeWorktree(parentCwd, worktree);
 	}
 }
 
+/** 0.40.0 — why a child failed, in the three words a person acts on: its
+ *  clock ran out (`timeout`: raise it, or split the task), the parent's
+ *  acceptance check RAN and failed (`acceptance`: the work is wrong), or
+ *  anything else (`error`: a crash, an abort, a missing or ambiguous
+ *  result, a spawn or collection failure — read the report). */
+export function failKindOf(status, verification) {
+	if (status === "timeout") return "timeout";
+	if (verification !== null && verification !== undefined && verification.passed === false) return "acceptance";
+	return "error";
+}
+
+const FAIL_KIND_ORDER = ["timeout", "acceptance", "error"];
+
+/** 0.40.0 — the settled row's marker: how many tasks ran, and — when any
+ *  failed — how many of each kind, in a fixed order with zero counts
+ *  omitted. A failed section without a kind counts as an error: a failure
+ *  is never silently left out of the count. */
+export function delegateSummary(sections, roles) {
+	const n = sections.length;
+	const toolCalls = sections.reduce((sum, s) => sum + (s.toolCalls ?? 0), 0);
+	const failedSections = sections.filter((s) => s.failed);
+	const counts = new Map(FAIL_KIND_ORDER.map((k) => [k, 0]));
+	for (const s of failedSections) {
+		const k = FAIL_KIND_ORDER.includes(s.failKind) ? s.failKind : "error";
+		counts.set(k, counts.get(k) + 1);
+	}
+	const kinds = FAIL_KIND_ORDER.filter((k) => counts.get(k) > 0).map((k) => `${counts.get(k)} ${k}`);
+	return `summary: ${n} task${n === 1 ? "" : "s"} · ${toolCalls} tool calls · ${roles} role${roles === 1 ? "" : "s"} · ${failedSections.length} failed${kinds.length > 0 ? ` (${kinds.join(", ")})` : ""}`;
+}
+
 /** DT-1a: the fixed trailer every task file ends with — the parser reads the section back. */
 export const UNRESOLVED_INSTRUCTION = 'When you finish, end your reply with a section titled UNRESOLVED listing what you could not do or verify, one item per line starting with "- ", or the single word none.';
 
@@ -363,9 +392,9 @@ function delegationConfig() {
 	try {
 		const raw = process.env.KISO_DELEGATION_CONFIG_JSON;
 		const parsed = raw === undefined ? {} : JSON.parse(raw);
-		return { checks: parsed.checks ?? {}, profiles: parsed.profiles ?? [] };
+		return { checks: parsed.checks ?? {}, profiles: parsed.profiles ?? [], sessionsDir: typeof parsed.sessionsDir === "string" && parsed.sessionsDir !== "" ? parsed.sessionsDir : undefined };
 	} catch {
-		return { checks: {}, profiles: [] };
+		return { checks: {}, profiles: [], sessionsDir: undefined };
 	}
 }
 
@@ -590,7 +619,7 @@ export function childArgs(bin, childId, taskPath, model) {
 	return [bin, ...(model !== undefined ? ["--model", model] : []), "chat", childId, "--task-file", taskPath];
 }
 
-function runProcess(childId, bin, cwd, policyDir, taskPath, timeout, signal, model) {
+function runProcess(childId, bin, cwd, policyDir, taskPath, timeout, signal, model, sessionsDir) {
 	const depth = Number.parseInt(process.env.KISO_SUBAGENT_DEPTH ?? "0", 10) || 0;
 	const child = spawn(process.execPath, childArgs(bin, childId, taskPath, model), {
 		cwd,
@@ -598,6 +627,9 @@ function runProcess(childId, bin, cwd, policyDir, taskPath, timeout, signal, mod
 			...process.env,
 			KISO_SUBAGENT_DEPTH: String(depth + 1),
 			KISO_EXTENSIONS_DIR: policyDir,
+			// 0.40.0: the child writes its log beside its parent's — the
+			// parent reads it back from there — whatever cwd it runs in
+			KISO_SESSIONS_DIR: sessionsDir,
 			// Modes: a headless child has no human — the mode tiers'
 			// ask would stall it. Bypass is the neutral tier here; the
 			// role policy dir (allow/deny only — a child must never
@@ -848,7 +880,7 @@ function removeWorktree(parentCwd, worktree) {
 }
 
 function failSection(childId, role, task, reason) {
-	return { failed: true, text: `[subagent] ${role}: ${task}\n  FAILED: ${reason}`, toolCalls: 0 };
+	return { failed: true, failKind: "error", text: `[subagent] ${role}: ${task}\n  FAILED: ${reason}`, toolCalls: 0 };
 }
 
 const msg = (err) => (err instanceof Error ? err.message : String(err));

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vincemakes/kiso-subagent-ext",
-	"version": "0.39.2",
+	"version": "0.40.0",
 	"description": "kiso official subagent extension \u2014 child kiso processes with role policies, kernel untouched",
 	"type": "module",
 	"license": "MIT",
@@ -23,7 +23,7 @@
 		"test": "vitest run"
 	},
 	"devDependencies": {
-		"@vincemakes/kiso-core": "0.39.2",
+		"@vincemakes/kiso-core": "0.40.0",
 		"@types/node": "^26.1.2",
 		"typescript": "^5.7.2",
 		"vitest": "^3.0.0"