@futdevpro/dynamo-eslint 1.15.25 → 1.15.28
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/.c8rc.json +26 -26
- package/.cursor/rules/__assistant_guide.mdc +30 -30
- package/.cursor/rules/__main.mdc +64 -64
- package/.cursor/rules/_ag_backend-structure.mdc +85 -85
- package/.cursor/rules/_ag_backend.mdc +16 -16
- package/.cursor/rules/_ag_debug.mdc +7 -7
- package/.cursor/rules/_ag_documentation_writing_rules.mdc +372 -372
- package/.cursor/rules/_ag_file-refactoring.mdc +113 -113
- package/.cursor/rules/_ag_fixes_rules.mdc +5 -5
- package/.cursor/rules/_ag_frontend-structure.mdc +86 -86
- package/.cursor/rules/_ag_frontend.mdc +39 -39
- package/.cursor/rules/_ag_import-rules.mdc +44 -44
- package/.cursor/rules/_ag_naming.mdc +115 -115
- package/.cursor/rules/_ag_running_commands.mdc +4 -4
- package/.cursor/rules/_ag_server-controller.mdc +5 -5
- package/.cursor/rules/_ag_should-be.mdc +6 -6
- package/.cursor/rules/_ag_swearing.mdc +47 -47
- package/.cursor/rules/ai_development_guide.md +60 -60
- package/.cursor/rules/ai_directives.md +114 -114
- package/.cursor/rules/cursor-rules.md +160 -160
- package/.cursor/rules/default-command.mdc +464 -464
- package/.cursor/rules/error_code_pattern.md +39 -39
- package/.cursor/rules/saved rule mcp server use.md +15 -15
- package/.dynamo/pipeline.cicd.config.json +184 -184
- package/.dynamo/version-bump.config.json +5 -5
- package/.eslintrc.json +4 -4
- package/.github/workflows/main.yml +436 -436
- package/.husky/pre-commit +2 -2
- package/.vscode/settings.json +11 -11
- package/README.md +319 -319
- package/__documentations/2026-04-18-ccap-review-integration.md +173 -173
- package/__documentations/2026-08-18-bfr-ctm-005-eslint9-validator-report.md +45 -0
- package/build/configs/base.d.ts +9 -9
- package/build/configs/base.js +150 -150
- package/build/configs/fsm.d.ts +1 -1
- package/build/configs/fsm.js +7 -7
- package/build/configs/ngx-package.d.ts +1 -1
- package/build/configs/ngx-package.js +7 -7
- package/build/configs/ngx.d.ts +4 -4
- package/build/configs/ngx.js +91 -91
- package/build/configs/nts-package.d.ts +1 -1
- package/build/configs/nts-package.js +17 -17
- package/build/configs/nts.d.ts +1 -1
- package/build/configs/nts.js +31 -31
- package/build/index.d.ts +1 -1
- package/build/index.js +2 -2
- package/build/plugin/index.d.ts +52 -52
- package/build/plugin/index.js +80 -80
- package/build/plugin/rules/angular/angular-for-track-logic.d.ts +7 -7
- package/build/plugin/rules/angular/angular-for-track-logic.js +81 -81
- package/build/plugin/rules/angular/effect-signal-loop.d.ts +9 -9
- package/build/plugin/rules/angular/effect-signal-loop.js +112 -112
- package/build/plugin/rules/angular/no-async-ngoninit.d.ts +8 -8
- package/build/plugin/rules/angular/no-async-ngoninit.js +42 -42
- package/build/plugin/rules/angular/no-component-direct-api.d.ts +9 -9
- package/build/plugin/rules/angular/no-component-direct-api.js +75 -75
- package/build/plugin/rules/angular/no-method-call-in-template.d.ts +8 -8
- package/build/plugin/rules/angular/no-method-call-in-template.js +79 -79
- package/build/plugin/rules/angular/no-ngfor-directive.d.ts +7 -7
- package/build/plugin/rules/angular/no-ngfor-directive.js +59 -59
- package/build/plugin/rules/angular/no-ngif-directive.d.ts +7 -7
- package/build/plugin/rules/angular/no-ngif-directive.js +59 -59
- package/build/plugin/rules/explicit-types.d.ts +3 -3
- package/build/plugin/rules/explicit-types.js +178 -178
- package/build/plugin/rules/import/import-order.d.ts +3 -3
- package/build/plugin/rules/import/import-order.js +154 -154
- package/build/plugin/rules/import/no-import-type.d.ts +3 -3
- package/build/plugin/rules/import/no-import-type.js +45 -45
- package/build/plugin/rules/import/no-js-import.d.ts +3 -3
- package/build/plugin/rules/import/no-js-import.js +40 -40
- package/build/plugin/rules/naming-patterns.d.ts +7 -7
- package/build/plugin/rules/naming-patterns.js +366 -366
- package/build/plugin/rules/no-getter-logic.d.ts +3 -3
- package/build/plugin/rules/no-getter-logic.js +194 -194
- package/build/plugin/rules/prefer-enum-over-string-union.d.ts +3 -3
- package/build/plugin/rules/prefer-enum-over-string-union.js +289 -289
- package/build/plugin/rules/prefer-interface-over-duplicate-types.d.ts +3 -3
- package/build/plugin/rules/prefer-interface-over-duplicate-types.js +264 -264
- package/build/plugin/rules/require-jsdoc-description.d.ts +3 -3
- package/build/plugin/rules/require-jsdoc-description.js +496 -496
- package/build/plugin/rules/require-test-description-prefix.d.ts +3 -3
- package/build/plugin/rules/require-test-description-prefix.js +135 -135
- package/build/plugin/rules/ts/data-model-extends.d.ts +8 -8
- package/build/plugin/rules/ts/data-model-extends.js +94 -94
- package/build/plugin/rules/ts/no-any-type.d.ts +8 -8
- package/build/plugin/rules/ts/no-any-type.js +36 -36
- package/build/plugin/rules/ts/no-as-cast.d.ts +8 -8
- package/build/plugin/rules/ts/no-as-cast.js +49 -49
- package/build/plugin/rules/ts/no-dynamic-imports.d.ts +8 -8
- package/build/plugin/rules/ts/no-dynamic-imports.js +50 -50
- package/build/plugin/rules/ts/no-native-browser-dialogs.d.ts +8 -8
- package/build/plugin/rules/ts/no-native-browser-dialogs.js +78 -78
- package/build/plugin/rules/ts/no-plain-function-export.d.ts +9 -9
- package/build/plugin/rules/ts/no-plain-function-export.js +69 -69
- package/build/plugin/rules/ts/one-export-per-file.d.ts +8 -8
- package/build/plugin/rules/ts/one-export-per-file.js +73 -73
- package/build/scripts/dynamo-fix.d.ts +26 -26
- package/build/scripts/dynamo-fix.js +146 -146
- package/build/scripts/eslintrc-audit.d.ts +26 -26
- package/build/scripts/eslintrc-audit.js +100 -100
- package/build/scripts/fix-return-types.d.ts +74 -74
- package/build/scripts/fix-return-types.js +226 -226
- package/build/scripts/validate-imports.d.ts +26 -26
- package/build/scripts/validate-imports.js +157 -146
- package/build/scripts/validate-imports.js.map +1 -1
- package/build/scripts/validate-naming.d.ts +26 -26
- package/build/scripts/validate-naming.js +157 -131
- package/build/scripts/validate-naming.js.map +1 -1
- package/eslint.config.js +15 -15
- package/package.json +105 -105
- package/samples/.vscode/settings.json +13 -13
- package/samples/base/.eslintrc.json +6 -6
- package/samples/base/eslint.config.js +3 -3
- package/samples/fsm/.eslintrc.json +4 -4
- package/samples/fsm/eslint.config.js +3 -3
- package/samples/ngx/.eslintrc.json +6 -6
- package/samples/ngx/eslint.config.js +3 -3
- package/samples/ngx-package/.eslintrc.json +4 -4
- package/samples/ngx-package/eslint.config.js +3 -3
- package/samples/nts/.eslintrc.json +6 -6
- package/samples/nts/eslint.config.js +3 -3
- package/samples/nts-package/.eslintrc.json +4 -4
- package/samples/nts-package/eslint.config.js +3 -3
- package/samples/package.json.example +26 -26
- package/samples/poc-sample.ts +38 -38
- package/samples/poc-violations.ts +54 -54
- package/scripts/run-coverage-tests.js +24 -24
- package/spec/support/helpers/ts-node-helper.js +9 -9
- package/spec/support/jasmine.coverage.json +23 -23
- package/spec/support/jasmine.json +24 -24
- package/src/configs/base.ts +157 -157
- package/src/configs/fsm.ts +10 -10
- package/src/configs/ngx-package.ts +10 -10
- package/src/configs/ngx.ts +96 -96
- package/src/configs/nts-package.ts +20 -20
- package/src/configs/nts.ts +34 -34
- package/src/index.ts +4 -4
- package/src/plugin/index.ts +86 -86
- package/src/plugin/rules/angular/angular-for-track-logic.spec.ts +44 -44
- package/src/plugin/rules/angular/angular-for-track-logic.ts +88 -88
- package/src/plugin/rules/angular/effect-signal-loop.spec.ts +89 -89
- package/src/plugin/rules/angular/effect-signal-loop.ts +125 -125
- package/src/plugin/rules/angular/no-async-ngoninit.spec.ts +50 -50
- package/src/plugin/rules/angular/no-async-ngoninit.ts +43 -43
- package/src/plugin/rules/angular/no-component-direct-api.spec.ts +78 -78
- package/src/plugin/rules/angular/no-component-direct-api.ts +82 -82
- package/src/plugin/rules/angular/no-method-call-in-template.spec.ts +63 -63
- package/src/plugin/rules/angular/no-method-call-in-template.ts +89 -89
- package/src/plugin/rules/angular/no-ngfor-directive.spec.ts +44 -44
- package/src/plugin/rules/angular/no-ngfor-directive.ts +62 -62
- package/src/plugin/rules/angular/no-ngif-directive.spec.ts +44 -44
- package/src/plugin/rules/angular/no-ngif-directive.ts +62 -62
- package/src/plugin/rules/explicit-types.spec.ts +190 -190
- package/src/plugin/rules/explicit-types.ts +186 -186
- package/src/plugin/rules/import/import-order.spec.ts +219 -219
- package/src/plugin/rules/import/import-order.ts +197 -197
- package/src/plugin/rules/import/no-import-type.spec.ts +69 -69
- package/src/plugin/rules/import/no-import-type.ts +46 -46
- package/src/plugin/rules/import/no-js-import.spec.ts +82 -82
- package/src/plugin/rules/import/no-js-import.ts +45 -45
- package/src/plugin/rules/naming-patterns.spec.ts +189 -189
- package/src/plugin/rules/naming-patterns.ts +426 -426
- package/src/plugin/rules/no-getter-logic.spec.ts +591 -591
- package/src/plugin/rules/no-getter-logic.ts +220 -220
- package/src/plugin/rules/prefer-enum-over-string-union.spec.ts +583 -583
- package/src/plugin/rules/prefer-enum-over-string-union.ts +333 -333
- package/src/plugin/rules/prefer-interface-over-duplicate-types.spec.ts +404 -404
- package/src/plugin/rules/prefer-interface-over-duplicate-types.ts +314 -314
- package/src/plugin/rules/require-jsdoc-description.spec.ts +779 -779
- package/src/plugin/rules/require-jsdoc-description.ts +572 -572
- package/src/plugin/rules/require-test-description-prefix.spec.ts +459 -459
- package/src/plugin/rules/require-test-description-prefix.ts +154 -154
- package/src/plugin/rules/ts/data-model-extends.spec.ts +84 -84
- package/src/plugin/rules/ts/data-model-extends.ts +100 -100
- package/src/plugin/rules/ts/no-any-type.spec.ts +28 -28
- package/src/plugin/rules/ts/no-any-type.ts +36 -36
- package/src/plugin/rules/ts/no-as-cast.spec.ts +51 -51
- package/src/plugin/rules/ts/no-as-cast.ts +51 -51
- package/src/plugin/rules/ts/no-dynamic-imports.spec.ts +54 -54
- package/src/plugin/rules/ts/no-dynamic-imports.ts +49 -49
- package/src/plugin/rules/ts/no-native-browser-dialogs.spec.ts +54 -54
- package/src/plugin/rules/ts/no-native-browser-dialogs.ts +84 -84
- package/src/plugin/rules/ts/no-plain-function-export.spec.ts +72 -72
- package/src/plugin/rules/ts/no-plain-function-export.ts +73 -73
- package/src/plugin/rules/ts/one-export-per-file.spec.ts +64 -64
- package/src/plugin/rules/ts/one-export-per-file.ts +77 -77
- package/src/scripts/dynamo-fix.ts +172 -172
- package/src/scripts/eslintrc-audit.ts +116 -116
- package/src/scripts/fix-return-types.spec.ts +65 -65
- package/src/scripts/fix-return-types.ts +288 -288
- package/src/scripts/validate-imports.ts +278 -261
- package/src/scripts/validate-naming.ts +204 -187
- package/src/scripts/validator-cli.integration.spec.ts +96 -0
- package/tsconfig.json +31 -31
|
@@ -1,49 +1,49 @@
|
|
|
1
|
-
import { Rule } from 'eslint';
|
|
2
|
-
|
|
3
|
-
/**
|
|
4
|
-
* No-dynamic-imports szabály: `import()` és `require()` dinamikus hívás tilos.
|
|
5
|
-
* Minden import a fájl tetején, top-level `import { … } from '…'` formában szerepeljen.
|
|
6
|
-
* CCAP "no-dynamic-imports" review ESLint AST-portja.
|
|
7
|
-
*/
|
|
8
|
-
const rule: Rule.RuleModule = {
|
|
9
|
-
meta: {
|
|
10
|
-
type: 'suggestion',
|
|
11
|
-
docs: {
|
|
12
|
-
description: 'Forbid dynamic `import()` and `require()` calls; all imports must be top-level.',
|
|
13
|
-
recommended: true,
|
|
14
|
-
},
|
|
15
|
-
schema: [],
|
|
16
|
-
messages: {
|
|
17
|
-
noDynamicImport: 'Dinamikus `import()` hívás tilos. Tedd top-level `import { … } from "…"` formába.',
|
|
18
|
-
noRequireCall: '`require()` hívás tilos. Használj top-level ES `import` szintaxist.',
|
|
19
|
-
},
|
|
20
|
-
},
|
|
21
|
-
create(context: Rule.RuleContext): Rule.RuleListener {
|
|
22
|
-
return {
|
|
23
|
-
ImportExpression(node: any): void {
|
|
24
|
-
try {
|
|
25
|
-
context.report({
|
|
26
|
-
node: node,
|
|
27
|
-
messageId: 'noDynamicImport',
|
|
28
|
-
});
|
|
29
|
-
} catch (error) {
|
|
30
|
-
console.error('[no-dynamic-imports] Error in ImportExpression visitor:', error);
|
|
31
|
-
}
|
|
32
|
-
},
|
|
33
|
-
CallExpression(node: any): void {
|
|
34
|
-
try {
|
|
35
|
-
if (node.callee && node.callee.type === 'Identifier' && node.callee.name === 'require') {
|
|
36
|
-
context.report({
|
|
37
|
-
node: node,
|
|
38
|
-
messageId: 'noRequireCall',
|
|
39
|
-
});
|
|
40
|
-
}
|
|
41
|
-
} catch (error) {
|
|
42
|
-
console.error('[no-dynamic-imports] Error in CallExpression visitor:', error);
|
|
43
|
-
}
|
|
44
|
-
},
|
|
45
|
-
};
|
|
46
|
-
},
|
|
47
|
-
};
|
|
48
|
-
|
|
49
|
-
export default rule;
|
|
1
|
+
import { Rule } from 'eslint';
|
|
2
|
+
|
|
3
|
+
/**
|
|
4
|
+
* No-dynamic-imports szabály: `import()` és `require()` dinamikus hívás tilos.
|
|
5
|
+
* Minden import a fájl tetején, top-level `import { … } from '…'` formában szerepeljen.
|
|
6
|
+
* CCAP "no-dynamic-imports" review ESLint AST-portja.
|
|
7
|
+
*/
|
|
8
|
+
const rule: Rule.RuleModule = {
|
|
9
|
+
meta: {
|
|
10
|
+
type: 'suggestion',
|
|
11
|
+
docs: {
|
|
12
|
+
description: 'Forbid dynamic `import()` and `require()` calls; all imports must be top-level.',
|
|
13
|
+
recommended: true,
|
|
14
|
+
},
|
|
15
|
+
schema: [],
|
|
16
|
+
messages: {
|
|
17
|
+
noDynamicImport: 'Dinamikus `import()` hívás tilos. Tedd top-level `import { … } from "…"` formába.',
|
|
18
|
+
noRequireCall: '`require()` hívás tilos. Használj top-level ES `import` szintaxist.',
|
|
19
|
+
},
|
|
20
|
+
},
|
|
21
|
+
create(context: Rule.RuleContext): Rule.RuleListener {
|
|
22
|
+
return {
|
|
23
|
+
ImportExpression(node: any): void {
|
|
24
|
+
try {
|
|
25
|
+
context.report({
|
|
26
|
+
node: node,
|
|
27
|
+
messageId: 'noDynamicImport',
|
|
28
|
+
});
|
|
29
|
+
} catch (error) {
|
|
30
|
+
console.error('[no-dynamic-imports] Error in ImportExpression visitor:', error);
|
|
31
|
+
}
|
|
32
|
+
},
|
|
33
|
+
CallExpression(node: any): void {
|
|
34
|
+
try {
|
|
35
|
+
if (node.callee && node.callee.type === 'Identifier' && node.callee.name === 'require') {
|
|
36
|
+
context.report({
|
|
37
|
+
node: node,
|
|
38
|
+
messageId: 'noRequireCall',
|
|
39
|
+
});
|
|
40
|
+
}
|
|
41
|
+
} catch (error) {
|
|
42
|
+
console.error('[no-dynamic-imports] Error in CallExpression visitor:', error);
|
|
43
|
+
}
|
|
44
|
+
},
|
|
45
|
+
};
|
|
46
|
+
},
|
|
47
|
+
};
|
|
48
|
+
|
|
49
|
+
export default rule;
|
|
@@ -1,54 +1,54 @@
|
|
|
1
|
-
import noNativeBrowserDialogsRule from './no-native-browser-dialogs';
|
|
2
|
-
|
|
3
|
-
describe('| no-native-browser-dialogs', () => {
|
|
4
|
-
it('| should be a valid ESLint rule', () => {
|
|
5
|
-
expect(noNativeBrowserDialogsRule.meta?.type).toBe('suggestion');
|
|
6
|
-
expect(noNativeBrowserDialogsRule.meta?.messages?.noNativeDialog).toBeDefined();
|
|
7
|
-
});
|
|
8
|
-
|
|
9
|
-
it('| should report `alert(...)`', () => {
|
|
10
|
-
let reportCalled: boolean = false;
|
|
11
|
-
const mockContext: any = {
|
|
12
|
-
report: (options: any): void => {
|
|
13
|
-
reportCalled = true;
|
|
14
|
-
expect(options.messageId).toBe('noNativeDialog');
|
|
15
|
-
expect(options.data.name).toBe('alert');
|
|
16
|
-
},
|
|
17
|
-
options: [],
|
|
18
|
-
};
|
|
19
|
-
const rule: any = noNativeBrowserDialogsRule.create(mockContext);
|
|
20
|
-
const mockNode: any = { callee: { type: 'Identifier', name: 'alert' } };
|
|
21
|
-
rule.CallExpression(mockNode);
|
|
22
|
-
expect(reportCalled).toBe(true);
|
|
23
|
-
});
|
|
24
|
-
|
|
25
|
-
it('| should report `window.confirm(...)`', () => {
|
|
26
|
-
let reportCalled: boolean = false;
|
|
27
|
-
const mockContext: any = {
|
|
28
|
-
report: (options: any): void => {
|
|
29
|
-
reportCalled = true;
|
|
30
|
-
expect(options.data.name).toBe('window.confirm');
|
|
31
|
-
},
|
|
32
|
-
options: [],
|
|
33
|
-
};
|
|
34
|
-
const rule: any = noNativeBrowserDialogsRule.create(mockContext);
|
|
35
|
-
const mockNode: any = {
|
|
36
|
-
callee: {
|
|
37
|
-
type: 'MemberExpression',
|
|
38
|
-
object: { type: 'Identifier', name: 'window' },
|
|
39
|
-
property: { type: 'Identifier', name: 'confirm' },
|
|
40
|
-
},
|
|
41
|
-
};
|
|
42
|
-
rule.CallExpression(mockNode);
|
|
43
|
-
expect(reportCalled).toBe(true);
|
|
44
|
-
});
|
|
45
|
-
|
|
46
|
-
it('| should not report unrelated calls', () => {
|
|
47
|
-
let reportCalled: boolean = false;
|
|
48
|
-
const mockContext: any = { report: () => { reportCalled = true; }, options: [] };
|
|
49
|
-
const rule: any = noNativeBrowserDialogsRule.create(mockContext);
|
|
50
|
-
const mockNode: any = { callee: { type: 'Identifier', name: 'doSomething' } };
|
|
51
|
-
rule.CallExpression(mockNode);
|
|
52
|
-
expect(reportCalled).toBe(false);
|
|
53
|
-
});
|
|
54
|
-
});
|
|
1
|
+
import noNativeBrowserDialogsRule from './no-native-browser-dialogs';
|
|
2
|
+
|
|
3
|
+
describe('| no-native-browser-dialogs', () => {
|
|
4
|
+
it('| should be a valid ESLint rule', () => {
|
|
5
|
+
expect(noNativeBrowserDialogsRule.meta?.type).toBe('suggestion');
|
|
6
|
+
expect(noNativeBrowserDialogsRule.meta?.messages?.noNativeDialog).toBeDefined();
|
|
7
|
+
});
|
|
8
|
+
|
|
9
|
+
it('| should report `alert(...)`', () => {
|
|
10
|
+
let reportCalled: boolean = false;
|
|
11
|
+
const mockContext: any = {
|
|
12
|
+
report: (options: any): void => {
|
|
13
|
+
reportCalled = true;
|
|
14
|
+
expect(options.messageId).toBe('noNativeDialog');
|
|
15
|
+
expect(options.data.name).toBe('alert');
|
|
16
|
+
},
|
|
17
|
+
options: [],
|
|
18
|
+
};
|
|
19
|
+
const rule: any = noNativeBrowserDialogsRule.create(mockContext);
|
|
20
|
+
const mockNode: any = { callee: { type: 'Identifier', name: 'alert' } };
|
|
21
|
+
rule.CallExpression(mockNode);
|
|
22
|
+
expect(reportCalled).toBe(true);
|
|
23
|
+
});
|
|
24
|
+
|
|
25
|
+
it('| should report `window.confirm(...)`', () => {
|
|
26
|
+
let reportCalled: boolean = false;
|
|
27
|
+
const mockContext: any = {
|
|
28
|
+
report: (options: any): void => {
|
|
29
|
+
reportCalled = true;
|
|
30
|
+
expect(options.data.name).toBe('window.confirm');
|
|
31
|
+
},
|
|
32
|
+
options: [],
|
|
33
|
+
};
|
|
34
|
+
const rule: any = noNativeBrowserDialogsRule.create(mockContext);
|
|
35
|
+
const mockNode: any = {
|
|
36
|
+
callee: {
|
|
37
|
+
type: 'MemberExpression',
|
|
38
|
+
object: { type: 'Identifier', name: 'window' },
|
|
39
|
+
property: { type: 'Identifier', name: 'confirm' },
|
|
40
|
+
},
|
|
41
|
+
};
|
|
42
|
+
rule.CallExpression(mockNode);
|
|
43
|
+
expect(reportCalled).toBe(true);
|
|
44
|
+
});
|
|
45
|
+
|
|
46
|
+
it('| should not report unrelated calls', () => {
|
|
47
|
+
let reportCalled: boolean = false;
|
|
48
|
+
const mockContext: any = { report: () => { reportCalled = true; }, options: [] };
|
|
49
|
+
const rule: any = noNativeBrowserDialogsRule.create(mockContext);
|
|
50
|
+
const mockNode: any = { callee: { type: 'Identifier', name: 'doSomething' } };
|
|
51
|
+
rule.CallExpression(mockNode);
|
|
52
|
+
expect(reportCalled).toBe(false);
|
|
53
|
+
});
|
|
54
|
+
});
|
|
@@ -1,84 +1,84 @@
|
|
|
1
|
-
import { Rule } from 'eslint';
|
|
2
|
-
|
|
3
|
-
/**
|
|
4
|
-
* Tiltott natív browser dialog nevek.
|
|
5
|
-
*/
|
|
6
|
-
const NATIVE_DIALOG_NAMES: Set<string> = new Set<string>([
|
|
7
|
-
'alert',
|
|
8
|
-
'confirm',
|
|
9
|
-
'prompt',
|
|
10
|
-
'showModalDialog',
|
|
11
|
-
]);
|
|
12
|
-
|
|
13
|
-
/**
|
|
14
|
-
* Globális objektum nevek, amelyeken a dialog hívás detektálandó.
|
|
15
|
-
*/
|
|
16
|
-
const GLOBAL_NAMES: Set<string> = new Set<string>([
|
|
17
|
-
'window',
|
|
18
|
-
'globalThis',
|
|
19
|
-
'self',
|
|
20
|
-
'top',
|
|
21
|
-
'parent',
|
|
22
|
-
]);
|
|
23
|
-
|
|
24
|
-
/**
|
|
25
|
-
* No-native-browser-dialogs szabály: tilos a natív `window.alert` / `window.confirm` /
|
|
26
|
-
* `window.prompt` / `showModalDialog` hívás. Helyette shared modal komponens (`S_Modal_Component`)
|
|
27
|
-
* használandó. CCAP "no-native-browser-dialogs" review ESLint AST-portja.
|
|
28
|
-
*/
|
|
29
|
-
const rule: Rule.RuleModule = {
|
|
30
|
-
meta: {
|
|
31
|
-
type: 'suggestion',
|
|
32
|
-
docs: {
|
|
33
|
-
description:
|
|
34
|
-
'Forbid native blocking browser dialogs (`alert`, `confirm`, `prompt`, `showModalDialog`); '
|
|
35
|
-
+ 'use shared modal component instead.',
|
|
36
|
-
recommended: true,
|
|
37
|
-
},
|
|
38
|
-
schema: [],
|
|
39
|
-
messages: {
|
|
40
|
-
noNativeDialog:
|
|
41
|
-
'Natív browser dialog `{{name}}` tilos. '
|
|
42
|
-
+ 'Használj shared modal komponenst (pl. `S_Modal_Component` / `app-s-modal`).',
|
|
43
|
-
},
|
|
44
|
-
},
|
|
45
|
-
create(context: Rule.RuleContext): Rule.RuleListener {
|
|
46
|
-
return {
|
|
47
|
-
CallExpression(node: any): void {
|
|
48
|
-
try {
|
|
49
|
-
const callee: any = node.callee;
|
|
50
|
-
if (!callee) {
|
|
51
|
-
return;
|
|
52
|
-
}
|
|
53
|
-
if (callee.type === 'Identifier' && NATIVE_DIALOG_NAMES.has(callee.name)) {
|
|
54
|
-
context.report({
|
|
55
|
-
node: node,
|
|
56
|
-
messageId: 'noNativeDialog',
|
|
57
|
-
data: { name: callee.name },
|
|
58
|
-
});
|
|
59
|
-
return;
|
|
60
|
-
}
|
|
61
|
-
if (
|
|
62
|
-
callee.type === 'MemberExpression'
|
|
63
|
-
&& callee.object
|
|
64
|
-
&& callee.object.type === 'Identifier'
|
|
65
|
-
&& GLOBAL_NAMES.has(callee.object.name)
|
|
66
|
-
&& callee.property
|
|
67
|
-
&& callee.property.type === 'Identifier'
|
|
68
|
-
&& NATIVE_DIALOG_NAMES.has(callee.property.name)
|
|
69
|
-
) {
|
|
70
|
-
context.report({
|
|
71
|
-
node: node,
|
|
72
|
-
messageId: 'noNativeDialog',
|
|
73
|
-
data: { name: `${callee.object.name}.${callee.property.name}` },
|
|
74
|
-
});
|
|
75
|
-
}
|
|
76
|
-
} catch (error) {
|
|
77
|
-
console.error('[no-native-browser-dialogs] Error in CallExpression visitor:', error);
|
|
78
|
-
}
|
|
79
|
-
},
|
|
80
|
-
};
|
|
81
|
-
},
|
|
82
|
-
};
|
|
83
|
-
|
|
84
|
-
export default rule;
|
|
1
|
+
import { Rule } from 'eslint';
|
|
2
|
+
|
|
3
|
+
/**
|
|
4
|
+
* Tiltott natív browser dialog nevek.
|
|
5
|
+
*/
|
|
6
|
+
const NATIVE_DIALOG_NAMES: Set<string> = new Set<string>([
|
|
7
|
+
'alert',
|
|
8
|
+
'confirm',
|
|
9
|
+
'prompt',
|
|
10
|
+
'showModalDialog',
|
|
11
|
+
]);
|
|
12
|
+
|
|
13
|
+
/**
|
|
14
|
+
* Globális objektum nevek, amelyeken a dialog hívás detektálandó.
|
|
15
|
+
*/
|
|
16
|
+
const GLOBAL_NAMES: Set<string> = new Set<string>([
|
|
17
|
+
'window',
|
|
18
|
+
'globalThis',
|
|
19
|
+
'self',
|
|
20
|
+
'top',
|
|
21
|
+
'parent',
|
|
22
|
+
]);
|
|
23
|
+
|
|
24
|
+
/**
|
|
25
|
+
* No-native-browser-dialogs szabály: tilos a natív `window.alert` / `window.confirm` /
|
|
26
|
+
* `window.prompt` / `showModalDialog` hívás. Helyette shared modal komponens (`S_Modal_Component`)
|
|
27
|
+
* használandó. CCAP "no-native-browser-dialogs" review ESLint AST-portja.
|
|
28
|
+
*/
|
|
29
|
+
const rule: Rule.RuleModule = {
|
|
30
|
+
meta: {
|
|
31
|
+
type: 'suggestion',
|
|
32
|
+
docs: {
|
|
33
|
+
description:
|
|
34
|
+
'Forbid native blocking browser dialogs (`alert`, `confirm`, `prompt`, `showModalDialog`); '
|
|
35
|
+
+ 'use shared modal component instead.',
|
|
36
|
+
recommended: true,
|
|
37
|
+
},
|
|
38
|
+
schema: [],
|
|
39
|
+
messages: {
|
|
40
|
+
noNativeDialog:
|
|
41
|
+
'Natív browser dialog `{{name}}` tilos. '
|
|
42
|
+
+ 'Használj shared modal komponenst (pl. `S_Modal_Component` / `app-s-modal`).',
|
|
43
|
+
},
|
|
44
|
+
},
|
|
45
|
+
create(context: Rule.RuleContext): Rule.RuleListener {
|
|
46
|
+
return {
|
|
47
|
+
CallExpression(node: any): void {
|
|
48
|
+
try {
|
|
49
|
+
const callee: any = node.callee;
|
|
50
|
+
if (!callee) {
|
|
51
|
+
return;
|
|
52
|
+
}
|
|
53
|
+
if (callee.type === 'Identifier' && NATIVE_DIALOG_NAMES.has(callee.name)) {
|
|
54
|
+
context.report({
|
|
55
|
+
node: node,
|
|
56
|
+
messageId: 'noNativeDialog',
|
|
57
|
+
data: { name: callee.name },
|
|
58
|
+
});
|
|
59
|
+
return;
|
|
60
|
+
}
|
|
61
|
+
if (
|
|
62
|
+
callee.type === 'MemberExpression'
|
|
63
|
+
&& callee.object
|
|
64
|
+
&& callee.object.type === 'Identifier'
|
|
65
|
+
&& GLOBAL_NAMES.has(callee.object.name)
|
|
66
|
+
&& callee.property
|
|
67
|
+
&& callee.property.type === 'Identifier'
|
|
68
|
+
&& NATIVE_DIALOG_NAMES.has(callee.property.name)
|
|
69
|
+
) {
|
|
70
|
+
context.report({
|
|
71
|
+
node: node,
|
|
72
|
+
messageId: 'noNativeDialog',
|
|
73
|
+
data: { name: `${callee.object.name}.${callee.property.name}` },
|
|
74
|
+
});
|
|
75
|
+
}
|
|
76
|
+
} catch (error) {
|
|
77
|
+
console.error('[no-native-browser-dialogs] Error in CallExpression visitor:', error);
|
|
78
|
+
}
|
|
79
|
+
},
|
|
80
|
+
};
|
|
81
|
+
},
|
|
82
|
+
};
|
|
83
|
+
|
|
84
|
+
export default rule;
|
|
@@ -1,72 +1,72 @@
|
|
|
1
|
-
import noPlainFunctionExportRule from './no-plain-function-export';
|
|
2
|
-
|
|
3
|
-
describe('| no-plain-function-export', () => {
|
|
4
|
-
it('| should be a valid ESLint rule', () => {
|
|
5
|
-
expect(noPlainFunctionExportRule.meta?.type).toBe('suggestion');
|
|
6
|
-
expect(noPlainFunctionExportRule.meta?.messages?.noFunctionExport).toBeDefined();
|
|
7
|
-
expect(noPlainFunctionExportRule.meta?.messages?.noConstFunctionExport).toBeDefined();
|
|
8
|
-
});
|
|
9
|
-
|
|
10
|
-
it('| should skip spec files', () => {
|
|
11
|
-
const mockContext: any = { filename: '/x/foo.spec.ts', report: () => {}, options: [] };
|
|
12
|
-
const result: any = noPlainFunctionExportRule.create(mockContext);
|
|
13
|
-
expect(result.ExportNamedDeclaration).toBeUndefined();
|
|
14
|
-
});
|
|
15
|
-
|
|
16
|
-
it('| should report `export function fn`', () => {
|
|
17
|
-
let reportCalled: boolean = false;
|
|
18
|
-
const mockContext: any = {
|
|
19
|
-
filename: '/x/foo.ts',
|
|
20
|
-
report: (options: any): void => {
|
|
21
|
-
reportCalled = true;
|
|
22
|
-
expect(options.messageId).toBe('noFunctionExport');
|
|
23
|
-
},
|
|
24
|
-
options: [],
|
|
25
|
-
};
|
|
26
|
-
const rule: any = noPlainFunctionExportRule.create(mockContext);
|
|
27
|
-
const mockNode: any = {
|
|
28
|
-
declaration: { type: 'FunctionDeclaration', id: { name: 'helperFn' } },
|
|
29
|
-
};
|
|
30
|
-
rule.ExportNamedDeclaration(mockNode);
|
|
31
|
-
expect(reportCalled).toBe(true);
|
|
32
|
-
});
|
|
33
|
-
|
|
34
|
-
it('| should report `export const fn = () => …`', () => {
|
|
35
|
-
let reportCalled: boolean = false;
|
|
36
|
-
const mockContext: any = {
|
|
37
|
-
filename: '/x/foo.ts',
|
|
38
|
-
report: (options: any): void => {
|
|
39
|
-
reportCalled = true;
|
|
40
|
-
expect(options.messageId).toBe('noConstFunctionExport');
|
|
41
|
-
},
|
|
42
|
-
options: [],
|
|
43
|
-
};
|
|
44
|
-
const rule: any = noPlainFunctionExportRule.create(mockContext);
|
|
45
|
-
const mockNode: any = {
|
|
46
|
-
declaration: {
|
|
47
|
-
type: 'VariableDeclaration',
|
|
48
|
-
declarations: [
|
|
49
|
-
{ id: { name: 'helperFn' }, init: { type: 'ArrowFunctionExpression' } },
|
|
50
|
-
],
|
|
51
|
-
},
|
|
52
|
-
};
|
|
53
|
-
rule.ExportNamedDeclaration(mockNode);
|
|
54
|
-
expect(reportCalled).toBe(true);
|
|
55
|
-
});
|
|
56
|
-
|
|
57
|
-
it('| should not report `export const data = { … }`', () => {
|
|
58
|
-
let reportCalled: boolean = false;
|
|
59
|
-
const mockContext: any = { filename: '/x/foo.ts', report: () => { reportCalled = true; }, options: [] };
|
|
60
|
-
const rule: any = noPlainFunctionExportRule.create(mockContext);
|
|
61
|
-
const mockNode: any = {
|
|
62
|
-
declaration: {
|
|
63
|
-
type: 'VariableDeclaration',
|
|
64
|
-
declarations: [
|
|
65
|
-
{ id: { name: 'data' }, init: { type: 'ObjectExpression' } },
|
|
66
|
-
],
|
|
67
|
-
},
|
|
68
|
-
};
|
|
69
|
-
rule.ExportNamedDeclaration(mockNode);
|
|
70
|
-
expect(reportCalled).toBe(false);
|
|
71
|
-
});
|
|
72
|
-
});
|
|
1
|
+
import noPlainFunctionExportRule from './no-plain-function-export';
|
|
2
|
+
|
|
3
|
+
describe('| no-plain-function-export', () => {
|
|
4
|
+
it('| should be a valid ESLint rule', () => {
|
|
5
|
+
expect(noPlainFunctionExportRule.meta?.type).toBe('suggestion');
|
|
6
|
+
expect(noPlainFunctionExportRule.meta?.messages?.noFunctionExport).toBeDefined();
|
|
7
|
+
expect(noPlainFunctionExportRule.meta?.messages?.noConstFunctionExport).toBeDefined();
|
|
8
|
+
});
|
|
9
|
+
|
|
10
|
+
it('| should skip spec files', () => {
|
|
11
|
+
const mockContext: any = { filename: '/x/foo.spec.ts', report: () => {}, options: [] };
|
|
12
|
+
const result: any = noPlainFunctionExportRule.create(mockContext);
|
|
13
|
+
expect(result.ExportNamedDeclaration).toBeUndefined();
|
|
14
|
+
});
|
|
15
|
+
|
|
16
|
+
it('| should report `export function fn`', () => {
|
|
17
|
+
let reportCalled: boolean = false;
|
|
18
|
+
const mockContext: any = {
|
|
19
|
+
filename: '/x/foo.ts',
|
|
20
|
+
report: (options: any): void => {
|
|
21
|
+
reportCalled = true;
|
|
22
|
+
expect(options.messageId).toBe('noFunctionExport');
|
|
23
|
+
},
|
|
24
|
+
options: [],
|
|
25
|
+
};
|
|
26
|
+
const rule: any = noPlainFunctionExportRule.create(mockContext);
|
|
27
|
+
const mockNode: any = {
|
|
28
|
+
declaration: { type: 'FunctionDeclaration', id: { name: 'helperFn' } },
|
|
29
|
+
};
|
|
30
|
+
rule.ExportNamedDeclaration(mockNode);
|
|
31
|
+
expect(reportCalled).toBe(true);
|
|
32
|
+
});
|
|
33
|
+
|
|
34
|
+
it('| should report `export const fn = () => …`', () => {
|
|
35
|
+
let reportCalled: boolean = false;
|
|
36
|
+
const mockContext: any = {
|
|
37
|
+
filename: '/x/foo.ts',
|
|
38
|
+
report: (options: any): void => {
|
|
39
|
+
reportCalled = true;
|
|
40
|
+
expect(options.messageId).toBe('noConstFunctionExport');
|
|
41
|
+
},
|
|
42
|
+
options: [],
|
|
43
|
+
};
|
|
44
|
+
const rule: any = noPlainFunctionExportRule.create(mockContext);
|
|
45
|
+
const mockNode: any = {
|
|
46
|
+
declaration: {
|
|
47
|
+
type: 'VariableDeclaration',
|
|
48
|
+
declarations: [
|
|
49
|
+
{ id: { name: 'helperFn' }, init: { type: 'ArrowFunctionExpression' } },
|
|
50
|
+
],
|
|
51
|
+
},
|
|
52
|
+
};
|
|
53
|
+
rule.ExportNamedDeclaration(mockNode);
|
|
54
|
+
expect(reportCalled).toBe(true);
|
|
55
|
+
});
|
|
56
|
+
|
|
57
|
+
it('| should not report `export const data = { … }`', () => {
|
|
58
|
+
let reportCalled: boolean = false;
|
|
59
|
+
const mockContext: any = { filename: '/x/foo.ts', report: () => { reportCalled = true; }, options: [] };
|
|
60
|
+
const rule: any = noPlainFunctionExportRule.create(mockContext);
|
|
61
|
+
const mockNode: any = {
|
|
62
|
+
declaration: {
|
|
63
|
+
type: 'VariableDeclaration',
|
|
64
|
+
declarations: [
|
|
65
|
+
{ id: { name: 'data' }, init: { type: 'ObjectExpression' } },
|
|
66
|
+
],
|
|
67
|
+
},
|
|
68
|
+
};
|
|
69
|
+
rule.ExportNamedDeclaration(mockNode);
|
|
70
|
+
expect(reportCalled).toBe(false);
|
|
71
|
+
});
|
|
72
|
+
});
|
|
@@ -1,73 +1,73 @@
|
|
|
1
|
-
import { Rule } from 'eslint';
|
|
2
|
-
|
|
3
|
-
/**
|
|
4
|
-
* No-plain-function-export szabály: tilos top-level `export function fn()` és
|
|
5
|
-
* `export const fn = () => …` arrow/function deklaráció.
|
|
6
|
-
* Megosztott logika static `_Util` / `UtilService` osztály metódusokban kell legyen.
|
|
7
|
-
* CCAP "no-plain-function-export" review ESLint AST-portja.
|
|
8
|
-
*/
|
|
9
|
-
const rule: Rule.RuleModule = {
|
|
10
|
-
meta: {
|
|
11
|
-
type: 'suggestion',
|
|
12
|
-
docs: {
|
|
13
|
-
description:
|
|
14
|
-
'Forbid top-level `export function` and `export const fn = () => …`; '
|
|
15
|
-
+ 'use static methods on `_Util` / `UtilService` classes.',
|
|
16
|
-
recommended: true,
|
|
17
|
-
},
|
|
18
|
-
schema: [],
|
|
19
|
-
messages: {
|
|
20
|
-
noFunctionExport:
|
|
21
|
-
'Top-level `export function {{name}}` tilos. '
|
|
22
|
-
+ 'Tedd static metódusba `_Util` / `UtilService` osztályon belül.',
|
|
23
|
-
noConstFunctionExport:
|
|
24
|
-
'Top-level `export const {{name}} = () => …` tilos. '
|
|
25
|
-
+ 'Tedd static metódusba `_Util` / `UtilService` osztályon belül.',
|
|
26
|
-
},
|
|
27
|
-
},
|
|
28
|
-
create(context: Rule.RuleContext): Rule.RuleListener {
|
|
29
|
-
const filename: string = context.filename ?? '';
|
|
30
|
-
if (filename.endsWith('.spec.ts') || filename.endsWith('.test.ts')) {
|
|
31
|
-
return {};
|
|
32
|
-
}
|
|
33
|
-
|
|
34
|
-
return {
|
|
35
|
-
ExportNamedDeclaration(node: any): void {
|
|
36
|
-
try {
|
|
37
|
-
const decl: any = node.declaration;
|
|
38
|
-
if (!decl) {
|
|
39
|
-
return;
|
|
40
|
-
}
|
|
41
|
-
if (decl.type === 'FunctionDeclaration' && decl.id && decl.id.name) {
|
|
42
|
-
context.report({
|
|
43
|
-
node: decl,
|
|
44
|
-
messageId: 'noFunctionExport',
|
|
45
|
-
data: { name: decl.id.name },
|
|
46
|
-
});
|
|
47
|
-
return;
|
|
48
|
-
}
|
|
49
|
-
if (decl.type === 'VariableDeclaration' && Array.isArray(decl.declarations)) {
|
|
50
|
-
for (const declarator of decl.declarations) {
|
|
51
|
-
const init: any = declarator.init;
|
|
52
|
-
if (!init) {
|
|
53
|
-
continue;
|
|
54
|
-
}
|
|
55
|
-
if (init.type === 'ArrowFunctionExpression' || init.type === 'FunctionExpression') {
|
|
56
|
-
const name: string = (declarator.id && declarator.id.name) ? declarator.id.name : 'anonymous';
|
|
57
|
-
context.report({
|
|
58
|
-
node: declarator,
|
|
59
|
-
messageId: 'noConstFunctionExport',
|
|
60
|
-
data: { name: name },
|
|
61
|
-
});
|
|
62
|
-
}
|
|
63
|
-
}
|
|
64
|
-
}
|
|
65
|
-
} catch (error) {
|
|
66
|
-
console.error('[no-plain-function-export] Error in ExportNamedDeclaration visitor:', error);
|
|
67
|
-
}
|
|
68
|
-
},
|
|
69
|
-
};
|
|
70
|
-
},
|
|
71
|
-
};
|
|
72
|
-
|
|
73
|
-
export default rule;
|
|
1
|
+
import { Rule } from 'eslint';
|
|
2
|
+
|
|
3
|
+
/**
|
|
4
|
+
* No-plain-function-export szabály: tilos top-level `export function fn()` és
|
|
5
|
+
* `export const fn = () => …` arrow/function deklaráció.
|
|
6
|
+
* Megosztott logika static `_Util` / `UtilService` osztály metódusokban kell legyen.
|
|
7
|
+
* CCAP "no-plain-function-export" review ESLint AST-portja.
|
|
8
|
+
*/
|
|
9
|
+
const rule: Rule.RuleModule = {
|
|
10
|
+
meta: {
|
|
11
|
+
type: 'suggestion',
|
|
12
|
+
docs: {
|
|
13
|
+
description:
|
|
14
|
+
'Forbid top-level `export function` and `export const fn = () => …`; '
|
|
15
|
+
+ 'use static methods on `_Util` / `UtilService` classes.',
|
|
16
|
+
recommended: true,
|
|
17
|
+
},
|
|
18
|
+
schema: [],
|
|
19
|
+
messages: {
|
|
20
|
+
noFunctionExport:
|
|
21
|
+
'Top-level `export function {{name}}` tilos. '
|
|
22
|
+
+ 'Tedd static metódusba `_Util` / `UtilService` osztályon belül.',
|
|
23
|
+
noConstFunctionExport:
|
|
24
|
+
'Top-level `export const {{name}} = () => …` tilos. '
|
|
25
|
+
+ 'Tedd static metódusba `_Util` / `UtilService` osztályon belül.',
|
|
26
|
+
},
|
|
27
|
+
},
|
|
28
|
+
create(context: Rule.RuleContext): Rule.RuleListener {
|
|
29
|
+
const filename: string = context.filename ?? '';
|
|
30
|
+
if (filename.endsWith('.spec.ts') || filename.endsWith('.test.ts')) {
|
|
31
|
+
return {};
|
|
32
|
+
}
|
|
33
|
+
|
|
34
|
+
return {
|
|
35
|
+
ExportNamedDeclaration(node: any): void {
|
|
36
|
+
try {
|
|
37
|
+
const decl: any = node.declaration;
|
|
38
|
+
if (!decl) {
|
|
39
|
+
return;
|
|
40
|
+
}
|
|
41
|
+
if (decl.type === 'FunctionDeclaration' && decl.id && decl.id.name) {
|
|
42
|
+
context.report({
|
|
43
|
+
node: decl,
|
|
44
|
+
messageId: 'noFunctionExport',
|
|
45
|
+
data: { name: decl.id.name },
|
|
46
|
+
});
|
|
47
|
+
return;
|
|
48
|
+
}
|
|
49
|
+
if (decl.type === 'VariableDeclaration' && Array.isArray(decl.declarations)) {
|
|
50
|
+
for (const declarator of decl.declarations) {
|
|
51
|
+
const init: any = declarator.init;
|
|
52
|
+
if (!init) {
|
|
53
|
+
continue;
|
|
54
|
+
}
|
|
55
|
+
if (init.type === 'ArrowFunctionExpression' || init.type === 'FunctionExpression') {
|
|
56
|
+
const name: string = (declarator.id && declarator.id.name) ? declarator.id.name : 'anonymous';
|
|
57
|
+
context.report({
|
|
58
|
+
node: declarator,
|
|
59
|
+
messageId: 'noConstFunctionExport',
|
|
60
|
+
data: { name: name },
|
|
61
|
+
});
|
|
62
|
+
}
|
|
63
|
+
}
|
|
64
|
+
}
|
|
65
|
+
} catch (error) {
|
|
66
|
+
console.error('[no-plain-function-export] Error in ExportNamedDeclaration visitor:', error);
|
|
67
|
+
}
|
|
68
|
+
},
|
|
69
|
+
};
|
|
70
|
+
},
|
|
71
|
+
};
|
|
72
|
+
|
|
73
|
+
export default rule;
|