@kyubiware/commit-mint 0.5.4 → 0.5.6

package/dist/cli.mjs

@@ -28,7 +28,7 @@ var __exportAll = (all, no_symbols) => {
 //#region package.json
 var package_default = {
 	name: "@kyubiware/commit-mint",
-	version: "0.5.4",
+	version: "0.5.6",
 	description: "🌿 A commit tool that actually handles hook failures",
 	type: "module",
 	bin: { "cmint": "./dist/cli.mjs" },
@@ -1031,23 +1031,44 @@ function buildCommand(command, files) {
 	return `${command} ${files.map((f) => f.includes(" ") ? `"${f}"` : f).join(" ")}`;
 }
 /**
-* Resolve config commands for a glob entry into an array of command strings.
-* Function commands receive matched filenames; string commands are used as-is.
+* Call a function command with matched files and normalize the result to ResolvedCommand[].
+*/
+function resolveFunction(fn, matchedFiles) {
+	const resolved = fn(matchedFiles);
+	return (Array.isArray(resolved) ? resolved : [resolved]).map((command) => ({
+		command,
+		fromFunction: true
+	}));
+}
+/**
+* Resolve config commands for a glob entry into an array of resolved commands.
+* Function commands are called with matched filenames; string commands are kept as-is.
+* Each resolved entry tracks whether it came from a function (for file-append behavior).
 */
 function resolveCommands(commands, matchedFiles) {
-	if (typeof commands === "function") {
-		const resolved = commands(matchedFiles);
-		return Array.isArray(resolved) ? resolved : [resolved];
+	if (typeof commands === "function") return resolveFunction(commands, matchedFiles);
+	if (Array.isArray(commands)) {
+		const result = [];
+		for (const cmd of commands) if (typeof cmd === "function") result.push(...resolveFunction(cmd, matchedFiles));
+		else result.push({
+			command: cmd,
+			fromFunction: false
+		});
+		return result;
 	}
-	return Array.isArray(commands) ? commands : [commands];
+	return [{
+		command: commands,
+		fromFunction: false
+	}];
 }
 /**
 * Run resolved commands for a single glob entry, appending results.
+* Function-originated commands run as-is; string commands get matched files appended.
 * Returns false if any command fails (for fail-fast signaling).
 */
-async function runCommandsForGlob(cmds, isFunction, matchedFiles, timeout, results, repoRoot) {
-	for (const cmd of cmds) {
-		const fullCommand = isFunction ? cmd : buildCommand(cmd, matchedFiles);
+async function runCommandsForGlob(cmds, matchedFiles, timeout, results, repoRoot) {
+	for (const { command, fromFunction } of cmds) {
+		const fullCommand = fromFunction ? command : buildCommand(command, matchedFiles);
 		debug("runCommandsForGlob: running '%s'", fullCommand);
 		const result = await runCommand(fullCommand, timeout, repoRoot);
 		results.push({
@@ -1080,13 +1101,12 @@ async function runAllChecks(repoRoot, stagedFiles, timeout) {
 	const results = [];
 	for (const [glob, commands] of Object.entries(config)) {
 		const matchedFiles = matchFiles(glob, stagedFiles);
-		const isFunction = typeof commands === "function";
 		if (matchedFiles.length === 0) {
 			debug("runAllChecks: no files matched pattern '%s'", glob);
 			continue;
 		}
 		debug("runAllChecks: pattern '%s' matched %d files", glob, matchedFiles.length);
-		if (!await runCommandsForGlob(resolveCommands(commands, matchedFiles), isFunction, matchedFiles, timeout, results, repoRoot)) return {
+		if (!await runCommandsForGlob(resolveCommands(commands, matchedFiles), matchedFiles, timeout, results, repoRoot)) return {
 			ok: false,
 			results
 		};
@@ -1274,6 +1294,188 @@ function validateGroups(groups, allFiles) {
 	return validated;
 }
 //#endregion
+//#region src/services/clipboard.ts
+/** Milliseconds to wait after stdin closes for quick exit failures. */
+const GRACE_PERIOD_MS = 150;
+async function copyToClipboard(content) {
+	for (const [cmd, args] of [
+		["wl-copy", []],
+		["xclip", ["-selection", "clipboard"]],
+		["xsel", ["--clipboard", "--input"]],
+		["pbcopy", []]
+	]) try {
+		if (await tryCopy(cmd, args, content)) return true;
+	} catch {}
+	return false;
+}
+/**
+* Try to copy content using a single clipboard tool.
+*
+* Waits a short grace period after stdin closes to detect quick failures
+* (e.g. wl-copy on non-Wayland, missing display). If the tool survives
+* the grace period, assumes success — clipboard tools like xclip and
+* wl-copy hold the selection open indefinitely, so we can't wait for exit.
+*/
+/**
+* Evaluate clipboard tool status after the grace period.
+* If the child already exited, report based on exit code.
+* If still alive, assume success (clipboard tools hold selection open).
+*/
+function handleGracePeriod(settled, exitCode, stderrChunks, child, done) {
+	if (settled) return;
+	if (exitCode !== null) {
+		if (exitCode === 0) done(true, "exited 0");
+		else {
+			const stderr = Buffer.concat(stderrChunks).toString().trim();
+			done(false, `exit ${exitCode}${stderr ? `: ${stderr}` : ""}`);
+		}
+		return;
+	}
+	child.unref();
+	done(true);
+}
+function tryCopy(cmd, args, content) {
+	return new Promise((resolve) => {
+		debug("clipboard: trying %s", cmd);
+		const child = spawn(cmd, args, { stdio: [
+			"pipe",
+			"ignore",
+			"pipe"
+		] });
+		let settled = false;
+		const stderrChunks = [];
+		child.stderr?.on("data", (chunk) => {
+			stderrChunks.push(chunk);
+		});
+		const done = (result, reason) => {
+			if (settled) return;
+			settled = true;
+			debug("clipboard: %s %s%s", cmd, result ? "ok" : "failed", reason ? ` (${reason})` : "");
+			resolve(result);
+		};
+		child.on("error", (err) => {
+			done(false, err.message);
+		});
+		let exitCode = null;
+		child.on("exit", (code) => {
+			exitCode = code;
+		});
+		child.stdin.write(content, (err) => {
+			if (err) {
+				done(false, "stdin write error");
+				return;
+			}
+			child.stdin.end(() => {
+				setTimeout(() => handleGracePeriod(settled, exitCode, stderrChunks, child, done), GRACE_PERIOD_MS);
+			});
+		});
+	});
+}
+//#endregion
+//#region src/ui/check-failure-menu.ts
+const MAX_TSC_DIAGNOSTICS = 3;
+const MAX_SUMMARY_LINE_LENGTH = 120;
+const TSC_DIAGNOSTIC = /^(.+?\.(?:ts|tsx|mts|cts|js|jsx|mjs|cjs))\((\d+),(\d+)\):\s+error\s+(TS\d+):\s+(.+)$/;
+function formatCheckFailureSummary(errors) {
+	if (errors.length === 0) return "No check error details were parsed. View full output for details.";
+	return errors.map((error) => formatCheckErrorSummary(error)).join("\n");
+}
+function formatCheckErrorSummary(error) {
+	if (error.tool === "tsc") {
+		const diagnostics = extractTscDiagnostics(error.raw || error.message);
+		if (diagnostics.length > 0) return formatTscSummary(diagnostics);
+	}
+	const message = firstMeaningfulLine(error.message || error.raw);
+	return `  ${red("•")} [${error.tool}] ${truncate(message, MAX_SUMMARY_LINE_LENGTH)}`;
+}
+function extractTscDiagnostics(raw) {
+	return raw.split("\n").map((line) => line.trim()).map((line) => {
+		const match = TSC_DIAGNOSTIC.exec(line);
+		if (!match) return null;
+		return {
+			file: match[1] ?? "",
+			line: match[2] ?? "",
+			column: match[3] ?? "",
+			code: match[4] ?? "",
+			message: match[5] ?? ""
+		};
+	}).filter((diagnostic) => diagnostic !== null);
+}
+function formatTscSummary(diagnostics) {
+	const visible = diagnostics.slice(0, MAX_TSC_DIAGNOSTICS);
+	const hidden = diagnostics.length - visible.length;
+	const lines = [`  ${red("•")} [tsc] ${diagnostics.length} TypeScript error${diagnostics.length !== 1 ? "s" : ""}`, ...visible.map((diagnostic) => `${diagnostic.file}:${diagnostic.line}:${diagnostic.column} — error ${diagnostic.code}: ${truncate(diagnostic.message, MAX_SUMMARY_LINE_LENGTH)}`)];
+	if (hidden > 0) lines.push(dim(`    +${hidden} more TypeScript error${hidden !== 1 ? "s" : ""}. View full output for details.`));
+	return lines.join("\n");
+}
+function firstMeaningfulLine(message) {
+	return message.split("\n").map((l) => l.trim()).find((l) => l.length > 0 && !l.startsWith(">") && !l.startsWith("ELIFECYCLE")) ?? message;
+}
+function truncate(message, maxLength) {
+	const collapsed = message.replace(/\s+/g, " ").trim();
+	if (collapsed.length <= maxLength) return collapsed;
+	return `${collapsed.slice(0, Math.max(0, maxLength - 1))}…`;
+}
+async function showCheckFailureMenu(errors, rawStderr, onRetry) {
+	debug("showCheckFailureMenu: %d errors", errors.length);
+	let clipboardCopied = false;
+	p.note(formatCheckFailureSummary(errors), red("Pre-commit check failed"));
+	while (true) {
+		const choice = await p.select({
+			message: "What do you want to do?",
+			options: [
+				{
+					label: clipboardCopied ? `${green("✓")} Copy error report to clipboard` : "Copy error report to clipboard",
+					value: "copy"
+				},
+				{
+					label: "View full error output",
+					value: "view",
+					hint: "Show the raw stderr from checks"
+				},
+				{
+					label: "Retry checks",
+					value: "retry",
+					hint: "Re-run checks after fixing errors"
+				},
+				{
+					label: "Skip checks and commit",
+					value: "skip"
+				},
+				{
+					label: "Cancel",
+					value: "cancel"
+				}
+			]
+		});
+		if (p.isCancel(choice)) {
+			debug("showCheckFailureMenu: user cancelled");
+			return "cancelled";
+		}
+		debug("showCheckFailureMenu: user chose %s", choice);
+		switch (choice) {
+			case "copy":
+				if (await copyToClipboard(rawStderr)) {
+					clipboardCopied = true;
+					p.log.step(green("Copied to clipboard."));
+				} else p.log.warn(red("No clipboard tool found. Install xclip, wl-copy, or xsel."));
+				continue;
+			case "view":
+				p.note(rawStderr.trim() || "(no raw output)", "Full error output");
+				continue;
+			case "retry":
+				if (onRetry) return "retried";
+				return "retried";
+			case "skip":
+				p.log.info("Skipping checks and proceeding with commit...");
+				return "skipped";
+			case "cancel":
+				p.outro(dim("Cancelled."));
+				return "cancelled";
+		}
+	}
+}
+//#endregion
 //#region src/ui/grouping.ts
 async function showGroupingConfirmation(groups, excluded) {
 	debug("showGroupingConfirmation: %d groups, %d excluded", groups.length, excluded.length);
@@ -1336,126 +1538,7 @@ function showGroupedFiles(groups, changedFiles) {
 	p.note(lines.join("\n"), "Commit groups");
 }
 //#endregion
-//#region src/services/clipboard.ts
-async function copyToClipboard(content) {
-	for (const [cmd, args] of [
-		["wl-copy", []],
-		["xclip", ["-selection", "clipboard"]],
-		["xsel", ["--clipboard", "--input"]],
-		["pbcopy", []]
-	]) try {
-		if (await new Promise((resolve) => {
-			const child = spawn(cmd, args, { stdio: [
-				"pipe",
-				"ignore",
-				"ignore"
-			] });
-			let settled = false;
-			const done = (result) => {
-				if (settled) return;
-				settled = true;
-				resolve(result);
-			};
-			child.on("error", () => done(false));
-			child.on("exit", (code) => {
-				if (code !== 0) done(false);
-			});
-			child.stdin.write(content, (err) => {
-				if (err) {
-					done(false);
-					return;
-				}
-				child.stdin.end(() => {
-					child.unref();
-					done(true);
-				});
-			});
-		})) return true;
-	} catch {}
-	return false;
-}
-//#endregion
-//#region src/ui/menu.ts
-async function showStagingMenu(files, hasChecks) {
-	debug("showStagingMenu: %d files", files.length);
-	const statusLabel = (status) => {
-		switch (status) {
-			case "M": return yellow("M");
-			case "A": return green("A");
-			case "D": return red("D");
-			case "?":
-			case "??": return cyan("?");
-			default: return dim(status);
-		}
-	};
-	const sorted = [...files].sort((a, b) => {
-		if (a.staged !== b.staged) return a.staged ? -1 : 1;
-		return a.path.localeCompare(b.path);
-	});
-	const stagedFiles = sorted.filter((f) => f.staged);
-	const unstagedFiles = sorted.filter((f) => !f.staged);
-	const lines = [];
-	if (stagedFiles.length > 0) lines.push(green(bold("Staged:")), ...stagedFiles.map((f) => `  ${statusLabel(f.status)}  ${f.path}`));
-	if (unstagedFiles.length > 0) {
-		if (lines.length > 0) lines.push("");
-		lines.push(yellow(bold("Changed:")), ...unstagedFiles.map((f) => `  ${statusLabel(f.status)}  ${f.path}`));
-	}
-	p.note(lines.join("\n"), `${files.length} file${files.length !== 1 ? "s" : ""}`);
-	const choice = await p.select({
-		message: "Stage files for commit:",
-		options: [
-			{
-				label: "Auto-group into commits",
-				value: "autogroup",
-				hint: "LLM groups files into logical commits"
-			},
-			...stagedFiles.length > 0 ? [{
-				label: "Commit staged files only",
-				value: "staged",
-				hint: `${stagedFiles.length} file${stagedFiles.length !== 1 ? "s" : ""} already staged`
-			}] : [],
-			{
-				label: "Stage all files",
-				value: "all",
-				hint: `${files.length} file${files.length !== 1 ? "s" : ""}`
-			},
-			...hasChecks ? [{
-				label: "Run checks",
-				value: "checks",
-				hint: "Pre-flight checks from cmint config"
-			}] : [],
-			{
-				label: "Select files...",
-				value: "select"
-			},
-			{
-				label: "Cancel",
-				value: "cancel"
-			}
-		]
-	});
-	if (p.isCancel(choice) || choice === "cancel") return null;
-	if (choice === "autogroup") return "autogroup";
-	if (choice === "checks") return "checks";
-	if (choice === "staged") return "staged";
-	if (choice === "all") return {
-		files: files.map((f) => f.path),
-		all: true
-	};
-	const selected = await p.multiselect({
-		message: "Select files to stage:",
-		options: sorted.map((f) => ({
-			label: `${statusLabel(f.status)}  ${f.path}`,
-			value: f.path
-		})),
-		required: true
-	});
-	if (p.isCancel(selected)) return null;
-	return {
-		files: selected,
-		all: false
-	};
-}
+//#region src/ui/recovery-menu.ts
 async function showRecoveryMenu(errors, onRetry, onSkipHooks, onRestage, message, rawStderr) {
 	debug("showRecoveryMenu: %d errors", errors.length);
 	let clipboardCopied = false;
@@ -1557,65 +1640,6 @@ async function showRecoveryMenu(errors, onRetry, onSkipHooks, onRestage, message
 		}
 	}
 }
-async function showCheckFailureMenu(errors, rawStderr, onRetry) {
-	debug("showCheckFailureMenu: %d errors", errors.length);
-	let clipboardCopied = false;
-	p.note(errors.map((e) => `  ${red("•")} [${e.tool}] ${e.message}`).join("\n"), red("Pre-commit check failed"));
-	while (true) {
-		const choice = await p.select({
-			message: "What do you want to do?",
-			options: [
-				{
-					label: clipboardCopied ? `${green("✓")} Copy error report to clipboard` : "Copy error report to clipboard",
-					value: "copy"
-				},
-				{
-					label: "View full error output",
-					value: "view",
-					hint: "Show the raw stderr from checks"
-				},
-				{
-					label: "Retry checks",
-					value: "retry",
-					hint: "Re-run checks after fixing errors"
-				},
-				{
-					label: "Skip checks and commit",
-					value: "skip"
-				},
-				{
-					label: "Cancel",
-					value: "cancel"
-				}
-			]
-		});
-		if (p.isCancel(choice)) {
-			debug("showCheckFailureMenu: user cancelled");
-			return "cancelled";
-		}
-		debug("showCheckFailureMenu: user chose %s", choice);
-		switch (choice) {
-			case "copy":
-				if (await copyToClipboard(rawStderr)) {
-					clipboardCopied = true;
-					p.log.step(green("Copied to clipboard."));
-				} else p.log.warn(red("No clipboard tool found. Install xclip, wl-copy, or xsel."));
-				continue;
-			case "view":
-				p.note(rawStderr.trim() || "(no raw output)", "Full error output");
-				continue;
-			case "retry":
-				if (onRetry) return "retried";
-				return "retried";
-			case "skip":
-				p.log.info("Skipping checks and proceeding with commit...");
-				return "skipped";
-			case "cancel":
-				p.outro(dim("Cancelled."));
-				return "cancelled";
-		}
-	}
-}
 //#endregion
 //#region src/commands/auto-group.ts
 async function runAutoGroupFlow(changedFiles, flags) {
@@ -1849,6 +1873,88 @@ async function handleRetry() {
 	else process.exit(1);
 }
 //#endregion
+//#region src/ui/staging-menu.ts
+async function showStagingMenu(files, hasChecks) {
+	debug("showStagingMenu: %d files", files.length);
+	const statusLabel = (status) => {
+		switch (status) {
+			case "M": return yellow("M");
+			case "A": return green("A");
+			case "D": return red("D");
+			case "?":
+			case "??": return cyan("?");
+			default: return dim(status);
+		}
+	};
+	const sorted = [...files].sort((a, b) => {
+		if (a.staged !== b.staged) return a.staged ? -1 : 1;
+		return a.path.localeCompare(b.path);
+	});
+	const stagedFiles = sorted.filter((f) => f.staged);
+	const unstagedFiles = sorted.filter((f) => !f.staged);
+	const lines = [];
+	if (stagedFiles.length > 0) lines.push(green(bold("Staged:")), ...stagedFiles.map((f) => `  ${statusLabel(f.status)}  ${f.path}`));
+	if (unstagedFiles.length > 0) {
+		if (lines.length > 0) lines.push("");
+		lines.push(yellow(bold("Changed:")), ...unstagedFiles.map((f) => `  ${statusLabel(f.status)}  ${f.path}`));
+	}
+	p.note(lines.join("\n"), `${files.length} file${files.length !== 1 ? "s" : ""}`);
+	const choice = await p.select({
+		message: "Stage files for commit:",
+		options: [
+			{
+				label: "Auto-group into commits",
+				value: "autogroup",
+				hint: "LLM groups files into logical commits"
+			},
+			...stagedFiles.length > 0 ? [{
+				label: "Commit staged files only",
+				value: "staged",
+				hint: `${stagedFiles.length} file${stagedFiles.length !== 1 ? "s" : ""} already staged`
+			}] : [],
+			{
+				label: "Stage all files",
+				value: "all",
+				hint: `${files.length} file${files.length !== 1 ? "s" : ""}`
+			},
+			...hasChecks ? [{
+				label: "Run checks",
+				value: "checks",
+				hint: "Pre-flight checks from cmint config"
+			}] : [],
+			{
+				label: "Select files...",
+				value: "select"
+			},
+			{
+				label: "Cancel",
+				value: "cancel"
+			}
+		]
+	});
+	if (p.isCancel(choice) || choice === "cancel") return null;
+	if (choice === "autogroup") return "autogroup";
+	if (choice === "checks") return "checks";
+	if (choice === "staged") return "staged";
+	if (choice === "all") return {
+		files: files.map((f) => f.path),
+		all: true
+	};
+	const selected = await p.multiselect({
+		message: "Select files to stage:",
+		options: sorted.map((f) => ({
+			label: `${statusLabel(f.status)}  ${f.path}`,
+			value: f.path
+		})),
+		required: true
+	});
+	if (p.isCancel(selected)) return null;
+	return {
+		files: selected,
+		all: false
+	};
+}
+//#endregion
 //#region src/commands/staging.ts
 /** Interactive staging loop for multiple changed files */
 async function handleStaging(changedFiles, flags) {