@aiready/consistency 0.21.4 → 0.21.6

package/dist/cli.js

@@ -820,6 +820,7 @@ async function analyzeNamingGeneralized(files) {
           if (!spec || spec === "*" || spec === "default") continue;
           if (exceptions.has(spec)) continue;
           if (COMMON_ABBREVIATIONS.has(spec.toLowerCase())) continue;
+          if (spec.includes(".")) continue;
           if (!conventions.variablePattern.test(spec) && !conventions.classPattern.test(spec) && (!conventions.constantPattern || !conventions.constantPattern.test(spec)) && (!conventions.typePattern || !conventions.typePattern.test(spec)) && (!conventions.interfacePattern || !conventions.interfacePattern.test(spec)) && !/^[A-Z][A-Z0-9_]*$/.test(spec)) {
             issues.push({
               type: "naming-inconsistency",
@@ -1030,7 +1031,8 @@ async function analyzeConsistency(options) {
   }
   for (const issue of patternIssues) {
     if (!shouldIncludeSeverity(issue.severity, minSeverity)) continue;
-    const fileName = issue.fileName || issue.file || issue.filePath || (Array.isArray(issue.files) ? issue.files[0] : "unknown");
+    const anyIssue = issue;
+    const fileName = anyIssue.fileName || anyIssue.file || anyIssue.filePath || (Array.isArray(issue.files) ? issue.files[0] : "unknown");
     if (!fileIssuesMap.has(fileName)) fileIssuesMap.set(fileName, []);
     fileIssuesMap.get(fileName).push(issue);
   }

package/dist/cli.mjs

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 import {
   analyzeConsistency
-} from "./chunk-ZG3KFSD3.mjs";
+} from "./chunk-7DKVZNY7.mjs";
 
 // src/cli.ts
 import { Command } from "commander";

package/dist/index.js

@@ -839,6 +839,7 @@ async function analyzeNamingGeneralized(files) {
           if (!spec || spec === "*" || spec === "default") continue;
           if (exceptions.has(spec)) continue;
           if (COMMON_ABBREVIATIONS.has(spec.toLowerCase())) continue;
+          if (spec.includes(".")) continue;
           if (!conventions.variablePattern.test(spec) && !conventions.classPattern.test(spec) && (!conventions.constantPattern || !conventions.constantPattern.test(spec)) && (!conventions.typePattern || !conventions.typePattern.test(spec)) && (!conventions.interfacePattern || !conventions.interfacePattern.test(spec)) && !/^[A-Z][A-Z0-9_]*$/.test(spec)) {
             issues.push({
               type: "naming-inconsistency",
@@ -1049,7 +1050,8 @@ async function analyzeConsistency(options) {
   }
   for (const issue of patternIssues) {
     if (!shouldIncludeSeverity(issue.severity, minSeverity)) continue;
-    const fileName = issue.fileName || issue.file || issue.filePath || (Array.isArray(issue.files) ? issue.files[0] : "unknown");
+    const anyIssue = issue;
+    const fileName = anyIssue.fileName || anyIssue.file || anyIssue.filePath || (Array.isArray(issue.files) ? issue.files[0] : "unknown");
     if (!fileIssuesMap.has(fileName)) fileIssuesMap.set(fileName, []);
     fileIssuesMap.get(fileName).push(issue);
   }
@@ -1187,7 +1189,7 @@ var ConsistencyProvider = (0, import_core7.createProvider)({
     return calculateConsistencyScore(
       allIssues,
       totalFiles,
-      options.costConfig
+      options?.costConfig
     );
   }
 });

package/dist/index.mjs

@@ -6,16 +6,13 @@ import {
   analyzePatterns,
   calculateConsistencyScore,
   detectNamingConventions
-} from "./chunk-ZG3KFSD3.mjs";
+} from "./chunk-7DKVZNY7.mjs";
 
 // src/index.ts
 import { ToolRegistry } from "@aiready/core";
 
 // src/provider.ts
-import {
-  createProvider,
-  ToolName
-} from "@aiready/core";
+import { createProvider, ToolName } from "@aiready/core";
 var ConsistencyProvider = createProvider({
   id: ToolName.NamingConsistency,
   alias: ["consistency", "naming", "standards"],
@@ -39,7 +36,7 @@ var ConsistencyProvider = createProvider({
     return calculateConsistencyScore(
       allIssues,
       totalFiles,
-      options.costConfig
+      options?.costConfig
     );
   }
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aiready/consistency",
-  "version": "0.21.4",
+  "version": "0.21.6",
   "description": "Detects consistency issues in naming, patterns, and architecture that confuse AI models",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",
@@ -43,7 +43,7 @@
     "@typescript-eslint/typescript-estree": "^8.53.0",
     "chalk": "^5.3.0",
     "commander": "^14.0.0",
-    "@aiready/core": "0.24.4"
+    "@aiready/core": "0.24.6"
   },
   "devDependencies": {
     "@types/node": "^24.0.0",

package/src/__tests__/naming-dotted.test.ts

@@ -0,0 +1,64 @@
+import { describe, it, expect, beforeAll, afterAll } from 'vitest';
+import { analyzeNamingGeneralized } from '../analyzers/naming-generalized';
+import { join } from 'path';
+import { writeFileSync, mkdirSync, rmSync } from 'fs';
+import { tmpdir } from 'os';
+
+describe('Naming Analysis - Dotted Identifiers', () => {
+  const tmpDir = join(tmpdir(), 'aiready-consistency-dotted-tests');
+
+  beforeAll(() => {
+    mkdirSync(tmpDir, { recursive: true });
+  });
+
+  afterAll(() => {
+    rmSync(tmpDir, { recursive: true, force: true });
+  });
+
+  function createTestFile(name: string, content: string): string {
+    const filePath = join(tmpDir, name);
+    writeFileSync(filePath, content, 'utf8');
+    return filePath;
+  }
+
+  it('should NOT flag dotted identifiers in Python imports', async () => {
+    const file = createTestFile(
+      'library_test.py',
+      `
+import urllib.parse
+import pytrends.request
+
+def analyze():
+    pass
+`
+    );
+
+    const issues = await analyzeNamingGeneralized([file]);
+
+    // urllib.parse and pytrends.request should be ignored because they contain dots
+    const namingIssues = issues.filter(
+      (i) => i.type === 'naming-inconsistency'
+    );
+    expect(namingIssues.length).toBe(0);
+  });
+
+  it('should still flag inconsistent top-level names without dots', async () => {
+    const file = createTestFile(
+      'bad_naming.py',
+      `
+def BadNamingConvention():
+    pass
+
+user_Name = "John"
+`
+    );
+
+    const issues = await analyzeNamingGeneralized([file]);
+    const namingIssues = issues.filter(
+      (i) => i.type === 'naming-inconsistency'
+    );
+
+    // BadNamingConvention (PascalCase for function) and user_Name (MixedCase) should be flagged
+    expect(namingIssues.length).toBeGreaterThanOrEqual(1);
+  });
+});

package/src/analyzer.ts

@@ -41,7 +41,22 @@ export async function analyzeConsistency(
   const filePaths = await scanFiles(scanOptions);
 
   // Collect issues by category
-  let namingIssues: any[] = [];
+  let namingIssues: Array<{
+    fileName?: string;
+    file?: string;
+    filePath?: string;
+    line?: number;
+    column?: number;
+    identifier?: string;
+    type?: string;
+    severity: Severity;
+    message?: string;
+    suggestion?: string;
+    description?: string;
+    files?: string[];
+    examples?: string[];
+    location?: { file: string; line: number; column: number };
+  }> = [];
   if (checkNaming) {
     // 1. Generalized naming analysis for all supported files
     namingIssues = await analyzeNamingGeneralized(filePaths);
@@ -66,10 +81,7 @@ export async function analyzeConsistency(
     if (!shouldIncludeSeverity(issue.severity, minSeverity)) continue;
 
     const fileName =
-      (issue as any).fileName ||
-      (issue as any).file ||
-      (issue as any).filePath ||
-      'unknown';
+      issue.fileName || issue.file || issue.filePath || 'unknown';
     if (!fileIssuesMap.has(fileName)) fileIssuesMap.set(fileName, []);
     fileIssuesMap.get(fileName)!.push(issue as unknown as ConsistencyIssue);
   }
@@ -78,13 +90,12 @@ export async function analyzeConsistency(
   for (const issue of patternIssues) {
     if (!shouldIncludeSeverity(issue.severity, minSeverity)) continue;
 
+    const anyIssue = issue as any;
     const fileName =
-      (issue as any).fileName ||
-      (issue as any).file ||
-      (issue as any).filePath ||
-      (Array.isArray((issue as any).files)
-        ? (issue as any).files[0]
-        : 'unknown');
+      (anyIssue.fileName as string) ||
+      (anyIssue.file as string) ||
+      (anyIssue.filePath as string) ||
+      (Array.isArray(issue.files) ? (issue.files[0] as string) : 'unknown');
     if (!fileIssuesMap.has(fileName)) fileIssuesMap.set(fileName, []);
     fileIssuesMap.get(fileName)!.push(issue as unknown as ConsistencyIssue);
   }

package/src/analyzers/naming-generalized.ts

@@ -147,6 +147,7 @@ export async function analyzeNamingGeneralized(
           if (!spec || spec === '*' || spec === 'default') continue;
           if (exceptions.has(spec)) continue;
           if (COMMON_ABBREVIATIONS.has(spec.toLowerCase())) continue;
+          if (spec.includes('.')) continue; // Library imports like urllib.parse (Issue: Library Naming Inconsistency)
 
           if (
             !conventions.variablePattern.test(spec) &&

package/src/cli.ts

@@ -91,7 +91,7 @@ EXAMPLES:
       checkNaming: options.naming !== false && mergedConfig.checkNaming,
       checkPatterns: options.patterns !== false && mergedConfig.checkPatterns,
       checkArchitecture: options.architecture || mergedConfig.checkArchitecture,
-      minSeverity: (options.minSeverity as any) || mergedConfig.minSeverity,
+      minSeverity: (options.minSeverity as string) || mergedConfig.minSeverity,
       include: options.include?.split(',') || mergedConfig.include,
       exclude: options.exclude?.split(',') || mergedConfig.exclude,
     };

package/src/provider.ts

@@ -1,9 +1,4 @@
-import {
-  AnalysisResult,
-  createProvider,
-  ToolName,
-  ScanOptions,
-} from '@aiready/core';
+import { createProvider, ToolName, ScanOptions } from '@aiready/core';
 import { analyzeConsistency } from './analyzer';
 import { calculateConsistencyScore } from './scoring';
 import { ConsistencyOptions, ConsistencyIssue } from './types';
@@ -20,22 +15,22 @@ export const ConsistencyProvider = createProvider({
     return analyzeConsistency(options as ConsistencyOptions);
   },
   getResults(report) {
-    return report.results as AnalysisResult[];
+    return report.results;
   },
   getSummary(report) {
     return report.summary;
   },
   score(output, options) {
-    const results = (output.results || []) as AnalysisResult[];
+    const results = output.results || [];
     const allIssues = results.flatMap(
-      (r) => (r.issues || []) as ConsistencyIssue[]
-    );
-    const totalFiles = (output.summary as any)?.filesAnalyzed || results.length;
+      (r) => r.issues || []
+    ) as ConsistencyIssue[];
+    const totalFiles = output.summary?.filesAnalyzed || results.length;
 
     return calculateConsistencyScore(
       allIssues,
       totalFiles,
-      (options as any).costConfig
+      options?.costConfig
     );
   },
 });

package/src/utils/ast-parser.ts

@@ -50,7 +50,7 @@ export function traverseAST(
 
   // Visit children
   for (const key of Object.keys(node)) {
-    const value = (node as any)[key];
+    const value = (node as unknown as Record<string, unknown>)[key];
 
     if (Array.isArray(value)) {
       for (const child of value) {