@atlisp/lint 0.2.1 → 0.2.3

package/atlisp-lint.default.json

@@ -12,7 +12,7 @@
     "command_shell": "warn",
     "startapp": "warn",
     "vl_registry_write": "off",
-    "vlax_without_loading": "warn",
+    "vlax_without_loading": "info",
     "token_in_url": "warn",
     "open_without_close": "warn",
     "bare_function_names": "off",
@@ -46,7 +46,7 @@
     "commented_code": "warn",
     "double_not": "warn",
     "setq_single_arg": "warn",
-    "assoc_without_cdr": "warn",
+    "assoc_without_cdr": "info",
     "identical_branches": "warn",
     "while_constant": "warn",
     "cond_duplicate": "warn",
@@ -71,9 +71,9 @@
     "setq_multiple": "warn",
     "format_indent": "off",
     "function_order": "off",
-    "magic_number": "off",
+    "magic_number": "info",
     "mixed_indent": "warn",
-    "long_function_call": "warn",
+    "long_function_call": "info",
     "no_return": "warn",
     "shadow_builtin": "warn",
     "dynamic_doc": "warn",
@@ -82,7 +82,7 @@
     "redundant_if": "warn",
     "undeclared_setq": "warn",
     "unused_catch_result": "off",
-    "string_concat_loop": "off",
+    "string_concat_loop": "info",
     "if_without_else": "off",
     "redundant_list": "warn",
     "entget_in_loop": "warn",

package/dist/checks/builtin-arg-count.js

@@ -35,7 +35,7 @@ const BUILTIN_ARITY = {
     minusp: { min: 1, max: 1 },
     plusp: { min: 1, max: 1 },
     eq: { min: 2, max: 2 },
-    equal: { min: 2, max: 2 },
+    equal: { min: 2, max: 3 },
     '=': { min: 2, max: -1 },
     '/=': { min: 2, max: -1 },
     '<': { min: 2, max: -1 },

package/dist/checks/magic-number.js

@@ -12,7 +12,7 @@ function checkMagicNumber(content, file) {
         if (m) {
             const num = m[1];
             if (num !== '10' && num !== '20' && num !== '100' && num !== '1000') {
-                issues.push({ file, line: i + 1, severity: 'warn', rule: 'magic_number', message: (0, locale_1.t)('magic_number', num) });
+                issues.push({ file, line: i + 1, severity: 'off', rule: 'magic_number', message: (0, locale_1.t)('magic_number', num) });
             }
         }
     }

package/dist/checks/string-concat-loop.js

@@ -47,7 +47,7 @@ function checkStringConcatInLoop(content, file) {
                 issues.push({
                     file,
                     line: i + 1,
-                    severity: 'warn',
+                    severity: 'info',
                     rule: 'string_concat_loop',
                     message: (0, locale_1.t)('string_concat_loop', name),
                 });

package/dist/config.js

@@ -113,7 +113,7 @@ const DEFAULT_CONFIG = {
         append_single: 'info',
         setq_multiple: 'warn',
         function_order: 'off',
-        magic_number: 'info',
+        magic_number: 'off',
         mixed_indent: 'info',
         long_function_call: 'info',
         no_return: 'info',

package/dist/presets.js

@@ -141,7 +141,7 @@ exports.PRESETS = {
             append_single: 'error',
             setq_multiple: 'error',
             function_order: 'off',
-            magic_number: 'warn',
+            magic_number: 'off',
             mixed_indent: 'error',
             long_function_call: 'error',
             no_return: 'error',
@@ -154,7 +154,7 @@ exports.PRESETS = {
             module_registration: 'error',
             namespace_header: 'error',
             unused_catch_result: 'error',
-            string_concat_loop: 'off',
+            string_concat_loop: 'info',
             if_without_else: 'off',
             redundant_list: 'error',
             entget_in_loop: 'warn',

package/dist/rules.js

@@ -104,7 +104,7 @@ exports.RULES = [
         description: '检测可优化的 while/assoc 循环',
         category: '最佳实践',
     },
-    { name: 'magic_number', defaultSeverity: 'info', description: '检测硬编码的魔法数字', category: '风格' },
+    { name: 'magic_number', defaultSeverity: 'off', description: '检测硬编码的魔法数字', category: '风格' },
     {
         name: 'misplaced_else',
         defaultSeverity: 'info',

package/package.json

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

package/dist/atlisp-lint.default.json

@@ -1,90 +0,0 @@
-{
-  "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/lib/lint-sbcl.lisp

@@ -1,161 +0,0 @@
-;; @lisp SBCL Lint — syntax validation via SBCL
-;; Usage: sbcl --script lint-sbcl.lisp <src-dir> <stub-packages.json> <walk-exclude-json> <defmacro-allow-files-json> <locale>
-
-(require :uiop)
-
-(defparameter *errors* 0)
-(defparameter *warnings* 0)
-(defparameter *checked* 0)
-(defparameter *locale* "zh")
-
-;; ── i18n ──────────────────────────────────────────────────────────────
-(defun i18n (key)
-  (cdr (assoc key
-    (if (string= *locale* "zh")
-      '((:not-found . "not found symbol")
-        (:syntax-error . "syntax error")
-        (:vec-syntax . "#( vector syntax")
-        (:char-syntax . "# backslash char literal")
-        (:eval-syntax . "#. read-time eval")
-        (:defmacro . "defmacro")
-        (:trailing-ws . "trailing whitespace")
-        (:ok . "OK")
-        (:fail . "FAIL")
-        (:header . "  @lisp SBCL Lint")
-        (:source . "  Source")
-        (:found . "  Found ~d .lsp files")
-        (:scanned . "  Scanned")
-        (:errors . "  Errors")
-        (:warnings . "  Warnings"))
-      '((:not-found . "not found symbol")
-        (:syntax-error . "syntax error")
-        (:vec-syntax . "#( vector syntax")
-        (:char-syntax . "# backslash char literal")
-        (:eval-syntax . "#. read-time eval")
-        (:defmacro . "defmacro")
-        (:trailing-ws . "trailing whitespace")
-        (:ok . "OK")
-        (:fail . "FAIL")
-        (:header . "  @lisp SBCL Lint")
-        (:source . "  Source")
-        (:found . "  Found ~d .lsp files")
-        (:scanned . "  Scanned")
-        (:errors . "  Errors")
-        (:warnings . "  Warnings"))))))
-
-;; ── Stub packages from external file ──────────────────────────────────
-(defun load-stub-packages (file-path)
-  (flet ((ensure-pkg (name &optional use-list)
-           (unless (find-package name)
-             (make-package name :use (or use-list '())))))
-    (with-open-file (s file-path :direction :input)
-      (let ((packages (read s)))
-        (dolist (pkg packages)
-          (ensure-pkg (first pkg) (second pkg)))))))
-
-;; ── File walking ──────────────────────────────────────────────────────
-(defun source-files (dir)
-  (sort
-    (remove-if-not
-      (lambda (p)
-        (let ((n (pathname-type p)))
-          (and n (string-equal n "lsp"))))
-      (uiop:directory-files dir))
-    'string< :key #'namestring))
-
-(defun walk-tree (dir exclude-dirs)
-  (let ((result (source-files dir)))
-    (dolist (sub (uiop:subdirectories dir))
-      (let ((name (car (last (pathname-directory sub)))))
-        (unless (find name exclude-dirs :test 'string=)
-          (setf result (append result (walk-tree sub exclude-dirs))))))
-    result))
-
-;; ── Per-file check ────────────────────────────────────────────────────
-(defun check-file (path src-dir defmacro-allow-files)
-  (incf *checked*)
-  (let* ((rel (enough-namestring path src-dir))
-         (fullname (format nil "~a.~a" (pathname-name path) (pathname-type path))))
-    (format t "  ~a ... " rel)
-    (force-output)
-    ;; Syntax check via READ
-    (handler-case
-        (with-open-file (s path :direction :input)
-          (loop for form = (read s nil :eof)
-                until (eq form :eof)))
-      (error (e)
-        (let ((msg (princ-to-string e)))
-          (cond
-            ((search "not found" msg)
-             (format t "~%  [NOTE] ~a: ~a~%" rel (i18n :not-found))
-             (incf *warnings*))
-            (t
-             (format t "~a~%  [ERROR] ~a: ~a~%" (i18n :fail) rel (i18n :syntax-error))
-             (incf *errors*)
-             (return-from check-file))))))
-    ;; Trailing whitespace
-    (with-open-file (s path :direction :input)
-      (loop for line = (read-line s nil nil)
-            for lineno from 1
-            while line
-            when (and (> (length line) 0)
-                      (find (char line (1- (length line))) '(#\Space #\Tab)))
-              do (progn
-                   (format t "~%  [WARN] ~a line ~d: ~a~%" rel lineno (i18n :trailing-ws))
-                   (incf *warnings*))))
-    ;; CL-ism checks: raw content scan
-    (with-open-file (s path :direction :input)
-      (let ((content (make-string (file-length s))))
-        (file-position s 0)
-        (read-sequence content s)
-        (when (search "#(" content)
-          (format t "~%  [WARN] ~a: ~a~%" rel (i18n :vec-syntax))
-          (incf *warnings*))
-        (when (search "#\\" content)
-          (format t "~%  [WARN] ~a: ~a~%" rel (i18n :char-syntax))
-          (incf *warnings*))
-        (when (search "#." content)
-          (format t "~%  [WARN] ~a: ~a~%" rel (i18n :eval-syntax))
-          (incf *warnings*))
-        (when (and (search "defmacro" content)
-                   (not (some (lambda (s) (search s fullname)) defmacro-allow-files)))
-          (format t "~%  [WARN] ~a: ~a~%" rel (i18n :defmacro))
-          (incf *warnings*))))
-    (format t "~a~%" (i18n :ok))))
-
-;; ── Main ──────────────────────────────────────────────────────────────
-(defun main ()
-  (let* ((args (uiop:command-line-arguments))
-         (src-dir (first args))
-         (stub-file (second args))
-         (walk-exclude (if (third args)
-                          (read-from-string (third args))
-                          '(".vscode" "test" "experiment" "tools" ".git")))
-         (defmacro-allow (if (fourth args)
-                            (read-from-string (fourth args))
-                            '("compat-cl")))
-         (locale (fifth args)))
-    (when locale (setf *locale* locale))
-    (load-stub-packages stub-file)
-
-    (format t "~%==================================================~%")
-    (format t "~a~%" (i18n :header))
-    (format t "~a: ~a~%" (i18n :source) (namestring (pathname src-dir)))
-    (format t "==================================================~%~%")
-    (setf *errors* 0 *warnings* 0 *checked* 0)
-
-    (let ((files (walk-tree src-dir walk-exclude)))
-      (format t "~a~%~%" (format nil (i18n :found) (length files)))
-      (dolist (f files)
-        (check-file f src-dir defmacro-allow)))
-
-    (format t "~%==================================================~%")
-    (format t "~a: ~d~%" (i18n :scanned) *checked*)
-    (format t "~a: ~d~%" (i18n :errors) *errors*)
-    (format t "~a: ~d~%" (i18n :warnings) *warnings*)
-    (format t "==================================================~%~%")
-    (if (> *errors* 0)
-        (uiop:quit 1)
-        (uiop:quit 0))))
-
-(main)

package/dist/stub-packages.json

@@ -1,41 +0,0 @@
-((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 ()))