@bermudi/pi-delegate 0.1.18 → 0.1.19

package/README.md

@@ -35,17 +35,16 @@ delegate({
 ```
 
 Parent extension/MCP tools are not copied, and project instructions are rebuilt
-for the task's `cwd`. Omit `agent` when you want an ad-hoc task using delegate's
-normal inline defaults instead.
+for the task's `cwd`. Omit `agent` when you want an inline task using delegate's
+normal defaults instead.
 
 The other built-ins are:
 
 - `scout` — read-only investigation with `read`, `grep`, `find`, and `ls`.
 - `coder` — implementation and verification with `read`, `write`, `edit`, and
   `bash` in the shared workspace.
-- `reviewer` — review with `read` and `bash`, using a disposable scratch copy by
-  default. Set `workspace: "shared"` when a reviewer needs a persistent
-  `sessionId`.
+- `reviewer` — review with `read` and `bash` in the shared workspace by default.
+  Set `workspace: "scratch"` when its project changes should be discarded.
 
 ### Shared-write safety
 
@@ -64,7 +63,7 @@ visible batch-level warning while it is active.
 A same-named Markdown file can override any built-in (first definition wins
 across `.pi/agents/`, `~/.pi/agent/agents/`, `~/.agents/`, `.claude/agents/`,
 `~/.claude/agents/`). A prompt-only override keeps the built-in's tools and
-workspace — `scout` stays read-only and `reviewer` stays scratch unless the
+workspace — `scout` stays read-only and `reviewer` stays shared unless the
 file explicitly sets `tools` or `workspace`. Fresh built-ins inherit the
 parent's exact model object and thinking level; an explicit `model`/`thinking`
 in the Markdown file replaces that inheritance. Task fields always win. For
@@ -106,7 +105,7 @@ host filesystem.
 
 ### Git-native isolated writers
 
-Use `workspace: "isolated"` for synchronous, one-shot coding tasks that should
+Use `workspace: "isolated"` for sync or async one-shot coding tasks that should
 run in parallel without sharing a working tree:
 
 ```ts
@@ -128,10 +127,12 @@ still matches the baseline.
 
 A clean reconciliation reports `applied_unverified`: textual merging does not
 prove the code builds or tests pass, so the result includes a suggested
-verification call. Isolated mode currently requires Git, rejects async and
-session reuse, disables whole-task retries, and rejects repositories with
-submodules. It is separation, not confinement: absolute-path writes can still
-reach the host.
+verification call. Async isolated tasks prepare after returning their ticket and
+settle only after reconciliation. Cancellation never applies unfinished work; a
+completed proposal cancelled before source application is retained as a private
+ref and full patch. Isolated mode requires Git, rejects session reuse, disables
+whole-task retries, and rejects repositories with submodules. It is separation,
+not confinement: absolute-path writes can still reach the host.
 
 ### Token accounting
 
@@ -216,8 +217,8 @@ over an installed extension.
   and workspace, and an explicit `model`/`thinking` replaces parent inheritance.
   For `default`, the `delegate.json` override maps are ignored (task fields still
   win); for the other built-ins, those maps win field-by-field over frontmatter.
-- **Ad-hoc subagent** — A subagent created from inline task fields instead of a
-  named Markdown agent profile. In current output this is labeled `ad-hoc`.
+- **Inline subagent** — A subagent created from inline task fields instead of a
+  named Markdown agent profile. In current output this is labeled `inline`.
 - **Inline task** — The task object itself when its configuration is supplied
   directly in the delegate call. Prefer this term over “inline agent” when
   talking about the API shape.

package/agents.ts

@@ -120,7 +120,7 @@ export const BUILTIN_AGENT_CONFIGS: Readonly<Record<string, AgentConfig>> = {
   [DEFAULT_AGENT_NAME]: {
     name: DEFAULT_AGENT_NAME,
     description:
-      "General-purpose subagent mirroring the live parent model, thinking level, native tools, and base prompt. Prefer it for general work; pick a specialist only when its role, tool limits, or scratch workspace genuinely fit.",
+      "General-purpose subagent mirroring the live parent model, thinking level, native tools, and base prompt. Prefer it for general work; pick a specialist when its role fits.",
     tools: DEFAULT_TOOLS,
     systemPrompt: "",
     builtin: true,

package/concurrency.ts

@@ -181,6 +181,11 @@ export function getModelKey(model: Model<Api> | undefined): string {
  *
  * The total number of concurrently running tasks is also capped by the
  * configured `maxConcurrent` value, shared across all `delegate` invocations.
+ *
+ * `beforeAcquire` runs per item before its global slot is acquired — a
+ * serialized successor awaits its predecessor there while holding only its
+ * per-model worker slot, never a global slot, so a serialized batch cannot
+ * pin the shared semaphore while running one task at a time.
  */
 export async function mapConcurrentByModel<T, R>(
   items: T[],
@@ -188,6 +193,7 @@ export async function mapConcurrentByModel<T, R>(
   getConcurrency: (modelKey: string) => number,
   fn: (item: T, index: number) => Promise<R>,
   signal?: AbortSignal,
+  beforeAcquire?: (index: number) => Promise<void>,
 ): Promise<R[]> {
   if (items.length === 0) return [];
   const results: R[] = new Array(items.length);
@@ -215,6 +221,7 @@ export async function mapConcurrentByModel<T, R>(
         group.limit,
         async (_item, localIdx) => {
           const globalIdx = group.indices[localIdx]!;
+          if (beforeAcquire) await beforeAcquire(globalIdx);
           const acquired = await acquireGlobal(signal);
           if (!acquired) {
             // Aborted while queued for a global slot: we hold no slot, so we

package/delegate.ts

@@ -52,11 +52,14 @@ export type { AgentOverride } from "./config.ts";
 export {
   checkout,
   commit,
+  recordUse,
   configFor,
   closePooledAgent,
   closeAllPooledAgents,
   listPooledAgents,
   withSessionLock,
+  SessionPool,
+  defaultSessionPool,
 } from "./pool.ts";
 export type {
   FrozenConfig,
@@ -79,8 +82,11 @@ export {
   resolveFinalTicketStatus,
   settleTicket,
   formatCompletedTicket,
+  TicketRegistry,
 } from "./tickets.ts";
 export type { TicketDelivery, SettleTicketOptions } from "./tickets.ts";
+export { createDelegateRuntime, getDefaultDelegateRuntime } from "./runtime.ts";
+export type { DelegateRuntime } from "./runtime.ts";
 export {
   recordTreeNavigation,
   getCurrentLeafId,