aislop 0.3.0 → 0.3.1

package/dist/cli.js

@@ -1937,12 +1937,14 @@ const parseBiomeJsonOutput = (output, rootDir) => {
 			const rawPath = entry.location?.path;
 			if (!rawPath) continue;
 			const severity = entry.severity === "error" ? "error" : "warning";
+			const rawMessage = entry.message ?? "";
+			const message = !rawMessage || rawMessage.toLowerCase().includes("would have printed") ? "File is not formatted correctly" : rawMessage;
 			diagnostics.push({
 				filePath: path.isAbsolute(rawPath) ? path.relative(rootDir, rawPath) : rawPath,
 				engine: "format",
 				rule: "formatting",
 				severity,
-				message: entry.message ?? "File is not formatted correctly",
+				message,
 				help: "Run `aislop fix` to auto-format",
 				line: entry.location?.start?.line ?? 0,
 				column: entry.location?.start?.column ?? 0,
@@ -2435,6 +2437,10 @@ const resolveOxlintBinary = () => {
 	}
 };
 const parseRuleCode = (code) => {
+	if (!code) return {
+		plugin: "unknown",
+		rule: "unknown"
+	};
 	const match = code.match(/^(.+)\((.+)\)$/);
 	if (!match) return {
 		plugin: "unknown",
@@ -2494,33 +2500,32 @@ const collectUnusedVarCandidates = (diagnostics) => diagnostics.filter((d) => d.
 const prefixIdentifierOnLine = (line, name, column, type) => {
 	const escaped = name.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
 	if (type === "parameter") {
+		const destructureMatch = line.match(/\{[^}]*\}/);
+		if (destructureMatch) {
+			const { 0: content, index: start } = destructureMatch;
+			const propPattern = new RegExp(`(?<!:\\s*)\\b${escaped}\\b(?!\\s*:)`);
+			if (propPattern.test(content)) {
+				const updated = content.replace(propPattern, `${name}: _${name}`);
+				if (updated !== content) return line.slice(0, start) + updated + line.slice(start + content.length);
+			}
+		}
 		const paramPattern = new RegExp(`\\b${escaped}\\b`);
-		if (paramPattern.test(line)) return line.replace(paramPattern, `_${name}`);
-		return line;
+		return paramPattern.test(line) ? line.replace(paramPattern, `_${name}`) : line;
 	}
 	const assignPattern = new RegExp(`(\\s*)(const|let|var)\\s+${escaped}\\s*=\\s*(.+)$`);
 	const assignMatch = line.match(assignPattern);
 	if (assignMatch) {
-		const indent = assignMatch[1];
-		const expression = assignMatch[3];
+		const [, indent, , expression] = assignMatch;
 		if (/await\s/.test(expression) || /\w+\s*\(/.test(expression)) return `${indent}${expression}`;
 		return "";
 	}
 	const destructureMatch = line.match(/\{[^}]*\}/);
 	if (destructureMatch) {
-		const destructureContent = destructureMatch[0];
-		const destructureStart = destructureMatch.index;
-		if (new RegExp(`\\b${escaped}\\b`).test(destructureContent)) {
-			let updated = destructureContent;
-			const propPattern = new RegExp(`\\b${escaped}\\b\\s*,?`);
-			updated = updated.replace(propPattern, (match) => {
-				if (match.endsWith(",")) return "";
-				return "";
-			});
-			updated = updated.replace(/,\s*\},/, "}");
-			updated = updated.replace(/\{,\s*/, "{");
-			updated = updated.replace(/\s*,\s*\}/, "}");
-			if (updated !== destructureContent) return line.slice(0, destructureStart) + updated + line.slice(destructureStart + destructureContent.length);
+		const { 0: content, index: start } = destructureMatch;
+		if (new RegExp(`\\b${escaped}\\b`).test(content)) {
+			let updated = content.replace(new RegExp(`\\b${escaped}\\b\\s*,?`), "");
+			updated = updated.replace(/,\s*\},/, "}").replace(/\{,\s*/, "{").replace(/\s*,\s*\}/, "}");
+			if (updated !== content) return line.slice(0, start) + updated + line.slice(start + content.length);
 		}
 	}
 	let bestStart = -1;
@@ -3371,7 +3376,7 @@ const logger = {
 * Application version — injected at build time by tsdown from package.json.
 * The fallback should always match the "version" field in package.json.
 */
-const APP_VERSION = "0.3.0";
+const APP_VERSION = "0.3.1";
 
 //#endregion
 //#region src/output/layout.ts

package/dist/{engine-info-CXT2Q_Jy.js → engine-info-D19chfzD.js}

@@ -3,7 +3,7 @@
 * Application version — injected at build time by tsdown from package.json.
 * The fallback should always match the "version" field in package.json.
 */
-const APP_VERSION = "0.3.0";
+const APP_VERSION = "0.3.1";
 
 //#endregion
 //#region src/output/engine-info.ts

package/dist/index.js

@@ -1,5 +1,5 @@
 import { i as runSubprocess, n as runExpoDoctor, r as isToolInstalled } from "./expo-doctor-Cm5892Y8.js";
-import { n as getEngineLabel, r as APP_VERSION, t as ENGINE_INFO } from "./engine-info-CXT2Q_Jy.js";
+import { n as getEngineLabel, r as APP_VERSION, t as ENGINE_INFO } from "./engine-info-D19chfzD.js";
 import { createRequire } from "node:module";
 import fs from "node:fs";
 import path from "node:path";
@@ -1587,12 +1587,14 @@ const parseBiomeJsonOutput = (output, rootDir) => {
 			const rawPath = entry.location?.path;
 			if (!rawPath) continue;
 			const severity = entry.severity === "error" ? "error" : "warning";
+			const rawMessage = entry.message ?? "";
+			const message = !rawMessage || rawMessage.toLowerCase().includes("would have printed") ? "File is not formatted correctly" : rawMessage;
 			diagnostics.push({
 				filePath: path.isAbsolute(rawPath) ? path.relative(rootDir, rawPath) : rawPath,
 				engine: "format",
 				rule: "formatting",
 				severity,
-				message: entry.message ?? "File is not formatted correctly",
+				message,
 				help: "Run `aislop fix` to auto-format",
 				line: entry.location?.start?.line ?? 0,
 				column: entry.location?.start?.column ?? 0,
@@ -1781,6 +1783,10 @@ const resolveOxlintBinary = () => {
 	}
 };
 const parseRuleCode = (code) => {
+	if (!code) return {
+		plugin: "unknown",
+		rule: "unknown"
+	};
 	const match = code.match(/^(.+)\((.+)\)$/);
 	if (!match) return {
 		plugin: "unknown",
@@ -1840,33 +1846,32 @@ const collectUnusedVarCandidates = (diagnostics) => diagnostics.filter((d) => d.
 const prefixIdentifierOnLine = (line, name, column, type) => {
 	const escaped = name.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
 	if (type === "parameter") {
+		const destructureMatch = line.match(/\{[^}]*\}/);
+		if (destructureMatch) {
+			const { 0: content, index: start } = destructureMatch;
+			const propPattern = new RegExp(`(?<!:\\s*)\\b${escaped}\\b(?!\\s*:)`);
+			if (propPattern.test(content)) {
+				const updated = content.replace(propPattern, `${name}: _${name}`);
+				if (updated !== content) return line.slice(0, start) + updated + line.slice(start + content.length);
+			}
+		}
 		const paramPattern = new RegExp(`\\b${escaped}\\b`);
-		if (paramPattern.test(line)) return line.replace(paramPattern, `_${name}`);
-		return line;
+		return paramPattern.test(line) ? line.replace(paramPattern, `_${name}`) : line;
 	}
 	const assignPattern = new RegExp(`(\\s*)(const|let|var)\\s+${escaped}\\s*=\\s*(.+)$`);
 	const assignMatch = line.match(assignPattern);
 	if (assignMatch) {
-		const indent = assignMatch[1];
-		const expression = assignMatch[3];
+		const [, indent, , expression] = assignMatch;
 		if (/await\s/.test(expression) || /\w+\s*\(/.test(expression)) return `${indent}${expression}`;
 		return "";
 	}
 	const destructureMatch = line.match(/\{[^}]*\}/);
 	if (destructureMatch) {
-		const destructureContent = destructureMatch[0];
-		const destructureStart = destructureMatch.index;
-		if (new RegExp(`\\b${escaped}\\b`).test(destructureContent)) {
-			let updated = destructureContent;
-			const propPattern = new RegExp(`\\b${escaped}\\b\\s*,?`);
-			updated = updated.replace(propPattern, (match) => {
-				if (match.endsWith(",")) return "";
-				return "";
-			});
-			updated = updated.replace(/,\s*\},/, "}");
-			updated = updated.replace(/\{,\s*/, "{");
-			updated = updated.replace(/\s*,\s*\}/, "}");
-			if (updated !== destructureContent) return line.slice(0, destructureStart) + updated + line.slice(destructureStart + destructureContent.length);
+		const { 0: content, index: start } = destructureMatch;
+		if (new RegExp(`\\b${escaped}\\b`).test(content)) {
+			let updated = content.replace(new RegExp(`\\b${escaped}\\b\\s*,?`), "");
+			updated = updated.replace(/,\s*\},/, "}").replace(/\{,\s*/, "{").replace(/\s*,\s*\}/, "}");
+			if (updated !== content) return line.slice(0, start) + updated + line.slice(start + content.length);
 		}
 	}
 	let bestStart = -1;
@@ -4536,7 +4541,7 @@ const scanCommand = async (directory, config, options) => {
 		});
 	}
 	if (options.json) {
-		const { buildJsonOutput } = await import("./json-C52xnH_4.js");
+		const { buildJsonOutput } = await import("./json-DD60WkDS.js");
 		const jsonOut = buildJsonOutput(results, scoreResult, projectInfo.sourceFileCount, elapsedMs);
 		console.log(JSON.stringify(jsonOut, null, 2));
 		return { exitCode };

package/dist/{json-C52xnH_4.js → json-DD60WkDS.js}

@@ -1,4 +1,4 @@
-import { r as APP_VERSION, t as ENGINE_INFO } from "./engine-info-CXT2Q_Jy.js";
+import { r as APP_VERSION, t as ENGINE_INFO } from "./engine-info-D19chfzD.js";
 
 //#region src/output/json.ts
 const buildJsonOutput = (results, scoreResult, fileCount, elapsedMs) => {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "aislop",
-	"version": "0.3.0",
+	"version": "0.3.1",
 	"description": "Stop AI slop from shipping. A unified code quality CLI that catches the lazy patterns AI coding tools leave behind.",
 	"type": "module",
 	"bin": {