@futdevpro/dynamo-eslint 1.15.22 → 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.d.ts.map +1 -1
- package/build/plugin/rules/naming-patterns.js +366 -350
- package/build/plugin/rules/naming-patterns.js.map +1 -1
- 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/dynamo-fix.js.map +1 -1
- 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 -137
- 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 -123
- 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 -127
- package/src/plugin/rules/naming-patterns.ts +426 -408
- 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 -249
- package/src/scripts/validate-naming.ts +204 -176
- package/src/scripts/validator-cli.integration.spec.ts +96 -0
- package/tsconfig.json +31 -31
|
@@ -1,100 +1,100 @@
|
|
|
1
|
-
import { Rule } from 'eslint';
|
|
2
|
-
|
|
3
|
-
/**
|
|
4
|
-
* Parallel date field nevek – ezeket egy DataModel sem tartalmazhatja,
|
|
5
|
-
* mert a `DyFM_Metadata` `__created` / `__updated` mezőit kell használni.
|
|
6
|
-
*/
|
|
7
|
-
const PARALLEL_DATE_FIELD_NAMES: Set<string> = new Set<string>([
|
|
8
|
-
'firstRegisteredAt',
|
|
9
|
-
'lastSeenAt',
|
|
10
|
-
'createdAt',
|
|
11
|
-
'updatedAt',
|
|
12
|
-
'registeredAt',
|
|
13
|
-
'seenAt',
|
|
14
|
-
'modifiedAt',
|
|
15
|
-
'lastModified',
|
|
16
|
-
'lastUpdated',
|
|
17
|
-
'dateCreated',
|
|
18
|
-
'dateUpdated',
|
|
19
|
-
]);
|
|
20
|
-
|
|
21
|
-
/**
|
|
22
|
-
* Data-model-extends szabály: `*.data-model.ts` osztályoknak `DyFM_Metadata`-t (vagy bármely `DyFM_*`-ot)
|
|
23
|
-
* kell kiterjeszteniük, és tilos párhuzamos dátum mezőket definiálni.
|
|
24
|
-
* CCAP "data-model-extends" review ESLint AST-portja.
|
|
25
|
-
*/
|
|
26
|
-
const rule: Rule.RuleModule = {
|
|
27
|
-
meta: {
|
|
28
|
-
type: 'suggestion',
|
|
29
|
-
docs: {
|
|
30
|
-
description: '`*.data-model.ts` classes must extend `DyFM_*`; no parallel date fields allowed.',
|
|
31
|
-
recommended: true,
|
|
32
|
-
},
|
|
33
|
-
schema: [],
|
|
34
|
-
messages: {
|
|
35
|
-
missingExtends:
|
|
36
|
-
'`{{className}}` nem terjeszti ki a `DyFM_Metadata`-t (vagy `DyFM_*` base-t). '
|
|
37
|
-
+ 'DataModel osztálynak `extends DyFM_Metadata` szükséges.',
|
|
38
|
-
parallelDateField:
|
|
39
|
-
'Párhuzamos dátum mező `{{fieldName}}` tilos. '
|
|
40
|
-
+ 'Használd a `DyFM_Metadata`-ból örökölt `__created` / `__updated` mezőket.',
|
|
41
|
-
},
|
|
42
|
-
},
|
|
43
|
-
create(context: Rule.RuleContext): Rule.RuleListener {
|
|
44
|
-
const filename: string = context.filename ?? '';
|
|
45
|
-
if (!filename.endsWith('.data-model.ts')) {
|
|
46
|
-
return {};
|
|
47
|
-
}
|
|
48
|
-
|
|
49
|
-
return {
|
|
50
|
-
ClassDeclaration(node: any): void {
|
|
51
|
-
try {
|
|
52
|
-
const superClass: any = node.superClass;
|
|
53
|
-
let extendsName: string | null = null;
|
|
54
|
-
if (superClass && superClass.type === 'Identifier') {
|
|
55
|
-
extendsName = superClass.name;
|
|
56
|
-
} else if (
|
|
57
|
-
superClass
|
|
58
|
-
&& superClass.type === 'MemberExpression'
|
|
59
|
-
&& superClass.property
|
|
60
|
-
&& superClass.property.name
|
|
61
|
-
) {
|
|
62
|
-
extendsName = superClass.property.name;
|
|
63
|
-
}
|
|
64
|
-
const isDyFmExtends: boolean = extendsName !== null && extendsName.startsWith('DyFM_');
|
|
65
|
-
if (!isDyFmExtends) {
|
|
66
|
-
const className: string = (node.id && node.id.name) ? node.id.name : 'anonymous';
|
|
67
|
-
context.report({
|
|
68
|
-
node: node.id ?? node,
|
|
69
|
-
messageId: 'missingExtends',
|
|
70
|
-
data: { className: className },
|
|
71
|
-
});
|
|
72
|
-
}
|
|
73
|
-
|
|
74
|
-
const body: any = node.body;
|
|
75
|
-
if (!body || !Array.isArray(body.body)) {
|
|
76
|
-
return;
|
|
77
|
-
}
|
|
78
|
-
for (const member of body.body) {
|
|
79
|
-
if (member.type !== 'PropertyDefinition') {
|
|
80
|
-
continue;
|
|
81
|
-
}
|
|
82
|
-
const key: any = member.key;
|
|
83
|
-
const fieldName: string | null = (key && key.type === 'Identifier' && key.name) ? key.name : null;
|
|
84
|
-
if (fieldName !== null && PARALLEL_DATE_FIELD_NAMES.has(fieldName)) {
|
|
85
|
-
context.report({
|
|
86
|
-
node: member,
|
|
87
|
-
messageId: 'parallelDateField',
|
|
88
|
-
data: { fieldName: fieldName },
|
|
89
|
-
});
|
|
90
|
-
}
|
|
91
|
-
}
|
|
92
|
-
} catch (error) {
|
|
93
|
-
console.error('[data-model-extends] Error in ClassDeclaration visitor:', error);
|
|
94
|
-
}
|
|
95
|
-
},
|
|
96
|
-
};
|
|
97
|
-
},
|
|
98
|
-
};
|
|
99
|
-
|
|
100
|
-
export default rule;
|
|
1
|
+
import { Rule } from 'eslint';
|
|
2
|
+
|
|
3
|
+
/**
|
|
4
|
+
* Parallel date field nevek – ezeket egy DataModel sem tartalmazhatja,
|
|
5
|
+
* mert a `DyFM_Metadata` `__created` / `__updated` mezőit kell használni.
|
|
6
|
+
*/
|
|
7
|
+
const PARALLEL_DATE_FIELD_NAMES: Set<string> = new Set<string>([
|
|
8
|
+
'firstRegisteredAt',
|
|
9
|
+
'lastSeenAt',
|
|
10
|
+
'createdAt',
|
|
11
|
+
'updatedAt',
|
|
12
|
+
'registeredAt',
|
|
13
|
+
'seenAt',
|
|
14
|
+
'modifiedAt',
|
|
15
|
+
'lastModified',
|
|
16
|
+
'lastUpdated',
|
|
17
|
+
'dateCreated',
|
|
18
|
+
'dateUpdated',
|
|
19
|
+
]);
|
|
20
|
+
|
|
21
|
+
/**
|
|
22
|
+
* Data-model-extends szabály: `*.data-model.ts` osztályoknak `DyFM_Metadata`-t (vagy bármely `DyFM_*`-ot)
|
|
23
|
+
* kell kiterjeszteniük, és tilos párhuzamos dátum mezőket definiálni.
|
|
24
|
+
* CCAP "data-model-extends" review ESLint AST-portja.
|
|
25
|
+
*/
|
|
26
|
+
const rule: Rule.RuleModule = {
|
|
27
|
+
meta: {
|
|
28
|
+
type: 'suggestion',
|
|
29
|
+
docs: {
|
|
30
|
+
description: '`*.data-model.ts` classes must extend `DyFM_*`; no parallel date fields allowed.',
|
|
31
|
+
recommended: true,
|
|
32
|
+
},
|
|
33
|
+
schema: [],
|
|
34
|
+
messages: {
|
|
35
|
+
missingExtends:
|
|
36
|
+
'`{{className}}` nem terjeszti ki a `DyFM_Metadata`-t (vagy `DyFM_*` base-t). '
|
|
37
|
+
+ 'DataModel osztálynak `extends DyFM_Metadata` szükséges.',
|
|
38
|
+
parallelDateField:
|
|
39
|
+
'Párhuzamos dátum mező `{{fieldName}}` tilos. '
|
|
40
|
+
+ 'Használd a `DyFM_Metadata`-ból örökölt `__created` / `__updated` mezőket.',
|
|
41
|
+
},
|
|
42
|
+
},
|
|
43
|
+
create(context: Rule.RuleContext): Rule.RuleListener {
|
|
44
|
+
const filename: string = context.filename ?? '';
|
|
45
|
+
if (!filename.endsWith('.data-model.ts')) {
|
|
46
|
+
return {};
|
|
47
|
+
}
|
|
48
|
+
|
|
49
|
+
return {
|
|
50
|
+
ClassDeclaration(node: any): void {
|
|
51
|
+
try {
|
|
52
|
+
const superClass: any = node.superClass;
|
|
53
|
+
let extendsName: string | null = null;
|
|
54
|
+
if (superClass && superClass.type === 'Identifier') {
|
|
55
|
+
extendsName = superClass.name;
|
|
56
|
+
} else if (
|
|
57
|
+
superClass
|
|
58
|
+
&& superClass.type === 'MemberExpression'
|
|
59
|
+
&& superClass.property
|
|
60
|
+
&& superClass.property.name
|
|
61
|
+
) {
|
|
62
|
+
extendsName = superClass.property.name;
|
|
63
|
+
}
|
|
64
|
+
const isDyFmExtends: boolean = extendsName !== null && extendsName.startsWith('DyFM_');
|
|
65
|
+
if (!isDyFmExtends) {
|
|
66
|
+
const className: string = (node.id && node.id.name) ? node.id.name : 'anonymous';
|
|
67
|
+
context.report({
|
|
68
|
+
node: node.id ?? node,
|
|
69
|
+
messageId: 'missingExtends',
|
|
70
|
+
data: { className: className },
|
|
71
|
+
});
|
|
72
|
+
}
|
|
73
|
+
|
|
74
|
+
const body: any = node.body;
|
|
75
|
+
if (!body || !Array.isArray(body.body)) {
|
|
76
|
+
return;
|
|
77
|
+
}
|
|
78
|
+
for (const member of body.body) {
|
|
79
|
+
if (member.type !== 'PropertyDefinition') {
|
|
80
|
+
continue;
|
|
81
|
+
}
|
|
82
|
+
const key: any = member.key;
|
|
83
|
+
const fieldName: string | null = (key && key.type === 'Identifier' && key.name) ? key.name : null;
|
|
84
|
+
if (fieldName !== null && PARALLEL_DATE_FIELD_NAMES.has(fieldName)) {
|
|
85
|
+
context.report({
|
|
86
|
+
node: member,
|
|
87
|
+
messageId: 'parallelDateField',
|
|
88
|
+
data: { fieldName: fieldName },
|
|
89
|
+
});
|
|
90
|
+
}
|
|
91
|
+
}
|
|
92
|
+
} catch (error) {
|
|
93
|
+
console.error('[data-model-extends] Error in ClassDeclaration visitor:', error);
|
|
94
|
+
}
|
|
95
|
+
},
|
|
96
|
+
};
|
|
97
|
+
},
|
|
98
|
+
};
|
|
99
|
+
|
|
100
|
+
export default rule;
|
|
@@ -1,28 +1,28 @@
|
|
|
1
|
-
import noAnyTypeRule from './no-any-type';
|
|
2
|
-
|
|
3
|
-
describe('| no-any-type', () => {
|
|
4
|
-
it('| should be a valid ESLint rule', () => {
|
|
5
|
-
expect(noAnyTypeRule.meta?.type).toBe('suggestion');
|
|
6
|
-
expect(noAnyTypeRule.meta?.messages?.noAny).toBeDefined();
|
|
7
|
-
});
|
|
8
|
-
|
|
9
|
-
it('| should have create function returning visitor object', () => {
|
|
10
|
-
const mockContext: any = { report: () => {}, options: [] };
|
|
11
|
-
const result: any = noAnyTypeRule.create(mockContext);
|
|
12
|
-
expect(typeof result.TSAnyKeyword).toBe('function');
|
|
13
|
-
});
|
|
14
|
-
|
|
15
|
-
it('| should report on `any` keyword', () => {
|
|
16
|
-
let reportCalled: boolean = false;
|
|
17
|
-
const mockContext: any = {
|
|
18
|
-
report: (options: any): void => {
|
|
19
|
-
reportCalled = true;
|
|
20
|
-
expect(options.messageId).toBe('noAny');
|
|
21
|
-
},
|
|
22
|
-
options: [],
|
|
23
|
-
};
|
|
24
|
-
const rule: any = noAnyTypeRule.create(mockContext);
|
|
25
|
-
rule.TSAnyKeyword({});
|
|
26
|
-
expect(reportCalled).toBe(true);
|
|
27
|
-
});
|
|
28
|
-
});
|
|
1
|
+
import noAnyTypeRule from './no-any-type';
|
|
2
|
+
|
|
3
|
+
describe('| no-any-type', () => {
|
|
4
|
+
it('| should be a valid ESLint rule', () => {
|
|
5
|
+
expect(noAnyTypeRule.meta?.type).toBe('suggestion');
|
|
6
|
+
expect(noAnyTypeRule.meta?.messages?.noAny).toBeDefined();
|
|
7
|
+
});
|
|
8
|
+
|
|
9
|
+
it('| should have create function returning visitor object', () => {
|
|
10
|
+
const mockContext: any = { report: () => {}, options: [] };
|
|
11
|
+
const result: any = noAnyTypeRule.create(mockContext);
|
|
12
|
+
expect(typeof result.TSAnyKeyword).toBe('function');
|
|
13
|
+
});
|
|
14
|
+
|
|
15
|
+
it('| should report on `any` keyword', () => {
|
|
16
|
+
let reportCalled: boolean = false;
|
|
17
|
+
const mockContext: any = {
|
|
18
|
+
report: (options: any): void => {
|
|
19
|
+
reportCalled = true;
|
|
20
|
+
expect(options.messageId).toBe('noAny');
|
|
21
|
+
},
|
|
22
|
+
options: [],
|
|
23
|
+
};
|
|
24
|
+
const rule: any = noAnyTypeRule.create(mockContext);
|
|
25
|
+
rule.TSAnyKeyword({});
|
|
26
|
+
expect(reportCalled).toBe(true);
|
|
27
|
+
});
|
|
28
|
+
});
|
|
@@ -1,36 +1,36 @@
|
|
|
1
|
-
import { Rule } from 'eslint';
|
|
2
|
-
|
|
3
|
-
/**
|
|
4
|
-
* No-any-type szabály: `any` típus annotáció tilos.
|
|
5
|
-
* CCAP "no-any-type" review ESLint AST-portja.
|
|
6
|
-
* Lefedi a `: any`, `<any>`, `as any`, `Array<any>`, `Promise<any>` eseteket egyaránt.
|
|
7
|
-
*/
|
|
8
|
-
const rule: Rule.RuleModule = {
|
|
9
|
-
meta: {
|
|
10
|
-
type: 'suggestion',
|
|
11
|
-
docs: {
|
|
12
|
-
description: 'Forbid `any` type annotation in TypeScript files.',
|
|
13
|
-
recommended: true,
|
|
14
|
-
},
|
|
15
|
-
schema: [],
|
|
16
|
-
messages: {
|
|
17
|
-
noAny: '`any` típus tilos. Adj meg konkrét típust vagy használj `unknown`-t.',
|
|
18
|
-
},
|
|
19
|
-
},
|
|
20
|
-
create(context: Rule.RuleContext): Rule.RuleListener {
|
|
21
|
-
return {
|
|
22
|
-
TSAnyKeyword(node: any): void {
|
|
23
|
-
try {
|
|
24
|
-
context.report({
|
|
25
|
-
node: node,
|
|
26
|
-
messageId: 'noAny',
|
|
27
|
-
});
|
|
28
|
-
} catch (error) {
|
|
29
|
-
console.error('[no-any-type] Error in TSAnyKeyword visitor:', error);
|
|
30
|
-
}
|
|
31
|
-
},
|
|
32
|
-
};
|
|
33
|
-
},
|
|
34
|
-
};
|
|
35
|
-
|
|
36
|
-
export default rule;
|
|
1
|
+
import { Rule } from 'eslint';
|
|
2
|
+
|
|
3
|
+
/**
|
|
4
|
+
* No-any-type szabály: `any` típus annotáció tilos.
|
|
5
|
+
* CCAP "no-any-type" review ESLint AST-portja.
|
|
6
|
+
* Lefedi a `: any`, `<any>`, `as any`, `Array<any>`, `Promise<any>` eseteket egyaránt.
|
|
7
|
+
*/
|
|
8
|
+
const rule: Rule.RuleModule = {
|
|
9
|
+
meta: {
|
|
10
|
+
type: 'suggestion',
|
|
11
|
+
docs: {
|
|
12
|
+
description: 'Forbid `any` type annotation in TypeScript files.',
|
|
13
|
+
recommended: true,
|
|
14
|
+
},
|
|
15
|
+
schema: [],
|
|
16
|
+
messages: {
|
|
17
|
+
noAny: '`any` típus tilos. Adj meg konkrét típust vagy használj `unknown`-t.',
|
|
18
|
+
},
|
|
19
|
+
},
|
|
20
|
+
create(context: Rule.RuleContext): Rule.RuleListener {
|
|
21
|
+
return {
|
|
22
|
+
TSAnyKeyword(node: any): void {
|
|
23
|
+
try {
|
|
24
|
+
context.report({
|
|
25
|
+
node: node,
|
|
26
|
+
messageId: 'noAny',
|
|
27
|
+
});
|
|
28
|
+
} catch (error) {
|
|
29
|
+
console.error('[no-any-type] Error in TSAnyKeyword visitor:', error);
|
|
30
|
+
}
|
|
31
|
+
},
|
|
32
|
+
};
|
|
33
|
+
},
|
|
34
|
+
};
|
|
35
|
+
|
|
36
|
+
export default rule;
|
|
@@ -1,51 +1,51 @@
|
|
|
1
|
-
import noAsCastRule from './no-as-cast';
|
|
2
|
-
|
|
3
|
-
describe('| no-as-cast', () => {
|
|
4
|
-
it('| should be a valid ESLint rule', () => {
|
|
5
|
-
expect(noAsCastRule.meta?.type).toBe('suggestion');
|
|
6
|
-
expect(noAsCastRule.meta?.messages?.noAsCast).toBeDefined();
|
|
7
|
-
});
|
|
8
|
-
|
|
9
|
-
it('| should have create function returning visitor object', () => {
|
|
10
|
-
const mockContext: any = {
|
|
11
|
-
report: () => {},
|
|
12
|
-
options: [],
|
|
13
|
-
sourceCode: { getText: () => 'X' },
|
|
14
|
-
};
|
|
15
|
-
const result: any = noAsCastRule.create(mockContext);
|
|
16
|
-
expect(typeof result.TSAsExpression).toBe('function');
|
|
17
|
-
});
|
|
18
|
-
|
|
19
|
-
it('| should not report `as const`', () => {
|
|
20
|
-
let reportCalled: boolean = false;
|
|
21
|
-
const mockContext: any = {
|
|
22
|
-
report: () => { reportCalled = true; },
|
|
23
|
-
options: [],
|
|
24
|
-
sourceCode: { getText: () => 'const' },
|
|
25
|
-
};
|
|
26
|
-
const rule: any = noAsCastRule.create(mockContext);
|
|
27
|
-
const mockNode: any = {
|
|
28
|
-
typeAnnotation: { type: 'TSTypeReference', typeName: { type: 'Identifier', name: 'const' } },
|
|
29
|
-
};
|
|
30
|
-
rule.TSAsExpression(mockNode);
|
|
31
|
-
expect(reportCalled).toBe(false);
|
|
32
|
-
});
|
|
33
|
-
|
|
34
|
-
it('| should report `as SomeType`', () => {
|
|
35
|
-
let reportCalled: boolean = false;
|
|
36
|
-
const mockContext: any = {
|
|
37
|
-
report: (options: any): void => {
|
|
38
|
-
reportCalled = true;
|
|
39
|
-
expect(options.messageId).toBe('noAsCast');
|
|
40
|
-
},
|
|
41
|
-
options: [],
|
|
42
|
-
sourceCode: { getText: () => 'MyType' },
|
|
43
|
-
};
|
|
44
|
-
const rule: any = noAsCastRule.create(mockContext);
|
|
45
|
-
const mockNode: any = {
|
|
46
|
-
typeAnnotation: { type: 'TSTypeReference', typeName: { type: 'Identifier', name: 'MyType' } },
|
|
47
|
-
};
|
|
48
|
-
rule.TSAsExpression(mockNode);
|
|
49
|
-
expect(reportCalled).toBe(true);
|
|
50
|
-
});
|
|
51
|
-
});
|
|
1
|
+
import noAsCastRule from './no-as-cast';
|
|
2
|
+
|
|
3
|
+
describe('| no-as-cast', () => {
|
|
4
|
+
it('| should be a valid ESLint rule', () => {
|
|
5
|
+
expect(noAsCastRule.meta?.type).toBe('suggestion');
|
|
6
|
+
expect(noAsCastRule.meta?.messages?.noAsCast).toBeDefined();
|
|
7
|
+
});
|
|
8
|
+
|
|
9
|
+
it('| should have create function returning visitor object', () => {
|
|
10
|
+
const mockContext: any = {
|
|
11
|
+
report: () => {},
|
|
12
|
+
options: [],
|
|
13
|
+
sourceCode: { getText: () => 'X' },
|
|
14
|
+
};
|
|
15
|
+
const result: any = noAsCastRule.create(mockContext);
|
|
16
|
+
expect(typeof result.TSAsExpression).toBe('function');
|
|
17
|
+
});
|
|
18
|
+
|
|
19
|
+
it('| should not report `as const`', () => {
|
|
20
|
+
let reportCalled: boolean = false;
|
|
21
|
+
const mockContext: any = {
|
|
22
|
+
report: () => { reportCalled = true; },
|
|
23
|
+
options: [],
|
|
24
|
+
sourceCode: { getText: () => 'const' },
|
|
25
|
+
};
|
|
26
|
+
const rule: any = noAsCastRule.create(mockContext);
|
|
27
|
+
const mockNode: any = {
|
|
28
|
+
typeAnnotation: { type: 'TSTypeReference', typeName: { type: 'Identifier', name: 'const' } },
|
|
29
|
+
};
|
|
30
|
+
rule.TSAsExpression(mockNode);
|
|
31
|
+
expect(reportCalled).toBe(false);
|
|
32
|
+
});
|
|
33
|
+
|
|
34
|
+
it('| should report `as SomeType`', () => {
|
|
35
|
+
let reportCalled: boolean = false;
|
|
36
|
+
const mockContext: any = {
|
|
37
|
+
report: (options: any): void => {
|
|
38
|
+
reportCalled = true;
|
|
39
|
+
expect(options.messageId).toBe('noAsCast');
|
|
40
|
+
},
|
|
41
|
+
options: [],
|
|
42
|
+
sourceCode: { getText: () => 'MyType' },
|
|
43
|
+
};
|
|
44
|
+
const rule: any = noAsCastRule.create(mockContext);
|
|
45
|
+
const mockNode: any = {
|
|
46
|
+
typeAnnotation: { type: 'TSTypeReference', typeName: { type: 'Identifier', name: 'MyType' } },
|
|
47
|
+
};
|
|
48
|
+
rule.TSAsExpression(mockNode);
|
|
49
|
+
expect(reportCalled).toBe(true);
|
|
50
|
+
});
|
|
51
|
+
});
|
|
@@ -1,51 +1,51 @@
|
|
|
1
|
-
import { Rule } from 'eslint';
|
|
2
|
-
|
|
3
|
-
/**
|
|
4
|
-
* No-as-cast szabály: TypeScript `as` type cast tilos.
|
|
5
|
-
* CCAP "no-as-cast" review ESLint AST-portja.
|
|
6
|
-
* Megjegyzés: az `as const` kifejezést megengedjük (immutable literal-ként standard).
|
|
7
|
-
*/
|
|
8
|
-
const rule: Rule.RuleModule = {
|
|
9
|
-
meta: {
|
|
10
|
-
type: 'suggestion',
|
|
11
|
-
docs: {
|
|
12
|
-
description: 'Forbid TypeScript `as` type casts (except `as const`); design issue indicator.',
|
|
13
|
-
recommended: true,
|
|
14
|
-
},
|
|
15
|
-
schema: [],
|
|
16
|
-
messages: {
|
|
17
|
-
noAsCast: '`as {{typeName}}` type cast tilos. Tervezési problémát jelezhet — használj rendes típust.',
|
|
18
|
-
},
|
|
19
|
-
},
|
|
20
|
-
create(context: Rule.RuleContext): Rule.RuleListener {
|
|
21
|
-
return {
|
|
22
|
-
TSAsExpression(node: any): void {
|
|
23
|
-
try {
|
|
24
|
-
const annotation: any = node.typeAnnotation;
|
|
25
|
-
if (!annotation) {
|
|
26
|
-
return;
|
|
27
|
-
}
|
|
28
|
-
// `as const` esetén megengedjük – immutable literal idiomatika.
|
|
29
|
-
if (
|
|
30
|
-
annotation.type === 'TSTypeReference'
|
|
31
|
-
&& annotation.typeName
|
|
32
|
-
&& annotation.typeName.type === 'Identifier'
|
|
33
|
-
&& annotation.typeName.name === 'const'
|
|
34
|
-
) {
|
|
35
|
-
return;
|
|
36
|
-
}
|
|
37
|
-
const typeName: string = context.sourceCode.getText(annotation);
|
|
38
|
-
context.report({
|
|
39
|
-
node: node,
|
|
40
|
-
messageId: 'noAsCast',
|
|
41
|
-
data: { typeName: typeName },
|
|
42
|
-
});
|
|
43
|
-
} catch (error) {
|
|
44
|
-
console.error('[no-as-cast] Error in TSAsExpression visitor:', error);
|
|
45
|
-
}
|
|
46
|
-
},
|
|
47
|
-
};
|
|
48
|
-
},
|
|
49
|
-
};
|
|
50
|
-
|
|
51
|
-
export default rule;
|
|
1
|
+
import { Rule } from 'eslint';
|
|
2
|
+
|
|
3
|
+
/**
|
|
4
|
+
* No-as-cast szabály: TypeScript `as` type cast tilos.
|
|
5
|
+
* CCAP "no-as-cast" review ESLint AST-portja.
|
|
6
|
+
* Megjegyzés: az `as const` kifejezést megengedjük (immutable literal-ként standard).
|
|
7
|
+
*/
|
|
8
|
+
const rule: Rule.RuleModule = {
|
|
9
|
+
meta: {
|
|
10
|
+
type: 'suggestion',
|
|
11
|
+
docs: {
|
|
12
|
+
description: 'Forbid TypeScript `as` type casts (except `as const`); design issue indicator.',
|
|
13
|
+
recommended: true,
|
|
14
|
+
},
|
|
15
|
+
schema: [],
|
|
16
|
+
messages: {
|
|
17
|
+
noAsCast: '`as {{typeName}}` type cast tilos. Tervezési problémát jelezhet — használj rendes típust.',
|
|
18
|
+
},
|
|
19
|
+
},
|
|
20
|
+
create(context: Rule.RuleContext): Rule.RuleListener {
|
|
21
|
+
return {
|
|
22
|
+
TSAsExpression(node: any): void {
|
|
23
|
+
try {
|
|
24
|
+
const annotation: any = node.typeAnnotation;
|
|
25
|
+
if (!annotation) {
|
|
26
|
+
return;
|
|
27
|
+
}
|
|
28
|
+
// `as const` esetén megengedjük – immutable literal idiomatika.
|
|
29
|
+
if (
|
|
30
|
+
annotation.type === 'TSTypeReference'
|
|
31
|
+
&& annotation.typeName
|
|
32
|
+
&& annotation.typeName.type === 'Identifier'
|
|
33
|
+
&& annotation.typeName.name === 'const'
|
|
34
|
+
) {
|
|
35
|
+
return;
|
|
36
|
+
}
|
|
37
|
+
const typeName: string = context.sourceCode.getText(annotation);
|
|
38
|
+
context.report({
|
|
39
|
+
node: node,
|
|
40
|
+
messageId: 'noAsCast',
|
|
41
|
+
data: { typeName: typeName },
|
|
42
|
+
});
|
|
43
|
+
} catch (error) {
|
|
44
|
+
console.error('[no-as-cast] Error in TSAsExpression visitor:', error);
|
|
45
|
+
}
|
|
46
|
+
},
|
|
47
|
+
};
|
|
48
|
+
},
|
|
49
|
+
};
|
|
50
|
+
|
|
51
|
+
export default rule;
|
|
@@ -1,54 +1,54 @@
|
|
|
1
|
-
import noDynamicImportsRule from './no-dynamic-imports';
|
|
2
|
-
|
|
3
|
-
describe('| no-dynamic-imports', () => {
|
|
4
|
-
it('| should be a valid ESLint rule', () => {
|
|
5
|
-
expect(noDynamicImportsRule.meta?.type).toBe('suggestion');
|
|
6
|
-
expect(noDynamicImportsRule.meta?.messages?.noDynamicImport).toBeDefined();
|
|
7
|
-
expect(noDynamicImportsRule.meta?.messages?.noRequireCall).toBeDefined();
|
|
8
|
-
});
|
|
9
|
-
|
|
10
|
-
it('| should have create function returning visitor object', () => {
|
|
11
|
-
const mockContext: any = { report: () => {}, options: [] };
|
|
12
|
-
const result: any = noDynamicImportsRule.create(mockContext);
|
|
13
|
-
expect(typeof result.ImportExpression).toBe('function');
|
|
14
|
-
expect(typeof result.CallExpression).toBe('function');
|
|
15
|
-
});
|
|
16
|
-
|
|
17
|
-
it('| should report dynamic import()', () => {
|
|
18
|
-
let reportCalled: boolean = false;
|
|
19
|
-
const mockContext: any = {
|
|
20
|
-
report: (options: any): void => {
|
|
21
|
-
reportCalled = true;
|
|
22
|
-
expect(options.messageId).toBe('noDynamicImport');
|
|
23
|
-
},
|
|
24
|
-
options: [],
|
|
25
|
-
};
|
|
26
|
-
const rule: any = noDynamicImportsRule.create(mockContext);
|
|
27
|
-
rule.ImportExpression({});
|
|
28
|
-
expect(reportCalled).toBe(true);
|
|
29
|
-
});
|
|
30
|
-
|
|
31
|
-
it('| should report require() call', () => {
|
|
32
|
-
let reportCalled: boolean = false;
|
|
33
|
-
const mockContext: any = {
|
|
34
|
-
report: (options: any): void => {
|
|
35
|
-
reportCalled = true;
|
|
36
|
-
expect(options.messageId).toBe('noRequireCall');
|
|
37
|
-
},
|
|
38
|
-
options: [],
|
|
39
|
-
};
|
|
40
|
-
const rule: any = noDynamicImportsRule.create(mockContext);
|
|
41
|
-
const mockNode: any = { callee: { type: 'Identifier', name: 'require' } };
|
|
42
|
-
rule.CallExpression(mockNode);
|
|
43
|
-
expect(reportCalled).toBe(true);
|
|
44
|
-
});
|
|
45
|
-
|
|
46
|
-
it('| should not report unrelated function calls', () => {
|
|
47
|
-
let reportCalled: boolean = false;
|
|
48
|
-
const mockContext: any = { report: () => { reportCalled = true; }, options: [] };
|
|
49
|
-
const rule: any = noDynamicImportsRule.create(mockContext);
|
|
50
|
-
const mockNode: any = { callee: { type: 'Identifier', name: 'someOtherCall' } };
|
|
51
|
-
rule.CallExpression(mockNode);
|
|
52
|
-
expect(reportCalled).toBe(false);
|
|
53
|
-
});
|
|
54
|
-
});
|
|
1
|
+
import noDynamicImportsRule from './no-dynamic-imports';
|
|
2
|
+
|
|
3
|
+
describe('| no-dynamic-imports', () => {
|
|
4
|
+
it('| should be a valid ESLint rule', () => {
|
|
5
|
+
expect(noDynamicImportsRule.meta?.type).toBe('suggestion');
|
|
6
|
+
expect(noDynamicImportsRule.meta?.messages?.noDynamicImport).toBeDefined();
|
|
7
|
+
expect(noDynamicImportsRule.meta?.messages?.noRequireCall).toBeDefined();
|
|
8
|
+
});
|
|
9
|
+
|
|
10
|
+
it('| should have create function returning visitor object', () => {
|
|
11
|
+
const mockContext: any = { report: () => {}, options: [] };
|
|
12
|
+
const result: any = noDynamicImportsRule.create(mockContext);
|
|
13
|
+
expect(typeof result.ImportExpression).toBe('function');
|
|
14
|
+
expect(typeof result.CallExpression).toBe('function');
|
|
15
|
+
});
|
|
16
|
+
|
|
17
|
+
it('| should report dynamic import()', () => {
|
|
18
|
+
let reportCalled: boolean = false;
|
|
19
|
+
const mockContext: any = {
|
|
20
|
+
report: (options: any): void => {
|
|
21
|
+
reportCalled = true;
|
|
22
|
+
expect(options.messageId).toBe('noDynamicImport');
|
|
23
|
+
},
|
|
24
|
+
options: [],
|
|
25
|
+
};
|
|
26
|
+
const rule: any = noDynamicImportsRule.create(mockContext);
|
|
27
|
+
rule.ImportExpression({});
|
|
28
|
+
expect(reportCalled).toBe(true);
|
|
29
|
+
});
|
|
30
|
+
|
|
31
|
+
it('| should report require() call', () => {
|
|
32
|
+
let reportCalled: boolean = false;
|
|
33
|
+
const mockContext: any = {
|
|
34
|
+
report: (options: any): void => {
|
|
35
|
+
reportCalled = true;
|
|
36
|
+
expect(options.messageId).toBe('noRequireCall');
|
|
37
|
+
},
|
|
38
|
+
options: [],
|
|
39
|
+
};
|
|
40
|
+
const rule: any = noDynamicImportsRule.create(mockContext);
|
|
41
|
+
const mockNode: any = { callee: { type: 'Identifier', name: 'require' } };
|
|
42
|
+
rule.CallExpression(mockNode);
|
|
43
|
+
expect(reportCalled).toBe(true);
|
|
44
|
+
});
|
|
45
|
+
|
|
46
|
+
it('| should not report unrelated function calls', () => {
|
|
47
|
+
let reportCalled: boolean = false;
|
|
48
|
+
const mockContext: any = { report: () => { reportCalled = true; }, options: [] };
|
|
49
|
+
const rule: any = noDynamicImportsRule.create(mockContext);
|
|
50
|
+
const mockNode: any = { callee: { type: 'Identifier', name: 'someOtherCall' } };
|
|
51
|
+
rule.CallExpression(mockNode);
|
|
52
|
+
expect(reportCalled).toBe(false);
|
|
53
|
+
});
|
|
54
|
+
});
|