@atlisp/lint 0.1.22 → 0.1.24

package/dist/atlisp-lint.default.json

@@ -0,0 +1,90 @@
+{
+  "locale": "zh",
+  "source": {
+    "globs": ["**/*.lsp"],
+    "exclude": ["**/node_modules/**", "**/vendor/**", "**/.git/**"]
+  },
+  "checks": {
+    "parens": "error",
+    "encoding": "warn",
+    "cl_syntax": "warn",
+    "quit_exit": "error",
+    "command_shell": "error",
+    "startapp": "warn",
+    "vl_registry_write": "warn",
+    "vlax_without_loading": "off",
+    "token_in_url": "warn",
+    "open_without_close": "warn",
+    "bare_function_names": "off",
+    "line_length": "warn",
+    "function_complexity": "warn",
+    "parameter_naming": "warn",
+    "unused_variable": "warn",
+    "missing_doc": "warn",
+    "error_handling": "warn",
+    "global_naming": "warn",
+    "extra_parens": "warn",
+    "arg_count": "warn",
+    "strcat_usage": "warn",
+    "cond_simplify": "warn",
+    "redundant_progn": "warn",
+    "quote_style": "warn",
+    "eq_usage": "warn",
+    "lambda_syntax": "warn",
+    "comment_style": "warn",
+    "empty_catch": "warn",
+    "nth_usage": "warn",
+    "append_single": "warn",
+    "setq_multiple": "warn",
+    "function_order": "off",
+    "magic_number": "off",
+    "mixed_indent": "warn",
+    "long_function_call": "warn",
+    "no_return": "warn",
+    "shadow_builtin": "warn",
+    "dynamic_doc": "warn",
+    "loop_optimization": "off",
+    "type_check": "off",
+    "redundant_if": "warn",
+    "trailing_whitespace": "warn",
+    "module_registration": "off",
+    "namespace_header": "off"
+  },
+  "line_length": {
+    "max": 100,
+    "tab_width": 2
+  },
+  "function_complexity": {
+    "max_lines": 60,
+    "max_nesting": 6
+  },
+  "cl_syntax": {
+    "keywords": ["&key"]
+  },
+  "dangerous_calls": {
+    "quit": "error",
+    "exit": "error",
+    "command_shell": "error",
+    "startapp": "warn",
+    "vl_registry_write": "warn"
+  },
+  "module_registration": {
+    "severity": "off",
+    "patterns": [],
+    "dirs": ["modules"]
+  },
+  "namespace_header": {
+    "severity": "off",
+    "package": "",
+    "deps": [],
+    "search_lines": 15
+  },
+  "bare_function_names": {
+    "allowlist": ["T", "nil"],
+    "namespace_pattern": "^[a-z]+:"
+  },
+  "sbcl": {
+    "walk_exclude": [".vscode", "vendor", ".git"],
+    "defmacro_allow_files": ["compat-cl"]
+  }
+}

package/dist/checks/arg-count.js

@@ -5,7 +5,7 @@ const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkArgCount(content, file) {
     const issues = [];
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     const defuns = (0, parser_1.astFindListWithHead)(ast, 'defun');
     for (const defun of defuns) {
         if (!defun.children || defun.children.length < 3)

package/dist/checks/assoc-without-cdr.js

@@ -5,7 +5,7 @@ exports.checkAssocWithoutCdrAst = checkAssocWithoutCdrAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkAssocWithoutCdr(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkAssocWithoutCdrAst(ast, file);
 }
 function checkAssocWithoutCdrAst(ast, file) {

package/dist/checks/cond-duplicate.js

@@ -18,7 +18,7 @@ function condKey(node) {
     return '';
 }
 function checkCondDuplicate(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkCondDuplicateAst(ast, file);
 }
 function checkCondDuplicateAst(ast, file) {

package/dist/checks/constant-condition.js

@@ -7,7 +7,7 @@ const parser_1 = require("@atlisp/parser");
 const CONDITION_FORMS = new Set(['if', 'while']);
 const CONSTANTS = new Set(['T', 'nil']);
 function checkConstantCondition(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkConstantConditionAst(ast, file);
 }
 function checkConstantConditionAst(ast, file) {
@@ -41,8 +41,8 @@ function checkConstantConditionAst(ast, file) {
                 continue;
             const condTest = clause.children[0];
             if (condTest.type === 'symbol' && condTest.name && CONSTANTS.has(condTest.name)) {
-                // Skip last clause with T — that's the standard else/default pattern
-                if (condTest.name === 'T' && i === condNode.children.length - 1)
+                // Skip last clause with T (only when there are multiple clauses — else/default pattern)
+                if (condTest.name === 'T' && i === condNode.children.length - 1 && i > 1)
                     continue;
                 issues.push({
                     file,

package/dist/checks/dangerous-calls.js

@@ -36,7 +36,7 @@ function isFunctionCall(list, name) {
     return list.children[0].type === 'symbol' && list.children[0].name === name;
 }
 function checkDangerousCalls(content, file, config) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkDangerousCallsAst(ast, file, config);
 }
 function checkDangerousCallsAst(ast, file, config) {

package/dist/checks/dangling-defun.js

@@ -40,7 +40,7 @@ const fs = __importStar(require("fs"));
 function checkDanglingDefun(file, allDefuns, allReferences) {
     const issues = [];
     const content = fs.readFileSync(file, 'utf-8');
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     const localDefuns = [];
     const defunNodes = (0, parser_1.astFindAll)(ast, n => (0, parser_1.astIsList)(n, 'defun') || (0, parser_1.astIsList)(n, 'defun-q'));
     for (const node of defunNodes) {

package/dist/checks/double-not.js

@@ -5,7 +5,7 @@ exports.checkDoubleNotAst = checkDoubleNotAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkDoubleNot(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkDoubleNotAst(ast, file);
 }
 function checkDoubleNotAst(ast, file) {

package/dist/checks/empty-branch.js

@@ -7,7 +7,7 @@ const parser_1 = require("@atlisp/parser");
 // AutoLISP only has `if` for conditional branching; `while` with no body is valid (no-op loop)
 const BRANCH_FORMS = new Set(['if']);
 function checkEmptyBranch(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkEmptyBranchAst(ast, file);
 }
 function checkEmptyBranchAst(ast, file) {

package/dist/checks/empty-catch.js

@@ -5,7 +5,7 @@ const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkEmptyCatch(content, file) {
     const issues = [];
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     const catchNodes = (0, parser_1.astFindAll)(ast, n => (0, parser_1.astIsList)(n, 'vl-catch-all-apply'));
     for (const node of catchNodes) {
         if (isInQuote(node))

package/dist/checks/function-complexity.js

@@ -5,7 +5,7 @@ exports.checkFunctionComplexityAst = checkFunctionComplexityAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkFunctionComplexity(content, file, maxLines, maxNesting) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkFunctionComplexityAst(ast, file, maxLines, maxNesting);
 }
 function checkFunctionComplexityAst(ast, file, maxLines, maxNesting) {

package/dist/checks/global-naming.js

@@ -25,7 +25,7 @@ function isInsideLocalScope(node) {
 }
 function checkGlobalNaming(content, file) {
     const issues = [];
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     const setqForms = (0, parser_1.astFindListWithHead)(ast, 'setq');
     for (const form of setqForms) {
         if (!form.children || form.children.length < 2)

package/dist/checks/identical-branches.js

@@ -27,7 +27,7 @@ function astEqual(a, b) {
     return false;
 }
 function checkIdenticalBranches(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkIdenticalBranchesAst(ast, file);
 }
 function checkIdenticalBranchesAst(ast, file) {

package/dist/checks/misplaced-else.js

@@ -5,7 +5,7 @@ exports.checkMisplacedElseAst = checkMisplacedElseAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkMisplacedElse(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkMisplacedElseAst(ast, file);
 }
 function checkMisplacedElseAst(ast, file) {

package/dist/checks/multiple-setq.js

@@ -5,7 +5,7 @@ exports.checkMultipleSetqAst = checkMultipleSetqAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkMultipleSetq(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkMultipleSetqAst(ast, file);
 }
 function checkMultipleSetqAst(ast, file) {

package/dist/checks/no-return.js

@@ -5,7 +5,7 @@ const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkNoReturn(content, file) {
     const issues = [];
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     const defuns = (0, parser_1.astFindListWithHead)(ast, 'defun');
     for (const defun of defuns) {
         if (!defun.children || defun.children.length < 3)

package/dist/checks/open-close.js

@@ -5,7 +5,7 @@ exports.checkOpenWithoutCloseAst = checkOpenWithoutCloseAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkOpenWithoutClose(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkOpenWithoutCloseAst(ast, file);
 }
 function checkOpenWithoutCloseAst(ast, file) {

package/dist/checks/parameter-naming.js

@@ -5,7 +5,7 @@ exports.checkParameterNamingAst = checkParameterNamingAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkParameterNaming(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkParameterNamingAst(ast, file);
 }
 function checkParameterNamingAst(ast, file) {

package/dist/checks/quote-vs-function.js

@@ -17,7 +17,7 @@ const HIGHER_ORDER_FUNCS = new Set([
     'vl-every',
 ]);
 function checkQuoteVsFunction(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkQuoteVsFunctionAst(ast, file);
 }
 function checkQuoteVsFunctionAst(ast, file) {

package/dist/checks/recursive-call.js

@@ -5,7 +5,7 @@ exports.checkRecursiveCallAst = checkRecursiveCallAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkRecursiveCall(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkRecursiveCallAst(ast, file);
 }
 /** Forms that provide conditional branching (exit condition for recursion) */

package/dist/checks/redundant-let.js

@@ -5,7 +5,7 @@ exports.checkRedundantLetAst = checkRedundantLetAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkRedundantLet(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkRedundantLetAst(ast, file);
 }
 function checkRedundantLetAst(ast, file) {

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

@@ -5,7 +5,7 @@ exports.checkRedundantNilElseAst = checkRedundantNilElseAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkRedundantNilElse(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkRedundantNilElseAst(ast, file);
 }
 function checkRedundantNilElseAst(ast, file) {

package/dist/checks/redundant-progn.js

@@ -5,7 +5,7 @@ exports.checkRedundantPrognAst = checkRedundantPrognAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkRedundantProgn(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkRedundantPrognAst(ast, file);
 }
 function checkRedundantPrognAst(ast, file) {

package/dist/checks/redundant-quotes.js

@@ -5,7 +5,7 @@ exports.checkRedundantQuotesAst = checkRedundantQuotesAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkRedundantQuotes(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkRedundantQuotesAst(ast, file);
 }
 function checkRedundantQuotesAst(ast, file) {

package/dist/checks/redundant-setq.js

@@ -5,7 +5,7 @@ exports.checkRedundantSetqAst = checkRedundantSetqAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkRedundantSetq(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkRedundantSetqAst(ast, file);
 }
 function checkRedundantSetqAst(ast, file) {

package/dist/checks/self-compare.js

@@ -6,7 +6,7 @@ const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 const COMPARE_FUNCTIONS = new Set(['=', '/=', '<', '>', '<=', '>=', 'eq', 'equal']);
 function checkSelfCompare(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkSelfCompareAst(ast, file);
 }
 function checkSelfCompareAst(ast, file) {

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

@@ -5,7 +5,7 @@ exports.checkSetqSingleArgAst = checkSetqSingleArgAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkSetqSingleArg(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkSetqSingleArgAst(ast, file);
 }
 function checkSetqSingleArgAst(ast, file) {

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

@@ -5,7 +5,7 @@ exports.checkSingleArgAndOrAst = checkSingleArgAndOrAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkSingleArgAndOr(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkSingleArgAndOrAst(ast, file);
 }
 function checkSingleArgAndOrAst(ast, file) {

package/dist/checks/undeclared-setq.js

@@ -42,7 +42,7 @@ function findEnclosingDefun(node) {
 }
 function checkUndeclaredSetq(content, file) {
     const issues = [];
-    const ast = (0, parser_1.parseAst)(content);
+    const 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)

package/dist/checks/unused-catch-result.js

@@ -5,7 +5,7 @@ const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkUnusedCatchResult(content, file) {
     const issues = [];
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     const catchNodes = (0, parser_1.astFindAll)(ast, n => (0, parser_1.astIsList)(n, 'vl-catch-all-apply'));
     for (const node of catchNodes) {
         if (isInQuote(node))

package/dist/checks/unused-let.js

@@ -5,7 +5,7 @@ exports.checkUnusedLetBindingAst = checkUnusedLetBindingAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkUnusedLetBinding(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkUnusedLetBindingAst(ast, file);
 }
 function checkUnusedLetBindingAst(ast, file) {

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

@@ -5,7 +5,7 @@ exports.checkUnusedLocalFunAst = checkUnusedLocalFunAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkUnusedLocalFun(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkUnusedLocalFunAst(ast, file);
 }
 function checkUnusedLocalFunAst(ast, file) {

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

@@ -40,7 +40,7 @@ const fs = __importStar(require("fs"));
 function checkUnusedPackageDep(file) {
     const issues = [];
     const content = fs.readFileSync(file, 'utf-8');
-    const ast = (0, parser_1.parseAst)(content);
+    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;

package/dist/checks/unused-param.js

@@ -5,7 +5,7 @@ exports.checkUnusedParameterAst = checkUnusedParameterAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkUnusedParameter(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkUnusedParameterAst(ast, file);
 }
 function checkUnusedParameterAst(ast, file) {

package/dist/checks/unused-variable.js

@@ -5,7 +5,7 @@ exports.checkUnusedVariableAst = checkUnusedVariableAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkUnusedVariable(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkUnusedVariableAst(ast, file);
 }
 function checkUnusedVariableAst(ast, file) {

package/dist/checks/variable-shadow.js

@@ -5,7 +5,7 @@ exports.checkVariableShadowAst = checkVariableShadowAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkVariableShadow(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkVariableShadowAst(ast, file);
 }
 function checkVariableShadowAst(ast, file) {

package/dist/checks/while-constant.js

@@ -5,7 +5,7 @@ exports.checkWhileConstantAst = checkWhileConstantAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function checkWhileConstant(content, file) {
-    const ast = (0, parser_1.parseAst)(content);
+    const ast = (0, parser_1.parseAst)(content, { errorRecovery: true });
     return checkWhileConstantAst(ast, file);
 }
 function checkWhileConstantAst(ast, file) {

package/dist/formatters.d.ts

@@ -1,5 +1,4 @@
 import { Issue } from './types';
 export declare function formatDefault(issues: Issue[], fileContents?: Map<string, string>): string;
-export declare function formatHtml(issues: Issue[], errorCount: number, warningCount: number): string;
 export declare function formatJson(issues: Issue[], errorCount?: number, warningCount?: number): string;
 //# sourceMappingURL=formatters.d.ts.map

package/dist/formatters.js

@@ -1,7 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.formatDefault = formatDefault;
-exports.formatHtml = formatHtml;
 exports.formatJson = formatJson;
 const locale_1 = require("./locale");
 const colors = {
@@ -69,21 +68,6 @@ function formatDefault(issues, fileContents) {
     }
     return lines.join('\n');
 }
-function formatHtml(issues, errorCount, warningCount) {
-    let html = `<!DOCTYPE html>
-<html lang="en">
-<head><meta charset="UTF-8"><title>@atlisp/lint Report</title></head>
-<body>
-<h1>@atlisp/lint Report</h1>
-<p>${errorCount} error(s), ${warningCount} warning(s)</p>
-<ul>
-`;
-    for (const iss of issues) {
-        html += `  <li>${iss.severity}: ${iss.file}:${iss.line} — ${iss.message}</li>\n`;
-    }
-    html += '</ul>\n</body>\n</html>';
-    return html;
-}
 function formatJson(issues, errorCount, warningCount) {
     return JSON.stringify({
         issues,