@sgtbeatdown/pi-tasks 0.1.3 → 0.1.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sgtbeatdown/pi-tasks",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "Durable Pi agent delegation through the Wolfpack task gateway.",
   "module": "src/index.ts",
   "exports": {

package/skills/wolfpack-pi-task-delegation/SKILL.md

@@ -31,6 +31,19 @@ For a multi-step project phase, retain one persistent implementer and one persis
 
 After every verified and acknowledged terminal task, explicitly retain a parent-spawned session only when concrete follow-up is likely; otherwise close it through canonical Wolfpack session control. Workers never close their own sessions. terminal `send` is only for explicit human steering; it is not task state, completion evidence, or a substitute for `agent_task_message`.
 
+## delegation lifecycle
+
+| event | owner | command | session outcome |
+| --- | --- | --- | --- |
+| assignment completion | worker | `agent_task_done` | unchanged |
+| task acknowledgment | parent | `agent_task_ack` | unchanged |
+| role session retention | parent | `none` | retained for reuse |
+| role session teardown | parent | `wolfpack kill <session-or-id> --json` | exact stable session id terminated |
+| teardown verification | parent | `wolfpack list --json` | exact stable session id absent |
+| Pi exit | none | `/quit` | not Wolfpack teardown |
+
+Assignment completion ends the assigned task, not the reusable role session. When no further reuse is likely, the parent—not the worker—must pass the exact stable session ID to `wolfpack kill <session-or-id> --json`, then inspect `wolfpack list --json` and verify that ID is absent. `/quit` exits Pi; it is not Wolfpack teardown. Workers never close themselves.
+
 ## v2 workflow
 
 1. create or select the role session, verify its structured project and terminal readiness, then obtain its registered `taskEndpoint`.

package/src/abortable-sleep.ts

@@ -0,0 +1,16 @@
+export async function abortableSleep(milliseconds: number, signal: AbortSignal | undefined, cancellationError: () => Error): Promise<void> {
+	if (signal?.aborted) throw cancellationError();
+	await new Promise<void>((resolve, reject) => {
+		const removeAbortListener = (): void => signal?.removeEventListener("abort", abort);
+		const timeout = setTimeout(() => {
+			removeAbortListener();
+			resolve();
+		}, milliseconds);
+		const abort = (): void => {
+			clearTimeout(timeout);
+			removeAbortListener();
+			reject(cancellationError());
+		};
+		signal?.addEventListener("abort", abort, { once: true });
+	});
+}

package/src/extension.ts

@@ -3,6 +3,7 @@ import type { AgentToolResult, ExtensionAPI, ExtensionContext } from "@earendil-
 import { Text } from "@earendil-works/pi-tui";
 import { Type } from "typebox";
 
+import { abortableSleep } from "./abortable-sleep";
 import { deliverTaskInbox, incorporatedTaskEvents } from "./task-inbox";
 import { TaskOutboxDeliveryError, TaskProtocolError } from "./task-protocol";
 import { createWolfpackTaskCore } from "./wolfpack-task-relay";
@@ -205,7 +206,7 @@ export function registerAgentTaskTools(pi: ExtensionAPI, core: TaskCore | undefi
 					if (terminal(task.status)) return toolResult(task, `## task status\n- task: \`${task.taskId}\`\n- status: ${task.status}`);
 					if (Date.now() >= deadline) return toolResult({ taskId: params.taskId, status: task.status }, `## task wait\n- task: \`${params.taskId}\`\n- status: ${task.status}\n- wait timed out`);
 					onUpdate?.({ content: [{ type: "text", text: `waiting for ${params.taskId}...` }], details: {} });
-					await sleep(Math.min(WAIT_POLL_MS, deadline - Date.now()), signal);
+					await abortableSleep(Math.max(1, Math.min(WAIT_POLL_MS, deadline - Date.now())), signal, () => new Error("task wait was cancelled"));
 				}
 			} catch (error) { return taskError(error); }
 		}, renderResult(result, _options, theme) { return new Text(theme.fg("accent", text(result))); },
@@ -311,11 +312,3 @@ function text(result: { readonly content?: readonly unknown[] }): string {
 	const first = result.content?.[0];
 	return first && typeof first === "object" && "type" in first && first.type === "text" && "text" in first && typeof first.text === "string" ? first.text : "";
 }
-
-async function sleep(milliseconds: number, signal: AbortSignal | undefined): Promise<void> {
-	if (signal?.aborted) throw new Error("task wait was cancelled");
-	await new Promise<void>((resolve, reject) => {
-		const timer = setTimeout(resolve, Math.max(1, milliseconds));
-		signal?.addEventListener("abort", () => { clearTimeout(timer); reject(new Error("task wait was cancelled")); }, { once: true });
-	});
-}

package/src/legacy-extension.ts

@@ -3,6 +3,7 @@ import type { AgentToolResult, ExtensionAPI, ExtensionContext } from "@earendil-
 import { Text } from "@earendil-works/pi-tui";
 import { Type } from "typebox";
 
+import { abortableSleep } from "./abortable-sleep";
 import {
 	DEFAULT_TASK_TIMEOUT_MS,
 	GatewayClientError,
@@ -126,7 +127,7 @@ export function registerAgentTaskTools(pi: ExtensionAPI, client: WolfpackGateway
 					if (terminal(status.status)) return statusResult(status);
 					if (Date.now() >= deadline) return toolResult({ taskId: params.taskId, status: status.status, error: { code: "WAIT_TIMEOUT", retryable: true } }, `## task wait\n- task: \`${params.taskId}\`\n- status: ${status.status}\n- wait timed out`);
 					onUpdate?.({ content: [{ type: "text", text: `waiting for ${params.taskId}...` }], details: {} });
-					await sleep(Math.min(WAIT_POLL_MS, Math.max(1, deadline - Date.now())), signal);
+					await abortableSleep(Math.min(WAIT_POLL_MS, Math.max(1, deadline - Date.now())), signal, () => new GatewayClientError("ABORTED", "task wait was cancelled", true));
 				}
 			} catch (error) { return clientError(error); }
 		}, renderResult(result, _options, theme) { return new Text(theme.fg("accent", text(result))); },
@@ -238,11 +239,3 @@ function text(result: { readonly content?: readonly unknown[] }): string {
 function throwIfAborted(signal: AbortSignal | undefined): void {
 	if (signal?.aborted) throw new GatewayClientError("ABORTED", "task wait was cancelled", true);
 }
-
-async function sleep(milliseconds: number, signal: AbortSignal | undefined): Promise<void> {
-	await new Promise<void>((resolve, reject) => {
-		const timeout = setTimeout(resolve, milliseconds);
-		const abort = (): void => { clearTimeout(timeout); reject(new GatewayClientError("ABORTED", "task wait was cancelled", true)); };
-		signal?.addEventListener("abort", abort, { once: true });
-	});
-}

package/src/task-inbox.ts

@@ -55,15 +55,6 @@ export async function deliverTaskInbox(pi: InboxPi, core: TaskCore, context: Inb
 	}
 }
 
-export function restoredInboxCursor(entries: readonly unknown[]): string {
-	let cursor = "0";
-	for (const entry of entries) {
-		if (!isRecord(entry) || entry.type !== "custom" || entry.customType !== TASK_CURSOR_CUSTOM_TYPE || !isRecord(entry.data) || !decimal(entry.data.cursor)) continue;
-		cursor = entry.data.cursor;
-	}
-	return cursor;
-}
-
 function inboxEvent(delivery: RelayDelivery): TaskEvent {
 	try {
 		const payload = JSON.parse(delivery.envelope.payload) as unknown;
@@ -113,7 +104,3 @@ function key(taskId: string, eventId: string): string {
 function isRecord(value: unknown): value is Record<string, unknown> {
 	return value !== null && typeof value === "object" && !Array.isArray(value);
 }
-
-function decimal(value: unknown): value is string {
-	return typeof value === "string" && /^(0|[1-9][0-9]*)$/.test(value);
-}