@futdevpro/dynamo-eslint 1.15.28 → 1.20.3

package/src/scripts/validate-naming.ts

@@ -31,10 +31,9 @@ import { DyFM_Log } from '@futdevpro/fsm-dynamo';
 
 import namingPatternsRule from '../plugin/rules/naming-patterns';
 
-// The parser is a consumer-provided peer whose package exports require Node16 resolution, while this package
-// intentionally compiles with legacy Node resolution for fleet compatibility.
-// eslint-disable-next-line @futdevpro/dynamo/no-dynamic-imports, @typescript-eslint/no-require-imports
-const tsParser = require('@typescript-eslint/parser');
+// The parser is provided by the unified typescript-eslint consumer peer.
+// eslint-disable-next-line @futdevpro/dynamo/no-dynamic-imports, @typescript-eslint/no-require-imports
+const tsParser = require('typescript-eslint').parser;
 
 interface LineColumnMessage_Interface {
   line: number;
@@ -47,12 +46,17 @@ interface LineColumnMessage_Interface {
 /**
  * Result of validating naming conventions in a single file
  */
-interface NamingValidationResult {
+interface NamingValidationResult {
   /** Path to the file that was validated */
   file: string;
   /** Array of naming validation errors found in the file */
   errors: LineColumnMessage_Interface[];
-}
+}
+
+/** Identifies the informational diagnostic emitted when isolated validators intentionally ignore inline config. */
+function isIgnoredInlineConfigDiagnostic(message: ESLint.LintResult['messages'][number]): boolean {
+  return !message.ruleId && message.message.includes("has no effect because you have 'noInlineConfig'");
+}
 
 /**
  * Validates naming conventions in a single TypeScript file
@@ -76,8 +80,9 @@ async function validateNamingInFile(filePath: string): Promise<NamingValidationR
     overrideConfigFile: true,
     overrideConfig: [{
       files: [ '**/*.{ts,tsx}' ],
-      linterOptions: {
-        reportUnusedDisableDirectives: 'off',
+      linterOptions: {
+        noInlineConfig: true,
+        reportUnusedDisableDirectives: 'off',
       },
       languageOptions: {
         parser: tsParser,
@@ -106,17 +111,14 @@ async function validateNamingInFile(filePath: string): Promise<NamingValidationR
   // Transform ESLint messages into our result format
   return {
     file: filePath,
-    errors: result.messages.map((msg: {
-      line: number;
-      column: number;
-      message: string;
-      ruleId?: string;
-    }): LineColumnMessage_Interface => ({
-      line: msg.line,
-      column: msg.column,
-      message: msg.message,
-      ruleId: msg.ruleId || 'unknown',
-    })),
+    errors: result.messages
+      .filter((message: ESLint.LintResult['messages'][number]): boolean => !isIgnoredInlineConfigDiagnostic(message))
+      .map((message: ESLint.LintResult['messages'][number]): LineColumnMessage_Interface => ({
+        line: message.line,
+        column: message.column,
+        message: message.message,
+        ruleId: message.ruleId || 'unknown',
+      })),
   };
 }
 
@@ -134,7 +136,14 @@ async function validateNamingInFile(filePath: string): Promise<NamingValidationR
 async function main(): Promise<void> {
   // Find all TypeScript files, excluding common directories and test files
   const files: string[] = await fg([ '**/*.{ts,tsx}' ], { 
-    ignore: [ '**/node_modules/**', '**/build/**', '**/dist/**', '**/*.spec.ts', '**/*.test.ts' ], 
+    ignore: [
+      '**/node_modules/**',
+      '**/build/**',
+      '**/dist/**',
+      '**/samples/**',
+      '**/*.spec.ts',
+      '**/*.test.ts',
+    ],
   });
   
   DyFM_Log.log(`[dynamo-validate-naming] Scanning ${files.length} files...`);

package/src/scripts/validator-cli.integration.spec.ts

@@ -46,14 +46,15 @@ describe('| Dynamo validator CLI flat-config integration', (): void => {
     const result: DyESL_ValidatorResult_Interface = runValidator(
       'validate-imports.js',
       'clean.ts',
-      '/* eslint-disable max-params */\n'
+      '/* eslint-disable @consumer-only/plugin-rule */\n'
         + 'import { join } from \'path\';\n\n'
         + 'export const value: string = join(\'a\', \'b\');\n',
     );
 
-    expect(result.status).toBe(0);
-    expect(result.output).toContain('All import validations passed');
-    expect(result.output).not.toContain('ConfigError');
+    expect(result.status).toBe(0);
+    expect(result.output).toContain('All import validations passed');
+    expect(result.output).not.toContain('ConfigError');
+    expect(result.output).not.toContain('Definition for rule');
   });
 
   it('| fails only for an import-order violation', (): void => {
@@ -74,12 +75,14 @@ describe('| Dynamo validator CLI flat-config integration', (): void => {
     const result: DyESL_ValidatorResult_Interface = runValidator(
       'validate-naming.js',
       'clean.control-service.ts',
-      '/* eslint-disable max-params */\nexport class DyESL_Clean_ControlService { public value: string = "ok"; }\n',
+      '/* eslint-disable @consumer-only/plugin-rule */\n'
+        + 'export class DyESL_Clean_ControlService { public value: string = "ok"; }\n',
     );
 
-    expect(result.status).toBe(0);
-    expect(result.output).toContain('All naming validations passed');
-    expect(result.output).not.toContain('JSDoc');
+    expect(result.status).toBe(0);
+    expect(result.output).toContain('All naming validations passed');
+    expect(result.output).not.toContain('JSDoc');
+    expect(result.output).not.toContain('Definition for rule');
   });
 
   it('| fails for a naming-pattern violation', (): void => {