@maxiaochao/pi-toolkit 0.10.22 → 0.10.24

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## 2026-10-12 - v0.10.24
+
+- bundle herdr-subagents v0.4.10: Stabilize concurrency checks and cancellation synchronization on slower CI runners.
+
+## 2026-10-12 - v0.10.23
+
+- bundle herdr-subagents v0.4.9: Stabilize concurrency checks on slower CI runners.
 ## 2026-10-12 - v0.10.22
 
 - bundle herdr-subagents v0.4.8: make blocking behave identically in serial and parallel runs and let respond continue a named failed task from its saved session

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@maxiaochao/pi-toolkit",
-  "version": "0.10.22",
+  "version": "0.10.24",
   "description": "Personal pi coding agent toolkit: scheduler, /btw, /cache_export, apply_patch, browser automation, and bundled theme/skills.",
   "type": "module",
   "workspaces": [

package/packages/herdr-subagents/CHANGELOG.md

@@ -1,3 +1,10 @@
+## 2026-10-12 - v0.4.10
+
+- Stabilize concurrency checks and cancellation synchronization on slower CI runners.
+
+## 2026-10-12 - v0.4.9
+
+- Stabilize concurrency checks on slower CI runners.
 ## 2026-10-12 - v0.4.8
 
 - make blocking behave identically in serial and parallel runs and let respond continue a named failed task from its saved session

package/packages/herdr-subagents/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@maxiaochao/pi-herdr-subagents",
-  "version": "0.4.8",
+  "version": "0.4.10",
   "description": "Run inspectable interactive Pi subagent batches through Herdr.",
   "type": "module",
   "keywords": [

package/packages/herdr-subagents/tests/run-tests.mjs

@@ -75,6 +75,7 @@ function fakeHerdr(mode = "completed") {
 	let active = 0;
 	let maxActive = 0;
 	let taskRuns = 0;
+	let promptBarrier;
 	const agents = new Map();
 	const writeAttemptReport = (directory, report) => {
 		const task = JSON.parse(readFileSync(join(directory, "task.json"), "utf8"));
@@ -87,6 +88,18 @@ function fakeHerdr(mode = "completed") {
 	return {
 		calls,
 		get maxActive() { return maxActive; },
+		expectConcurrentPrompts(count) {
+			let release;
+			const barrier = { count, arrivals: 0, promise: new Promise((resolve) => { release = resolve; }) };
+			const finish = () => {
+				if (promptBarrier !== barrier) return;
+				promptBarrier = undefined;
+				clearTimeout(barrier.timeout);
+				release();
+			};
+			barrier.finish = finish;
+			promptBarrier = barrier;
+		},
 		async createTab(request) {
 			calls.push(["createTab", request]);
 			taskDirectory = request.env.PI_HERDR_SUBAGENTS_TASK_DIR;
@@ -104,6 +117,13 @@ function fakeHerdr(mode = "completed") {
 			const runNumber = ++taskRuns;
 			active += 1;
 			maxActive = Math.max(maxActive, active);
+			const barrier = promptBarrier;
+			if (barrier) {
+				barrier.timeout ??= setTimeout(barrier.finish, 10_000);
+				barrier.arrivals += 1;
+				if (barrier.arrivals >= barrier.count) barrier.finish();
+				await barrier.promise;
+			}
 			await new Promise((resolve) => setTimeout(resolve, mode === "ui-blocked" ? 30 : request.prompt.includes("slow") ? 15 : 2));
 			if (mode === "delivery-unknown-late") {
 				const task = JSON.parse(readFileSync(join(directory, "task.json"), "utf8"));
@@ -599,6 +619,7 @@ try {
 	check("configured persona uses displayed effective values", !configuredRun.isError && configuredStart.args.includes("task/model") && configuredStart.args.includes("high") && configuredStart.args.includes("--skill") && configuredStart.args.includes("project-skill"));
 
 	const analystHerdr = fakeHerdr();
+	analystHerdr.expectConcurrentPrompts(3);
 	const analystBatch = await executeHerdrSubagents({ label: "independent-analysis", concurrency: 3, tasks: [
 		{ name: "sol-audit", prompt: "slow audit", agent: "analyst", model: "vendor/sol" },
 		{ name: "grok-replicate", prompt: "slow replicate", agent: "analyst", model: "vendor/grok" },
@@ -611,6 +632,7 @@ try {
 	check("completed analyst batch closes its run tab", analystHerdr.calls.at(-1)?.[0] === "closeTab");
 
 	const batchHerdr = fakeHerdr();
+	batchHerdr.expectConcurrentPrompts(2);
 	const batchUpdates = [];
 	const batch = await executeHerdrSubagents({ label: "reviews", concurrency: 2, tasks: [
 		{ name: "one", prompt: "one slow", agent: "explorer" },
@@ -649,10 +671,32 @@ try {
 	check("the parallel run closes only after its last blocked lane is answered", !secondOfBoth.isError && secondOfBoth.details.status === "completed" && secondOfBoth.details.tasks.every((entry) => entry.status === "completed") && bothBlockedHerdr.calls.filter(([name]) => name === "closeTab").length === 1);
 	const mixedCancelHerdr = fakeHerdr();
 	const mixedCancelPrompt = mixedCancelHerdr.promptAgent.bind(mixedCancelHerdr);
-	mixedCancelHerdr.promptAgent = async (request) => request.prompt.includes("WAIT") ? await new Promise((_resolve, reject) => request.signal?.addEventListener("abort", () => reject(request.signal.reason), { once: true })) : mixedCancelPrompt(request);
+	let markMixedFailureSettled;
+	let markMixedWaitStarted;
+	const mixedFailureSettled = new Promise((resolve) => { markMixedFailureSettled = resolve; });
+	const mixedWaitStarted = new Promise((resolve) => { markMixedWaitStarted = resolve; });
+	mixedCancelHerdr.promptAgent = async (request) => {
+		if (request.prompt.includes("WAIT")) {
+			markMixedWaitStarted();
+			if (!request.signal) throw new Error("WAIT prompt requires an AbortSignal");
+			return await new Promise((_resolve, reject) => {
+				if (request.signal.aborted) reject(request.signal.reason);
+				else request.signal.addEventListener("abort", () => reject(request.signal.reason), { once: true });
+			});
+		}
+		return mixedCancelPrompt(request);
+	};
 	const mixedCancelController = new AbortController();
-	const mixedCancelPromise = executeHerdrSubagents({ concurrency: 2, tasks: [{ name: "fail-first", prompt: "FAIL", agent: "explorer" }, { name: "cancel-second", prompt: "WAIT", agent: "reviewer" }] }, mixedCancelController.signal, undefined, { cwd: root }, { agentsDirectory: join(process.cwd(), "agents"), herdr: mixedCancelHerdr });
-	await new Promise((resolve) => setTimeout(resolve, 10));
+	const mixedCancelPromise = executeHerdrSubagents(
+		{ concurrency: 2, tasks: [{ name: "fail-first", prompt: "FAIL", agent: "explorer" }, { name: "cancel-second", prompt: "WAIT", agent: "reviewer" }] },
+		mixedCancelController.signal,
+		(update) => {
+			if (update.details?.tasks?.some((task) => task.status === "failed")) markMixedFailureSettled();
+		},
+		{ cwd: root },
+		{ agentsDirectory: join(process.cwd(), "agents"), herdr: mixedCancelHerdr },
+	);
+	await Promise.all([mixedFailureSettled, mixedWaitStarted]);
 	mixedCancelController.abort(new Error("Parent Pi session closed."));
 	const mixedCancel = await mixedCancelPromise;
 	check("failed and cancelled production tasks aggregate to partial", mixedCancel.details.status === "partial" && mixedCancel.details.tasks.some((task) => task.status === "failed") && mixedCancel.details.tasks.some((task) => task.status === "cancelled"));
@@ -1724,6 +1768,7 @@ try {
 	const queuedRecoveryHerdr = fakeHerdr();
 	await reconcileSubagentRuns(queuedRecoveryRoot, queuedRecoveryHerdr);
 	check("reconciliation preserves unstarted task records as queued", JSON.parse(readFileSync(join(secondRecoveryTask, "task.json"), "utf8")).status === "queued" && JSON.parse(readFileSync(join(thirdRecoveryTask, "task.json"), "utf8")).status === "queued");
+	queuedRecoveryHerdr.expectConcurrentPrompts(2);
 	await continueQueuedRun(queuedRunDirectory, queuedRecoveryHerdr);
 	check("startup continuation restores bounded concurrency", queuedRecoveryHerdr.maxActive === 2 && JSON.parse(readFileSync(join(secondRecoveryTask, "task.json"), "utf8")).status === "completed" && JSON.parse(readFileSync(join(thirdRecoveryTask, "task.json"), "utf8")).status === "completed" && JSON.parse(readFileSync(join(queuedRunDirectory, "run.json"), "utf8")).status === "completed");
 	const staleContinuationRoot = join(root, "stale-continuation");