@atlisp/lint 0.1.22 → 0.1.23

package/dist/runner.d.ts

@@ -3,6 +3,7 @@ export declare function runChecks(content: string, file: string, config: LintCon
 export declare function runChecksWithVisitorWrapper(content: string, file: string, config: LintConfig): Issue[];
 export declare function lintFiles(files: string[], config: LintConfig, rootDir: string): Issue[];
 export declare function lintFilesParallel(files: string[], config: LintConfig, rootDir: string): Promise<Issue[]>;
+export declare const FIXABLE_RULES: Set<string>;
 export declare function applyFixes(issues: Issue[], content: string, filepath: string): string;
 export declare function parseIgnoreFile(rootDir: string): string[];
 export declare function fixFile(filepath: string): string[];

package/dist/runner.js

@@ -33,6 +33,7 @@ var __importStar = (this && this.__importStar) || (function () {
     };
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.FIXABLE_RULES = void 0;
 exports.runChecks = runChecks;
 exports.runChecksWithVisitorWrapper = runChecksWithVisitorWrapper;
 exports.lintFiles = lintFiles;
@@ -308,6 +309,16 @@ function extractForm(s, start) {
     const end = findMatchingParen(s, i);
     return [s.slice(i, end + 1), end + 1];
 }
+// ===== FIX APPLICATION =====
+exports.FIXABLE_RULES = new Set([
+    'redundant_quotes', 'double_not', 'redundant_nil_else',
+    'single_arg_and_or', 'redundant_setq', 'redundant_progn',
+    'redundant_let', 'quote_style', 'redundant_if', 'misplaced_else',
+    'setq_multiple', 'append_single', 'nth_usage', 'setq_single_arg',
+    'extra_parens', 'empty_branch',
+    'comment_style', 'eq_usage', 'cond_simplify', 'self_compare',
+    'while_constant', 'redundant_list',
+]);
 function applyFixes(issues, content, filepath) {
     const lines = content.split('\n');
     const fixesByRule = new Map();
@@ -318,13 +329,7 @@ function applyFixes(issues, content, filepath) {
         set.add(iss.line);
         fixesByRule.set(iss.rule, set);
     }
-    const fixableRules = new Set([
-        'redundant_quotes', 'double_not', 'redundant_nil_else',
-        'single_arg_and_or', 'redundant_setq', 'redundant_progn',
-        'redundant_let', 'quote_style', 'redundant_if', 'misplaced_else',
-        'setq_multiple', 'append_single', 'nth_usage', 'setq_single_arg',
-        'extra_parens', 'empty_branch',
-    ]);
+    const fixableRules = exports.FIXABLE_RULES;
     let result = content;
     for (const [rule, lineSet] of fixesByRule) {
         if (!fixableRules.has(rule))
@@ -503,6 +508,67 @@ function applyFixes(issues, content, filepath) {
                         result = replaceLine(result, idx, fixed);
                     break;
                 }
+                case 'comment_style': {
+                    const fixed = lineContent.replace(/^(\s*;)([^ ;|].*)$/gm, '$1 $2');
+                    if (fixed !== lineContent)
+                        result = replaceLine(result, idx, fixed);
+                    break;
+                }
+                case 'eq_usage': {
+                    const fixed = lineContent.replace(/\(eq\s+(\S+)\s+(\d+)\s*\)/g, '(= $1 $2)');
+                    if (fixed !== lineContent)
+                        result = replaceLine(result, idx, fixed);
+                    break;
+                }
+                case 'cond_simplify': {
+                    const condIdx = lineContent.indexOf('(cond ');
+                    if (condIdx !== -1) {
+                        const condEnd = findMatchingParen(lineContent, condIdx);
+                        if (condEnd > condIdx) {
+                            const inner = lineContent.slice(condIdx + 6, condEnd).trim();
+                            const branchOpen = inner.indexOf('(');
+                            if (branchOpen !== -1) {
+                                const branchEnd = findMatchingParen(inner, branchOpen);
+                                if (branchEnd > branchOpen) {
+                                    const indent = lineContent.slice(0, condIdx);
+                                    const branchContent = inner.slice(branchOpen + 1, branchEnd).trim();
+                                    const space = branchContent.search(/\s+/);
+                                    if (space !== -1) {
+                                        const testExpr = branchContent.slice(0, space);
+                                        const bodyExpr = branchContent.slice(space).trim();
+                                        const rest = lineContent.slice(condEnd + 1);
+                                        const fixed = indent + '(if ' + testExpr + ' ' + bodyExpr + ')' + rest;
+                                        if (fixed !== lineContent)
+                                            result = replaceLine(result, idx, fixed);
+                                    }
+                                }
+                            }
+                        }
+                    }
+                    break;
+                }
+                case 'self_compare': {
+                    const fixed = lineContent.replace(/\((?:[=/<>]+|eq|equal)\s+(\S+)\s+\1\s*\)/g, 'T');
+                    if (fixed !== lineContent)
+                        result = replaceLine(result, idx, fixed);
+                    break;
+                }
+                case 'while_constant': {
+                    const allLines = result.split('\n');
+                    const trimmed = allLines[idx].trim();
+                    const nilMatch = trimmed.match(/^\(\s*while\s+nil\s+(.*)\)\s*$/);
+                    if (nilMatch) {
+                        allLines[idx] = allLines[idx].replace(trimmed, 'nil');
+                        result = allLines.join('\n');
+                    }
+                    break;
+                }
+                case 'redundant_list': {
+                    const fixed = lineContent.replace(/\(\s*list\s*\)/g, 'nil');
+                    if (fixed !== lineContent)
+                        result = replaceLine(result, idx, fixed);
+                    break;
+                }
             }
         }
     }

package/dist/visitor-runner.js

@@ -17,7 +17,7 @@ function runChecksWithVisitor(ast, file, config, disableMap) {
         openCount: 0,
         closeCount: 0,
         setqDefs: new Map(),
-        symbolRefs: new Set(),
+        symbolRefs: new Map(),
         condKeys: new Map(),
         defunScopes: [],
         letScopes: [],
@@ -379,9 +379,9 @@ function runChecksWithVisitor(ast, file, config, disableMap) {
             const seen = new Set();
             for (let i = 1; i < node.children.length; i++) {
                 const clause = node.children[i];
-                if (clause.type !== 'list' || !clause.children || clause.children.length < 2)
+                if (clause.type !== 'list' || !clause.children || clause.children.length < 1)
                     continue;
-                const condTest = clause.children[1];
+                const condTest = clause.children[0];
                 const key = getCondKey(condTest);
                 if (seen.has(key)) {
                     addIssue('cond_duplicate', clause.pos.line, (0, locale_1.t)('cond_duplicate'));
@@ -574,7 +574,7 @@ function runChecksWithVisitor(ast, file, config, disableMap) {
     if (checks['unused_variable'] !== 'off') {
         visitor.on('*', node => {
             if (node.type === 'symbol' && node.name) {
-                state.symbolRefs.add(node.name);
+                state.symbolRefs.set(node.name, (state.symbolRefs.get(node.name) || 0) + 1);
             }
         });
     }
@@ -672,11 +672,7 @@ function runChecksWithVisitor(ast, file, config, disableMap) {
     if (checks['unused_variable'] !== 'off') {
         for (const [name, def] of state.setqDefs) {
             if (state.setqDefs.size > 1) {
-                let refCount = 0;
-                for (const ref of state.symbolRefs) {
-                    if (ref === name)
-                        refCount++;
-                }
+                const refCount = state.symbolRefs.get(name) || 0;
                 if (refCount <= 1) {
                     addIssue('unused_variable', def.line, (0, locale_1.t)('unused_variable', name));
                 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlisp/lint",
-  "version": "0.1.22",
+  "version": "0.1.23",
   "description": "AutoLISP static analysis tool — parens, security, conventions, SBCL syntax validation",
   "keywords": [
     "CAD",