@atlisp/lint 0.1.1 → 0.1.3
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/LICENSE +21 -0
- package/README.md +2 -2
- package/atlisp-lint.default.json +28 -3
- package/bin/atlisp-lint +1 -1
- package/dist/atlisp-lint.default.json +28 -3
- package/dist/checks/append-single.d.ts +3 -0
- package/dist/checks/append-single.d.ts.map +1 -0
- package/dist/checks/append-single.js +17 -0
- package/dist/checks/append-single.js.map +1 -0
- package/dist/checks/arg-count.d.ts +3 -0
- package/dist/checks/arg-count.d.ts.map +1 -0
- package/dist/checks/arg-count.js +120 -0
- package/dist/checks/arg-count.js.map +1 -0
- package/dist/checks/comment-style.d.ts +3 -0
- package/dist/checks/comment-style.d.ts.map +1 -0
- package/dist/checks/comment-style.js +21 -0
- package/dist/checks/comment-style.js.map +1 -0
- package/dist/checks/cond-simplify.d.ts +3 -0
- package/dist/checks/cond-simplify.d.ts.map +1 -0
- package/dist/checks/cond-simplify.js +42 -0
- package/dist/checks/cond-simplify.js.map +1 -0
- package/dist/checks/dangerous-calls.d.ts.map +1 -1
- package/dist/checks/dangerous-calls.js +9 -7
- package/dist/checks/dangerous-calls.js.map +1 -1
- package/dist/checks/dynamic-doc.d.ts +3 -0
- package/dist/checks/dynamic-doc.d.ts.map +1 -0
- package/dist/checks/dynamic-doc.js +21 -0
- package/dist/checks/dynamic-doc.js.map +1 -0
- package/dist/checks/empty-catch.d.ts +3 -0
- package/dist/checks/empty-catch.d.ts.map +1 -0
- package/dist/checks/empty-catch.js +31 -0
- package/dist/checks/empty-catch.js.map +1 -0
- package/dist/checks/eq-usage.d.ts +3 -0
- package/dist/checks/eq-usage.d.ts.map +1 -0
- package/dist/checks/eq-usage.js +22 -0
- package/dist/checks/eq-usage.js.map +1 -0
- package/dist/checks/error-handling.d.ts +3 -0
- package/dist/checks/error-handling.d.ts.map +1 -0
- package/dist/checks/error-handling.js +53 -0
- package/dist/checks/error-handling.js.map +1 -0
- package/dist/checks/extra-parens.d.ts +3 -0
- package/dist/checks/extra-parens.d.ts.map +1 -0
- package/dist/checks/extra-parens.js +42 -0
- package/dist/checks/extra-parens.js.map +1 -0
- package/dist/checks/function-order.d.ts +3 -0
- package/dist/checks/function-order.d.ts.map +1 -0
- package/dist/checks/function-order.js +33 -0
- package/dist/checks/function-order.js.map +1 -0
- package/dist/checks/global-naming.d.ts +3 -0
- package/dist/checks/global-naming.d.ts.map +1 -0
- package/dist/checks/global-naming.js +51 -0
- package/dist/checks/global-naming.js.map +1 -0
- package/dist/checks/index.d.ts +3 -0
- package/dist/checks/index.d.ts.map +1 -0
- package/dist/checks/index.js +108 -0
- package/dist/checks/index.js.map +1 -0
- package/dist/checks/lambda-syntax.d.ts +3 -0
- package/dist/checks/lambda-syntax.d.ts.map +1 -0
- package/dist/checks/lambda-syntax.js +22 -0
- package/dist/checks/lambda-syntax.js.map +1 -0
- package/dist/checks/long-function-call.d.ts +3 -0
- package/dist/checks/long-function-call.d.ts.map +1 -0
- package/dist/checks/long-function-call.js +48 -0
- package/dist/checks/long-function-call.js.map +1 -0
- package/dist/checks/loop-optimization.d.ts +3 -0
- package/dist/checks/loop-optimization.d.ts.map +1 -0
- package/dist/checks/loop-optimization.js +17 -0
- package/dist/checks/loop-optimization.js.map +1 -0
- package/dist/checks/magic-number.d.ts +3 -0
- package/dist/checks/magic-number.d.ts.map +1 -0
- package/dist/checks/magic-number.js +21 -0
- package/dist/checks/magic-number.js.map +1 -0
- package/dist/checks/missing-doc.js +2 -2
- package/dist/checks/missing-doc.js.map +1 -1
- package/dist/checks/mixed-indent.d.ts +3 -0
- package/dist/checks/mixed-indent.d.ts.map +1 -0
- package/dist/checks/mixed-indent.js +19 -0
- package/dist/checks/mixed-indent.js.map +1 -0
- package/dist/checks/no-return.d.ts +3 -0
- package/dist/checks/no-return.d.ts.map +1 -0
- package/dist/checks/no-return.js +45 -0
- package/dist/checks/no-return.js.map +1 -0
- package/dist/checks/nth-usage.d.ts +3 -0
- package/dist/checks/nth-usage.d.ts.map +1 -0
- package/dist/checks/nth-usage.js +17 -0
- package/dist/checks/nth-usage.js.map +1 -0
- package/dist/checks/parameter-naming.d.ts.map +1 -1
- package/dist/checks/parameter-naming.js +4 -1
- package/dist/checks/parameter-naming.js.map +1 -1
- package/dist/checks/quote-style.d.ts +3 -0
- package/dist/checks/quote-style.d.ts.map +1 -0
- package/dist/checks/quote-style.js +22 -0
- package/dist/checks/quote-style.js.map +1 -0
- package/dist/checks/redundant-if.d.ts +3 -0
- package/dist/checks/redundant-if.d.ts.map +1 -0
- package/dist/checks/redundant-if.js +17 -0
- package/dist/checks/redundant-if.js.map +1 -0
- package/dist/checks/redundant-progn.d.ts +3 -0
- package/dist/checks/redundant-progn.d.ts.map +1 -0
- package/dist/checks/redundant-progn.js +22 -0
- package/dist/checks/redundant-progn.js.map +1 -0
- package/dist/checks/setq-multiple.d.ts +3 -0
- package/dist/checks/setq-multiple.d.ts.map +1 -0
- package/dist/checks/setq-multiple.js +17 -0
- package/dist/checks/setq-multiple.js.map +1 -0
- package/dist/checks/shadow-builtin.d.ts +3 -0
- package/dist/checks/shadow-builtin.d.ts.map +1 -0
- package/dist/checks/shadow-builtin.js +29 -0
- package/dist/checks/shadow-builtin.js.map +1 -0
- package/dist/checks/strcat-usage.d.ts +3 -0
- package/dist/checks/strcat-usage.d.ts.map +1 -0
- package/dist/checks/strcat-usage.js +22 -0
- package/dist/checks/strcat-usage.js.map +1 -0
- package/dist/checks/type-check.d.ts +3 -0
- package/dist/checks/type-check.d.ts.map +1 -0
- package/dist/checks/type-check.js +26 -0
- package/dist/checks/type-check.js.map +1 -0
- package/dist/checks/unused-variable.d.ts.map +1 -1
- package/dist/checks/unused-variable.js +16 -4
- package/dist/checks/unused-variable.js.map +1 -1
- package/dist/config.d.ts.map +1 -1
- package/dist/config.js +28 -3
- package/dist/config.js.map +1 -1
- package/dist/formatters.d.ts +1 -0
- package/dist/formatters.d.ts.map +1 -1
- package/dist/formatters.js +55 -0
- package/dist/formatters.js.map +1 -1
- package/dist/index.d.ts +1 -1
- package/dist/index.d.ts.map +1 -1
- package/dist/index.js +164 -18
- package/dist/index.js.map +1 -1
- package/dist/lib/lint-sbcl.lisp +54 -19
- package/dist/locale.d.ts.map +1 -1
- package/dist/locale.js +66 -2
- package/dist/locale.js.map +1 -1
- package/dist/rules.d.ts +9 -0
- package/dist/rules.d.ts.map +1 -0
- package/dist/rules.js +58 -0
- package/dist/rules.js.map +1 -0
- package/dist/runner.d.ts +11 -2
- package/dist/runner.d.ts.map +1 -1
- package/dist/runner.js +69 -54
- package/dist/runner.js.map +1 -1
- package/dist/sbcl.d.ts.map +1 -1
- package/dist/sbcl.js +2 -0
- package/dist/sbcl.js.map +1 -1
- package/dist/validate.d.ts +8 -0
- package/dist/validate.d.ts.map +1 -0
- package/dist/validate.js +59 -0
- package/dist/validate.js.map +1 -0
- package/dist/watch.d.ts +9 -0
- package/dist/watch.d.ts.map +1 -0
- package/dist/watch.js +109 -0
- package/dist/watch.js.map +1 -0
- package/dist/worker.js +2 -47
- package/dist/worker.js.map +1 -1
- package/lib/lint-sbcl.lisp +54 -19
- package/package.json +8 -1
|
@@ -0,0 +1,45 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.checkNoReturn = checkNoReturn;
|
|
4
|
+
const locale_1 = require("../locale");
|
|
5
|
+
const utils_1 = require("../utils");
|
|
6
|
+
function checkNoReturn(content, file) {
|
|
7
|
+
const issues = [];
|
|
8
|
+
const lines = content.split('\n');
|
|
9
|
+
let inDefun = false;
|
|
10
|
+
let defunName = '';
|
|
11
|
+
let defunStart = 0;
|
|
12
|
+
let depth = 0;
|
|
13
|
+
let hasReturn = false;
|
|
14
|
+
for (let i = 0; i < lines.length; i++) {
|
|
15
|
+
const stripped = (0, utils_1.stripLine)(lines[i]);
|
|
16
|
+
if (!inDefun) {
|
|
17
|
+
const m = stripped.match(/\(defun\s+(\S+)/);
|
|
18
|
+
if (m) {
|
|
19
|
+
inDefun = true;
|
|
20
|
+
defunName = m[1];
|
|
21
|
+
defunStart = i + 1;
|
|
22
|
+
depth = 0;
|
|
23
|
+
hasReturn = false;
|
|
24
|
+
}
|
|
25
|
+
}
|
|
26
|
+
if (inDefun) {
|
|
27
|
+
for (const ch of stripped) {
|
|
28
|
+
if (ch === '(')
|
|
29
|
+
depth++;
|
|
30
|
+
else if (ch === ')')
|
|
31
|
+
depth--;
|
|
32
|
+
}
|
|
33
|
+
if (stripped.trim().startsWith('(') && !stripped.includes('defun'))
|
|
34
|
+
hasReturn = true;
|
|
35
|
+
if (depth === 0 && defunName && !defunName.startsWith('c:')) {
|
|
36
|
+
if (!hasReturn) {
|
|
37
|
+
issues.push({ file, line: defunStart, severity: 'warn', rule: 'no_return', message: (0, locale_1.t)('no_return', defunName) });
|
|
38
|
+
}
|
|
39
|
+
inDefun = false;
|
|
40
|
+
}
|
|
41
|
+
}
|
|
42
|
+
}
|
|
43
|
+
return issues;
|
|
44
|
+
}
|
|
45
|
+
//# sourceMappingURL=no-return.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"no-return.js","sourceRoot":"","sources":["../../src/checks/no-return.ts"],"names":[],"mappings":";;AAIA,sCA2BC;AA9BD,sCAA8B;AAC9B,oCAAqC;AAErC,SAAgB,aAAa,CAAC,OAAe,EAAE,IAAY;IACzD,MAAM,MAAM,GAAY,EAAE,CAAC;IAC3B,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAClC,IAAI,OAAO,GAAG,KAAK,CAAC;IACpB,IAAI,SAAS,GAAG,EAAE,CAAC;IACnB,IAAI,UAAU,GAAG,CAAC,CAAC;IACnB,IAAI,KAAK,GAAG,CAAC,CAAC;IACd,IAAI,SAAS,GAAG,KAAK,CAAC;IAEtB,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACtC,MAAM,QAAQ,GAAG,IAAA,iBAAS,EAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QACrC,IAAI,CAAC,OAAO,EAAE,CAAC;YACb,MAAM,CAAC,GAAG,QAAQ,CAAC,KAAK,CAAC,iBAAiB,CAAC,CAAC;YAC5C,IAAI,CAAC,EAAE,CAAC;gBAAC,OAAO,GAAG,IAAI,CAAC;gBAAC,SAAS,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC;gBAAC,UAAU,GAAG,CAAC,GAAG,CAAC,CAAC;gBAAC,KAAK,GAAG,CAAC,CAAC;gBAAC,SAAS,GAAG,KAAK,CAAC;YAAC,CAAC;QAChG,CAAC;QACD,IAAI,OAAO,EAAE,CAAC;YACZ,KAAK,MAAM,EAAE,IAAI,QAAQ,EAAE,CAAC;gBAAC,IAAI,EAAE,KAAK,GAAG;oBAAE,KAAK,EAAE,CAAC;qBAAM,IAAI,EAAE,KAAK,GAAG;oBAAE,KAAK,EAAE,CAAC;YAAC,CAAC;YACrF,IAAI,QAAQ,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC,OAAO,CAAC;gBAAE,SAAS,GAAG,IAAI,CAAC;YACrF,IAAI,KAAK,KAAK,CAAC,IAAI,SAAS,IAAI,CAAC,SAAS,CAAC,UAAU,CAAC,IAAI,CAAC,EAAE,CAAC;gBAC5D,IAAI,CAAC,SAAS,EAAE,CAAC;oBACf,MAAM,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,WAAW,EAAE,OAAO,EAAE,IAAA,UAAC,EAAC,WAAW,EAAE,SAAS,CAAC,EAAE,CAAC,CAAC;gBACnH,CAAC;gBACD,OAAO,GAAG,KAAK,CAAC;YAClB,CAAC;QACH,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"nth-usage.d.ts","sourceRoot":"","sources":["../../src/checks/nth-usage.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,UAAU,CAAC;AAIjC,wBAAgB,aAAa,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,KAAK,EAAE,CAUpE"}
|
|
@@ -0,0 +1,17 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.checkNthUsage = checkNthUsage;
|
|
4
|
+
const locale_1 = require("../locale");
|
|
5
|
+
const utils_1 = require("../utils");
|
|
6
|
+
function checkNthUsage(content, file) {
|
|
7
|
+
const issues = [];
|
|
8
|
+
const lines = content.split('\n');
|
|
9
|
+
for (let i = 0; i < lines.length; i++) {
|
|
10
|
+
const stripped = (0, utils_1.stripLine)(lines[i]);
|
|
11
|
+
if (/\(nth\s+0\b/.test(stripped)) {
|
|
12
|
+
issues.push({ file, line: i + 1, severity: 'warn', rule: 'nth_usage', message: (0, locale_1.t)('nth_usage') });
|
|
13
|
+
}
|
|
14
|
+
}
|
|
15
|
+
return issues;
|
|
16
|
+
}
|
|
17
|
+
//# sourceMappingURL=nth-usage.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"nth-usage.js","sourceRoot":"","sources":["../../src/checks/nth-usage.ts"],"names":[],"mappings":";;AAIA,sCAUC;AAbD,sCAA8B;AAC9B,oCAAqC;AAErC,SAAgB,aAAa,CAAC,OAAe,EAAE,IAAY;IACzD,MAAM,MAAM,GAAY,EAAE,CAAC;IAC3B,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAClC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACtC,MAAM,QAAQ,GAAG,IAAA,iBAAS,EAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QACrC,IAAI,aAAa,CAAC,IAAI,CAAC,QAAQ,CAAC,EAAE,CAAC;YACjC,MAAM,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,CAAC,GAAG,CAAC,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,WAAW,EAAE,OAAO,EAAE,IAAA,UAAC,EAAC,WAAW,CAAC,EAAE,CAAC,CAAC;QACnG,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"parameter-naming.d.ts","sourceRoot":"","sources":["../../src/checks/parameter-naming.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,UAAU,CAAC;AAIjC,wBAAgB,oBAAoB,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,KAAK,EAAE,
|
|
1
|
+
{"version":3,"file":"parameter-naming.d.ts","sourceRoot":"","sources":["../../src/checks/parameter-naming.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,UAAU,CAAC;AAIjC,wBAAgB,oBAAoB,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,KAAK,EAAE,CAqB3E"}
|
|
@@ -11,7 +11,10 @@ function checkParameterNaming(content, file) {
|
|
|
11
11
|
const m = stripped.match(/\(defun\s+(\S+)\s+\(([^)]*)\)/);
|
|
12
12
|
if (!m)
|
|
13
13
|
continue;
|
|
14
|
-
const
|
|
14
|
+
const all = m[2];
|
|
15
|
+
const slashIdx = all.indexOf('/');
|
|
16
|
+
const paramStr = slashIdx >= 0 ? all.slice(0, slashIdx).trim() : all;
|
|
17
|
+
const params = paramStr.split(/\s+/).filter(Boolean);
|
|
15
18
|
for (const p of params) {
|
|
16
19
|
if (/^[A-Z]/.test(p) && p !== 'T' && p !== 'nil') {
|
|
17
20
|
issues.push({
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"parameter-naming.js","sourceRoot":"","sources":["../../src/checks/parameter-naming.ts"],"names":[],"mappings":";;AAIA,
|
|
1
|
+
{"version":3,"file":"parameter-naming.js","sourceRoot":"","sources":["../../src/checks/parameter-naming.ts"],"names":[],"mappings":";;AAIA,oDAqBC;AAxBD,sCAA8B;AAC9B,oCAAqC;AAErC,SAAgB,oBAAoB,CAAC,OAAe,EAAE,IAAY;IAChE,MAAM,MAAM,GAAY,EAAE,CAAC;IAC3B,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAClC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACtC,MAAM,QAAQ,GAAG,IAAA,iBAAS,EAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QACrC,MAAM,CAAC,GAAG,QAAQ,CAAC,KAAK,CAAC,+BAA+B,CAAC,CAAC;QAC1D,IAAI,CAAC,CAAC;YAAE,SAAS;QACjB,MAAM,GAAG,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC;QACjB,MAAM,QAAQ,GAAG,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;QAClC,MAAM,QAAQ,GAAG,QAAQ,IAAI,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC,EAAE,QAAQ,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,GAAG,CAAC;QACrE,MAAM,MAAM,GAAG,QAAQ,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;QACrD,KAAK,MAAM,CAAC,IAAI,MAAM,EAAE,CAAC;YACvB,IAAI,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,GAAG,IAAI,CAAC,KAAK,KAAK,EAAE,CAAC;gBACjD,MAAM,CAAC,IAAI,CAAC;oBACV,IAAI,EAAE,IAAI,EAAE,CAAC,GAAG,CAAC,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,kBAAkB;oBAC7D,OAAO,EAAE,IAAA,UAAC,EAAC,kBAAkB,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC;iBACxC,CAAC,CAAC;YACL,CAAC;QACH,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"quote-style.d.ts","sourceRoot":"","sources":["../../src/checks/quote-style.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,UAAU,CAAC;AAIjC,wBAAgB,eAAe,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,KAAK,EAAE,CAetE"}
|
|
@@ -0,0 +1,22 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.checkQuoteStyle = checkQuoteStyle;
|
|
4
|
+
const locale_1 = require("../locale");
|
|
5
|
+
const utils_1 = require("../utils");
|
|
6
|
+
function checkQuoteStyle(content, file) {
|
|
7
|
+
const issues = [];
|
|
8
|
+
const lines = content.split('\n');
|
|
9
|
+
for (let i = 0; i < lines.length; i++) {
|
|
10
|
+
const stripped = (0, utils_1.stripLine)(lines[i]);
|
|
11
|
+
// Detect (quote ...) that could be '
|
|
12
|
+
const m = stripped.match(/\(quote\s+/);
|
|
13
|
+
if (m) {
|
|
14
|
+
issues.push({
|
|
15
|
+
file, line: i + 1, severity: 'warn', rule: 'quote_style',
|
|
16
|
+
message: (0, locale_1.t)('quote_style'),
|
|
17
|
+
});
|
|
18
|
+
}
|
|
19
|
+
}
|
|
20
|
+
return issues;
|
|
21
|
+
}
|
|
22
|
+
//# sourceMappingURL=quote-style.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"quote-style.js","sourceRoot":"","sources":["../../src/checks/quote-style.ts"],"names":[],"mappings":";;AAIA,0CAeC;AAlBD,sCAA8B;AAC9B,oCAAqC;AAErC,SAAgB,eAAe,CAAC,OAAe,EAAE,IAAY;IAC3D,MAAM,MAAM,GAAY,EAAE,CAAC;IAC3B,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAClC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACtC,MAAM,QAAQ,GAAG,IAAA,iBAAS,EAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QACrC,qCAAqC;QACrC,MAAM,CAAC,GAAG,QAAQ,CAAC,KAAK,CAAC,YAAY,CAAC,CAAC;QACvC,IAAI,CAAC,EAAE,CAAC;YACN,MAAM,CAAC,IAAI,CAAC;gBACV,IAAI,EAAE,IAAI,EAAE,CAAC,GAAG,CAAC,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,aAAa;gBACxD,OAAO,EAAE,IAAA,UAAC,EAAC,aAAa,CAAC;aAC1B,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"redundant-if.d.ts","sourceRoot":"","sources":["../../src/checks/redundant-if.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,UAAU,CAAC;AAIjC,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,KAAK,EAAE,CAUvE"}
|
|
@@ -0,0 +1,17 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.checkRedundantIf = checkRedundantIf;
|
|
4
|
+
const locale_1 = require("../locale");
|
|
5
|
+
const utils_1 = require("../utils");
|
|
6
|
+
function checkRedundantIf(content, file) {
|
|
7
|
+
const issues = [];
|
|
8
|
+
const lines = content.split('\n');
|
|
9
|
+
for (let i = 0; i < lines.length; i++) {
|
|
10
|
+
const stripped = (0, utils_1.stripLine)(lines[i]);
|
|
11
|
+
if (/\((if|when|unless)\s+\S+\s+\(progn\b/.test(stripped)) {
|
|
12
|
+
issues.push({ file, line: i + 1, severity: 'warn', rule: 'redundant_if', message: (0, locale_1.t)('redundant_if') });
|
|
13
|
+
}
|
|
14
|
+
}
|
|
15
|
+
return issues;
|
|
16
|
+
}
|
|
17
|
+
//# sourceMappingURL=redundant-if.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"redundant-if.js","sourceRoot":"","sources":["../../src/checks/redundant-if.ts"],"names":[],"mappings":";;AAIA,4CAUC;AAbD,sCAA8B;AAC9B,oCAAqC;AAErC,SAAgB,gBAAgB,CAAC,OAAe,EAAE,IAAY;IAC5D,MAAM,MAAM,GAAY,EAAE,CAAC;IAC3B,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAClC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACtC,MAAM,QAAQ,GAAG,IAAA,iBAAS,EAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QACrC,IAAI,sCAAsC,CAAC,IAAI,CAAC,QAAQ,CAAC,EAAE,CAAC;YAC1D,MAAM,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,CAAC,GAAG,CAAC,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,cAAc,EAAE,OAAO,EAAE,IAAA,UAAC,EAAC,cAAc,CAAC,EAAE,CAAC,CAAC;QACzG,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"redundant-progn.d.ts","sourceRoot":"","sources":["../../src/checks/redundant-progn.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,UAAU,CAAC;AAIjC,wBAAgB,mBAAmB,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,KAAK,EAAE,CAe1E"}
|
|
@@ -0,0 +1,22 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.checkRedundantProgn = checkRedundantProgn;
|
|
4
|
+
const locale_1 = require("../locale");
|
|
5
|
+
const utils_1 = require("../utils");
|
|
6
|
+
function checkRedundantProgn(content, file) {
|
|
7
|
+
const issues = [];
|
|
8
|
+
const lines = content.split('\n');
|
|
9
|
+
for (let i = 0; i < lines.length; i++) {
|
|
10
|
+
const stripped = (0, utils_1.stripLine)(lines[i]).trim();
|
|
11
|
+
// Detect (if expr (progn ...) ...) or (cond ((t) (progn ...)))
|
|
12
|
+
const m = stripped.match(/\((if|when|unless|cond)\s+.+\(progn\b/);
|
|
13
|
+
if (m) {
|
|
14
|
+
issues.push({
|
|
15
|
+
file, line: i + 1, severity: 'warn', rule: 'redundant_progn',
|
|
16
|
+
message: (0, locale_1.t)('redundant_progn'),
|
|
17
|
+
});
|
|
18
|
+
}
|
|
19
|
+
}
|
|
20
|
+
return issues;
|
|
21
|
+
}
|
|
22
|
+
//# sourceMappingURL=redundant-progn.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"redundant-progn.js","sourceRoot":"","sources":["../../src/checks/redundant-progn.ts"],"names":[],"mappings":";;AAIA,kDAeC;AAlBD,sCAA8B;AAC9B,oCAAqC;AAErC,SAAgB,mBAAmB,CAAC,OAAe,EAAE,IAAY;IAC/D,MAAM,MAAM,GAAY,EAAE,CAAC;IAC3B,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAClC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACtC,MAAM,QAAQ,GAAG,IAAA,iBAAS,EAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QAC5C,+DAA+D;QAC/D,MAAM,CAAC,GAAG,QAAQ,CAAC,KAAK,CAAC,uCAAuC,CAAC,CAAC;QAClE,IAAI,CAAC,EAAE,CAAC;YACN,MAAM,CAAC,IAAI,CAAC;gBACV,IAAI,EAAE,IAAI,EAAE,CAAC,GAAG,CAAC,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,iBAAiB;gBAC5D,OAAO,EAAE,IAAA,UAAC,EAAC,iBAAiB,CAAC;aAC9B,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"setq-multiple.d.ts","sourceRoot":"","sources":["../../src/checks/setq-multiple.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,UAAU,CAAC;AAIjC,wBAAgB,iBAAiB,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,KAAK,EAAE,CAUxE"}
|
|
@@ -0,0 +1,17 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.checkSetqMultiple = checkSetqMultiple;
|
|
4
|
+
const locale_1 = require("../locale");
|
|
5
|
+
const utils_1 = require("../utils");
|
|
6
|
+
function checkSetqMultiple(content, file) {
|
|
7
|
+
const issues = [];
|
|
8
|
+
const lines = content.split('\n');
|
|
9
|
+
for (let i = 0; i < lines.length; i++) {
|
|
10
|
+
const stripped = (0, utils_1.stripLine)(lines[i]);
|
|
11
|
+
if (/\(setq\s+\S+\s+[^\s)]+\)\s*\(setq\b/.test(stripped)) {
|
|
12
|
+
issues.push({ file, line: i + 1, severity: 'warn', rule: 'setq_multiple', message: (0, locale_1.t)('setq_multiple') });
|
|
13
|
+
}
|
|
14
|
+
}
|
|
15
|
+
return issues;
|
|
16
|
+
}
|
|
17
|
+
//# sourceMappingURL=setq-multiple.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"setq-multiple.js","sourceRoot":"","sources":["../../src/checks/setq-multiple.ts"],"names":[],"mappings":";;AAIA,8CAUC;AAbD,sCAA8B;AAC9B,oCAAqC;AAErC,SAAgB,iBAAiB,CAAC,OAAe,EAAE,IAAY;IAC7D,MAAM,MAAM,GAAY,EAAE,CAAC;IAC3B,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAClC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACtC,MAAM,QAAQ,GAAG,IAAA,iBAAS,EAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QACrC,IAAI,qCAAqC,CAAC,IAAI,CAAC,QAAQ,CAAC,EAAE,CAAC;YACzD,MAAM,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,CAAC,GAAG,CAAC,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,eAAe,EAAE,OAAO,EAAE,IAAA,UAAC,EAAC,eAAe,CAAC,EAAE,CAAC,CAAC;QAC3G,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"shadow-builtin.d.ts","sourceRoot":"","sources":["../../src/checks/shadow-builtin.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,UAAU,CAAC;AAgBjC,wBAAgB,kBAAkB,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,KAAK,EAAE,CAWzE"}
|
|
@@ -0,0 +1,29 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.checkShadowBuiltin = checkShadowBuiltin;
|
|
4
|
+
const locale_1 = require("../locale");
|
|
5
|
+
const utils_1 = require("../utils");
|
|
6
|
+
const BUILTINS = new Set([
|
|
7
|
+
'car', 'cdr', 'cons', 'list', 'append', 'length', 'member', 'assoc',
|
|
8
|
+
'subst', 'apply', 'mapcar', 'foreach', 'while', 'repeat', 'if', 'cond',
|
|
9
|
+
'progn', 'setq', 'set', 'quote', 'function', 'lambda', 'defun', 'princ',
|
|
10
|
+
'print', 'prompt', 'getint', 'getreal', 'getstring', 'getpoint', 'getvar',
|
|
11
|
+
'setvar', 'command', 'vlax-', 'vla-', 'vl-', 'ssget', 'ssname', 'sslength',
|
|
12
|
+
'entget', 'entmod', 'entmake', 'entdel', 'entlast', 'entsel', 'tblobjname',
|
|
13
|
+
'tblsearch', 'tblnext', 'regapp', 'startapp', 'vl-registry-write',
|
|
14
|
+
'vl-registry-read', 'vl-registry-delete', 'vl-catch-all-apply',
|
|
15
|
+
'vl-catch-all-error-p', 'vl-load-com', 'vlax-get-or-create-object',
|
|
16
|
+
]);
|
|
17
|
+
function checkShadowBuiltin(content, file) {
|
|
18
|
+
const issues = [];
|
|
19
|
+
const lines = content.split('\n');
|
|
20
|
+
for (let i = 0; i < lines.length; i++) {
|
|
21
|
+
const stripped = (0, utils_1.stripLine)(lines[i]);
|
|
22
|
+
const m = stripped.match(/\(defun\s+(\S+)\b/);
|
|
23
|
+
if (m && BUILTINS.has(m[1].toLowerCase())) {
|
|
24
|
+
issues.push({ file, line: i + 1, severity: 'warn', rule: 'shadow_builtin', message: (0, locale_1.t)('shadow_builtin', m[1]) });
|
|
25
|
+
}
|
|
26
|
+
}
|
|
27
|
+
return issues;
|
|
28
|
+
}
|
|
29
|
+
//# sourceMappingURL=shadow-builtin.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"shadow-builtin.js","sourceRoot":"","sources":["../../src/checks/shadow-builtin.ts"],"names":[],"mappings":";;AAgBA,gDAWC;AA1BD,sCAA8B;AAC9B,oCAAqC;AAErC,MAAM,QAAQ,GAAG,IAAI,GAAG,CAAC;IACvB,KAAK,EAAE,KAAK,EAAE,MAAM,EAAE,MAAM,EAAE,QAAQ,EAAE,QAAQ,EAAE,QAAQ,EAAE,OAAO;IACnE,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,SAAS,EAAE,OAAO,EAAE,QAAQ,EAAE,IAAI,EAAE,MAAM;IACtE,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,OAAO,EAAE,OAAO;IACvE,OAAO,EAAE,QAAQ,EAAE,QAAQ,EAAE,SAAS,EAAE,WAAW,EAAE,UAAU,EAAE,QAAQ;IACzE,QAAQ,EAAE,SAAS,EAAE,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,UAAU;IAC1E,QAAQ,EAAE,QAAQ,EAAE,SAAS,EAAE,QAAQ,EAAE,SAAS,EAAE,QAAQ,EAAE,YAAY;IAC1E,WAAW,EAAE,SAAS,EAAE,QAAQ,EAAE,UAAU,EAAE,mBAAmB;IACjE,kBAAkB,EAAE,oBAAoB,EAAE,oBAAoB;IAC9D,sBAAsB,EAAE,aAAa,EAAE,2BAA2B;CACnE,CAAC,CAAA;AAEF,SAAgB,kBAAkB,CAAC,OAAe,EAAE,IAAY;IAC9D,MAAM,MAAM,GAAY,EAAE,CAAC;IAC3B,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAClC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACtC,MAAM,QAAQ,GAAG,IAAA,iBAAS,EAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QACrC,MAAM,CAAC,GAAG,QAAQ,CAAC,KAAK,CAAC,mBAAmB,CAAC,CAAC;QAC9C,IAAI,CAAC,IAAI,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC,EAAE,CAAC;YAC1C,MAAM,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,CAAC,GAAG,CAAC,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,gBAAgB,EAAE,OAAO,EAAE,IAAA,UAAC,EAAC,gBAAgB,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;QACnH,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"strcat-usage.d.ts","sourceRoot":"","sources":["../../src/checks/strcat-usage.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,UAAU,CAAC;AAIjC,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,KAAK,EAAE,CAgBvE"}
|
|
@@ -0,0 +1,22 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.checkStrcatUsage = checkStrcatUsage;
|
|
4
|
+
const locale_1 = require("../locale");
|
|
5
|
+
const utils_1 = require("../utils");
|
|
6
|
+
function checkStrcatUsage(content, file) {
|
|
7
|
+
const issues = [];
|
|
8
|
+
const lines = content.split('\n');
|
|
9
|
+
for (let i = 0; i < lines.length; i++) {
|
|
10
|
+
const stripped = (0, utils_1.stripLineRaw)(lines[i]);
|
|
11
|
+
// Check for (+ "str" ...) pattern
|
|
12
|
+
const m = stripped.match(/\(\+\s+"[^"]*"/);
|
|
13
|
+
if (m) {
|
|
14
|
+
issues.push({
|
|
15
|
+
file, line: i + 1, severity: 'warn', rule: 'strcat_usage',
|
|
16
|
+
message: (0, locale_1.t)('strcat_usage'),
|
|
17
|
+
});
|
|
18
|
+
}
|
|
19
|
+
}
|
|
20
|
+
return issues;
|
|
21
|
+
}
|
|
22
|
+
//# sourceMappingURL=strcat-usage.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"strcat-usage.js","sourceRoot":"","sources":["../../src/checks/strcat-usage.ts"],"names":[],"mappings":";;AAIA,4CAgBC;AAnBD,sCAA8B;AAC9B,oCAAwC;AAExC,SAAgB,gBAAgB,CAAC,OAAe,EAAE,IAAY;IAC5D,MAAM,MAAM,GAAY,EAAE,CAAC;IAC3B,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAElC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACtC,MAAM,QAAQ,GAAG,IAAA,oBAAY,EAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QACxC,kCAAkC;QAClC,MAAM,CAAC,GAAG,QAAQ,CAAC,KAAK,CAAC,gBAAgB,CAAC,CAAC;QAC3C,IAAI,CAAC,EAAE,CAAC;YACN,MAAM,CAAC,IAAI,CAAC;gBACV,IAAI,EAAE,IAAI,EAAE,CAAC,GAAG,CAAC,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,cAAc;gBACzD,OAAO,EAAE,IAAA,UAAC,EAAC,cAAc,CAAC;aAC3B,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"type-check.d.ts","sourceRoot":"","sources":["../../src/checks/type-check.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,UAAU,CAAC;AAIjC,wBAAgB,cAAc,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,KAAK,EAAE,CAkBrE"}
|
|
@@ -0,0 +1,26 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.checkTypeCheck = checkTypeCheck;
|
|
4
|
+
const locale_1 = require("../locale");
|
|
5
|
+
const utils_1 = require("../utils");
|
|
6
|
+
function checkTypeCheck(content, file) {
|
|
7
|
+
const issues = [];
|
|
8
|
+
const lines = content.split('\n');
|
|
9
|
+
for (let i = 0; i < lines.length; i++) {
|
|
10
|
+
const stripped = (0, utils_1.stripLine)(lines[i]);
|
|
11
|
+
const m = stripped.match(/\(setq\s+\S+\s+\(getvar\b/);
|
|
12
|
+
if (m) {
|
|
13
|
+
const varName = m[0].match(/\(setq\s+(\S+)/)?.[1] || '';
|
|
14
|
+
if (!varName)
|
|
15
|
+
continue;
|
|
16
|
+
const body = lines.slice(i, i + 5).join('\n');
|
|
17
|
+
const typeChk = '(type ' + varName;
|
|
18
|
+
const eqTypeChk = '(= (type ' + varName;
|
|
19
|
+
if (!body.includes(typeChk) && !body.includes(eqTypeChk)) {
|
|
20
|
+
issues.push({ file, line: i + 1, severity: 'warn', rule: 'type_check', message: (0, locale_1.t)('type_check', varName) });
|
|
21
|
+
}
|
|
22
|
+
}
|
|
23
|
+
}
|
|
24
|
+
return issues;
|
|
25
|
+
}
|
|
26
|
+
//# sourceMappingURL=type-check.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"type-check.js","sourceRoot":"","sources":["../../src/checks/type-check.ts"],"names":[],"mappings":";;AAIA,wCAkBC;AArBD,sCAA8B;AAC9B,oCAAqC;AAErC,SAAgB,cAAc,CAAC,OAAe,EAAE,IAAY;IAC1D,MAAM,MAAM,GAAY,EAAE,CAAC;IAC3B,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAClC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACtC,MAAM,QAAQ,GAAG,IAAA,iBAAS,EAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QACrC,MAAM,CAAC,GAAG,QAAQ,CAAC,KAAK,CAAC,2BAA2B,CAAC,CAAC;QACtD,IAAI,CAAC,EAAE,CAAC;YACN,MAAM,OAAO,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,gBAAgB,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;YACxD,IAAI,CAAC,OAAO;gBAAE,SAAS;YACvB,MAAM,IAAI,GAAG,KAAK,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAC9C,MAAM,OAAO,GAAG,QAAQ,GAAG,OAAO,CAAC;YACnC,MAAM,SAAS,GAAG,WAAW,GAAG,OAAO,CAAC;YACxC,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,SAAS,CAAC,EAAE,CAAC;gBACzD,MAAM,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,CAAC,GAAG,CAAC,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,YAAY,EAAE,OAAO,EAAE,IAAA,UAAC,EAAC,YAAY,EAAE,OAAO,CAAC,EAAE,CAAC,CAAC;YAC9G,CAAC;QACH,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"unused-variable.d.ts","sourceRoot":"","sources":["../../src/checks/unused-variable.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,UAAU,CAAC;
|
|
1
|
+
{"version":3,"file":"unused-variable.d.ts","sourceRoot":"","sources":["../../src/checks/unused-variable.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,UAAU,CAAC;AAOjC,wBAAgB,mBAAmB,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,KAAK,EAAE,CAqD1E"}
|
|
@@ -3,6 +3,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
|
|
|
3
3
|
exports.checkUnusedVariable = checkUnusedVariable;
|
|
4
4
|
const locale_1 = require("../locale");
|
|
5
5
|
const utils_1 = require("../utils");
|
|
6
|
+
// AutoLISP variables that are implicitly used by the runtime
|
|
7
|
+
const IMPLICITLY_USED = ['*error*', 't', 'nil'];
|
|
6
8
|
function checkUnusedVariable(content, file) {
|
|
7
9
|
const issues = [];
|
|
8
10
|
const lines = content.split('\n');
|
|
@@ -15,11 +17,11 @@ function checkUnusedVariable(content, file) {
|
|
|
15
17
|
const setqMatch = stripped.match(/\(setq\s+(\S+)/);
|
|
16
18
|
if (setqMatch) {
|
|
17
19
|
const v = setqMatch[1];
|
|
18
|
-
if (!setqVars.has(v) && v
|
|
20
|
+
if (!setqVars.has(v) && !IMPLICITLY_USED.includes(v.toLowerCase())) {
|
|
19
21
|
setqVars.set(v, i + 1);
|
|
20
22
|
}
|
|
21
23
|
}
|
|
22
|
-
//
|
|
24
|
+
// Track string state for usage detection
|
|
23
25
|
const strippedNoStr = lines[i].replace(/{[^}]*}/g, '');
|
|
24
26
|
for (let j = 0; j < strippedNoStr.length; j++) {
|
|
25
27
|
if (strippedNoStr[j] === '"')
|
|
@@ -28,9 +30,19 @@ function checkUnusedVariable(content, file) {
|
|
|
28
30
|
if (!inStr) {
|
|
29
31
|
for (const [v] of setqVars) {
|
|
30
32
|
if (v.length > 0) {
|
|
31
|
-
const
|
|
33
|
+
const escaped = v.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
|
|
34
|
+
// Check if name has only word characters (a-z, A-Z, 0-9, _)
|
|
35
|
+
const isSimpleName = /^[a-zA-Z0-9_]+$/.test(v);
|
|
36
|
+
let re;
|
|
37
|
+
if (isSimpleName) {
|
|
38
|
+
re = new RegExp('\\b' + escaped + '\\b');
|
|
39
|
+
}
|
|
40
|
+
else {
|
|
41
|
+
// For names with special chars (%, *), use safe boundary matching
|
|
42
|
+
re = new RegExp('(?:^|[^a-zA-Z0-9_])' + escaped + '(?:$|[^a-zA-Z0-9_])');
|
|
43
|
+
}
|
|
32
44
|
// Don't count the setq line itself
|
|
33
|
-
if (re.test(stripped) && !stripped.match(new RegExp('\\(setq\\s+' +
|
|
45
|
+
if (re.test(stripped) && !stripped.match(new RegExp('\\(setq\\s+' + escaped + '\\b'))) {
|
|
34
46
|
usedVars.add(v);
|
|
35
47
|
}
|
|
36
48
|
}
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"unused-variable.js","sourceRoot":"","sources":["../../src/checks/unused-variable.ts"],"names":[],"mappings":";;
|
|
1
|
+
{"version":3,"file":"unused-variable.js","sourceRoot":"","sources":["../../src/checks/unused-variable.ts"],"names":[],"mappings":";;AAOA,kDAqDC;AA3DD,sCAA8B;AAC9B,oCAAqC;AAErC,6DAA6D;AAC7D,MAAM,eAAe,GAAG,CAAC,SAAS,EAAE,GAAG,EAAE,KAAK,CAAC,CAAA;AAE/C,SAAgB,mBAAmB,CAAC,OAAe,EAAE,IAAY;IAC/D,MAAM,MAAM,GAAY,EAAE,CAAC;IAC3B,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAClC,MAAM,QAAQ,GAAwB,IAAI,GAAG,EAAE,CAAC;IAChD,MAAM,QAAQ,GAAgB,IAAI,GAAG,EAAE,CAAC;IACxC,IAAI,KAAK,GAAG,KAAK,CAAC;IAElB,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACtC,MAAM,QAAQ,GAAG,IAAA,iBAAS,EAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QACrC,sBAAsB;QACtB,MAAM,SAAS,GAAG,QAAQ,CAAC,KAAK,CAAC,gBAAgB,CAAC,CAAC;QACnD,IAAI,SAAS,EAAE,CAAC;YACd,MAAM,CAAC,GAAG,SAAS,CAAC,CAAC,CAAC,CAAC;YACvB,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,eAAe,CAAC,QAAQ,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC,EAAE,CAAC;gBACnE,QAAQ,CAAC,GAAG,CAAC,CAAC,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;YACzB,CAAC;QACH,CAAC;QACD,yCAAyC;QACzC,MAAM,aAAa,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,UAAU,EAAE,EAAE,CAAC,CAAC;QACvD,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,aAAa,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;YAC9C,IAAI,aAAa,CAAC,CAAC,CAAC,KAAK,GAAG;gBAAE,KAAK,GAAG,CAAC,KAAK,CAAC;QAC/C,CAAC;QACD,IAAI,CAAC,KAAK,EAAE,CAAC;YACX,KAAK,MAAM,CAAC,CAAC,CAAC,IAAI,QAAQ,EAAE,CAAC;gBAC3B,IAAI,CAAC,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;oBACjB,MAAM,OAAO,GAAG,CAAC,CAAC,OAAO,CAAC,qBAAqB,EAAE,MAAM,CAAC,CAAC;oBACzD,4DAA4D;oBAC5D,MAAM,YAAY,GAAG,iBAAiB,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;oBAC/C,IAAI,EAAU,CAAC;oBACf,IAAI,YAAY,EAAE,CAAC;wBACjB,EAAE,GAAG,IAAI,MAAM,CAAC,KAAK,GAAG,OAAO,GAAG,KAAK,CAAC,CAAC;oBAC3C,CAAC;yBAAM,CAAC;wBACN,kEAAkE;wBAClE,EAAE,GAAG,IAAI,MAAM,CAAC,qBAAqB,GAAG,OAAO,GAAG,qBAAqB,CAAC,CAAC;oBAC3E,CAAC;oBACD,mCAAmC;oBACnC,IAAI,EAAE,CAAC,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC,IAAI,MAAM,CAAC,aAAa,GAAG,OAAO,GAAG,KAAK,CAAC,CAAC,EAAE,CAAC;wBACtF,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC;oBAClB,CAAC;gBACH,CAAC;YACH,CAAC;QACH,CAAC;IACH,CAAC;IAED,KAAK,MAAM,CAAC,CAAC,EAAE,IAAI,CAAC,IAAI,QAAQ,EAAE,CAAC;QACjC,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;YAC3C,MAAM,CAAC,IAAI,CAAC;gBACV,IAAI,EAAE,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,iBAAiB;gBACrD,OAAO,EAAE,IAAA,UAAC,EAAC,iBAAiB,EAAE,CAAC,CAAC;aACjC,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
package/dist/config.d.ts.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"config.d.ts","sourceRoot":"","sources":["../src/config.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAwB,MAAM,SAAS,CAAC;
|
|
1
|
+
{"version":3,"file":"config.d.ts","sourceRoot":"","sources":["../src/config.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAwB,MAAM,SAAS,CAAC;AA6FrE,wBAAgB,UAAU,CAAC,UAAU,CAAC,EAAE,MAAM,GAAG,UAAU,CAe1D;AAmBD,wBAAgB,gBAAgB,CAAC,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,GAAG,QAAQ,CAIhF;AAED,wBAAgB,aAAa,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,UAAU,CAAC,GAAG,IAAI,CAQ1E;AAED,wBAAgB,cAAc,CAAC,IAAI,EAAE,UAAU,EAAE,SAAS,EAAE,OAAO,CAAC,UAAU,CAAC,GAAG,UAAU,CAI3F"}
|
package/dist/config.js
CHANGED
|
@@ -47,22 +47,47 @@ const DEFAULT_CONFIG = {
|
|
|
47
47
|
},
|
|
48
48
|
checks: {
|
|
49
49
|
parens: 'error',
|
|
50
|
-
encoding: '
|
|
50
|
+
encoding: 'warn',
|
|
51
51
|
cl_syntax: 'warn',
|
|
52
52
|
quit_exit: 'error',
|
|
53
53
|
command_shell: 'error',
|
|
54
54
|
startapp: 'warn',
|
|
55
55
|
vl_registry_write: 'warn',
|
|
56
|
-
vlax_without_loading: '
|
|
56
|
+
vlax_without_loading: 'off',
|
|
57
57
|
token_in_url: 'warn',
|
|
58
58
|
open_without_close: 'warn',
|
|
59
|
-
bare_function_names: '
|
|
59
|
+
bare_function_names: 'off',
|
|
60
60
|
trailing_whitespace: 'warn',
|
|
61
61
|
line_length: 'warn',
|
|
62
62
|
function_complexity: 'warn',
|
|
63
63
|
parameter_naming: 'warn',
|
|
64
64
|
unused_variable: 'warn',
|
|
65
65
|
missing_doc: 'warn',
|
|
66
|
+
error_handling: 'warn',
|
|
67
|
+
global_naming: 'warn',
|
|
68
|
+
extra_parens: 'warn',
|
|
69
|
+
arg_count: 'warn',
|
|
70
|
+
strcat_usage: 'warn',
|
|
71
|
+
cond_simplify: 'warn',
|
|
72
|
+
redundant_progn: 'warn',
|
|
73
|
+
quote_style: 'warn',
|
|
74
|
+
eq_usage: 'warn',
|
|
75
|
+
lambda_syntax: 'warn',
|
|
76
|
+
comment_style: 'warn',
|
|
77
|
+
empty_catch: 'warn',
|
|
78
|
+
nth_usage: 'warn',
|
|
79
|
+
append_single: 'warn',
|
|
80
|
+
setq_multiple: 'warn',
|
|
81
|
+
function_order: 'off',
|
|
82
|
+
magic_number: 'off',
|
|
83
|
+
mixed_indent: 'warn',
|
|
84
|
+
long_function_call: 'warn',
|
|
85
|
+
no_return: 'warn',
|
|
86
|
+
shadow_builtin: 'warn',
|
|
87
|
+
dynamic_doc: 'warn',
|
|
88
|
+
loop_optimization: 'off',
|
|
89
|
+
type_check: 'off',
|
|
90
|
+
redundant_if: 'warn',
|
|
66
91
|
module_registration: 'off',
|
|
67
92
|
namespace_header: 'off',
|
|
68
93
|
},
|
package/dist/config.js.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"config.js","sourceRoot":"","sources":["../src/config.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
|
|
1
|
+
{"version":3,"file":"config.js","sourceRoot":"","sources":["../src/config.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AA+FA,gCAeC;AAmBD,4CAIC;AAED,sCAQC;AAED,wCAIC;AArJD,uCAAyB;AACzB,2CAA6B;AAG7B,MAAM,cAAc,GAAe;IACjC,MAAM,EAAE,IAAI;IACZ,MAAM,EAAE;QACN,KAAK,EAAE,CAAC,UAAU,CAAC;QACnB,OAAO,EAAE,CAAC,oBAAoB,EAAE,cAAc,EAAE,YAAY,CAAC;KAC9D;IACD,MAAM,EAAE;QACN,MAAM,EAAE,OAAO;QACf,QAAQ,EAAE,MAAM;QAChB,SAAS,EAAE,MAAM;QACjB,SAAS,EAAE,OAAO;QAClB,aAAa,EAAE,OAAO;QACtB,QAAQ,EAAE,MAAM;QAChB,iBAAiB,EAAE,MAAM;QACzB,oBAAoB,EAAE,KAAK;QAC3B,YAAY,EAAE,MAAM;QACpB,kBAAkB,EAAE,MAAM;QAC1B,mBAAmB,EAAE,KAAK;QAC1B,mBAAmB,EAAE,MAAM;QAC3B,WAAW,EAAE,MAAM;QACnB,mBAAmB,EAAE,MAAM;QAC3B,gBAAgB,EAAE,MAAM;QACxB,eAAe,EAAE,MAAM;QACvB,WAAW,EAAE,MAAM;QACnB,cAAc,EAAE,MAAM;QACtB,aAAa,EAAE,MAAM;QACrB,YAAY,EAAE,MAAM;QACpB,SAAS,EAAE,MAAM;QACjB,YAAY,EAAE,MAAM;QACpB,aAAa,EAAE,MAAM;QACrB,eAAe,EAAE,MAAM;QACvB,WAAW,EAAE,MAAM;QACnB,QAAQ,EAAE,MAAM;QAChB,aAAa,EAAE,MAAM;QACrB,aAAa,EAAE,MAAM;QACrB,WAAW,EAAE,MAAM;QACnB,SAAS,EAAE,MAAM;QACjB,aAAa,EAAE,MAAM;QACrB,aAAa,EAAE,MAAM;QACrB,cAAc,EAAE,KAAK;QACrB,YAAY,EAAE,KAAK;QACnB,YAAY,EAAE,MAAM;QACpB,kBAAkB,EAAE,MAAM;QAC1B,SAAS,EAAE,MAAM;QACjB,cAAc,EAAE,MAAM;QACtB,WAAW,EAAE,MAAM;QACnB,iBAAiB,EAAE,KAAK;QACxB,UAAU,EAAE,KAAK;QACjB,YAAY,EAAE,MAAM;QACpB,mBAAmB,EAAE,KAAK;QAC1B,gBAAgB,EAAE,KAAK;KACxB;IACD,WAAW,EAAE;QACX,GAAG,EAAE,GAAG;QACR,SAAS,EAAE,CAAC;KACb;IACD,mBAAmB,EAAE;QACnB,SAAS,EAAE,EAAE;QACb,WAAW,EAAE,CAAC;KACf;IACD,SAAS,EAAE;QACT,QAAQ,EAAE,CAAC,MAAM,CAAC;KACnB;IACD,eAAe,EAAE;QACf,IAAI,EAAE,OAAO;QACb,IAAI,EAAE,OAAO;QACb,aAAa,EAAE,OAAO;QACtB,QAAQ,EAAE,MAAM;QAChB,iBAAiB,EAAE,MAAM;KAC1B;IACD,mBAAmB,EAAE;QACnB,QAAQ,EAAE,KAAK;QACf,QAAQ,EAAE,EAAE;QACZ,IAAI,EAAE,CAAC,SAAS,CAAC;KAClB;IACD,gBAAgB,EAAE;QAChB,QAAQ,EAAE,KAAK;QACf,OAAO,EAAE,EAAE;QACX,IAAI,EAAE,EAAE;QACR,YAAY,EAAE,EAAE;KACjB;IACD,mBAAmB,EAAE;QACnB,SAAS,EAAE,CAAC,GAAG,EAAE,KAAK,CAAC;QACvB,iBAAiB,EAAE,UAAU;KAC9B;IACD,IAAI,EAAE;QACJ,YAAY,EAAE,CAAC,SAAS,EAAE,QAAQ,EAAE,MAAM,CAAC;QAC3C,oBAAoB,EAAE,CAAC,WAAW,CAAC;KACpC;CACF,CAAC;AAEF,SAAgB,UAAU,CAAC,UAAmB;IAC5C,IAAI,UAAU,EAAE,CAAC;QACf,MAAM,OAAO,GAAG,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC;QACzC,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,OAAO,CAAC,EAAE,CAAC;YAC5B,MAAM,IAAI,KAAK,CAAC,0BAA0B,OAAO,EAAE,CAAC,CAAC;QACvD,CAAC;QACD,MAAM,GAAG,GAAG,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,OAAO,CAAC,CAAC;QAC9C,MAAM,IAAI,GAAG,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;QAC7B,sDAAsD;QACtD,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,cAAc,CAAC,CAAC,CAAC;QAC1D,SAAS,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;QACxB,OAAO,MAAoB,CAAC;IAC9B,CAAC;IAED,OAAO,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,cAAc,CAAC,CAAe,CAAC;AAClE,CAAC;AAED,8DAA8D;AAC9D,SAAS,SAAS,CAAC,MAAW,EAAE,MAAW;IACzC,KAAK,MAAM,GAAG,IAAI,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC;QACtC,MAAM,GAAG,GAAG,MAAM,CAAC,GAAG,CAAC,CAAC;QACxB,IAAI,GAAG,KAAK,IAAI,IAAI,OAAO,GAAG,KAAK,QAAQ,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC;YACnE,IAAI,CAAC,MAAM,CAAC,GAAG,CAAC,IAAI,OAAO,MAAM,CAAC,GAAG,CAAC,KAAK,QAAQ,EAAE,CAAC;gBACpD,MAAM,CAAC,GAAG,CAAC,GAAG,EAAE,CAAC;YACnB,CAAC;YACD,SAAS,CAAC,MAAM,CAAC,GAAG,CAAC,EAAE,GAAG,CAAC,CAAC;QAC9B,CAAC;aAAM,CAAC;YACN,MAAM,CAAC,GAAG,CAAC,GAAG,GAAG,CAAC;QACpB,CAAC;IACH,CAAC;AACH,CAAC;AAID,SAAgB,gBAAgB,CAAC,MAAkB,EAAE,SAAiB;IACpE,MAAM,GAAG,GAAI,MAAM,CAAC,MAAiC,CAAC,SAAS,CAAC,CAAC;IACjE,IAAI,GAAG,KAAK,OAAO,IAAI,GAAG,KAAK,MAAM;QAAE,OAAO,GAAG,CAAC;IAClD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,SAAgB,aAAa,CAAC,QAAgB;IAC5C,MAAM,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;IACnC,MAAM,UAAU,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,mBAAmB,CAAC,CAAC;IACvD,IAAI,EAAE,CAAC,UAAU,CAAC,UAAU,CAAC,EAAE,CAAC;QAC9B,MAAM,GAAG,GAAG,EAAE,CAAC,YAAY,CAAC,UAAU,EAAE,OAAO,CAAC,CAAC;QACjD,OAAO,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;IACzB,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAED,SAAgB,cAAc,CAAC,IAAgB,EAAE,SAA8B;IAC7E,MAAM,MAAM,GAAe,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC;IAC5D,SAAS,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC;IAC7B,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
package/dist/formatters.d.ts
CHANGED
|
@@ -1,4 +1,5 @@
|
|
|
1
1
|
import { Issue } from './types';
|
|
2
2
|
export declare function formatDefault(issues: Issue[], fileContents?: Map<string, string>): string;
|
|
3
3
|
export declare function formatJson(issues: Issue[]): string;
|
|
4
|
+
export declare function formatHtml(issues: Issue[], totalErrors: number, totalWarnings: number): string;
|
|
4
5
|
//# sourceMappingURL=formatters.d.ts.map
|
package/dist/formatters.d.ts.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"formatters.d.ts","sourceRoot":"","sources":["../src/formatters.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,SAAS,CAAC;AAehC,wBAAgB,aAAa,CAAC,MAAM,EAAE,KAAK,EAAE,EAAE,YAAY,CAAC,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,MAAM,CAyDzF;AAED,wBAAgB,UAAU,CAAC,MAAM,EAAE,KAAK,EAAE,GAAG,MAAM,CAElD"}
|
|
1
|
+
{"version":3,"file":"formatters.d.ts","sourceRoot":"","sources":["../src/formatters.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,SAAS,CAAC;AAehC,wBAAgB,aAAa,CAAC,MAAM,EAAE,KAAK,EAAE,EAAE,YAAY,CAAC,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,MAAM,CAyDzF;AAED,wBAAgB,UAAU,CAAC,MAAM,EAAE,KAAK,EAAE,GAAG,MAAM,CAElD;AAED,wBAAgB,UAAU,CAAC,MAAM,EAAE,KAAK,EAAE,EAAE,WAAW,EAAE,MAAM,EAAE,aAAa,EAAE,MAAM,GAAG,MAAM,CAsD9F"}
|
package/dist/formatters.js
CHANGED
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
3
|
exports.formatDefault = formatDefault;
|
|
4
4
|
exports.formatJson = formatJson;
|
|
5
|
+
exports.formatHtml = formatHtml;
|
|
5
6
|
const locale_1 = require("./locale");
|
|
6
7
|
const colors = {
|
|
7
8
|
red: '\x1b[31m',
|
|
@@ -73,4 +74,58 @@ function formatDefault(issues, fileContents) {
|
|
|
73
74
|
function formatJson(issues) {
|
|
74
75
|
return JSON.stringify({ issues }, null, 2);
|
|
75
76
|
}
|
|
77
|
+
function formatHtml(issues, totalErrors, totalWarnings) {
|
|
78
|
+
const rows = issues.map(iss => {
|
|
79
|
+
const sevClass = iss.severity === 'error' ? 'error' : 'warn';
|
|
80
|
+
const loc = iss.line > 0 ? `Line ${iss.line}` : '';
|
|
81
|
+
return `<tr class="${sevClass}">
|
|
82
|
+
<td><span class="badge badge-${sevClass}">${sevClass === 'error' ? 'FAIL' : 'WARN'}</span></td>
|
|
83
|
+
<td>${iss.file}</td>
|
|
84
|
+
<td>${loc}</td>
|
|
85
|
+
<td>${iss.rule}</td>
|
|
86
|
+
<td>${iss.message}</td>
|
|
87
|
+
</tr>`;
|
|
88
|
+
}).join('\n');
|
|
89
|
+
return `<!DOCTYPE html>
|
|
90
|
+
<html lang="zh">
|
|
91
|
+
<head>
|
|
92
|
+
<meta charset="UTF-8">
|
|
93
|
+
<meta name="viewport" content="width=device-width, initial-scale=1.0">
|
|
94
|
+
<title>@atlisp/lint Report</title>
|
|
95
|
+
<style>
|
|
96
|
+
body { font-family: -apple-system, BlinkMacSystemFont, sans-serif; padding: 20px; background: #f5f5f5; }
|
|
97
|
+
h1 { color: #333; }
|
|
98
|
+
.summary { display: flex; gap: 20px; margin: 20px 0; }
|
|
99
|
+
.stat { background: #fff; padding: 15px 25px; border-radius: 8px; box-shadow: 0 1px 3px rgba(0,0,0,0.1); }
|
|
100
|
+
.stat-errors { border-left: 4px solid #e74c3c; }
|
|
101
|
+
.stat-warnings { border-left: 4px solid #f39c12; }
|
|
102
|
+
.stat-ok { border-left: 4px solid #27ae60; }
|
|
103
|
+
.stat-value { font-size: 28px; font-weight: bold; }
|
|
104
|
+
.stat-label { color: #666; font-size: 12px; text-transform: uppercase; }
|
|
105
|
+
table { width: 100%; border-collapse: collapse; background: #fff; border-radius: 8px; overflow: hidden; box-shadow: 0 1px 3px rgba(0,0,0,0.1); }
|
|
106
|
+
th { background: #333; color: #fff; padding: 12px; text-align: left; }
|
|
107
|
+
td { padding: 10px 12px; border-bottom: 1px solid #eee; }
|
|
108
|
+
tr.error td { background: #fef0ef; }
|
|
109
|
+
tr.warn td { background: #fef9e7; }
|
|
110
|
+
.badge { padding: 2px 8px; border-radius: 4px; font-size: 12px; font-weight: bold; color: #fff; }
|
|
111
|
+
.badge-error { background: #e74c3c; }
|
|
112
|
+
.badge-warn { background: #f39c12; }
|
|
113
|
+
.footer { margin-top: 20px; color: #999; font-size: 12px; text-align: center; }
|
|
114
|
+
</style>
|
|
115
|
+
</head>
|
|
116
|
+
<body>
|
|
117
|
+
<h1>@atlisp/lint Report</h1>
|
|
118
|
+
<div class="summary">
|
|
119
|
+
<div class="stat stat-errors"><div class="stat-value">${totalErrors}</div><div class="stat-label">Errors</div></div>
|
|
120
|
+
<div class="stat stat-warnings"><div class="stat-value">${totalWarnings}</div><div class="stat-label">Warnings</div></div>
|
|
121
|
+
<div class="stat stat-ok"><div class="stat-value">${issues.length}</div><div class="stat-label">Total Issues</div></div>
|
|
122
|
+
</div>
|
|
123
|
+
<table>
|
|
124
|
+
<thead><tr><th>Severity</th><th>File</th><th>Line</th><th>Rule</th><th>Message</th></tr></thead>
|
|
125
|
+
<tbody>${rows.length ? rows : '<tr><td colspan="5" style="text-align:center;color:#999;">No issues found</td></tr>'}</tbody>
|
|
126
|
+
</table>
|
|
127
|
+
<div class="footer">Generated by @atlisp/lint</div>
|
|
128
|
+
</body>
|
|
129
|
+
</html>`;
|
|
130
|
+
}
|
|
76
131
|
//# sourceMappingURL=formatters.js.map
|