@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
package/LICENSE
ADDED
|
@@ -0,0 +1,21 @@
|
|
|
1
|
+
MIT License
|
|
2
|
+
|
|
3
|
+
Copyright (c) 2024 VitalGG
|
|
4
|
+
|
|
5
|
+
Permission is hereby granted, free of charge, to any person obtaining a copy
|
|
6
|
+
of this software and associated documentation files (the "Software"), to deal
|
|
7
|
+
in the Software without restriction, including without limitation the rights
|
|
8
|
+
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
|
|
9
|
+
copies of the Software, and to permit persons to whom the Software is
|
|
10
|
+
furnished to do so, subject to the following conditions:
|
|
11
|
+
|
|
12
|
+
The above copyright notice and this permission notice shall be included in all
|
|
13
|
+
copies or substantial portions of the Software.
|
|
14
|
+
|
|
15
|
+
THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
16
|
+
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
17
|
+
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
|
|
18
|
+
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
|
19
|
+
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
|
|
20
|
+
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
|
|
21
|
+
SOFTWARE.
|
package/README.md
CHANGED
|
@@ -34,7 +34,7 @@ npx @atlisp/lint --fix
|
|
|
34
34
|
--config <path> 指定配置文件路径
|
|
35
35
|
--staged 仅检查 git 暂存区中的 .lsp 文件
|
|
36
36
|
--format <format> 输出格式:default(默认)| json
|
|
37
|
-
--init 在当前目录生成 atlisp-lint.json
|
|
37
|
+
--init 在当前目录生成 .atlisp-lint.json
|
|
38
38
|
--install-hook 安装 git pre-commit hook
|
|
39
39
|
--fix 自动修复尾部空格、UTF-8 BOM 等问题
|
|
40
40
|
--hook-args <args> 自定义 pre-commit hook 参数(配合 --install-hook)
|
|
@@ -85,7 +85,7 @@ npx @atlisp/lint --fix
|
|
|
85
85
|
|
|
86
86
|
## 配置文件
|
|
87
87
|
|
|
88
|
-
|
|
88
|
+
`.atlisp-lint.json`:
|
|
89
89
|
|
|
90
90
|
```json
|
|
91
91
|
{
|
package/atlisp-lint.default.json
CHANGED
|
@@ -6,21 +6,46 @@
|
|
|
6
6
|
},
|
|
7
7
|
"checks": {
|
|
8
8
|
"parens": "error",
|
|
9
|
-
"encoding": "
|
|
9
|
+
"encoding": "warn",
|
|
10
10
|
"cl_syntax": "warn",
|
|
11
11
|
"quit_exit": "error",
|
|
12
12
|
"command_shell": "error",
|
|
13
13
|
"startapp": "warn",
|
|
14
14
|
"vl_registry_write": "warn",
|
|
15
|
-
"vlax_without_loading": "
|
|
15
|
+
"vlax_without_loading": "off",
|
|
16
16
|
"token_in_url": "warn",
|
|
17
17
|
"open_without_close": "warn",
|
|
18
|
-
"bare_function_names": "
|
|
18
|
+
"bare_function_names": "off",
|
|
19
19
|
"line_length": "warn",
|
|
20
20
|
"function_complexity": "warn",
|
|
21
21
|
"parameter_naming": "warn",
|
|
22
22
|
"unused_variable": "warn",
|
|
23
23
|
"missing_doc": "warn",
|
|
24
|
+
"error_handling": "warn",
|
|
25
|
+
"global_naming": "warn",
|
|
26
|
+
"extra_parens": "warn",
|
|
27
|
+
"arg_count": "warn",
|
|
28
|
+
"strcat_usage": "warn",
|
|
29
|
+
"cond_simplify": "warn",
|
|
30
|
+
"redundant_progn": "warn",
|
|
31
|
+
"quote_style": "warn",
|
|
32
|
+
"eq_usage": "warn",
|
|
33
|
+
"lambda_syntax": "warn",
|
|
34
|
+
"comment_style": "warn",
|
|
35
|
+
"empty_catch": "warn",
|
|
36
|
+
"nth_usage": "warn",
|
|
37
|
+
"append_single": "warn",
|
|
38
|
+
"setq_multiple": "warn",
|
|
39
|
+
"function_order": "off",
|
|
40
|
+
"magic_number": "off",
|
|
41
|
+
"mixed_indent": "warn",
|
|
42
|
+
"long_function_call": "warn",
|
|
43
|
+
"no_return": "warn",
|
|
44
|
+
"shadow_builtin": "warn",
|
|
45
|
+
"dynamic_doc": "warn",
|
|
46
|
+
"loop_optimization": "off",
|
|
47
|
+
"type_check": "off",
|
|
48
|
+
"redundant_if": "warn",
|
|
24
49
|
"trailing_whitespace": "warn",
|
|
25
50
|
"module_registration": "off",
|
|
26
51
|
"namespace_header": "off"
|
package/bin/atlisp-lint
CHANGED
|
@@ -1,2 +1,2 @@
|
|
|
1
1
|
#!/usr/bin/env node
|
|
2
|
-
require('../dist/index.js').main();
|
|
2
|
+
require('../dist/index.js').main().catch(e => { console.error(e); process.exit(1); });
|
|
@@ -6,21 +6,46 @@
|
|
|
6
6
|
},
|
|
7
7
|
"checks": {
|
|
8
8
|
"parens": "error",
|
|
9
|
-
"encoding": "
|
|
9
|
+
"encoding": "warn",
|
|
10
10
|
"cl_syntax": "warn",
|
|
11
11
|
"quit_exit": "error",
|
|
12
12
|
"command_shell": "error",
|
|
13
13
|
"startapp": "warn",
|
|
14
14
|
"vl_registry_write": "warn",
|
|
15
|
-
"vlax_without_loading": "
|
|
15
|
+
"vlax_without_loading": "off",
|
|
16
16
|
"token_in_url": "warn",
|
|
17
17
|
"open_without_close": "warn",
|
|
18
|
-
"bare_function_names": "
|
|
18
|
+
"bare_function_names": "off",
|
|
19
19
|
"line_length": "warn",
|
|
20
20
|
"function_complexity": "warn",
|
|
21
21
|
"parameter_naming": "warn",
|
|
22
22
|
"unused_variable": "warn",
|
|
23
23
|
"missing_doc": "warn",
|
|
24
|
+
"error_handling": "warn",
|
|
25
|
+
"global_naming": "warn",
|
|
26
|
+
"extra_parens": "warn",
|
|
27
|
+
"arg_count": "warn",
|
|
28
|
+
"strcat_usage": "warn",
|
|
29
|
+
"cond_simplify": "warn",
|
|
30
|
+
"redundant_progn": "warn",
|
|
31
|
+
"quote_style": "warn",
|
|
32
|
+
"eq_usage": "warn",
|
|
33
|
+
"lambda_syntax": "warn",
|
|
34
|
+
"comment_style": "warn",
|
|
35
|
+
"empty_catch": "warn",
|
|
36
|
+
"nth_usage": "warn",
|
|
37
|
+
"append_single": "warn",
|
|
38
|
+
"setq_multiple": "warn",
|
|
39
|
+
"function_order": "off",
|
|
40
|
+
"magic_number": "off",
|
|
41
|
+
"mixed_indent": "warn",
|
|
42
|
+
"long_function_call": "warn",
|
|
43
|
+
"no_return": "warn",
|
|
44
|
+
"shadow_builtin": "warn",
|
|
45
|
+
"dynamic_doc": "warn",
|
|
46
|
+
"loop_optimization": "off",
|
|
47
|
+
"type_check": "off",
|
|
48
|
+
"redundant_if": "warn",
|
|
24
49
|
"trailing_whitespace": "warn",
|
|
25
50
|
"module_registration": "off",
|
|
26
51
|
"namespace_header": "off"
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"append-single.d.ts","sourceRoot":"","sources":["../../src/checks/append-single.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.checkAppendSingle = checkAppendSingle;
|
|
4
|
+
const locale_1 = require("../locale");
|
|
5
|
+
const utils_1 = require("../utils");
|
|
6
|
+
function checkAppendSingle(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 (/\(append\s+\(list\b/.test(stripped)) {
|
|
12
|
+
issues.push({ file, line: i + 1, severity: 'warn', rule: 'append_single', message: (0, locale_1.t)('append_single') });
|
|
13
|
+
}
|
|
14
|
+
}
|
|
15
|
+
return issues;
|
|
16
|
+
}
|
|
17
|
+
//# sourceMappingURL=append-single.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"append-single.js","sourceRoot":"","sources":["../../src/checks/append-single.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,qBAAqB,CAAC,IAAI,CAAC,QAAQ,CAAC,EAAE,CAAC;YACzC,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":"arg-count.d.ts","sourceRoot":"","sources":["../../src/checks/arg-count.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,UAAU,CAAC;AAUjC,wBAAgB,aAAa,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,KAAK,EAAE,CAoFpE"}
|
|
@@ -0,0 +1,120 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.checkArgCount = checkArgCount;
|
|
4
|
+
const locale_1 = require("../locale");
|
|
5
|
+
const utils_1 = require("../utils");
|
|
6
|
+
function checkArgCount(content, file) {
|
|
7
|
+
const issues = [];
|
|
8
|
+
const lines = content.split('\n');
|
|
9
|
+
const defuns = [];
|
|
10
|
+
// Collect all defun definitions
|
|
11
|
+
for (let i = 0; i < lines.length; i++) {
|
|
12
|
+
const stripped = (0, utils_1.stripLine)(lines[i]);
|
|
13
|
+
const m = stripped.match(/\(defun\s+(\S+)\s+\(([^)]*)\)/);
|
|
14
|
+
if (m) {
|
|
15
|
+
const all = m[2];
|
|
16
|
+
const slashIdx = all.indexOf('/');
|
|
17
|
+
const paramStr = slashIdx >= 0 ? all.slice(0, slashIdx).trim() : all;
|
|
18
|
+
const params = paramStr.split(/\s+/).filter(Boolean);
|
|
19
|
+
defuns.push({ name: m[1], line: i + 1, paramCount: params.length });
|
|
20
|
+
}
|
|
21
|
+
}
|
|
22
|
+
// Check calls against definitions
|
|
23
|
+
for (const fn of defuns) {
|
|
24
|
+
if (fn.name.startsWith('c:') || fn.name.includes('/'))
|
|
25
|
+
continue;
|
|
26
|
+
if (fn.paramCount === 0)
|
|
27
|
+
continue;
|
|
28
|
+
for (let i = 0; i < lines.length; i++) {
|
|
29
|
+
const stripped = (0, utils_1.stripLine)(lines[i]);
|
|
30
|
+
const re = new RegExp('\\(\\s*' + fn.name.replace(/[.*+?^${}()|[\]\\]/g, '\\$&') + '\\s+');
|
|
31
|
+
// Don't match the definition line
|
|
32
|
+
if (stripped.includes('defun') && stripped.includes(fn.name))
|
|
33
|
+
continue;
|
|
34
|
+
const callMatch = re.exec(stripped);
|
|
35
|
+
if (callMatch) {
|
|
36
|
+
// Collect all text from the call across lines
|
|
37
|
+
let callText = stripped.slice(callMatch.index);
|
|
38
|
+
let parenDepth = 0;
|
|
39
|
+
for (const ch of callText) {
|
|
40
|
+
if (ch === '(')
|
|
41
|
+
parenDepth++;
|
|
42
|
+
else if (ch === ')')
|
|
43
|
+
parenDepth--;
|
|
44
|
+
}
|
|
45
|
+
// If not closed on this line, read next lines
|
|
46
|
+
let lineIdx = i + 1;
|
|
47
|
+
while (parenDepth > 0 && lineIdx < lines.length) {
|
|
48
|
+
const nextStripped = (0, utils_1.stripLine)(lines[lineIdx]);
|
|
49
|
+
callText += ' ' + nextStripped;
|
|
50
|
+
for (const ch of nextStripped) {
|
|
51
|
+
if (ch === '(')
|
|
52
|
+
parenDepth++;
|
|
53
|
+
else if (ch === ')')
|
|
54
|
+
parenDepth--;
|
|
55
|
+
}
|
|
56
|
+
lineIdx++;
|
|
57
|
+
}
|
|
58
|
+
// Extract args: remove the function call prefix
|
|
59
|
+
const afterName = callText.slice(callMatch[0].length - 1); // keep the leading (
|
|
60
|
+
// Find matching closing paren
|
|
61
|
+
let closeIdx = -1;
|
|
62
|
+
let pd = 1;
|
|
63
|
+
for (let j = 0; j < afterName.length; j++) {
|
|
64
|
+
if (afterName[j] === '(')
|
|
65
|
+
pd++;
|
|
66
|
+
else if (afterName[j] === ')') {
|
|
67
|
+
pd--;
|
|
68
|
+
if (pd === 0) {
|
|
69
|
+
closeIdx = j;
|
|
70
|
+
break;
|
|
71
|
+
}
|
|
72
|
+
}
|
|
73
|
+
}
|
|
74
|
+
const argsStr = closeIdx >= 0 ? afterName.slice(0, closeIdx).trim() : afterName.trim();
|
|
75
|
+
if (!argsStr)
|
|
76
|
+
continue;
|
|
77
|
+
// Count space-delimited args at depth 0
|
|
78
|
+
let argCount = 0;
|
|
79
|
+
let depth = 0;
|
|
80
|
+
let inArg = false;
|
|
81
|
+
for (const ch of argsStr) {
|
|
82
|
+
if (ch === '(') {
|
|
83
|
+
depth++;
|
|
84
|
+
if (!inArg) {
|
|
85
|
+
inArg = true;
|
|
86
|
+
}
|
|
87
|
+
}
|
|
88
|
+
else if (ch === ')') {
|
|
89
|
+
depth--;
|
|
90
|
+
if (depth === 0) {
|
|
91
|
+
if (inArg) {
|
|
92
|
+
argCount++;
|
|
93
|
+
inArg = false;
|
|
94
|
+
}
|
|
95
|
+
}
|
|
96
|
+
}
|
|
97
|
+
else if (depth === 0 && (ch === ' ' || ch === '\t')) {
|
|
98
|
+
if (inArg) {
|
|
99
|
+
argCount++;
|
|
100
|
+
inArg = false;
|
|
101
|
+
}
|
|
102
|
+
}
|
|
103
|
+
else if (!inArg) {
|
|
104
|
+
inArg = true;
|
|
105
|
+
}
|
|
106
|
+
}
|
|
107
|
+
if (inArg)
|
|
108
|
+
argCount++;
|
|
109
|
+
if (argCount !== fn.paramCount) {
|
|
110
|
+
issues.push({
|
|
111
|
+
file, line: i + 1, severity: 'warn', rule: 'arg_count',
|
|
112
|
+
message: (0, locale_1.t)('arg_count', fn.name, argCount, fn.paramCount),
|
|
113
|
+
});
|
|
114
|
+
}
|
|
115
|
+
}
|
|
116
|
+
}
|
|
117
|
+
}
|
|
118
|
+
return issues;
|
|
119
|
+
}
|
|
120
|
+
//# sourceMappingURL=arg-count.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"arg-count.js","sourceRoot":"","sources":["../../src/checks/arg-count.ts"],"names":[],"mappings":";;AAUA,sCAoFC;AA7FD,sCAA8B;AAC9B,oCAAqC;AAQrC,SAAgB,aAAa,CAAC,OAAe,EAAE,IAAY;IACzD,MAAM,MAAM,GAAY,EAAE,CAAC;IAC3B,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAClC,MAAM,MAAM,GAAe,EAAE,CAAC;IAE9B,gCAAgC;IAChC,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,EAAE,CAAC;YACN,MAAM,GAAG,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC;YACjB,MAAM,QAAQ,GAAG,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;YAClC,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;YACrE,MAAM,MAAM,GAAG,QAAQ,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;YACrD,MAAM,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,CAAC,GAAG,CAAC,EAAE,UAAU,EAAE,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC;QACtE,CAAC;IACH,CAAC;IAED,kCAAkC;IAClC,KAAK,MAAM,EAAE,IAAI,MAAM,EAAE,CAAC;QACxB,IAAI,EAAE,CAAC,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC;YAAE,SAAS;QAChE,IAAI,EAAE,CAAC,UAAU,KAAK,CAAC;YAAE,SAAS;QAElC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;YACtC,MAAM,QAAQ,GAAG,IAAA,iBAAS,EAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;YACrC,MAAM,EAAE,GAAG,IAAI,MAAM,CAAC,SAAS,GAAG,EAAE,CAAC,IAAI,CAAC,OAAO,CAAC,qBAAqB,EAAE,MAAM,CAAC,GAAG,MAAM,CAAC,CAAC;YAC3F,kCAAkC;YAClC,IAAI,QAAQ,CAAC,QAAQ,CAAC,OAAO,CAAC,IAAI,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC,IAAI,CAAC;gBAAE,SAAS;YAEvE,MAAM,SAAS,GAAG,EAAE,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;YACpC,IAAI,SAAS,EAAE,CAAC;gBACd,8CAA8C;gBAC9C,IAAI,QAAQ,GAAG,QAAQ,CAAC,KAAK,CAAC,SAAS,CAAC,KAAK,CAAC,CAAC;gBAC/C,IAAI,UAAU,GAAG,CAAC,CAAC;gBACnB,KAAK,MAAM,EAAE,IAAI,QAAQ,EAAE,CAAC;oBAC1B,IAAI,EAAE,KAAK,GAAG;wBAAE,UAAU,EAAE,CAAC;yBACxB,IAAI,EAAE,KAAK,GAAG;wBAAE,UAAU,EAAE,CAAC;gBACpC,CAAC;gBACD,8CAA8C;gBAC9C,IAAI,OAAO,GAAG,CAAC,GAAG,CAAC,CAAC;gBACpB,OAAO,UAAU,GAAG,CAAC,IAAI,OAAO,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC;oBAChD,MAAM,YAAY,GAAG,IAAA,iBAAS,EAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC;oBAC/C,QAAQ,IAAI,GAAG,GAAG,YAAY,CAAC;oBAC/B,KAAK,MAAM,EAAE,IAAI,YAAY,EAAE,CAAC;wBAC9B,IAAI,EAAE,KAAK,GAAG;4BAAE,UAAU,EAAE,CAAC;6BACxB,IAAI,EAAE,KAAK,GAAG;4BAAE,UAAU,EAAE,CAAC;oBACpC,CAAC;oBACD,OAAO,EAAE,CAAC;gBACZ,CAAC;gBAED,gDAAgD;gBAChD,MAAM,SAAS,GAAG,QAAQ,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,qBAAqB;gBAChF,8BAA8B;gBAC9B,IAAI,QAAQ,GAAG,CAAC,CAAC,CAAC;gBAClB,IAAI,EAAE,GAAG,CAAC,CAAC;gBACX,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,SAAS,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;oBAC1C,IAAI,SAAS,CAAC,CAAC,CAAC,KAAK,GAAG;wBAAE,EAAE,EAAE,CAAC;yBAC1B,IAAI,SAAS,CAAC,CAAC,CAAC,KAAK,GAAG,EAAE,CAAC;wBAAC,EAAE,EAAE,CAAC;wBAAC,IAAI,EAAE,KAAK,CAAC,EAAE,CAAC;4BAAC,QAAQ,GAAG,CAAC,CAAC;4BAAC,MAAM;wBAAC,CAAC;oBAAC,CAAC;gBACjF,CAAC;gBACD,MAAM,OAAO,GAAG,QAAQ,IAAI,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,KAAK,CAAC,CAAC,EAAE,QAAQ,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,SAAS,CAAC,IAAI,EAAE,CAAC;gBACvF,IAAI,CAAC,OAAO;oBAAE,SAAS;gBAEvB,wCAAwC;gBACxC,IAAI,QAAQ,GAAG,CAAC,CAAC;gBACjB,IAAI,KAAK,GAAG,CAAC,CAAC;gBACd,IAAI,KAAK,GAAG,KAAK,CAAC;gBAClB,KAAK,MAAM,EAAE,IAAI,OAAO,EAAE,CAAC;oBACzB,IAAI,EAAE,KAAK,GAAG,EAAE,CAAC;wBAAC,KAAK,EAAE,CAAC;wBAAC,IAAI,CAAC,KAAK,EAAE,CAAC;4BAAC,KAAK,GAAG,IAAI,CAAC;wBAAC,CAAC;oBAAC,CAAC;yBACrD,IAAI,EAAE,KAAK,GAAG,EAAE,CAAC;wBAAC,KAAK,EAAE,CAAC;wBAAC,IAAI,KAAK,KAAK,CAAC,EAAE,CAAC;4BAAC,IAAI,KAAK,EAAE,CAAC;gCAAC,QAAQ,EAAE,CAAC;gCAAC,KAAK,GAAG,KAAK,CAAC;4BAAC,CAAC;wBAAC,CAAC;oBAAC,CAAC;yBAC3F,IAAI,KAAK,KAAK,CAAC,IAAI,CAAC,EAAE,KAAK,GAAG,IAAI,EAAE,KAAK,IAAI,CAAC,EAAE,CAAC;wBAAC,IAAI,KAAK,EAAE,CAAC;4BAAC,QAAQ,EAAE,CAAC;4BAAC,KAAK,GAAG,KAAK,CAAC;wBAAC,CAAC;oBAAC,CAAC;yBAC7F,IAAI,CAAC,KAAK,EAAE,CAAC;wBAAC,KAAK,GAAG,IAAI,CAAC;oBAAC,CAAC;gBACpC,CAAC;gBACD,IAAI,KAAK;oBAAE,QAAQ,EAAE,CAAC;gBAEtB,IAAI,QAAQ,KAAK,EAAE,CAAC,UAAU,EAAE,CAAC;oBAC/B,MAAM,CAAC,IAAI,CAAC;wBACV,IAAI,EAAE,IAAI,EAAE,CAAC,GAAG,CAAC,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,WAAW;wBACtD,OAAO,EAAE,IAAA,UAAC,EAAC,WAAW,EAAE,EAAE,CAAC,IAAI,EAAE,QAAQ,EAAE,EAAE,CAAC,UAAU,CAAC;qBAC1D,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;QACH,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"comment-style.d.ts","sourceRoot":"","sources":["../../src/checks/comment-style.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,UAAU,CAAC;AAGjC,wBAAgB,iBAAiB,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,KAAK,EAAE,CAexE"}
|
|
@@ -0,0 +1,21 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.checkCommentStyle = checkCommentStyle;
|
|
4
|
+
const locale_1 = require("../locale");
|
|
5
|
+
function checkCommentStyle(content, file) {
|
|
6
|
+
const issues = [];
|
|
7
|
+
const lines = content.split('\n');
|
|
8
|
+
for (let i = 0; i < lines.length; i++) {
|
|
9
|
+
const line = lines[i];
|
|
10
|
+
const trimmed = line.trim();
|
|
11
|
+
// Detect ; not followed by space (except ;; and ;|)
|
|
12
|
+
if (/^;[^ ;|]/.test(trimmed)) {
|
|
13
|
+
issues.push({
|
|
14
|
+
file, line: i + 1, severity: 'warn', rule: 'comment_style',
|
|
15
|
+
message: (0, locale_1.t)('comment_style'),
|
|
16
|
+
});
|
|
17
|
+
}
|
|
18
|
+
}
|
|
19
|
+
return issues;
|
|
20
|
+
}
|
|
21
|
+
//# sourceMappingURL=comment-style.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"comment-style.js","sourceRoot":"","sources":["../../src/checks/comment-style.ts"],"names":[],"mappings":";;AAGA,8CAeC;AAjBD,sCAA8B;AAE9B,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,IAAI,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;QACtB,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,EAAE,CAAC;QAC5B,oDAAoD;QACpD,IAAI,UAAU,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC;YAC7B,MAAM,CAAC,IAAI,CAAC;gBACV,IAAI,EAAE,IAAI,EAAE,CAAC,GAAG,CAAC,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,eAAe;gBAC1D,OAAO,EAAE,IAAA,UAAC,EAAC,eAAe,CAAC;aAC5B,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"cond-simplify.d.ts","sourceRoot":"","sources":["../../src/checks/cond-simplify.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,CAmCxE"}
|
|
@@ -0,0 +1,42 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.checkCondSimplify = checkCondSimplify;
|
|
4
|
+
const locale_1 = require("../locale");
|
|
5
|
+
const utils_1 = require("../utils");
|
|
6
|
+
function checkCondSimplify(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
|
+
// Match (cond (t ...)) or (cond (expr ...)) - single branch cond
|
|
12
|
+
// A single-branch cond typically looks like: (cond (expr body)) on one line
|
|
13
|
+
const m = stripped.match(/\(cond\s+\(/);
|
|
14
|
+
if (m) {
|
|
15
|
+
// Count how many top-level branches inside this cond
|
|
16
|
+
const condStart = stripped.indexOf('(cond');
|
|
17
|
+
const afterCond = stripped.slice(condStart + 5).trim();
|
|
18
|
+
let branchCount = 0;
|
|
19
|
+
let parenDepth = 0;
|
|
20
|
+
for (const ch of afterCond) {
|
|
21
|
+
if (ch === '(') {
|
|
22
|
+
if (parenDepth === 0)
|
|
23
|
+
branchCount++;
|
|
24
|
+
parenDepth++;
|
|
25
|
+
}
|
|
26
|
+
else if (ch === ')') {
|
|
27
|
+
parenDepth--;
|
|
28
|
+
if (parenDepth < 0)
|
|
29
|
+
break;
|
|
30
|
+
}
|
|
31
|
+
}
|
|
32
|
+
if (branchCount === 1) {
|
|
33
|
+
issues.push({
|
|
34
|
+
file, line: i + 1, severity: 'warn', rule: 'cond_simplify',
|
|
35
|
+
message: (0, locale_1.t)('cond_simplify'),
|
|
36
|
+
});
|
|
37
|
+
}
|
|
38
|
+
}
|
|
39
|
+
}
|
|
40
|
+
return issues;
|
|
41
|
+
}
|
|
42
|
+
//# sourceMappingURL=cond-simplify.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"cond-simplify.js","sourceRoot":"","sources":["../../src/checks/cond-simplify.ts"],"names":[],"mappings":";;AAIA,8CAmCC;AAtCD,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;IAElC,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,iEAAiE;QACjE,4EAA4E;QAC5E,MAAM,CAAC,GAAG,QAAQ,CAAC,KAAK,CAAC,aAAa,CAAC,CAAC;QACxC,IAAI,CAAC,EAAE,CAAC;YACN,qDAAqD;YACrD,MAAM,SAAS,GAAG,QAAQ,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC;YAC5C,MAAM,SAAS,GAAG,QAAQ,CAAC,KAAK,CAAC,SAAS,GAAG,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;YACvD,IAAI,WAAW,GAAG,CAAC,CAAC;YACpB,IAAI,UAAU,GAAG,CAAC,CAAC;YAEnB,KAAK,MAAM,EAAE,IAAI,SAAS,EAAE,CAAC;gBAC3B,IAAI,EAAE,KAAK,GAAG,EAAE,CAAC;oBACf,IAAI,UAAU,KAAK,CAAC;wBAAE,WAAW,EAAE,CAAC;oBACpC,UAAU,EAAE,CAAC;gBACf,CAAC;qBAAM,IAAI,EAAE,KAAK,GAAG,EAAE,CAAC;oBACtB,UAAU,EAAE,CAAC;oBACb,IAAI,UAAU,GAAG,CAAC;wBAAE,MAAM;gBAC5B,CAAC;YACH,CAAC;YAED,IAAI,WAAW,KAAK,CAAC,EAAE,CAAC;gBACtB,MAAM,CAAC,IAAI,CAAC;oBACV,IAAI,EAAE,IAAI,EAAE,CAAC,GAAG,CAAC,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,eAAe;oBAC1D,OAAO,EAAE,IAAA,UAAC,EAAC,eAAe,CAAC;iBAC5B,CAAC,CAAC;YACL,CAAC;QACH,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"dangerous-calls.d.ts","sourceRoot":"","sources":["../../src/checks/dangerous-calls.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,oBAAoB,EAAE,MAAM,UAAU,CAAC;
|
|
1
|
+
{"version":3,"file":"dangerous-calls.d.ts","sourceRoot":"","sources":["../../src/checks/dangerous-calls.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,oBAAoB,EAAE,MAAM,UAAU,CAAC;AA2BvD,wBAAgB,mBAAmB,CACjC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,oBAAoB,GAC1D,KAAK,EAAE,CAwBT"}
|
|
@@ -4,8 +4,8 @@ exports.checkDangerousCalls = checkDangerousCalls;
|
|
|
4
4
|
const utils_1 = require("../utils");
|
|
5
5
|
const locale_1 = require("../locale");
|
|
6
6
|
const PATTERNS = [
|
|
7
|
-
{ rule: 'quit', check: l => /\bquit\b/.test(l), key: 'quit' },
|
|
8
|
-
{ rule: 'exit', check: l => /\bexit\b/.test(l), key: 'exit' },
|
|
7
|
+
{ rule: 'quit', check: l => /\bquit\b/.test(l), key: 'quit', useStripped: true },
|
|
8
|
+
{ rule: 'exit', check: l => /\bexit\b/.test(l), key: 'exit', useStripped: true },
|
|
9
9
|
{ rule: 'command_shell', check: l => /\(command\s+"shell"\s/.test(l), key: 'command_shell' },
|
|
10
10
|
{ rule: 'startapp', check: l => /\bstartapp\b/.test(l), key: 'startapp' },
|
|
11
11
|
{ rule: 'vl_registry_write', check: l => /\bvl-registry-write\b/.test(l), key: 'vl_registry_write' },
|
|
@@ -22,15 +22,17 @@ function checkDangerousCalls(content, file, config) {
|
|
|
22
22
|
const lines = content.split('\n');
|
|
23
23
|
for (let lineno = 1; lineno <= lines.length; lineno++) {
|
|
24
24
|
const line = lines[lineno - 1];
|
|
25
|
-
//
|
|
26
|
-
const
|
|
27
|
-
|
|
28
|
-
continue;
|
|
25
|
+
// Two versions: without strings (for bare word checks) and with strings (for quoted arg checks)
|
|
26
|
+
const strippedCode = (0, utils_1.stripLine)(line).trim(); // strings removed
|
|
27
|
+
const strippedRaw = (0, utils_1.stripLineRaw)(line).trim(); // strings preserved
|
|
29
28
|
for (const p of PATTERNS) {
|
|
30
29
|
const configuredSeverity = config[p.key];
|
|
31
30
|
if (configuredSeverity === 'off')
|
|
32
31
|
continue;
|
|
33
|
-
|
|
32
|
+
const testStr = p.useStripped ? strippedCode : strippedRaw;
|
|
33
|
+
if (!testStr)
|
|
34
|
+
continue;
|
|
35
|
+
if (p.check(testStr)) {
|
|
34
36
|
issues.push({
|
|
35
37
|
file, line: lineno, severity: configuredSeverity, rule: p.rule,
|
|
36
38
|
message: (0, locale_1.t)(MESSAGE_KEYS[p.rule]),
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"dangerous-calls.js","sourceRoot":"","sources":["../../src/checks/dangerous-calls.ts"],"names":[],"mappings":";;
|
|
1
|
+
{"version":3,"file":"dangerous-calls.js","sourceRoot":"","sources":["../../src/checks/dangerous-calls.ts"],"names":[],"mappings":";;AA2BA,kDA0BC;AApDD,oCAAmD;AACnD,sCAA8B;AAS9B,MAAM,QAAQ,GAAc;IAC1B,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,CAAC,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,GAAG,EAAE,MAAM,EAAE,WAAW,EAAE,IAAI,EAAE;IAChF,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,CAAC,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,GAAG,EAAE,MAAM,EAAE,WAAW,EAAE,IAAI,EAAE;IAChF,EAAE,IAAI,EAAE,eAAe,EAAE,KAAK,EAAE,CAAC,CAAC,EAAE,CAAC,uBAAuB,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,GAAG,EAAE,eAAe,EAAE;IAC5F,EAAE,IAAI,EAAE,UAAU,EAAE,KAAK,EAAE,CAAC,CAAC,EAAE,CAAC,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,GAAG,EAAE,UAAU,EAAE;IACzE,EAAE,IAAI,EAAE,mBAAmB,EAAE,KAAK,EAAE,CAAC,CAAC,EAAE,CAAC,uBAAuB,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,GAAG,EAAE,mBAAmB,EAAE;CACrG,CAAC;AAEF,MAAM,YAAY,GAA2B;IAC3C,IAAI,EAAE,gBAAgB;IACtB,IAAI,EAAE,gBAAgB;IACtB,aAAa,EAAE,yBAAyB;IACxC,QAAQ,EAAE,oBAAoB;IAC9B,iBAAiB,EAAE,6BAA6B;CACjD,CAAC;AAEF,SAAgB,mBAAmB,CACjC,OAAe,EAAE,IAAY,EAAE,MAA4B;IAE3D,MAAM,MAAM,GAAY,EAAE,CAAC;IAC3B,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAElC,KAAK,IAAI,MAAM,GAAG,CAAC,EAAE,MAAM,IAAI,KAAK,CAAC,MAAM,EAAE,MAAM,EAAE,EAAE,CAAC;QACtD,MAAM,IAAI,GAAG,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;QAC/B,gGAAgG;QAChG,MAAM,YAAY,GAAG,IAAA,iBAAS,EAAC,IAAI,CAAC,CAAC,IAAI,EAAE,CAAC,CAAG,kBAAkB;QACjE,MAAM,WAAW,GAAG,IAAA,oBAAY,EAAC,IAAI,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,oBAAoB;QAEnE,KAAK,MAAM,CAAC,IAAI,QAAQ,EAAE,CAAC;YACzB,MAAM,kBAAkB,GAAG,MAAM,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC;YACzC,IAAI,kBAAkB,KAAK,KAAK;gBAAE,SAAS;YAC3C,MAAM,OAAO,GAAG,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,WAAW,CAAC;YAC3D,IAAI,CAAC,OAAO;gBAAE,SAAS;YACvB,IAAI,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,EAAE,CAAC;gBACrB,MAAM,CAAC,IAAI,CAAC;oBACV,IAAI,EAAE,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,kBAAkB,EAAE,IAAI,EAAE,CAAC,CAAC,IAAI;oBAC9D,OAAO,EAAE,IAAA,UAAC,EAAC,YAAY,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;iBACjC,CAAC,CAAC;YACL,CAAC;QACH,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"dynamic-doc.d.ts","sourceRoot":"","sources":["../../src/checks/dynamic-doc.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,CActE"}
|
|
@@ -0,0 +1,21 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.checkDynamicDoc = checkDynamicDoc;
|
|
4
|
+
const locale_1 = require("../locale");
|
|
5
|
+
const utils_1 = require("../utils");
|
|
6
|
+
function checkDynamicDoc(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(/\(defun\s+(C:\S+)\b/);
|
|
12
|
+
if (m) {
|
|
13
|
+
const body = lines.slice(i + 1).join('\n');
|
|
14
|
+
if (!body.includes('(princ)')) {
|
|
15
|
+
issues.push({ file, line: i + 1, severity: 'warn', rule: 'dynamic_doc', message: (0, locale_1.t)('dynamic_doc', m[1]) });
|
|
16
|
+
}
|
|
17
|
+
}
|
|
18
|
+
}
|
|
19
|
+
return issues;
|
|
20
|
+
}
|
|
21
|
+
//# sourceMappingURL=dynamic-doc.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"dynamic-doc.js","sourceRoot":"","sources":["../../src/checks/dynamic-doc.ts"],"names":[],"mappings":";;AAIA,0CAcC;AAjBD,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,MAAM,CAAC,GAAG,QAAQ,CAAC,KAAK,CAAC,qBAAqB,CAAC,CAAC;QAChD,IAAI,CAAC,EAAE,CAAC;YACN,MAAM,IAAI,GAAG,KAAK,CAAC,KAAK,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAC3C,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,SAAS,CAAC,EAAE,CAAC;gBAC9B,MAAM,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,CAAC,GAAG,CAAC,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,aAAa,EAAE,OAAO,EAAE,IAAA,UAAC,EAAC,aAAa,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;YAC7G,CAAC;QACH,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"empty-catch.d.ts","sourceRoot":"","sources":["../../src/checks/empty-catch.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,CAwBtE"}
|
|
@@ -0,0 +1,31 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.checkEmptyCatch = checkEmptyCatch;
|
|
4
|
+
const locale_1 = require("../locale");
|
|
5
|
+
const utils_1 = require("../utils");
|
|
6
|
+
function checkEmptyCatch(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 (vl-catch-all-apply ...) not inside (vl-catch-all-error-p ...) check
|
|
12
|
+
if (stripped.includes('vl-catch-all-apply')) {
|
|
13
|
+
// Check subsequent lines for error-p check
|
|
14
|
+
let hasErrorCheck = false;
|
|
15
|
+
for (let j = i + 1; j < Math.min(i + 5, lines.length); j++) {
|
|
16
|
+
if ((0, utils_1.stripLine)(lines[j]).includes('vl-catch-all-error-p')) {
|
|
17
|
+
hasErrorCheck = true;
|
|
18
|
+
break;
|
|
19
|
+
}
|
|
20
|
+
}
|
|
21
|
+
if (!hasErrorCheck) {
|
|
22
|
+
issues.push({
|
|
23
|
+
file, line: i + 1, severity: 'warn', rule: 'empty_catch',
|
|
24
|
+
message: (0, locale_1.t)('empty_catch'),
|
|
25
|
+
});
|
|
26
|
+
}
|
|
27
|
+
}
|
|
28
|
+
}
|
|
29
|
+
return issues;
|
|
30
|
+
}
|
|
31
|
+
//# sourceMappingURL=empty-catch.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"empty-catch.js","sourceRoot":"","sources":["../../src/checks/empty-catch.ts"],"names":[],"mappings":";;AAIA,0CAwBC;AA3BD,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,8EAA8E;QAC9E,IAAI,QAAQ,CAAC,QAAQ,CAAC,oBAAoB,CAAC,EAAE,CAAC;YAC5C,2CAA2C;YAC3C,IAAI,aAAa,GAAG,KAAK,CAAC;YAC1B,KAAK,IAAI,CAAC,GAAG,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,GAAG,CAAC,EAAE,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC;gBAC3D,IAAI,IAAA,iBAAS,EAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,sBAAsB,CAAC,EAAE,CAAC;oBACzD,aAAa,GAAG,IAAI,CAAC;oBACrB,MAAM;gBACR,CAAC;YACH,CAAC;YACD,IAAI,CAAC,aAAa,EAAE,CAAC;gBACnB,MAAM,CAAC,IAAI,CAAC;oBACV,IAAI,EAAE,IAAI,EAAE,CAAC,GAAG,CAAC,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,aAAa;oBACxD,OAAO,EAAE,IAAA,UAAC,EAAC,aAAa,CAAC;iBAC1B,CAAC,CAAC;YACL,CAAC;QACH,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"eq-usage.d.ts","sourceRoot":"","sources":["../../src/checks/eq-usage.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,UAAU,CAAC;AAIjC,wBAAgB,YAAY,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,KAAK,EAAE,CAenE"}
|
|
@@ -0,0 +1,22 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.checkEqUsage = checkEqUsage;
|
|
4
|
+
const locale_1 = require("../locale");
|
|
5
|
+
const utils_1 = require("../utils");
|
|
6
|
+
function checkEqUsage(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 (eq num ...) where num is literal number
|
|
12
|
+
const m = stripped.match(/\(eq\s+\S+\s+(\d+)/);
|
|
13
|
+
if (m) {
|
|
14
|
+
issues.push({
|
|
15
|
+
file, line: i + 1, severity: 'warn', rule: 'eq_usage',
|
|
16
|
+
message: (0, locale_1.t)('eq_usage'),
|
|
17
|
+
});
|
|
18
|
+
}
|
|
19
|
+
}
|
|
20
|
+
return issues;
|
|
21
|
+
}
|
|
22
|
+
//# sourceMappingURL=eq-usage.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"eq-usage.js","sourceRoot":"","sources":["../../src/checks/eq-usage.ts"],"names":[],"mappings":";;AAIA,oCAeC;AAlBD,sCAA8B;AAC9B,oCAAqC;AAErC,SAAgB,YAAY,CAAC,OAAe,EAAE,IAAY;IACxD,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,kDAAkD;QAClD,MAAM,CAAC,GAAG,QAAQ,CAAC,KAAK,CAAC,oBAAoB,CAAC,CAAC;QAC/C,IAAI,CAAC,EAAE,CAAC;YACN,MAAM,CAAC,IAAI,CAAC;gBACV,IAAI,EAAE,IAAI,EAAE,CAAC,GAAG,CAAC,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,UAAU;gBACrD,OAAO,EAAE,IAAA,UAAC,EAAC,UAAU,CAAC;aACvB,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"error-handling.d.ts","sourceRoot":"","sources":["../../src/checks/error-handling.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,UAAU,CAAC;AAIjC,wBAAgB,kBAAkB,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,KAAK,EAAE,CAgDzE"}
|