pi-cohort 5.1.2 → 5.1.3

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # 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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-cohort",
-  "version": "5.1.2",
+  "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 {}
 

package/src/extension/schemas.ts

@@ -259,9 +259,6 @@ export const SubagentParams = Type.Object({
 	id: Type.Optional(Type.String({
 		description: "Run id/prefix for status/interrupt/resume."
 	})),
-	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)