aislop 0.2.0 → 0.2.1

package/dist/index.js

@@ -1,10 +1,10 @@
-import { n as getEngineLabel, r as APP_VERSION, t as ENGINE_INFO } from "./engine-info-DpU0WTTj.js";
+import { n as getEngineLabel, r as APP_VERSION, t as ENGINE_INFO } from "./engine-info-DFze-2GQ.js";
 import { n as runSubprocess, t as isToolInstalled } from "./subprocess-99puEEGl.js";
 import { createRequire } from "node:module";
 import fs from "node:fs";
 import path from "node:path";
 import pc from "picocolors";
-import { spawn, spawnSync } from "node:child_process";
+import { spawnSync } from "node:child_process";
 import { fileURLToPath } from "node:url";
 import { performance } from "node:perf_hooks";
 import os from "node:os";
@@ -52,8 +52,7 @@ const logger = {
 //#region src/output/layout.ts
 const formatElapsed$1 = (elapsedMs) => elapsedMs < 1e3 ? `${Math.round(elapsedMs)}ms` : `${(elapsedMs / 1e3).toFixed(1)}s`;
 const printCommandHeader = (commandName) => {
-	logger.log(highlighter.bold(`aislop ${commandName}`));
-	logger.log(highlighter.dim(`v${APP_VERSION}`));
+	logger.log(`${highlighter.bold(`aislop ${commandName.toLowerCase()}`)} ${highlighter.dim(`v${APP_VERSION}`)}`);
 	logger.break();
 };
 const formatProjectSummary = (project) => `Project ${highlighter.info(project.projectName)} (${highlighter.info(project.languages.join(", "))})`;
@@ -530,193 +529,272 @@ const doctorCommand = async (directory) => {
 };
 
 //#endregion
-//#region src/engines/format/biome.ts
-const esmRequire$1 = createRequire(import.meta.url);
-const resolveLocalBiomeScript = () => {
-	try {
-		const packageJsonPath = esmRequire$1.resolve("@biomejs/biome/package.json");
-		return path.join(path.dirname(packageJsonPath), "bin", "biome");
-	} catch {
-		return null;
-	}
-};
-const runBiome = async (args, rootDirectory, timeout) => {
-	const localScript = resolveLocalBiomeScript();
-	if (localScript) return runSubprocess(process.execPath, [localScript, ...args], {
-		cwd: rootDirectory,
-		timeout
-	});
-	return runSubprocess("biome", args, {
-		cwd: rootDirectory,
-		timeout
-	});
-};
-const BIOME_EXTENSIONS = new Set([
-	".js",
-	".jsx",
+//#region src/engines/ai-slop/unused-imports.ts
+const JS_EXTENSIONS$1 = new Set([
 	".ts",
 	".tsx",
+	".js",
+	".jsx",
 	".mjs",
 	".cjs"
 ]);
-const getBiomeTargets = (context) => getSourceFiles(context).filter((filePath) => BIOME_EXTENSIONS.has(path.extname(filePath))).map((filePath) => path.relative(context.rootDirectory, filePath));
-const projectUsesDecorators = (rootDir) => {
-	try {
-		const tsconfigPath = path.join(rootDir, "tsconfig.json");
-		if (!fs.existsSync(tsconfigPath)) return false;
-		const content = fs.readFileSync(tsconfigPath, "utf-8");
-		return /experimentalDecorators.*true/i.test(content);
-	} catch {
-		return false;
-	}
-};
-const runBiomeFormat = async (context) => {
-	const targets = getBiomeTargets(context);
-	if (targets.length === 0) return [];
-	const args = [
-		"format",
-		"--reporter=json",
-		...targets
-	];
-	try {
-		const result = await runBiome(args, context.rootDirectory, 6e4);
-		const output = [result.stdout, result.stderr].filter(Boolean).join("\n");
-		if (!output) return [];
-		let diagnostics = parseBiomeJsonOutput(output, context.rootDirectory);
-		if (projectUsesDecorators(context.rootDirectory)) diagnostics = diagnostics.filter((d) => {
-			const msg = d.message.toLowerCase();
-			return !msg.includes("decorator") && !msg.includes("parsing error");
+const PY_EXTENSIONS = new Set([".py"]);
+const REMOVE_MARKER = "\0__AISLOP_REMOVE__";
+const extractJsImportedSymbols = (lines) => {
+	const symbols = [];
+	const importLines = /* @__PURE__ */ new Set();
+	for (let i = 0; i < lines.length; i++) {
+		const trimmed = lines[i].trim();
+		if (!trimmed.startsWith("import ")) continue;
+		importLines.add(i);
+		if (/^import\s+["']/.test(trimmed)) continue;
+		if (/^import\s+type\s/.test(trimmed)) continue;
+		let fullImport = trimmed;
+		let endLine = i;
+		while (!fullImport.includes("from") && endLine < lines.length - 1) {
+			endLine++;
+			fullImport += ` ${lines[endLine].trim()}`;
+			importLines.add(endLine);
+		}
+		const namespaceMatch = fullImport.match(/import\s+\*\s+as\s+(\w+)\s+from/);
+		if (namespaceMatch) {
+			symbols.push({
+				name: namespaceMatch[1],
+				line: i + 1,
+				isDefault: false,
+				isNamespace: true
+			});
+			continue;
+		}
+		const defaultMatch = fullImport.match(/import\s+(\w+)\s*(?:,\s*\{[^}]*\})?\s+from/);
+		if (defaultMatch && defaultMatch[1] !== "type") symbols.push({
+			name: defaultMatch[1],
+			line: i + 1,
+			isDefault: true,
+			isNamespace: false
 		});
-		return diagnostics;
-	} catch {
-		return [];
+		const namedMatch = fullImport.match(/\{([^}]+)\}/);
+		if (namedMatch) {
+			const namedImports = namedMatch[1].split(",");
+			for (const ni of namedImports) {
+				const parts = ni.trim().split(/\s+as\s+/);
+				if (parts.length === 0 || !parts[0]) continue;
+				const cleanParts = parts.map((p) => p.trim().replace(/^type\s+/, ""));
+				const localName = cleanParts.length > 1 ? cleanParts[1] : cleanParts[0];
+				if (localName && /^\w+$/.test(localName)) symbols.push({
+					name: localName,
+					line: i + 1,
+					isDefault: false,
+					isNamespace: false
+				});
+			}
+		}
 	}
+	return {
+		symbols,
+		importLines
+	};
 };
-const parseBiomeJsonOutput = (output, rootDir) => {
-	const diagnostics = [];
-	for (const line of output.split("\n")) {
-		const trimmed = line.trim();
-		if (!trimmed.startsWith("{")) continue;
-		let parsed = null;
-		try {
-			parsed = JSON.parse(trimmed);
-		} catch {
-			parsed = null;
+const extractPyImportedSymbols = (lines) => {
+	const symbols = [];
+	const importLines = /* @__PURE__ */ new Set();
+	for (let i = 0; i < lines.length; i++) {
+		const trimmed = lines[i].trim();
+		const fromMatch = trimmed.match(/^from\s+[\w.]+\s+import\s+(.+)/);
+		if (fromMatch) {
+			importLines.add(i);
+			const importPart = fromMatch[1].replace(/#.*$/, "").trim();
+			if (importPart === "*") continue;
+			const cleaned = importPart.replace(/[()]/g, "");
+			for (const item of cleaned.split(",")) {
+				const parts = item.trim().split(/\s+as\s+/);
+				const localName = parts.length > 1 ? parts[1].trim() : parts[0].trim();
+				if (localName && /^\w+$/.test(localName)) symbols.push({
+					name: localName,
+					line: i + 1,
+					isDefault: false,
+					isNamespace: false
+				});
+			}
+			continue;
 		}
-		if (!parsed || !Array.isArray(parsed.diagnostics)) continue;
-		for (const entry of parsed.diagnostics) {
-			const rawPath = entry.location?.path;
-			if (!rawPath) continue;
-			const severity = entry.severity === "error" ? "error" : "warning";
-			diagnostics.push({
-				filePath: path.isAbsolute(rawPath) ? path.relative(rootDir, rawPath) : rawPath,
-				engine: "format",
-				rule: "formatting",
-				severity,
-				message: entry.message ?? "File is not formatted correctly",
-				help: "Run `aislop fix` to auto-format",
-				line: entry.location?.start?.line ?? 0,
-				column: entry.location?.start?.column ?? 0,
-				category: "Format",
-				fixable: true
+		const importMatch = trimmed.match(/^import\s+([\w.]+)(?:\s+as\s+(\w+))?/);
+		if (importMatch) {
+			importLines.add(i);
+			const simpleName = (importMatch[2] ?? importMatch[1]).split(".")[0];
+			if (simpleName && /^\w+$/.test(simpleName)) symbols.push({
+				name: simpleName,
+				line: i + 1,
+				isDefault: false,
+				isNamespace: true
 			});
 		}
 	}
-	return diagnostics;
-};
-const fixBiomeFormat = async (context) => {
-	const targets = getBiomeTargets(context);
-	if (targets.length === 0) return;
-	const result = await runBiome([
-		"check",
-		"--write",
-		"--formatter-enabled=true",
-		"--linter-enabled=false",
-		...targets
-	], context.rootDirectory, 6e4);
-	if (result.exitCode !== 0) throw new Error(result.stderr || result.stdout || `Biome exited with code ${result.exitCode}`);
+	return {
+		symbols,
+		importLines
+	};
 };
-
-//#endregion
-//#region src/engines/format/gofmt.ts
-const runGofmt = async (context) => {
-	try {
-		const result = await runSubprocess("gofmt", ["-l", context.rootDirectory], {
-			cwd: context.rootDirectory,
-			timeout: 6e4
-		});
-		if (!result.stdout) return [];
-		return result.stdout.split("\n").filter((f) => f.length > 0).map((file) => ({
-			filePath: path.relative(context.rootDirectory, file),
-			engine: "format",
-			rule: "go-formatting",
-			severity: "warning",
-			message: "Go file is not formatted correctly",
-			help: "Run `aislop fix` to auto-format with gofmt",
-			line: 0,
-			column: 0,
-			category: "Format",
-			fixable: true
-		}));
-	} catch {
-		return [];
+const isSymbolUsed = (name, content, importLines, lines) => {
+	const pattern = new RegExp(`\\b${name}\\b`, "g");
+	let match;
+	while ((match = pattern.exec(content)) !== null) {
+		const lineIndex = content.slice(0, match.index).split("\n").length - 1;
+		if (!importLines.has(lineIndex)) return true;
 	}
+	for (let i = 0; i < lines.length; i++) {
+		if (importLines.has(i)) continue;
+		if (lines[i].includes(name)) return true;
+	}
+	return false;
 };
-const fixGofmt = async (rootDirectory) => {
-	const result = await runSubprocess("gofmt", ["-w", rootDirectory], {
-		cwd: rootDirectory,
-		timeout: 6e4
-	});
-	if (result.exitCode !== 0) throw new Error(result.stderr || result.stdout || `gofmt exited with code ${result.exitCode}`);
-};
-
-//#endregion
-//#region src/engines/format/ruff-format.ts
-const runRuffFormat = async (context) => {
-	const ruffBinary = resolveToolBinary("ruff");
+const analyzeFile = (filePath) => {
+	if (isAutoGenerated(filePath)) return null;
+	let content;
 	try {
-		const result = await runSubprocess(ruffBinary, [
-			"format",
-			"--check",
-			"--diff",
-			context.rootDirectory
-		], {
-			cwd: context.rootDirectory,
-			timeout: 6e4
-		});
-		if (result.exitCode === 0) return [];
-		return parseRuffFormatOutput(result.stdout || result.stderr, context.rootDirectory);
+		content = fs.readFileSync(filePath, "utf-8");
 	} catch {
-		return [];
+		return null;
 	}
+	const ext = path.extname(filePath);
+	const lines = content.split("\n");
+	let symbols;
+	let importLines;
+	if (JS_EXTENSIONS$1.has(ext)) {
+		const result = extractJsImportedSymbols(lines);
+		symbols = result.symbols;
+		importLines = result.importLines;
+	} else if (PY_EXTENSIONS.has(ext)) {
+		const result = extractPyImportedSymbols(lines);
+		symbols = result.symbols;
+		importLines = result.importLines;
+	} else return null;
+	return {
+		lines,
+		symbols,
+		importLines,
+		ext
+	};
 };
-const parseRuffFormatOutput = (output, rootDir) => {
+const getUnusedSymbols = (lines, symbols, importLines) => {
+	const content = lines.join("\n");
+	return symbols.filter((symbol) => !isSymbolUsed(symbol.name, content, importLines, lines));
+};
+const detectUnusedImports = async (context) => {
+	const files = getSourceFiles(context);
 	const diagnostics = [];
-	const filePattern = /^--- (.+)$/gm;
-	let match;
-	while ((match = filePattern.exec(output)) !== null) {
-		const filePath = match[1].replace(/^a\//, "");
-		diagnostics.push({
-			filePath: path.relative(rootDir, filePath),
-			engine: "format",
-			rule: "python-formatting",
+	for (const filePath of files) {
+		const analysis = analyzeFile(filePath);
+		if (!analysis) continue;
+		const relativePath = path.relative(context.rootDirectory, filePath);
+		const unused = getUnusedSymbols(analysis.lines, analysis.symbols, analysis.importLines);
+		for (const symbol of unused) diagnostics.push({
+			filePath: relativePath,
+			engine: "ai-slop",
+			rule: "ai-slop/unused-import",
 			severity: "warning",
-			message: "Python file is not formatted correctly",
-			help: "Run `aislop fix` to auto-format with ruff",
-			line: 0,
+			message: `Imported symbol '${symbol.name}' is never used`,
+			help: "Remove unused imports to keep the code clean",
+			line: symbol.line,
 			column: 0,
-			category: "Format",
+			category: "AI Slop",
 			fixable: true
 		});
 	}
 	return diagnostics;
 };
-const fixRuffFormat = async (rootDirectory) => {
-	const result = await runSubprocess(resolveToolBinary("ruff"), ["format", rootDirectory], {
-		cwd: rootDirectory,
-		timeout: 6e4
+
+//#endregion
+//#region src/engines/ai-slop/unused-imports-fix.ts
+const fixUnusedImports = async (context) => {
+	const files = getSourceFiles(context);
+	for (const filePath of files) {
+		const analysis = analyzeFile(filePath);
+		if (!analysis) continue;
+		const unused = getUnusedSymbols(analysis.lines, analysis.symbols, analysis.importLines);
+		if (unused.length === 0) continue;
+		const unusedNames = new Set(unused.map((u) => u.name));
+		const lines = [...analysis.lines];
+		const symbolsByLine = /* @__PURE__ */ new Map();
+		for (const sym of analysis.symbols) {
+			const arr = symbolsByLine.get(sym.line) ?? [];
+			arr.push(sym);
+			symbolsByLine.set(sym.line, arr);
+		}
+		const linesToRemove = /* @__PURE__ */ new Set();
+		for (const [lineNo, syms] of symbolsByLine) {
+			const lineIdx = lineNo - 1;
+			const allUnused = syms.every((s) => unusedNames.has(s.name));
+			const importSpan = getImportSpan(lineIdx, analysis.importLines);
+			if (allUnused) for (const idx of importSpan) linesToRemove.add(idx);
+			else if (JS_EXTENSIONS$1.has(analysis.ext)) rewriteJsImportSpan(lines, importSpan, syms, unusedNames);
+			else if (PY_EXTENSIONS.has(analysis.ext)) rewritePyImportLine(lines, lineIdx, syms, unusedNames);
+		}
+		if (linesToRemove.size === 0 && unused.length === 0) continue;
+		const sortedRemove = [...linesToRemove].sort((a, b) => b - a);
+		for (const idx of sortedRemove) lines.splice(idx, 1);
+		const filtered = lines.filter((l) => l !== REMOVE_MARKER);
+		while (filtered.length > 0 && filtered[0].trim() === "") filtered.shift();
+		fs.writeFileSync(filePath, filtered.join("\n"));
+	}
+};
+const getImportSpan = (startIdx, importLines) => {
+	const span = [startIdx];
+	let idx = startIdx + 1;
+	while (importLines.has(idx)) {
+		span.push(idx);
+		idx++;
+	}
+	return span;
+};
+const rewriteJsImportSpan = (lines, span, syms, unusedNames) => {
+	const fullImport = span.map((i) => lines[i]).join("\n");
+	const namedMatch = fullImport.match(/\{([^}]+)\}/s);
+	if (!namedMatch) return;
+	const unusedNamed = syms.filter((s) => !s.isDefault && !s.isNamespace && unusedNames.has(s.name));
+	if (unusedNamed.length === 0) return;
+	const unusedNamedSet = new Set(unusedNamed.map((s) => s.name));
+	const keptSpecifiers = namedMatch[1].split(",").map((s) => s.trim()).filter(Boolean).filter((spec) => {
+		const parts = spec.split(/\s+as\s+/);
+		const localName = parts.length > 1 ? parts[1].trim().replace(/^type\s+/, "") : parts[0].trim().replace(/^type\s+/, "");
+		return !unusedNamedSet.has(localName);
 	});
-	if (result.exitCode !== 0) throw new Error(result.stderr || result.stdout || `ruff format exited with code ${result.exitCode}`);
+	if (keptSpecifiers.length === 0) {
+		if (syms.find((s) => s.isDefault && !unusedNames.has(s.name))) {
+			const rewritten = fullImport.replace(/,\s*\{[^}]*\}/s, "").replace(/\{[^}]*\}\s*,?\s*/s, "");
+			lines[span[0]] = rewritten.replace(/\n/g, " ").replace(/\s+/g, " ");
+			for (let i = 1; i < span.length; i++) lines[span[i]] = REMOVE_MARKER;
+		}
+		return;
+	}
+	const fromMatch = fullImport.match(/\}\s*(from\s+.+)$/s);
+	const fromClause = fromMatch ? fromMatch[1].trim() : "";
+	const importPrefix = fullImport.match(/^(import\s+(?:\w+\s*,\s*)?)/);
+	const prefix = importPrefix ? importPrefix[1] : "import ";
+	const wasMultiLine = span.length > 1;
+	let newImport;
+	if (wasMultiLine && keptSpecifiers.length > 2) {
+		const indentMatch = lines[span[1]]?.match(/^(\s+)/);
+		const indent = indentMatch ? indentMatch[1] : "	";
+		newImport = `${prefix}{\n${keptSpecifiers.map((s) => `${indent}${s},`).join("\n")}\n} ${fromClause}`;
+	} else newImport = `${prefix}{ ${keptSpecifiers.join(", ")} } ${fromClause}`;
+	lines[span[0]] = newImport;
+	for (let i = 1; i < span.length; i++) lines[span[i]] = REMOVE_MARKER;
+};
+const rewritePyImportLine = (lines, lineIdx, syms, unusedNames) => {
+	const fromMatch = lines[lineIdx].match(/^(\s*from\s+[\w.]+\s+import\s+)(.+)$/);
+	if (!fromMatch) return;
+	const prefix = fromMatch[1];
+	const importPart = fromMatch[2].replace(/#.*$/, "").trim();
+	const hasParen = importPart.startsWith("(");
+	const keptSpecifiers = importPart.replace(/[()]/g, "").split(",").map((s) => s.trim()).filter((spec) => {
+		const parts = spec.split(/\s+as\s+/);
+		const localName = parts.length > 1 ? parts[1].trim() : parts[0].trim();
+		return !unusedNames.has(localName);
+	});
+	if (keptSpecifiers.length === 0) return;
+	const joined = keptSpecifiers.join(", ");
+	lines[lineIdx] = hasParen ? `${prefix}(${joined})` : `${prefix}${joined}`;
 };
 
 //#endregion
@@ -862,26 +940,213 @@ const runKnip = async (rootDirectory) => {
 			engine: "code-quality",
 			rule: "knip/files",
 			severity: "warning",
-			message: KNIP_MESSAGE_MAP.files,
-			help: "This file is not imported by any other file in the project.",
+			message: KNIP_MESSAGE_MAP.files,
+			help: "This file is not imported by any other file in the project.",
+			line: 0,
+			column: 0,
+			category: "Dead Code",
+			fixable: false
+		});
+		const issues = parsed.issues ?? [];
+		const issueTypes = [
+			...DEPENDENCY_TYPES,
+			"exports",
+			"types",
+			"duplicates"
+		];
+		for (const fileIssue of issues) for (const type of issueTypes) diagnostics.push(...collectIssues(fileIssue, type, rootDirectory, knipRuntime.cwd));
+		return diagnostics;
+	} catch {
+		return [];
+	}
+};
+
+//#endregion
+//#region src/engines/format/biome.ts
+const esmRequire$1 = createRequire(import.meta.url);
+const resolveLocalBiomeScript = () => {
+	try {
+		const packageJsonPath = esmRequire$1.resolve("@biomejs/biome/package.json");
+		return path.join(path.dirname(packageJsonPath), "bin", "biome");
+	} catch {
+		return null;
+	}
+};
+const runBiome = async (args, rootDirectory, timeout) => {
+	const localScript = resolveLocalBiomeScript();
+	if (localScript) return runSubprocess(process.execPath, [localScript, ...args], {
+		cwd: rootDirectory,
+		timeout
+	});
+	return runSubprocess("biome", args, {
+		cwd: rootDirectory,
+		timeout
+	});
+};
+const BIOME_EXTENSIONS = new Set([
+	".js",
+	".jsx",
+	".ts",
+	".tsx",
+	".mjs",
+	".cjs"
+]);
+const getBiomeTargets = (context) => getSourceFiles(context).filter((filePath) => BIOME_EXTENSIONS.has(path.extname(filePath))).map((filePath) => path.relative(context.rootDirectory, filePath));
+const projectUsesDecorators = (rootDir) => {
+	try {
+		const tsconfigPath = path.join(rootDir, "tsconfig.json");
+		if (!fs.existsSync(tsconfigPath)) return false;
+		const content = fs.readFileSync(tsconfigPath, "utf-8");
+		return /experimentalDecorators.*true/i.test(content);
+	} catch {
+		return false;
+	}
+};
+const runBiomeFormat = async (context) => {
+	const targets = getBiomeTargets(context);
+	if (targets.length === 0) return [];
+	const args = [
+		"format",
+		"--reporter=json",
+		...targets
+	];
+	try {
+		const result = await runBiome(args, context.rootDirectory, 6e4);
+		const output = [result.stdout, result.stderr].filter(Boolean).join("\n");
+		if (!output) return [];
+		let diagnostics = parseBiomeJsonOutput(output, context.rootDirectory);
+		if (projectUsesDecorators(context.rootDirectory)) diagnostics = diagnostics.filter((d) => {
+			const msg = d.message.toLowerCase();
+			return !msg.includes("decorator") && !msg.includes("parsing error");
+		});
+		return diagnostics;
+	} catch {
+		return [];
+	}
+};
+const parseBiomeJsonOutput = (output, rootDir) => {
+	const diagnostics = [];
+	for (const line of output.split("\n")) {
+		const trimmed = line.trim();
+		if (!trimmed.startsWith("{")) continue;
+		let parsed = null;
+		try {
+			parsed = JSON.parse(trimmed);
+		} catch {
+			parsed = null;
+		}
+		if (!parsed || !Array.isArray(parsed.diagnostics)) continue;
+		for (const entry of parsed.diagnostics) {
+			const rawPath = entry.location?.path;
+			if (!rawPath) continue;
+			const severity = entry.severity === "error" ? "error" : "warning";
+			diagnostics.push({
+				filePath: path.isAbsolute(rawPath) ? path.relative(rootDir, rawPath) : rawPath,
+				engine: "format",
+				rule: "formatting",
+				severity,
+				message: entry.message ?? "File is not formatted correctly",
+				help: "Run `aislop fix` to auto-format",
+				line: entry.location?.start?.line ?? 0,
+				column: entry.location?.start?.column ?? 0,
+				category: "Format",
+				fixable: true
+			});
+		}
+	}
+	return diagnostics;
+};
+const fixBiomeFormat = async (context) => {
+	const targets = getBiomeTargets(context);
+	if (targets.length === 0) return;
+	await runBiome([
+		"format",
+		"--write",
+		...targets
+	], context.rootDirectory, 6e4);
+};
+
+//#endregion
+//#region src/engines/format/gofmt.ts
+const runGofmt = async (context) => {
+	try {
+		const result = await runSubprocess("gofmt", ["-l", context.rootDirectory], {
+			cwd: context.rootDirectory,
+			timeout: 6e4
+		});
+		if (!result.stdout) return [];
+		return result.stdout.split("\n").filter((f) => f.length > 0).map((file) => ({
+			filePath: path.relative(context.rootDirectory, file),
+			engine: "format",
+			rule: "go-formatting",
+			severity: "warning",
+			message: "Go file is not formatted correctly",
+			help: "Run `aislop fix` to auto-format with gofmt",
 			line: 0,
 			column: 0,
-			category: "Dead Code",
-			fixable: false
+			category: "Format",
+			fixable: true
+		}));
+	} catch {
+		return [];
+	}
+};
+const fixGofmt = async (rootDirectory) => {
+	const result = await runSubprocess("gofmt", ["-w", rootDirectory], {
+		cwd: rootDirectory,
+		timeout: 6e4
+	});
+	if (result.exitCode !== 0) throw new Error(result.stderr || result.stdout || `gofmt exited with code ${result.exitCode}`);
+};
+
+//#endregion
+//#region src/engines/format/ruff-format.ts
+const runRuffFormat = async (context) => {
+	const ruffBinary = resolveToolBinary("ruff");
+	try {
+		const result = await runSubprocess(ruffBinary, [
+			"format",
+			"--check",
+			"--diff",
+			context.rootDirectory
+		], {
+			cwd: context.rootDirectory,
+			timeout: 6e4
 		});
-		const issues = parsed.issues ?? [];
-		const issueTypes = [
-			...DEPENDENCY_TYPES,
-			"exports",
-			"types",
-			"duplicates"
-		];
-		for (const fileIssue of issues) for (const type of issueTypes) diagnostics.push(...collectIssues(fileIssue, type, rootDirectory, knipRuntime.cwd));
-		return diagnostics;
+		if (result.exitCode === 0) return [];
+		return parseRuffFormatOutput(result.stdout || result.stderr, context.rootDirectory);
 	} catch {
 		return [];
 	}
 };
+const parseRuffFormatOutput = (output, rootDir) => {
+	const diagnostics = [];
+	const filePattern = /^--- (.+)$/gm;
+	let match;
+	while ((match = filePattern.exec(output)) !== null) {
+		const filePath = match[1].replace(/^a\//, "");
+		diagnostics.push({
+			filePath: path.relative(rootDir, filePath),
+			engine: "format",
+			rule: "python-formatting",
+			severity: "warning",
+			message: "Python file is not formatted correctly",
+			help: "Run `aislop fix` to auto-format with ruff",
+			line: 0,
+			column: 0,
+			category: "Format",
+			fixable: true
+		});
+	}
+	return diagnostics;
+};
+const fixRuffFormat = async (rootDirectory) => {
+	const result = await runSubprocess(resolveToolBinary("ruff"), ["format", rootDirectory], {
+		cwd: rootDirectory,
+		timeout: 6e4
+	});
+	if (result.exitCode !== 0) throw new Error(result.stderr || result.stdout || `ruff format exited with code ${result.exitCode}`);
+};
 
 //#endregion
 //#region src/engines/lint/oxlint-config.ts
@@ -1057,6 +1322,7 @@ const fixOxlint = async (context) => {
 			"-c",
 			configPath,
 			"--fix",
+			"--fix-suggestions",
 			"."
 		];
 		const result = await runSubprocess(process.execPath, args, {
@@ -1111,81 +1377,6 @@ const fixRuffLint = async (rootDirectory) => {
 	if (result.exitCode !== 0) throw new Error(result.stderr || result.stdout || `ruff check --fix exited with code ${result.exitCode}`);
 };
 
-//#endregion
-//#region src/output/pager.ts
-const DEFAULT_COLUMNS = 80;
-const DEFAULT_ROWS = 24;
-const ANSI_PATTERN = new RegExp(String.raw`\u001B\[[0-?]*[ -/]*[@-~]`, "g");
-const stripAnsi = (text) => text.replace(ANSI_PATTERN, "");
-const resolvePagerCommand = () => {
-	const pager = process.env.PAGER?.trim();
-	if (pager) {
-		const [command, ...args] = pager.split(/\s+/);
-		if (command) return {
-			command,
-			args
-		};
-	}
-	return {
-		command: "less",
-		args: [
-			"-R",
-			"-F",
-			"-X"
-		]
-	};
-};
-const writeToStdout = (text) => {
-	process.stdout.write(text);
-};
-const pipeToPager = async (command, args, text) => new Promise((resolve) => {
-	let settled = false;
-	const finish = (success) => {
-		if (settled) return;
-		settled = true;
-		resolve(success);
-	};
-	try {
-		const child = spawn(command, args, {
-			stdio: [
-				"pipe",
-				"inherit",
-				"inherit"
-			],
-			windowsHide: true
-		});
-		child.once("error", () => finish(false));
-		child.once("close", (code) => finish(code === 0));
-		child.stdin?.on("error", () => void 0);
-		child.stdin?.end(text);
-	} catch {
-		finish(false);
-	}
-});
-const countRenderedLines = (text, columns = DEFAULT_COLUMNS) => {
-	const width = Math.max(1, columns);
-	return text.split("\n").reduce((count, line) => {
-		const visibleLine = stripAnsi(line).replaceAll("	", "    ");
-		return count + Math.max(1, Math.ceil(visibleLine.length / width));
-	}, 0);
-};
-const shouldPageOutput = (text, options = {}) => {
-	if (text.trim().length === 0) return false;
-	const stdinIsTTY = options.stdinIsTTY ?? Boolean(process.stdin.isTTY);
-	const stdoutIsTTY = options.stdoutIsTTY ?? Boolean(process.stdout.isTTY);
-	if (!stdinIsTTY || !stdoutIsTTY) return false;
-	const rows = Math.max(1, options.rows ?? process.stdout.rows ?? DEFAULT_ROWS);
-	return countRenderedLines(text, Math.max(1, options.columns ?? process.stdout.columns ?? DEFAULT_COLUMNS)) > rows - 1;
-};
-const printMaybePaged = async (text) => {
-	if (!shouldPageOutput(text)) {
-		writeToStdout(text);
-		return;
-	}
-	const pager = resolvePagerCommand();
-	if (!await pipeToPager(pager.command, pager.args, text)) writeToStdout(text);
-};
-
 //#endregion
 //#region src/utils/telemetry.ts
 /**
@@ -1333,7 +1524,7 @@ const runFixStep = async (name, detect, applyFix, options) => {
 	}
 	lines.push(...getFilePreviewLines("Affected", uniqueFiles(before), options.verbose));
 	if (after.length > 0) lines.push(...getFilePreviewLines("Remaining", uniqueFiles(after), options.verbose));
-	await printMaybePaged(`${lines.join("\n")}\n\n`);
+	process.stdout.write(`${lines.join("\n")}\n\n`);
 	return result;
 };
 const createEngineContext = (rootDirectory, projectInfo, config) => ({
@@ -1376,13 +1567,7 @@ const fixCommand = async (directory, config, options = {
 	printProjectMetadata(projectInfo);
 	const context = createEngineContext(resolvedDir, projectInfo, config);
 	const steps = [];
-	if (config.engines.format) {
-		if (projectInfo.languages.includes("typescript") || projectInfo.languages.includes("javascript")) steps.push(await runFixStep("JS/TS formatting", () => runBiomeFormat(context), () => fixBiomeFormat(context), options));
-		if (projectInfo.languages.includes("python") && projectInfo.installedTools.ruff) steps.push(await runFixStep("Python formatting", () => runRuffFormat(context), () => fixRuffFormat(resolvedDir), options));
-		else if (projectInfo.languages.includes("python")) logger.warn("  Python detected but ruff is not installed; skipping Python formatting fixes.");
-		if (projectInfo.languages.includes("go") && projectInfo.installedTools.gofmt) steps.push(await runFixStep("Go formatting", () => runGofmt(context), () => fixGofmt(resolvedDir), options));
-		else if (projectInfo.languages.includes("go")) logger.warn("  Go detected but gofmt is not installed; skipping Go formatting fixes.");
-	}
+	if (config.engines["ai-slop"]) steps.push(await runFixStep("Unused imports", () => detectUnusedImports(context), () => fixUnusedImports(context), options));
 	if (config.engines.lint) {
 		if (projectInfo.languages.includes("typescript") || projectInfo.languages.includes("javascript")) steps.push(await runFixStep("JS/TS lint fixes", () => runOxlint(context), () => fixOxlint(context), options));
 		if (projectInfo.languages.includes("python") && projectInfo.installedTools.ruff) steps.push(await runFixStep("Python lint fixes", () => runRuffLint(context), () => fixRuffLint(resolvedDir), options));
@@ -1391,6 +1576,13 @@ const fixCommand = async (directory, config, options = {
 	if (config.engines["code-quality"]) {
 		if (projectInfo.languages.includes("typescript") || projectInfo.languages.includes("javascript")) steps.push(await runFixStep("Unused dependencies", () => runKnipDependencyCheck(resolvedDir), () => fixUnusedDependencies(resolvedDir), options));
 	}
+	if (config.engines.format) {
+		if (projectInfo.languages.includes("typescript") || projectInfo.languages.includes("javascript")) steps.push(await runFixStep("JS/TS formatting", () => runBiomeFormat(context), () => fixBiomeFormat(context), options));
+		if (projectInfo.languages.includes("python") && projectInfo.installedTools.ruff) steps.push(await runFixStep("Python formatting", () => runRuffFormat(context), () => fixRuffFormat(resolvedDir), options));
+		else if (projectInfo.languages.includes("python")) logger.warn("  Python detected but ruff is not installed; skipping Python formatting fixes.");
+		if (projectInfo.languages.includes("go") && projectInfo.installedTools.gofmt) steps.push(await runFixStep("Go formatting", () => runGofmt(context), () => fixGofmt(resolvedDir), options));
+		else if (projectInfo.languages.includes("go")) logger.warn("  Go detected but gofmt is not installed; skipping Go formatting fixes.");
+	}
 	if (steps.length === 0) logger.dim("  No applicable auto-fixers found for this project.");
 	else {
 		logger.break();
@@ -1873,7 +2065,7 @@ const detectTrivialComments = async (context) => {
 
 //#endregion
 //#region src/engines/ai-slop/dead-patterns.ts
-const JS_EXTENSIONS$1 = new Set([
+const JS_EXTENSIONS = new Set([
 	".ts",
 	".tsx",
 	".js",
@@ -1885,7 +2077,7 @@ const CONSOLE_LOG_PATTERN = /\bconsole\.(?:log|debug|info|trace|dir|table)\s*\(/
 const LOGGER_FILE_PATTERN = /(?:^|\/)(?:logger|logging|log)\.[^/]+$/i;
 const SCRIPT_DIR_PATTERN = /(?:^|\/)(scripts|bin)\//;
 const detectConsoleLeftovers = (content, relativePath, ext) => {
-	if (!JS_EXTENSIONS$1.has(ext)) return [];
+	if (!JS_EXTENSIONS.has(ext)) return [];
 	if (LOGGER_FILE_PATTERN.test(relativePath)) return [];
 	if (SCRIPT_DIR_PATTERN.test(relativePath)) return [];
 	const diagnostics = [];
@@ -1945,7 +2137,7 @@ const detectDeadCodePatterns = (content, relativePath, ext) => {
 	for (let i = 0; i < lines.length; i++) {
 		const trimmed = lines[i].trim();
 		const nextLine = i + 1 < lines.length ? lines[i + 1]?.trim() : void 0;
-		if (JS_EXTENSIONS$1.has(ext) && /^(?:return|throw)\b/.test(trimmed) && trimmed.endsWith(";") && nextLine && nextLine.length > 0 && !nextLine.startsWith("}") && !nextLine.startsWith("//") && !nextLine.startsWith("/*") && !nextLine.startsWith("case ") && !nextLine.startsWith("default:") && !nextLine.startsWith("if ") && !nextLine.startsWith("if(") && !nextLine.startsWith("else")) diagnostics.push({
+		if (JS_EXTENSIONS.has(ext) && /^(?:return|throw)\b/.test(trimmed) && trimmed.endsWith(";") && nextLine && nextLine.length > 0 && !nextLine.startsWith("}") && !nextLine.startsWith("//") && !nextLine.startsWith("/*") && !nextLine.startsWith("case ") && !nextLine.startsWith("default:") && !nextLine.startsWith("if ") && !nextLine.startsWith("if(") && !nextLine.startsWith("else")) diagnostics.push({
 			filePath: relativePath,
 			engine: "ai-slop",
 			rule: "ai-slop/unreachable-code",
@@ -1969,7 +2161,7 @@ const detectDeadCodePatterns = (content, relativePath, ext) => {
 			category: "AI Slop",
 			fixable: false
 		});
-		if (JS_EXTENSIONS$1.has(ext) && /(?:function\s+\w+|=>\s*)\s*\{\s*\}\s*;?\s*$/.test(trimmed) && !trimmed.startsWith("interface") && !trimmed.startsWith("type ")) diagnostics.push({
+		if (JS_EXTENSIONS.has(ext) && /(?:function\s+\w+|=>\s*)\s*\{\s*\}\s*;?\s*$/.test(trimmed) && !trimmed.startsWith("interface") && !trimmed.startsWith("type ")) diagnostics.push({
 			filePath: relativePath,
 			engine: "ai-slop",
 			rule: "ai-slop/empty-function",
@@ -2145,166 +2337,6 @@ const detectSwallowedExceptions = async (context) => {
 	return diagnostics;
 };
 
-//#endregion
-//#region src/engines/ai-slop/unused-imports.ts
-const JS_EXTENSIONS = new Set([
-	".ts",
-	".tsx",
-	".js",
-	".jsx",
-	".mjs",
-	".cjs"
-]);
-const PY_EXTENSIONS = new Set([".py"]);
-const extractJsImportedSymbols = (lines) => {
-	const symbols = [];
-	const importLines = /* @__PURE__ */ new Set();
-	for (let i = 0; i < lines.length; i++) {
-		const trimmed = lines[i].trim();
-		if (!trimmed.startsWith("import ")) continue;
-		importLines.add(i);
-		if (/^import\s+["']/.test(trimmed)) continue;
-		if (/^import\s+type\s/.test(trimmed)) continue;
-		let fullImport = trimmed;
-		let endLine = i;
-		while (!fullImport.includes("from") && endLine < lines.length - 1) {
-			endLine++;
-			fullImport += ` ${lines[endLine].trim()}`;
-			importLines.add(endLine);
-		}
-		const namespaceMatch = fullImport.match(/import\s+\*\s+as\s+(\w+)\s+from/);
-		if (namespaceMatch) {
-			symbols.push({
-				name: namespaceMatch[1],
-				line: i + 1,
-				isDefault: false,
-				isNamespace: true
-			});
-			continue;
-		}
-		const defaultMatch = fullImport.match(/import\s+(\w+)\s*(?:,\s*\{[^}]*\})?\s+from/);
-		if (defaultMatch && defaultMatch[1] !== "type") symbols.push({
-			name: defaultMatch[1],
-			line: i + 1,
-			isDefault: true,
-			isNamespace: false
-		});
-		const namedMatch = fullImport.match(/\{([^}]+)\}/);
-		if (namedMatch) {
-			const namedImports = namedMatch[1].split(",");
-			for (const ni of namedImports) {
-				const parts = ni.trim().split(/\s+as\s+/);
-				if (parts.length === 0 || !parts[0]) continue;
-				const cleanParts = parts.map((p) => p.trim().replace(/^type\s+/, ""));
-				const localName = cleanParts.length > 1 ? cleanParts[1] : cleanParts[0];
-				if (localName && /^\w+$/.test(localName)) symbols.push({
-					name: localName,
-					line: i + 1,
-					isDefault: false,
-					isNamespace: false
-				});
-			}
-		}
-	}
-	return {
-		symbols,
-		importLines
-	};
-};
-const extractPyImportedSymbols = (lines) => {
-	const symbols = [];
-	const importLines = /* @__PURE__ */ new Set();
-	for (let i = 0; i < lines.length; i++) {
-		const trimmed = lines[i].trim();
-		const fromMatch = trimmed.match(/^from\s+[\w.]+\s+import\s+(.+)/);
-		if (fromMatch) {
-			importLines.add(i);
-			const importPart = fromMatch[1].replace(/#.*$/, "").trim();
-			if (importPart === "*") continue;
-			const cleaned = importPart.replace(/[()]/g, "");
-			for (const item of cleaned.split(",")) {
-				const parts = item.trim().split(/\s+as\s+/);
-				const localName = parts.length > 1 ? parts[1].trim() : parts[0].trim();
-				if (localName && /^\w+$/.test(localName)) symbols.push({
-					name: localName,
-					line: i + 1,
-					isDefault: false,
-					isNamespace: false
-				});
-			}
-			continue;
-		}
-		const importMatch = trimmed.match(/^import\s+([\w.]+)(?:\s+as\s+(\w+))?/);
-		if (importMatch) {
-			importLines.add(i);
-			const simpleName = (importMatch[2] ?? importMatch[1]).split(".")[0];
-			if (simpleName && /^\w+$/.test(simpleName)) symbols.push({
-				name: simpleName,
-				line: i + 1,
-				isDefault: false,
-				isNamespace: true
-			});
-		}
-	}
-	return {
-		symbols,
-		importLines
-	};
-};
-const isSymbolUsed = (name, content, importLines, lines) => {
-	const pattern = new RegExp(`\\b${name}\\b`, "g");
-	let match;
-	while ((match = pattern.exec(content)) !== null) {
-		const lineIndex = content.slice(0, match.index).split("\n").length - 1;
-		if (!importLines.has(lineIndex)) return true;
-	}
-	for (let i = 0; i < lines.length; i++) {
-		if (importLines.has(i)) continue;
-		if (lines[i].includes(name)) return true;
-	}
-	return false;
-};
-const detectUnusedImports = async (context) => {
-	const files = getSourceFiles(context);
-	const diagnostics = [];
-	for (const filePath of files) {
-		if (isAutoGenerated(filePath)) continue;
-		let content;
-		try {
-			content = fs.readFileSync(filePath, "utf-8");
-		} catch {
-			continue;
-		}
-		const ext = path.extname(filePath);
-		const relativePath = path.relative(context.rootDirectory, filePath);
-		const lines = content.split("\n");
-		let symbols;
-		let importLinesSet;
-		if (JS_EXTENSIONS.has(ext)) {
-			const result = extractJsImportedSymbols(lines);
-			symbols = result.symbols;
-			importLinesSet = result.importLines;
-		} else if (PY_EXTENSIONS.has(ext)) {
-			const result = extractPyImportedSymbols(lines);
-			symbols = result.symbols;
-			importLinesSet = result.importLines;
-		} else continue;
-		for (const symbol of symbols) if (!isSymbolUsed(symbol.name, content, importLinesSet, lines)) diagnostics.push({
-			filePath: relativePath,
-			engine: "ai-slop",
-			rule: "ai-slop/unused-import",
-			severity: "warning",
-			message: `Imported symbol '${symbol.name}' is never used`,
-			help: "Remove unused imports to keep the code clean",
-			line: symbol.line,
-			column: 0,
-			category: "AI Slop",
-			fixable: true
-		});
-	}
-	return diagnostics;
-};
-
 //#endregion
 //#region src/engines/ai-slop/index.ts
 const aiSlopEngine = {
@@ -4069,17 +4101,18 @@ const scanCommand = async (directory, config, options) => {
 		});
 	}
 	if (options.json) {
-		const { buildJsonOutput } = await import("./json-UG8l_sLC.js");
+		const { buildJsonOutput } = await import("./json-Ci_gvHLS.js");
 		const jsonOut = buildJsonOutput(results, scoreResult, projectInfo.sourceFileCount, elapsedMs);
 		console.log(JSON.stringify(jsonOut, null, 2));
 		return { exitCode };
 	}
-	await printMaybePaged([
+	const output = [
 		"",
 		allDiagnostics.length === 0 ? `${highlighter.success("  ✓ No issues found.")}\n` : renderDiagnostics(allDiagnostics, options.verbose),
 		renderSummary(allDiagnostics, scoreResult, elapsedMs, projectInfo.sourceFileCount, config.scoring.thresholds),
 		""
-	].join("\n"));
+	].join("\n");
+	process.stdout.write(output);
 	return { exitCode };
 };