@futdevpro/dynamo-eslint 1.15.25 → 1.20.2

package/src/plugin/rules/angular/no-async-ngoninit.spec.ts

@@ -1,50 +1,50 @@
-import noAsyncNgOnInitRule from './no-async-ngoninit';
-
-describe('| no-async-ngoninit', () => {
-  it('| should be a valid ESLint rule', () => {
-    expect(noAsyncNgOnInitRule.meta?.type).toBe('suggestion');
-    expect(noAsyncNgOnInitRule.meta?.messages?.noAsyncNgOnInit).toBeDefined();
-  });
-
-  it('| should report `async ngOnInit`', () => {
-    let reportCalled: boolean = false;
-    const mockContext: any = {
-      report: (options: any): void => {
-        reportCalled = true;
-        expect(options.messageId).toBe('noAsyncNgOnInit');
-      },
-      options: [],
-    };
-    const rule: any = noAsyncNgOnInitRule.create(mockContext);
-    const mockNode: any = {
-      key: { type: 'Identifier', name: 'ngOnInit' },
-      value: { async: true },
-    };
-    rule.MethodDefinition(mockNode);
-    expect(reportCalled).toBe(true);
-  });
-
-  it('| should not report sync `ngOnInit`', () => {
-    let reportCalled: boolean = false;
-    const mockContext: any = { report: () => { reportCalled = true; }, options: [] };
-    const rule: any = noAsyncNgOnInitRule.create(mockContext);
-    const mockNode: any = {
-      key: { type: 'Identifier', name: 'ngOnInit' },
-      value: { async: false },
-    };
-    rule.MethodDefinition(mockNode);
-    expect(reportCalled).toBe(false);
-  });
-
-  it('| should not report unrelated async methods', () => {
-    let reportCalled: boolean = false;
-    const mockContext: any = { report: () => { reportCalled = true; }, options: [] };
-    const rule: any = noAsyncNgOnInitRule.create(mockContext);
-    const mockNode: any = {
-      key: { type: 'Identifier', name: 'loadData' },
-      value: { async: true },
-    };
-    rule.MethodDefinition(mockNode);
-    expect(reportCalled).toBe(false);
-  });
-});
+import noAsyncNgOnInitRule from './no-async-ngoninit';
+
+describe('| no-async-ngoninit', () => {
+  it('| should be a valid ESLint rule', () => {
+    expect(noAsyncNgOnInitRule.meta?.type).toBe('suggestion');
+    expect(noAsyncNgOnInitRule.meta?.messages?.noAsyncNgOnInit).toBeDefined();
+  });
+
+  it('| should report `async ngOnInit`', () => {
+    let reportCalled: boolean = false;
+    const mockContext: any = {
+      report: (options: any): void => {
+        reportCalled = true;
+        expect(options.messageId).toBe('noAsyncNgOnInit');
+      },
+      options: [],
+    };
+    const rule: any = noAsyncNgOnInitRule.create(mockContext);
+    const mockNode: any = {
+      key: { type: 'Identifier', name: 'ngOnInit' },
+      value: { async: true },
+    };
+    rule.MethodDefinition(mockNode);
+    expect(reportCalled).toBe(true);
+  });
+
+  it('| should not report sync `ngOnInit`', () => {
+    let reportCalled: boolean = false;
+    const mockContext: any = { report: () => { reportCalled = true; }, options: [] };
+    const rule: any = noAsyncNgOnInitRule.create(mockContext);
+    const mockNode: any = {
+      key: { type: 'Identifier', name: 'ngOnInit' },
+      value: { async: false },
+    };
+    rule.MethodDefinition(mockNode);
+    expect(reportCalled).toBe(false);
+  });
+
+  it('| should not report unrelated async methods', () => {
+    let reportCalled: boolean = false;
+    const mockContext: any = { report: () => { reportCalled = true; }, options: [] };
+    const rule: any = noAsyncNgOnInitRule.create(mockContext);
+    const mockNode: any = {
+      key: { type: 'Identifier', name: 'loadData' },
+      value: { async: true },
+    };
+    rule.MethodDefinition(mockNode);
+    expect(reportCalled).toBe(false);
+  });
+});

package/src/plugin/rules/angular/no-async-ngoninit.ts

@@ -1,43 +1,43 @@
-import { Rule } from 'eslint';
-
-/**
- * No-async-ngOnInit szabály: `ngOnInit` metódus nem lehet `async`.
- * Angular nem várja le az `async ngOnInit`-et — ez néma hibát okoz.
- * CCAP "no-async-ngoninit" review ESLint AST-portja.
- */
-const rule: Rule.RuleModule = {
-  meta: {
-    type: 'suggestion',
-    docs: {
-      description: 'Forbid `async ngOnInit` in Angular components — Angular does not await it.',
-      recommended: true,
-    },
-    schema: [],
-    messages: {
-      noAsyncNgOnInit:
-        '`async ngOnInit` tilos — Angular nem várja le. '
-        + 'Tedd a logikát egy külön `async` privát metódusba és hívd meg `ngOnInit`-ből.',
-    },
-  },
-  create(context: Rule.RuleContext): Rule.RuleListener {
-    return {
-      MethodDefinition(node: any): void {
-        try {
-          if (!node.key || node.key.type !== 'Identifier' || node.key.name !== 'ngOnInit') {
-            return;
-          }
-          if (node.value && node.value.async === true) {
-            context.report({
-              node: node.key,
-              messageId: 'noAsyncNgOnInit',
-            });
-          }
-        } catch (error) {
-          console.error('[no-async-ngoninit] Error in MethodDefinition visitor:', error);
-        }
-      },
-    };
-  },
-};
-
-export default rule;
+import { Rule } from 'eslint';
+
+/**
+ * No-async-ngOnInit szabály: `ngOnInit` metódus nem lehet `async`.
+ * Angular nem várja le az `async ngOnInit`-et — ez néma hibát okoz.
+ * CCAP "no-async-ngoninit" review ESLint AST-portja.
+ */
+const rule: Rule.RuleModule = {
+  meta: {
+    type: 'suggestion',
+    docs: {
+      description: 'Forbid `async ngOnInit` in Angular components — Angular does not await it.',
+      recommended: true,
+    },
+    schema: [],
+    messages: {
+      noAsyncNgOnInit:
+        '`async ngOnInit` tilos — Angular nem várja le. '
+        + 'Tedd a logikát egy külön `async` privát metódusba és hívd meg `ngOnInit`-ből.',
+    },
+  },
+  create(context: Rule.RuleContext): Rule.RuleListener {
+    return {
+      MethodDefinition(node: any): void {
+        try {
+          if (!node.key || node.key.type !== 'Identifier' || node.key.name !== 'ngOnInit') {
+            return;
+          }
+          if (node.value && node.value.async === true) {
+            context.report({
+              node: node.key,
+              messageId: 'noAsyncNgOnInit',
+            });
+          }
+        } catch (error) {
+          console.error('[no-async-ngoninit] Error in MethodDefinition visitor:', error);
+        }
+      },
+    };
+  },
+};
+
+export default rule;

package/src/plugin/rules/angular/no-component-direct-api.spec.ts

@@ -1,78 +1,78 @@
-import noComponentDirectApiRule from './no-component-direct-api';
-
-describe('| no-component-direct-api', () => {
-  it('| should be a valid ESLint rule', () => {
-    expect(noComponentDirectApiRule.meta?.type).toBe('suggestion');
-    expect(noComponentDirectApiRule.meta?.messages?.noApiServiceImport).toBeDefined();
-    expect(noComponentDirectApiRule.meta?.messages?.noHttpClientImport).toBeDefined();
-  });
-
-  it('| should skip non-component files', () => {
-    const mockContext: any = { filename: '/x/foo.ts', report: () => {}, options: [] };
-    const result: any = noComponentDirectApiRule.create(mockContext);
-    expect(result.ImportDeclaration).toBeUndefined();
-  });
-
-  it('| should report `*ApiService` import in component', () => {
-    let reportCalled: boolean = false;
-    const mockContext: any = {
-      filename: '/x/user.component.ts',
-      report: (options: any): void => {
-        if (options.messageId === 'noApiServiceImport') {
-          reportCalled = true;
-          expect(options.data.names).toBe('User_ApiService');
-        }
-      },
-      options: [],
-    };
-    const rule: any = noComponentDirectApiRule.create(mockContext);
-    const mockNode: any = {
-      source: { value: './user.api-service' },
-      specifiers: [
-        { type: 'ImportSpecifier', imported: { name: 'User_ApiService' } },
-      ],
-    };
-    rule.ImportDeclaration(mockNode);
-    expect(reportCalled).toBe(true);
-  });
-
-  it('| should report `HttpClient` import in component', () => {
-    let httpReported: boolean = false;
-    const mockContext: any = {
-      filename: '/x/user.component.ts',
-      report: (options: any): void => {
-        if (options.messageId === 'noHttpClientImport') {
-          httpReported = true;
-        }
-      },
-      options: [],
-    };
-    const rule: any = noComponentDirectApiRule.create(mockContext);
-    const mockNode: any = {
-      source: { value: '@angular/common/http' },
-      specifiers: [
-        { type: 'ImportSpecifier', imported: { name: 'HttpClient' } },
-      ],
-    };
-    rule.ImportDeclaration(mockNode);
-    expect(httpReported).toBe(true);
-  });
-
-  it('| should not report data-service import in component', () => {
-    let reportCalled: boolean = false;
-    const mockContext: any = {
-      filename: '/x/user.component.ts',
-      report: () => { reportCalled = true; },
-      options: [],
-    };
-    const rule: any = noComponentDirectApiRule.create(mockContext);
-    const mockNode: any = {
-      source: { value: './user.data-service' },
-      specifiers: [
-        { type: 'ImportSpecifier', imported: { name: 'User_DataService' } },
-      ],
-    };
-    rule.ImportDeclaration(mockNode);
-    expect(reportCalled).toBe(false);
-  });
-});
+import noComponentDirectApiRule from './no-component-direct-api';
+
+describe('| no-component-direct-api', () => {
+  it('| should be a valid ESLint rule', () => {
+    expect(noComponentDirectApiRule.meta?.type).toBe('suggestion');
+    expect(noComponentDirectApiRule.meta?.messages?.noApiServiceImport).toBeDefined();
+    expect(noComponentDirectApiRule.meta?.messages?.noHttpClientImport).toBeDefined();
+  });
+
+  it('| should skip non-component files', () => {
+    const mockContext: any = { filename: '/x/foo.ts', report: () => {}, options: [] };
+    const result: any = noComponentDirectApiRule.create(mockContext);
+    expect(result.ImportDeclaration).toBeUndefined();
+  });
+
+  it('| should report `*ApiService` import in component', () => {
+    let reportCalled: boolean = false;
+    const mockContext: any = {
+      filename: '/x/user.component.ts',
+      report: (options: any): void => {
+        if (options.messageId === 'noApiServiceImport') {
+          reportCalled = true;
+          expect(options.data.names).toBe('User_ApiService');
+        }
+      },
+      options: [],
+    };
+    const rule: any = noComponentDirectApiRule.create(mockContext);
+    const mockNode: any = {
+      source: { value: './user.api-service' },
+      specifiers: [
+        { type: 'ImportSpecifier', imported: { name: 'User_ApiService' } },
+      ],
+    };
+    rule.ImportDeclaration(mockNode);
+    expect(reportCalled).toBe(true);
+  });
+
+  it('| should report `HttpClient` import in component', () => {
+    let httpReported: boolean = false;
+    const mockContext: any = {
+      filename: '/x/user.component.ts',
+      report: (options: any): void => {
+        if (options.messageId === 'noHttpClientImport') {
+          httpReported = true;
+        }
+      },
+      options: [],
+    };
+    const rule: any = noComponentDirectApiRule.create(mockContext);
+    const mockNode: any = {
+      source: { value: '@angular/common/http' },
+      specifiers: [
+        { type: 'ImportSpecifier', imported: { name: 'HttpClient' } },
+      ],
+    };
+    rule.ImportDeclaration(mockNode);
+    expect(httpReported).toBe(true);
+  });
+
+  it('| should not report data-service import in component', () => {
+    let reportCalled: boolean = false;
+    const mockContext: any = {
+      filename: '/x/user.component.ts',
+      report: () => { reportCalled = true; },
+      options: [],
+    };
+    const rule: any = noComponentDirectApiRule.create(mockContext);
+    const mockNode: any = {
+      source: { value: './user.data-service' },
+      specifiers: [
+        { type: 'ImportSpecifier', imported: { name: 'User_DataService' } },
+      ],
+    };
+    rule.ImportDeclaration(mockNode);
+    expect(reportCalled).toBe(false);
+  });
+});

package/src/plugin/rules/angular/no-component-direct-api.ts

@@ -1,82 +1,82 @@
-import { Rule } from 'eslint';
-
-/** ApiService class név regex – `_ApiService` vagy `ApiService` végződésű class. */
-const API_SERVICE_NAME_REGEX: RegExp = /\w+(?:_)?ApiService$/;
-
-/**
- * No-component-direct-api szabály: `*.component.ts` fájlok nem importálhatnak közvetlenül
- * `*ApiService` class-okat vagy `HttpClient`-et. Single Source of Truth elv – data-service
- * vagy control-service réteg kötelező.
- * CCAP "no-component-direct-api" review ESLint AST-portja.
- */
-const rule: Rule.RuleModule = {
-  meta: {
-    type: 'suggestion',
-    docs: {
-      description:
-        'Components must not directly import `*ApiService` or `HttpClient`; '
-        + 'use data-service / control-service layer (Single Source of Truth).',
-      recommended: true,
-    },
-    schema: [],
-    messages: {
-      noApiServiceImport:
-        'Komponens közvetlenül importál ApiService class-t: `{{names}}`. '
-        + 'Használj data-service-t vagy control-service-t (Single Source of Truth).',
-      noHttpClientImport:
-        'Komponens közvetlenül importálja a `HttpClient`-et. '
-        + 'Használj api-service / data-service / control-service réteget.',
-    },
-  },
-  create(context: Rule.RuleContext): Rule.RuleListener {
-    const filename: string = context.filename ?? '';
-    if (!filename.endsWith('.component.ts')) {
-      return {};
-    }
-
-    return {
-      ImportDeclaration(node: any): void {
-        try {
-          const source: string = (node.source && typeof node.source.value === 'string')
-            ? node.source.value
-            : '';
-          const apiServiceNames: string[] = [];
-          let hasHttpClient: boolean = false;
-
-          for (const specifier of node.specifiers ?? []) {
-            if (specifier.type !== 'ImportSpecifier') {
-              continue;
-            }
-            const importedName: string = (specifier.imported && specifier.imported.name)
-              ? specifier.imported.name
-              : '';
-            if (importedName === 'HttpClient' && source === '@angular/common/http') {
-              hasHttpClient = true;
-            }
-            if (API_SERVICE_NAME_REGEX.test(importedName)) {
-              apiServiceNames.push(importedName);
-            }
-          }
-
-          if (apiServiceNames.length > 0) {
-            context.report({
-              node: node,
-              messageId: 'noApiServiceImport',
-              data: { names: apiServiceNames.join(', ') },
-            });
-          }
-          if (hasHttpClient) {
-            context.report({
-              node: node,
-              messageId: 'noHttpClientImport',
-            });
-          }
-        } catch (error) {
-          console.error('[no-component-direct-api] Error in ImportDeclaration visitor:', error);
-        }
-      },
-    };
-  },
-};
-
-export default rule;
+import { Rule } from 'eslint';
+
+/** ApiService class név regex – `_ApiService` vagy `ApiService` végződésű class. */
+const API_SERVICE_NAME_REGEX: RegExp = /\w+(?:_)?ApiService$/;
+
+/**
+ * No-component-direct-api szabály: `*.component.ts` fájlok nem importálhatnak közvetlenül
+ * `*ApiService` class-okat vagy `HttpClient`-et. Single Source of Truth elv – data-service
+ * vagy control-service réteg kötelező.
+ * CCAP "no-component-direct-api" review ESLint AST-portja.
+ */
+const rule: Rule.RuleModule = {
+  meta: {
+    type: 'suggestion',
+    docs: {
+      description:
+        'Components must not directly import `*ApiService` or `HttpClient`; '
+        + 'use data-service / control-service layer (Single Source of Truth).',
+      recommended: true,
+    },
+    schema: [],
+    messages: {
+      noApiServiceImport:
+        'Komponens közvetlenül importál ApiService class-t: `{{names}}`. '
+        + 'Használj data-service-t vagy control-service-t (Single Source of Truth).',
+      noHttpClientImport:
+        'Komponens közvetlenül importálja a `HttpClient`-et. '
+        + 'Használj api-service / data-service / control-service réteget.',
+    },
+  },
+  create(context: Rule.RuleContext): Rule.RuleListener {
+    const filename: string = context.filename ?? '';
+    if (!filename.endsWith('.component.ts')) {
+      return {};
+    }
+
+    return {
+      ImportDeclaration(node: any): void {
+        try {
+          const source: string = (node.source && typeof node.source.value === 'string')
+            ? node.source.value
+            : '';
+          const apiServiceNames: string[] = [];
+          let hasHttpClient: boolean = false;
+
+          for (const specifier of node.specifiers ?? []) {
+            if (specifier.type !== 'ImportSpecifier') {
+              continue;
+            }
+            const importedName: string = (specifier.imported && specifier.imported.name)
+              ? specifier.imported.name
+              : '';
+            if (importedName === 'HttpClient' && source === '@angular/common/http') {
+              hasHttpClient = true;
+            }
+            if (API_SERVICE_NAME_REGEX.test(importedName)) {
+              apiServiceNames.push(importedName);
+            }
+          }
+
+          if (apiServiceNames.length > 0) {
+            context.report({
+              node: node,
+              messageId: 'noApiServiceImport',
+              data: { names: apiServiceNames.join(', ') },
+            });
+          }
+          if (hasHttpClient) {
+            context.report({
+              node: node,
+              messageId: 'noHttpClientImport',
+            });
+          }
+        } catch (error) {
+          console.error('[no-component-direct-api] Error in ImportDeclaration visitor:', error);
+        }
+      },
+    };
+  },
+};
+
+export default rule;

package/src/plugin/rules/angular/no-method-call-in-template.spec.ts

@@ -1,63 +1,63 @@
-import noMethodCallInTemplateRule from './no-method-call-in-template';
-
-describe('| no-method-call-in-template', () => {
-  it('| should be a valid ESLint rule', () => {
-    expect(noMethodCallInTemplateRule.meta?.type).toBe('suggestion');
-    expect(noMethodCallInTemplateRule.meta?.messages?.interpolationCall).toBeDefined();
-    expect(noMethodCallInTemplateRule.meta?.messages?.propertyBindingCall).toBeDefined();
-    expect(noMethodCallInTemplateRule.meta?.messages?.blockCall).toBeDefined();
-  });
-
-  it('| should skip non-html files', () => {
-    const mockContext: any = { filename: '/x/foo.ts', report: () => {}, options: [], sourceCode: { text: '' } };
-    const result: any = noMethodCallInTemplateRule.create(mockContext);
-    expect(result.Program).toBeUndefined();
-  });
-
-  it('| should report interpolation method call `{{ getName() }}`', () => {
-    let reportCount: number = 0;
-    const mockContext: any = {
-      filename: '/x/user.component.html',
-      report: (options: any): void => {
-        if (options.messageId === 'interpolationCall') {
-          reportCount += 1;
-        }
-      },
-      options: [],
-      sourceCode: { text: '<div>{{ getName() }}</div>' },
-    };
-    const rule: any = noMethodCallInTemplateRule.create(mockContext);
-    rule.Program();
-    expect(reportCount).toBeGreaterThan(0);
-  });
-
-  it('| should NOT report signal call `{{ name_$() }}`', () => {
-    let reportCount: number = 0;
-    const mockContext: any = {
-      filename: '/x/user.component.html',
-      report: (): void => { reportCount += 1; },
-      options: [],
-      sourceCode: { text: '<div>{{ name_$() }}</div>' },
-    };
-    const rule: any = noMethodCallInTemplateRule.create(mockContext);
-    rule.Program();
-    expect(reportCount).toBe(0);
-  });
-
-  it('| should report `@if (isActive())` block call', () => {
-    let blockReported: boolean = false;
-    const mockContext: any = {
-      filename: '/x/user.component.html',
-      report: (options: any): void => {
-        if (options.messageId === 'blockCall') {
-          blockReported = true;
-        }
-      },
-      options: [],
-      sourceCode: { text: '@if (isActive()) { <span>X</span> }' },
-    };
-    const rule: any = noMethodCallInTemplateRule.create(mockContext);
-    rule.Program();
-    expect(blockReported).toBe(true);
-  });
-});
+import noMethodCallInTemplateRule from './no-method-call-in-template';
+
+describe('| no-method-call-in-template', () => {
+  it('| should be a valid ESLint rule', () => {
+    expect(noMethodCallInTemplateRule.meta?.type).toBe('suggestion');
+    expect(noMethodCallInTemplateRule.meta?.messages?.interpolationCall).toBeDefined();
+    expect(noMethodCallInTemplateRule.meta?.messages?.propertyBindingCall).toBeDefined();
+    expect(noMethodCallInTemplateRule.meta?.messages?.blockCall).toBeDefined();
+  });
+
+  it('| should skip non-html files', () => {
+    const mockContext: any = { filename: '/x/foo.ts', report: () => {}, options: [], sourceCode: { text: '' } };
+    const result: any = noMethodCallInTemplateRule.create(mockContext);
+    expect(result.Program).toBeUndefined();
+  });
+
+  it('| should report interpolation method call `{{ getName() }}`', () => {
+    let reportCount: number = 0;
+    const mockContext: any = {
+      filename: '/x/user.component.html',
+      report: (options: any): void => {
+        if (options.messageId === 'interpolationCall') {
+          reportCount += 1;
+        }
+      },
+      options: [],
+      sourceCode: { text: '<div>{{ getName() }}</div>' },
+    };
+    const rule: any = noMethodCallInTemplateRule.create(mockContext);
+    rule.Program();
+    expect(reportCount).toBeGreaterThan(0);
+  });
+
+  it('| should NOT report signal call `{{ name_$() }}`', () => {
+    let reportCount: number = 0;
+    const mockContext: any = {
+      filename: '/x/user.component.html',
+      report: (): void => { reportCount += 1; },
+      options: [],
+      sourceCode: { text: '<div>{{ name_$() }}</div>' },
+    };
+    const rule: any = noMethodCallInTemplateRule.create(mockContext);
+    rule.Program();
+    expect(reportCount).toBe(0);
+  });
+
+  it('| should report `@if (isActive())` block call', () => {
+    let blockReported: boolean = false;
+    const mockContext: any = {
+      filename: '/x/user.component.html',
+      report: (options: any): void => {
+        if (options.messageId === 'blockCall') {
+          blockReported = true;
+        }
+      },
+      options: [],
+      sourceCode: { text: '@if (isActive()) { <span>X</span> }' },
+    };
+    const rule: any = noMethodCallInTemplateRule.create(mockContext);
+    rule.Program();
+    expect(blockReported).toBe(true);
+  });
+});