@futdevpro/dynamo-eslint 1.15.25 → 1.20.2

package/src/configs/ngx.ts

@@ -1,96 +1,93 @@
 const ngxBaseConfig = require('./base');
-const angularPlugin = require('@angular-eslint/eslint-plugin');
-const angularTemplatePlugin = require('@angular-eslint/eslint-plugin-template');
-const angularTemplateParser = require('@angular-eslint/template-parser');
-
-/**
- * Rulesets For Dynamo Based Systems
- * NGX: Angular Projects
- * ESLint v9 Flat Config Format
- */
-module.exports = [
-  ...ngxBaseConfig,
-  {
-    files: ['**/*.ts', '**/*.tsx'],
-    languageOptions: {
-      ...ngxBaseConfig[1].languageOptions,
-      globals: {
-        // Angular globals
-        window: 'readonly',
-        document: 'readonly',
-        console: 'readonly',
-        process: 'readonly',
-        Buffer: 'readonly',
-        global: 'readonly',
-        __dirname: 'readonly',
-        __filename: 'readonly',
-        module: 'readonly',
-        require: 'readonly',
-        exports: 'readonly'
-      }
-    },
-    plugins: {
-      ...ngxBaseConfig[1].plugins,
-      '@angular-eslint': angularPlugin,
-    },
-    rules: {
-      ...ngxBaseConfig[1].rules,
-      // Angular-specific rules
-      '@angular-eslint/directive-selector': 'error',
-      '@angular-eslint/component-selector': 'error',
-      '@angular-eslint/no-conflicting-lifecycle': 'error',
-      '@angular-eslint/no-host-metadata-property': 'error',
-      '@angular-eslint/no-input-rename': 'error',
-      '@angular-eslint/no-output-rename': 'error',
-      '@angular-eslint/no-outputs-metadata-property': 'error',
-      '@angular-eslint/use-lifecycle-interface': 'error',
-      '@angular-eslint/use-pipe-transform-interface': 'error',
-
-      // CCAP-portolt Angular TypeScript review-k.
-      '@futdevpro/dynamo/no-async-ngoninit': 'warn',
-      '@futdevpro/dynamo/no-component-direct-api': 'warn',
-      '@futdevpro/dynamo/effect-signal-loop': 'warn',
-    },
-  },
-  {
-    files: ['**/*.html'],
-    languageOptions: {
-      parser: angularTemplateParser,
-      parserOptions: {
-        ecmaVersion: 'latest',
-        sourceType: 'module',
-      },
-    },
-    plugins: {
-      ...ngxBaseConfig[1].plugins,
-      '@angular-eslint/template': angularTemplatePlugin,
-    },
-    rules: {
-      '@angular-eslint/template/banana-in-box': 'error',
-      '@angular-eslint/template/cyclomatic-complexity': 'error',
-      '@angular-eslint/template/eqeqeq': 'error',
-      // Signal-first kódban (Angular 17+) a `params.data_$()`, `model_$()` stb.
-      // szignal-getter hivasok DESIGN-ALTAL kerulnek a templatebe — olcso, ket
-      // resugarzo, change-detection-friendly. Az eredeti `'error'` szint még a
-      // zone.js-CD korszakra szabva volt; a deprecation altalanos.
-      // 2026-06-05: warn-ra allitva, hogy a Signal-first komponensek (button-new,
-      // form-new, field-new, ...) build-jet ne tortje, viszont legacy
-      // zone.js-alapú expensive-call patternek meg felgyulladjanak.
-      // 2026-06-23: a signal-getterek (`_$`-suffix) NARROW exemptet kapnak (allowSuffix),
-      // mert a `()` invocation Angular 18 signal-ben a KANONIKUS read-pattern, nem anti-pattern;
-      // a valodi (nem-signal) metodus-hivasok templateben tovabbra is warn-olnak.
-      '@angular-eslint/template/no-call-expression': [ 'warn', { allowSuffix: '_$' }],
-      '@angular-eslint/template/no-duplicate-attributes': 'error',
-      '@angular-eslint/template/no-negated-async': 'error',
-      '@angular-eslint/template/use-track-by-function': 'error',
-
-      // CCAP-portolt Angular template review-k (regex-alapú text scan).
-      '@futdevpro/dynamo/no-method-call-in-template': 'warn',
-      '@futdevpro/dynamo/angular-for-track-logic': 'warn',
-      '@futdevpro/dynamo/no-ngif-directive': 'warn',
-      '@futdevpro/dynamo/no-ngfor-directive': 'warn',
-    },
-  },
-];
-
-
+const angularEslint = require('angular-eslint');
+
+/**
+ * Rulesets For Dynamo Based Systems
+ * NGX: Angular Projects
+ * ESLint v9 Flat Config Format
+ */
+module.exports = [
+  ...ngxBaseConfig,
+  {
+    files: ['**/*.ts', '**/*.tsx'],
+    languageOptions: {
+      ...ngxBaseConfig[1].languageOptions,
+      globals: {
+        // Angular globals
+        window: 'readonly',
+        document: 'readonly',
+        console: 'readonly',
+        process: 'readonly',
+        Buffer: 'readonly',
+        global: 'readonly',
+        __dirname: 'readonly',
+        __filename: 'readonly',
+        module: 'readonly',
+        require: 'readonly',
+        exports: 'readonly'
+      }
+    },
+    plugins: {
+      ...ngxBaseConfig[1].plugins,
+      '@angular-eslint': angularEslint.tsPlugin,
+    },
+    rules: {
+      ...ngxBaseConfig[1].rules,
+      // Angular-specific rules
+      '@angular-eslint/directive-selector': 'error',
+      '@angular-eslint/component-selector': 'error',
+      '@angular-eslint/no-conflicting-lifecycle': 'error',
+      '@angular-eslint/no-input-rename': 'error',
+      '@angular-eslint/no-output-rename': 'error',
+      '@angular-eslint/no-outputs-metadata-property': 'error',
+      '@angular-eslint/use-lifecycle-interface': 'error',
+      '@angular-eslint/use-pipe-transform-interface': 'error',
+
+      // CCAP-portolt Angular TypeScript review-k.
+      '@futdevpro/dynamo/no-async-ngoninit': 'warn',
+      '@futdevpro/dynamo/no-component-direct-api': 'warn',
+      '@futdevpro/dynamo/effect-signal-loop': 'warn',
+    },
+  },
+  {
+    files: ['**/*.html'],
+    languageOptions: {
+      parser: angularEslint.templateParser,
+      parserOptions: {
+        ecmaVersion: 'latest',
+        sourceType: 'module',
+      },
+    },
+    plugins: {
+      ...ngxBaseConfig[1].plugins,
+      '@angular-eslint/template': angularEslint.templatePlugin,
+    },
+    rules: {
+      '@angular-eslint/template/banana-in-box': 'error',
+      '@angular-eslint/template/cyclomatic-complexity': 'error',
+      '@angular-eslint/template/eqeqeq': 'error',
+      // Signal-first kódban (Angular 17+) a `params.data_$()`, `model_$()` stb.
+      // szignal-getter hivasok DESIGN-ALTAL kerulnek a templatebe — olcso, ket
+      // resugarzo, change-detection-friendly. Az eredeti `'error'` szint még a
+      // zone.js-CD korszakra szabva volt; a deprecation altalanos.
+      // 2026-06-05: warn-ra allitva, hogy a Signal-first komponensek (button-new,
+      // form-new, field-new, ...) build-jet ne tortje, viszont legacy
+      // zone.js-alapú expensive-call patternek meg felgyulladjanak.
+      // 2026-06-23: a signal-getterek (`_$`-suffix) NARROW exemptet kapnak (allowSuffix),
+      // mert a `()` invocation Angular 18 signal-ben a KANONIKUS read-pattern, nem anti-pattern;
+      // a valodi (nem-signal) metodus-hivasok templateben tovabbra is warn-olnak.
+      '@angular-eslint/template/no-call-expression': [ 'warn', { allowSuffix: '_$' }],
+      '@angular-eslint/template/no-duplicate-attributes': 'error',
+      '@angular-eslint/template/no-negated-async': 'error',
+      '@angular-eslint/template/use-track-by-function': 'error',
+
+      // CCAP-portolt Angular template review-k (regex-alapú text scan).
+      '@futdevpro/dynamo/no-method-call-in-template': 'warn',
+      '@futdevpro/dynamo/angular-for-track-logic': 'warn',
+      '@futdevpro/dynamo/no-ngif-directive': 'warn',
+      '@futdevpro/dynamo/no-ngfor-directive': 'warn',
+    },
+  },
+];
+
+

package/src/configs/nts-package.ts

@@ -1,20 +1,20 @@
-const ntsConfig = require('./nts');
-
-/**
- * Rulesets For Dynamo Based Systems
- * NTS-PACKAGE: Node TypeScript Package
- * ESLint v9 Flat Config Format
- */
-module.exports = [
-  ...ntsConfig,
-  {
-    files: ['**/*.ts', '**/*.tsx'],
-    rules: {
-      ...ntsConfig[1].rules,
-      '@typescript-eslint/ban-types': 'off',
-      '@typescript-eslint/no-explicit-any': 'off',
-    },
-  },
-];
-
-
+const ntsConfig = require('./nts');
+
+/**
+ * Rulesets For Dynamo Based Systems
+ * NTS-PACKAGE: Node TypeScript Package
+ * ESLint v9 Flat Config Format
+ */
+module.exports = [
+  ...ntsConfig,
+  {
+    files: ['**/*.ts', '**/*.tsx'],
+    rules: {
+      ...ntsConfig[1].rules,
+      '@typescript-eslint/ban-types': 'off',
+      '@typescript-eslint/no-explicit-any': 'off',
+    },
+  },
+];
+
+

package/src/configs/nts.ts

@@ -1,34 +1,34 @@
-const ntsBaseConfig = require('./base');
-
-/**
- * Rulesets For Dynamo Based Systems
- * NTS: Node TypeScript
- * ESLint v9 Flat Config Format
- */
-module.exports = [
-  ...ntsBaseConfig,
-  {
-    files: ['**/*.ts', '**/*.tsx'],
-    languageOptions: {
-      ...ntsBaseConfig[1].languageOptions,
-      globals: {
-        // Node.js globals
-        process: 'readonly',
-        Buffer: 'readonly',
-        global: 'readonly',
-        __dirname: 'readonly',
-        __filename: 'readonly',
-        module: 'readonly',
-        require: 'readonly',
-        exports: 'readonly',
-        console: 'readonly'
-      }
-    },
-    rules: {
-      ...ntsBaseConfig[1].rules,
-      'max-lines': ['warn', 1500],
-    },
-  },
-];
-
-
+const ntsBaseConfig = require('./base');
+
+/**
+ * Rulesets For Dynamo Based Systems
+ * NTS: Node TypeScript
+ * ESLint v9 Flat Config Format
+ */
+module.exports = [
+  ...ntsBaseConfig,
+  {
+    files: ['**/*.ts', '**/*.tsx'],
+    languageOptions: {
+      ...ntsBaseConfig[1].languageOptions,
+      globals: {
+        // Node.js globals
+        process: 'readonly',
+        Buffer: 'readonly',
+        global: 'readonly',
+        __dirname: 'readonly',
+        __filename: 'readonly',
+        module: 'readonly',
+        require: 'readonly',
+        exports: 'readonly',
+        console: 'readonly'
+      }
+    },
+    rules: {
+      ...ntsBaseConfig[1].rules,
+      'max-lines': ['warn', 1500],
+    },
+  },
+];
+
+

package/src/index.ts

@@ -1,4 +1,4 @@
-// Entry point kept minimal; configs are exported via subpath exports in package.json
-export {};
-
-
+// Entry point kept minimal; configs are exported via subpath exports in package.json
+export {};
+
+

package/src/plugin/index.ts

@@ -1,86 +1,86 @@
-import explicitTypesRule from './rules/explicit-types';
-import importOrderRule from './rules/import/import-order';
-import namingPatternsRule from './rules/naming-patterns';
-import noImportTypeRule from './rules/import/no-import-type';
-import noJsExtensionRule from './rules/import/no-js-import';
-import preferInterfaceRule from './rules/prefer-interface-over-duplicate-types';
-import requireJSDocRule from './rules/require-jsdoc-description';
-import preferEnumRule from './rules/prefer-enum-over-string-union';
-import requireTestPrefixRule from './rules/require-test-description-prefix';
-import noGetterLogicRule from './rules/no-getter-logic';
-
-import noAsCastRule from './rules/ts/no-as-cast';
-import noAnyTypeRule from './rules/ts/no-any-type';
-import noDynamicImportsRule from './rules/ts/no-dynamic-imports';
-import oneExportPerFileRule from './rules/ts/one-export-per-file';
-import noPlainFunctionExportRule from './rules/ts/no-plain-function-export';
-import dataModelExtendsRule from './rules/ts/data-model-extends';
-import noNativeBrowserDialogsRule from './rules/ts/no-native-browser-dialogs';
-
-import noAsyncNgOnInitRule from './rules/angular/no-async-ngoninit';
-import noComponentDirectApiRule from './rules/angular/no-component-direct-api';
-import effectSignalLoopRule from './rules/angular/effect-signal-loop';
-import noMethodCallInTemplateRule from './rules/angular/no-method-call-in-template';
-import angularForTrackLogicRule from './rules/angular/angular-for-track-logic';
-import noNgIfDirectiveRule from './rules/angular/no-ngif-directive';
-import noNgForDirectiveRule from './rules/angular/no-ngfor-directive';
-
-export = {
-  rules: {
-    'explicit-types': explicitTypesRule,
-    'import-order': importOrderRule,
-    'no-import-type': noImportTypeRule,
-    'no-js-import': noJsExtensionRule,
-    'prefer-interface-over-duplicate-types': preferInterfaceRule,
-    'require-jsdoc-description': requireJSDocRule,
-    'prefer-enum-over-string-union': preferEnumRule,
-    'require-test-description-prefix': requireTestPrefixRule,
-    'no-getter-logic': noGetterLogicRule,
-
-    'naming-patterns': namingPatternsRule,
-
-    // CCAP-portolt TypeScript review-k (AST-alapú).
-    'no-as-cast': noAsCastRule,
-    'no-any-type': noAnyTypeRule,
-    'no-dynamic-imports': noDynamicImportsRule,
-    'one-export-per-file': oneExportPerFileRule,
-    'no-plain-function-export': noPlainFunctionExportRule,
-    'data-model-extends': dataModelExtendsRule,
-    'no-native-browser-dialogs': noNativeBrowserDialogsRule,
-
-    // CCAP-portolt Angular review-k (TS + template-szöveg AST-alapú).
-    'no-async-ngoninit': noAsyncNgOnInitRule,
-    'no-component-direct-api': noComponentDirectApiRule,
-    'effect-signal-loop': effectSignalLoopRule,
-    'no-method-call-in-template': noMethodCallInTemplateRule,
-    'angular-for-track-logic': angularForTrackLogicRule,
-    'no-ngif-directive': noNgIfDirectiveRule,
-    'no-ngfor-directive': noNgForDirectiveRule,
-  },
-  configs: {
-    recommended: {
-      rules: {
-        '@futdevpro/dynamo/explicit-types': 'warn',
-        '@futdevpro/dynamo/import-order': 'warn',
-        '@futdevpro/dynamo/no-import-type': 'warn',
-        '@futdevpro/dynamo/no-js-import': 'warn',
-        '@futdevpro/dynamo/prefer-interface-over-duplicate-types': 'warn',
-        '@futdevpro/dynamo/require-jsdoc-description': 'warn',
-        '@futdevpro/dynamo/prefer-enum-over-string-union': 'warn',
-        '@futdevpro/dynamo/require-test-description-prefix': 'warn',
-        '@futdevpro/dynamo/no-getter-logic': 'warn',
-
-        '@futdevpro/dynamo/naming-patterns': 'warn',
-
-        // CCAP-portolt TypeScript review-k (AST-alapú).
-        '@futdevpro/dynamo/no-as-cast': 'warn',
-        '@futdevpro/dynamo/no-any-type': 'warn',
-        '@futdevpro/dynamo/no-dynamic-imports': 'warn',
-        '@futdevpro/dynamo/one-export-per-file': 'warn',
-        '@futdevpro/dynamo/no-plain-function-export': 'warn',
-        '@futdevpro/dynamo/data-model-extends': 'warn',
-        '@futdevpro/dynamo/no-native-browser-dialogs': 'warn',
-      },
-    },
-  },
-};
+import explicitTypesRule from './rules/explicit-types';
+import importOrderRule from './rules/import/import-order';
+import namingPatternsRule from './rules/naming-patterns';
+import noImportTypeRule from './rules/import/no-import-type';
+import noJsExtensionRule from './rules/import/no-js-import';
+import preferInterfaceRule from './rules/prefer-interface-over-duplicate-types';
+import requireJSDocRule from './rules/require-jsdoc-description';
+import preferEnumRule from './rules/prefer-enum-over-string-union';
+import requireTestPrefixRule from './rules/require-test-description-prefix';
+import noGetterLogicRule from './rules/no-getter-logic';
+
+import noAsCastRule from './rules/ts/no-as-cast';
+import noAnyTypeRule from './rules/ts/no-any-type';
+import noDynamicImportsRule from './rules/ts/no-dynamic-imports';
+import oneExportPerFileRule from './rules/ts/one-export-per-file';
+import noPlainFunctionExportRule from './rules/ts/no-plain-function-export';
+import dataModelExtendsRule from './rules/ts/data-model-extends';
+import noNativeBrowserDialogsRule from './rules/ts/no-native-browser-dialogs';
+
+import noAsyncNgOnInitRule from './rules/angular/no-async-ngoninit';
+import noComponentDirectApiRule from './rules/angular/no-component-direct-api';
+import effectSignalLoopRule from './rules/angular/effect-signal-loop';
+import noMethodCallInTemplateRule from './rules/angular/no-method-call-in-template';
+import angularForTrackLogicRule from './rules/angular/angular-for-track-logic';
+import noNgIfDirectiveRule from './rules/angular/no-ngif-directive';
+import noNgForDirectiveRule from './rules/angular/no-ngfor-directive';
+
+export = {
+  rules: {
+    'explicit-types': explicitTypesRule,
+    'import-order': importOrderRule,
+    'no-import-type': noImportTypeRule,
+    'no-js-import': noJsExtensionRule,
+    'prefer-interface-over-duplicate-types': preferInterfaceRule,
+    'require-jsdoc-description': requireJSDocRule,
+    'prefer-enum-over-string-union': preferEnumRule,
+    'require-test-description-prefix': requireTestPrefixRule,
+    'no-getter-logic': noGetterLogicRule,
+
+    'naming-patterns': namingPatternsRule,
+
+    // CCAP-portolt TypeScript review-k (AST-alapú).
+    'no-as-cast': noAsCastRule,
+    'no-any-type': noAnyTypeRule,
+    'no-dynamic-imports': noDynamicImportsRule,
+    'one-export-per-file': oneExportPerFileRule,
+    'no-plain-function-export': noPlainFunctionExportRule,
+    'data-model-extends': dataModelExtendsRule,
+    'no-native-browser-dialogs': noNativeBrowserDialogsRule,
+
+    // CCAP-portolt Angular review-k (TS + template-szöveg AST-alapú).
+    'no-async-ngoninit': noAsyncNgOnInitRule,
+    'no-component-direct-api': noComponentDirectApiRule,
+    'effect-signal-loop': effectSignalLoopRule,
+    'no-method-call-in-template': noMethodCallInTemplateRule,
+    'angular-for-track-logic': angularForTrackLogicRule,
+    'no-ngif-directive': noNgIfDirectiveRule,
+    'no-ngfor-directive': noNgForDirectiveRule,
+  },
+  configs: {
+    recommended: {
+      rules: {
+        '@futdevpro/dynamo/explicit-types': 'warn',
+        '@futdevpro/dynamo/import-order': 'warn',
+        '@futdevpro/dynamo/no-import-type': 'warn',
+        '@futdevpro/dynamo/no-js-import': 'warn',
+        '@futdevpro/dynamo/prefer-interface-over-duplicate-types': 'warn',
+        '@futdevpro/dynamo/require-jsdoc-description': 'warn',
+        '@futdevpro/dynamo/prefer-enum-over-string-union': 'warn',
+        '@futdevpro/dynamo/require-test-description-prefix': 'warn',
+        '@futdevpro/dynamo/no-getter-logic': 'warn',
+
+        '@futdevpro/dynamo/naming-patterns': 'warn',
+
+        // CCAP-portolt TypeScript review-k (AST-alapú).
+        '@futdevpro/dynamo/no-as-cast': 'warn',
+        '@futdevpro/dynamo/no-any-type': 'warn',
+        '@futdevpro/dynamo/no-dynamic-imports': 'warn',
+        '@futdevpro/dynamo/one-export-per-file': 'warn',
+        '@futdevpro/dynamo/no-plain-function-export': 'warn',
+        '@futdevpro/dynamo/data-model-extends': 'warn',
+        '@futdevpro/dynamo/no-native-browser-dialogs': 'warn',
+      },
+    },
+  },
+};

package/src/plugin/rules/angular/angular-for-track-logic.spec.ts

@@ -1,44 +1,44 @@
-import angularForTrackLogicRule from './angular-for-track-logic';
-
-describe('| angular-for-track-logic', () => {
-  it('| should be a valid ESLint rule', () => {
-    expect(angularForTrackLogicRule.meta?.type).toBe('problem');
-    expect(angularForTrackLogicRule.meta?.messages?.trackLogic).toBeDefined();
-  });
-
-  it('| should skip non-html files', () => {
-    const mockContext: any = { filename: '/x/foo.ts', report: () => {}, options: [], sourceCode: { text: '' } };
-    const result: any = angularForTrackLogicRule.create(mockContext);
-    expect(result.Program).toBeUndefined();
-  });
-
-  it('| should NOT report `track item.id` (simple property access)', () => {
-    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 = angularForTrackLogicRule.create(mockContext);
-    rule.Program();
-    expect(reportCount).toBe(0);
-  });
-
-  it('| should report `track method(item)` (logic in track)', () => {
-    let reportCount: number = 0;
-    const mockContext: any = {
-      filename: '/x/list.component.html',
-      report: (options: any): void => {
-        if (options.messageId === 'trackLogic') {
-          reportCount += 1;
-        }
-      },
-      options: [],
-      sourceCode: { text: '@for (item of items; track trackFn(item)) { <div>X</div> }' },
-    };
-    const rule: any = angularForTrackLogicRule.create(mockContext);
-    rule.Program();
-    expect(reportCount).toBeGreaterThan(0);
-  });
-});
+import angularForTrackLogicRule from './angular-for-track-logic';
+
+describe('| angular-for-track-logic', () => {
+  it('| should be a valid ESLint rule', () => {
+    expect(angularForTrackLogicRule.meta?.type).toBe('problem');
+    expect(angularForTrackLogicRule.meta?.messages?.trackLogic).toBeDefined();
+  });
+
+  it('| should skip non-html files', () => {
+    const mockContext: any = { filename: '/x/foo.ts', report: () => {}, options: [], sourceCode: { text: '' } };
+    const result: any = angularForTrackLogicRule.create(mockContext);
+    expect(result.Program).toBeUndefined();
+  });
+
+  it('| should NOT report `track item.id` (simple property access)', () => {
+    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 = angularForTrackLogicRule.create(mockContext);
+    rule.Program();
+    expect(reportCount).toBe(0);
+  });
+
+  it('| should report `track method(item)` (logic in track)', () => {
+    let reportCount: number = 0;
+    const mockContext: any = {
+      filename: '/x/list.component.html',
+      report: (options: any): void => {
+        if (options.messageId === 'trackLogic') {
+          reportCount += 1;
+        }
+      },
+      options: [],
+      sourceCode: { text: '@for (item of items; track trackFn(item)) { <div>X</div> }' },
+    };
+    const rule: any = angularForTrackLogicRule.create(mockContext);
+    rule.Program();
+    expect(reportCount).toBeGreaterThan(0);
+  });
+});