@atlisp/lint 0.2.16 → 0.2.17

package/dist/checks/redundant-let.js

@@ -12,7 +12,7 @@ function checkRedundantLetAst(ast, file) {
     const issues = [];
     const letNodes = (0, parser_1.astFindAll)(ast, n => (0, parser_1.astIsList)(n, 'let'));
     for (const node of letNodes) {
-        if (!node.children || node.children.length < 2)
+        if (!(0, parser_1.astIsList)(node) || node.children.length < 2)
             continue;
         const bindings = node.children[1];
         // (let () ...) — binding list exists but is empty

package/dist/checks/redundant-nil-else.js

@@ -13,7 +13,7 @@ function checkRedundantNilElseAst(ast, file) {
     // Find (if cond then nil) — else branch is nil which is the default
     const ifNodes = (0, parser_1.astFindAll)(ast, n => (0, parser_1.astIsList)(n, 'if'));
     for (const node of ifNodes) {
-        if (!node.children || node.children.length !== 4)
+        if (!(0, parser_1.astIsList)(node) || node.children.length !== 4)
             continue;
         // (if cond then nil) — 4 children: if, cond, then, nil-else
         const elseBranch = node.children[3];

package/dist/checks/redundant-progn.js

@@ -12,7 +12,7 @@ function checkRedundantPrognAst(ast, file) {
     const issues = [];
     const prognNodes = (0, parser_1.astFindAll)(ast, n => (0, parser_1.astIsList)(n, 'progn'));
     for (const node of prognNodes) {
-        if (!node.children)
+        if (!(0, parser_1.astIsList)(node))
             continue;
         // count forms after 'progn'
         const bodyCount = node.children.length - 1;

package/dist/checks/redundant-setq.js

@@ -12,7 +12,7 @@ function checkRedundantSetqAst(ast, file) {
     const issues = [];
     const setqNodes = (0, parser_1.astFindAll)(ast, n => (0, parser_1.astIsList)(n, 'setq'));
     for (const node of setqNodes) {
-        if (!node.children)
+        if (!(0, parser_1.astIsList)(node))
             continue;
         // (setq x x) — variable is set to itself
         for (let i = 1; i < node.children.length - 1; i += 2) {

package/dist/checks/self-compare.js

@@ -14,7 +14,7 @@ function checkSelfCompareAst(ast, file) {
     for (const fn of COMPARE_FUNCTIONS) {
         const nodes = (0, parser_1.astFindAll)(ast, n => (0, parser_1.astIsList)(n, fn));
         for (const node of nodes) {
-            if (!node.children)
+            if (!(0, parser_1.astIsList)(node))
                 continue;
             // (= x x) — 3 children: '=', x, x
             if (node.children.length === 3) {

package/dist/checks/setq-single-arg.js

@@ -12,16 +12,17 @@ function checkSetqSingleArgAst(ast, file) {
     const issues = [];
     const nodes = (0, parser_1.astFindAll)(ast, n => (0, parser_1.astIsList)(n, 'setq'));
     for (const node of nodes) {
-        if (!node.children)
+        if (!(0, parser_1.astIsList)(node))
             continue;
         // (setq x) — 2 children: 'setq', x — missing value
         if (node.children.length === 2) {
+            const varNode = node.children[1];
             issues.push({
                 file,
                 line: node.pos.line,
                 severity: 'warn',
                 rule: 'setq_single_arg',
-                message: (0, locale_1.t)('setq_single_arg', node.children[1].name || ''),
+                message: (0, locale_1.t)('setq_single_arg', varNode.type === 'symbol' && varNode.name ? varNode.name : ''),
             });
         }
     }

package/dist/checks/single-arg-and-or.js

@@ -13,7 +13,7 @@ function checkSingleArgAndOrAst(ast, file) {
     for (const form of ['and', 'or']) {
         const nodes = (0, parser_1.astFindAll)(ast, n => (0, parser_1.astIsList)(n, form));
         for (const node of nodes) {
-            if (!node.children)
+            if (!(0, parser_1.astIsList)(node))
                 continue;
             // (and x) — 2 children: 'and', x
             if (node.children.length === 2) {

package/dist/checks/undeclared-setq.d.ts

@@ -1,3 +1,4 @@
 import { Issue } from '../types';
-export declare function checkUndeclaredSetq(content: string, file: string): Issue[];
+import { AstNode } from '@atlisp/parser';
+export declare function checkUndeclaredSetq(content: string, file: string, ast?: AstNode): Issue[];
 //# sourceMappingURL=undeclared-setq.d.ts.map

package/dist/checks/undeclared-setq.js

@@ -6,10 +6,10 @@ const parser_1 = require("@atlisp/parser");
 /** Extract declared variable names from a defun's parameter list (both sides of /) */
 function getDeclaredVars(defun) {
     const vars = new Set();
-    if (!defun.children || defun.children.length < 3)
+    if (!(0, parser_1.astIsList)(defun) || defun.children.length < 3)
         return vars;
     const paramList = defun.children[2];
-    if (paramList.type !== 'list' || !paramList.children)
+    if (!(0, parser_1.astIsList)(paramList))
         return vars;
     for (const p of paramList.children) {
         if (p.type === 'symbol' && p.name && p.name !== '/' && p.name !== 'T' && p.name !== 'nil') {
@@ -40,12 +40,13 @@ function findEnclosingDefun(node) {
     }
     return null;
 }
-function checkUndeclaredSetq(content, file) {
+function checkUndeclaredSetq(content, file, ast) {
     const issues = [];
-    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
+    if (!ast)
+        ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     const setqForms = (0, parser_1.astFindListWithHead)(ast, 'setq');
     for (const setq of setqForms) {
-        if (!setq.children || setq.children.length < 2)
+        if (!(0, parser_1.astIsList)(setq) || setq.children.length < 2)
             continue;
         // Skip top-level setq (not inside any defun)
         const defun = findEnclosingDefun(setq);

package/dist/checks/unused-let.js

@@ -12,15 +12,15 @@ function checkUnusedLetBindingAst(ast, file) {
     const issues = [];
     const letNodes = (0, parser_1.astFindAll)(ast, n => (0, parser_1.astIsList)(n, 'let'));
     for (const node of letNodes) {
-        if (!node.children || node.children.length < 2)
+        if (!(0, parser_1.astIsList)(node) || node.children.length < 2)
             continue;
         const bindingList = node.children[1];
-        if (bindingList.type !== 'list' || !bindingList.children)
+        if (!(0, parser_1.astIsList)(bindingList))
             continue;
         // Collect let-bound variables
         const bindings = [];
         for (const binding of bindingList.children) {
-            if (binding.type !== 'list' || !binding.children || binding.children.length === 0)
+            if (!(0, parser_1.astIsList)(binding) || binding.children.length === 0)
                 continue;
             const varNode = binding.children[0];
             if (varNode.type !== 'symbol' || !varNode.name)

package/dist/checks/unused-local-fun.js

@@ -12,10 +12,10 @@ function checkUnusedLocalFunAst(ast, file) {
     const issues = [];
     const defuns = (0, parser_1.astFindListWithHead)(ast, 'defun');
     for (const defun of defuns) {
-        if (!defun.children || defun.children.length < 3)
+        if (!(0, parser_1.astIsList)(defun) || defun.children.length < 3)
             continue;
         const paramList = defun.children[2];
-        if (paramList.type !== 'list' || !paramList.children)
+        if (!(0, parser_1.astIsList)(paramList))
             continue;
         // Collect local variables after `/` in param list
         // (defun foo (x / y z) ...) → y, z are local vars

package/dist/checks/unused-package-dep.d.ts

@@ -1,4 +1,3 @@
 import { Issue } from '../types';
-export declare function checkUnusedPackageDep(file: string): Issue[];
 export declare function checkUnusedPackageDepFromContent(file: string, content: string): Issue[];
 //# sourceMappingURL=unused-package-dep.d.ts.map

package/dist/checks/unused-package-dep.js

@@ -1,96 +1,8 @@
 "use strict";
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || (function () {
-    var ownKeys = function(o) {
-        ownKeys = Object.getOwnPropertyNames || function (o) {
-            var ar = [];
-            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
-            return ar;
-        };
-        return ownKeys(o);
-    };
-    return function (mod) {
-        if (mod && mod.__esModule) return mod;
-        var result = {};
-        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
-        __setModuleDefault(result, mod);
-        return result;
-    };
-})();
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.checkUnusedPackageDep = checkUnusedPackageDep;
 exports.checkUnusedPackageDepFromContent = checkUnusedPackageDepFromContent;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
-const fs = __importStar(require("fs"));
-function checkUnusedPackageDep(file) {
-    const issues = [];
-    const content = fs.readFileSync(file, 'utf-8');
-    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
-    const inPackageNodes = (0, parser_1.astFindAll)(ast, n => (0, parser_1.astIsList)(n, 'in-package'));
-    if (inPackageNodes.length === 0)
-        return issues;
-    const allPackages = new Set();
-    for (const node of inPackageNodes) {
-        if (node.children && node.children.length >= 2 && (0, parser_1.astIsSymbol)(node.children[1])) {
-            allPackages.add(node.children[1].name);
-        }
-    }
-    const usedSymbols = new Set();
-    const allLists = (0, parser_1.astFindAll)(ast, n => n.type === 'list');
-    for (const node of allLists) {
-        if (!node.children || node.children.length === 0)
-            continue;
-        const car = node.children[0];
-        if (car.type === 'symbol' && car.name) {
-            const parts = car.name.split(':');
-            if (parts.length === 2 && parts[0] && parts[1]) {
-                usedSymbols.add(parts[0]);
-            }
-        }
-        for (let i = 1; i < node.children.length; i++) {
-            const child = node.children[i];
-            if (child.type === 'symbol' && child.name) {
-                const parts = child.name.split(':');
-                if (parts.length === 2 && parts[0] && parts[1]) {
-                    usedSymbols.add(parts[0]);
-                }
-            }
-        }
-    }
-    const mainPackage = inPackageNodes.length > 0 && inPackageNodes[0].children && inPackageNodes[0].children[1]
-        ? inPackageNodes[0].children[1].name || ''
-        : '';
-    allPackages.delete(mainPackage);
-    for (const pkg of allPackages) {
-        if (!usedSymbols.has(pkg)) {
-            const node = inPackageNodes.find(n => n.children && n.children.length >= 2 && (0, parser_1.astIsSymbol)(n.children[1]) && n.children[1].name === pkg);
-            issues.push({
-                file,
-                line: node ? node.pos.line : 1,
-                severity: 'warn',
-                rule: 'unused_package_dep',
-                message: (0, locale_1.t)('unused_package_dep', pkg),
-            });
-        }
-    }
-    return issues;
-}
 function checkUnusedPackageDepFromContent(file, content) {
     const issues = [];
     const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
@@ -99,14 +11,14 @@ function checkUnusedPackageDepFromContent(file, content) {
         return issues;
     const allPackages = new Set();
     for (const node of inPackageNodes) {
-        if (node.children && node.children.length >= 2 && (0, parser_1.astIsSymbol)(node.children[1])) {
+        if ((0, parser_1.astIsList)(node) && node.children.length >= 2 && (0, parser_1.astIsSymbol)(node.children[1])) {
             allPackages.add(node.children[1].name);
         }
     }
     const usedSymbols = new Set();
     const allLists = (0, parser_1.astFindAll)(ast, n => n.type === 'list');
     for (const node of allLists) {
-        if (!node.children || node.children.length === 0)
+        if (!(0, parser_1.astIsList)(node) || node.children.length === 0)
             continue;
         const car = node.children[0];
         if (car.type === 'symbol' && car.name) {
@@ -125,13 +37,14 @@ function checkUnusedPackageDepFromContent(file, content) {
             }
         }
     }
-    const mainPackage = inPackageNodes.length > 0 && inPackageNodes[0].children && inPackageNodes[0].children[1]
-        ? inPackageNodes[0].children[1].name || ''
+    const mainPkgNode = inPackageNodes.length > 0 && (0, parser_1.astIsList)(inPackageNodes[0]) ? inPackageNodes[0] : null;
+    const mainPackage = mainPkgNode && mainPkgNode.children.length >= 2 && (0, parser_1.astIsSymbol)(mainPkgNode.children[1])
+        ? mainPkgNode.children[1].name || ''
         : '';
     allPackages.delete(mainPackage);
     for (const pkg of allPackages) {
         if (!usedSymbols.has(pkg)) {
-            const node = inPackageNodes.find(n => n.children && n.children.length >= 2 && (0, parser_1.astIsSymbol)(n.children[1]) && n.children[1].name === pkg);
+            const node = inPackageNodes.find(n => (0, parser_1.astIsList)(n) && n.children.length >= 2 && (0, parser_1.astIsSymbol)(n.children[1]) && n.children[1].name === pkg);
             issues.push({
                 file,
                 line: node ? node.pos.line : 1,

package/dist/checks/unused-param.js

@@ -12,14 +12,14 @@ function checkUnusedParameterAst(ast, file) {
     const issues = [];
     const defuns = (0, parser_1.astFindListWithHead)(ast, 'defun');
     for (const defun of defuns) {
-        if (!defun.children || defun.children.length < 3)
+        if (!(0, parser_1.astIsList)(defun) || defun.children.length < 3)
             continue;
         const nameNode = defun.children[1];
         const funcName = nameNode.type === 'symbol' && nameNode.name ? nameNode.name : '<unknown>';
         if (funcName.startsWith('c:') || funcName.startsWith('C:'))
             continue;
         const paramList = defun.children[2];
-        if (paramList.type !== 'list' || !paramList.children)
+        if (!(0, parser_1.astIsList)(paramList))
             continue;
         // Collect params before / (AutoLISP: (defun foo (a b / x y) ...) — a,b are params, x,y are locals)
         const params = [];

package/dist/checks/unused-variable.js

@@ -14,7 +14,7 @@ function checkUnusedVariableAst(ast, file) {
     const setqForms = (0, parser_1.astFindListWithHead)(ast, 'setq');
     const setqVars = new Map();
     for (const form of setqForms) {
-        if (!form.children)
+        if (!(0, parser_1.astIsList)(form))
             continue;
         for (let i = 1; i < form.children.length; i += 2) {
             const v = form.children[i];
@@ -37,7 +37,7 @@ function checkUnusedVariableAst(ast, file) {
     // Set of defined var nodes (to exclude from usage)
     const defNodes = new Set();
     for (const form of setqForms) {
-        if (!form.children)
+        if (!(0, parser_1.astIsList)(form))
             continue;
         for (let i = 1; i < form.children.length; i += 2) {
             defNodes.add(form.children[i]);

package/dist/checks/variable-shadow.js

@@ -12,15 +12,15 @@ function checkVariableShadowAst(ast, file) {
     const issues = [];
     const letNodes = (0, parser_1.astFindAll)(ast, n => (0, parser_1.astIsList)(n, 'let'));
     for (const node of letNodes) {
-        if (!node.children || node.children.length < 2)
+        if (!(0, parser_1.astIsList)(node) || node.children.length < 2)
             continue;
         const bindingList = node.children[1];
-        if (bindingList.type !== 'list' || !bindingList.children)
+        if (!(0, parser_1.astIsList)(bindingList))
             continue;
         // Collect let-bound variable names
         const letVars = new Set();
         for (const binding of bindingList.children) {
-            if (binding.type !== 'list' || !binding.children || binding.children.length === 0)
+            if (!(0, parser_1.astIsList)(binding) || binding.children.length === 0)
                 continue;
             const varNode = binding.children[0];
             if (varNode.type !== 'symbol' || !varNode.name)

package/dist/checks/while-constant.js

@@ -12,7 +12,7 @@ function checkWhileConstantAst(ast, file) {
     const issues = [];
     const whileNodes = (0, parser_1.astFindAll)(ast, n => (0, parser_1.astIsList)(n, 'while'));
     for (const node of whileNodes) {
-        if (!node.children || node.children.length < 2)
+        if (!(0, parser_1.astIsList)(node) || node.children.length < 2)
             continue;
         const cond = node.children[1];
         // Check for constant conditions: nil, T, 1, 0, etc.

package/dist/cli/collector.js

@@ -41,6 +41,22 @@ const path = __importStar(require("path"));
 const child_process_1 = require("child_process");
 const GLOB_PLACEHOLDER = '\x00GLOB\x00';
 const GLOB_RE = new RegExp(GLOB_PLACEHOLDER, 'g');
+const globCache = new Map();
+function toRegex(pattern) {
+    const cached = globCache.get(pattern);
+    if (cached)
+        return cached;
+    const str = pattern
+        .replace(/\./g, '\\.')
+        .replace(/\*\*\//g, GLOB_PLACEHOLDER)
+        .replace(/\*\*/g, '.*')
+        .replace(/\*/g, '[^/]*')
+        .replace(/\?/g, '.')
+        .replace(GLOB_RE, '(.*/)?');
+    const re = new RegExp(`^${str}$`);
+    globCache.set(pattern, re);
+    return re;
+}
 function* walkFiles(dir, rootDir, regex) {
     let entries;
     try {
@@ -88,15 +104,7 @@ function collectChangedFiles(rootDir, baseBranch) {
     }
 }
 function globFiles(pattern, rootDir) {
-    const regexStr = pattern
-        .replace(/\./g, '\\.')
-        .replace(/\*\*\//g, GLOB_PLACEHOLDER)
-        .replace(/\*\*/g, '.*')
-        .replace(/\*/g, '[^/]*')
-        .replace(/\?/g, '.')
-        .replace(GLOB_RE, '(.*/)?');
-    const regex = new RegExp(`^${regexStr}$`);
-    return Array.from(walkFiles(rootDir, rootDir, regex));
+    return Array.from(walkFiles(rootDir, rootDir, toRegex(pattern)));
 }
 function collectFiles(rootDir, opts, config) {
     const files = new Set();
@@ -127,23 +135,22 @@ function collectFiles(rootDir, opts, config) {
             }
         }
     }
-    const excludePatterns = config.source.exclude.map(e => {
-        const str = e
+    const excludeRe = config.source.exclude.length > 0
+        ? new RegExp(`^(${config.source.exclude
+            .map(e => e
             .replace(/\./g, '\\.')
             .replace(/\*\*\//g, GLOB_PLACEHOLDER)
             .replace(/\*\*/g, '.*')
             .replace(/\*/g, '[^/]*')
-            .replace(GLOB_RE, '(.*/)?');
-        return new RegExp(`^${str}$`);
-    });
+            .replace(GLOB_RE, '(.*/)?'))
+            .join('|')})$`)
+        : null;
     return Array.from(files)
         .filter(f => {
+        if (!excludeRe)
+            return true;
         const rel = path.relative(rootDir, f).replace(/\\/g, '/');
-        for (const re of excludePatterns) {
-            if (re.test(rel))
-                return false;
-        }
-        return true;
+        return !excludeRe.test(rel);
     })
         .sort();
 }

package/dist/config.d.ts

@@ -1,6 +1,6 @@
 import { LintConfig } from './types';
 export declare function loadConfig(configPath?: string): LintConfig;
-export declare function getCheckSeverity(config: LintConfig, rule: string): string;
 export declare function findOverrides(filepath: string): Partial<LintConfig> | null;
+export declare function clearOverrideCache(): void;
 export declare function mergeOverrides(base: LintConfig, overrides: Partial<LintConfig>): LintConfig;
 //# sourceMappingURL=config.d.ts.map

package/dist/config.js

@@ -34,12 +34,13 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.loadConfig = loadConfig;
-exports.getCheckSeverity = getCheckSeverity;
 exports.findOverrides = findOverrides;
+exports.clearOverrideCache = clearOverrideCache;
 exports.mergeOverrides = mergeOverrides;
 const fs = __importStar(require("fs"));
 const path = __importStar(require("path"));
 const presets_1 = require("./presets");
+const overrideCache = new Map();
 const DEFAULT_CONFIG = {
     locale: 'zh',
     source: {
@@ -199,7 +200,7 @@ function loadConfig(configPath) {
         const raw = fs.readFileSync(absPath, 'utf-8');
         const user = JSON.parse(raw);
         // Start from default config
-        const merged = JSON.parse(JSON.stringify(DEFAULT_CONFIG));
+        const merged = structuredClone(DEFAULT_CONFIG);
         // Apply preset if specified (preset applies before user overrides)
         const preset = user.preset;
         if (preset && presets_1.PRESETS[preset]) {
@@ -211,7 +212,7 @@ function loadConfig(configPath) {
         delete merged.preset;
         return merged;
     }
-    return JSON.parse(JSON.stringify(DEFAULT_CONFIG));
+    return structuredClone(DEFAULT_CONFIG);
 }
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 function deepMerge(target, source) {
@@ -228,29 +229,37 @@ function deepMerge(target, source) {
         }
     }
 }
-function getCheckSeverity(config, rule) {
-    return config.checks[rule] || 'off';
-}
 function findOverrides(filepath) {
+    const cached = overrideCache.get(filepath);
+    if (cached !== undefined)
+        return cached;
     let dir = path.dirname(filepath);
     for (;;) {
         const configPath = path.join(dir, '.atlisp-lint.json');
         if (fs.existsSync(configPath)) {
             try {
-                return JSON.parse(fs.readFileSync(configPath, 'utf-8'));
+                const result = JSON.parse(fs.readFileSync(configPath, 'utf-8'));
+                overrideCache.set(filepath, result);
+                return result;
             }
             catch {
+                overrideCache.set(filepath, null);
                 return null;
             }
         }
         const parent = path.dirname(dir);
-        if (parent === dir)
+        if (parent === dir) {
+            overrideCache.set(filepath, null);
             return null;
+        }
         dir = parent;
     }
 }
+function clearOverrideCache() {
+    overrideCache.clear();
+}
 function mergeOverrides(base, overrides) {
-    const merged = JSON.parse(JSON.stringify(base));
+    const merged = structuredClone(base);
     deepMerge(merged, overrides);
     return merged;
 }

package/dist/fixes/index.d.ts

@@ -1,8 +1,12 @@
 import { Issue } from '../types';
+import { AstNode, AstPosition } from '@atlisp/parser';
 export type FixResult = string | null;
 export interface FixProvider {
     (line: string): string;
 }
+export interface ContentFixProvider {
+    (issues: Issue[], content: string, ast?: AstNode, posToOffset?: (pos: AstPosition) => number): string;
+}
 export declare const FIXABLE_RULES: Set<string>;
 export declare function applyFixesFromProviders(issues: Issue[], content: string, filepath: string): string;
 //# sourceMappingURL=index.d.ts.map