@atlisp/lint 0.1.8 → 0.1.10

package/atlisp-lint.default.json

@@ -9,7 +9,7 @@
     "encoding": "error",
     "cl_syntax": "warn",
     "quit_exit": "error",
-    "command_shell": "error",
+    "command_shell": "warn",
     "startapp": "warn",
     "vl_registry_write": "warn",
     "vlax_without_loading": "warn",
@@ -95,7 +95,7 @@
   "dangerous_calls": {
     "quit": "error",
     "exit": "error",
-    "command_shell": "error",
+    "command_shell": "warn",
     "startapp": "warn",
     "vl_registry_write": "warn"
   },

package/dist/checks/format-indent.d.ts

@@ -1,3 +1,3 @@
 import { Issue } from '../types';
-export declare function checkFormatIndent(content: string, file: string): Issue[];
+export declare function checkFormatIndent(content: string, file: string, indentSize?: number): Issue[];
 //# sourceMappingURL=format-indent.d.ts.map

package/dist/checks/format-indent.js

@@ -2,8 +2,8 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.checkFormatIndent = checkFormatIndent;
 const formatter_1 = require("../formatter");
-function checkFormatIndent(content, file) {
-    const formatted = (0, formatter_1.formatCode)(content);
+function checkFormatIndent(content, file, indentSize = 2) {
+    const formatted = (0, formatter_1.formatCode)(content, indentSize);
     if (content === formatted)
         return [];
     // Find first line that differs for better error location

package/dist/config.js

@@ -51,7 +51,7 @@ const DEFAULT_CONFIG = {
         encoding: 'error',
         cl_syntax: 'warn',
         quit_exit: 'error',
-        command_shell: 'error',
+        command_shell: 'warn',
         startapp: 'warn',
         vl_registry_write: 'warn',
         vlax_without_loading: 'warn',
@@ -137,7 +137,7 @@ const DEFAULT_CONFIG = {
     dangerous_calls: {
         quit: 'error',
         exit: 'error',
-        command_shell: 'error',
+        command_shell: 'warn',
         startapp: 'warn',
         vl_registry_write: 'warn',
     },

package/dist/index.js

@@ -164,10 +164,11 @@ function* walkFiles(dir, rootDir, regex) {
 function globFiles(pattern, rootDir) {
     const regexStr = pattern
         .replace(/\./g, '\\.')
-        .replace(/\*\*\//g, '(.*/)?')
+        .replace(/\*\*\//g, '\x00GLOB\x00')
         .replace(/\*\*/g, '.*')
         .replace(/\*/g, '[^/]*')
-        .replace(/\?/g, '.');
+        .replace(/\?/g, '.')
+        .replace(/\x00GLOB\x00/g, '(.*/)?');
     const regex = new RegExp(`^${regexStr}$`);
     return Array.from(walkFiles(rootDir, rootDir, regex));
 }
@@ -195,7 +196,11 @@ function collectFiles(rootDir, opts, config) {
         }
     }
     const excludePatterns = config.source.exclude.map(e => {
-        const str = e.replace(/\./g, '\\.').replace(/\*\*/g, '.*').replace(/\*/g, '[^/]*');
+        const str = e
+            .replace(/\./g, '\\.')
+            .replace(/\*\*/g, '\x00STAR\x00')
+            .replace(/\*/g, '[^/]*')
+            .replace(/\x00STAR\x00/g, '.*');
         return new RegExp(`^${str}$`);
     });
     return Array.from(files)
@@ -290,10 +295,11 @@ async function main() {
         }
         // --format-code mode: format files in place
         if (opts.formatCode || opts.formatCheck) {
+            const indentSize = config.line_length.tab_width;
             let needsFormat = false;
             for (const f of files) {
                 const content = fs.readFileSync(f, 'utf-8');
-                const formatted = (0, formatter_1.formatCode)(content);
+                const formatted = (0, formatter_1.formatCode)(content, indentSize);
                 if (content !== formatted) {
                     if (opts.formatCheck) {
                         console.log(`${path.relative(rootDir, f)} — needs formatting`);
@@ -306,7 +312,7 @@ async function main() {
                 }
                 // Idempotency check: format again to verify stability
                 if (!opts.formatCheck) {
-                    const reFormatted = (0, formatter_1.formatCode)(fs.readFileSync(f, 'utf-8'));
+                    const reFormatted = (0, formatter_1.formatCode)(fs.readFileSync(f, 'utf-8'), indentSize);
                     if (reFormatted !== fs.readFileSync(f, 'utf-8')) {
                         console.log(`${(0, locale_1.t)('summary.tag_warn')}: ${path.relative(rootDir, f)} — ${(0, locale_1.t)('index.format_unstable')}`);
                     }
@@ -331,10 +337,11 @@ async function main() {
             }
             // format_indent fix for non-staged mode (no lint results available yet)
             if (config.checks['format_indent'] !== 'off') {
+                const indentSize = config.line_length.tab_width;
                 for (const f of files) {
                     try {
                         const content = fs.readFileSync(f, 'utf-8');
-                        const formatted = (0, formatter_1.formatCode)(content);
+                        const formatted = (0, formatter_1.formatCode)(content, indentSize);
                         if (formatted !== content) {
                             fs.writeFileSync(f, formatted, 'utf-8');
                             console.log(`${(0, locale_1.t)('summary.tag_fail')}: ${path.relative(rootDir, f)} — ${(0, locale_1.t)('index.fixed', 'format_indent')}`);
@@ -402,13 +409,14 @@ async function main() {
             }
             // format_indent fix: apply formatCode to files with format_indent issues
             if (config.checks['format_indent'] !== 'off') {
+                const indentSize = config.line_length.tab_width;
                 for (const f of filesToLint) {
                     const relPath = path.relative(rootDir, f);
                     const hasFormatIssue = issues.some(i => i.file === relPath && i.rule === 'format_indent');
                     if (hasFormatIssue) {
                         try {
                             const content = fs.readFileSync(f, 'utf-8');
-                            const formatted = (0, formatter_1.formatCode)(content);
+                            const formatted = (0, formatter_1.formatCode)(content, indentSize);
                             if (formatted !== content) {
                                 fs.writeFileSync(f, formatted, 'utf-8');
                                 console.log(`${(0, locale_1.t)('summary.tag_fail')}: ${relPath} — ${(0, locale_1.t)('index.fixed', 'format_indent')}`);

package/dist/presets.js

@@ -8,7 +8,7 @@ exports.PRESETS = {
             encoding: 'error',
             cl_syntax: 'warn',
             quit_exit: 'error',
-            command_shell: 'error',
+            command_shell: 'warn',
             startapp: 'warn',
             vl_registry_write: 'warn',
             token_in_url: 'warn',
@@ -74,7 +74,7 @@ exports.PRESETS = {
             encoding: 'error',
             cl_syntax: 'error',
             quit_exit: 'error',
-            command_shell: 'error',
+            command_shell: 'warn',
             startapp: 'error',
             vl_registry_write: 'error',
             vlax_without_loading: 'error',
@@ -152,7 +152,7 @@ exports.PRESETS = {
             parens: 'error',
             encoding: 'error',
             quit_exit: 'error',
-            command_shell: 'error',
+            command_shell: 'warn',
         },
     },
 };

package/dist/runner.js

@@ -235,7 +235,7 @@ function runChecks(content, file, config) {
     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('format_indent', () => (0, format_indent_1.checkFormatIndent)(content, file));
+    addIfEnabled('format_indent', () => (0, format_indent_1.checkFormatIndent)(content, file, config.line_length.tab_width));
     return issues;
 }
 /** Single-pass visitor-based runChecks using AstVisitor */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlisp/lint",
-  "version": "0.1.8",
+  "version": "0.1.10",
   "description": "AutoLISP static analysis tool — parens, security, conventions, SBCL syntax validation",
   "keywords": [
     "CAD",
@@ -31,8 +31,8 @@
     "test:watch": "jest --watch",
     "lint": "eslint src",
     "typecheck": "tsc --noEmit",
-    "format": "prettier --write \"src/**/*.ts\" \"test/**/*.ts\"",
-    "format:check": "prettier --check \"src/**/*.ts\" \"test/**/*.ts\"",
+    "format": "prettier --write \"src/**/*.ts\" \"test/**/*.ts\" \"*.json\" \"!stub-packages.json\"",
+    "format:check": "prettier --check \"src/**/*.ts\" \"test/**/*.ts\" \"*.json\" \"!stub-packages.json\"",
     "prepublishOnly": "npm run build",
     "prepack": "npm run build"
   },