@docker-doctor/cli 0.6.0 → 0.7.0

package/dist/cli.mjs

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { a as runDockerfileRules, c as createComposeLocator, d as discoverProject, f as version, i as runComposeRules, l as parseCompose, n as calculateScore, o as allRules, r as loadConfig, s as findRule, t as toJsonReport, u as parseDockerfile } from "./src-BLeXr_hR.mjs";
+import { a as runDockerfileRules, c as createComposeLocator, d as discoverProject, f as version, i as runComposeRules, l as parseCompose, n as calculateScore, o as allRules, r as loadConfig, s as findRule, t as toJsonReport, u as parseDockerfile } from "./src-D2DWrv5B.mjs";
 import fs from "node:fs/promises";
 import path from "node:path";
 import os from "node:os";
@@ -586,6 +586,7 @@ const writeDiagnosticsDirectory = async (diagnostics, report, rootDir) => {
 			message: sanitizeMessage(failure.message)
 		}))
 	};
+	await fs.writeFile(path.join(dir, ".gitignore"), "*\n", "utf-8");
 	await fs.writeFile(path.join(dir, "diagnostics.json"), JSON.stringify({
 		note: TRUST_BOUNDARY_NOTE,
 		...sanitizedReport
@@ -656,6 +657,7 @@ const buildHandoffPayload = (input) => {
 	const ruleWord = groups.length === 1 ? "rule" : "rules";
 	const lines = [
 		`Fix the ${issueCount} Docker Doctor ${issueWord} (${groups.length} ${ruleWord}) in ${sanitizePath(input.projectName)}.`,
+		`Every path below is relative to ${sanitizePath(input.projectDir)}. Work from that directory.`,
 		TRUST_BOUNDARY_NOTE,
 		""
 	];
@@ -720,8 +722,11 @@ const detectLaunchableAgents = () => {
 
 //#endregion
 //#region src/agents/launch-agent.ts
-const launchAgent = (agentId, prompt) => new Promise((resolve) => {
-	const child = spawn(AGENT_BINARIES[agentId], [...AGENT_AUTO_FLAGS[agentId], prompt], { stdio: "inherit" });
+const launchAgent = (agentId, prompt, cwd) => new Promise((resolve) => {
+	const child = spawn(AGENT_BINARIES[agentId], [...AGENT_AUTO_FLAGS[agentId], prompt], {
+		cwd,
+		stdio: "inherit"
+	});
 	child.once("error", () => {
 		resolve(false);
 	});
@@ -775,9 +780,9 @@ const printDiscoveredFiles = (project) => {
 	console.log(`  Dockerfile(s): ${project.dockerfiles.length ? project.dockerfiles.map((f) => chalk.cyan(f)).join(", ") : chalk.dim("None")}`);
 	console.log(`  Compose file(s):  ${project.composeFiles.length ? project.composeFiles.map((f) => chalk.cyan(f)).join(", ") : chalk.dim("None")}`);
 };
-const printDiagnostics = (diagnostics, verbose, fileContents, categoryIssueCounts) => {
+const printDiagnostics = (diagnostics, verbose, fileContents, categoryIssueCounts, unanalyzedCount) => {
 	if (diagnostics.length === 0) {
-		console.log(`\n${chalk.green.bold("✔ No issues found! Your Docker setup looks healthy.")}`);
+		console.log(unanalyzedCount === 0 ? `\n${chalk.green.bold("✔ No issues found! Your Docker setup looks healthy.")}` : `\n${chalk.yellow.bold("⚠ No issues found in the files that could be analyzed.")}`);
 		return;
 	}
 	if (!verbose) {
@@ -866,7 +871,7 @@ const getWhaleMascot = (score, border) => {
 	];
 };
 const easeOutCubic = (x) => 1 - (1 - x) ** 3;
-const printScoreBox = async (score, label, warningsCount, errorsCount) => {
+const printScoreBox = async (score, label, warningsCount, errorsCount, unanalyzedCount) => {
 	const { isTTY } = process.stdout;
 	const shouldAnimate = isTTY && !process.env.CI && !process.env.NO_ANIMATION && process.env.TERM !== "dumb" && process.env.NODE_ENV !== "test";
 	let scoreColor = chalk.red.bold;
@@ -910,15 +915,21 @@ const printScoreBox = async (score, label, warningsCount, errorsCount) => {
 		console.log(`  ${whaleLines[3]}`);
 	}
 	console.log(`\n  ${chalk.dim("────────────────────────────────────────────────────────────")}\n`);
-	console.log(`  Share: ${chalk.cyan(shareUrl)}`);
-	console.log(`  Tell others how you did on socials\n`);
+	if (unanalyzedCount === 0) {
+		console.log(`  Share: ${chalk.cyan(shareUrl)}`);
+		console.log(`  Tell others how you did on socials\n`);
+	} else {
+		const fileWord = unanalyzedCount === 1 ? "file" : "files";
+		console.log(`  ${chalk.yellow(`⚠ Incomplete scan: ${unanalyzedCount} ${fileWord} could not be analyzed.`)}`);
+		console.log(`  The score covers only the analyzed files.\n`);
+	}
 	console.log(`  Docs: ${chalk.cyan("https://docker-doctor.vercel.app")}`);
 	console.log(`  Learn more about fixing issues, setting up CI/CD, and`);
 	console.log(`  configuring rules with a config file\n`);
 	console.log(`  GitHub: ${chalk.cyan("https://github.com/PunGrumpy/docker-doctor")}`);
 	console.log(`  Report issues and star the repository!`);
 };
-const formatTerminal = async (diagnostics, score, label, project, verbose = false, fileContents = {}) => {
+const formatTerminal = async (diagnostics, score, label, project, verbose = false, fileContents = {}, unanalyzedCount = 0) => {
 	if (verbose) {
 		console.log(`\n${chalk.bold("Docker Doctor Diagnostics")}`);
 		console.log(`=================================`);
@@ -939,8 +950,8 @@ const formatTerminal = async (diagnostics, score, label, project, verbose = fals
 		if (d.severity === "warning") warningsCount += 1;
 		else if (d.severity === "error") errorsCount += 1;
 	}
-	printDiagnostics(diagnostics, verbose, fileContents, categoryIssueCounts);
-	await printScoreBox(score, label, warningsCount, errorsCount);
+	printDiagnostics(diagnostics, verbose, fileContents, categoryIssueCounts, unanalyzedCount);
+	await printScoreBox(score, label, warningsCount, errorsCount, unanalyzedCount);
 };
 
 //#endregion
@@ -954,8 +965,25 @@ const fileExists = async (filePath) => {
 		return false;
 	}
 };
+const selfAndAncestors = (dir) => {
+	const dirs = [dir];
+	let parent = path.dirname(dir);
+	while (parent !== dirs.at(-1)) {
+		dirs.push(parent);
+		parent = path.dirname(parent);
+	}
+	return dirs;
+};
+const findRepositoryRoot = async (dir) => {
+	const candidates = selfAndAncestors(dir);
+	const nearest = (await Promise.all(candidates.map((candidate) => fileExists(path.join(candidate, ".git"))))).indexOf(true);
+	return nearest === -1 ? null : candidates[nearest];
+};
 const scaffoldActionWorkflow = async (options) => {
-	const workflowDir = path.join(options.rootDir, ".github", "workflows");
+	const repositoryRoot = await findRepositoryRoot(options.rootDir) ?? options.rootDir;
+	const scanDirectory = path.relative(repositoryRoot, options.rootDir).split(path.sep).join("/");
+	const directoryInput = scanDirectory === "" ? "" : `        with:\n          directory: ${JSON.stringify(scanDirectory)}\n`;
+	const workflowDir = path.join(repositoryRoot, ".github", "workflows");
 	const workflowPath = path.join(workflowDir, "docker-doctor.yml");
 	const existing = await fileExists(workflowPath);
 	if (existing && !await options.confirmOverwrite()) return "kept";
@@ -973,7 +1001,7 @@ jobs:
     steps:
       - uses: actions/checkout@v7
       - uses: ${options.actionRef}
-`;
+${directoryInput}`;
 	await fs.mkdir(workflowDir, { recursive: true });
 	await fs.writeFile(workflowPath, workflowYaml, "utf-8");
 	return existing ? "updated" : "created";
@@ -1165,6 +1193,7 @@ const runAgentHandoff = async (context) => {
 	await ensureGitignoreEntry(context.rootDir);
 	const payload = buildHandoffPayload({
 		diagnostics: context.diagnostics,
+		projectDir: context.rootDir,
 		projectName: path.basename(context.rootDir)
 	});
 	if (choice === clipboardIndex) {
@@ -1173,14 +1202,14 @@ const runAgentHandoff = async (context) => {
 		return;
 	}
 	const agentId = launchable[choice];
-	if (!await askConfirm(`Launch ${AGENT_BINARIES[agentId]} with ${AGENT_AUTO_FLAGS[agentId].join(" ")}? It will edit files without asking for approval.`)) {
+	if (!await askConfirm(`Launch ${AGENT_BINARIES[agentId]} with ${AGENT_AUTO_FLAGS[agentId].join(" ")} in ${context.rootDir}? It will edit files without asking for approval.`)) {
 		printAgentPrompt(payload);
 		return;
 	}
 	const installResult = await installSkillForAgents([agentId], { projectRoot: context.rootDir });
 	if (installResult && installResult.installed.length > 0) console.log(`\n  ${chalk.green("✔")} Installed the docker-doctor skill for ${agentDisplayName(agentId)}`);
 	console.log(`\n  Handing off to ${agentDisplayName(agentId)}...\n`);
-	if (!await launchAgent(agentId, payload)) {
+	if (!await launchAgent(agentId, payload, context.rootDir)) {
 		console.log(`  ${chalk.yellow("⚠")} Couldn't launch ${AGENT_BINARIES[agentId]}. Here's the prompt instead:`);
 		printAgentPrompt(payload);
 	}
@@ -1276,6 +1305,13 @@ const reportScanFailures = (failures) => {
 	console.error(`Docker Doctor could not analyze ${failures.length} ${fileWord}; the score below covers only the files that were analyzed.`);
 	for (const failure of failures) console.error(`  - ${failure.file}: ${flattenMessage(failure.message)}`);
 };
+const noFilesFoundMessage = (rootDir) => `No Dockerfiles or Compose files found in ${rootDir}`;
+const printNothingScanned = (rootDir) => {
+	console.log(`\n${chalk.yellow("⚠")} ${noFilesFoundMessage(rootDir)}.`);
+	console.log(`  docker-doctor scans files named Dockerfile, Dockerfile.*, *.dockerfile,`);
+	console.log(`  compose.yaml, docker-compose.yaml and variants such as compose.prod.yaml`);
+	console.log(`  (.yaml or .yml). Check the directory and your ignore.files patterns.`);
+};
 const pathExists = async (filePath) => {
 	try {
 		await fs.access(filePath);
@@ -1305,8 +1341,9 @@ const program = new Command();
 program.name("docker-doctor").description("Static analysis for Dockerfile and Docker Compose files").version(version, "-V, --version", "display the version number");
 program.argument("[dir]", "directory to scan", ".").option("-v, --verbose", "show verbose diagnostics description", false).option("-s, --score", "only output numeric health score", false).option("-j, --json", "output results as JSON report", false).option("-c, --config <path>", "custom config file path").action(async (dir, options) => {
 	const isSilent = options.score || options.json;
+	const showProgress = Boolean(process.stdout.isTTY) && !isSilent;
 	const restoreCursor = () => {
-		if (process.stdout.isTTY && !isSilent) process.stdout.write("\x1B[?25h");
+		if (showProgress) process.stdout.write("\x1B[?25h");
 	};
 	process.on("exit", restoreCursor);
 	process.once("SIGINT", () => {
@@ -1344,7 +1381,7 @@ program.argument("[dir]", "directory to scan", ".").option("-v, --verbose", "sho
 		const setStatus = (text) => {
 			statusText = text;
 		};
-		if (process.stdout.isTTY && !isSilent) {
+		if (showProgress) {
 			process.stdout.write("\x1B[?25l");
 			process.stdout.write(`${chalk.cyan(spinnerFrames[0])} ${statusText}`);
 			spinnerInterval = setInterval(() => {
@@ -1353,7 +1390,7 @@ program.argument("[dir]", "directory to scan", ".").option("-v, --verbose", "sho
 			}, 80);
 		}
 		try {
-			if (process.stdout.isTTY && !isSilent) await setTimeout(150);
+			if (showProgress) await setTimeout(150);
 			setStatus("Loading configuration...");
 			const configPath = await resolveConfigPath(options.config, rootDir);
 			const config = await loadConfig(rootDir, configPath, (message) => {
@@ -1372,8 +1409,9 @@ program.argument("[dir]", "directory to scan", ".").option("-v, --verbose", "sho
 				...project.composeFiles,
 				...project.dockerignores || []
 			];
+			const scannedFileCount = project.dockerfiles.length + project.composeFiles.length;
 			const { diagnostics, failures } = await runRulesEngine(rootDir, project, config.rules, config.categories, projectFilesList, fileContents, options, setStatus);
-			const { score, label } = calculateScore(diagnostics);
+			const { score, label } = calculateScore(diagnostics, scannedFileCount - failures.length);
 			const duration = ((Date.now() - startTime) / 1e3).toFixed(1);
 			const concurrency = os.cpus().length;
 			if (spinnerInterval !== null) {
@@ -1381,9 +1419,17 @@ program.argument("[dir]", "directory to scan", ".").option("-v, --verbose", "sho
 				spinnerInterval = null;
 				process.stdout.write("\r\x1B[K\x1B[?25h");
 			}
-			if (process.stdout.isTTY && !isSilent) console.log(`${chalk.green("✔")} Scanned ${projectFilesList.length} files in ${duration}s [~${concurrency} workers]`);
+			if (showProgress) console.log(`${chalk.green("✔")} Scanned ${scannedFileCount} files in ${duration}s [~${concurrency} workers]`);
 			reportScanFailures(failures);
 			const scanIncomplete = failures.length > 0;
+			if (scannedFileCount === 0) {
+				if (isSilent) console.error(`Warning: ${noFilesFoundMessage(rootDir)}`);
+				else {
+					printNothingScanned(rootDir);
+					process.exitCode = 0;
+					return;
+				}
+			}
 			const hasErrors = diagnostics.some((d) => d.severity === "error");
 			if (options.score) {
 				console.log(score);
@@ -1395,7 +1441,7 @@ program.argument("[dir]", "directory to scan", ".").option("-v, --verbose", "sho
 				process.exitCode = scanExitCode(scanIncomplete, hasErrors);
 				return;
 			}
-			await formatTerminal(diagnostics, score, label, project, options.verbose, fileContents);
+			await formatTerminal(diagnostics, score, label, project, options.verbose, fileContents, failures.length);
 			let wizardOk = true;
 			if (process.stdout.isTTY && process.stdin.isTTY) wizardOk = await runInteractiveWizard({
 				diagnostics,