/**
 * Mapping from pi-review reviewer ids → pi-subagents runtime agent names,
 * plus default turn/tool budgets for the token-lean directive path.
 *
 * Agents live in `agents/*.md` and are registered via package.json
 * `pi.subagents.agents` so pi-subagents discovers them as package agents
 * (`pi-review.<id>`).
 */

export const LEAN_AGENT_PACKAGE = "pi-review";

/** Runtime agent name for a reviewer id (e.g. bugbot → pi-review.bugbot). */
export function leanAgentName(reviewerId: string): string {
	return `${LEAN_AGENT_PACKAGE}.${reviewerId}`;
}

/** Gate agent runtime name. */
export const LEAN_GATE_AGENT = leanAgentName("gate");

export interface ToolBudgetSpec {
	soft: number;
	hard: number;
}

export interface LeanBudgetSpec {
	/** Top-level turn budget for the parallel fan-out call. */
	turnBudget: { maxTurns: number; graceTurns: number };
	/** Per-task tool budget (overrides by reviewer id when present). */
	defaultToolBudget: ToolBudgetSpec;
	/** Stricter budget for history-context. */
	historyToolBudget: ToolBudgetSpec;
	/** Gate subagent budgets. */
	gateTurnBudget: { maxTurns: number; graceTurns: number };
	gateToolBudget: ToolBudgetSpec;
	/** Wall-clock timeout for the fan-out / gate calls (ms). */
	timeoutMs: number;
}

/** Defaults (v0.5.2): more headroom; shallow prompts keep real usage lower. */
export const LEAN_BUDGETS: LeanBudgetSpec = {
	turnBudget: { maxTurns: 20, graceTurns: 2 },
	defaultToolBudget: { soft: 20, hard: 32 },
	historyToolBudget: { soft: 14, hard: 24 },
	gateTurnBudget: { maxTurns: 6, graceTurns: 1 },
	gateToolBudget: { soft: 5, hard: 10 },
	timeoutMs: 600_000,
};

export function toolBudgetForReviewer(id: string): ToolBudgetSpec {
	if (id === "history-context") return LEAN_BUDGETS.historyToolBudget;
	return LEAN_BUDGETS.defaultToolBudget;
}

/** Merge optional config.budgets.turnBudget over defaults. */
export function resolveLeanBudgets(override?: {
	turnBudget?: { maxTurns?: number; graceTurns?: number };
}): LeanBudgetSpec {
	const base = { ...LEAN_BUDGETS, turnBudget: { ...LEAN_BUDGETS.turnBudget } };
	if (override?.turnBudget?.maxTurns != null && override.turnBudget.maxTurns >= 1) {
		base.turnBudget.maxTurns = Math.min(48, Math.floor(override.turnBudget.maxTurns));
	}
	if (override?.turnBudget?.graceTurns != null && override.turnBudget.graceTurns >= 0) {
		base.turnBudget.graceTurns = Math.floor(override.turnBudget.graceTurns);
	}
	return base;
}

/** Append :thinking to a model id when thinking is set (gate path). */
export function withThinkingSuffix(model: string, thinking?: string): string {
	if (!thinking || thinking === "off" || thinking === "false") return model;
	const colon = model.lastIndexOf(":");
	const known = ["minimal", "low", "medium", "high", "xhigh", "max", "min"];
	if (colon !== -1 && known.includes(model.slice(colon + 1))) {
		return `${model.slice(0, colon)}:${thinking}`;
	}
	return `${model}:${thinking}`;
}

/** Shared false-positive list (injected once into the directive). */
export const FALSE_POSITIVE_GUIDANCE = [
	"Pre-existing issues on lines the author did not modify",
	"Pedantic nitpicks a senior engineer would not call out",
	"Issues a linter, typechecker, or CI would catch",
	"Generic quality (missing tests/docs) unless a project rule explicitly requires it",
	"Something that looks like a bug but is intentional given the change",
].join("; ");