@futdevpro/dynamo-eslint 1.15.25 → 1.20.2

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

@@ -1,89 +1,89 @@
-import { Rule } from 'eslint';
-
-/**
- * Interpoláció metódushívás regex: `{{ ... method() ... }}`. Signal hívás (`_$()`) kizárva.
- * Capture group 1: a hívott metódus neve.
- */
-const INTERPOLATION_CALL_REGEX: RegExp =
-  /\{\{[^}]*\b([a-zA-Z_$][a-zA-Z0-9_$]*(?<!_\$))\s*\([^)]*\)[^}]*\}\}/g;
-
-/**
- * Property binding metódushívás regex: `[attr]="method()"`. Esemény binding `(event)="..."` kizárva.
- */
-const PROPERTY_BINDING_CALL_REGEX: RegExp =
-  /\[[^\]]+\]="[^"]*\b[a-zA-Z_$][a-zA-Z0-9_$]*(?<!_\$)\s*\([^)]*\)[^"]*"/g;
-
-/**
- * Angular block (`@if`, `@switch`, `@case`, `@else if`) metódushívás regex.
- */
-const BLOCK_CALL_REGEX: RegExp =
-  /@(?:if|switch|case|else if)\s*\([^)]*\b[a-zA-Z_$][a-zA-Z0-9_$]*(?<!_\$)\s*\([^)]*\)/g;
-
-/**
- * Adott offset alapján a sor és oszlop kiszámítása az alap szövegből.
- */
-function computeLoc(text: string, index: number): { line: number; column: number } {
-  const before: string = text.substring(0, index);
-  const newlineMatches: RegExpMatchArray | null = before.match(/\n/g);
-  const line: number = (newlineMatches ? newlineMatches.length : 0) + 1;
-  const lastNewline: number = before.lastIndexOf('\n');
-  const column: number = lastNewline === -1 ? index : index - lastNewline - 1;
-  return { line: line, column: column };
-}
-
-/**
- * No-method-call-in-template szabály: Angular template-ekben tilos metódust hívni
- * adat megjelenítéshez. Megengedett: signal `mySignal_$()` és esemény `(click)="handler()"`.
- * CCAP "no-method-call-in-template" review ESLint port (template parser source-text scan).
- */
-const rule: Rule.RuleModule = {
-  meta: {
-    type: 'suggestion',
-    docs: {
-      description:
-        'Forbid method calls in Angular templates for data display; '
-        + 'use signals / computed / pipes instead. Event bindings `(click)="..."` allowed.',
-      recommended: true,
-    },
-    schema: [],
-    messages: {
-      interpolationCall: 'Metódushívás interpolációban: `{{ method() }}`. Használj signal-t vagy pipe-ot.',
-      propertyBindingCall: 'Metódushívás property bindingban. Használj signal-t vagy computed-t.',
-      blockCall: 'Metódushívás Angular block direktívában (`@if` / `@switch`). Használj signal-t vagy computed-t.',
-    },
-  },
-  create(context: Rule.RuleContext): Rule.RuleListener {
-    const filename: string = context.filename ?? '';
-    if (!filename.endsWith('.html')) {
-      return {};
-    }
-    const text: string = context.sourceCode.text ?? '';
-
-    function reportMatches(regex: RegExp, messageId: string): void {
-      const localRegex: RegExp = new RegExp(regex.source, 'g');
-      let match: RegExpExecArray | null = localRegex.exec(text);
-      while (match !== null) {
-        const loc: { line: number; column: number } = computeLoc(text, match.index);
-        context.report({
-          loc: { start: loc, end: { line: loc.line, column: loc.column + match[0].length } },
-          messageId: messageId,
-        });
-        match = localRegex.exec(text);
-      }
-    }
-
-    return {
-      Program(): void {
-        try {
-          reportMatches(INTERPOLATION_CALL_REGEX, 'interpolationCall');
-          reportMatches(PROPERTY_BINDING_CALL_REGEX, 'propertyBindingCall');
-          reportMatches(BLOCK_CALL_REGEX, 'blockCall');
-        } catch (error) {
-          console.error('[no-method-call-in-template] Error in Program visitor:', error);
-        }
-      },
-    };
-  },
-};
-
-export default rule;
+import { Rule } from 'eslint';
+
+/**
+ * Interpoláció metódushívás regex: `{{ ... method() ... }}`. Signal hívás (`_$()`) kizárva.
+ * Capture group 1: a hívott metódus neve.
+ */
+const INTERPOLATION_CALL_REGEX: RegExp =
+  /\{\{[^}]*\b([a-zA-Z_$][a-zA-Z0-9_$]*(?<!_\$))\s*\([^)]*\)[^}]*\}\}/g;
+
+/**
+ * Property binding metódushívás regex: `[attr]="method()"`. Esemény binding `(event)="..."` kizárva.
+ */
+const PROPERTY_BINDING_CALL_REGEX: RegExp =
+  /\[[^\]]+\]="[^"]*\b[a-zA-Z_$][a-zA-Z0-9_$]*(?<!_\$)\s*\([^)]*\)[^"]*"/g;
+
+/**
+ * Angular block (`@if`, `@switch`, `@case`, `@else if`) metódushívás regex.
+ */
+const BLOCK_CALL_REGEX: RegExp =
+  /@(?:if|switch|case|else if)\s*\([^)]*\b[a-zA-Z_$][a-zA-Z0-9_$]*(?<!_\$)\s*\([^)]*\)/g;
+
+/**
+ * Adott offset alapján a sor és oszlop kiszámítása az alap szövegből.
+ */
+function computeLoc(text: string, index: number): { line: number; column: number } {
+  const before: string = text.substring(0, index);
+  const newlineMatches: RegExpMatchArray | null = before.match(/\n/g);
+  const line: number = (newlineMatches ? newlineMatches.length : 0) + 1;
+  const lastNewline: number = before.lastIndexOf('\n');
+  const column: number = lastNewline === -1 ? index : index - lastNewline - 1;
+  return { line: line, column: column };
+}
+
+/**
+ * No-method-call-in-template szabály: Angular template-ekben tilos metódust hívni
+ * adat megjelenítéshez. Megengedett: signal `mySignal_$()` és esemény `(click)="handler()"`.
+ * CCAP "no-method-call-in-template" review ESLint port (template parser source-text scan).
+ */
+const rule: Rule.RuleModule = {
+  meta: {
+    type: 'suggestion',
+    docs: {
+      description:
+        'Forbid method calls in Angular templates for data display; '
+        + 'use signals / computed / pipes instead. Event bindings `(click)="..."` allowed.',
+      recommended: true,
+    },
+    schema: [],
+    messages: {
+      interpolationCall: 'Metódushívás interpolációban: `{{ method() }}`. Használj signal-t vagy pipe-ot.',
+      propertyBindingCall: 'Metódushívás property bindingban. Használj signal-t vagy computed-t.',
+      blockCall: 'Metódushívás Angular block direktívában (`@if` / `@switch`). Használj signal-t vagy computed-t.',
+    },
+  },
+  create(context: Rule.RuleContext): Rule.RuleListener {
+    const filename: string = context.filename ?? '';
+    if (!filename.endsWith('.html')) {
+      return {};
+    }
+    const text: string = context.sourceCode.text ?? '';
+
+    function reportMatches(regex: RegExp, messageId: string): void {
+      const localRegex: RegExp = new RegExp(regex.source, 'g');
+      let match: RegExpExecArray | null = localRegex.exec(text);
+      while (match !== null) {
+        const loc: { line: number; column: number } = computeLoc(text, match.index);
+        context.report({
+          loc: { start: loc, end: { line: loc.line, column: loc.column + match[0].length } },
+          messageId: messageId,
+        });
+        match = localRegex.exec(text);
+      }
+    }
+
+    return {
+      Program(): void {
+        try {
+          reportMatches(INTERPOLATION_CALL_REGEX, 'interpolationCall');
+          reportMatches(PROPERTY_BINDING_CALL_REGEX, 'propertyBindingCall');
+          reportMatches(BLOCK_CALL_REGEX, 'blockCall');
+        } catch (error) {
+          console.error('[no-method-call-in-template] Error in Program visitor:', error);
+        }
+      },
+    };
+  },
+};
+
+export default rule;

package/src/plugin/rules/angular/no-ngfor-directive.spec.ts

@@ -1,44 +1,44 @@
-import noNgForDirectiveRule from './no-ngfor-directive';
-
-describe('| no-ngfor-directive', () => {
-  it('| should be a valid ESLint rule', () => {
-    expect(noNgForDirectiveRule.meta?.type).toBe('suggestion');
-    expect(noNgForDirectiveRule.meta?.messages?.noNgFor).toBeDefined();
-  });
-
-  it('| should skip non-html files', () => {
-    const mockContext: any = { filename: '/x/foo.ts', report: () => {}, options: [], sourceCode: { text: '' } };
-    const result: any = noNgForDirectiveRule.create(mockContext);
-    expect(result.Program).toBeUndefined();
-  });
-
-  it('| should report `*ngFor`', () => {
-    let reportCount: number = 0;
-    const mockContext: any = {
-      filename: '/x/list.component.html',
-      report: (options: any): void => {
-        if (options.messageId === 'noNgFor') {
-          reportCount += 1;
-        }
-      },
-      options: [],
-      sourceCode: { text: '<div *ngFor="let item of items">X</div>' },
-    };
-    const rule: any = noNgForDirectiveRule.create(mockContext);
-    rule.Program();
-    expect(reportCount).toBeGreaterThan(0);
-  });
-
-  it('| should NOT report `@for` block', () => {
-    let reportCount: number = 0;
-    const mockContext: any = {
-      filename: '/x/list.component.html',
-      report: (): void => { reportCount += 1; },
-      options: [],
-      sourceCode: { text: '@for (item of items; track item.id) { <div>X</div> }' },
-    };
-    const rule: any = noNgForDirectiveRule.create(mockContext);
-    rule.Program();
-    expect(reportCount).toBe(0);
-  });
-});
+import noNgForDirectiveRule from './no-ngfor-directive';
+
+describe('| no-ngfor-directive', () => {
+  it('| should be a valid ESLint rule', () => {
+    expect(noNgForDirectiveRule.meta?.type).toBe('suggestion');
+    expect(noNgForDirectiveRule.meta?.messages?.noNgFor).toBeDefined();
+  });
+
+  it('| should skip non-html files', () => {
+    const mockContext: any = { filename: '/x/foo.ts', report: () => {}, options: [], sourceCode: { text: '' } };
+    const result: any = noNgForDirectiveRule.create(mockContext);
+    expect(result.Program).toBeUndefined();
+  });
+
+  it('| should report `*ngFor`', () => {
+    let reportCount: number = 0;
+    const mockContext: any = {
+      filename: '/x/list.component.html',
+      report: (options: any): void => {
+        if (options.messageId === 'noNgFor') {
+          reportCount += 1;
+        }
+      },
+      options: [],
+      sourceCode: { text: '<div *ngFor="let item of items">X</div>' },
+    };
+    const rule: any = noNgForDirectiveRule.create(mockContext);
+    rule.Program();
+    expect(reportCount).toBeGreaterThan(0);
+  });
+
+  it('| should NOT report `@for` block', () => {
+    let reportCount: number = 0;
+    const mockContext: any = {
+      filename: '/x/list.component.html',
+      report: (): void => { reportCount += 1; },
+      options: [],
+      sourceCode: { text: '@for (item of items; track item.id) { <div>X</div> }' },
+    };
+    const rule: any = noNgForDirectiveRule.create(mockContext);
+    rule.Program();
+    expect(reportCount).toBe(0);
+  });
+});

package/src/plugin/rules/angular/no-ngfor-directive.ts

@@ -1,62 +1,62 @@
-import { Rule } from 'eslint';
-
-/** `*ngFor` használat detektálása HTML template-ekben. */
-const NGFOR_REGEX: RegExp = /\*ngFor/g;
-
-/**
- * Adott offset alapján a sor és oszlop kiszámítása az alap szövegből.
- */
-function computeLoc(text: string, index: number): { line: number; column: number } {
-  const before: string = text.substring(0, index);
-  const newlineMatches: RegExpMatchArray | null = before.match(/\n/g);
-  const line: number = (newlineMatches ? newlineMatches.length : 0) + 1;
-  const lastNewline: number = before.lastIndexOf('\n');
-  const column: number = lastNewline === -1 ? index : index - lastNewline - 1;
-  return { line: line, column: column };
-}
-
-/**
- * No-ngFor-directive szabály: Angular template-ekben `*ngFor` helyett `@for` szintaxist használj.
- * CCAP "no-ngfor-directive" review ESLint template-text-scan portja.
- */
-const rule: Rule.RuleModule = {
-  meta: {
-    type: 'suggestion',
-    docs: {
-      description: 'Forbid `*ngFor` directive in Angular templates; use `@for` block syntax instead.',
-      recommended: true,
-    },
-    schema: [],
-    messages: {
-      noNgFor: '`*ngFor` direktíva tilos. Használj `@for` block szintaxist.',
-    },
-  },
-  create(context: Rule.RuleContext): Rule.RuleListener {
-    const filename: string = context.filename ?? '';
-    if (!filename.endsWith('.html')) {
-      return {};
-    }
-    const text: string = context.sourceCode.text ?? '';
-
-    return {
-      Program(): void {
-        try {
-          const localRegex: RegExp = new RegExp(NGFOR_REGEX.source, 'g');
-          let match: RegExpExecArray | null = localRegex.exec(text);
-          while (match !== null) {
-            const loc: { line: number; column: number } = computeLoc(text, match.index);
-            context.report({
-              loc: { start: loc, end: { line: loc.line, column: loc.column + match[0].length } },
-              messageId: 'noNgFor',
-            });
-            match = localRegex.exec(text);
-          }
-        } catch (error) {
-          console.error('[no-ngfor-directive] Error in Program visitor:', error);
-        }
-      },
-    };
-  },
-};
-
-export default rule;
+import { Rule } from 'eslint';
+
+/** `*ngFor` használat detektálása HTML template-ekben. */
+const NGFOR_REGEX: RegExp = /\*ngFor/g;
+
+/**
+ * Adott offset alapján a sor és oszlop kiszámítása az alap szövegből.
+ */
+function computeLoc(text: string, index: number): { line: number; column: number } {
+  const before: string = text.substring(0, index);
+  const newlineMatches: RegExpMatchArray | null = before.match(/\n/g);
+  const line: number = (newlineMatches ? newlineMatches.length : 0) + 1;
+  const lastNewline: number = before.lastIndexOf('\n');
+  const column: number = lastNewline === -1 ? index : index - lastNewline - 1;
+  return { line: line, column: column };
+}
+
+/**
+ * No-ngFor-directive szabály: Angular template-ekben `*ngFor` helyett `@for` szintaxist használj.
+ * CCAP "no-ngfor-directive" review ESLint template-text-scan portja.
+ */
+const rule: Rule.RuleModule = {
+  meta: {
+    type: 'suggestion',
+    docs: {
+      description: 'Forbid `*ngFor` directive in Angular templates; use `@for` block syntax instead.',
+      recommended: true,
+    },
+    schema: [],
+    messages: {
+      noNgFor: '`*ngFor` direktíva tilos. Használj `@for` block szintaxist.',
+    },
+  },
+  create(context: Rule.RuleContext): Rule.RuleListener {
+    const filename: string = context.filename ?? '';
+    if (!filename.endsWith('.html')) {
+      return {};
+    }
+    const text: string = context.sourceCode.text ?? '';
+
+    return {
+      Program(): void {
+        try {
+          const localRegex: RegExp = new RegExp(NGFOR_REGEX.source, 'g');
+          let match: RegExpExecArray | null = localRegex.exec(text);
+          while (match !== null) {
+            const loc: { line: number; column: number } = computeLoc(text, match.index);
+            context.report({
+              loc: { start: loc, end: { line: loc.line, column: loc.column + match[0].length } },
+              messageId: 'noNgFor',
+            });
+            match = localRegex.exec(text);
+          }
+        } catch (error) {
+          console.error('[no-ngfor-directive] Error in Program visitor:', error);
+        }
+      },
+    };
+  },
+};
+
+export default rule;

package/src/plugin/rules/angular/no-ngif-directive.spec.ts

@@ -1,44 +1,44 @@
-import noNgIfDirectiveRule from './no-ngif-directive';
-
-describe('| no-ngif-directive', () => {
-  it('| should be a valid ESLint rule', () => {
-    expect(noNgIfDirectiveRule.meta?.type).toBe('suggestion');
-    expect(noNgIfDirectiveRule.meta?.messages?.noNgIf).toBeDefined();
-  });
-
-  it('| should skip non-html files', () => {
-    const mockContext: any = { filename: '/x/foo.ts', report: () => {}, options: [], sourceCode: { text: '' } };
-    const result: any = noNgIfDirectiveRule.create(mockContext);
-    expect(result.Program).toBeUndefined();
-  });
-
-  it('| should report `*ngIf`', () => {
-    let reportCount: number = 0;
-    const mockContext: any = {
-      filename: '/x/user.component.html',
-      report: (options: any): void => {
-        if (options.messageId === 'noNgIf') {
-          reportCount += 1;
-        }
-      },
-      options: [],
-      sourceCode: { text: '<div *ngIf="condition">X</div>' },
-    };
-    const rule: any = noNgIfDirectiveRule.create(mockContext);
-    rule.Program();
-    expect(reportCount).toBeGreaterThan(0);
-  });
-
-  it('| should NOT report `@if` block', () => {
-    let reportCount: number = 0;
-    const mockContext: any = {
-      filename: '/x/user.component.html',
-      report: (): void => { reportCount += 1; },
-      options: [],
-      sourceCode: { text: '@if (condition) { <div>X</div> }' },
-    };
-    const rule: any = noNgIfDirectiveRule.create(mockContext);
-    rule.Program();
-    expect(reportCount).toBe(0);
-  });
-});
+import noNgIfDirectiveRule from './no-ngif-directive';
+
+describe('| no-ngif-directive', () => {
+  it('| should be a valid ESLint rule', () => {
+    expect(noNgIfDirectiveRule.meta?.type).toBe('suggestion');
+    expect(noNgIfDirectiveRule.meta?.messages?.noNgIf).toBeDefined();
+  });
+
+  it('| should skip non-html files', () => {
+    const mockContext: any = { filename: '/x/foo.ts', report: () => {}, options: [], sourceCode: { text: '' } };
+    const result: any = noNgIfDirectiveRule.create(mockContext);
+    expect(result.Program).toBeUndefined();
+  });
+
+  it('| should report `*ngIf`', () => {
+    let reportCount: number = 0;
+    const mockContext: any = {
+      filename: '/x/user.component.html',
+      report: (options: any): void => {
+        if (options.messageId === 'noNgIf') {
+          reportCount += 1;
+        }
+      },
+      options: [],
+      sourceCode: { text: '<div *ngIf="condition">X</div>' },
+    };
+    const rule: any = noNgIfDirectiveRule.create(mockContext);
+    rule.Program();
+    expect(reportCount).toBeGreaterThan(0);
+  });
+
+  it('| should NOT report `@if` block', () => {
+    let reportCount: number = 0;
+    const mockContext: any = {
+      filename: '/x/user.component.html',
+      report: (): void => { reportCount += 1; },
+      options: [],
+      sourceCode: { text: '@if (condition) { <div>X</div> }' },
+    };
+    const rule: any = noNgIfDirectiveRule.create(mockContext);
+    rule.Program();
+    expect(reportCount).toBe(0);
+  });
+});

package/src/plugin/rules/angular/no-ngif-directive.ts

@@ -1,62 +1,62 @@
-import { Rule } from 'eslint';
-
-/** `*ngIf` használat detektálása HTML template-ekben. */
-const NGIF_REGEX: RegExp = /\*ngIf/g;
-
-/**
- * Adott offset alapján a sor és oszlop kiszámítása az alap szövegből.
- */
-function computeLoc(text: string, index: number): { line: number; column: number } {
-  const before: string = text.substring(0, index);
-  const newlineMatches: RegExpMatchArray | null = before.match(/\n/g);
-  const line: number = (newlineMatches ? newlineMatches.length : 0) + 1;
-  const lastNewline: number = before.lastIndexOf('\n');
-  const column: number = lastNewline === -1 ? index : index - lastNewline - 1;
-  return { line: line, column: column };
-}
-
-/**
- * No-ngIf-directive szabály: Angular template-ekben `*ngIf` helyett `@if` szintaxist használj.
- * CCAP "no-ngif-directive" review ESLint template-text-scan portja.
- */
-const rule: Rule.RuleModule = {
-  meta: {
-    type: 'suggestion',
-    docs: {
-      description: 'Forbid `*ngIf` directive in Angular templates; use `@if` block syntax instead.',
-      recommended: true,
-    },
-    schema: [],
-    messages: {
-      noNgIf: '`*ngIf` direktíva tilos. Használj `@if` block szintaxist.',
-    },
-  },
-  create(context: Rule.RuleContext): Rule.RuleListener {
-    const filename: string = context.filename ?? '';
-    if (!filename.endsWith('.html')) {
-      return {};
-    }
-    const text: string = context.sourceCode.text ?? '';
-
-    return {
-      Program(): void {
-        try {
-          const localRegex: RegExp = new RegExp(NGIF_REGEX.source, 'g');
-          let match: RegExpExecArray | null = localRegex.exec(text);
-          while (match !== null) {
-            const loc: { line: number; column: number } = computeLoc(text, match.index);
-            context.report({
-              loc: { start: loc, end: { line: loc.line, column: loc.column + match[0].length } },
-              messageId: 'noNgIf',
-            });
-            match = localRegex.exec(text);
-          }
-        } catch (error) {
-          console.error('[no-ngif-directive] Error in Program visitor:', error);
-        }
-      },
-    };
-  },
-};
-
-export default rule;
+import { Rule } from 'eslint';
+
+/** `*ngIf` használat detektálása HTML template-ekben. */
+const NGIF_REGEX: RegExp = /\*ngIf/g;
+
+/**
+ * Adott offset alapján a sor és oszlop kiszámítása az alap szövegből.
+ */
+function computeLoc(text: string, index: number): { line: number; column: number } {
+  const before: string = text.substring(0, index);
+  const newlineMatches: RegExpMatchArray | null = before.match(/\n/g);
+  const line: number = (newlineMatches ? newlineMatches.length : 0) + 1;
+  const lastNewline: number = before.lastIndexOf('\n');
+  const column: number = lastNewline === -1 ? index : index - lastNewline - 1;
+  return { line: line, column: column };
+}
+
+/**
+ * No-ngIf-directive szabály: Angular template-ekben `*ngIf` helyett `@if` szintaxist használj.
+ * CCAP "no-ngif-directive" review ESLint template-text-scan portja.
+ */
+const rule: Rule.RuleModule = {
+  meta: {
+    type: 'suggestion',
+    docs: {
+      description: 'Forbid `*ngIf` directive in Angular templates; use `@if` block syntax instead.',
+      recommended: true,
+    },
+    schema: [],
+    messages: {
+      noNgIf: '`*ngIf` direktíva tilos. Használj `@if` block szintaxist.',
+    },
+  },
+  create(context: Rule.RuleContext): Rule.RuleListener {
+    const filename: string = context.filename ?? '';
+    if (!filename.endsWith('.html')) {
+      return {};
+    }
+    const text: string = context.sourceCode.text ?? '';
+
+    return {
+      Program(): void {
+        try {
+          const localRegex: RegExp = new RegExp(NGIF_REGEX.source, 'g');
+          let match: RegExpExecArray | null = localRegex.exec(text);
+          while (match !== null) {
+            const loc: { line: number; column: number } = computeLoc(text, match.index);
+            context.report({
+              loc: { start: loc, end: { line: loc.line, column: loc.column + match[0].length } },
+              messageId: 'noNgIf',
+            });
+            match = localRegex.exec(text);
+          }
+        } catch (error) {
+          console.error('[no-ngif-directive] Error in Program visitor:', error);
+        }
+      },
+    };
+  },
+};
+
+export default rule;