@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.
Files changed (197) hide show
  1. package/.c8rc.json +26 -26
  2. package/.cursor/rules/__assistant_guide.mdc +30 -30
  3. package/.cursor/rules/__main.mdc +64 -64
  4. package/.cursor/rules/_ag_backend-structure.mdc +85 -85
  5. package/.cursor/rules/_ag_backend.mdc +16 -16
  6. package/.cursor/rules/_ag_debug.mdc +7 -7
  7. package/.cursor/rules/_ag_documentation_writing_rules.mdc +372 -372
  8. package/.cursor/rules/_ag_file-refactoring.mdc +113 -113
  9. package/.cursor/rules/_ag_fixes_rules.mdc +5 -5
  10. package/.cursor/rules/_ag_frontend-structure.mdc +86 -86
  11. package/.cursor/rules/_ag_frontend.mdc +39 -39
  12. package/.cursor/rules/_ag_import-rules.mdc +44 -44
  13. package/.cursor/rules/_ag_naming.mdc +115 -115
  14. package/.cursor/rules/_ag_running_commands.mdc +4 -4
  15. package/.cursor/rules/_ag_server-controller.mdc +5 -5
  16. package/.cursor/rules/_ag_should-be.mdc +6 -6
  17. package/.cursor/rules/_ag_swearing.mdc +47 -47
  18. package/.cursor/rules/ai_development_guide.md +60 -60
  19. package/.cursor/rules/ai_directives.md +114 -114
  20. package/.cursor/rules/cursor-rules.md +160 -160
  21. package/.cursor/rules/default-command.mdc +464 -464
  22. package/.cursor/rules/error_code_pattern.md +39 -39
  23. package/.cursor/rules/saved rule mcp server use.md +15 -15
  24. package/.dynamo/pipeline.cicd.config.json +184 -184
  25. package/.dynamo/version-bump.config.json +5 -5
  26. package/.eslintrc.json +4 -4
  27. package/.github/workflows/main.yml +436 -436
  28. package/.husky/pre-commit +2 -2
  29. package/.vscode/settings.json +11 -11
  30. package/README.md +319 -319
  31. package/__documentations/2026-04-18-ccap-review-integration.md +173 -173
  32. package/__documentations/2026-08-18-bfr-ctm-005-eslint9-validator-report.md +45 -0
  33. package/build/configs/base.d.ts +9 -9
  34. package/build/configs/base.js +150 -150
  35. package/build/configs/fsm.d.ts +1 -1
  36. package/build/configs/fsm.js +7 -7
  37. package/build/configs/ngx-package.d.ts +1 -1
  38. package/build/configs/ngx-package.js +7 -7
  39. package/build/configs/ngx.d.ts +4 -4
  40. package/build/configs/ngx.js +91 -91
  41. package/build/configs/nts-package.d.ts +1 -1
  42. package/build/configs/nts-package.js +17 -17
  43. package/build/configs/nts.d.ts +1 -1
  44. package/build/configs/nts.js +31 -31
  45. package/build/index.d.ts +1 -1
  46. package/build/index.js +2 -2
  47. package/build/plugin/index.d.ts +52 -52
  48. package/build/plugin/index.js +80 -80
  49. package/build/plugin/rules/angular/angular-for-track-logic.d.ts +7 -7
  50. package/build/plugin/rules/angular/angular-for-track-logic.js +81 -81
  51. package/build/plugin/rules/angular/effect-signal-loop.d.ts +9 -9
  52. package/build/plugin/rules/angular/effect-signal-loop.js +112 -112
  53. package/build/plugin/rules/angular/no-async-ngoninit.d.ts +8 -8
  54. package/build/plugin/rules/angular/no-async-ngoninit.js +42 -42
  55. package/build/plugin/rules/angular/no-component-direct-api.d.ts +9 -9
  56. package/build/plugin/rules/angular/no-component-direct-api.js +75 -75
  57. package/build/plugin/rules/angular/no-method-call-in-template.d.ts +8 -8
  58. package/build/plugin/rules/angular/no-method-call-in-template.js +79 -79
  59. package/build/plugin/rules/angular/no-ngfor-directive.d.ts +7 -7
  60. package/build/plugin/rules/angular/no-ngfor-directive.js +59 -59
  61. package/build/plugin/rules/angular/no-ngif-directive.d.ts +7 -7
  62. package/build/plugin/rules/angular/no-ngif-directive.js +59 -59
  63. package/build/plugin/rules/explicit-types.d.ts +3 -3
  64. package/build/plugin/rules/explicit-types.js +178 -178
  65. package/build/plugin/rules/import/import-order.d.ts +3 -3
  66. package/build/plugin/rules/import/import-order.js +154 -154
  67. package/build/plugin/rules/import/no-import-type.d.ts +3 -3
  68. package/build/plugin/rules/import/no-import-type.js +45 -45
  69. package/build/plugin/rules/import/no-js-import.d.ts +3 -3
  70. package/build/plugin/rules/import/no-js-import.js +40 -40
  71. package/build/plugin/rules/naming-patterns.d.ts +7 -7
  72. package/build/plugin/rules/naming-patterns.d.ts.map +1 -1
  73. package/build/plugin/rules/naming-patterns.js +366 -350
  74. package/build/plugin/rules/naming-patterns.js.map +1 -1
  75. package/build/plugin/rules/no-getter-logic.d.ts +3 -3
  76. package/build/plugin/rules/no-getter-logic.js +194 -194
  77. package/build/plugin/rules/prefer-enum-over-string-union.d.ts +3 -3
  78. package/build/plugin/rules/prefer-enum-over-string-union.js +289 -289
  79. package/build/plugin/rules/prefer-interface-over-duplicate-types.d.ts +3 -3
  80. package/build/plugin/rules/prefer-interface-over-duplicate-types.js +264 -264
  81. package/build/plugin/rules/require-jsdoc-description.d.ts +3 -3
  82. package/build/plugin/rules/require-jsdoc-description.js +496 -496
  83. package/build/plugin/rules/require-test-description-prefix.d.ts +3 -3
  84. package/build/plugin/rules/require-test-description-prefix.js +135 -135
  85. package/build/plugin/rules/ts/data-model-extends.d.ts +8 -8
  86. package/build/plugin/rules/ts/data-model-extends.js +94 -94
  87. package/build/plugin/rules/ts/no-any-type.d.ts +8 -8
  88. package/build/plugin/rules/ts/no-any-type.js +36 -36
  89. package/build/plugin/rules/ts/no-as-cast.d.ts +8 -8
  90. package/build/plugin/rules/ts/no-as-cast.js +49 -49
  91. package/build/plugin/rules/ts/no-dynamic-imports.d.ts +8 -8
  92. package/build/plugin/rules/ts/no-dynamic-imports.js +50 -50
  93. package/build/plugin/rules/ts/no-native-browser-dialogs.d.ts +8 -8
  94. package/build/plugin/rules/ts/no-native-browser-dialogs.js +78 -78
  95. package/build/plugin/rules/ts/no-plain-function-export.d.ts +9 -9
  96. package/build/plugin/rules/ts/no-plain-function-export.js +69 -69
  97. package/build/plugin/rules/ts/one-export-per-file.d.ts +8 -8
  98. package/build/plugin/rules/ts/one-export-per-file.js +73 -73
  99. package/build/scripts/dynamo-fix.d.ts +26 -26
  100. package/build/scripts/dynamo-fix.js +146 -146
  101. package/build/scripts/dynamo-fix.js.map +1 -1
  102. package/build/scripts/eslintrc-audit.d.ts +26 -26
  103. package/build/scripts/eslintrc-audit.js +100 -100
  104. package/build/scripts/fix-return-types.d.ts +74 -74
  105. package/build/scripts/fix-return-types.js +226 -226
  106. package/build/scripts/validate-imports.d.ts +26 -26
  107. package/build/scripts/validate-imports.js +157 -137
  108. package/build/scripts/validate-imports.js.map +1 -1
  109. package/build/scripts/validate-naming.d.ts +26 -26
  110. package/build/scripts/validate-naming.js +157 -123
  111. package/build/scripts/validate-naming.js.map +1 -1
  112. package/eslint.config.js +15 -15
  113. package/package.json +105 -105
  114. package/samples/.vscode/settings.json +13 -13
  115. package/samples/base/.eslintrc.json +6 -6
  116. package/samples/base/eslint.config.js +3 -3
  117. package/samples/fsm/.eslintrc.json +4 -4
  118. package/samples/fsm/eslint.config.js +3 -3
  119. package/samples/ngx/.eslintrc.json +6 -6
  120. package/samples/ngx/eslint.config.js +3 -3
  121. package/samples/ngx-package/.eslintrc.json +4 -4
  122. package/samples/ngx-package/eslint.config.js +3 -3
  123. package/samples/nts/.eslintrc.json +6 -6
  124. package/samples/nts/eslint.config.js +3 -3
  125. package/samples/nts-package/.eslintrc.json +4 -4
  126. package/samples/nts-package/eslint.config.js +3 -3
  127. package/samples/package.json.example +26 -26
  128. package/samples/poc-sample.ts +38 -38
  129. package/samples/poc-violations.ts +54 -54
  130. package/scripts/run-coverage-tests.js +24 -24
  131. package/spec/support/helpers/ts-node-helper.js +9 -9
  132. package/spec/support/jasmine.coverage.json +23 -23
  133. package/spec/support/jasmine.json +24 -24
  134. package/src/configs/base.ts +157 -157
  135. package/src/configs/fsm.ts +10 -10
  136. package/src/configs/ngx-package.ts +10 -10
  137. package/src/configs/ngx.ts +96 -96
  138. package/src/configs/nts-package.ts +20 -20
  139. package/src/configs/nts.ts +34 -34
  140. package/src/index.ts +4 -4
  141. package/src/plugin/index.ts +86 -86
  142. package/src/plugin/rules/angular/angular-for-track-logic.spec.ts +44 -44
  143. package/src/plugin/rules/angular/angular-for-track-logic.ts +88 -88
  144. package/src/plugin/rules/angular/effect-signal-loop.spec.ts +89 -89
  145. package/src/plugin/rules/angular/effect-signal-loop.ts +125 -125
  146. package/src/plugin/rules/angular/no-async-ngoninit.spec.ts +50 -50
  147. package/src/plugin/rules/angular/no-async-ngoninit.ts +43 -43
  148. package/src/plugin/rules/angular/no-component-direct-api.spec.ts +78 -78
  149. package/src/plugin/rules/angular/no-component-direct-api.ts +82 -82
  150. package/src/plugin/rules/angular/no-method-call-in-template.spec.ts +63 -63
  151. package/src/plugin/rules/angular/no-method-call-in-template.ts +89 -89
  152. package/src/plugin/rules/angular/no-ngfor-directive.spec.ts +44 -44
  153. package/src/plugin/rules/angular/no-ngfor-directive.ts +62 -62
  154. package/src/plugin/rules/angular/no-ngif-directive.spec.ts +44 -44
  155. package/src/plugin/rules/angular/no-ngif-directive.ts +62 -62
  156. package/src/plugin/rules/explicit-types.spec.ts +190 -190
  157. package/src/plugin/rules/explicit-types.ts +186 -186
  158. package/src/plugin/rules/import/import-order.spec.ts +219 -219
  159. package/src/plugin/rules/import/import-order.ts +197 -197
  160. package/src/plugin/rules/import/no-import-type.spec.ts +69 -69
  161. package/src/plugin/rules/import/no-import-type.ts +46 -46
  162. package/src/plugin/rules/import/no-js-import.spec.ts +82 -82
  163. package/src/plugin/rules/import/no-js-import.ts +45 -45
  164. package/src/plugin/rules/naming-patterns.spec.ts +189 -127
  165. package/src/plugin/rules/naming-patterns.ts +426 -408
  166. package/src/plugin/rules/no-getter-logic.spec.ts +591 -591
  167. package/src/plugin/rules/no-getter-logic.ts +220 -220
  168. package/src/plugin/rules/prefer-enum-over-string-union.spec.ts +583 -583
  169. package/src/plugin/rules/prefer-enum-over-string-union.ts +333 -333
  170. package/src/plugin/rules/prefer-interface-over-duplicate-types.spec.ts +404 -404
  171. package/src/plugin/rules/prefer-interface-over-duplicate-types.ts +314 -314
  172. package/src/plugin/rules/require-jsdoc-description.spec.ts +779 -779
  173. package/src/plugin/rules/require-jsdoc-description.ts +572 -572
  174. package/src/plugin/rules/require-test-description-prefix.spec.ts +459 -459
  175. package/src/plugin/rules/require-test-description-prefix.ts +154 -154
  176. package/src/plugin/rules/ts/data-model-extends.spec.ts +84 -84
  177. package/src/plugin/rules/ts/data-model-extends.ts +100 -100
  178. package/src/plugin/rules/ts/no-any-type.spec.ts +28 -28
  179. package/src/plugin/rules/ts/no-any-type.ts +36 -36
  180. package/src/plugin/rules/ts/no-as-cast.spec.ts +51 -51
  181. package/src/plugin/rules/ts/no-as-cast.ts +51 -51
  182. package/src/plugin/rules/ts/no-dynamic-imports.spec.ts +54 -54
  183. package/src/plugin/rules/ts/no-dynamic-imports.ts +49 -49
  184. package/src/plugin/rules/ts/no-native-browser-dialogs.spec.ts +54 -54
  185. package/src/plugin/rules/ts/no-native-browser-dialogs.ts +84 -84
  186. package/src/plugin/rules/ts/no-plain-function-export.spec.ts +72 -72
  187. package/src/plugin/rules/ts/no-plain-function-export.ts +73 -73
  188. package/src/plugin/rules/ts/one-export-per-file.spec.ts +64 -64
  189. package/src/plugin/rules/ts/one-export-per-file.ts +77 -77
  190. package/src/scripts/dynamo-fix.ts +172 -172
  191. package/src/scripts/eslintrc-audit.ts +116 -116
  192. package/src/scripts/fix-return-types.spec.ts +65 -65
  193. package/src/scripts/fix-return-types.ts +288 -288
  194. package/src/scripts/validate-imports.ts +278 -249
  195. package/src/scripts/validate-naming.ts +204 -176
  196. package/src/scripts/validator-cli.integration.spec.ts +96 -0
  197. 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;