pi-crew 0.9.51 → 0.9.53

package/src/config/types.ts

@@ -1,278 +0,0 @@
-// 2.9 — config interface types extracted from src/config/config.ts.
-//
-// All public surface types live here so that hot-path callers (loadConfig,
-// merging helpers, schema validators) can import just the types without
-// pulling in the parser graph. config.ts re-exports every name from this
-// file for backwards compat — existing `import { CrewUiConfig } from "../config/config.ts"`
-// continues to work.
-
-export type PiTeamsAutonomyProfile = "manual" | "suggested" | "assisted" | "aggressive";
-
-export interface PiTeamsAutonomousConfig {
-	profile?: PiTeamsAutonomyProfile;
-	enabled?: boolean;
-	injectPolicy?: boolean;
-	preferAsyncForLongTasks?: boolean;
-	allowWorktreeSuggestion?: boolean;
-	magicKeywords?: Record<string, string[]>;
-}
-
-export interface CrewLimitsConfig {
-	maxConcurrentWorkers?: number;
-	allowUnboundedConcurrency?: boolean;
-	maxTaskDepth?: number;
-	maxChildrenPerTask?: number;
-	maxRunMinutes?: number;
-	maxRetriesPerTask?: number;
-	maxTasksPerRun?: number;
-	heartbeatStaleMs?: number;
-	/** Round 25 (M5): serialize on write-path overlap. Default false (off).
-	 *  When true, the scheduler skips ready tasks whose declared `step.output`
-	 *  overlaps with already-picked tasks, so two workers never write the same
-	 *  file in parallel. See src/runtime/path-overlap.ts. */
-	serializeOnPathOverlap?: boolean;
-}
-
-export type CrewRuntimeMode = "auto" | "scaffold" | "child-process" | "live-session";
-
-export type CompletionMutationGuardMode = "off" | "warn" | "fail";
-export type EffectivenessGuardMode = "off" | "warn" | "block" | "fail";
-
-export interface CrewRuntimeConfig {
-	mode?: CrewRuntimeMode;
-	preferLiveSession?: boolean;
-	allowChildProcessFallback?: boolean;
-	maxTurns?: number;
-	graceTurns?: number;
-	/**
-	 * W2 fix — wall-clock timeout per task in milliseconds. When the task
-	 * exceeds this limit, input.signal is aborted which triggers the existing
-	 * SIGTERM → SIGKILL escalation in child-pi.ts. Default 0 (no timeout).
-	 * Prevents runaway agent loops (e.g. 11_build in oh-my-pi distill run that
-	 * re-verified completed files 14+ times).
-	 */
-	taskTimeoutMs?: number;
-	inheritContext?: boolean;
-	promptMode?: "replace" | "append";
-	groupJoin?: "off" | "group" | "smart";
-	groupJoinAckTimeoutMs?: number;
-	requirePlanApproval?: boolean;
-	completionMutationGuard?: CompletionMutationGuardMode;
-	effectivenessGuard?: EffectivenessGuardMode;
-	yield?: {
-		enabled?: boolean;
-		maxReminders?: number;
-		reminderPrompt?: string;
-	};
-	/** Policy for per-role runtime selection. Not sensitive — safe to keep in project config. */
-	isolationPolicy?: {
-		/** Roles that should use child-process for crash isolation. Default: no roles. */
-		isolatedRoles?: string[];
-		/** Default runtime for roles not in isolatedRoles. Default: "live-session" (uses live-session). */
-		defaultRuntime?: "live-session" | "child-process";
-	};
-	/** Mark certain bash commands as excludeFromContext to reduce context tokens. Default: false */
-	excludeContextBash?: boolean;
-}
-
-export interface CrewControlConfig {
-	enabled?: boolean;
-	needsAttentionAfterMs?: number;
-}
-
-export interface CrewWorktreeConfig {
-	setupHook?: string;
-	setupHookTimeoutMs?: number;
-	linkNodeModules?: boolean;
-	seedPaths?: string[];
-}
-
-/** Goal-wrap config (RFC v0.5 vision: apply `goal` completion-guarantee to builtin workflows). */
-export interface GoalWrapWorkflowConfig {
-	enabled?: boolean;
-	maxTurns?: number;
-	evaluatorModel?: string;
-	verification?: { commands: string[]; mode?: "text-only" };
-	budgetTotal?: number;
-	budgetUnlimited?: boolean;
-}
-
-export interface CrewUiConfig {
-	widgetPlacement?: "aboveEditor" | "belowEditor";
-	widgetMaxLines?: number;
-	powerbar?: boolean;
-	dashboardPlacement?: "center" | "right";
-	dashboardWidth?: number;
-	dashboardLiveRefreshMs?: number;
-	autoOpenDashboard?: boolean;
-	autoOpenDashboardForForegroundRuns?: boolean;
-	autoCloseDashboardMs?: number;
-	showModel?: boolean;
-	showTokens?: boolean;
-	showTools?: boolean;
-	transcriptTailBytes?: number;
-	mascotStyle?: "cat" | "armin";
-	mascotEffect?: "random" | "none" | "typewriter" | "scanline" | "rain" | "fade" | "crt" | "glitch" | "dissolve";
-}
-
-export interface AgentOverrideConfig {
-	disabled?: boolean;
-	model?: string | false;
-	fallbackModels?: string[] | false;
-	thinking?: string | false;
-	tools?: string[] | false;
-	skills?: string[] | false;
-}
-
-export interface CrewAgentsConfig {
-	disableBuiltins?: boolean;
-	overrides?: Record<string, AgentOverrideConfig>;
-}
-
-export interface CrewToolsConfig {
-	enableClaudeStyleAliases?: boolean;
-	enableSteer?: boolean;
-	terminateOnForeground?: boolean;
-}
-
-export interface CrewTelemetryConfig {
-	enabled?: boolean;
-}
-
-export interface CrewPolicyConfig {
-	requireIntentForDestructiveActions?: boolean;
-	disabledCapabilities?: string[];
-}
-
-export type CrewNotificationSeverity = "info" | "warning" | "error" | "critical";
-
-export interface CrewNotificationsConfig {
-	enabled?: boolean;
-	severityFilter?: CrewNotificationSeverity[];
-	dedupWindowMs?: number;
-	batchWindowMs?: number;
-	quietHours?: string;
-	sinkRetentionDays?: number;
-}
-
-export interface CrewObservabilityConfig {
-	enabled?: boolean;
-	pollIntervalMs?: number;
-	metricRetentionDays?: number;
-}
-
-export interface CrewRetryPolicyConfig {
-	maxAttempts?: number;
-	backoffMs?: number;
-	jitterRatio?: number;
-	exponentialFactor?: number;
-	retryableErrors?: string[];
-}
-
-export interface CrewReliabilityConfig {
-	autoRetry?: boolean;
-	retryPolicy?: CrewRetryPolicyConfig;
-	autoRecover?: boolean;
-	deadletterThreshold?: number;
-	/** Interval (ms) for periodic stale-run auto-repair. Default 60_000 (60s). Set to 0 to disable. */
-	autoRepairIntervalMs?: number;
-	/** Remove /tmp/pi-crew-* directories after their orphaned runs are reconciled. Default: true. */
-	cleanupOrphanedTempDirs?: boolean;
-	/** Bypass the preflight topology validator (workflow threshold rule, .crew/knowledge.md
-	 *  'pi-crew USAGE THRESHOLD RULE'). When true, runs that would be BLOCKED or WARNED
-	 *  proceed without intervention. Audit-trail the override via events.jsonl.
-	 *  Default: false (enforce). Use only for legitimate audit/debug sessions. */
-	forcePreflight?: boolean;
-	/** Inject a compact ambient crew-status note into the agent's context on every LLM call while crew runs are in-flight, so the agent stays continuously aware of active runs without calling the `team` tool. No-op when no runs are active. Default: true. */
-	ambientStatusInjection?: boolean;
-	/**
-	 * Per-write validation (T5). On every `write`/`edit` tool result, run a
-	 * zero-cost synchronous validator for the file type and append a `🔴`
-	 * blocker to the tool result on failure (e.g. malformed JSON). v1 ships
-	 * JSON only (`JSON.parse` — instant, no process spawn); process-spawning
-	 * validators (.js/.sh/.py) are a future opt-in. Default: true (opt-out).
-	 * Set to `false` to disable.
-	 */
-	perWriteValidation?: boolean;
-	/**
-	 * Opt-in model scope enforcement (F7). When true, subagent model choices
-	 * that fall outside the user's pi `enabledModels` allowlist are flagged:
-	 * caller-supplied out-of-scope → hard error before spawn; frontmatter-
-	 * pinned out-of-scope → warning + runs anyway. Default: false (no
-	 * enforcement, fully back-compat).
-	 */
-	scopeModels?: boolean;
-}
-
-export interface CrewOtlpConfig {
-	enabled?: boolean;
-	endpoint?: string;
-	headers?: Record<string, string>;
-	intervalMs?: number;
-}
-
-export interface PiTeamsConfig {
-	asyncByDefault?: boolean;
-	executeWorkers?: boolean;
-	notifierIntervalMs?: number;
-	requireCleanWorktreeLeader?: boolean;
-	ignoreMethod?: "gitignore" | "exclude";
-	autonomous?: PiTeamsAutonomousConfig;
-	limits?: CrewLimitsConfig;
-	runtime?: CrewRuntimeConfig;
-	control?: CrewControlConfig;
-	worktree?: CrewWorktreeConfig;
-	goalWrap?: Record<string, GoalWrapWorkflowConfig>;
-	agents?: CrewAgentsConfig;
-	tools?: CrewToolsConfig;
-	telemetry?: CrewTelemetryConfig;
-	policy?: CrewPolicyConfig;
-	notifications?: CrewNotificationsConfig;
-	observability?: CrewObservabilityConfig;
-	reliability?: CrewReliabilityConfig;
-	otlp?: CrewOtlpConfig;
-	ui?: CrewUiConfig;
-	/**
-	 * Inter-pi broker (Phase 0). Local-only socket transport between parent
-	 * and child pi workers. Default is OFF (`enabled:false`) — keep it that
-	 * way until Phase 4 soak completes. Numeric limits are bounded by the
-	 * schema (`src/schema/config-schema.ts`).
-	 */
-	broker?: CrewBrokerConfig;
-}
-
-/** CrewBroker config (Phase 0 inter-pi broker). */
-export interface CrewBrokerConfig {
-	/** Master switch. `false` keeps the broker fully dormant. */
-	enabled?: boolean;
-	/** Length of the SHA-256 hex prefix used in the socket filename. 4..32. */
-	pathHashLen?: number;
-	/** Maximum NDJSON frame size in UTF-8 bytes (default 262144 = 256 KiB). 1024..1048576. */
-	maxFrameBytes?: number;
-	/** Per-connection outbound queue cap. 32..4096 (default 256). */
-	outboundQueueCap?: number;
-}
-
-export interface LoadedPiTeamsConfig {
-	config: PiTeamsConfig;
-	path: string;
-	paths: string[];
-	error?: string;
-	warnings?: string[];
-}
-
-export interface ConfigValidationResult {
-	config: PiTeamsConfig;
-	warnings: string[];
-}
-
-export interface SavedPiTeamsConfig {
-	config: PiTeamsConfig;
-	path: string;
-}
-
-export interface UpdateConfigOptions {
-	cwd?: string;
-	scope?: "user" | "project";
-	unsetPaths?: string[];
-}

package/src/errors.ts

@@ -1,194 +0,0 @@
-// pi-crew structured error module — taxonomy mapping E001–E006.
-/**
- * @fileoverview Error types and structured error handling for pi-crew.
- *
- * This module defines pi-crew's error taxonomy, mapping to semantic categories
- * matching fallow's E001-E004 pattern. It exports three main constructs:
- *
- * - {@link ErrorCode} — a `const` object and string-literal union type alias
- *   enumerating machine-readable error codes (E001–E006). Implemented as a
- *   `const` object rather than a TypeScript `enum` so that Node's
- *   `--experimental-strip-types` can load this module (enum syntax is not
- *   supported in strip-only mode).
- *
- * - {@link CrewError} — a structured `Error` subclass that carries an
- *   {@link ErrorCode}, an optional human-readable `help` hint, and an optional
- *   `context` string. Its `toString()` renders the display format:
- *   `error[E001]: Failed to read manifest.json: not found`.
- *
- * - {@link errors} — a factory object exposing convenience constructors
- *   (e.g. `errors.fileRead`, `errors.taskNotFound`) that build `CrewError`
- *   instances with sensible defaults and pre-attached context.
- */
-// Implemented as const object + type alias (not `enum`) so that Node's
-// `--experimental-strip-types` can load this module. TypeScript `enum`
-// syntax is not supported in strip-only mode.
-export const ErrorCode = {
-	FileReadError: "E001", // Cannot read a file
-	FileWriteError: "E002", // Cannot write a file
-	TaskNotFound: "E003", // Referenced task ID does not exist
-	InvalidStatusTransition: "E004", // Run/task status cannot legally transition
-	ConfigError: "E005", // Malformed config or missing required field
-	ResourceNotFound: "E006", // Agent/team/workflow not found in discovery paths
-	// E1 (Round 15): runtime failure categories that previously threw raw Error
-	// with no code, no help hint, and no context. Surfaces actionable guidance.
-	ChildTimeout: "E007", // Child Pi worker became unresponsive and was killed
-	ModelExhausted: "E008", // All model candidates in the fallback chain failed
-	PreStepFailed: "E009", // A pre-step hook script returned a non-zero exit
-	EventLogLockTimeout: "E010", // Could not acquire the event-log file lock
-	DepthLimitExceeded: "E011", // Pipeline/chain recursion depth limit hit (circular dep)
-	RunStale: "E012", // Run reconciled as stale/zombie (heartbeat expired)
-	ModelOutOfScope: "E013", // Caller-supplied model is not in pi's enabledModels allowlist (F7 scope gate)
-} as const;
-
-export type ErrorCode = (typeof ErrorCode)[keyof typeof ErrorCode];
-
-const DEFAULT_HELP: Record<ErrorCode, string | undefined> = {
-	[ErrorCode.FileReadError]: "Check that the file exists and that the process has read permission.",
-	[ErrorCode.FileWriteError]: "Check that the disk is not full and that the process has write permission.",
-	[ErrorCode.TaskNotFound]: "The task may have been removed or the run may be in an inconsistent state. Use `team status` to verify.",
-	[ErrorCode.InvalidStatusTransition]: "Verify the run status using `team status` before retrying.",
-	[ErrorCode.ConfigError]: "Check the configuration file for syntax errors or missing required fields.",
-	[ErrorCode.ResourceNotFound]: "Use `team list` to see available agents, teams, and workflows.",
-	// E1 (Round 15): help hints for the new runtime categories.
-	[ErrorCode.ChildTimeout]:
-		"The child Pi worker produced no output for too long and was terminated. Re-run the team; if it recurs, raise the response timeout in config or reduce the task scope.",
-	[ErrorCode.ModelExhausted]:
-		"Every model in the fallback chain failed. Check your API key/quota and the per-attempt errors, then retry or swap the model in config.",
-	[ErrorCode.PreStepFailed]:
-		"The pre-step hook script exited non-zero. Inspect its stderr, or mark it optional in the workflow step (preStepOptional).",
-	[ErrorCode.EventLogLockTimeout]:
-		"Another process holds the event-log lock. Check for orphaned `.lock` files or stale pi-crew processes, then retry.",
-	[ErrorCode.DepthLimitExceeded]:
-		"A pipeline/chain exceeded the recursion depth limit, which usually indicates a circular stage dependency. Review step `dependsOn` chains.",
-	[ErrorCode.RunStale]:
-		"The worker stopped heartbeating and was treated as a zombie. Re-run the team (resume or fresh); if it recurs, check `runtime.executeWorkers` / system load.",
-	[ErrorCode.ModelOutOfScope]:
-		"The requested model is not in your pi `enabledModels` allowlist. Either pick a model listed in `enabledModels` (settings.json) or extend the allowlist. The scope gate is opt-in — disable `runtime.reliability.scopeModels` to allow any model.",
-};
-
-/**
- * Structured error type for pi-crew.
- * Display format:
- *   error[E001]: Failed to read manifest.json: not found
- *     context: while loading run state
- *     help: Check that the file exists and that the process has read permission.
- */
-export class CrewError extends Error {
-	readonly code: ErrorCode;
-	help?: string;
-	private _context?: string;
-
-	constructor(code: ErrorCode, message: string, help?: string) {
-		super(message);
-		this.name = "CrewError";
-		this.code = code;
-		this.help = help ?? DEFAULT_HELP[code];
-		Object.defineProperty(this, "message", { enumerable: true });
-		Object.defineProperty(this, "code", { enumerable: true });
-	}
-
-	withContext(context: string): this {
-		this._context = context;
-		return this;
-	}
-
-	withHelp(help: string): this {
-		this.help = help;
-		return this;
-	}
-
-	toString(): string {
-		let out = `error[${this.code}]: ${this.message}`;
-		if (this._context) out += `\n  context: ${this._context}`;
-		if (this.help) out += `\n  help: ${this.help}`;
-		return out;
-	}
-}
-
-export const errors = {
-	fileRead(path: string, source: NodeJS.ErrnoException): CrewError {
-		return new CrewError(ErrorCode.FileReadError, `Failed to read ${path}: ${source.code?.toLowerCase() ?? "unknown"}`).withContext(
-			"file system read operation",
-		);
-	},
-
-	fileWrite(path: string, source: NodeJS.ErrnoException): CrewError {
-		return new CrewError(ErrorCode.FileWriteError, `Failed to write ${path}: ${source.code?.toLowerCase() ?? "unknown"}`).withContext(
-			"file system write operation",
-		);
-	},
-
-	taskNotFound(taskId: string, runId?: string): CrewError {
-		const msg = runId ? `Task '${taskId}' not found in run '${runId}'` : `Task '${taskId}' not found`;
-		return new CrewError(ErrorCode.TaskNotFound, msg);
-	},
-
-	invalidStatusTransition(from: string, to: string): CrewError {
-		return new CrewError(ErrorCode.InvalidStatusTransition, `Invalid run status transition: ${from} -> ${to}`);
-	},
-
-	config(message: string): CrewError {
-		return new CrewError(ErrorCode.ConfigError, message).withContext("configuration loading");
-	},
-
-	resourceNotFound(type: string, name: string): CrewError {
-		return new CrewError(ErrorCode.ResourceNotFound, `${type} '${name}' not found in any discovery path`);
-	},
-
-	// E1 (Round 15): runtime failure constructors. These wrap the raw-throw
-	// sites identified in the Round 15 error-experience audit so failures carry
-	// a machine-readable code, a help hint, and structured context.
-	childTimeout(detail: { timeoutMs?: number; taskId?: string; stderr?: string }): CrewError {
-		const tail = detail.stderr ? ` Stderr tail: ${detail.stderr.slice(-400)}` : "";
-		const dur = detail.timeoutMs ? ` after ${detail.timeoutMs}ms of no output` : "";
-		return new CrewError(ErrorCode.ChildTimeout, `Child Pi worker became unresponsive${dur} and was terminated.${tail}`).withContext(
-			`worker execution${detail.taskId ? ` (task ${detail.taskId})` : ""}`,
-		);
-	},
-
-	modelExhausted(chain: string[], lastFailure?: string): CrewError {
-		const tried = chain.join(" → ");
-		const last = lastFailure ? ` Last failure: ${lastFailure}` : "";
-		return new CrewError(
-			ErrorCode.ModelExhausted,
-			`All ${chain.length} model candidates exhausted (tried: ${tried}).${last}`,
-		).withContext("model fallback chain");
-	},
-
-	preStepFailed(script: string, exitCode: number | undefined, stderr?: string): CrewError {
-		const tail = stderr ? ` Stderr: ${stderr.slice(-400)}` : "";
-		return new CrewError(ErrorCode.PreStepFailed, `preStepScript '${script}' exited ${exitCode ?? "non-zero"}.${tail}`).withContext(
-			"pre-step hook execution",
-		);
-	},
-
-	eventLogLockTimeout(eventsPath: string, timeoutMs: number): CrewError {
-		return new CrewError(
-			ErrorCode.EventLogLockTimeout,
-			`Event log lock timeout for ${eventsPath}: could not acquire lock within ${timeoutMs}ms`,
-		).withContext("event-log append");
-	},
-
-	depthLimitExceeded(depth: number, kind = "pipeline"): CrewError {
-		return new CrewError(
-			ErrorCode.DepthLimitExceeded,
-			`${kind[0].toUpperCase() + kind.slice(1)} recursion depth limit exceeded (${depth}). Possible circular dependency.`,
-		).withContext(`${kind} execution`);
-	},
-
-	runStale(reason: string, heartbeatAgeSeconds?: number): CrewError {
-		const age = heartbeatAgeSeconds !== undefined ? ` Last heartbeat was ${heartbeatAgeSeconds}s ago.` : "";
-		return new CrewError(
-			ErrorCode.RunStale,
-			`Stale run reconciled (reason=${reason}).${age} The worker stopped heartbeating and was treated as dead/zombie.`,
-		).withContext("stale-run reconciliation");
-	},
-
-	modelOutOfScope(model: string, patterns: string[]): CrewError {
-		return new CrewError(
-			ErrorCode.ModelOutOfScope,
-			`Requested model "${model}" is not in enabledModels scope (allowlist: [${patterns.join(", ")}])`,
-		).withContext("F7 model scope gate — caller override rejected");
-	},
-} as const;

package/src/extension/action-suggestions.ts

@@ -1,117 +0,0 @@
-/**
- * action-suggestions.ts — "Did you mean?" suggestions for team actions (DX: F1).
- *
- * Round 16 DX audit found that a typo'd action (`action: 'stat'`,
- * `action: 'summery'`) hits a dead-end "Unknown action: stat" with no path
- * forward. pi-crew already ships a Levenshtein fuzzy-matcher
- * (`src/config/suggestions.ts → suggestConfigKey`); this module applies it to
- * the known set of team actions.
- *
- * The known-action list mirrors the `action` enum in
- * `src/schema/team-tool-schema.ts`. Kept as a hand-maintained constant (not
- * derived from the TypeBox schema at runtime) so it is trivially testable and
- * avoids pulling the schema into low-level error paths.
- */
-
-import { findClosestKey } from "../config/suggestions.ts";
-
-/**
- * The complete set of valid top-level `team` actions (mirrors the action enum
- * in `src/schema/team-tool-schema.ts`). Exported so callers and tests can use
- * the single source of truth.
- */
-export const KNOWN_TEAM_ACTIONS = [
-	"run",
-	"parallel",
-	"plan",
-	"status",
-	"wait",
-	"list",
-	"get",
-	"cancel",
-	"retry",
-	"resume",
-	"respond",
-	"create",
-	"update",
-	"delete",
-	"doctor",
-	"cleanup",
-	"events",
-	"artifacts",
-	"worktrees",
-	"forget",
-	"summary",
-	"prune",
-	"export",
-	"import",
-	"imports",
-	"help",
-	"validate",
-	"config",
-	"init",
-	"recommend",
-	"autonomy",
-	"api",
-	"settings",
-	"steer",
-	"invalidate",
-	"health",
-	"graph",
-	"onboard",
-	"explain",
-	"cache",
-	"checkpoint",
-	"search",
-	"orchestrate",
-	"schedule",
-	"scheduled",
-	"anchor",
-	"auto-summarize",
-	"auto_boomerang",
-	"goal",
-	"workflow-create",
-	"workflow-get",
-	"workflow-list",
-	"workflow-save",
-	"workflow-delete",
-] as const;
-
-/**
- * Suggest the closest known team action for a (likely typo'd) input.
- * Returns `null` when no action is close enough — callers should then omit
- * the "Did you mean …?" hint rather than suggesting a poor match.
- *
- * Uses a tighter edit-distance budget than the generic config-key suggester
- * (2 instead of 3): team actions are short command words, so distance-3
- * matches against a short input (e.g. "" → "run") produce low-quality hints.
- * Empty/whitespace input always returns null.
- *
- * Exported for unit testing.
- */
-export function suggestAction(input: string): string | null {
-	const trimmed = input.trim();
-	if (!trimmed) return null;
-	// Defense-in-depth (Round 18 security F1): levenshtein is O(n×m). A hostile
-	// very-long input would waste cycles. The action is enum-validated upstream
-	// so this is unreachable in practice, but cap input length cheaply.
-	if (trimmed.length > 64) return null;
-	return findClosestKey(trimmed, KNOWN_TEAM_ACTIONS, 2);
-}
-
-/**
- * Build a "Did you mean?" suffix for an unknown-action error message.
- * Returns "" when there is no good suggestion (so the caller can just append
- * it unconditionally). Keeps error formatting centralized.
- *
- * Exported for unit testing + use in the dispatch default-case.
- *
- * Example:
- *   formatActionSuggestion("stat")    // "\n\nDid you mean 'status'? Use action='status'."
- *   formatActionSuggestion("xyzzy")   // ""
- */
-export function formatActionSuggestion(input: string): string {
-	const suggestion = suggestAction(input);
-	if (!suggestion || suggestion === input) return "";
-	return `\n\nDid you mean '${suggestion}'? Use action='${suggestion}'. Run action='help' to see all actions.`;
-}