pi-cohort 5.1.1 → 5.1.3

package/CHANGELOG.md

@@ -1,5 +1,29 @@
 # Changelog
 
+## [5.1.3] - 2026-08-18
+
+### Fixed
+
+- Foreground control notices are steered to a turn boundary while the session
+  is streaming instead of splicing into an open tool cycle (which corrupted
+  the session with a duplicated `tool_use_id`); idle delivery still appends
+  without triggering a turn. (#7)
+
+### Changed
+
+- `dir` no longer advertised by the subagent tool schema (follows `runId`,
+  #5) - still accepted at runtime for async status/resume, `id` preferred.
+
+## [5.1.2] - 2026-08-07
+
+### Changed
+
+- Schema diet round three (subagent tool schema 9,483 -> 8,246 bytes serialized):
+  `tasks[]` items collapsed to the round-two stub form (overrides pass through,
+  documented in the pi-cohort skill), `ChainItem` allOf conditionals removed
+  (runtime-enforced at chain parse), and `runId` no longer advertised by the
+  tool schema - still accepted at runtime, `id` preferred.
+
 ## [5.1.1] - 2026-08-07
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-cohort",
-  "version": "5.1.1",
+  "version": "5.1.3",
   "description": "Delegate Pi work to focused child agents: code review, scouting, implementation, parallel audits, saved chains, and background jobs.",
   "author": "Jacek Juraszek",
   "license": "MIT",
@@ -36,8 +36,8 @@
   "scripts": {
     "check:agents-core": "node scripts/check-agents-core.mjs",
     "test": "npm run test:unit",
-    "test:unit": "node --experimental-strip-types --test test/unit/*.test.ts",
-    "test:integration": "node --experimental-transform-types --import ./test/support/register-loader.mjs --test test/integration/*.test.ts",
+    "test:unit": "node --test test/unit/*.test.ts",
+    "test:integration": "node --import ./test/support/register-loader.mjs --test test/integration/*.test.ts",
     "test:all": "npm run check:agents-core && npm run test:unit && npm run test:integration"
   },
   "pi": {

package/src/extension/control-notices.ts

@@ -39,6 +39,7 @@ function deliverControlNotice(input: {
 	pi: Pick<ExtensionAPI, "sendMessage">;
 	visibleControlNotices: Set<string>;
 	details: SubagentControlMessageDetails;
+	isIdle: () => boolean;
 }): void {
 	const childIntercomTarget = controlNoticeTarget(input.details);
 	const key = controlNotificationKey(input.details.event, childIntercomTarget);
@@ -52,7 +53,7 @@ function deliverControlNotice(input: {
 			display: true,
 			details: { ...input.details, childIntercomTarget, noticeText },
 		},
-		{ triggerTurn: input.details.source !== "foreground" },
+		{ triggerTurn: input.details.source !== "foreground" || !input.isIdle() },
 	);
 }
 
@@ -72,8 +73,9 @@ export function handleSubagentControlNotice(input: {
 	foregroundDelayMs?: number;
 }): void {
 	if (!input.details?.event || input.details.event.type === "active_long_running") return;
+	const isIdle = () => input.state.lastUiContext?.isIdle() ?? false;
 	if (input.details.source !== "foreground") {
-		deliverControlNotice(input);
+		deliverControlNotice({ ...input, isIdle });
 		return;
 	}
 
@@ -85,7 +87,7 @@ export function handleSubagentControlNotice(input: {
 	const timer = setTimeout(() => {
 		pending.delete(timerKey);
 		if (!isForegroundNoticeStillActionable(input.state, input.details)) return;
-		deliverControlNotice(input);
+		deliverControlNotice({ ...input, isIdle });
 	}, input.foregroundDelayMs ?? 1000);
 	timer.unref?.();
 	pending.set(timerKey, timer);

package/src/extension/index.ts

@@ -183,10 +183,13 @@ function parseSubagentNotifyContent(content: string): SubagentNotifyDetails | un
 }
 
 class SubagentControlNoticeComponent implements Component {
-	constructor(
-		private readonly details: SubagentControlMessageDetails,
-		private readonly theme: ExtensionContext["ui"]["theme"],
-	) {}
+	private readonly details: SubagentControlMessageDetails;
+	private readonly theme: ExtensionContext["ui"]["theme"];
+
+	constructor(details: SubagentControlMessageDetails, theme: ExtensionContext["ui"]["theme"]) {
+		this.details = details;
+		this.theme = theme;
+	}
 
 	invalidate(): void {}
 
@@ -441,7 +444,7 @@ Example: { chain: [{agent:"agent-a", task:"Analyze {task}"}, {agent:"agent-b", t
 
 MANAGEMENT: action=list/get/create/update/delete (chainName for chains; packaged agents use dotted names); config fields: pi-cohort skill reference/config-fields.md.
 
-CONTROL: action=status/interrupt/resume (id/runId, message, index); details: pi-cohort skill.
+CONTROL: action=status/interrupt/resume (id, message, index); details: pi-cohort skill.
 
 DIAGNOSTICS:
 • { action: "doctor" } - read-only report for runtime paths, discovery, sessions, and intercom`,

package/src/extension/schemas.ts

@@ -137,18 +137,22 @@ const AcceptanceOverrideStub = Type.Unsafe({
 	description: "Acceptance override; same shape as top-level acceptance.",
 });
 
-const TaskItem = Type.Object({
-	agent: Type.String(), 
-	task: Type.String(), 
-	cwd: Type.Optional(Type.String()),
-	count: Type.Optional(Type.Integer({ minimum: 1, description: "Repeat N times (same settings)." })),
-	output: Type.Optional(brief(OutputOverride, "Output file path, or false.")),
-	outputMode: Type.Optional(brief(OutputModeOverride, "Default: inline.")),
-	reads: Type.Optional(ReadsOverride),
-	progress: Type.Optional(Type.Boolean({ description: "true enables progress.md tracking; omit or false disables." })),
-	model: Type.Optional(Type.String()),
-	skill: Type.Optional(brief(SkillOverride, "Skill override.")),
-	acceptance: Type.Optional(AcceptanceOverrideStub),
+// tasks[] stub (round three): same treatment as ParallelTaskSchema below; structural keys
+// stay explicit (agent/task required, count = arity); every per-task override (model, output,
+// outputMode, reads, progress, skill, cwd, acceptance) passes through additionalProperties
+// and is documented in the pi-cohort skill. No `as` (no {outputs.name} wiring here) and no
+// `label` (TaskParam has no such field at this position). Widening-only: unknown keys were
+// already ignored at runtime.
+const TaskItem = Type.Unsafe({
+	type: "object",
+	additionalProperties: true,
+	required: ["agent", "task"],
+	properties: {
+		agent: { type: "string" },
+		task: { type: "string" },
+		count: { type: "integer", minimum: 1, description: "Repeat N times (same settings)." },
+	},
+	description: "Per-task overrides (model, output, reads, skill, ...): pi-cohort skill.",
 });
 
 // Parallel-item stubs (round two): structural/wiring keys stay explicit (agent/task,
@@ -232,11 +236,6 @@ const ChainItem = Type.Object({
 }, {
 	description: "{agent,task?} seq; {parallel:[..]} concurrent; {expand,parallel,collect} fanout.",
 	additionalProperties: false,
-	allOf: [
-		{ if: { required: ["expand"] }, then: { required: ["parallel", "collect"], properties: { parallel: { type: "object" } } } },
-		{ if: { required: ["collect"] }, then: { required: ["expand", "parallel"], properties: { parallel: { type: "object" } } } },
-		{ not: { required: ["expand"], properties: { parallel: { type: "array", items: {} } } } },
-	],
 });
 
 // Control stub (round two): 10 fields collapsed; the full table lives in the pi-cohort
@@ -260,12 +259,6 @@ export const SubagentParams = Type.Object({
 	id: Type.Optional(Type.String({
 		description: "Run id/prefix for status/interrupt/resume."
 	})),
-	runId: Type.Optional(Type.String({
-		description: "Run ID (interrupt/resume); defaults to latest. Prefer id."
-	})),
-	dir: Type.Optional(Type.String({
-		description: "Async run dir for status/resume."
-	})),
 	index: Type.Optional(Type.Integer({ minimum: 0, description: "Zero-based index for per-child actions." })),
 	message: Type.Optional(Type.String({ description: "Follow-up message for resume." })),
 	// Chain identifier for management (can't reuse 'chain' — that's the execution array)