@oh-my-pi/pi-coding-agent 15.5.15 → 15.7.0

package/src/tools/recipe/runners/just.ts

@@ -1,73 +0,0 @@
-import * as fs from "node:fs/promises";
-import * as path from "node:path";
-import { $which, isEnoent, logger } from "@oh-my-pi/pi-utils";
-import type { DetectedRunner, RunnerTask, TaskRunner } from "../runner";
-
-interface JustDumpRecipeRaw {
-	name?: string;
-	doc?: string | null;
-	private?: boolean;
-	parameters?: Array<{ name?: string }>;
-}
-
-interface JustDump {
-	recipes?: Record<string, JustDumpRecipeRaw>;
-}
-
-const JUSTFILE_NAMES = ["justfile", "Justfile", ".justfile"] as const;
-
-async function hasJustfile(cwd: string): Promise<boolean> {
-	for (const name of JUSTFILE_NAMES) {
-		try {
-			const stat = await fs.stat(path.join(cwd, name));
-			if (stat.isFile()) return true;
-		} catch (err) {
-			if (!isEnoent(err)) throw err;
-		}
-	}
-	return false;
-}
-
-async function dumpJustTasks(cwd: string): Promise<RunnerTask[] | null> {
-	try {
-		const proc = Bun.spawn(["just", "--dump", "--dump-format=json"], {
-			cwd,
-			stdin: "ignore",
-			stdout: "pipe",
-			stderr: "pipe",
-		});
-		const [stdout, exit] = await Promise.all([new Response(proc.stdout).text(), proc.exited]);
-		if (exit !== 0) return null;
-		const dump = JSON.parse(stdout) as JustDump;
-		const tasks: RunnerTask[] = [];
-		for (const recipe of Object.values(dump.recipes ?? {})) {
-			if (!recipe.name || recipe.private) continue;
-			const parameters = (recipe.parameters ?? [])
-				.map(parameter => parameter.name)
-				.filter((name): name is string => typeof name === "string" && name.length > 0);
-			const doc = typeof recipe.doc === "string" && recipe.doc.length > 0 ? recipe.doc : undefined;
-			tasks.push({ name: recipe.name, doc, parameters });
-		}
-		return tasks;
-	} catch (err) {
-		logger.debug("just task detection failed", { error: err instanceof Error ? err.message : String(err) });
-		return null;
-	}
-}
-
-export const justRunner: TaskRunner = {
-	id: "just",
-	label: "Just",
-	async detect(cwd: string): Promise<DetectedRunner | null> {
-		try {
-			if (!$which("just")) return null;
-			if (!(await hasJustfile(cwd))) return null;
-			const tasks = await dumpJustTasks(cwd);
-			if (!tasks || tasks.length === 0) return null;
-			return { id: "just", label: "Just", commandPrefix: "just", tasks };
-		} catch (err) {
-			logger.debug("just runner probe failed", { error: err instanceof Error ? err.message : String(err) });
-			return null;
-		}
-	},
-};

package/src/tools/recipe/runners/make.ts

@@ -1,101 +0,0 @@
-import * as fs from "node:fs/promises";
-import * as path from "node:path";
-import { $which, isEnoent, logger } from "@oh-my-pi/pi-utils";
-import type { DetectedRunner, RunnerTask, TaskRunner } from "../runner";
-
-const MAKEFILE_NAMES = ["Makefile", "makefile", "GNUmakefile"] as const;
-const TARGET_PATTERN = /^(?<name>[A-Za-z_][A-Za-z0-9_-]*)\s*:(?!=).*?(?:##\s*(?<doc>.+))?$/u;
-const PHONY_PATTERN = /^\.PHONY\s*:\s*(?<targets>.*)$/u;
-
-interface MakeTargetInfo {
-	name: string;
-	doc?: string;
-	order: number;
-	phony: boolean;
-}
-
-async function findMakefile(cwd: string): Promise<string | null> {
-	for (const name of MAKEFILE_NAMES) {
-		const candidate = path.join(cwd, name);
-		try {
-			const stat = await fs.stat(candidate);
-			if (stat.isFile()) return candidate;
-		} catch (err) {
-			if (!isEnoent(err)) throw err;
-		}
-	}
-	return null;
-}
-
-function isVariableAssignment(line: string, name: string): boolean {
-	return new RegExp(`^\\s*${name}\\s*[:?+]?=`, "u").test(line);
-}
-
-function parsePhonyTargets(line: string): string[] {
-	const match = PHONY_PATTERN.exec(line);
-	if (!match?.groups?.targets) return [];
-	return match.groups.targets
-		.split(/\s+/u)
-		.map(target => target.trim())
-		.filter(target => /^[A-Za-z_][A-Za-z0-9_-]*$/u.test(target));
-}
-
-function parseMakeTargets(text: string): RunnerTask[] {
-	const targets = new Map<string, MakeTargetInfo>();
-	const phonyTargets: string[] = [];
-	let order = 0;
-
-	for (const line of text.split("\n")) {
-		for (const target of parsePhonyTargets(line)) {
-			if (!phonyTargets.includes(target)) phonyTargets.push(target);
-		}
-
-		const match = TARGET_PATTERN.exec(line);
-		const name = match?.groups?.name;
-		if (!name || name === ".PHONY" || isVariableAssignment(line, name)) continue;
-		if (targets.has(name)) continue;
-		const rawDoc = match?.groups?.doc?.trim();
-		const doc = rawDoc && rawDoc.length > 0 ? rawDoc : undefined;
-		targets.set(name, { name, doc, order, phony: false });
-		order += 1;
-	}
-
-	for (const phony of phonyTargets) {
-		const existing = targets.get(phony);
-		if (existing) {
-			existing.phony = true;
-			continue;
-		}
-		targets.set(phony, { name: phony, order, phony: true });
-		order += 1;
-	}
-
-	const hasPhonyTargets = phonyTargets.length > 0;
-	return [...targets.values()]
-		.sort((left, right) => left.order - right.order)
-		.flatMap(target => {
-			if (!hasPhonyTargets || target.phony) {
-				return [{ name: target.name, doc: target.doc, parameters: [] }];
-			}
-			if (!target.doc) return [];
-			return [{ name: target.name, doc: `${target.doc} (file target)`, parameters: [] }];
-		});
-}
-
-export const makeRunner: TaskRunner = {
-	id: "make",
-	label: "Make",
-	async detect(cwd: string): Promise<DetectedRunner | null> {
-		try {
-			if (!$which("make")) return null;
-			const makefile = await findMakefile(cwd);
-			if (!makefile) return null;
-			const tasks = parseMakeTargets(await Bun.file(makefile).text());
-			if (tasks.length === 0) return null;
-			return { id: "make", label: "Make", commandPrefix: "make", tasks };
-		} catch (err) {
-			logger.debug("make runner probe failed", { error: err instanceof Error ? err.message : String(err) });
-			return null;
-		}
-	},
-};

package/src/tools/recipe/runners/pkg.ts

@@ -1,167 +0,0 @@
-import * as fs from "node:fs/promises";
-import * as path from "node:path";
-import { $which, isEnoent, logger } from "@oh-my-pi/pi-utils";
-import type { DetectedRunner, RunnerTask, TaskRunner } from "../runner";
-
-interface PackageJsonInfo {
-	name?: string;
-	scripts: string[];
-	workspaces: string[];
-}
-
-async function resolvePackageRunner(cwd: string): Promise<string> {
-	const [bunLock, bunLockb, pnpmLock, yarnLock, npmLock, npmShrink] = await Promise.all([
-		isFile(path.join(cwd, "bun.lock")),
-		isFile(path.join(cwd, "bun.lockb")),
-		isFile(path.join(cwd, "pnpm-lock.yaml")),
-		isFile(path.join(cwd, "yarn.lock")),
-		isFile(path.join(cwd, "package-lock.json")),
-		isFile(path.join(cwd, "npm-shrinkwrap.json")),
-	]);
-	if (bunLock || bunLockb) return "bun run";
-	if (pnpmLock) return "pnpm run";
-	if (yarnLock) return "yarn";
-	if (npmLock || npmShrink) return "npm run";
-	if ($which("bun")) return "bun run";
-	return "npm run";
-}
-
-function isRecord(value: unknown): value is Record<string, unknown> {
-	return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-
-function shellQuote(value: string): string {
-	return `'${value.replaceAll("'", `'\\''`)}'`;
-}
-
-async function isFile(filePath: string): Promise<boolean> {
-	try {
-		const stat = await fs.stat(filePath);
-		return stat.isFile();
-	} catch (err) {
-		if (isEnoent(err)) return false;
-		throw err;
-	}
-}
-
-function parseWorkspacePatterns(pkg: Record<string, unknown>): string[] {
-	const { workspaces } = pkg;
-	if (Array.isArray(workspaces)) return workspaces.filter((entry): entry is string => typeof entry === "string");
-	if (isRecord(workspaces) && Array.isArray(workspaces.packages)) {
-		return workspaces.packages.filter((entry): entry is string => typeof entry === "string");
-	}
-	return [];
-}
-
-function normalizeWorkspacePattern(pattern: string): string {
-	const negated = pattern.startsWith("!");
-	const body = negated ? pattern.slice(1) : pattern;
-	const normalizedBody = body.endsWith("package.json") ? body : `${body.replace(/\/+$/u, "")}/package.json`;
-	return negated ? `!${normalizedBody}` : normalizedBody;
-}
-
-async function readPackageJson(filePath: string): Promise<PackageJsonInfo | null> {
-	try {
-		const pkg = (await Bun.file(filePath).json()) as unknown;
-		if (!isRecord(pkg)) return null;
-		const scripts = isRecord(pkg.scripts)
-			? Object.entries(pkg.scripts)
-					.filter((entry): entry is [string, string] => typeof entry[1] === "string" && entry[0].length > 0)
-					.map(([name]) => name)
-			: [];
-		const name = typeof pkg.name === "string" && pkg.name.length > 0 ? pkg.name : undefined;
-		return { name, scripts, workspaces: parseWorkspacePatterns(pkg) };
-	} catch (err) {
-		if (!isEnoent(err)) {
-			logger.debug("package.json script detection failed", {
-				error: err instanceof Error ? err.message : String(err),
-			});
-		}
-		return null;
-	}
-}
-
-async function findWorkspacePackageJsons(cwd: string, patterns: string[]): Promise<string[]> {
-	const includePatterns = patterns.filter(pattern => !pattern.startsWith("!")).map(normalizeWorkspacePattern);
-	const excludePatterns = patterns.filter(pattern => pattern.startsWith("!")).map(normalizeWorkspacePattern);
-
-	const collect = async (pattern: string): Promise<string[]> => {
-		const out: string[] = [];
-		for await (const entry of new Bun.Glob(pattern).scan({ cwd, onlyFiles: true })) {
-			out.push(path.normalize(String(entry)));
-		}
-		return out;
-	};
-
-	const [excludedLists, includedLists] = await Promise.all([
-		Promise.all(excludePatterns.map(pattern => collect(pattern.slice(1)))),
-		Promise.all(includePatterns.map(pattern => collect(pattern))),
-	]);
-	const excluded = new Set<string>(excludedLists.flat());
-	const files = new Set<string>();
-	for (const entry of includedLists.flat()) {
-		if (entry !== "package.json" && !excluded.has(entry)) files.add(entry);
-	}
-	return [...files].sort((left, right) => left.localeCompare(right));
-}
-
-function packageTaskName(packageName: string | undefined, packageDir: string, scriptName: string): string {
-	return `${packageName ?? packageDir}/${scriptName}`;
-}
-
-function tasksForPackage(options: { pkg: PackageJsonInfo; packageDir: string; namespaced: boolean }): RunnerTask[] {
-	return options.pkg.scripts.map(scriptName => ({
-		name: options.namespaced ? packageTaskName(options.pkg.name, options.packageDir, scriptName) : scriptName,
-		doc: options.namespaced ? options.packageDir : undefined,
-		parameters: [],
-		cwd: options.namespaced ? options.packageDir : undefined,
-		commandName: shellQuote(scriptName),
-	}));
-}
-
-async function readPackageTasks(cwd: string): Promise<RunnerTask[] | null> {
-	const rootPkg = await readPackageJson(path.join(cwd, "package.json"));
-	if (!rootPkg) return null;
-	const workspacePackageJsons = await findWorkspacePackageJsons(cwd, rootPkg.workspaces);
-	const tasks: RunnerTask[] = [];
-
-	if (rootPkg.scripts.length > 0) {
-		tasks.push(
-			...tasksForPackage({
-				pkg: rootPkg,
-				packageDir: ".",
-				namespaced: false,
-			}),
-		);
-	}
-
-	const pkgs = await Promise.all(workspacePackageJsons.map(p => readPackageJson(path.join(cwd, p))));
-	pkgs.forEach((pkg, index) => {
-		if (!pkg || pkg.scripts.length === 0) return;
-		const packageDir = path.dirname(workspacePackageJsons[index]);
-		tasks.push(
-			...tasksForPackage({
-				pkg,
-				packageDir,
-				namespaced: true,
-			}),
-		);
-	});
-
-	return tasks.length > 0 ? tasks : null;
-}
-
-export const pkgRunner: TaskRunner = {
-	id: "pkg",
-	label: "Pkg",
-	async detect(cwd: string): Promise<DetectedRunner | null> {
-		try {
-			const [commandPrefix, tasks] = await Promise.all([resolvePackageRunner(cwd), readPackageTasks(cwd)]);
-			if (!tasks || tasks.length === 0) return null;
-			return { id: "pkg", label: "Pkg", commandPrefix, tasks };
-		} catch (err) {
-			logger.debug("package runner probe failed", { error: err instanceof Error ? err.message : String(err) });
-			return null;
-		}
-	},
-};

package/src/tools/recipe/runners/task.ts

@@ -1,72 +0,0 @@
-import * as fs from "node:fs/promises";
-import * as path from "node:path";
-import { $which, isEnoent, logger } from "@oh-my-pi/pi-utils";
-import type { DetectedRunner, RunnerTask, TaskRunner } from "../runner";
-
-interface TaskListEntry {
-	name?: string;
-	desc?: string;
-	summary?: string;
-}
-
-interface TaskListJson {
-	tasks?: TaskListEntry[];
-}
-
-const TASKFILE_NAMES = ["Taskfile.yml", "Taskfile.yaml"] as const;
-
-async function hasTaskfile(cwd: string): Promise<boolean> {
-	for (const name of TASKFILE_NAMES) {
-		try {
-			const stat = await fs.stat(path.join(cwd, name));
-			if (stat.isFile()) return true;
-		} catch (err) {
-			if (!isEnoent(err)) throw err;
-		}
-	}
-	return false;
-}
-
-async function listTaskfileTasks(cwd: string): Promise<RunnerTask[] | null> {
-	try {
-		const proc = Bun.spawn(["task", "--list-all", "--json"], {
-			cwd,
-			stdin: "ignore",
-			stdout: "pipe",
-			stderr: "pipe",
-		});
-		const [stdout, exit] = await Promise.all([new Response(proc.stdout).text(), proc.exited]);
-		if (exit !== 0) return null;
-		const list = JSON.parse(stdout) as TaskListJson;
-		const tasks = (list.tasks ?? [])
-			.filter(
-				(task): task is TaskListEntry & { name: string } => typeof task.name === "string" && task.name.length > 0,
-			)
-			.map(task => {
-				const desc = typeof task.desc === "string" && task.desc.length > 0 ? task.desc : undefined;
-				const summary = typeof task.summary === "string" && task.summary.length > 0 ? task.summary : undefined;
-				return { name: task.name, doc: desc ?? summary, parameters: [] };
-			});
-		return tasks.length > 0 ? tasks : null;
-	} catch (err) {
-		logger.debug("task runner list failed", { error: err instanceof Error ? err.message : String(err) });
-		return null;
-	}
-}
-
-export const taskRunner: TaskRunner = {
-	id: "task",
-	label: "Task",
-	async detect(cwd: string): Promise<DetectedRunner | null> {
-		try {
-			if (!$which("task")) return null;
-			if (!(await hasTaskfile(cwd))) return null;
-			const tasks = await listTaskfileTasks(cwd);
-			if (!tasks || tasks.length === 0) return null;
-			return { id: "task", label: "Task", commandPrefix: "task", tasks };
-		} catch (err) {
-			logger.debug("task runner probe failed", { error: err instanceof Error ? err.message : String(err) });
-			return null;
-		}
-	},
-};