@pi-unipi/background-tasks 2.16.1 → 2.17.0

package/src/tools.ts

@@ -6,9 +6,7 @@
  */
 
 import { Text } from "@earendil-works/pi-tui";
-import { getInstalledPackageVersion } from "@pi-unipi/core";
-import { dirname } from "node:path";
-import { fileURLToPath } from "node:url";
+import { renderCompletionCard, renderLaunchCard } from "./cards.js";
 import { Type, type Static } from "typebox";
 import {
   COMMAND_PREVIEW_CHARS,
@@ -27,7 +25,6 @@ import {
   type BgRunDetails,
   type BgStatusDetails,
   type BgTaskSnapshot,
-  type StartAttestedPiTaskOptions,
   type StartTaskOptions,
 } from "./types.js";
 import type { BackgroundTaskRegistry } from "./registry.js";
@@ -88,31 +85,12 @@ export const BgKillParams = Type.Object({
   taskId: Type.String({ description: "Task ID or unambiguous prefix of a running task" }),
 });
 
-export const BgPiAttestedParams = Type.Object({
-  name: Type.String({ description: "Short human-readable task name" }),
-  provider: Type.String({ description: "Pi provider id (e.g. anthropic)" }),
-  model: Type.String({ description: "Model id under the provider" }),
-  prompt: Type.String({ description: "Final user prompt for the child Pi run" }),
-  reportPath: Type.String({
-    description: "Relative path (inside cwd) where the child must write its report before exit",
-  }),
-  extraPiArgs: Type.Optional(
-    Type.Array(Type.String(), { description: "Extra literal pi CLI args (restricted)" }),
-  ),
-  thinking: Type.Optional(Type.String({ description: "Thinking level passed as --thinking" })),
-  timeoutSeconds: Type.Optional(Type.Number({ description: "Kill after this many seconds" })),
-});
-
 // ── Registration ────────────────────────────────────────────────────────────
 
 export interface RegisterSurfaceOptions {
   pi: import("@earendil-works/pi-coding-agent").ExtensionAPI;
   registry: BackgroundTaskRegistry;
   startTask: (ctx: any, command: string, options?: StartTaskOptions) => Promise<any>;
-  startAttestedPiTask: (
-    ctx: any,
-    options: StartAttestedPiTaskOptions,
-  ) => Promise<any>;
   openTaskManager: (ctx: any, initialTaskId?: string) => Promise<void>;
   clearFinishedNotices: (ctx: any) => number;
   openSettings: (ctx: any) => Promise<void>;
@@ -135,25 +113,6 @@ export function registerToolsAndCommands(options: RegisterSurfaceOptions): void
 
   // ── Commands (/unipi:* namespace — ours) ──────────────────────────────────
 
-  // Update info lives with OUR updater module; this only reports versions.
-  pi.registerCommand("unipi:bg-update", {
-    description: "Show the installed background-tasks version and how to update",
-    handler: (_args, ctx) => {
-      const here = dirname(fileURLToPath(import.meta.url));
-      const current = getInstalledPackageVersion(here, "@pi-unipi/background-tasks");
-      const lines = [
-        `@pi-unipi/background-tasks ${current} is installed.`,
-        "Background tasks ship inside the @pi-unipi/unipi umbrella package.",
-        "Update from npm:",
-        "  pi install npm:@pi-unipi/unipi@latest",
-        "Or use /unipi:updater-settings to check for updates.",
-        "This command only prints update instructions; it does not install or self-update.",
-      ];
-      ctx.ui.notify(lines.join("\n"), "info");
-      return Promise.resolve();
-    },
-  });
-
   pi.registerCommand("unipi:bg", {
     description:
       'Start a shell command as a tracked background task: /unipi:bg [--agent] [--name "Task name"] <command>',
@@ -180,14 +139,6 @@ export function registerToolsAndCommands(options: RegisterSurfaceOptions): void
     },
   });
 
-  pi.registerCommand("unipi:tasks", {
-    description: "Open the background task manager UI",
-    handler: async (args, ctx) => {
-      const taskId = typeof args === "string" ? args.trim() : "";
-      await options.openTaskManager(ctx, taskId || undefined);
-    },
-  });
-
   pi.registerCommand("unipi:bg-tasks", {
     description: "Open the background task manager UI",
     handler: async (args, ctx) => {
@@ -196,86 +147,6 @@ export function registerToolsAndCommands(options: RegisterSurfaceOptions): void
     },
   });
 
-  pi.registerCommand("unipi:bg-clear", {
-    description: "Clear finished background task footer notices",
-    handler: (_args, ctx) => {
-      options.clearFinishedNotices(ctx);
-      return Promise.resolve();
-    },
-  });
-
-  pi.registerCommand("unipi:jobs", {
-    description: "List running and recent background tasks",
-    handler: (_args, ctx) => {
-      ctx.ui.notify(
-        formatSnapshotList(registry.allTasks().map((task) => registry.snapshot(task))),
-        "info",
-      );
-      return Promise.resolve();
-    },
-  });
-
-  pi.registerCommand("unipi:logs", {
-    description: "Show bounded output from a background task: /unipi:logs <id> [maxBytes]",
-    getArgumentCompletions: (prefix: string) => {
-      const matches = registry
-        .allTasks()
-        .filter((task) => task.id.startsWith(prefix.trim()))
-        .slice(0, 20)
-        .map((task) => ({
-          value: task.id,
-          label: `${task.id} ${taskDisplayName(task)}`,
-          description: `${task.status} — ${truncateChars(task.command, 60)}`,
-        }));
-      return matches.length > 0 ? matches : null;
-    },
-    handler: async (args, ctx) => {
-      try {
-        const [id, bytes] = args.trim().split(/\s+/, 2);
-        const task = registry.resolveTask(id ?? "");
-        const maxBytes = normalizeMaxBytes(Number(bytes), DEFAULT_LOG_BYTES);
-        const logs = await registry.getTaskLogs(task, maxBytes, true);
-        ctx.ui.notify(logs.text, "info");
-      } catch (error) {
-        ctx.ui.notify(
-          `Background logs error: ${error instanceof Error ? error.message : String(error)}`,
-          "error",
-        );
-      }
-    },
-  });
-
-  pi.registerCommand("unipi:kill", {
-    description: "Stop a running background task: /unipi:kill <id>",
-    getArgumentCompletions: (prefix: string) => {
-      const matches = registry
-        .allTasks()
-        .filter((task) => task.status === "running" && task.id.startsWith(prefix.trim()))
-        .slice(0, 20)
-        .map((task) => ({
-          value: task.id,
-          label: `${task.id} ${taskDisplayName(task)}`,
-          description: truncateChars(task.command, 70),
-        }));
-      return matches.length > 0 ? matches : null;
-    },
-    handler: async (args, ctx) => {
-      try {
-        const task = registry.resolveTask(args.trim());
-        await registry.stopTask(task, "user");
-        ctx.ui.notify(
-          `Killed ${taskDisplayName(task)} (${task.id}). Output: ${task.outputPath}`,
-          "info",
-        );
-      } catch (error) {
-        ctx.ui.notify(
-          `Background kill error: ${error instanceof Error ? error.message : String(error)}`,
-          "error",
-        );
-      }
-    },
-  });
-
   // Shortcuts (same keys as reference; documented in our README)
   pi.registerCommand("unipi:bg-settings", {
     description: "Open background-tasks settings (master toggle, defaults)",
@@ -301,21 +172,8 @@ export function registerToolsAndCommands(options: RegisterSurfaceOptions): void
 
   pi.registerMessageRenderer<BgTaskSnapshot>(
     "background-task-notification",
-    (message: { details?: BgTaskSnapshot }, _options: unknown, theme: any) => {
-      const task = message.details;
-      const status = task?.status ?? "completed";
-      const color: string =
-        status === "completed" ? "success" : status === "failed" ? "error" : status === "killed" ? "warning" : "accent";
-      const id = task?.id ?? "background task";
-      const name = task ? taskDisplayName(task) : "Background task";
-      const output = task?.outputPath ? `\n${theme.fg("dim", `Output: ${task.outputPath}`)}` : "";
-      const error = task?.error ? `\n${theme.fg("error", task.error)}` : "";
-      return new Text(
-        `${theme.fg(color, `[bg ${status}]`)} ${theme.fg("accent", name)} ${theme.fg("dim", `(${id})`)}${output}${error}`,
-        0,
-        0,
-      );
-    },
+    (message: { details?: BgTaskSnapshot }, _options: unknown, theme: any) =>
+      renderCompletionCard(theme, message.details),
   );
 
   // ── Tools (reference names kept) ─────────────────────────────────────────
@@ -371,12 +229,7 @@ export function registerToolsAndCommands(options: RegisterSurfaceOptions): void
       );
     },
     renderResult(result: { details: BgRunDetails }, _options: unknown, theme: any) {
-      const { task } = result.details;
-      return new Text(
-        `${theme.fg("success", "✓ started")} ${theme.fg("accent", taskDisplayName(task))} ${theme.fg("dim", `(${task.id})`)}\n${theme.fg("dim", `Output: ${task.outputPath}`)}`,
-        0,
-        0,
-      );
+      return renderLaunchCard(theme, result.details.task);
     },
   });
 
@@ -491,42 +344,4 @@ export function registerToolsAndCommands(options: RegisterSurfaceOptions): void
       );
     },
   });
-
-  pi.registerTool<typeof BgPiAttestedParams, BgRunDetails>({
-    name: "bg_run_pi_attested",
-    label: "Attested Pi Run",
-    description:
-      "Opt-in evidence-oriented direct Pi spawn. Launches exactly one `pi --mode json` child, records raw events/stderr, hashes prompt/report/output, observes OAuth through ModelRegistry, and emits a strict attestation sidecar only after successful completion.",
-    promptSnippet: "Start an attested direct Pi agent task and return its task ID plus output path",
-    promptGuidelines: [
-      "Use only when the user explicitly asks for an attested Pi evidence-producing task; ordinary background work should use bg_run unchanged.",
-      "Provide provider/model as structured fields and a relative reportPath that the child Pi prompt will write before exit.",
-      "Do not provide channel, auth, route, or hash claims; the producer observes those facts itself and fails loudly if it cannot attest them.",
-    ],
-    parameters: BgPiAttestedParams,
-    async execute(_toolCallId, params, _signal, _onUpdate, ctx) {
-      const task = await options.startAttestedPiTask(ctx, params);
-      return {
-        content: textContent(
-          `Started attested Pi task ${taskDisplayName(task)} (${task.id})\nStatus: ${task.status}\nPID: ${String(task.pid ?? "unknown")}\nOutput: ${task.outputPath}\nAttestation: ${task.attestationPath ?? "pending until completion"}`,
-        ),
-        details: { task: registry.snapshot(task) },
-      };
-    },
-    renderCall(args: Static<typeof BgPiAttestedParams>, theme: any) {
-      return new Text(
-        `${theme.fg("toolTitle", theme.bold("bg_run_pi_attested "))}${theme.fg("muted", truncateChars(args.name, COMMAND_PREVIEW_CHARS))}`,
-        0,
-        0,
-      );
-    },
-    renderResult(result: { details: BgRunDetails }, _options: unknown, theme: any) {
-      const { task } = result.details;
-      return new Text(
-        `${theme.fg("success", "✓ started")} ${theme.fg("accent", taskDisplayName(task))} ${theme.fg("dim", `(${task.id})`)}\n${theme.fg("dim", `Output: ${task.outputPath}`)}\n${theme.fg("dim", `Attestation: ${task.attestationPath ?? "pending"}`)}`,
-        0,
-        0,
-      );
-    },
-  });
 }

package/src/types.ts

@@ -12,7 +12,6 @@
 import { statSync, type WriteStream } from "node:fs";
 import { join } from "node:path";
 import type { BackgroundTaskChildProcess } from "./child-process.js";
-import type { FusionResultDetails, FusionUsage, FusionWorkflowId } from "./fusion/types.js";
 
 export const TASK_STATUS_VALUES = ["running", "completed", "failed", "killed"] as const;
 export const TERMINAL_TASK_STATUS_VALUES = ["completed", "failed", "killed"] as const;
@@ -69,20 +68,9 @@ export interface BgTaskSnapshot {
   toolUsage?: TaskToolUsage | undefined;
   model?: string | undefined;
   telemetryUnavailableReason?: string | undefined;
-  attestationPath?: string | undefined;
   delegate?: DelegateTaskFacts | undefined;
-  fusion?: FusionTaskFacts | undefined;
-}
-
-export interface AttestedPiTaskFiles {
-  eventsPath: string;
-  stderrPath: string;
-  wrapperPath: string;
-  attestationPath: string;
-}
-
-export interface AttestedPiTaskSnapshot extends BgTaskSnapshot {
-  attestedPi?: AttestedPiTaskFiles | undefined;
+  /** Last few non-empty output lines (bounded), for compact chat cards. */
+  outputTail?: string[] | undefined;
 }
 
 /** Delegate-specific task facts surfaced through snapshots and `bg_result`. */
@@ -114,25 +102,6 @@ export interface DelegateTaskOutcome {
 export type DelegateBudgetRouteSource = import("./delegate/types.js").DelegateBudgetRouteSource;
 export type DelegateExtensionMode = import("./delegate/types.js").DelegateExtensionMode;
 
-/** Fusion-specific task facts surfaced through snapshots and `bg_result`. */
-export interface FusionTaskFacts {
-  runId: string;
-  workflow: FusionWorkflowId;
-  artifactDir: string;
-  artifactDirAbs: string;
-  state: string;
-  outcome?: FusionTaskOutcome | undefined;
-  /** Durable once-only accounting claim made by the first successful bg_result retrieval. */
-  usageDelivered: boolean;
-}
-
-export interface FusionTaskOutcome {
-  status: "committed" | "failed" | "cancelled";
-  resultDetails?: FusionResultDetails | undefined;
-  usage?: FusionUsage | undefined;
-  error?: string | undefined;
-}
-
 export interface BgTask extends Omit<BgTaskSnapshot, "name"> {
   name: string;
   outputAbsPath: string;
@@ -160,14 +129,8 @@ export interface BgTask extends Omit<BgTaskSnapshot, "name"> {
   /** Partial trailing stdout line held between chunks while reconstructing wrapped-agent control lines. */
   agentStdoutBuffer?: string | undefined;
   telemetryUnavailableReason?: string | undefined;
-  attestationPath?: string | undefined;
-  attestedPi?: AttestedPiTaskFiles | undefined;
   delegate?: DelegateTaskFacts | undefined;
-  fusion?: FusionTaskFacts | undefined;
-  /** Cancellation hook for an in-process managed task such as Fusion. */
-  managedCancel?: (() => void) | undefined;
-  managedCancelRequested?: boolean | undefined;
-  managedStopWaitMs?: number | undefined;
+  outputTail?: string[] | undefined;
   metadataWriteChain?: Promise<void> | undefined;
   waiters: Array<() => void>;
 }
@@ -264,23 +227,6 @@ export interface StartTaskOptions {
   terminalPublicationGate?: Promise<void> | undefined;
 }
 
-/** Prepared managed launch handed to the registry after preflight has succeeded. */
-export interface StartManagedTaskOptions {
-  id: string;
-  name: string;
-  command: string;
-  description?: string | undefined;
-  isAgent: boolean;
-  completion: Promise<void>;
-  cancel: () => void;
-  notifyOnCompletion: boolean;
-  triggerOnCompletion: boolean;
-  fusion: FusionTaskFacts;
-  stopWaitMs?: number | undefined;
-  /** Prevent terminal publication until the launch receipt handoff is observable. */
-  terminalPublicationGate?: Promise<void> | undefined;
-}
-
 export interface StartDelegateTaskOptions {
   name: string;
   argv: readonly string[];
@@ -293,17 +239,6 @@ export interface StartDelegateTaskOptions {
   timeoutSeconds?: number | undefined;
 }
 
-export interface StartAttestedPiTaskOptions {
-  name: string;
-  provider: string;
-  model: string;
-  prompt: string;
-  reportPath: string;
-  extraPiArgs?: string[] | undefined;
-  thinking?: string | undefined;
-  timeoutSeconds?: number | undefined;
-}
-
 /** Default tool-output byte cap (mirrors pi's DEFAULT_MAX_BYTES). */
 export const DEFAULT_MAX_BYTES = 30 * 1024;
 
@@ -830,12 +765,24 @@ export function snapshot(task: BgTask): BgTaskSnapshot {
     toolUsage: task.toolUsage,
     model: task.model,
     telemetryUnavailableReason: task.telemetryUnavailableReason,
-    attestationPath: task.attestationPath,
     delegate: task.delegate,
-    fusion: task.fusion,
+    outputTail: task.outputTail === undefined ? undefined : [...task.outputTail],
   };
 }
 
+export const OUTPUT_TAIL_LINES = 3;
+const OUTPUT_TAIL_LINE_CHARS = 200;
+
+/** Fold a raw output chunk into the task's bounded tail ring (non-empty lines only). */
+export function appendOutputTail(task: { outputTail?: string[] | undefined }, chunk: string): void {
+  if (chunk.length === 0) return;
+  const lines = chunk.split(/\r?\n/u).map((line) => line.replace(/\x1b\[[0-9;]*[A-Za-z]/gu, "").trimEnd()).filter((line) => line.trim().length > 0);
+  if (lines.length === 0) return;
+  const tail = task.outputTail ?? [];
+  for (const line of lines) tail.push(line.length > OUTPUT_TAIL_LINE_CHARS ? `${line.slice(0, OUTPUT_TAIL_LINE_CHARS)}…` : line);
+  task.outputTail = tail.slice(-OUTPUT_TAIL_LINES);
+}
+
 export async function boundedRead(
   filePath: string,
   maxBytes: number,

package/extensions/anthropic-attribution.ts

@@ -1 +0,0 @@
-export { default } from '../src/anthropic-attribution.js';

package/extensions/fusion-child.ts

@@ -1 +0,0 @@
-export { default } from '../src/fusion-child-extension.js';

package/src/anthropic-attribution-path.ts

@@ -1,21 +0,0 @@
-/**
- * @pi-unipi/background-tasks — Anthropic attribution extension path
- *
- * Resolves this package's own attribution extension entry so isolated child Pi
- * processes can load it explicitly. Mirrors the reference layout:
- * extensions/anthropic-attribution.ts re-exports src/anthropic-attribution.ts.
- */
-
-import { dirname, resolve } from "node:path";
-import { fileURLToPath } from "node:url";
-import { existsSync } from "node:fs";
-
-export function resolveAnthropicAttributionExtensionPath(): string {
-  // src/anthropic-attribution-path.ts -> packages/background-tasks/extensions/
-  const here = dirname(fileURLToPath(import.meta.url));
-  const candidate = resolve(here, "..", "extensions", "anthropic-attribution.ts");
-  if (!existsSync(candidate)) {
-    throw new Error(`Anthropic attribution extension is missing: ${candidate}`);
-  }
-  return candidate;
-}