@atlisp/lint 0.1.23 → 0.1.25

package/dist/runner.js

@@ -35,7 +35,6 @@ 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;
 exports.lintFilesParallel = lintFilesParallel;
 exports.applyFixes = applyFixes;
@@ -60,6 +59,7 @@ const error_handling_1 = require("./checks/error-handling");
 const global_naming_1 = require("./checks/global-naming");
 const extra_parens_1 = require("./checks/extra-parens");
 const arg_count_1 = require("./checks/arg-count");
+const builtin_arg_count_1 = require("./checks/builtin-arg-count");
 const strcat_usage_1 = require("./checks/strcat-usage");
 const cond_simplify_1 = require("./checks/cond-simplify");
 const quote_style_1 = require("./checks/quote-style");
@@ -100,12 +100,17 @@ function runChecks(content, file, config) {
         const severity = checks[rule];
         if (!severity || severity === 'off')
             return;
-        const results = fn();
-        for (const r of results) {
-            if (!(0, disable_1.isDisabled)(r.rule, r.line, disableMap)) {
-                issues.push(r);
+        try {
+            const results = fn();
+            for (const r of results) {
+                if (!(0, disable_1.isDisabled)(r.rule, r.line, disableMap)) {
+                    issues.push(r);
+                }
             }
         }
+        catch {
+            // Check failed, continue with remaining checks
+        }
     }
     function addIssues(ruleIssues) {
         for (const r of ruleIssues) {
@@ -133,6 +138,7 @@ function runChecks(content, file, config) {
     addIfEnabled('global_naming', () => (0, global_naming_1.checkGlobalNaming)(content, file));
     addIfEnabled('extra_parens', () => (0, extra_parens_1.checkExtraParens)(content, file));
     addIfEnabled('arg_count', () => (0, arg_count_1.checkArgCount)(content, file));
+    addIfEnabled('builtin_arg_count', () => (0, builtin_arg_count_1.checkBuiltinArgCount)(content, file));
     addIfEnabled('strcat_usage', () => (0, strcat_usage_1.checkStrcatUsage)(content, file));
     addIfEnabled('cond_simplify', () => (0, cond_simplify_1.checkCondSimplify)(content, file));
     addIfEnabled('quote_style', () => (0, quote_style_1.checkQuoteStyle)(content, file));
@@ -173,17 +179,17 @@ function runChecks(content, file, config) {
     ]);
     const needsAst = Array.from(astRules).some(r => checks[r] !== 'off');
     if (needsAst) {
-        const ast = (0, parser_1.parseAst)(content);
-        const visitorIssues = (0, visitor_runner_1.runChecksWithVisitor)(ast, file, config, disableMap);
-        addIssues(visitorIssues);
+        try {
+            const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
+            const visitorIssues = (0, visitor_runner_1.runChecksWithVisitor)(ast, file, config, disableMap);
+            addIssues(visitorIssues);
+        }
+        catch {
+            // AST-based checks are skipped, text-based checks above still report issues
+        }
     }
     return issues;
 }
-function runChecksWithVisitorWrapper(content, file, config) {
-    const disableMap = (0, disable_1.parseDisableComments)(content);
-    const ast = (0, parser_1.parseAst)(content);
-    return (0, visitor_runner_1.runChecksWithVisitor)(ast, file, config, disableMap);
-}
 function lintFiles(files, config, rootDir) {
     (0, locale_1.setLocale)(config.locale || 'zh');
     const allIssues = [];
@@ -321,6 +327,7 @@ exports.FIXABLE_RULES = new Set([
 ]);
 function applyFixes(issues, content, filepath) {
     const lines = content.split('\n');
+    const fixedLines = new Set();
     const fixesByRule = new Map();
     for (const iss of issues) {
         if (iss.file !== filepath)
@@ -330,7 +337,6 @@ function applyFixes(issues, content, filepath) {
         fixesByRule.set(iss.rule, set);
     }
     const fixableRules = exports.FIXABLE_RULES;
-    let result = content;
     for (const [rule, lineSet] of fixesByRule) {
         if (!fixableRules.has(rule))
             continue;
@@ -339,30 +345,38 @@ function applyFixes(issues, content, filepath) {
             const idx = line - 1;
             if (idx < 0 || idx >= lines.length)
                 continue;
-            const lineContent = result.split('\n')[idx];
+            const lineContent = lines[idx];
             switch (rule) {
                 case 'redundant_quotes': {
                     const fixed = lineContent.replace(/''(\S)/g, "'$1");
-                    if (fixed !== lineContent)
-                        result = replaceLine(result, idx, fixed);
+                    if (fixed !== lineContent) {
+                        lines[idx] = fixed;
+                        fixedLines.add(idx);
+                    }
                     break;
                 }
                 case 'double_not': {
                     const fixed = lineContent.replace(/\(not\s+\(not\s+/g, '(not ');
-                    if (fixed !== lineContent)
-                        result = replaceLine(result, idx, fixed);
+                    if (fixed !== lineContent) {
+                        lines[idx] = fixed;
+                        fixedLines.add(idx);
+                    }
                     break;
                 }
                 case 'redundant_nil_else': {
                     const fixed = lineContent.replace(/\s+nil\s*\)$/, ')');
-                    if (fixed !== lineContent)
-                        result = replaceLine(result, idx, fixed);
+                    if (fixed !== lineContent) {
+                        lines[idx] = fixed;
+                        fixedLines.add(idx);
+                    }
                     break;
                 }
                 case 'single_arg_and_or': {
                     const m = lineContent.match(/\((and|or)\s+([^)\s]+)\s*\)/);
-                    if (m)
-                        result = replaceLine(result, idx, m[2]);
+                    if (m) {
+                        lines[idx] = m[2];
+                        fixedLines.add(idx);
+                    }
                     break;
                 }
                 case 'redundant_progn': {
@@ -374,16 +388,20 @@ function applyFixes(issues, content, filepath) {
                             const inner = lineContent.slice(proIdx + 7, close);
                             const rest = lineContent.slice(close + 1);
                             const fixed = beforePro + inner + rest;
-                            if (fixed !== lineContent)
-                                result = replaceLine(result, idx, fixed);
+                            if (fixed !== lineContent) {
+                                lines[idx] = fixed;
+                                fixedLines.add(idx);
+                            }
                         }
                     }
                     break;
                 }
                 case 'redundant_let': {
                     const fixed = lineContent.replace(/\(let\s+\(\)\s*/, '(progn ');
-                    if (fixed !== lineContent)
-                        result = replaceLine(result, idx, fixed);
+                    if (fixed !== lineContent) {
+                        lines[idx] = fixed;
+                        fixedLines.add(idx);
+                    }
                     break;
                 }
                 case 'quote_style': {
@@ -395,8 +413,10 @@ function applyFixes(issues, content, filepath) {
                             const quoted = lineContent.slice(qIdx + 7, close);
                             const rest = lineContent.slice(close + 1);
                             const fixed = beforeQ + "'" + quoted + rest;
-                            if (fixed !== lineContent)
-                                result = replaceLine(result, idx, fixed);
+                            if (fixed !== lineContent) {
+                                lines[idx] = fixed;
+                                fixedLines.add(idx);
+                            }
                         }
                     }
                     break;
@@ -410,8 +430,10 @@ function applyFixes(issues, content, filepath) {
                             const inner = lineContent.slice(pIdx + 7, close);
                             const rest = lineContent.slice(close + 1);
                             const fixed = beforeP + inner + rest;
-                            if (fixed !== lineContent)
-                                result = replaceLine(result, idx, fixed);
+                            if (fixed !== lineContent) {
+                                lines[idx] = fixed;
+                                fixedLines.add(idx);
+                            }
                         }
                     }
                     break;
@@ -436,18 +458,18 @@ function applyFixes(issues, content, filepath) {
                         if (thenForm && elseForm) {
                             const rest = lineContent.slice(ifClose + 1);
                             const fixed = indent + '(if ' + condVar + ' ' + elseForm + ' ' + thenForm + ')' + rest;
-                            if (fixed !== lineContent)
-                                result = replaceLine(result, idx, fixed);
+                            if (fixed !== lineContent) {
+                                lines[idx] = fixed;
+                                fixedLines.add(idx);
+                            }
                         }
                     }
                     break;
                 }
                 case 'redundant_setq': {
-                    const allLines = result.split('\n');
-                    const trimmed = allLines[idx].trim();
+                    const trimmed = lines[idx].trim();
                     if (/^\(setq\s+(\S+)\s+\1\s*\)$/.test(trimmed)) {
-                        allLines.splice(idx, 1);
-                        result = allLines.join('\n');
+                        lines.splice(idx, 1);
                     }
                     break;
                 }
@@ -456,12 +478,15 @@ function applyFixes(issues, content, filepath) {
                     if (fixed !== lineContent) {
                         const merged = lineContent.replace(/^\(\s*setq\s+/, '(setq ');
                         if (merged.includes('setq') && !merged.match(/\)\(/)) {
-                            result = replaceLine(result, idx, merged);
+                            lines[idx] = merged;
+                            fixedLines.add(idx);
                         }
                         else {
                             const simplified = lineContent.replace(/\)\(setq\s+/g, '\n');
-                            if (simplified !== lineContent)
-                                result = replaceLine(result, idx, simplified);
+                            if (simplified !== lineContent) {
+                                lines[idx] = simplified;
+                                fixedLines.add(idx);
+                            }
                         }
                     }
                     break;
@@ -475,8 +500,10 @@ function applyFixes(issues, content, filepath) {
                             const listMatch = inner.match(/^\(list\s+(.*)\)\s*$/);
                             if (listMatch) {
                                 const fixed = lineContent.slice(0, aIdx) + '(cons ' + listMatch[1] + ')' + lineContent.slice(close + 1);
-                                if (fixed !== lineContent)
-                                    result = replaceLine(result, idx, fixed);
+                                if (fixed !== lineContent) {
+                                    lines[idx] = fixed;
+                                    fixedLines.add(idx);
+                                }
                             }
                         }
                     }
@@ -484,40 +511,48 @@ function applyFixes(issues, content, filepath) {
                 }
                 case 'nth_usage': {
                     const fixed = lineContent.replace(/\(nth\s+0\s+/g, '(car ');
-                    if (fixed !== lineContent)
-                        result = replaceLine(result, idx, fixed);
+                    if (fixed !== lineContent) {
+                        lines[idx] = fixed;
+                        fixedLines.add(idx);
+                    }
                     break;
                 }
                 case 'setq_single_arg': {
-                    const allLines = result.split('\n');
-                    if (/^\s*\(setq\s+\S+\s*\)\s*$/.test(allLines[idx])) {
-                        allLines.splice(idx, 1);
-                        result = allLines.join('\n');
+                    if (/^\s*\(setq\s+\S+\s*\)\s*$/.test(lines[idx])) {
+                        lines.splice(idx, 1);
                     }
                     break;
                 }
                 case 'extra_parens': {
                     const fixed = lineContent.replace(/\){2,}/g, ')');
-                    if (fixed !== lineContent)
-                        result = replaceLine(result, idx, fixed);
+                    if (fixed !== lineContent) {
+                        lines[idx] = fixed;
+                        fixedLines.add(idx);
+                    }
                     break;
                 }
                 case 'empty_branch': {
                     const fixed = lineContent.replace(/\(if\s+\S+\s*\)/g, m => m.slice(0, -1) + ' nil)');
-                    if (fixed !== lineContent)
-                        result = replaceLine(result, idx, fixed);
+                    if (fixed !== lineContent) {
+                        lines[idx] = fixed;
+                        fixedLines.add(idx);
+                    }
                     break;
                 }
                 case 'comment_style': {
                     const fixed = lineContent.replace(/^(\s*;)([^ ;|].*)$/gm, '$1 $2');
-                    if (fixed !== lineContent)
-                        result = replaceLine(result, idx, fixed);
+                    if (fixed !== lineContent) {
+                        lines[idx] = fixed;
+                        fixedLines.add(idx);
+                    }
                     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);
+                    if (fixed !== lineContent) {
+                        lines[idx] = fixed;
+                        fixedLines.add(idx);
+                    }
                     break;
                 }
                 case 'cond_simplify': {
@@ -538,8 +573,10 @@ function applyFixes(issues, content, filepath) {
                                         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);
+                                        if (fixed !== lineContent) {
+                                            lines[idx] = fixed;
+                                            fixedLines.add(idx);
+                                        }
                                     }
                                 }
                             }
@@ -549,34 +586,31 @@ function applyFixes(issues, content, filepath) {
                 }
                 case 'self_compare': {
                     const fixed = lineContent.replace(/\((?:[=/<>]+|eq|equal)\s+(\S+)\s+\1\s*\)/g, 'T');
-                    if (fixed !== lineContent)
-                        result = replaceLine(result, idx, fixed);
+                    if (fixed !== lineContent) {
+                        lines[idx] = fixed;
+                        fixedLines.add(idx);
+                    }
                     break;
                 }
                 case 'while_constant': {
-                    const allLines = result.split('\n');
-                    const trimmed = allLines[idx].trim();
+                    const trimmed = lines[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');
+                        lines[idx] = lines[idx].replace(trimmed, 'nil');
                     }
                     break;
                 }
                 case 'redundant_list': {
                     const fixed = lineContent.replace(/\(\s*list\s*\)/g, 'nil');
-                    if (fixed !== lineContent)
-                        result = replaceLine(result, idx, fixed);
+                    if (fixed !== lineContent) {
+                        lines[idx] = fixed;
+                        fixedLines.add(idx);
+                    }
                     break;
                 }
             }
         }
     }
-    return result;
-}
-function replaceLine(content, lineIdx, newLine) {
-    const lines = content.split('\n');
-    lines[lineIdx] = newLine;
     return lines.join('\n');
 }
 function parseIgnoreFile(rootDir) {
@@ -593,23 +627,51 @@ const UTF8_BOM = 0xfeff;
 function fixFile(filepath) {
     const fixes = [];
     let content = fs.readFileSync(filepath, 'utf-8');
-    const wsFixed = content
-        .split('\n')
-        .map(l => l.replace(/[ \t]+$/, ''))
-        .join('\n');
-    if (wsFixed !== content) {
-        content = wsFixed;
-        fixes.push('trailing_whitespace');
-    }
+    // 1. Remove BOM
     if (content.charCodeAt(0) === UTF8_BOM) {
         content = content.slice(1);
         fixes.push('encoding (BOM)');
     }
+    // 2. Single line pass: trailing whitespace + empty comment lines
+    const origLines = content.split('\n');
+    let wsChanged = false;
+    let ecChanged = false;
+    const processed = origLines.map(l => {
+        const stripped = l.replace(/[ \t]+$/, '');
+        if (stripped !== l)
+            wsChanged = true;
+        if (/^\s*;\s*$/.test(stripped)) {
+            ecChanged = true;
+            return '';
+        }
+        return stripped;
+    });
+    if (wsChanged)
+        fixes.push('trailing_whitespace');
+    if (ecChanged)
+        fixes.push('empty_comment');
+    if (wsChanged || ecChanged)
+        content = processed.join('\n');
+    // 3. Count parens (string/comment aware), fix trailing excess
     let openCount = 0;
     let closeCount = 0;
-    let fixContent = content;
-    for (let i = 0; i < fixContent.length; i++) {
-        const ch = fixContent[i];
+    let inStr = false;
+    for (let i = 0; i < content.length; i++) {
+        const ch = content[i];
+        if (ch === '"') {
+            inStr = !inStr;
+            continue;
+        }
+        if (inStr) {
+            if (ch === '\\')
+                i++;
+            continue;
+        }
+        if (ch === ';') {
+            const nl = content.indexOf('\n', i);
+            i = nl !== -1 ? nl : content.length;
+            continue;
+        }
         if (ch === '(')
             openCount++;
         else if (ch === ')')
@@ -617,29 +679,17 @@ function fixFile(filepath) {
     }
     if (closeCount > openCount) {
         let excess = closeCount - openCount;
-        while (excess > 0 && fixContent.endsWith(')')) {
-            fixContent = fixContent.slice(0, -1);
+        let fixed = content;
+        while (excess > 0 && fixed.endsWith(')')) {
+            fixed = fixed.slice(0, -1);
             excess--;
         }
-        fixContent = fixContent.replace(/\n[ \t]*$/, '');
-        if (fixContent !== content) {
-            content = fixContent;
+        fixed = fixed.replace(/\n[ \t]*$/, '');
+        if (fixed !== content) {
+            content = fixed;
             fixes.push('trailing_paren');
         }
     }
-    const lines = content.split('\n');
-    let changed = false;
-    const fixedLines = lines.map(l => {
-        if (/^\s*;\s*$/.test(l)) {
-            changed = true;
-            return '';
-        }
-        return l;
-    });
-    if (changed) {
-        content = fixedLines.join('\n');
-        fixes.push('empty_comment');
-    }
     if (fixes.length > 0) {
         fs.writeFileSync(filepath, content, 'utf-8');
     }

package/dist/stub-packages.json

@@ -0,0 +1,41 @@
+((AUTOLISP ())
+ (@ (COMMON-LISP AUTOLISP))
+ (C ())
+ (JSON ())
+ (SIDEBAR ())
+ (FUN ())
+ (DCL ())
+ (BASE ())
+ (STRING ())
+ (BLOCK ())
+ (ENTITY ())
+ (LAYER ())
+ (LAYOUT ())
+ (CURVE ())
+ (LIST ())
+ (M ())
+ (P ())
+ (UI ())
+ (SYS ())
+ (EXCEL ())
+ (WORD ())
+ (RE ())
+ (VECTRA ())
+ (DOS ())
+ (AT-PM ())
+ (QRENCODE ())
+ (NETWORK ())
+ (PKGMAN ())
+ (DATETIME ())
+ (DICT ())
+ (@NLP ())
+ (AT-SIDEBAR ())
+ (S ())
+ (VL ())
+ (VLR ())
+ (VLAX ())
+ (THEME ())
+ (AJAX ())
+ (ATLISP ())
+ (LOG ())
+ (ATLISP-CORE ()))

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlisp/lint",
-  "version": "0.1.23",
+  "version": "0.1.25",
   "description": "AutoLISP static analysis tool — parens, security, conventions, SBCL syntax validation",
   "keywords": [
     "CAD",
@@ -37,7 +37,7 @@
     "prepack": "npm run build"
   },
   "dependencies": {
-    "@atlisp/parser": "^0.1.2"
+    "@atlisp/parser": "^0.1.3"
   },
   "devDependencies": {
     "@types/jest": "^29",