pi-harness-delegate 0.1.0 → 0.1.1

package/extensions/config.ts

@@ -4,155 +4,161 @@ import { join } from 'node:path';
 import { DEFAULT_TIMEOUT_MS } from './harnesses/types.ts';
 
 export interface HarnessConfig {
-	model?: string;
-	timeoutMs?: number;
-	allowDangerous?: boolean;
-	maxBudgetUsd?: number;
+  model?: string;
+  timeoutMs?: number;
+  allowDangerous?: boolean;
+  maxBudgetUsd?: number;
 }
 
 export interface DelegateConfig {
-	model?: string;
-	timeoutMs: number;
-	defaultMode: string;
-	defaultHarness: string;
-	allowDangerous: boolean;
-	inspectThinking: boolean;
-	maxBudgetUsd?: number;
-	autoDelegateHints: boolean;
-	modelAliases: Record<string, string>;
-	maxConcurrent: number | { global?: number; perHarness?: Record<string, number> };
-	maxTranscripts: number;
-	harnesses: Record<string, HarnessConfig>;
+  model?: string;
+  timeoutMs: number;
+  defaultMode: string;
+  defaultHarness: string;
+  allowDangerous: boolean;
+  inspectThinking: boolean;
+  maxBudgetUsd?: number;
+  autoDelegateHints: boolean;
+  modelAliases: Record<string, string>;
+  maxConcurrent: number | { global?: number; perHarness?: Record<string, number> };
+  maxTranscripts: number;
+  harnesses: Record<string, HarnessConfig>;
 }
 
 export function agentDir(): string {
-	return process.env.PI_CODING_AGENT_DIR ?? join(homedir(), '.pi', 'agent');
+  return process.env.PI_CODING_AGENT_DIR ?? join(homedir(), '.pi', 'agent');
 }
 
 export function outputsDir(harness?: string): string {
-	const base = join(agentDir(), 'delegate', 'outputs');
-	return harness ? join(base, harness) : base;
+  const base = join(agentDir(), 'delegate', 'outputs');
+  return harness ? join(base, harness) : base;
 }
 
 export function legacyOutputsDir(): string {
-	return join(agentDir(), 'claude-delegate', 'outputs');
+  return join(agentDir(), 'claude-delegate', 'outputs');
 }
 
 export function loadConfig(): DelegateConfig {
-	const cfg: DelegateConfig = {
-		timeoutMs: DEFAULT_TIMEOUT_MS,
-		defaultMode: 'general',
-		defaultHarness: 'claude',
-		allowDangerous: false,
-		inspectThinking: false,
-		autoDelegateHints: false,
-		modelAliases: { economy: 'haiku', balanced: 'sonnet', max: 'opus' },
-		maxConcurrent: 1,
-		maxTranscripts: 100,
-		harnesses: {},
-	};
-	try {
-		const file = join(agentDir(), 'settings.json');
-		if (!existsSync(file)) return cfg;
-		const settings = JSON.parse(readFileSync(file, 'utf8')) as {
-			delegate?: Partial<DelegateConfig & { harnesses: Record<string, HarnessConfig> }>;
-			claudeDelegate?: Partial<DelegateConfig & { model?: string }>;
-		};
-		// Legacy claudeDelegate -> delegate.harnesses.claude migration
-		if (settings.claudeDelegate && !settings.delegate) {
-			const c = settings.claudeDelegate as Partial<DelegateConfig>;
-			if (typeof c.model === 'string') cfg.harnesses['claude'] = { ...(cfg.harnesses['claude'] ?? {}), model: c.model };
-			if (typeof c.timeoutMs === 'number' && c.timeoutMs > 0) cfg.timeoutMs = c.timeoutMs;
-			if (typeof c.defaultMode === 'string') cfg.defaultMode = c.defaultMode;
-			if (typeof c.allowDangerous === 'boolean') cfg.allowDangerous = c.allowDangerous;
-			if (typeof c.inspectThinking === 'boolean') cfg.inspectThinking = c.inspectThinking;
-			if (typeof (c as DelegateConfig).maxBudgetUsd === 'number' && (c as DelegateConfig).maxBudgetUsd! > 0) cfg.maxBudgetUsd = (c as DelegateConfig).maxBudgetUsd;
-			if (typeof c.autoDelegateHints === 'boolean') cfg.autoDelegateHints = c.autoDelegateHints;
-			if ((c as DelegateConfig).modelAliases && typeof (c as DelegateConfig).modelAliases === 'object') {
-				for (const [k, v] of Object.entries((c as DelegateConfig).modelAliases!)) {
-					if (typeof v === 'string' && v) cfg.modelAliases[k] = v;
-				}
-			}
-			if (typeof c.maxConcurrent === 'number' && c.maxConcurrent >= 0) cfg.maxConcurrent = c.maxConcurrent;
-			else if (c.maxConcurrent && typeof c.maxConcurrent === 'object') {
-				const obj = c.maxConcurrent as { global?: unknown; perHarness?: unknown };
-				const out: { global?: number; perHarness?: Record<string, number> } = {};
-				if (typeof obj.global === 'number' && obj.global >= 0) out.global = obj.global;
-				if (obj.perHarness && typeof obj.perHarness === 'object') {
-					const ph: Record<string, number> = {};
-					for (const [k, v] of Object.entries(obj.perHarness as Record<string, unknown>)) {
-						if (typeof v === 'number' && v >= 0) ph[k] = v;
-					}
-					if (Object.keys(ph).length > 0) out.perHarness = ph;
-				}
-				if (out.global !== undefined || out.perHarness) cfg.maxConcurrent = out;
-			}
-			if (typeof c.maxTranscripts === 'number' && c.maxTranscripts >= 0) cfg.maxTranscripts = c.maxTranscripts;
-			if (c.harnesses && typeof c.harnesses === 'object') {
-				for (const [k, v] of Object.entries(c.harnesses)) {
-					if (v && typeof v === 'object') cfg.harnesses[k] = { ...(cfg.harnesses[k] ?? {}), ...(v as HarnessConfig) };
-				}
-			}
-			// also map harnesses.claude if any
-			return cfg;
-		}
-		const d = settings.delegate ?? {};
-		if (typeof d.defaultHarness === 'string' && d.defaultHarness) cfg.defaultHarness = d.defaultHarness;
-		if (typeof d.defaultMode === 'string') cfg.defaultMode = d.defaultMode;
-		if (typeof d.model === 'string') cfg.model = d.model;
-		if (typeof d.timeoutMs === 'number' && d.timeoutMs > 0) cfg.timeoutMs = d.timeoutMs;
-		if (typeof d.allowDangerous === 'boolean') cfg.allowDangerous = d.allowDangerous;
-		if (typeof d.inspectThinking === 'boolean') cfg.inspectThinking = d.inspectThinking;
-		if (typeof d.maxBudgetUsd === 'number' && d.maxBudgetUsd > 0) cfg.maxBudgetUsd = d.maxBudgetUsd;
-		if (typeof d.autoDelegateHints === 'boolean') cfg.autoDelegateHints = d.autoDelegateHints;
-		if (d.modelAliases && typeof d.modelAliases === 'object') {
-			for (const [k, v] of Object.entries(d.modelAliases)) {
-				if (typeof v === 'string' && v) cfg.modelAliases[k] = v;
-			}
-		}
-		if (typeof d.maxConcurrent === 'number' && d.maxConcurrent >= 0) cfg.maxConcurrent = d.maxConcurrent;
-		else if (d.maxConcurrent && typeof d.maxConcurrent === 'object') {
-			const obj = d.maxConcurrent as { global?: unknown; perHarness?: unknown };
-			const out: { global?: number; perHarness?: Record<string, number> } = {};
-			if (typeof obj.global === 'number' && obj.global >= 0) out.global = obj.global;
-			if (obj.perHarness && typeof obj.perHarness === 'object') {
-				const ph: Record<string, number> = {};
-				for (const [k, v] of Object.entries(obj.perHarness as Record<string, unknown>)) {
-					if (typeof v === 'number' && v >= 0) ph[k] = v;
-				}
-				if (Object.keys(ph).length > 0) out.perHarness = ph;
-			}
-			if (out.global !== undefined || out.perHarness) cfg.maxConcurrent = out;
-		}
-		if (typeof d.maxTranscripts === 'number' && d.maxTranscripts >= 0) cfg.maxTranscripts = d.maxTranscripts;
-		if (d.harnesses && typeof d.harnesses === 'object') {
-			for (const [k, v] of Object.entries(d.harnesses)) {
-				if (v && typeof v === 'object') cfg.harnesses[k] = { ...(v as HarnessConfig) };
-			}
-		}
-		// also support legacy claudeDelegate merged when delegate also present (delegate wins)
-		if (settings.claudeDelegate) {
-			const c = settings.claudeDelegate as Partial<DelegateConfig>;
-			if (typeof c.model === 'string' && !cfg.harnesses['claude']?.model) {
-				cfg.harnesses['claude'] = { ...(cfg.harnesses['claude'] ?? {}), model: c.model };
-			}
-		}
-	} catch {
-		// invalid settings — fall back to defaults
-	}
-	return cfg;
+  const cfg: DelegateConfig = {
+    timeoutMs: DEFAULT_TIMEOUT_MS,
+    defaultMode: 'general',
+    defaultHarness: 'claude',
+    allowDangerous: false,
+    inspectThinking: false,
+    autoDelegateHints: false,
+    modelAliases: { economy: 'haiku', balanced: 'sonnet', max: 'opus' },
+    maxConcurrent: 1,
+    maxTranscripts: 100,
+    harnesses: {},
+  };
+  try {
+    const file = join(agentDir(), 'settings.json');
+    if (!existsSync(file)) return cfg;
+    const settings = JSON.parse(readFileSync(file, 'utf8')) as {
+      delegate?: Partial<DelegateConfig & { harnesses: Record<string, HarnessConfig> }>;
+      claudeDelegate?: Partial<DelegateConfig & { model?: string }>;
+    };
+    // Legacy claudeDelegate -> delegate.harnesses.claude migration
+    if (settings.claudeDelegate && !settings.delegate) {
+      const c = settings.claudeDelegate as Partial<DelegateConfig>;
+      if (typeof c.model === 'string') cfg.harnesses['claude'] = { ...(cfg.harnesses['claude'] ?? {}), model: c.model };
+      if (typeof c.timeoutMs === 'number' && c.timeoutMs > 0) cfg.timeoutMs = c.timeoutMs;
+      if (typeof c.defaultMode === 'string') cfg.defaultMode = c.defaultMode;
+      if (typeof c.allowDangerous === 'boolean') cfg.allowDangerous = c.allowDangerous;
+      if (typeof c.inspectThinking === 'boolean') cfg.inspectThinking = c.inspectThinking;
+      if (typeof (c as DelegateConfig).maxBudgetUsd === 'number' && (c as DelegateConfig).maxBudgetUsd! > 0)
+        cfg.maxBudgetUsd = (c as DelegateConfig).maxBudgetUsd;
+      if (typeof c.autoDelegateHints === 'boolean') cfg.autoDelegateHints = c.autoDelegateHints;
+      if ((c as DelegateConfig).modelAliases && typeof (c as DelegateConfig).modelAliases === 'object') {
+        for (const [k, v] of Object.entries((c as DelegateConfig).modelAliases!)) {
+          if (typeof v === 'string' && v) cfg.modelAliases[k] = v;
+        }
+      }
+      if (typeof c.maxConcurrent === 'number' && c.maxConcurrent >= 0) cfg.maxConcurrent = c.maxConcurrent;
+      else if (c.maxConcurrent && typeof c.maxConcurrent === 'object') {
+        const obj = c.maxConcurrent as { global?: unknown; perHarness?: unknown };
+        const out: { global?: number; perHarness?: Record<string, number> } = {};
+        if (typeof obj.global === 'number' && obj.global >= 0) out.global = obj.global;
+        if (obj.perHarness && typeof obj.perHarness === 'object') {
+          const ph: Record<string, number> = {};
+          for (const [k, v] of Object.entries(obj.perHarness as Record<string, unknown>)) {
+            if (typeof v === 'number' && v >= 0) ph[k] = v;
+          }
+          if (Object.keys(ph).length > 0) out.perHarness = ph;
+        }
+        if (out.global !== undefined || out.perHarness) cfg.maxConcurrent = out;
+      }
+      if (typeof c.maxTranscripts === 'number' && c.maxTranscripts >= 0) cfg.maxTranscripts = c.maxTranscripts;
+      if (c.harnesses && typeof c.harnesses === 'object') {
+        for (const [k, v] of Object.entries(c.harnesses)) {
+          if (v && typeof v === 'object') cfg.harnesses[k] = { ...(cfg.harnesses[k] ?? {}), ...(v as HarnessConfig) };
+        }
+      }
+      // also map harnesses.claude if any
+      return cfg;
+    }
+    const d = settings.delegate ?? {};
+    if (typeof d.defaultHarness === 'string' && d.defaultHarness) cfg.defaultHarness = d.defaultHarness;
+    if (typeof d.defaultMode === 'string') cfg.defaultMode = d.defaultMode;
+    if (typeof d.model === 'string') cfg.model = d.model;
+    if (typeof d.timeoutMs === 'number' && d.timeoutMs > 0) cfg.timeoutMs = d.timeoutMs;
+    if (typeof d.allowDangerous === 'boolean') cfg.allowDangerous = d.allowDangerous;
+    if (typeof d.inspectThinking === 'boolean') cfg.inspectThinking = d.inspectThinking;
+    if (typeof d.maxBudgetUsd === 'number' && d.maxBudgetUsd > 0) cfg.maxBudgetUsd = d.maxBudgetUsd;
+    if (typeof d.autoDelegateHints === 'boolean') cfg.autoDelegateHints = d.autoDelegateHints;
+    if (d.modelAliases && typeof d.modelAliases === 'object') {
+      for (const [k, v] of Object.entries(d.modelAliases)) {
+        if (typeof v === 'string' && v) cfg.modelAliases[k] = v;
+      }
+    }
+    if (typeof d.maxConcurrent === 'number' && d.maxConcurrent >= 0) cfg.maxConcurrent = d.maxConcurrent;
+    else if (d.maxConcurrent && typeof d.maxConcurrent === 'object') {
+      const obj = d.maxConcurrent as { global?: unknown; perHarness?: unknown };
+      const out: { global?: number; perHarness?: Record<string, number> } = {};
+      if (typeof obj.global === 'number' && obj.global >= 0) out.global = obj.global;
+      if (obj.perHarness && typeof obj.perHarness === 'object') {
+        const ph: Record<string, number> = {};
+        for (const [k, v] of Object.entries(obj.perHarness as Record<string, unknown>)) {
+          if (typeof v === 'number' && v >= 0) ph[k] = v;
+        }
+        if (Object.keys(ph).length > 0) out.perHarness = ph;
+      }
+      if (out.global !== undefined || out.perHarness) cfg.maxConcurrent = out;
+    }
+    if (typeof d.maxTranscripts === 'number' && d.maxTranscripts >= 0) cfg.maxTranscripts = d.maxTranscripts;
+    if (d.harnesses && typeof d.harnesses === 'object') {
+      for (const [k, v] of Object.entries(d.harnesses)) {
+        if (v && typeof v === 'object') cfg.harnesses[k] = { ...(v as HarnessConfig) };
+      }
+    }
+    // also support legacy claudeDelegate merged when delegate also present (delegate wins)
+    if (settings.claudeDelegate) {
+      const c = settings.claudeDelegate as Partial<DelegateConfig>;
+      if (typeof c.model === 'string' && !cfg.harnesses['claude']?.model) {
+        cfg.harnesses['claude'] = { ...(cfg.harnesses['claude'] ?? {}), model: c.model };
+      }
+    }
+  } catch {
+    // invalid settings — fall back to defaults
+  }
+  return cfg;
 }
 
-export function resolveModelForHarness(cfg: DelegateConfig, harness: string, model?: string, templateModel?: string): string | undefined {
-	const resolve = (m?: string) => (m ? (cfg.modelAliases[m] ?? m) : undefined);
-	return resolve(model) ?? resolve(templateModel) ?? resolve(cfg.harnesses[harness]?.model) ?? resolve(cfg.model);
+export function resolveModelForHarness(
+  cfg: DelegateConfig,
+  harness: string,
+  model?: string,
+  templateModel?: string,
+): string | undefined {
+  const resolve = (m?: string) => (m ? (cfg.modelAliases[m] ?? m) : undefined);
+  return resolve(model) ?? resolve(templateModel) ?? resolve(cfg.harnesses[harness]?.model) ?? resolve(cfg.model);
 }
 
 export function getMaxConcurrent(cfg: DelegateConfig, harness?: string): number {
-	if (typeof cfg.maxConcurrent === 'number') return cfg.maxConcurrent;
-	// if object shape {global, perHarness}
-	const mc = cfg.maxConcurrent as unknown as { global?: number; perHarness?: Record<string, number> };
-	if (harness && mc.perHarness && typeof mc.perHarness[harness] === 'number') return mc.perHarness[harness]!;
-	if (typeof mc.global === 'number') return mc.global;
-	return 1;
+  if (typeof cfg.maxConcurrent === 'number') return cfg.maxConcurrent;
+  // if object shape {global, perHarness}
+  const mc = cfg.maxConcurrent as unknown as { global?: number; perHarness?: Record<string, number> };
+  if (harness && mc.perHarness && typeof mc.perHarness[harness] === 'number') return mc.perHarness[harness]!;
+  if (typeof mc.global === 'number') return mc.global;
+  return 1;
 }

package/extensions/harnesses/amp.ts

@@ -1,137 +1,144 @@
 import { execFile } from 'node:child_process';
 import { promisify } from 'node:util';
-import type { Harness, BuildArgsOpts, NormalizedPermission, ParseOutcome, ParseState, StreamedResult } from './types.ts';
+import type {
+  BuildArgsOpts,
+  Harness,
+  NormalizedPermission,
+  ParseOutcome,
+  ParseState,
+  StreamedResult,
+} from './types.ts';
 
 const execFileAsync = promisify(execFile);
 
 function isRecord(v: unknown): v is Record<string, unknown> {
-	return typeof v === 'object' && v !== null && !Array.isArray(v);
+  return typeof v === 'object' && v !== null && !Array.isArray(v);
 }
 
 const PERMISSION_MAP: Record<NormalizedPermission, string> = {
-	readonly: 'read-only',
-	edit: 'workspace',
-	danger: 'danger',
+  readonly: 'read-only',
+  edit: 'workspace',
+  danger: 'danger',
 };
 
 function extractAmpText(o: Record<string, unknown>): string | undefined {
-	if (typeof o.text === 'string') return o.text;
-	if (typeof o.output === 'string') return o.output;
-	if (typeof o.delta === 'string') return o.delta;
-	if (typeof o.content === 'string') return o.content;
-	return undefined;
+  if (typeof o.text === 'string') return o.text;
+  if (typeof o.output === 'string') return o.output;
+  if (typeof o.delta === 'string') return o.delta;
+  if (typeof o.content === 'string') return o.content;
+  return undefined;
 }
 
 export function parseAmpLine(line: string, state: ParseState): ParseOutcome {
-	let o: unknown;
-	try {
-		o = JSON.parse(line);
-	} catch {
-		if (line.trim().length > 0) return { streamedText: line + '\n', activities: [] };
-		return { activities: [] };
-	}
-	if (!isRecord(o)) return { activities: [] };
-	const activities: ParseOutcome['activities'] = [];
-	let streamedText: string | undefined;
-	const typeStr = typeof o.type === 'string' ? o.type : '';
+  let o: unknown;
+  try {
+    o = JSON.parse(line);
+  } catch {
+    if (line.trim().length > 0) return { streamedText: line + '\n', activities: [] };
+    return { activities: [] };
+  }
+  if (!isRecord(o)) return { activities: [] };
+  const activities: ParseOutcome['activities'] = [];
+  let streamedText: string | undefined;
+  const typeStr = typeof o.type === 'string' ? o.type : '';
 
-	if (typeStr.includes('tool') || o.type === 'tool_use') {
-		const name = typeof o.name === 'string' ? o.name : 'tool';
-		if (typeStr.includes('start') || o.type === 'tool_use') {
-			activities.push({ kind: 'tool_start', name });
-			if (isRecord(o.input)) activities.push({ kind: 'tool_input', name, input: o.input as Record<string, unknown> });
-		} else {
-			activities.push({ kind: 'tool_result', isError: o.is_error === true });
-		}
-	}
-	if (typeStr.includes('thinking')) activities.push({ kind: 'thinking', chars: 10 });
+  if (typeStr.includes('tool') || o.type === 'tool_use') {
+    const name = typeof o.name === 'string' ? o.name : 'tool';
+    if (typeStr.includes('start') || o.type === 'tool_use') {
+      activities.push({ kind: 'tool_start', name });
+      if (isRecord(o.input)) activities.push({ kind: 'tool_input', name, input: o.input as Record<string, unknown> });
+    } else {
+      activities.push({ kind: 'tool_result', isError: o.is_error === true });
+    }
+  }
+  if (typeStr.includes('thinking')) activities.push({ kind: 'thinking', chars: 10 });
 
-	const text = extractAmpText(o);
-	if (text && !typeStr.includes('tool')) streamedText = text;
+  const text = extractAmpText(o);
+  if (text && !typeStr.includes('tool')) streamedText = text;
 
-	if (o.type === 'result' || o.type === 'done' || o.type === 'completed') {
-		const usage = isRecord(o.usage) ? o.usage : null;
-		const result: StreamedResult = {
-			result: typeof o.result === 'string' ? o.result : state.streamedText + (streamedText ?? ''),
-			isError: o.is_error === true,
-			numTurns: typeof o.num_turns === 'number' ? o.num_turns : 0,
-			totalCostUsd: typeof o.total_cost_usd === 'number' ? o.total_cost_usd : 0,
-			sessionId: typeof o.session_id === 'string' ? o.session_id : null,
-			stopReason: typeof o.stop_reason === 'string' ? o.stop_reason : null,
-			permissionDenials: [],
-			durationMs: typeof o.duration_ms === 'number' ? o.duration_ms : null,
-			durationApiMs: null,
-			ttftMs: null,
-			model: typeof o.model === 'string' ? o.model : null,
-			contextWindow: null,
-			maxOutputTokens: null,
-			usage: usage
-				? {
-						inputTokens: typeof usage.input_tokens === 'number' ? usage.input_tokens : 0,
-						outputTokens: typeof usage.output_tokens === 'number' ? usage.output_tokens : 0,
-						cacheCreationInputTokens: 0,
-						cacheReadInputTokens: 0,
-					}
-				: null,
-		};
-		return { activities, streamedText, result };
-	}
-	return { activities, streamedText };
+  if (o.type === 'result' || o.type === 'done' || o.type === 'completed') {
+    const usage = isRecord(o.usage) ? o.usage : null;
+    const result: StreamedResult = {
+      result: typeof o.result === 'string' ? o.result : state.streamedText + (streamedText ?? ''),
+      isError: o.is_error === true,
+      numTurns: typeof o.num_turns === 'number' ? o.num_turns : 0,
+      totalCostUsd: typeof o.total_cost_usd === 'number' ? o.total_cost_usd : 0,
+      sessionId: typeof o.session_id === 'string' ? o.session_id : null,
+      stopReason: typeof o.stop_reason === 'string' ? o.stop_reason : null,
+      permissionDenials: [],
+      durationMs: typeof o.duration_ms === 'number' ? o.duration_ms : null,
+      durationApiMs: null,
+      ttftMs: null,
+      model: typeof o.model === 'string' ? o.model : null,
+      contextWindow: null,
+      maxOutputTokens: null,
+      usage: usage
+        ? {
+            inputTokens: typeof usage.input_tokens === 'number' ? usage.input_tokens : 0,
+            outputTokens: typeof usage.output_tokens === 'number' ? usage.output_tokens : 0,
+            cacheCreationInputTokens: 0,
+            cacheReadInputTokens: 0,
+          }
+        : null,
+    };
+    return { activities, streamedText, result };
+  }
+  return { activities, streamedText };
 }
 
 export const ampHarness: Harness = {
-	name: 'amp',
-	displayName: 'Amp',
-	binary: 'amp',
-	async detect() {
-		try {
-			const { stdout } = await execFileAsync('amp', ['--version'], { timeout: 5000 });
-			return { ok: true, version: stdout.trim() };
-		} catch {
-			try {
-				const { stdout } = await execFileAsync('omp', ['--version'], { timeout: 5000 });
-				return { ok: true, version: stdout.trim() };
-			} catch {
-				return { ok: false, hint: 'Install Amp: https://ampcode.com' };
-			}
-		}
-	},
-	buildArgs(opts: BuildArgsOpts): string[] {
-		const perm = opts.nativePermission ?? PERMISSION_MAP[opts.permission] ?? 'workspace';
-		const args = ['--output', 'jsonl', opts.prompt, '--permission', perm];
-		if (opts.model) args.push('--model', opts.model);
-		if (opts.resumeSessionId) args.push('--resume', opts.resumeSessionId);
-		for (const dir of opts.addDirs ?? []) args.push('--add-dir', dir);
-		return args;
-	},
-	parseLine(line: string, state: ParseState): ParseOutcome {
-		return parseAmpLine(line, state);
-	},
-	extractResult(state: ParseState): StreamedResult | null {
-		if (state.result) return state.result;
-		if (state.streamedText.trim().length > 0) {
-			return {
-				result: state.streamedText,
-				isError: false,
-				numTurns: 1,
-				totalCostUsd: 0,
-				sessionId: null,
-				stopReason: null,
-				permissionDenials: [],
-				durationMs: null,
-				durationApiMs: null,
-				ttftMs: null,
-				model: null,
-				contextWindow: null,
-				maxOutputTokens: null,
-				usage: null,
-			};
-		}
-		return null;
-	},
-	permissionMap: {
-		readonly: ['read-only'],
-		edit: ['workspace'],
-		danger: ['danger'],
-	},
+  name: 'amp',
+  displayName: 'Amp',
+  binary: 'amp',
+  async detect() {
+    try {
+      const { stdout } = await execFileAsync('amp', ['--version'], { timeout: 5000 });
+      return { ok: true, version: stdout.trim() };
+    } catch {
+      try {
+        const { stdout } = await execFileAsync('omp', ['--version'], { timeout: 5000 });
+        return { ok: true, version: stdout.trim() };
+      } catch {
+        return { ok: false, hint: 'Install Amp: https://ampcode.com' };
+      }
+    }
+  },
+  buildArgs(opts: BuildArgsOpts): string[] {
+    const perm = opts.nativePermission ?? PERMISSION_MAP[opts.permission] ?? 'workspace';
+    const args = ['--output', 'jsonl', opts.prompt, '--permission', perm];
+    if (opts.model) args.push('--model', opts.model);
+    if (opts.resumeSessionId) args.push('--resume', opts.resumeSessionId);
+    for (const dir of opts.addDirs ?? []) args.push('--add-dir', dir);
+    return args;
+  },
+  parseLine(line: string, state: ParseState): ParseOutcome {
+    return parseAmpLine(line, state);
+  },
+  extractResult(state: ParseState): StreamedResult | null {
+    if (state.result) return state.result;
+    if (state.streamedText.trim().length > 0) {
+      return {
+        result: state.streamedText,
+        isError: false,
+        numTurns: 1,
+        totalCostUsd: 0,
+        sessionId: null,
+        stopReason: null,
+        permissionDenials: [],
+        durationMs: null,
+        durationApiMs: null,
+        ttftMs: null,
+        model: null,
+        contextWindow: null,
+        maxOutputTokens: null,
+        usage: null,
+      };
+    }
+    return null;
+  },
+  permissionMap: {
+    readonly: ['read-only'],
+    edit: ['workspace'],
+    danger: ['danger'],
+  },
 };