@visorcraft/idlehands 4.2.3 → 4.2.5

package/dist/{anton-Dn_Pc4P0.js → anton-Cw4-BgtA.js}

@@ -1,5 +1,5 @@
-import "./paths-DQpv9a3Q.js";
-import { t as CONFIG_DIR } from "./utils-D9wGoiU9.js";
+import "./paths-BYNVLNi_.js";
+import { t as CONFIG_DIR } from "./utils-CTFLl_ji.js";
 import fs from "node:fs/promises";
 import path from "node:path";
 import { promisify } from "node:util";
@@ -77,13 +77,126 @@ function markTaskDone(markdown, lineNo) {
 	lines[idx] = (lines[idx] ?? "").replace(/^(\s*)- \[ \] /, "$1- [x] ");
 	return `${lines.join("\n")}\n`;
 }
-function buildDirectTaskPrompt(task) {
+function isJsonTaskFile(taskFile) {
+	return taskFile.toLowerCase().endsWith(".json");
+}
+function parsePendingTasksFromJson(jsonText) {
+	const data = JSON.parse(jsonText);
+	const pending = [];
+	const joinDetails = (parentDetails, subtasks) => {
+		const details = [];
+		if (typeof parentDetails === "string" && parentDetails.trim()) details.push(parentDetails.trim());
+		const lines = [];
+		for (const sub of subtasks) {
+			if (!sub || typeof sub !== "object") continue;
+			const subObj = sub;
+			if (subObj.done === true) continue;
+			const name = typeof subObj.name === "string" ? subObj.name.trim() : "";
+			if (!name) continue;
+			const subDetails = typeof subObj.details === "string" ? subObj.details.trim() : "";
+			lines.push(subDetails ? `- ${name}: ${subDetails}` : `- ${name}`);
+		}
+		if (lines.length > 0) details.push(`Subtasks:\n${lines.join("\n")}`);
+		return details.length > 0 ? details.join("\n\n") : void 0;
+	};
+	const pushTask = (jsonPath, obj) => {
+		if (obj.done === true) return;
+		const text = typeof obj.name === "string" ? obj.name.trim() : "";
+		if (!text) return;
+		const subtasks = Array.isArray(obj.subtasks) ? obj.subtasks : [];
+		pending.push({
+			text,
+			details: joinDetails(obj.details, subtasks),
+			jsonPath
+		});
+	};
+	if (data && typeof data === "object") {
+		const root = data;
+		const tasks = root.tasks;
+		if (Array.isArray(tasks)) for (let i = 0; i < tasks.length; i++) {
+			const task = tasks[i];
+			if (!task || typeof task !== "object") continue;
+			pushTask(["tasks", i], task);
+		}
+		else if ("name" in root || "subtasks" in root) pushTask([], root);
+	}
+	return {
+		pending,
+		data
+	};
+}
+function markJsonTaskDone(data, jsonPath) {
+	if (!jsonPath) return;
+	let cur = data;
+	for (const key of jsonPath) {
+		if (typeof key === "number") {
+			if (!Array.isArray(cur)) return;
+			cur = cur[key];
+			continue;
+		}
+		if (!cur || typeof cur !== "object" || Array.isArray(cur)) return;
+		cur = cur[key];
+	}
+	if (!cur || typeof cur !== "object" || Array.isArray(cur)) return;
+	cur.done = true;
+}
+function formatTaskForPrompt(task) {
+	if (task.details) return `${task.text}
+Details: ${task.details}`;
+	return task.text;
+}
+function looksLikeVersionBumpTask(taskText) {
+	return /bump\s+version/i.test(taskText) || /version\s+(to|\d+\.)/i.test(taskText);
+}
+function allowsNoRepoChanges(taskText) {
+	return /(review|verify|ensure|confirm|check)/i.test(taskText) && /(doc|docs|documentation|readme)/i.test(taskText);
+}
+async function ensureRepoCleanOrThrow(cwd) {
+	const { stdout } = await execFile$1("git", ["status", "--porcelain"], { cwd });
+	if (stdout.split(/\r?\n/).map((line) => line.trimEnd()).filter(Boolean).map((line) => line.slice(3).trim().replace(/\\/g, "/")).filter((file) => file && !file.startsWith(".agents/tasks/")).length > 0) throw new Error(`Anton requires a clean git working tree. Dirty repo at: ${cwd}`);
+}
+async function hardCleanRepo(cwd) {
+	await execFile$1("git", ["reset", "--hard"], { cwd });
+	await execFile$1("git", [
+		"clean",
+		"-fd",
+		"-e",
+		".agents/tasks",
+		"-e",
+		"node_modules"
+	], { cwd });
+}
+async function commitAllIfDirty(cwd, message) {
+	const { stdout } = await execFile$1("git", ["status", "--porcelain"], { cwd });
+	if (stdout.trim().length === 0) return;
+	await execFile$1("git", ["add", "-A"], { cwd });
+	try {
+		await execFile$1("git", [
+			"restore",
+			"--staged",
+			".agents/tasks"
+		], { cwd });
+	} catch {}
+	const { stdout: staged } = await execFile$1("git", [
+		"diff",
+		"--cached",
+		"--name-only"
+	], { cwd });
+	if (staged.trim().length === 0) return;
+	await execFile$1("git", [
+		"commit",
+		"-m",
+		message
+	], { cwd });
+}
+function buildDirectTaskPrompt(task, opts) {
+	const skipTests = Boolean(opts?.skipTests);
 	return [
 		"You are executing one item from a managed checklist.",
 		`Task: ${task}`,
 		"Rules:",
 		"1) Make the minimal code changes required for this task.",
-		"2) Run targeted tests for your change.",
+		skipTests ? "2) Skip tests for this task unless you changed executable code." : "2) Run targeted tests for your change.",
 		"3) Return a concise completion summary."
 	].join("\n");
 }
@@ -137,7 +250,8 @@ function buildReviewPrompt(planFilePath) {
 
 Do not skip reading the file. Do not return JSON without first reading and updating the plan.`;
 }
-function buildImplementationPrompt(task, planFilePath) {
+function buildImplementationPrompt(task, planFilePath, opts) {
+	const skipTests = Boolean(opts?.skipTests);
 	return [
 		`Complete this task: ${task}`,
 		"",
@@ -148,11 +262,12 @@ function buildImplementationPrompt(task, planFilePath) {
 		"Describing changes in text or showing diffs without applying them does NOT count.",
 		"Do not ask for confirmation — apply the changes now.",
 		"",
-		"After making the edits, run any verification steps from the plan.",
+		skipTests ? "Skip tests for this task unless you changed executable code." : "After making the edits, run any verification steps from the plan.",
 		"Then return a brief summary of what you changed."
 	].join("\n");
 }
-function buildImplementationRetryPrompt(task, planFilePath) {
+function buildImplementationRetryPrompt(task, planFilePath, opts) {
+	const skipTests = Boolean(opts?.skipTests);
 	return [
 		"Your previous implementation attempt made zero repository changes.",
 		`Task: ${task}`,
@@ -162,12 +277,27 @@ function buildImplementationRetryPrompt(task, planFilePath) {
 		"Required sequence:",
 		"1) Read the plan file.",
 		"2) Use edit/write to modify repository files.",
-		"3) Optionally run verification.",
+		skipTests ? "3) Skip tests for this task unless you changed executable code." : "3) Optionally run verification.",
 		"4) Return a short summary.",
 		"",
 		"If you only provide analysis/diff text without edit/write tool calls, this attempt fails."
 	].join("\n");
 }
+function resolveAntonExecutionConfig(params) {
+	const antonCfg = params.config ?? {};
+	const preflightCfg = antonCfg.preflight ?? {};
+	const requestedMode = params.modeOverride ?? antonCfg.mode ?? "direct";
+	const mode = requestedMode === "preflight" && preflightCfg.enabled === false ? "direct" : requestedMode;
+	const taskTimeoutSec = antonCfg.taskTimeoutSec ?? 1200;
+	return {
+		mode,
+		requirementsReview: antonCfg.requirementsReview ?? preflightCfg.requirementsReview ?? false,
+		taskTimeoutSec,
+		discoveryTimeoutSec: antonCfg.discoveryTimeoutSec ?? preflightCfg.discoveryTimeoutSec ?? taskTimeoutSec,
+		reviewTimeoutSec: antonCfg.reviewTimeoutSec ?? preflightCfg.reviewTimeoutSec ?? taskTimeoutSec,
+		preflightMaxRetries: antonCfg.preflightMaxRetries ?? preflightCfg.maxRetries ?? 2
+	};
+}
 function formatDuration(ms) {
 	const seconds = Math.floor(ms / 1e3);
 	if (seconds < 60) return `${seconds}s`;
@@ -225,14 +355,14 @@ async function antonStop(runtime) {
 /** Load Anton config from the IdleHands config file. */
 async function loadAntonConfig() {
 	try {
-		const { loadConfig } = await import("./model-selection-C5hI1dum.js").then((n) => n.I);
+		const { loadConfig } = await import("./model-selection-1MMYqY6M.js").then((n) => n.I);
 		return loadConfig().anton ?? {};
 	} catch {
 		return {};
 	}
 }
 async function runAgentTask(args) {
-	const { agentCliCommand } = await import("./agent-via-gateway-BsTP4WyI.js");
+	const { agentCliCommand } = await import("./agent-via-gateway-DVTkXwmI.js");
 	const extraSystemPrompt = args.workspaceDir ? `Your working directory is: ${args.workspaceDir}\nAll file paths are relative to this directory. Use this as your cwd for all operations.` : void 0;
 	const result = await agentCliCommand({
 		message: args.message,
@@ -376,8 +506,11 @@ async function runDiscoveryPhase(args) {
 				task: `Discovery (attempt ${attempt + 1})`,
 				sessionId
 			});
+			const discoveryRepoCwd = args.workspaceDir ? path.resolve(args.workspaceDir) : path.dirname(args.taskFile);
+			await hardCleanRepo(discoveryRepoCwd);
+			await ensureRepoCleanOrThrow(discoveryRepoCwd);
 			const firstPass = await runAgentTask({
-				message: buildDiscoveryPrompt(args.task.text, args.taskFile, planFile),
+				message: buildDiscoveryPrompt(formatTaskForPrompt(args.task), args.taskFile, planFile),
 				sessionId,
 				timeout: args.timeout,
 				agent: args.agent,
@@ -445,8 +578,10 @@ async function runDiscoveryPhase(args) {
 				task: `Discovery repair (attempt ${attempt + 1})`,
 				sessionId: repairSessionId
 			});
+			await hardCleanRepo(discoveryRepoCwd);
+			await ensureRepoCleanOrThrow(discoveryRepoCwd);
 			const repairPass = await runAgentTask({
-				message: buildDiscoveryRepairPrompt(args.task.text, args.taskFile, declaredPlanFile),
+				message: buildDiscoveryRepairPrompt(formatTaskForPrompt(args.task), args.taskFile, declaredPlanFile),
 				sessionId: repairSessionId,
 				timeout: args.timeout,
 				agent: args.agent,
@@ -544,6 +679,9 @@ async function runReviewPhase(args) {
 			task: "Requirements review",
 			sessionId
 		});
+		const reviewRepoCwd = args.workspaceDir ? path.resolve(args.workspaceDir) : process.cwd();
+		await hardCleanRepo(reviewRepoCwd);
+		await ensureRepoCleanOrThrow(reviewRepoCwd);
 		await runAgentTask({
 			message: buildReviewPrompt(args.planFile),
 			sessionId,
@@ -579,7 +717,14 @@ async function runReviewPhase(args) {
 }
 async function runAnton(args) {
 	const filePath = path.resolve(args.taskFile);
-	const pending = parsePendingTasks(await fs.readFile(filePath, "utf8"));
+	const raw = await fs.readFile(filePath, "utf8");
+	let pending = [];
+	let jsonData;
+	if (isJsonTaskFile(filePath)) {
+		const parsed = parsePendingTasksFromJson(raw);
+		pending = parsed.pending;
+		jsonData = parsed.data;
+	} else pending = parsePendingTasks(raw);
 	const notify = args.onProgress ?? (async () => {});
 	if (args.dryRun) {
 		args.runtime.log(`Dry run: ${pending.length} pending task(s)`);
@@ -598,16 +743,22 @@ async function runAnton(args) {
 		return;
 	}
 	const antonCfg = await loadAntonConfig();
-	const mode = args.mode ?? antonCfg.mode ?? "direct";
-	const requirementsReview = antonCfg.requirementsReview ?? false;
-	const taskTimeout = antonCfg.taskTimeoutSec ?? 1200;
-	const discoveryTimeout = antonCfg.discoveryTimeoutSec ?? taskTimeout;
-	const reviewTimeout = antonCfg.reviewTimeoutSec ?? taskTimeout;
-	const preflightMaxRetries = antonCfg.preflightMaxRetries ?? 2;
+	const execution = resolveAntonExecutionConfig({
+		config: antonCfg,
+		modeOverride: args.mode
+	});
+	const mode = execution.mode;
+	const requirementsReview = execution.requirementsReview;
+	const taskTimeout = execution.taskTimeoutSec;
+	const discoveryTimeout = execution.discoveryTimeoutSec;
+	const reviewTimeout = execution.reviewTimeoutSec;
+	const preflightMaxRetries = execution.preflightMaxRetries;
 	const planDir = antonCfg.planDir ? path.resolve(antonCfg.planDir) : path.resolve(path.dirname(filePath), ".agents", "tasks");
-	const { loadConfig } = await import("./model-selection-C5hI1dum.js").then((n) => n.I);
+	const { loadConfig } = await import("./model-selection-1MMYqY6M.js").then((n) => n.I);
 	const cfg = loadConfig();
 	const defaultTimeout = String(Number.isFinite(args.timeoutSec) && (args.timeoutSec ?? 0) > 0 ? args.timeoutSec : cfg.agents?.defaults?.timeoutSeconds ?? taskTimeout);
+	const repoCwd = args.workspaceDir ? path.resolve(args.workspaceDir) : path.dirname(filePath);
+	await ensureRepoCleanOrThrow(repoCwd);
 	await acquireLock(Boolean(args.force));
 	const startedAt = Date.now();
 	await writeState({
@@ -642,6 +793,9 @@ async function runAnton(args) {
 			const task = pending[i];
 			if (!task) continue;
 			const taskNum = i + 1;
+			const taskPromptText = formatTaskForPrompt(task);
+			const skipTests = looksLikeVersionBumpTask(task.text) || allowsNoRepoChanges(task.text);
+			const allowNoChanges = allowsNoRepoChanges(task.text);
 			await writeState({
 				running: true,
 				taskFile: filePath,
@@ -665,6 +819,8 @@ async function runAnton(args) {
 				const taskFileRelForBaseline = path.relative(gitCwdForBaseline, filePath).replace(/\\/g, "/");
 				const changedBeforeTask = await getGitChangedFileCount(gitCwdForBaseline, taskFileRelForBaseline.startsWith("..") ? [] : [taskFileRelForBaseline]);
 				if (mode === "preflight") {
+					await hardCleanRepo(repoCwd);
+					await ensureRepoCleanOrThrow(repoCwd);
 					const discoveryResult = await runDiscoveryPhase({
 						task,
 						taskNum,
@@ -687,8 +843,17 @@ async function runAnton(args) {
 							total: pending.length,
 							task: task.text
 						});
-						const updated = markTaskDone(await fs.readFile(filePath, "utf8"), task.line);
-						await fs.writeFile(filePath, updated, "utf8");
+						const latest = await fs.readFile(filePath, "utf8");
+						if (isJsonTaskFile(filePath)) {
+							if (jsonData && task.jsonPath) {
+								markJsonTaskDone(jsonData, task.jsonPath);
+								await fs.writeFile(filePath, JSON.stringify(jsonData, null, 2) + "\n", "utf8");
+							}
+						} else {
+							const updated = markTaskDone(latest, task.line ?? 0);
+							await fs.writeFile(filePath, updated, "utf8");
+						}
+						await commitAllIfDirty(repoCwd, `anton: ${task.text}`);
 						completed += 1;
 						await notify({
 							phase: "task_complete",
@@ -731,10 +896,12 @@ async function runAnton(args) {
 					task: task.text,
 					sessionId: implSessionId
 				});
-				const implPrompt = planFile ? buildImplementationPrompt(task.text, planFile) : buildDirectTaskPrompt(task.text);
+				const implPrompt = planFile ? buildImplementationPrompt(taskPromptText, planFile, { skipTests }) : buildDirectTaskPrompt(taskPromptText, { skipTests });
 				const gitCwd = args.workspaceDir ? path.resolve(args.workspaceDir) : path.dirname(filePath);
 				const taskFileRel = path.relative(gitCwd, filePath).replace(/\\/g, "/");
 				const changeIgnores = taskFileRel.startsWith("..") ? [] : [taskFileRel];
+				await hardCleanRepo(repoCwd);
+				await ensureRepoCleanOrThrow(repoCwd);
 				await runAgentTask({
 					message: implPrompt,
 					sessionId: implSessionId,
@@ -746,30 +913,44 @@ async function runAnton(args) {
 					workspaceDir: args.workspaceDir
 				});
 				let changedAfter = await getGitChangedFileCount(gitCwd, changeIgnores);
+				if (mode === "direct" && changedBeforeTask !== null && changedAfter !== null && changedAfter <= changedBeforeTask && !allowNoChanges) throw new Error("Implementation made no repository changes; refusing to mark task complete");
 				if (mode === "preflight" && changedBeforeTask !== null && changedAfter !== null && changedAfter <= changedBeforeTask) {
-					const retrySessionId = `anton-impl-retry-${Date.now()}-${taskNum}`;
-					await notify({
-						phase: "task_agent_spawned",
-						index: taskNum,
-						total: pending.length,
-						task: `${task.text} (implementation retry)`,
-						sessionId: retrySessionId
-					});
-					await runAgentTask({
-						message: buildImplementationRetryPrompt(task.text, planFile ?? ""),
-						sessionId: retrySessionId,
-						timeout: defaultTimeout,
-						agent: args.agent,
-						to: args.to,
-						runtime: args.runtime,
-						deps: args.deps,
-						workspaceDir: args.workspaceDir
-					});
-					changedAfter = await getGitChangedFileCount(gitCwd, changeIgnores);
-					if (changedAfter !== null && changedAfter <= changedBeforeTask) throw new Error("Implementation made no repository changes after retry; refusing to mark task complete");
+					if (!allowNoChanges) {
+						const retrySessionId = `anton-impl-retry-${Date.now()}-${taskNum}`;
+						await notify({
+							phase: "task_agent_spawned",
+							index: taskNum,
+							total: pending.length,
+							task: `${task.text} (implementation retry)`,
+							sessionId: retrySessionId
+						});
+						await hardCleanRepo(repoCwd);
+						await ensureRepoCleanOrThrow(repoCwd);
+						await runAgentTask({
+							message: buildImplementationRetryPrompt(taskPromptText, planFile ?? "", { skipTests }),
+							sessionId: retrySessionId,
+							timeout: defaultTimeout,
+							agent: args.agent,
+							to: args.to,
+							runtime: args.runtime,
+							deps: args.deps,
+							workspaceDir: args.workspaceDir
+						});
+						changedAfter = await getGitChangedFileCount(gitCwd, changeIgnores);
+						if (changedAfter !== null && changedAfter <= changedBeforeTask) throw new Error("Implementation made no repository changes after retry; refusing to mark task complete");
+					}
+				}
+				const latest = await fs.readFile(filePath, "utf8");
+				if (isJsonTaskFile(filePath)) {
+					if (jsonData && task.jsonPath) {
+						markJsonTaskDone(jsonData, task.jsonPath);
+						await fs.writeFile(filePath, JSON.stringify(jsonData, null, 2) + "\n", "utf8");
+					}
+				} else {
+					const updated = markTaskDone(latest, task.line ?? 0);
+					await fs.writeFile(filePath, updated, "utf8");
 				}
-				const updated = markTaskDone(await fs.readFile(filePath, "utf8"), task.line);
-				await fs.writeFile(filePath, updated, "utf8");
+				await commitAllIfDirty(repoCwd, `anton: ${task.text}`);
 				completed += 1;
 				await notify({
 					phase: "task_complete",
@@ -778,6 +959,9 @@ async function runAnton(args) {
 					task: task.text
 				});
 			} catch (err) {
+				try {
+					await hardCleanRepo(repoCwd);
+				} catch {}
 				skipped += 1;
 				const errorMsg = err instanceof Error ? err.message : String(err);
 				args.runtime.error(`[Anton] Task failed and was skipped: ${task.text}`);