@kyubiware/commit-mint 0.5.1 → 0.5.3

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.1",
+	version: "0.5.3",
 	description: "🌿 A commit tool that actually handles hook failures",
 	type: "module",
 	bin: { "cmint": "./dist/cli.mjs" },
@@ -104,11 +104,11 @@ const PROVIDER_CONFIGS = {
 		defaultModel: "openai/gpt-oss-20b"
 	},
 	cerebras: {
-		baseURL: "https://api.cerebras.ai",
+		baseURL: "https://api.cerebras.ai/v1",
 		defaultModel: "gpt-oss-120b"
 	},
 	mistral: {
-		baseURL: "https://api.mistral.ai",
+		baseURL: "https://api.mistral.ai/v1",
 		defaultModel: "mistral-small"
 	}
 };
@@ -124,17 +124,52 @@ function formatProviderName(provider) {
 function isValidProvider(name) {
 	return ALLOWED_PROVIDERS.includes(name);
 }
+/**
+* Generic OpenAI-compatible chat completions client using fetch.
+* Used for non-Groq providers where the Groq SDK's hardcoded `/openai/v1/` path
+* prefix doesn't match the provider's actual API path.
+*/
+function createFetchClient(baseURL, apiKey, timeout) {
+	return { chat: { completions: { async create(params) {
+		const url = `${baseURL}/chat/completions`;
+		debug("fetchClient: POST %s, model=%s", url, params.model);
+		const controller = new AbortController();
+		const timer = setTimeout(() => controller.abort(), timeout);
+		try {
+			const response = await fetch(url, {
+				method: "POST",
+				headers: {
+					"Content-Type": "application/json",
+					Authorization: `Bearer ${apiKey}`
+				},
+				body: JSON.stringify(params),
+				signal: controller.signal
+			});
+			if (!response.ok) {
+				const text = await response.text().catch(() => "");
+				throw new Error(`${response.status} ${text}`);
+			}
+			return await response.json();
+		} finally {
+			clearTimeout(timer);
+		}
+	} } } };
+}
 function createProvider(options) {
 	if (!isValidProvider(options.provider)) throw new Error(`Invalid provider "${options.provider}". Allowed values: ${ALLOWED_PROVIDERS.join(", ")}`);
 	const providerConfig = PROVIDER_CONFIGS[options.provider];
 	const model = options.modelOverride ?? providerConfig.defaultModel;
 	const baseURL = options.baseURLOverride ?? providerConfig.baseURL;
+	const timeout = options.timeout ?? 6e4;
+	let client;
+	if (options.provider === "groq") client = new Groq({
+		apiKey: options.apiKey,
+		baseURL,
+		timeout
+	});
+	else client = createFetchClient(baseURL, options.apiKey, timeout);
 	return {
-		client: new Groq({
-			apiKey: options.apiKey,
-			baseURL,
-			timeout: options.timeout
-		}),
+		client,
 		model
 	};
 }
@@ -191,8 +226,17 @@ async function getProviderApiKey(provider) {
 	debug("getProviderApiKey(%s): not found", provider);
 	throw new Error(`Please set your ${formatProviderName(provider)} API key via \`cmint config set ${envVar}=<your token>\``);
 }
+/** Check if a model name is the default for a provider OTHER than the given one. */
+function isOtherProviderDefault(model, provider) {
+	for (const [name, config] of Object.entries(PROVIDER_CONFIGS)) if (name !== provider && config.defaultModel === model) return true;
+	return false;
+}
 function getModelForProvider(config, provider, defaultModel) {
-	return config[`model_${provider}`] ?? config.model ?? defaultModel;
+	const providerModel = config[`model_${provider}`];
+	if (providerModel) return providerModel;
+	const globalModel = config.model;
+	if (globalModel && !isOtherProviderDefault(globalModel, provider)) return globalModel;
+	return defaultModel;
 }
 //#endregion
 //#region src/services/hooks.ts
@@ -293,6 +337,50 @@ function parseEslintErrors(output) {
 	return errors;
 }
 /**
+* Parse cmint check output into structured errors.
+* Check output uses [tool] prefix format (built by callers in auto-group.ts / staging.ts).
+* This is distinct from parseHookErrors which handles raw git hook stderr.
+*/
+function parseCheckErrors(output) {
+	if (!output.trim()) return [];
+	const errors = [];
+	const lines = output.split("\n");
+	let currentTool = "";
+	let currentLines = [];
+	const flush = () => {
+		if (currentTool && currentLines.length > 0) {
+			const message = currentLines.join("\n").trim();
+			if (message) errors.push({
+				tool: currentTool,
+				message,
+				raw: currentLines.join("\n")
+			});
+		}
+		currentTool = "";
+		currentLines = [];
+	};
+	for (const line of lines) {
+		const match = line.match(/^\[(\S+)\]\s*(.*)/);
+		if (match) {
+			flush();
+			currentTool = match[1];
+			const rest = match[2].trim();
+			if (rest) currentLines.push(rest);
+		} else if (currentTool) currentLines.push(line);
+		else currentLines.push(line);
+	}
+	flush();
+	if (errors.length === 0) {
+		const trimmed = output.trim();
+		if (trimmed) errors.push({
+			tool: "checks",
+			message: trimmed,
+			raw: output
+		});
+	}
+	return errors;
+}
+/**
 * Parse lint-staged/hook stderr output to discover which tools ran
 * and whether they succeeded. Used for clean post-commit summary.
 */
@@ -651,6 +739,7 @@ function mapGroqError(error, providerLabel) {
 	if (error instanceof Groq.RateLimitError) return /* @__PURE__ */ new Error(`Rate limited by ${label}. Please wait and try again.`);
 	if (error instanceof Groq.APIConnectionTimeoutError) return /* @__PURE__ */ new Error("Request timed out. Check your network or try a smaller diff.");
 	if (error instanceof Groq.APIError) return /* @__PURE__ */ new Error(`${label} API error: ${error.message}`);
+	if (error instanceof Error && /^4\d{2}\s/.test(error.message)) return /* @__PURE__ */ new Error(`${label} API error: ${error.message}`);
 	return /* @__PURE__ */ new Error(`Unexpected error: ${error instanceof Error ? error.message : String(error)}`);
 }
 const CONVENTIONAL_COMMIT_REGEX = /^(build|chore|ci|docs|feat|fix|perf|refactor|revert|style|test)(\(.+\))?!?: .+$/;
@@ -757,6 +846,7 @@ async function generateCommitMessage(diff, options) {
 		debug("callAI: %s — model=%s, promptLen=%d, systemLen=%d", !!strictSystemPrompt ? "RETRY (strict)" : "INITIAL", model, userPrompt.length, (strictSystemPrompt ?? systemPrompt).length);
 		try {
 			const isReasoningModel = /^(o[1-9]|.*gpt-oss.*|.*gpt-5.*)/i.test(model);
+			const isGroq = (options.provider ?? "groq") === "groq";
 			const completion = await client.chat.completions.create({
 				messages: [{
 					role: "system",
@@ -768,7 +858,7 @@ async function generateCommitMessage(diff, options) {
 				model,
 				temperature: .3,
 				...isReasoningModel ? { max_completion_tokens: 1024 } : { max_tokens: 1024 },
-				reasoning_format: "parsed"
+				...isGroq && isReasoningModel ? { reasoning_format: "parsed" } : {}
 			});
 			const elapsed = Date.now() - callStart;
 			const rawContent = completion.choices[0]?.message?.content;
@@ -1523,35 +1613,44 @@ async function runAutoGroupFlow(changedFiles, flags) {
 	if (excluded.length > 0) {
 		debug("Committing %d excluded files upfront:", excluded.length, excluded);
 		const message = buildExcludedFilesMessage(excluded);
+		log.info(excluded.map((f) => `     ${f}`).join("\n"));
 		await resetStaging();
 		await stageFiles(excluded);
 		const headBefore = await getHead();
 		const commitResult = await attemptCommit(message);
 		const headAfter = await getHead();
-		if (commitResult.ok || headBefore !== headAfter) debug("Excluded files committed:", message);
-		else debug("Excluded files commit failed, continuing without them");
+		if (commitResult.ok || headBefore !== headAfter) {
+			debug("Excluded files committed:", message);
+			log.success(dim(message));
+		} else {
+			debug("Excluded files commit failed, continuing without them");
+			log.warn(red("Failed to commit excluded files."));
+		}
 	}
 	if (included.length === 0) {
 		debug("No included files to group, done");
-		outro(green("Done."));
+		if (excluded.length > 0) outro(green("Committed excluded files. No other changes to group."));
+		else outro(dim("Nothing to commit."));
 		return "committed";
 	}
 	if (!flags.noCheck) {
 		const { getRepoRoot } = await Promise.resolve().then(() => git_exports);
 		const repoRoot = await getRepoRoot();
 		const allFiles = included.filter((f) => f.status !== "D").map((f) => f.path);
-		debug("Running user checks on %d files...", allFiles.length);
-		const ck = spinner();
-		ck.start("Running checks...");
-		const checkResults = await runAllChecks(repoRoot, allFiles, 6e4);
-		debug("Check results: ok=%s, count=%d", checkResults.ok, checkResults.results.length);
-		if (!checkResults.ok) {
-			ck.stop(`${checkResults.results.filter((r) => !r.ok).length} check(s) failed`);
-			const rawStderr = checkResults.results.filter((r) => !r.ok).map((r) => `[${r.tool}] ${r.stderr}`).join("\n");
-			if (await showCheckFailureMenu(parseHookErrors(rawStderr), rawStderr) === "cancelled") return "cancelled";
-		} else {
-			ck.stop("All checks passed");
-			if (checkResults.results.length > 0) log.info(checkResults.results.map((r) => `  ${green("✓")} ${r.tool}`).join("\n"));
+		if (await detectConfig(repoRoot)) {
+			debug("Running user checks on %d files...", allFiles.length);
+			const ck = spinner();
+			ck.start("Running checks...");
+			const checkResults = await runAllChecks(repoRoot, allFiles, 6e4);
+			debug("Check results: ok=%s, count=%d", checkResults.ok, checkResults.results.length);
+			if (!checkResults.ok) {
+				ck.stop(`${checkResults.results.filter((r) => !r.ok).length} check(s) failed`);
+				const rawOutput = checkResults.results.filter((r) => !r.ok).map((r) => `[${r.tool}]\n${r.stdout}\n${r.stderr}`.trim()).join("\n\n");
+				if (await showCheckFailureMenu(parseCheckErrors(rawOutput), rawOutput) === "cancelled") return "cancelled";
+			} else {
+				ck.stop("All checks passed");
+				if (checkResults.results.length > 0) log.info(checkResults.results.map((r) => `  ${green("✓")} ${r.tool}`).join("\n"));
+			}
 		}
 	}
 	const config = await readConfig();
@@ -1750,16 +1849,19 @@ async function handleStaging(changedFiles, flags) {
 		}
 		if (stagingResult === "checks") {
 			await stageAll();
-			const ckSpinner = spinner();
-			ckSpinner.start("Running checks...");
-			const ckResult = await runAllChecks(repoRoot, currentFiles.filter((f) => f.status !== "D").map((f) => f.path), 6e4);
-			if (ckResult.ok) {
-				ckSpinner.stop("All checks passed");
-				for (const r of ckResult.results) if (r.stdout.trim()) log.info(dim(r.stdout.trim()));
-			} else {
-				const failed = ckResult.results.filter((r) => !r.ok);
-				ckSpinner.stop(`${failed.length} check${failed.length !== 1 ? "s" : ""} failed`);
-				for (const r of failed) log.info(r.stderr?.trim() || r.stdout?.trim() || `Check failed: ${r.command}`);
+			const allFiles = currentFiles.filter((f) => f.status !== "D").map((f) => f.path);
+			if (await detectConfig(repoRoot)) {
+				const ckSpinner = spinner();
+				ckSpinner.start("Running checks...");
+				const ckResult = await runAllChecks(repoRoot, allFiles, 6e4);
+				if (ckResult.ok) {
+					ckSpinner.stop("All checks passed");
+					for (const r of ckResult.results) if (r.stdout.trim()) log.info(dim(r.stdout.trim()));
+				} else {
+					const failed = ckResult.results.filter((r) => !r.ok);
+					ckSpinner.stop(`${failed.length} check${failed.length !== 1 ? "s" : ""} failed`);
+					for (const r of failed) log.info(r.stderr?.trim() || r.stdout?.trim() || `Check failed: ${r.command}`);
+				}
 			}
 			currentFiles = await getChangedFiles();
 			continue;
@@ -1798,8 +1900,8 @@ async function runPreCommitChecks(changedFiles, noCheck) {
 	const checkResults = await runAllChecks(checkRoot, stagedFileList, 6e4);
 	debug("Check results: ok=%s, count=%d", checkResults.ok, checkResults.results.length);
 	if (!checkResults.ok) {
-		const rawStderr = checkResults.results.filter((r) => !r.ok).map((r) => `[${r.tool}] ${r.stderr}`).join("\n");
-		if (await showCheckFailureMenu(parseHookErrors(rawStderr), rawStderr) === "cancelled") process.exit(1);
+		const rawOutput = checkResults.results.filter((r) => !r.ok).map((r) => `[${r.tool}]\n${r.stdout}\n${r.stderr}`.trim()).join("\n\n");
+		if (await showCheckFailureMenu(parseCheckErrors(rawOutput), rawOutput) === "cancelled") process.exit(1);
 	}
 }
 //#endregion
@@ -1937,10 +2039,11 @@ function maskKey(key) {
 function buildConfigDisplay(config) {
 	const provider = isValidProvider(config.provider ?? "groq") ? config.provider : "groq";
 	const apiKey = config[PROVIDER_ENV_KEYS[provider]];
+	const effectiveModel = getModelForProvider(config, provider, PROVIDER_CONFIGS[provider].defaultModel);
 	return [
 		`Provider:     ${bold(formatProviderName(provider))}`,
 		`API Key:      ${maskKey(apiKey)}`,
-		`Model:        ${config.model ?? "(none)"}`,
+		`Model:        ${effectiveModel}`,
 		`Locale:       ${config.locale ?? "en"}`,
 		`Max Length:   ${config["max-length"] ?? "100"}`,
 		`Commit Type:  ${config.type || dim("(none)")}`,
@@ -2007,11 +2110,23 @@ function getSettingHandlers(config) {
 		provider: async () => {
 			const result = await promptProvider();
 			if (p.isCancel(result)) return result;
-			await writeConfig({ provider: result });
-			debug("config: provider set to %s", result);
+			const newProvider = result;
+			const newDefaultModel = PROVIDER_CONFIGS[newProvider].defaultModel;
+			await writeConfig({
+				provider: newProvider,
+				model: newDefaultModel
+			});
+			debug("config: provider set to %s, model set to %s", newProvider, newDefaultModel);
+			const keyName = PROVIDER_ENV_KEYS[newProvider];
+			if (!(await readConfig())[keyName]) {
+				const keyResult = await promptApiKey(newProvider);
+				if (p.isCancel(keyResult)) return keyResult;
+			}
 		},
 		apikey: async () => promptApiKey(provider),
-		model: async () => promptTextSetting("Model ID:", "model", config.model),
+		model: async () => {
+			return promptTextSetting("Model ID:", "model", getModelForProvider(config, provider, PROVIDER_CONFIGS[provider].defaultModel));
+		},
 		locale: async () => promptTextSetting("Locale (e.g. en, ja, ko):", "locale", config.locale),
 		maxlen: async () => promptTextSetting("Max commit message length:", "max-length", config["max-length"], requireNumber),
 		type: async () => promptTextSetting("Commit type prefix (e.g. conventional):", "type", config.type),
@@ -2030,6 +2145,7 @@ async function editSettingsLoop(initialConfig) {
 	while (true) {
 		config = await readConfig();
 		const provider = getProvider(config);
+		const effectiveModel = getModelForProvider(config, provider, PROVIDER_CONFIGS[provider].defaultModel);
 		const setting = await p.select({
 			message: "Select a setting to edit:",
 			options: [
@@ -2042,7 +2158,7 @@ async function editSettingsLoop(initialConfig) {
 					value: "apikey"
 				},
 				{
-					label: `Model  ${dim(`(${config.model ?? "(none)"})`)}`,
+					label: `Model  ${dim(`(${effectiveModel})`)}`,
 					value: "model"
 				},
 				{