pi-roundtable-coding 0.7.14 → 0.7.16

package/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 ## [Unreleased]
 
+## [0.7.16] - 2026-10-04
+
+- The worker's shell calls follow pi-roundtable 0.7.16's `shellHoldRule`: an `rm` whose operands all resolve inside the job's workspace, without being it, runs without a hold. Every `git push` stays held.
+- A finished job frees its repository before its report is delivered, so the report turn can run `repo_change_report`, `repo_push` or the next `repo_task` there; `idle()` still waits for the delivery.
+
 ## [0.7.5] - 2026-10-03
 
 - Add `workerBlockText`, `limits` and `diagnosticChars`; git, gh and worker error text and the report and held-action bounds can be raised or lifted by the host.

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-roundtable-coding",
-	"version": "0.7.14",
+	"version": "0.7.16",
 	"description": "Repository shelves and owner-approved Pi coding workers for pi-roundtable",
 	"type": "module",
 	"license": "MIT",
@@ -44,7 +44,7 @@
 		"@earendil-works/pi-ai": ">=1.0.0 <2",
 		"@biomejs/biome": "2.5.15",
 		"@types/bun": "1.4.2",
-		"pi-roundtable": "0.7.14",
+		"pi-roundtable": "0.7.16",
 		"typescript": "7.0.2"
 	}
 }

package/src/coding-desk.ts

@@ -99,6 +99,8 @@ function bounded(text: string, max: number): string {
 /** One worker per repository, at most three per channel; results and jobs are in memory. */
 export class CodingDesk {
 	readonly #options: CodingDeskOptions;
+	/** Every run until its report is delivered, for idle(). */
+	readonly #pending = new Set<Promise<void>>();
 	readonly #running = new Map<
 		number,
 		{ job: CodingJob; controller: AbortController; done?: Promise<void> }
@@ -129,7 +131,7 @@ export class CodingDesk {
 			throw new AgentError(`A coding worker is still using ${repo}.`);
 	}
 	async idle(): Promise<void> {
-		await Promise.all([...this.#running.values()].map((run) => run.done));
+		await Promise.all(this.#pending);
 	}
 	async stop(): Promise<void> {
 		this.#stopped = true;
@@ -183,6 +185,9 @@ export class CodingDesk {
 				);
 			})
 			.finally(() => this.#running.delete(job.id));
+		const done = run.done;
+		this.#pending.add(done);
+		void done.finally(() => this.#pending.delete(done));
 		return job;
 	}
 	async #run(
@@ -332,6 +337,8 @@ export class CodingDesk {
 		} catch {
 			this.#options.logger.warn({ job: job.id }, "Coding thread close failed.");
 		}
+		// The work is over: free the repository before the report turn, so that turn can ship it.
+		this.#running.delete(job.id);
 		await deliver(result);
 	}
 }