@futdevpro/dynamo-eslint 1.20.3 → 1.20.5
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 +304 -304
- package/build/plugin/rules/ts/no-as-cast.d.ts.map +1 -1
- package/build/plugin/rules/ts/no-as-cast.js +43 -0
- package/build/plugin/rules/ts/no-as-cast.js.map +1 -1
- package/package.json +14 -7
- package/.c8rc.json +0 -26
- package/.cursor/rules/__assistant_guide.mdc +0 -30
- package/.cursor/rules/__main.mdc +0 -64
- package/.cursor/rules/_ag_backend-structure.mdc +0 -86
- package/.cursor/rules/_ag_backend.mdc +0 -16
- package/.cursor/rules/_ag_debug.mdc +0 -8
- package/.cursor/rules/_ag_documentation_writing_rules.mdc +0 -372
- package/.cursor/rules/_ag_file-refactoring.mdc +0 -113
- package/.cursor/rules/_ag_fixes_rules.mdc +0 -6
- package/.cursor/rules/_ag_frontend-structure.mdc +0 -87
- package/.cursor/rules/_ag_frontend.mdc +0 -40
- package/.cursor/rules/_ag_import-rules.mdc +0 -45
- package/.cursor/rules/_ag_naming.mdc +0 -116
- package/.cursor/rules/_ag_running_commands.mdc +0 -5
- package/.cursor/rules/_ag_server-controller.mdc +0 -6
- package/.cursor/rules/_ag_should-be.mdc +0 -7
- package/.cursor/rules/_ag_swearing.mdc +0 -47
- package/.cursor/rules/ai_development_guide.md +0 -61
- package/.cursor/rules/ai_directives.md +0 -114
- package/.cursor/rules/cursor-rules.md +0 -160
- package/.cursor/rules/default-command.mdc +0 -465
- package/.cursor/rules/error_code_pattern.md +0 -40
- package/.cursor/rules/saved rule mcp server use.md +0 -16
- package/.dynamo/pipeline.cicd.config.json +0 -184
- package/.dynamo/version-bump.config.json +0 -5
- package/.eslintrc.json +0 -4
- package/.github/workflows/main.yml +0 -436
- package/.husky/pre-commit +0 -2
- package/.vscode/settings.json +0 -11
- package/__documentations/2026-04-18-ccap-review-integration.md +0 -173
- package/__documentations/2026-08-18-bfr-ctm-005-eslint9-validator-report.md +0 -45
- package/__documentations/2026-08-22-angular-20-phase2-release.md +0 -83
- package/eslint.config.js +0 -15
- package/pnpm-workspace.yaml +0 -2
- package/samples/.vscode/settings.json +0 -13
- package/samples/base/.eslintrc.json +0 -6
- package/samples/base/eslint.config.js +0 -3
- package/samples/fsm/.eslintrc.json +0 -4
- package/samples/fsm/eslint.config.js +0 -3
- package/samples/ngx/.eslintrc.json +0 -6
- package/samples/ngx/eslint.config.js +0 -3
- package/samples/ngx-package/.eslintrc.json +0 -4
- package/samples/ngx-package/eslint.config.js +0 -3
- package/samples/nts/.eslintrc.json +0 -6
- package/samples/nts/eslint.config.js +0 -3
- package/samples/nts-package/.eslintrc.json +0 -4
- package/samples/nts-package/eslint.config.js +0 -3
- package/samples/package.json.example +0 -22
- package/samples/poc-sample.ts +0 -38
- package/samples/poc-violations.ts +0 -54
- package/scripts/run-coverage-tests.js +0 -24
- package/spec/support/helpers/ts-node-helper.js +0 -9
- package/spec/support/jasmine.coverage.json +0 -23
- package/spec/support/jasmine.json +0 -24
- package/src/configs/angular20-compatibility.spec.ts +0 -73
- package/src/configs/base.ts +0 -156
- package/src/configs/fsm.ts +0 -10
- package/src/configs/ngx-package.ts +0 -10
- package/src/configs/ngx.ts +0 -93
- package/src/configs/nts-package.ts +0 -20
- package/src/configs/nts.ts +0 -34
- package/src/index.ts +0 -4
- package/src/plugin/index.ts +0 -86
- package/src/plugin/rules/angular/angular-for-track-logic.spec.ts +0 -44
- package/src/plugin/rules/angular/angular-for-track-logic.ts +0 -88
- package/src/plugin/rules/angular/effect-signal-loop.spec.ts +0 -89
- package/src/plugin/rules/angular/effect-signal-loop.ts +0 -125
- package/src/plugin/rules/angular/no-async-ngoninit.spec.ts +0 -50
- package/src/plugin/rules/angular/no-async-ngoninit.ts +0 -43
- package/src/plugin/rules/angular/no-component-direct-api.spec.ts +0 -78
- package/src/plugin/rules/angular/no-component-direct-api.ts +0 -82
- package/src/plugin/rules/angular/no-method-call-in-template.spec.ts +0 -63
- package/src/plugin/rules/angular/no-method-call-in-template.ts +0 -89
- package/src/plugin/rules/angular/no-ngfor-directive.spec.ts +0 -44
- package/src/plugin/rules/angular/no-ngfor-directive.ts +0 -62
- package/src/plugin/rules/angular/no-ngif-directive.spec.ts +0 -44
- package/src/plugin/rules/angular/no-ngif-directive.ts +0 -62
- package/src/plugin/rules/explicit-types.spec.ts +0 -190
- package/src/plugin/rules/explicit-types.ts +0 -186
- package/src/plugin/rules/import/import-order.spec.ts +0 -219
- package/src/plugin/rules/import/import-order.ts +0 -197
- package/src/plugin/rules/import/no-import-type.spec.ts +0 -69
- package/src/plugin/rules/import/no-import-type.ts +0 -46
- package/src/plugin/rules/import/no-js-import.spec.ts +0 -82
- package/src/plugin/rules/import/no-js-import.ts +0 -45
- package/src/plugin/rules/naming-patterns.spec.ts +0 -189
- package/src/plugin/rules/naming-patterns.ts +0 -426
- package/src/plugin/rules/no-getter-logic.spec.ts +0 -591
- package/src/plugin/rules/no-getter-logic.ts +0 -220
- package/src/plugin/rules/prefer-enum-over-string-union.spec.ts +0 -583
- package/src/plugin/rules/prefer-enum-over-string-union.ts +0 -333
- package/src/plugin/rules/prefer-interface-over-duplicate-types.spec.ts +0 -404
- package/src/plugin/rules/prefer-interface-over-duplicate-types.ts +0 -314
- package/src/plugin/rules/require-jsdoc-description.spec.ts +0 -779
- package/src/plugin/rules/require-jsdoc-description.ts +0 -572
- package/src/plugin/rules/require-test-description-prefix.spec.ts +0 -459
- package/src/plugin/rules/require-test-description-prefix.ts +0 -154
- package/src/plugin/rules/ts/data-model-extends.spec.ts +0 -84
- package/src/plugin/rules/ts/data-model-extends.ts +0 -100
- package/src/plugin/rules/ts/no-any-type.spec.ts +0 -28
- package/src/plugin/rules/ts/no-any-type.ts +0 -36
- package/src/plugin/rules/ts/no-as-cast.spec.ts +0 -51
- package/src/plugin/rules/ts/no-as-cast.ts +0 -51
- package/src/plugin/rules/ts/no-dynamic-imports.spec.ts +0 -54
- package/src/plugin/rules/ts/no-dynamic-imports.ts +0 -49
- package/src/plugin/rules/ts/no-native-browser-dialogs.spec.ts +0 -54
- package/src/plugin/rules/ts/no-native-browser-dialogs.ts +0 -84
- package/src/plugin/rules/ts/no-plain-function-export.spec.ts +0 -72
- package/src/plugin/rules/ts/no-plain-function-export.ts +0 -73
- package/src/plugin/rules/ts/one-export-per-file.spec.ts +0 -64
- package/src/plugin/rules/ts/one-export-per-file.ts +0 -77
- package/src/scripts/dynamo-fix.ts +0 -172
- package/src/scripts/eslintrc-audit.ts +0 -116
- package/src/scripts/fix-return-types.spec.ts +0 -65
- package/src/scripts/fix-return-types.ts +0 -289
- package/src/scripts/validate-imports.ts +0 -292
- package/src/scripts/validate-naming.ts +0 -213
- package/src/scripts/validator-cli.integration.spec.ts +0 -99
- package/tsconfig.json +0 -32
|
@@ -1,100 +0,0 @@
|
|
|
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 +0,0 @@
|
|
|
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 +0,0 @@
|
|
|
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 +0,0 @@
|
|
|
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 +0,0 @@
|
|
|
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 +0,0 @@
|
|
|
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,49 +0,0 @@
|
|
|
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 +0,0 @@
|
|
|
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 +0,0 @@
|
|
|
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 +0,0 @@
|
|
|
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 +0,0 @@
|
|
|
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;
|