@bermudi/pi-delegate 0.1.11 → 0.1.13

package/tickets.ts

@@ -169,6 +169,7 @@ export function cancelTicketForShutdown(ticket: AsyncTicket): void {
         total_cost: totalCost,
       },
       ticket.telemetryGeneration,
+      ticket.telemetryConfig,
     );
   }
 }
@@ -238,6 +239,9 @@ export function formatCompletedTicket(
   parts.push(
     `${statusTag}${succeeded}/${ticket.results.length} ${completionLabel} · ${fmtDuration(elapsedTotal)} wall time\n`,
   );
+  if (ticket.dispatchWarning) {
+    parts.push(`WARNING: ${ticket.dispatchWarning}`);
+  }
 
   const pendingLabelFor = (index: number): string => {
     if (ticket.status !== "cancelled") {
@@ -258,7 +262,7 @@ export function formatCompletedTicket(
       parts.push(`[${pendingLabelFor(i)}]`);
       continue;
     }
-    parts.push(...formatCompletedTask(t, r));
+    parts.push(...formatCompletedTask(t, r, ticket.config));
   }
 
   const completedResults = ticket.results.filter(
@@ -294,6 +298,7 @@ export function formatCompletedTicket(
       ticketId: ticket.id,
       status: ticket.status,
       overlapWarning: overlapWarning || undefined,
+      dispatchWarning: ticket.dispatchWarning,
     },
   };
 }
@@ -336,9 +341,21 @@ function buildWaitDetails(ticket: AsyncTicket): DelegateDetails {
     ticketId: ticket.id,
     status: ticket.status,
     overlapWarning: overlapWarning || undefined,
+    dispatchWarning: ticket.dispatchWarning,
   };
 }
 
+function appendDispatchWarnings(
+  base: string,
+  details: DelegateDetails,
+): string {
+  const warnings = [
+    details.dispatchWarning ? `WARNING: ${details.dispatchWarning}` : undefined,
+    details.overlapWarning,
+  ].filter((warning): warning is string => warning !== undefined);
+  return warnings.length ? `${base}\n\n${warnings.join("\n\n")}` : base;
+}
+
 function buildWaitRunningUpdate(
   ticket: AsyncTicket,
 ): AgentToolResult<DelegateDetails> {
@@ -358,9 +375,7 @@ function buildWaitRunningUpdate(
   if (pending > 0) parts.push(`${pending} queued`);
 
   const details = buildWaitDetails(ticket);
-  const text =
-    parts.join(" · ") +
-    (details.overlapWarning ? `\n\n${details.overlapWarning}` : "");
+  const text = appendDispatchWarnings(parts.join(" · "), details);
   return {
     content: [{ type: "text", text }],
     details,
@@ -398,8 +413,7 @@ function buildWaitAbortResult(
 ): AgentToolResult<DelegateDetails> {
   const details = buildWaitDetails(ticket);
   const base = `Wait for ticket ${ticket.id} aborted · ticket continues ${ticket.status} in the background`;
-  const text =
-    base + (details.overlapWarning ? `\n\n${details.overlapWarning}` : "");
+  const text = appendDispatchWarnings(base, details);
   return {
     content: [{ type: "text", text }],
     details,
@@ -613,6 +627,7 @@ export function handlePoll(
         ticketId: ticket.id,
         status: ticket.status,
         overlapWarning: snapshot.overlapWarning || undefined,
+        dispatchWarning: ticket.dispatchWarning,
       },
     };
   }
@@ -656,9 +671,7 @@ export function handleCancel(params: {
   }
   if (!params.force) {
     const details = buildWaitDetails(ticket);
-    const text =
-      formatCancelPreview(ticket) +
-      (details.overlapWarning ? `\n\n${details.overlapWarning}` : "");
+    const text = appendDispatchWarnings(formatCancelPreview(ticket), details);
     return {
       content: [{ type: "text", text }],
       details,
@@ -667,8 +680,7 @@ export function handleCancel(params: {
   requestTicketCancel(ticket);
   const details = buildWaitDetails(ticket);
   const base = `Ticket '${ticketId}' is cancelling; workers are settling. Poll for final status.`;
-  const text =
-    base + (details.overlapWarning ? `\n\n${details.overlapWarning}` : "");
+  const text = appendDispatchWarnings(base, details);
   return {
     content: [{ type: "text", text }],
     details,

package/types.ts

@@ -99,8 +99,19 @@ export interface AsyncTicket {
   callRecord?: CallRecord;
   /** Runtime generation for rejecting shutdown writes from stale tickets. */
   telemetryGeneration?: number;
+  /** Telemetry config captured at dispatch; binds shutdown aggregate rows to the
+   *  same backend the call span wrote to. */
+  telemetryConfig?: import("./config.ts").TelemetryConfig;
   /** Resolves after every async worker has settled, including shutdown aborts. */
   completion?: Promise<void>;
+  /** False from admission until every worker has quiesced. Unlike `status`,
+   * this remains false during shutdown's early terminal transition. */
+  workersSettled?: boolean;
+  /** Batch-level warning attached to this dispatch, not to any one task. */
+  dispatchWarning?: string;
+  /** Immutable dispatch-scoped delegate.json snapshot used by async workers and
+   *  later result formatting. */
+  config?: import("./config.ts").DelegateConfig;
 }
 
 /** Live parent settings captured when a delegate call starts. The built-in
@@ -138,6 +149,11 @@ export interface ResolvedTask {
   warnings: string[];
   /** Explicit settings that must match a live pooled session on reuse. */
   reuseIntent?: ReuseIntent;
+  /** Stable signature of the provider-scoped extension allowlist for this task's
+   *  model provider. Pool reuse compares this to the frozen session value so a
+   *  revoked or reconfigured extension cannot continue executing in a reused
+   *  session. */
+  providerExtensionSources?: string;
 }
 
 export interface ToolActivity {
@@ -195,6 +211,8 @@ export interface DelegateDetails {
   /** Global overlap warning derived from result.attributedFiles, surfaced in both
    *  the textual content and the custom TUI. */
   overlapWarning?: string;
+  /** Warning that applies to the whole dispatch rather than an individual task. */
+  dispatchWarning?: string;
 }
 
 export interface TaskResult {
@@ -227,8 +245,58 @@ export interface TaskResult {
    *  for overlap detection so concurrent tasks in the same repo do not
    *  fabricate false conflicts from shared git snapshots. */
   attributedFiles?: string[];
+  /** Git-native proposal/reconciliation outcome for workspace:"isolated". */
+  integration?: TaskIntegration;
 }
 
+export type TaskIntegrationStatus =
+  | "applied_unverified"
+  | "no_changes"
+  | "conflict"
+  | "discarded"
+  | "apply_failed";
+
+interface TaskIntegrationFiles {
+  proposedFiles: string[];
+  appliedFiles: string[];
+}
+
+interface TaskIntegrationWithoutRecovery extends TaskIntegrationFiles {
+  conflicts?: never;
+  baselineRef?: never;
+  proposalRef?: never;
+  patchPath?: never;
+  worktreePath?: never;
+}
+
+export type TaskIntegration =
+  | (TaskIntegrationWithoutRecovery & {
+      status: "applied_unverified";
+    })
+  | (TaskIntegrationWithoutRecovery & {
+      status: "no_changes";
+    })
+  | (TaskIntegrationWithoutRecovery & {
+      status: "discarded";
+    })
+  | (TaskIntegrationFiles & {
+      status: "conflict";
+      conflicts: Array<{ path: string; reason: string }>;
+      baselineRef: string;
+      proposalRef: string;
+      patchPath: string;
+      worktreePath: string;
+    })
+  | (TaskIntegrationFiles & {
+      status: "apply_failed";
+      conflicts: Array<{ path: string; reason: string }>;
+      /** Recovery metadata exists when reconciliation reached a proposal. */
+      baselineRef?: string;
+      proposalRef?: string;
+      patchPath?: string;
+      worktreePath?: string;
+    });
+
 /** Single source of truth for a subagent's runtime configuration.
  *  Passed to `createAgentSession` as `model` / `thinkingLevel` / `tools` / `cwd`. */
 export interface AgentRunConfig {
@@ -265,8 +333,15 @@ export interface TaskRunEnv {
   telemetryCallId?: string;
   /** Runtime generation for rejecting writes from a stale shutdown worker. */
   telemetryGeneration?: number;
+  /** Telemetry config captured at dispatch; binds task rows to the same backend
+   *  as the call span. */
+  telemetryConfig?: import("./config.ts").TelemetryConfig;
   /** Whether this task is part of an async ticket. */
   async?: boolean;
+  /** Immutable dispatch-scoped delegate.json snapshot. Long-lived async workers
+   *  use this instead of the live singleton so retry/stall/output/provider
+   *  settings stay stable for the ticket's lifetime. */
+  config?: import("./config.ts").DelegateConfig;
 }
 
 /** Structural subset of Pi's `ExtensionContext` used by delegate's
@@ -288,6 +363,10 @@ export interface DelegateToolCtx {
     | undefined;
   /** Optional hook Pi exposes for extensions to read the live system prompt. */
   getSystemPrompt?: () => string | undefined;
+  /** Optional TUI notice boundary used for migration warnings. */
+  ui?: {
+    notify(message: string, level?: "info" | "warning" | "error"): void;
+  };
 }
 
 /** Shape returned by the delegate tool's `execute`. Mirrors Pi's