@atlisp/lint 0.1.11 → 0.1.14

package/dist/checks/unused-variable.js

@@ -24,6 +24,9 @@ function checkUnusedVariableAst(ast, file) {
                 continue;
             if (v.name.startsWith('/'))
                 continue;
+            // Skip earmuffed globals (*var*) and namespaced earmuffed (ns:*var*) — typically cross-file
+            if (isEarmuffed(v.name))
+                continue;
             if (!setqVars.has(v.name)) {
                 setqVars.set(v.name, v.pos.line);
             }
@@ -66,6 +69,11 @@ function checkUnusedVariableAst(ast, file) {
     }
     return issues;
 }
+/** Check if a variable name is earmuffed (*var*), optionally with a namespace prefix (ns:*var*) */
+function isEarmuffed(name) {
+    const localName = name.includes(':') ? name.slice(name.lastIndexOf(':') + 1) : name;
+    return localName.startsWith('*') && localName.endsWith('*') && localName.length > 2;
+}
 function isInsideLetBinding(node, varName) {
     let cur = node.parent;
     while (cur) {

package/dist/index.js

@@ -151,24 +151,29 @@ function* walkFiles(dir, rootDir, regex) {
         catch {
             continue;
         }
+        const rel = path.relative(rootDir, fullPath).replace(/\\/g, '/');
+        if (rel.startsWith('..'))
+            continue;
         if (stat.isDirectory()) {
             yield* walkFiles(fullPath, rootDir, regex);
         }
         else if (stat.isFile()) {
-            const rel = path.relative(rootDir, fullPath).replace(/\\/g, '/');
             if (regex.test(rel))
                 yield fullPath;
         }
     }
 }
+// Placeholder regex (dynamically constructed to avoid no-control-regex)
+const GLOB_PLACEHOLDER = '\x00GLOB\x00';
+const GLOB_RE = new RegExp(GLOB_PLACEHOLDER, 'g');
 function globFiles(pattern, rootDir) {
     const regexStr = pattern
         .replace(/\./g, '\\.')
-        .replace(/\*\*\//g, '\x00GLOB\x00')
+        .replace(/\*\*\//g, GLOB_PLACEHOLDER)
         .replace(/\*\*/g, '.*')
         .replace(/\*/g, '[^/]*')
         .replace(/\?/g, '.')
-        .replace(/\x00GLOB\x00/g, '(.*/)?');
+        .replace(GLOB_RE, '(.*/)?');
     const regex = new RegExp(`^${regexStr}$`);
     return Array.from(walkFiles(rootDir, rootDir, regex));
 }
@@ -198,9 +203,10 @@ function collectFiles(rootDir, opts, config) {
     const excludePatterns = config.source.exclude.map(e => {
         const str = e
             .replace(/\./g, '\\.')
-            .replace(/\*\*/g, '\x00STAR\x00')
+            .replace(/\*\*\//g, GLOB_PLACEHOLDER)
+            .replace(/\*\*/g, '.*')
             .replace(/\*/g, '[^/]*')
-            .replace(/\x00STAR\x00/g, '.*');
+            .replace(GLOB_RE, '(.*/)?');
         return new RegExp(`^${str}$`);
     });
     return Array.from(files)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlisp/lint",
-  "version": "0.1.11",
+  "version": "0.1.14",
   "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 ()))