pi-harness-delegate 0.1.0 → 0.2.0

package/README.md

@@ -1,5 +1,7 @@
 # pi-harness-delegate
 
+[![npm version](https://img.shields.io/npm/v/pi-harness-delegate?logo=npm&color=CB3837)](https://www.npmjs.com/package/pi-harness-delegate) [![CI](https://github.com/yorch/pi-harness-delegate/actions/workflows/ci.yml/badge.svg)](https://github.com/yorch/pi-harness-delegate/actions/workflows/ci.yml) [![Release](https://github.com/yorch/pi-harness-delegate/actions/workflows/release.yml/badge.svg)](https://github.com/yorch/pi-harness-delegate/actions/workflows/release.yml) [![Node](https://img.shields.io/badge/node-26.x-brightgreen?logo=node.js)](https://nodejs.org) [![Bun](https://img.shields.io/badge/bun-1.3.14-black?logo=bun)](https://bun.sh) [![Biome](https://img.shields.io/badge/Biome-2.5.10-60a5fa)](https://biomejs.dev) [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](LICENSE)
+
 Delegate work to **any harness** ([Claude Code](https://github.com/anthropics/claude-code), [Muse](https://github.com/openai/codex), [OpenCode](https://opencode.ai), [Amp](https://ampcode.com)) from the [pi coding agent](https://github.com/badlogic/pi-mono): code reviews, detailed plans, implementation, security audits, docs — or your own custom templates.
 
 Each harness runs headless in your repo with a normalized permission (`readonly` / `edit` / `danger`). Results stream back live, and token/cost usage feeds into pi's footer stats. Templates are portable — prompt bodies live in `templates/shared/`, harness-specific frontmatter selects the native permission.
@@ -44,7 +46,7 @@ The `delegate` tool takes: `harness`, `task`, `mode`, `scope` (`diff` = git diff
 ## Harnesses
 
 | Harness | Binary | Permission mapping | Notes |
-|---|---|---|---|
+| --- | --- | --- | --- |
 | `claude` | `claude` | `readonly→plan`, `edit→acceptEdits`, `danger→bypassPermissions` | Full stream-json, cost + context% |
 | `codex` | `codex` | `readonly→read-only`, `edit→workspace-write`, `danger→danger-full-access` | `codex exec --json`, best-effort JSONL |
 | `opencode` | `opencode` | `readonly→read-only`, `edit→allow-edit`, `danger→danger` | `opencode run --format json` |
@@ -55,7 +57,7 @@ Detect availability: `delegate` checks `harness --version` at startup; missing h
 ## Modes (templates)
 
 | Mode | Permission | Purpose |
-|---|---|---|
+| --- | --- | --- |
 | `review` | `readonly` | Code review, cites `file:line`, prioritized findings |
 | `plan` | `readonly` | Detailed implementation plan with steps + risks |
 | `implement` | `edit` | Implements a task, runs checks, reports changes |
@@ -167,12 +169,12 @@ Review what the harness is asked to do before granting broad permissions.
 ## Development
 
 ```bash
-npm install
-npm run typecheck
-npm test
+bun install
+bun run typecheck
+bun test
 ```
 
-See `AGENTS.md` for architecture. Release: bump `version` in `package.json`, `npm publish --access public`, `pi update --extensions`.
+See [CONTRIBUTING.md](CONTRIBUTING.md) for the project layout, the release dev-loop, and the npm publish gotchas. Agents working in this repo should read [AGENTS.md](AGENTS.md).
 
 ## License
 

package/extensions/activity.ts

@@ -3,252 +3,276 @@ import { join } from 'node:path';
 import type { ActivityEvent } from './harnesses/types.ts';
 
 function truncate(s: string, max: number): string {
-	return s.length > max ? `${s.slice(0, max - 1)}…` : s;
+  return s.length > max ? `${s.slice(0, max - 1)}…` : s;
 }
 
 /** Make a template/mode name safe for use in a filename. */
 export function safeSegmentName(name: string): string {
-	const safe = name.replace(/[^a-zA-Z0-9_-]+/g, '_').replace(/^_+|_+$/g, '');
-	return safe.length > 0 ? safe : 'delegate';
+  const safe = name.replace(/[^a-zA-Z0-9_-]+/g, '_').replace(/^_+|_+$/g, '');
+  return safe.length > 0 ? safe : 'delegate';
 }
 
 export interface MetricsInput {
-	numTurns: number;
-	totalCostUsd: number;
-	promptTokens: number;
-	contextPercent: number | null;
-	durationMs: number | null;
+  numTurns: number;
+  totalCostUsd: number;
+  promptTokens: number;
+  contextPercent: number | null;
+  durationMs: number | null;
 }
 
 /** Compact run summary: `3 turn(s) · $0.54 · 62k tok · 6.2% ctx · 12s`. */
 export function formatMetrics(m: MetricsInput): string {
-	const parts: Array<string | null> = [
-		`${m.numTurns} turn(s)`,
-		`$${m.totalCostUsd.toFixed(3)}`,
-		m.promptTokens > 0 ? `${Math.round(m.promptTokens / 1000)}k tok` : null,
-		typeof m.contextPercent === 'number' ? `${m.contextPercent.toFixed(1)}% ctx` : null,
-		typeof m.durationMs === 'number' && m.durationMs !== null ? `${(m.durationMs / 1000).toFixed(0)}s` : null,
-	];
-	return parts.filter((p): p is string => Boolean(p)).join(' · ');
+  const parts: Array<string | null> = [
+    `${m.numTurns} turn(s)`,
+    `$${m.totalCostUsd.toFixed(3)}`,
+    m.promptTokens > 0 ? `${Math.round(m.promptTokens / 1000)}k tok` : null,
+    typeof m.contextPercent === 'number' ? `${m.contextPercent.toFixed(1)}% ctx` : null,
+    typeof m.durationMs === 'number' && m.durationMs !== null ? `${(m.durationMs / 1000).toFixed(0)}s` : null,
+  ];
+  return parts.filter((p): p is string => Boolean(p)).join(' · ');
 }
 
 /** Parse the metadata header of a transcript file (without loading the whole body). */
 export function parseTranscriptMeta(head: string): {
-	mode: string;
-	cost: number;
-	sessionId: string | null;
-	harness: string | null;
+  mode: string;
+  cost: number;
+  sessionId: string | null;
+  harness: string | null;
 } {
-	let mode = 'delegate';
-	let cost = 0;
-	let sessionId: string | null = null;
-	let harness: string | null = null;
-	const mm = /^# Delegated (?:Claude|Harness) run — (.+)$/m.exec(head);
-	if (mm) mode = mm[1];
-	// Also match new header: # Delegated <harness> run — <mode>
-	const hm = /^# Delegated (\w+) run —/m.exec(head);
-	if (hm) harness = hm[1].toLowerCase();
-	const cm = /\bcost: \$([\d.]+)/.exec(head);
-	if (cm) cost = Number(cm[1]);
-	const sm = /\bsession: ([0-9a-f-]+)/.exec(head);
-	if (sm) sessionId = sm[1];
-	// harness explicit field
-	const hfm = /^-\s*harness:\s*(\w+)/m.exec(head);
-	if (hfm) harness = hfm[1];
-	return { mode, cost, sessionId, harness };
+  let mode = 'delegate';
+  let cost = 0;
+  let sessionId: string | null = null;
+  let harness: string | null = null;
+  const mm = /^# Delegated (?:Claude|Harness) run — (.+)$/m.exec(head);
+  if (mm) mode = mm[1];
+  // Also match new header: # Delegated <harness> run — <mode>
+  const hm = /^# Delegated (\w+) run —/m.exec(head);
+  if (hm) harness = hm[1].toLowerCase();
+  const cm = /\bcost: \$([\d.]+)/.exec(head);
+  if (cm) cost = Number(cm[1]);
+  const sm = /\bsession: ([0-9a-f-]+)/.exec(head);
+  if (sm) sessionId = sm[1];
+  // harness explicit field
+  const hfm = /^-\s*harness:\s*(\w+)/m.exec(head);
+  if (hfm) harness = hfm[1];
+  return { mode, cost, sessionId, harness };
 }
 
 /** Build the markdown report content injected into the session on the next turn. */
 export function buildReportContent(opts: {
-	harness?: string;
-	mode: string;
-	metrics: string;
-	body: string;
-	file?: string;
-	sessionId?: string;
+  harness?: string;
+  mode: string;
+  metrics: string;
+  body: string;
+  file?: string;
+  sessionId?: string;
 }): string {
-	const harness = opts.harness ?? 'claude';
-	const header = `## ${harness} ${opts.mode} (${opts.metrics})`;
-	const foot: string[] = [];
-	if (opts.file) foot.push(`transcript: ${opts.file}`);
-	if (opts.sessionId) foot.push(`resume: \`/delegate --harness=${opts.harness} --resume=${opts.sessionId} <prompt>\` (or /${opts.harness} --resume=${opts.sessionId})`);
-	return [header, '', opts.body, foot.length > 0 ? `\n_${foot.join(' · ')}_` : ''].join('\n');
+  const harness = opts.harness ?? 'claude';
+  const header = `## ${harness} ${opts.mode} (${opts.metrics})`;
+  const foot: string[] = [];
+  if (opts.file) foot.push(`transcript: ${opts.file}`);
+  if (opts.sessionId)
+    foot.push(
+      `resume: \`/delegate --harness=${opts.harness} --resume=${opts.sessionId} <prompt>\` (or /${opts.harness} --resume=${opts.sessionId})`,
+    );
+  return [header, '', opts.body, foot.length > 0 ? `\n_${foot.join(' · ')}_` : ''].join('\n');
 }
 
 /** Legacy wrapper for compat */
 export function buildClaudeReportContent(opts: {
-	mode: string;
-	metrics: string;
-	body: string;
-	file?: string;
-	sessionId?: string;
+  mode: string;
+  metrics: string;
+  body: string;
+  file?: string;
+  sessionId?: string;
 }): string {
-	return buildReportContent({ harness: 'claude', ...opts });
+  return buildReportContent({ harness: 'claude', ...opts });
 }
 
 /** Delete oldest transcript files beyond `maxCount` (0 = keep everything). */
 export function pruneOutputs(dir: string, maxCount: number): void {
-	if (maxCount <= 0) return;
-	let files: string[];
-	try {
-		files = readdirSync(dir);
-	} catch {
-		return;
-	}
-	const byMtime = files
-		.filter((f) => f.endsWith('.md'))
-		.map((f) => ({ f, mtime: statSync(join(dir, f), { throwIfNoEntry: false })?.mtimeMs ?? 0 }))
-		.sort((a, b) => b.mtime - a.mtime);
-	for (const { f } of byMtime.slice(maxCount)) {
-		try {
-			rmSync(join(dir, f));
-		} catch {
-			// best-effort
-		}
-	}
+  if (maxCount <= 0) return;
+  let files: string[];
+  try {
+    files = readdirSync(dir);
+  } catch {
+    return;
+  }
+  const byMtime = files
+    .filter(f => f.endsWith('.md'))
+    .map(f => ({ f, mtime: statSync(join(dir, f), { throwIfNoEntry: false })?.mtimeMs ?? 0 }))
+    .sort((a, b) => b.mtime - a.mtime);
+  for (const { f } of byMtime.slice(maxCount)) {
+    try {
+      rmSync(join(dir, f));
+    } catch {
+      // best-effort
+    }
+  }
 }
 
 /** Human-readable one-liner for a tool call (uses Claude's `description` when present). */
 export function formatToolUse(name: string, input: Record<string, unknown>): string {
-	if (typeof input.description === 'string' && input.description) {
-		return `${name}: ${truncate(input.description, 90)}`;
-	}
-	if (typeof input.command === 'string') return `${name}: ${truncate(input.command.split('\n')[0], 90)}`;
-	if (typeof input.file_path === 'string') return `${name}: ${input.file_path}`;
-	if (typeof input.pattern === 'string') return `${name}: ${input.pattern}`;
-	if (typeof input.url === 'string') return `${name}: ${input.url}`;
-	const first = Object.values(input).find((v): v is string => typeof v === 'string' && v.length > 0);
-	return first ? `${name}: ${truncate(first, 90)}` : name;
+  if (typeof input.description === 'string' && input.description) {
+    return `${name}: ${truncate(input.description, 90)}`;
+  }
+  if (typeof input.command === 'string') return `${name}: ${truncate(input.command.split('\n')[0], 90)}`;
+  if (typeof input.file_path === 'string') return `${name}: ${input.file_path}`;
+  if (typeof input.pattern === 'string') return `${name}: ${input.pattern}`;
+  if (typeof input.url === 'string') return `${name}: ${input.url}`;
+  const first = Object.values(input).find((v): v is string => typeof v === 'string' && v.length > 0);
+  return first ? `${name}: ${truncate(first, 90)}` : name;
 }
 
 /** Compact per-line activity log for the transcript (tool_input + results only). */
 export function collectActivityLog(events: ActivityEvent[]): string[] {
-	const log: string[] = [];
-	for (const ev of events) {
-		if (ev.kind === 'tool_input') {
-			log.push(`▶ ${formatToolUse(ev.name, ev.input)}`);
-		} else if (ev.kind === 'tool_result') {
-			const last = log.length - 1;
-			if (last >= 0 && log[last].startsWith('▶')) {
-				log[last] += ev.isError ? '  ✗ error' : '  ✓';
-			}
-		}
-	}
-	return log;
+  const log: string[] = [];
+  for (const ev of events) {
+    if (ev.kind === 'tool_input') {
+      log.push(`▶ ${formatToolUse(ev.name, ev.input)}`);
+    } else if (ev.kind === 'tool_result') {
+      const last = log.length - 1;
+      if (last >= 0 && log[last].startsWith('▶')) {
+        log[last] += ev.isError ? '  ✗ error' : '  ✓';
+      }
+    }
+  }
+  return log;
 }
 
 /** Full transcript written to the outputs dir: metadata + activity + output. */
-export function buildTranscript(opts: {
-	harness?: string;
-	mode: string;
-	permission?: string;
-	permissionMode?: string;
-	nativePermission?: string;
-	model: string | null;
-	cwd: string;
-	sessionId: string | null;
-	resumed: boolean;
-	numTurns: number;
-	totalCostUsd: number;
-	isError: boolean;
-	stopReason: string | null;
-	durationMs: number | null;
-	usage: { inputTokens: number; outputTokens: number; cacheCreationInputTokens: number; cacheReadInputTokens: number } | null;
-	contextPercent: number | null;
-	contextWindow: number | null;
-	activityLog: string[];
-	output: string;
-} & Record<string, unknown>): string {
-	const harness = (opts.harness as string | undefined) ?? 'claude';
-	const permissionRaw = (opts.permission as string | undefined) ?? (opts.permissionMode as string | undefined) ?? 'edit';
-	let permission = permissionRaw;
-	let nativePermission = opts.nativePermission as string | undefined;
-	// map legacy permissionMode to normalized if needed
-	if ((opts as Record<string, unknown>).permissionMode && !opts.permission) {
-		const pm = (opts as Record<string, unknown>).permissionMode as string;
-		if (pm === 'plan') { permission = 'readonly'; nativePermission = pm; }
-		else if (pm === 'bypassPermissions') { permission = 'danger'; nativePermission = pm; }
-		else { permission = 'edit'; nativePermission = pm; }
-	}
-	const u = opts.usage;
-	const tokens = u
-		? [
-				`input ${u.inputTokens}`,
-				`output ${u.outputTokens}`,
-				`cache+${u.cacheCreationInputTokens}`,
-				`cache ${u.cacheReadInputTokens}`,
-			].join(' · ')
-		: null;
-	const context =
-		opts.contextPercent !== null && opts.contextWindow
-			? `${opts.contextPercent.toFixed(1)}% of ${opts.contextWindow.toLocaleString()} window`
-			: null;
-	const duration = opts.durationMs !== null ? `${(opts.durationMs / 1000).toFixed(1)}s` : null;
-	const permLine = nativePermission
-		? `- permission: ${permission} (${nativePermission})`
-		: `- permission: ${permission}`;
+export function buildTranscript(
+  opts: {
+    harness?: string;
+    mode: string;
+    permission?: string;
+    permissionMode?: string;
+    nativePermission?: string;
+    model: string | null;
+    cwd: string;
+    sessionId: string | null;
+    resumed: boolean;
+    numTurns: number;
+    totalCostUsd: number;
+    isError: boolean;
+    stopReason: string | null;
+    durationMs: number | null;
+    usage: {
+      inputTokens: number;
+      outputTokens: number;
+      cacheCreationInputTokens: number;
+      cacheReadInputTokens: number;
+    } | null;
+    contextPercent: number | null;
+    contextWindow: number | null;
+    activityLog: string[];
+    output: string;
+  } & Record<string, unknown>,
+): string {
+  const harness = (opts.harness as string | undefined) ?? 'claude';
+  const permissionRaw =
+    (opts.permission as string | undefined) ?? (opts.permissionMode as string | undefined) ?? 'edit';
+  let permission = permissionRaw;
+  let nativePermission = opts.nativePermission as string | undefined;
+  // map legacy permissionMode to normalized if needed
+  if ((opts as Record<string, unknown>).permissionMode && !opts.permission) {
+    const pm = (opts as Record<string, unknown>).permissionMode as string;
+    if (pm === 'plan') {
+      permission = 'readonly';
+      nativePermission = pm;
+    } else if (pm === 'bypassPermissions') {
+      permission = 'danger';
+      nativePermission = pm;
+    } else {
+      permission = 'edit';
+      nativePermission = pm;
+    }
+  }
+  const u = opts.usage;
+  const tokens = u
+    ? [
+        `input ${u.inputTokens}`,
+        `output ${u.outputTokens}`,
+        `cache+${u.cacheCreationInputTokens}`,
+        `cache ${u.cacheReadInputTokens}`,
+      ].join(' · ')
+    : null;
+  const context =
+    opts.contextPercent !== null && opts.contextWindow
+      ? `${opts.contextPercent.toFixed(1)}% of ${opts.contextWindow.toLocaleString()} window`
+      : null;
+  const duration = opts.durationMs !== null ? `${(opts.durationMs / 1000).toFixed(1)}s` : null;
+  const permLine = nativePermission
+    ? `- permission: ${permission} (${nativePermission})`
+    : `- permission: ${permission}`;
 
-	return [
-		`# Delegated ${harness.charAt(0).toUpperCase() + harness.slice(1)} run — ${opts.mode}`,
-		'',
-		`- harness: ${harness}`,
-		`- mode: ${opts.mode}`,
-		permLine,
-		`- model: ${opts.model ?? 'default'}`,
-		`- cwd: ${opts.cwd}`,
-		`- session: ${opts.sessionId ?? 'n/a'}${opts.resumed ? ' (resumed)' : ''}`,
-		`- turns: ${opts.numTurns} · cost: $${opts.totalCostUsd.toFixed(4)} · isError: ${opts.isError}`,
-		`- tokens: ${tokens ?? 'n/a'}`,
-		`- context: ${context ?? 'n/a'}`,
-		`- duration: ${duration ?? 'n/a'}`,
-		`- stop reason: ${opts.stopReason ?? 'n/a'}`,
-		'',
-		'## Activity',
-		opts.activityLog.length > 0 ? opts.activityLog.join('\n') : '(no tool activity)',
-		'',
-		'## Output',
-		opts.output || '(empty)',
-		'',
-	].join('\n');
+  return [
+    `# Delegated ${harness.charAt(0).toUpperCase() + harness.slice(1)} run — ${opts.mode}`,
+    '',
+    `- harness: ${harness}`,
+    `- mode: ${opts.mode}`,
+    permLine,
+    `- model: ${opts.model ?? 'default'}`,
+    `- cwd: ${opts.cwd}`,
+    `- session: ${opts.sessionId ?? 'n/a'}${opts.resumed ? ' (resumed)' : ''}`,
+    `- turns: ${opts.numTurns} · cost: $${opts.totalCostUsd.toFixed(4)} · isError: ${opts.isError}`,
+    `- tokens: ${tokens ?? 'n/a'}`,
+    `- context: ${context ?? 'n/a'}`,
+    `- duration: ${duration ?? 'n/a'}`,
+    `- stop reason: ${opts.stopReason ?? 'n/a'}`,
+    '',
+    '## Activity',
+    opts.activityLog.length > 0 ? opts.activityLog.join('\n') : '(no tool activity)',
+    '',
+    '## Output',
+    opts.output || '(empty)',
+    '',
+  ].join('\n');
 }
 
 /** Legacy wrapper */
 export function buildClaudeTranscript(opts: {
-	mode: string;
-	permissionMode: string;
-	model: string | null;
-	cwd: string;
-	sessionId: string | null;
-	resumed: boolean;
-	numTurns: number;
-	totalCostUsd: number;
-	isError: boolean;
-	stopReason: string | null;
-	durationMs: number | null;
-	usage: { inputTokens: number; outputTokens: number; cacheCreationInputTokens: number; cacheReadInputTokens: number } | null;
-	contextPercent: number | null;
-	contextWindow: number | null;
-	activityLog: string[];
-	output: string;
+  mode: string;
+  permissionMode: string;
+  model: string | null;
+  cwd: string;
+  sessionId: string | null;
+  resumed: boolean;
+  numTurns: number;
+  totalCostUsd: number;
+  isError: boolean;
+  stopReason: string | null;
+  durationMs: number | null;
+  usage: {
+    inputTokens: number;
+    outputTokens: number;
+    cacheCreationInputTokens: number;
+    cacheReadInputTokens: number;
+  } | null;
+  contextPercent: number | null;
+  contextWindow: number | null;
+  activityLog: string[];
+  output: string;
 }): string {
-	return buildTranscript({
-		harness: 'claude',
-		mode: opts.mode,
-		permission: opts.permissionMode === 'plan' ? 'readonly' : opts.permissionMode === 'bypassPermissions' ? 'danger' : 'edit',
-		nativePermission: opts.permissionMode,
-		model: opts.model,
-		cwd: opts.cwd,
-		sessionId: opts.sessionId,
-		resumed: opts.resumed,
-		numTurns: opts.numTurns,
-		totalCostUsd: opts.totalCostUsd,
-		isError: opts.isError,
-		stopReason: opts.stopReason,
-		durationMs: opts.durationMs,
-		usage: opts.usage,
-		contextPercent: opts.contextPercent,
-		contextWindow: opts.contextWindow,
-		activityLog: opts.activityLog,
-		output: opts.output,
-	});
+  return buildTranscript({
+    harness: 'claude',
+    mode: opts.mode,
+    permission:
+      opts.permissionMode === 'plan' ? 'readonly' : opts.permissionMode === 'bypassPermissions' ? 'danger' : 'edit',
+    nativePermission: opts.permissionMode,
+    model: opts.model,
+    cwd: opts.cwd,
+    sessionId: opts.sessionId,
+    resumed: opts.resumed,
+    numTurns: opts.numTurns,
+    totalCostUsd: opts.totalCostUsd,
+    isError: opts.isError,
+    stopReason: opts.stopReason,
+    durationMs: opts.durationMs,
+    usage: opts.usage,
+    contextPercent: opts.contextPercent,
+    contextWindow: opts.contextWindow,
+    activityLog: opts.activityLog,
+    output: opts.output,
+  });
 }