@atlisp/lint 0.2.5 → 0.2.6

package/README.md

@@ -123,7 +123,7 @@ npx @atlisp/lint --format-check
 | `nth_usage` | warn | 风格 | (nth 0 ...) 建议 car |
 | `append_single` | warn | 风格 | (append (list ...)) 建议 cons |
 | `mixed_indent` | warn | 风格 | 缩进混用 tab 和空格 |
-| `long_function_call` | warn | 风格 | 单行参数过多 |
+| `long_function_call` | off | 风格 | 单行参数过多 |
 | `no_return` | warn | 最佳实践 | 函数无返回值 |
 | `shadow_builtin` | warn | 最佳实践 | defun 覆盖内置函数 |
 | `dynamic_doc` | warn | 最佳实践 | C: 命令缺少 (princ) |
@@ -138,7 +138,6 @@ npx @atlisp/lint --format-check
 | `module_registration` | off | 架构 | 模块文件未注册到 \*modules\* |
 | `namespace_header` | off | 架构 | 缺少 (in-package ...) 头 |
 | `loop_optimization` | off | 最佳实践 | 优化 while/assoc 循环 |
-| `type_check` | off | 最佳实践 | getvar 结果未类型检查 |
 | `builtin_arg_count` | error | 正确性 | 内置函数调用参数个数不匹配 |
 | `unused_catch_result` | info | 正确性 | vl-catch-all-apply 结果被当布尔值用 |
 | `string_concat_loop` | info | 性能 | 循环内字符串拼接 |

package/atlisp-lint.default.json

@@ -73,12 +73,11 @@
     "function_order": "off",
     "magic_number": "info",
     "mixed_indent": "warn",
-    "long_function_call": "info",
+    "long_function_call": "off",
     "no_return": "warn",
     "shadow_builtin": "warn",
     "dynamic_doc": "warn",
     "loop_optimization": "off",
-    "type_check": "off",
     "redundant_if": "warn",
     "undeclared_setq": "warn",
     "unused_catch_result": "off",

package/dist/checks/cond-duplicate.js

@@ -5,7 +5,7 @@ exports.checkCondDuplicateAst = checkCondDuplicateAst;
 const locale_1 = require("../locale");
 const parser_1 = require("@atlisp/parser");
 function condKey(node) {
-    if (node.type === 'symbol')
+    if (node.type === 'symbol' || node.type === 'keyword')
         return `s:${node.name}`;
     if (node.type === 'number')
         return `n:${node.value}`;

package/dist/config.js

@@ -115,12 +115,11 @@ const DEFAULT_CONFIG = {
         function_order: 'off',
         magic_number: 'off',
         mixed_indent: 'info',
-        long_function_call: 'info',
+        long_function_call: 'off',
         no_return: 'info',
         shadow_builtin: 'warn',
         dynamic_doc: 'warn',
         loop_optimization: 'info',
-        type_check: 'info',
         format_indent: 'off',
         redundant_if: 'info',
         unused_catch_result: 'info',

package/dist/presets.js

@@ -61,7 +61,7 @@ exports.PRESETS = {
             append_single: 'info',
             setq_multiple: 'warn',
             mixed_indent: 'info',
-            long_function_call: 'info',
+            long_function_call: 'off',
             no_return: 'info',
             shadow_builtin: 'warn',
             dynamic_doc: 'warn',
@@ -143,12 +143,11 @@ exports.PRESETS = {
             function_order: 'off',
             magic_number: 'off',
             mixed_indent: 'error',
-            long_function_call: 'error',
+            long_function_call: 'off',
             no_return: 'error',
             shadow_builtin: 'error',
             dynamic_doc: 'error',
             loop_optimization: 'warn',
-            type_check: 'warn',
             format_indent: 'warn',
             redundant_if: 'error',
             module_registration: 'error',

package/dist/rules.js

@@ -97,7 +97,7 @@ exports.RULES = [
         category: '风格',
     },
     { name: 'line_length', defaultSeverity: 'info', description: '检查行长度是否超过限制', category: '风格' },
-    { name: 'long_function_call', defaultSeverity: 'info', description: '检测单行参数过多的函数调用', category: '风格' },
+    { name: 'long_function_call', defaultSeverity: 'off', description: '检测单行参数过多的函数调用', category: '风格' },
     {
         name: 'loop_optimization',
         defaultSeverity: 'info',
@@ -192,7 +192,6 @@ exports.RULES = [
     },
     { name: 'trailing_paren', defaultSeverity: 'warn', description: '检测多余闭合括号', category: '语法' },
     { name: 'trailing_whitespace', defaultSeverity: 'info', description: '检测行尾多余空格', category: '风格' },
-    { name: 'type_check', defaultSeverity: 'info', description: '检测 getvar 结果未做类型检查', category: '最佳实践' },
     {
         name: 'undeclared_setq',
         defaultSeverity: 'warn',

package/dist/runner.js

@@ -78,7 +78,6 @@ const no_return_1 = require("./checks/no-return");
 const shadow_builtin_1 = require("./checks/shadow-builtin");
 const dynamic_doc_1 = require("./checks/dynamic-doc");
 const loop_optimization_1 = require("./checks/loop-optimization");
-const type_check_1 = require("./checks/type-check");
 const redundant_if_1 = require("./checks/redundant-if");
 const undeclared_setq_1 = require("./checks/undeclared-setq");
 const format_indent_1 = require("./checks/format-indent");
@@ -157,7 +156,6 @@ function runChecks(content, file, config) {
     addIfEnabled('shadow_builtin', () => (0, shadow_builtin_1.checkShadowBuiltin)(content, file));
     addIfEnabled('dynamic_doc', () => (0, dynamic_doc_1.checkDynamicDoc)(content, file));
     addIfEnabled('loop_optimization', () => (0, loop_optimization_1.checkLoopOptimization)(content, file));
-    addIfEnabled('type_check', () => (0, type_check_1.checkTypeCheck)(content, file));
     addIfEnabled('redundant_if', () => (0, redundant_if_1.checkRedundantIf)(content, file));
     addIfEnabled('undeclared_setq', () => (0, undeclared_setq_1.checkUndeclaredSetq)(content, file));
     addIfEnabled('format_indent', () => (0, format_indent_1.checkFormatIndent)(content, file, config.line_length.tab_width));

package/dist/validate.js

@@ -66,7 +66,6 @@ const VALID_RULES = [
     'token_in_url',
     'trailing_paren',
     'trailing_whitespace',
-    'type_check',
     'undeclared_setq',
     'unused_let_binding',
     'unused_local_fun',

package/dist/visitor-runner.js

@@ -70,7 +70,7 @@ function runChecksWithVisitor(ast, file, config, disableMap) {
         return false;
     }
     function getCondKey(node) {
-        if (node.type === 'symbol')
+        if (node.type === 'symbol' || node.type === 'keyword')
             return node.name || '';
         if (node.type === 'string' || node.type === 'number')
             return String(node.value);

package/package.json

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