@futdevpro/dynamo-eslint 1.15.25 → 1.20.2

package/scripts/run-coverage-tests.js

@@ -1,24 +1,24 @@
-const { execSync } = require('child_process');
-const path = require('path');
-
-const configPath = path.join(__dirname, '../spec/support/jasmine.coverage.json');
-const tsNodeRegister = require.resolve('ts-node/register/transpile-only');
-
-const env = {
-  ...process.env,
-  NODE_OPTIONS: `--require ${tsNodeRegister}`
-};
-
-try {
-  execSync(`npx jasmine --config="${configPath}"`, {
-    stdio: 'inherit',
-    cwd: path.join(__dirname, '..'),
-    shell: true,
-    env: env
-  });
-} catch (error) {
-  // Exit with the same status code as jasmine (3 = test failures, 1 = other errors)
-  // This allows c8 to still generate coverage reports even if tests fail
-  process.exit(error.status || 1);
-}
-
+const { execSync } = require('child_process');
+const path = require('path');
+
+const configPath = path.join(__dirname, '../spec/support/jasmine.coverage.json');
+const tsNodeRegister = require.resolve('ts-node/register/transpile-only');
+
+const env = {
+  ...process.env,
+  NODE_OPTIONS: `--require ${tsNodeRegister}`
+};
+
+try {
+  execSync(`npx jasmine --config="${configPath}"`, {
+    stdio: 'inherit',
+    cwd: path.join(__dirname, '..'),
+    shell: true,
+    env: env
+  });
+} catch (error) {
+  // Exit with the same status code as jasmine (3 = test failures, 1 = other errors)
+  // This allows c8 to still generate coverage reports even if tests fail
+  process.exit(error.status || 1);
+}
+

package/spec/support/helpers/ts-node-helper.js

@@ -1,9 +1,9 @@
-require('ts-node').register({
-  transpileOnly: true,
-  compilerOptions: {
-    module: 'commonjs',
-    esModuleInterop: true,
-    allowSyntheticDefaultImports: true
-  }
-});
-
+require('ts-node').register({
+  transpileOnly: true,
+  compilerOptions: {
+    module: 'commonjs',
+    esModuleInterop: true,
+    allowSyntheticDefaultImports: true
+  }
+});
+

package/spec/support/jasmine.coverage.json

@@ -1,23 +1,23 @@
-{
-  "reporters": [
-    {
-      "name": "jasmine-json-reporter",
-      "options": {
-        "pretty": true
-      }
-    }
-  ],
-  "spec_dir": "src/",
-  "spec_files": [
-    "**/*.spec.ts",
-    "**/*[sS]pec.ts"
-  ],
-  "helpers": [
-    "spec/support/helpers/ts-node-helper.js"
-  ],
-  "env": {
-    "stopSpecOnExpectationFailure": false,
-    "random": true
-  }
-}
-
+{
+  "reporters": [
+    {
+      "name": "jasmine-json-reporter",
+      "options": {
+        "pretty": true
+      }
+    }
+  ],
+  "spec_dir": "src/",
+  "spec_files": [
+    "**/*.spec.ts",
+    "**/*[sS]pec.ts"
+  ],
+  "helpers": [
+    "spec/support/helpers/ts-node-helper.js"
+  ],
+  "env": {
+    "stopSpecOnExpectationFailure": false,
+    "random": true
+  }
+}
+

package/spec/support/jasmine.json

@@ -1,24 +1,24 @@
-{
-  "reporters": [
-    {
-      "name": "jasmine-json-reporter",
-      "options": {
-        "pretty": true
-      }
-    }
-  ],
-  "spec_dir": "build/",
-  "spec_files": [
-    "**/*.spec.ts",
-    "**/*.spec.js",
-    "**/*[sS]pec.ts"
-  ],
-  "helpers": [
-    "helpers/**/*.ts",
-    "helpers/**/*.js"
-  ],
-  "env": {
-    "stopSpecOnExpectationFailure": false,
-    "random": true
-  }
-}
+{
+  "reporters": [
+    {
+      "name": "jasmine-json-reporter",
+      "options": {
+        "pretty": true
+      }
+    }
+  ],
+  "spec_dir": "build/",
+  "spec_files": [
+    "**/*.spec.ts",
+    "**/*.spec.js",
+    "**/*[sS]pec.ts"
+  ],
+  "helpers": [
+    "helpers/**/*.ts",
+    "helpers/**/*.js"
+  ],
+  "env": {
+    "stopSpecOnExpectationFailure": false,
+    "random": true
+  }
+}

package/src/configs/angular20-compatibility.spec.ts

@@ -0,0 +1,62 @@
+/*
+ * The published config is intentionally CommonJS and ESLint's config surface is dynamically typed.
+ * These narrow test-only exceptions keep the compatibility assertions faithful to the consumer API.
+ */
+/* eslint-disable @futdevpro/dynamo/no-dynamic-imports, @typescript-eslint/no-require-imports */
+/* eslint-disable @futdevpro/dynamo/no-any-type */
+const angularEslint = require('angular-eslint');
+const ngxConfig = require('./ngx');
+
+export {};
+
+describe('| Angular 20 flat-config compatibility', (): void => {
+  it('| resolves every configured Angular TypeScript rule from the unified plugin', (): void => {
+    const typeScriptConfig: any = ngxConfig.find(
+      (config: any): boolean => !!config.plugins?.['@angular-eslint'],
+    );
+
+    expect(typeScriptConfig).toBeDefined();
+    expect(typeScriptConfig.rules['@angular-eslint/no-host-metadata-property']).toBeUndefined();
+
+    Object.keys(typeScriptConfig.rules)
+      .filter((ruleName: string): boolean => ruleName.startsWith('@angular-eslint/'))
+      .forEach((ruleName: string): void => {
+        const pluginRuleName: string = ruleName.replace('@angular-eslint/', '');
+
+        expect(angularEslint.tsPlugin.rules[pluginRuleName])
+          .withContext(`Missing Angular 20 TypeScript rule: ${ruleName}`)
+          .toBeDefined();
+      });
+  });
+
+  it('| resolves every configured Angular template rule from the unified plugin', (): void => {
+    const templateConfig: any = ngxConfig.find(
+      (config: any): boolean => !!config.plugins?.['@angular-eslint/template'],
+    );
+
+    expect(templateConfig).toBeDefined();
+
+    Object.keys(templateConfig.rules)
+      .filter((ruleName: string): boolean => ruleName.startsWith('@angular-eslint/template/'))
+      .forEach((ruleName: string): void => {
+        const pluginRuleName: string = ruleName.replace('@angular-eslint/template/', '');
+
+        expect(angularEslint.templatePlugin.rules[pluginRuleName])
+          .withContext(`Missing Angular 20 template rule: ${ruleName}`)
+          .toBeDefined();
+      });
+  });
+
+  it('| exposes the unified Angular parser and plugin objects in the preset', (): void => {
+    const typeScriptConfig: any = ngxConfig.find(
+      (config: any): boolean => !!config.plugins?.['@angular-eslint'],
+    );
+    const templateConfig: any = ngxConfig.find(
+      (config: any): boolean => !!config.plugins?.['@angular-eslint/template'],
+    );
+
+    expect(typeScriptConfig.plugins['@angular-eslint']).toBe(angularEslint.tsPlugin);
+    expect(templateConfig.plugins['@angular-eslint/template']).toBe(angularEslint.templatePlugin);
+    expect(templateConfig.languageOptions.parser).toBe(angularEslint.templateParser);
+  });
+});

package/src/configs/base.ts

@@ -1,157 +1,156 @@
-
-/**
- * Base Rulesets For Dynamo Based Systems
- * ESLint v9 Flat Config Format
- */
-const tsParser = require('@typescript-eslint/parser');
-const tsPlugin = require('@typescript-eslint/eslint-plugin');
-const unusedImportsPlugin = require('eslint-plugin-unused-imports');
-const maxParamsNoConstructorPlugin = require('eslint-plugin-max-params-no-constructor');
-const dynamoPlugin = require('../plugin');
-
-module.exports = [
-  {
-    ignores: [
-      'node_modules/**',
-      'dist/**',
-      'build/**',
-      '*.d.ts',
-    ],
-  },
-  {
-    files: [ '**/*.ts', '**/*.tsx' ],
-    languageOptions: {
-      parser: tsParser,
-      parserOptions: { 
-        ecmaVersion: 'latest',
-        sourceType: 'module',
-        projectService: true,
-        tsconfigRootDir: process.cwd(),
-      },
-      globals: {
-        // Browser globals
-        window: 'readonly',
-        document: 'readonly',
-        console: 'readonly',
-        process: 'readonly',
-        Buffer: 'readonly',
-        global: 'readonly',
-        __dirname: 'readonly',
-        __filename: 'readonly',
-        module: 'readonly',
-        require: 'readonly',
-        exports: 'readonly',
-      },
-    },
-    plugins: {
-      '@typescript-eslint': tsPlugin,
-      'unused-imports': unusedImportsPlugin,
-      'max-params-no-constructor': maxParamsNoConstructorPlugin,
-      '@futdevpro/dynamo': dynamoPlugin,
-    },
-    rules: {
-      // ESLint recommended rules
-      'no-warning-comments': [ 'warn', { terms: [ 'todo', 'fixme', 'removable', '??' ], location: 'anywhere' }],
-      'indent': [ 'warn', 2, { SwitchCase: 1 }],
-      'max-len': [ 'warn', { code: 120, comments: 120 }],
-      'max-lines': [ 'warn', 1000 ],
-      'linebreak-style': 'off',
-      'semi': [ 'warn', 'always' ],
-      'no-unused-vars': 'off',
-      'no-prototype-builtins': 'off',
-      'no-empty': 'warn',
-      'comma-dangle': [ 
-        'warn', 
-        { 
-          arrays: 'always-multiline', 
-          objects: 'always-multiline', 
-          functions: 'only-multiline', 
-          imports: 'never', 
-          exports: 'never', 
-        },
-      ],
-      'brace-style': [ 'warn', '1tbs', { allowSingleLine: false }],
-      'curly': [ 'warn', 'all' ],
-      'object-curly-spacing': [ 'warn', 'always' ],
-      'array-bracket-spacing': [ 'warn', 'always', { objectsInArrays: false, arraysInArrays: false }],
-      'padding-line-between-statements': [
-        'warn',
-        { blankLine: 'always', prev: '*', next: [ 'return','throw','if','for','while','switch','default' ] },
-        { blankLine: 'always', prev: [ 'const','let','var','break' ], next: '*' },
-        { blankLine: 'any', prev: [ 'const','let','var' ], next: [ 'const','let','var' ] },
-        { blankLine: 'any', prev: '*', next: 'import' },
-        { blankLine: 'any', prev: 'import', next: '*' },
-      ],
-      'prefer-const': 'warn',
-      'no-case-declarations': 'off',
-      'no-fallthrough': 'off',
-      'keyword-spacing': 'warn',
-      'no-else-return': 'warn',
-      'no-duplicate-imports': 'warn',
-      // CCAP method-input-count: 3+ paraméter helyett egyetlen object input kötelező
-      'max-params': [ 'warn', 3 ],
-      'max-params-no-constructor/max-params-no-constructor': [ 'warn', 5 ],
-      'quotes': [ 'warn', 'single', { allowTemplateLiterals: true }],
-      // CCAP no-object-shorthand: tilos a `{ foo }` shorthand, mindig `{ foo: foo }`
-      'object-shorthand': [ 'warn', 'never' ],
-      // CCAP no-bracket-property-access: dot-notation előnyben (ahol lehetséges).
-      // A STOCK `dot-notation`-t KIKAPCSOLJUK és a type-aware
-      // `@typescript-eslint/dot-notation`-t hasznaljuk helyette: a stock variant
-      // a `service['protectedMember']` bracket-access-t is `service.protectedMember`-re
-      // javitana (--fix), ami a class-hierarchian KIVULROL (pl. spec fajlbol) elert
-      // protected/private tagnal TS2445 fordito-hibat okoz. A bracket-forma ott a
-      // kanonikus TS escape-hatch. A type-aware variant a checker-bol latja a
-      // modositokat, ezert NEM nyul a protected/private/index-signature elerese-hez.
-      'dot-notation': 'off',
-      '@typescript-eslint/dot-notation': [
-        'warn',
-        {
-          allowProtectedClassPropertyAccess: true,
-          allowPrivateClassPropertyAccess: true,
-          allowIndexSignaturePropertyAccess: true,
-        },
-      ],
-
-      // TypeScript rules
-      '@typescript-eslint/no-unused-vars': 'warn',
-      '@typescript-eslint/explicit-function-return-type': [ 'warn', { allowTypedFunctionExpressions: false }],
-      '@typescript-eslint/no-explicit-any': 'warn',
-      '@typescript-eslint/typedef': [
-        'warn',
-        {
-          parameter: true, // Enforce parameter type annotations
-        },
-      ],
-      '@typescript-eslint/no-unnecessary-type-assertion': 'warn',
-      // CCAP no-array-generic: `Array<T>` helyett `T[]`
-      '@typescript-eslint/array-type': [ 'warn', { default: 'array' }],
-      // CCAP prefer-optional-chaining: `a && a.b` helyett `a?.b`
-      '@typescript-eslint/prefer-optional-chain': 'warn',
-      // CCAP no-dynamic-imports (CommonJS variant): `require()` helyett ES import
-      '@typescript-eslint/no-require-imports': 'warn',
-      
-      // Dynamo custom rules
-      '@futdevpro/dynamo/explicit-types': 'warn',
-      '@futdevpro/dynamo/import-order': 'warn',
-      '@futdevpro/dynamo/naming-patterns': 'warn',
-      '@futdevpro/dynamo/no-import-type': 'warn',
-      '@futdevpro/dynamo/no-js-import': 'warn',
-      '@futdevpro/dynamo/prefer-interface-over-duplicate-types': [ 'warn', { threshold: 3 }],
-      '@futdevpro/dynamo/require-jsdoc-description': [ 'warn', { scope: 'public' }],
-      '@futdevpro/dynamo/prefer-enum-over-string-union': [ 'warn', { minValues: 3 }],
-      '@futdevpro/dynamo/require-test-description-prefix': [ 'warn', { prefix: '| ' }],
-      '@futdevpro/dynamo/no-getter-logic': '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',
-    },
-  },
-];
-
-
+
+/**
+ * Base Rulesets For Dynamo Based Systems
+ * ESLint v9 Flat Config Format
+ */
+const tsEslint = require('typescript-eslint');
+const unusedImportsPlugin = require('eslint-plugin-unused-imports');
+const maxParamsNoConstructorPlugin = require('eslint-plugin-max-params-no-constructor');
+const dynamoPlugin = require('../plugin');
+
+module.exports = [
+  {
+    ignores: [
+      'node_modules/**',
+      'dist/**',
+      'build/**',
+      '*.d.ts',
+    ],
+  },
+  {
+    files: [ '**/*.ts', '**/*.tsx' ],
+    languageOptions: {
+      parser: tsEslint.parser,
+      parserOptions: { 
+        ecmaVersion: 'latest',
+        sourceType: 'module',
+        projectService: true,
+        tsconfigRootDir: process.cwd(),
+      },
+      globals: {
+        // Browser globals
+        window: 'readonly',
+        document: 'readonly',
+        console: 'readonly',
+        process: 'readonly',
+        Buffer: 'readonly',
+        global: 'readonly',
+        __dirname: 'readonly',
+        __filename: 'readonly',
+        module: 'readonly',
+        require: 'readonly',
+        exports: 'readonly',
+      },
+    },
+    plugins: {
+      '@typescript-eslint': tsEslint.plugin,
+      'unused-imports': unusedImportsPlugin,
+      'max-params-no-constructor': maxParamsNoConstructorPlugin,
+      '@futdevpro/dynamo': dynamoPlugin,
+    },
+    rules: {
+      // ESLint recommended rules
+      'no-warning-comments': [ 'warn', { terms: [ 'todo', 'fixme', 'removable', '??' ], location: 'anywhere' }],
+      'indent': [ 'warn', 2, { SwitchCase: 1 }],
+      'max-len': [ 'warn', { code: 120, comments: 120 }],
+      'max-lines': [ 'warn', 1000 ],
+      'linebreak-style': 'off',
+      'semi': [ 'warn', 'always' ],
+      'no-unused-vars': 'off',
+      'no-prototype-builtins': 'off',
+      'no-empty': 'warn',
+      'comma-dangle': [ 
+        'warn', 
+        { 
+          arrays: 'always-multiline', 
+          objects: 'always-multiline', 
+          functions: 'only-multiline', 
+          imports: 'never', 
+          exports: 'never', 
+        },
+      ],
+      'brace-style': [ 'warn', '1tbs', { allowSingleLine: false }],
+      'curly': [ 'warn', 'all' ],
+      'object-curly-spacing': [ 'warn', 'always' ],
+      'array-bracket-spacing': [ 'warn', 'always', { objectsInArrays: false, arraysInArrays: false }],
+      'padding-line-between-statements': [
+        'warn',
+        { blankLine: 'always', prev: '*', next: [ 'return','throw','if','for','while','switch','default' ] },
+        { blankLine: 'always', prev: [ 'const','let','var','break' ], next: '*' },
+        { blankLine: 'any', prev: [ 'const','let','var' ], next: [ 'const','let','var' ] },
+        { blankLine: 'any', prev: '*', next: 'import' },
+        { blankLine: 'any', prev: 'import', next: '*' },
+      ],
+      'prefer-const': 'warn',
+      'no-case-declarations': 'off',
+      'no-fallthrough': 'off',
+      'keyword-spacing': 'warn',
+      'no-else-return': 'warn',
+      'no-duplicate-imports': 'warn',
+      // CCAP method-input-count: 3+ paraméter helyett egyetlen object input kötelező
+      'max-params': [ 'warn', 3 ],
+      'max-params-no-constructor/max-params-no-constructor': [ 'warn', 5 ],
+      'quotes': [ 'warn', 'single', { allowTemplateLiterals: true }],
+      // CCAP no-object-shorthand: tilos a `{ foo }` shorthand, mindig `{ foo: foo }`
+      'object-shorthand': [ 'warn', 'never' ],
+      // CCAP no-bracket-property-access: dot-notation előnyben (ahol lehetséges).
+      // A STOCK `dot-notation`-t KIKAPCSOLJUK és a type-aware
+      // `@typescript-eslint/dot-notation`-t hasznaljuk helyette: a stock variant
+      // a `service['protectedMember']` bracket-access-t is `service.protectedMember`-re
+      // javitana (--fix), ami a class-hierarchian KIVULROL (pl. spec fajlbol) elert
+      // protected/private tagnal TS2445 fordito-hibat okoz. A bracket-forma ott a
+      // kanonikus TS escape-hatch. A type-aware variant a checker-bol latja a
+      // modositokat, ezert NEM nyul a protected/private/index-signature elerese-hez.
+      'dot-notation': 'off',
+      '@typescript-eslint/dot-notation': [
+        'warn',
+        {
+          allowProtectedClassPropertyAccess: true,
+          allowPrivateClassPropertyAccess: true,
+          allowIndexSignaturePropertyAccess: true,
+        },
+      ],
+
+      // TypeScript rules
+      '@typescript-eslint/no-unused-vars': 'warn',
+      '@typescript-eslint/explicit-function-return-type': [ 'warn', { allowTypedFunctionExpressions: false }],
+      '@typescript-eslint/no-explicit-any': 'warn',
+      '@typescript-eslint/typedef': [
+        'warn',
+        {
+          parameter: true, // Enforce parameter type annotations
+        },
+      ],
+      '@typescript-eslint/no-unnecessary-type-assertion': 'warn',
+      // CCAP no-array-generic: `Array<T>` helyett `T[]`
+      '@typescript-eslint/array-type': [ 'warn', { default: 'array' }],
+      // CCAP prefer-optional-chaining: `a && a.b` helyett `a?.b`
+      '@typescript-eslint/prefer-optional-chain': 'warn',
+      // CCAP no-dynamic-imports (CommonJS variant): `require()` helyett ES import
+      '@typescript-eslint/no-require-imports': 'warn',
+      
+      // Dynamo custom rules
+      '@futdevpro/dynamo/explicit-types': 'warn',
+      '@futdevpro/dynamo/import-order': 'warn',
+      '@futdevpro/dynamo/naming-patterns': 'warn',
+      '@futdevpro/dynamo/no-import-type': 'warn',
+      '@futdevpro/dynamo/no-js-import': 'warn',
+      '@futdevpro/dynamo/prefer-interface-over-duplicate-types': [ 'warn', { threshold: 3 }],
+      '@futdevpro/dynamo/require-jsdoc-description': [ 'warn', { scope: 'public' }],
+      '@futdevpro/dynamo/prefer-enum-over-string-union': [ 'warn', { minValues: 3 }],
+      '@futdevpro/dynamo/require-test-description-prefix': [ 'warn', { prefix: '| ' }],
+      '@futdevpro/dynamo/no-getter-logic': '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/configs/fsm.ts

@@ -1,10 +1,10 @@
-const fsmBaseConfig = require('./base');
-
-/**
- * Rulesets For Dynamo Based Systems
- * FSM: Full Stack Module
- * ESLint v9 Flat Config Format
- */
-module.exports = fsmBaseConfig;
-
-
+const fsmBaseConfig = require('./base');
+
+/**
+ * Rulesets For Dynamo Based Systems
+ * FSM: Full Stack Module
+ * ESLint v9 Flat Config Format
+ */
+module.exports = fsmBaseConfig;
+
+

package/src/configs/ngx-package.ts

@@ -1,10 +1,10 @@
-const ngxConfig = require('./ngx');
-
-/**
- * Rulesets For Dynamo Based Systems
- * NGX-PACKAGE: Angular Package
- * ESLint v9 Flat Config Format
- */
-module.exports = ngxConfig;
-
-
+const ngxConfig = require('./ngx');
+
+/**
+ * Rulesets For Dynamo Based Systems
+ * NGX-PACKAGE: Angular Package
+ * ESLint v9 Flat Config Format
+ */
+module.exports = ngxConfig;
+
+