@futdevpro/dynamo-eslint 1.15.25 → 1.15.28

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);
+  });
+});

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

@@ -1,88 +1,88 @@
-import { Rule } from 'eslint';
-
-/**
- * `@for (...; track <expr>)` kifejezés keresése — capture group 1: a track expression.
- */
-const FOR_TRACK_REGEX: RegExp = /@for\s*\([^)]*;\s*track\s+([^;)]+)/g;
-
-/**
- * Logika detektáló minták a track kifejezésben.
- * Tiltott: `??`, `||`, `&&`, `()`, `+-*\/`, ternary `?`.
- * Megengedett: egyszerű property access (`item.id`).
- */
-const TRACK_LOGIC_PATTERNS: RegExp[] = [
-  /\?\?/,
-  /\|\|/,
-  /&&/,
-  /\(/,
-  /\)/,
-  /[+\-*/%]/,
-  /\?\s*[^:]/,
-];
-
-/**
- * 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 };
-}
-
-/**
- * Angular-for-track-logic szabály: tilos logika `@for (...; track <expr>)` kifejezésben.
- * CCAP "angular-for-track-logic" review ESLint template-text-scan portja.
- */
-const rule: Rule.RuleModule = {
-  meta: {
-    type: 'problem',
-    docs: {
-      description:
-        'Forbid logic in Angular `@for (...; track <expr>)`; '
-        + 'only direct property access (`item.id`) allowed.',
-      recommended: true,
-    },
-    schema: [],
-    messages: {
-      trackLogic: 'Logika a `@for track` kifejezésben: `track {{expr}}`.',
-    },
-  },
-  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(FOR_TRACK_REGEX.source, 'g');
-          let match: RegExpExecArray | null = localRegex.exec(text);
-          while (match !== null) {
-            const expr: string = match[1].trim();
-            const hasLogic: boolean = TRACK_LOGIC_PATTERNS.some(
-              (p: RegExp): boolean => p.test(expr),
-            );
-            if (hasLogic) {
-              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: 'trackLogic',
-                data: { expr: expr },
-              });
-            }
-            match = localRegex.exec(text);
-          }
-        } catch (error) {
-          console.error('[angular-for-track-logic] Error in Program visitor:', error);
-        }
-      },
-    };
-  },
-};
-
-export default rule;
+import { Rule } from 'eslint';
+
+/**
+ * `@for (...; track <expr>)` kifejezés keresése — capture group 1: a track expression.
+ */
+const FOR_TRACK_REGEX: RegExp = /@for\s*\([^)]*;\s*track\s+([^;)]+)/g;
+
+/**
+ * Logika detektáló minták a track kifejezésben.
+ * Tiltott: `??`, `||`, `&&`, `()`, `+-*\/`, ternary `?`.
+ * Megengedett: egyszerű property access (`item.id`).
+ */
+const TRACK_LOGIC_PATTERNS: RegExp[] = [
+  /\?\?/,
+  /\|\|/,
+  /&&/,
+  /\(/,
+  /\)/,
+  /[+\-*/%]/,
+  /\?\s*[^:]/,
+];
+
+/**
+ * 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 };
+}
+
+/**
+ * Angular-for-track-logic szabály: tilos logika `@for (...; track <expr>)` kifejezésben.
+ * CCAP "angular-for-track-logic" review ESLint template-text-scan portja.
+ */
+const rule: Rule.RuleModule = {
+  meta: {
+    type: 'problem',
+    docs: {
+      description:
+        'Forbid logic in Angular `@for (...; track <expr>)`; '
+        + 'only direct property access (`item.id`) allowed.',
+      recommended: true,
+    },
+    schema: [],
+    messages: {
+      trackLogic: 'Logika a `@for track` kifejezésben: `track {{expr}}`.',
+    },
+  },
+  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(FOR_TRACK_REGEX.source, 'g');
+          let match: RegExpExecArray | null = localRegex.exec(text);
+          while (match !== null) {
+            const expr: string = match[1].trim();
+            const hasLogic: boolean = TRACK_LOGIC_PATTERNS.some(
+              (p: RegExp): boolean => p.test(expr),
+            );
+            if (hasLogic) {
+              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: 'trackLogic',
+                data: { expr: expr },
+              });
+            }
+            match = localRegex.exec(text);
+          }
+        } catch (error) {
+          console.error('[angular-for-track-logic] Error in Program visitor:', error);
+        }
+      },
+    };
+  },
+};
+
+export default rule;