@aiready/consistency 0.6.16 → 0.7.0

package/dist/cli.js

@@ -27,7 +27,7 @@ var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__ge
 var import_commander = require("commander");
 
 // src/analyzer.ts
-var import_core3 = require("@aiready/core");
+var import_core4 = require("@aiready/core");
 
 // src/utils/ast-parser.ts
 var import_typescript_estree = require("@typescript-eslint/typescript-estree");
@@ -1083,8 +1083,120 @@ function extractIdentifiersFromPattern(pattern, scopeTracker, isParameter, ances
   }
 }
 
-// src/analyzers/patterns.ts
+// src/analyzers/naming-python.ts
 var import_core2 = require("@aiready/core");
+async function analyzePythonNaming(files) {
+  const issues = [];
+  const parser = (0, import_core2.getParser)("dummy.py");
+  if (!parser) {
+    console.warn("Python parser not available");
+    return issues;
+  }
+  const pythonFiles = files.filter((f) => f.toLowerCase().endsWith(".py"));
+  for (const file of pythonFiles) {
+    try {
+      const fs = await import("fs");
+      const code = await fs.promises.readFile(file, "utf-8");
+      const result = parser.parse(code, file);
+      for (const exp of result.exports) {
+        const nameIssue = checkPythonNaming(exp.name, exp.type, file, exp.loc?.start.line || 0);
+        if (nameIssue) {
+          issues.push(nameIssue);
+        }
+      }
+      for (const imp of result.imports) {
+        for (const spec of imp.specifiers) {
+          if (spec !== "*" && spec !== "default") {
+            const nameIssue = checkPythonNaming(spec, "variable", file, imp.loc?.start.line || 0);
+            if (nameIssue) {
+              issues.push(nameIssue);
+            }
+          }
+        }
+      }
+    } catch (error) {
+      console.warn(`Skipping ${file} due to error:`, error);
+    }
+  }
+  return issues;
+}
+function checkPythonNaming(identifier, type, file, line) {
+  const parser = (0, import_core2.getParser)("dummy.py");
+  const conventions = parser?.getNamingConventions();
+  if (!conventions) return null;
+  if (conventions.exceptions?.includes(identifier)) {
+    return null;
+  }
+  if (type === "class") {
+    if (!conventions.classPattern.test(identifier)) {
+      return {
+        type: "poor-naming",
+        identifier,
+        file,
+        line,
+        column: 0,
+        severity: "major",
+        category: "naming",
+        suggestion: `Class names should use PascalCase (e.g., ${toPascalCase(identifier)})`
+      };
+    }
+  } else if (type === "function") {
+    if (!conventions.functionPattern.test(identifier)) {
+      if (/^[a-z][a-zA-Z0-9]*$/.test(identifier) && /[A-Z]/.test(identifier)) {
+        return {
+          type: "convention-mix",
+          identifier,
+          file,
+          line,
+          column: 0,
+          severity: "major",
+          category: "naming",
+          suggestion: `Function names should use snake_case, not camelCase (e.g., ${toSnakeCase(identifier)})`
+        };
+      }
+    }
+  } else if (type === "const" || type === "variable") {
+    if (identifier === identifier.toUpperCase() && identifier.length > 1) {
+      if (!conventions.constantPattern.test(identifier)) {
+        return {
+          type: "poor-naming",
+          identifier,
+          file,
+          line,
+          column: 0,
+          severity: "minor",
+          category: "naming",
+          suggestion: "Constants should use UPPER_CASE_WITH_UNDERSCORES"
+        };
+      }
+    } else {
+      if (!conventions.variablePattern.test(identifier)) {
+        if (/^[a-z][a-zA-Z0-9]*$/.test(identifier) && /[A-Z]/.test(identifier)) {
+          return {
+            type: "convention-mix",
+            identifier,
+            file,
+            line,
+            column: 0,
+            severity: "major",
+            category: "naming",
+            suggestion: `Variable names should use snake_case, not camelCase (e.g., ${toSnakeCase(identifier)})`
+          };
+        }
+      }
+    }
+  }
+  return null;
+}
+function toSnakeCase(str) {
+  return str.replace(/([A-Z])/g, "_$1").toLowerCase().replace(/^_/, "");
+}
+function toPascalCase(str) {
+  return str.split("_").map((word) => word.charAt(0).toUpperCase() + word.slice(1).toLowerCase()).join("");
+}
+
+// src/analyzers/patterns.ts
+var import_core3 = require("@aiready/core");
 async function analyzePatterns(files) {
   const issues = [];
   const errorHandlingIssues = await analyzeErrorHandling(files);
@@ -1103,7 +1215,7 @@ async function analyzeErrorHandling(files) {
     returnsError: []
   };
   for (const file of files) {
-    const content = await (0, import_core2.readFileContent)(file);
+    const content = await (0, import_core3.readFileContent)(file);
     if (content.includes("try {") || content.includes("} catch")) {
       patterns.tryCatch.push(file);
     }
@@ -1147,7 +1259,7 @@ async function analyzeAsyncPatterns(files) {
     callbacks: []
   };
   for (const file of files) {
-    const content = await (0, import_core2.readFileContent)(file);
+    const content = await (0, import_core3.readFileContent)(file);
     if (content.match(/async\s+(function|\(|[a-zA-Z])/)) {
       patterns.asyncAwait.push(file);
     }
@@ -1189,7 +1301,7 @@ async function analyzeImportStyles(files) {
     mixed: []
   };
   for (const file of files) {
-    const content = await (0, import_core2.readFileContent)(file);
+    const content = await (0, import_core3.readFileContent)(file);
     const hasESM = content.match(/^import\s+/m);
     const hasCJS = hasActualRequireCalls(content);
     if (hasESM && hasCJS) {
@@ -1247,8 +1359,15 @@ async function analyzeConsistency(options) {
     minSeverity = "info",
     ...scanOptions
   } = options;
-  const filePaths = await (0, import_core3.scanFiles)(scanOptions);
-  const namingIssues = checkNaming ? await analyzeNamingAST(filePaths) : [];
+  const filePaths = await (0, import_core4.scanFiles)(scanOptions);
+  const tsJsFiles = filePaths.filter((f) => /\.(ts|tsx|js|jsx)$/i.test(f));
+  const pythonFiles = filePaths.filter((f) => /\.py$/i.test(f));
+  let namingIssues = [];
+  if (checkNaming) {
+    const tsJsNamingIssues = tsJsFiles.length > 0 ? await analyzeNamingAST(tsJsFiles) : [];
+    const pythonNamingIssues = pythonFiles.length > 0 ? await analyzePythonNaming(pythonFiles) : [];
+    namingIssues = [...tsJsNamingIssues, ...pythonNamingIssues];
+  }
   const patternIssues = checkPatterns ? await analyzePatterns(filePaths) : [];
   const results = [];
   const fileIssuesMap = /* @__PURE__ */ new Map();
@@ -1389,7 +1508,7 @@ function generateRecommendations(namingIssues, patternIssues) {
 var import_chalk = __toESM(require("chalk"));
 var import_fs2 = require("fs");
 var import_path2 = require("path");
-var import_core4 = require("@aiready/core");
+var import_core5 = require("@aiready/core");
 var program = new import_commander.Command();
 program.name("aiready-consistency").description("Detect consistency patterns in naming, code structure, and architecture").version("0.1.0").addHelpText("after", `
 LANGUAGE SUPPORT:
@@ -1413,7 +1532,7 @@ EXAMPLES:
 `).argument("<directory>", "Directory to analyze").option("--naming", "Check naming conventions and quality (default: true)").option("--no-naming", "Skip naming analysis").option("--patterns", "Check code pattern consistency (default: true)").option("--no-patterns", "Skip pattern analysis").option("--architecture", "Check architectural consistency (not yet implemented)").option("--min-severity <level>", "Minimum severity: info|minor|major|critical. Default: info").option("--include <patterns>", "File patterns to include (comma-separated)").option("--exclude <patterns>", "File patterns to exclude (comma-separated)").option("-o, --output <format>", "Output format: console|json|markdown", "console").option("--output-file <path>", "Output file path (for json/markdown)").action(async (directory, options) => {
   console.log(import_chalk.default.blue("\u{1F50D} Analyzing consistency...\n"));
   const startTime = Date.now();
-  const config = await (0, import_core4.loadConfig)(directory);
+  const config = await (0, import_core5.loadConfig)(directory);
   const defaults = {
     checkNaming: true,
     checkPatterns: true,
@@ -1422,7 +1541,7 @@ EXAMPLES:
     include: void 0,
     exclude: void 0
   };
-  const mergedConfig = (0, import_core4.mergeConfigWithDefaults)(config, defaults);
+  const mergedConfig = (0, import_core5.mergeConfigWithDefaults)(config, defaults);
   const finalOptions = {
     rootDir: directory,
     checkNaming: options.naming !== false && mergedConfig.checkNaming,
@@ -1436,7 +1555,7 @@ EXAMPLES:
   const elapsedTime = ((Date.now() - startTime) / 1e3).toFixed(2);
   if (options.output === "json") {
     const output = JSON.stringify(report, null, 2);
-    const outputPath = (0, import_core4.resolveOutputPath)(
+    const outputPath = (0, import_core5.resolveOutputPath)(
       options.outputFile,
       `consistency-report-${(/* @__PURE__ */ new Date()).toISOString().split("T")[0]}.json`,
       directory
@@ -1449,7 +1568,7 @@ EXAMPLES:
     console.log(import_chalk.default.green(`\u2713 Report saved to ${outputPath}`));
   } else if (options.output === "markdown") {
     const markdown = generateMarkdownReport(report, elapsedTime);
-    const outputPath = (0, import_core4.resolveOutputPath)(
+    const outputPath = (0, import_core5.resolveOutputPath)(
       options.outputFile,
       `consistency-report-${(/* @__PURE__ */ new Date()).toISOString().split("T")[0]}.md`,
       directory

package/dist/cli.mjs

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

package/dist/index.d.mts

@@ -54,8 +54,7 @@ interface ConsistencyReport {
 
 /**
  * Main consistency analyzer that orchestrates all analysis types
- * Note: Currently only supports TypeScript/JavaScript files (.ts, .tsx, .js, .jsx)
- * Python and other language files will be ignored during naming analysis.
+ * Supports: TypeScript, JavaScript, Python
  */
 declare function analyzeConsistency(options: ConsistencyOptions): Promise<ConsistencyReport>;
 

package/dist/index.d.ts

@@ -54,8 +54,7 @@ interface ConsistencyReport {
 
 /**
  * Main consistency analyzer that orchestrates all analysis types
- * Note: Currently only supports TypeScript/JavaScript files (.ts, .tsx, .js, .jsx)
- * Python and other language files will be ignored during naming analysis.
+ * Supports: TypeScript, JavaScript, Python
  */
 declare function analyzeConsistency(options: ConsistencyOptions): Promise<ConsistencyReport>;
 

package/dist/index.js

@@ -1,7 +1,9 @@
 "use strict";
+var __create = Object.create;
 var __defProp = Object.defineProperty;
 var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
 var __getOwnPropNames = Object.getOwnPropertyNames;
+var __getProtoOf = Object.getPrototypeOf;
 var __hasOwnProp = Object.prototype.hasOwnProperty;
 var __export = (target, all) => {
   for (var name in all)
@@ -15,6 +17,14 @@ var __copyProps = (to, from, except, desc) => {
   }
   return to;
 };
+var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
+  // If the importer is in node compatibility mode or this is not an ESM
+  // file that has been converted to a CommonJS file using a Babel-
+  // compatible transform (i.e. "__esModule" has not been set), then set
+  // "default" to the CommonJS "module.exports" for node compatibility.
+  isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
+  mod
+));
 var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
 
 // src/index.ts
@@ -30,7 +40,7 @@ __export(index_exports, {
 module.exports = __toCommonJS(index_exports);
 
 // src/analyzer.ts
-var import_core3 = require("@aiready/core");
+var import_core4 = require("@aiready/core");
 
 // src/utils/ast-parser.ts
 var import_typescript_estree = require("@typescript-eslint/typescript-estree");
@@ -1097,8 +1107,120 @@ function extractIdentifiersFromPattern(pattern, scopeTracker, isParameter, ances
   }
 }
 
-// src/analyzers/patterns.ts
+// src/analyzers/naming-python.ts
 var import_core2 = require("@aiready/core");
+async function analyzePythonNaming(files) {
+  const issues = [];
+  const parser = (0, import_core2.getParser)("dummy.py");
+  if (!parser) {
+    console.warn("Python parser not available");
+    return issues;
+  }
+  const pythonFiles = files.filter((f) => f.toLowerCase().endsWith(".py"));
+  for (const file of pythonFiles) {
+    try {
+      const fs = await import("fs");
+      const code = await fs.promises.readFile(file, "utf-8");
+      const result = parser.parse(code, file);
+      for (const exp of result.exports) {
+        const nameIssue = checkPythonNaming(exp.name, exp.type, file, exp.loc?.start.line || 0);
+        if (nameIssue) {
+          issues.push(nameIssue);
+        }
+      }
+      for (const imp of result.imports) {
+        for (const spec of imp.specifiers) {
+          if (spec !== "*" && spec !== "default") {
+            const nameIssue = checkPythonNaming(spec, "variable", file, imp.loc?.start.line || 0);
+            if (nameIssue) {
+              issues.push(nameIssue);
+            }
+          }
+        }
+      }
+    } catch (error) {
+      console.warn(`Skipping ${file} due to error:`, error);
+    }
+  }
+  return issues;
+}
+function checkPythonNaming(identifier, type, file, line) {
+  const parser = (0, import_core2.getParser)("dummy.py");
+  const conventions = parser?.getNamingConventions();
+  if (!conventions) return null;
+  if (conventions.exceptions?.includes(identifier)) {
+    return null;
+  }
+  if (type === "class") {
+    if (!conventions.classPattern.test(identifier)) {
+      return {
+        type: "poor-naming",
+        identifier,
+        file,
+        line,
+        column: 0,
+        severity: "major",
+        category: "naming",
+        suggestion: `Class names should use PascalCase (e.g., ${toPascalCase(identifier)})`
+      };
+    }
+  } else if (type === "function") {
+    if (!conventions.functionPattern.test(identifier)) {
+      if (/^[a-z][a-zA-Z0-9]*$/.test(identifier) && /[A-Z]/.test(identifier)) {
+        return {
+          type: "convention-mix",
+          identifier,
+          file,
+          line,
+          column: 0,
+          severity: "major",
+          category: "naming",
+          suggestion: `Function names should use snake_case, not camelCase (e.g., ${toSnakeCase(identifier)})`
+        };
+      }
+    }
+  } else if (type === "const" || type === "variable") {
+    if (identifier === identifier.toUpperCase() && identifier.length > 1) {
+      if (!conventions.constantPattern.test(identifier)) {
+        return {
+          type: "poor-naming",
+          identifier,
+          file,
+          line,
+          column: 0,
+          severity: "minor",
+          category: "naming",
+          suggestion: "Constants should use UPPER_CASE_WITH_UNDERSCORES"
+        };
+      }
+    } else {
+      if (!conventions.variablePattern.test(identifier)) {
+        if (/^[a-z][a-zA-Z0-9]*$/.test(identifier) && /[A-Z]/.test(identifier)) {
+          return {
+            type: "convention-mix",
+            identifier,
+            file,
+            line,
+            column: 0,
+            severity: "major",
+            category: "naming",
+            suggestion: `Variable names should use snake_case, not camelCase (e.g., ${toSnakeCase(identifier)})`
+          };
+        }
+      }
+    }
+  }
+  return null;
+}
+function toSnakeCase(str) {
+  return str.replace(/([A-Z])/g, "_$1").toLowerCase().replace(/^_/, "");
+}
+function toPascalCase(str) {
+  return str.split("_").map((word) => word.charAt(0).toUpperCase() + word.slice(1).toLowerCase()).join("");
+}
+
+// src/analyzers/patterns.ts
+var import_core3 = require("@aiready/core");
 async function analyzePatterns(files) {
   const issues = [];
   const errorHandlingIssues = await analyzeErrorHandling(files);
@@ -1117,7 +1239,7 @@ async function analyzeErrorHandling(files) {
     returnsError: []
   };
   for (const file of files) {
-    const content = await (0, import_core2.readFileContent)(file);
+    const content = await (0, import_core3.readFileContent)(file);
     if (content.includes("try {") || content.includes("} catch")) {
       patterns.tryCatch.push(file);
     }
@@ -1161,7 +1283,7 @@ async function analyzeAsyncPatterns(files) {
     callbacks: []
   };
   for (const file of files) {
-    const content = await (0, import_core2.readFileContent)(file);
+    const content = await (0, import_core3.readFileContent)(file);
     if (content.match(/async\s+(function|\(|[a-zA-Z])/)) {
       patterns.asyncAwait.push(file);
     }
@@ -1203,7 +1325,7 @@ async function analyzeImportStyles(files) {
     mixed: []
   };
   for (const file of files) {
-    const content = await (0, import_core2.readFileContent)(file);
+    const content = await (0, import_core3.readFileContent)(file);
     const hasESM = content.match(/^import\s+/m);
     const hasCJS = hasActualRequireCalls(content);
     if (hasESM && hasCJS) {
@@ -1261,8 +1383,15 @@ async function analyzeConsistency(options) {
     minSeverity = "info",
     ...scanOptions
   } = options;
-  const filePaths = await (0, import_core3.scanFiles)(scanOptions);
-  const namingIssues = checkNaming ? await analyzeNamingAST(filePaths) : [];
+  const filePaths = await (0, import_core4.scanFiles)(scanOptions);
+  const tsJsFiles = filePaths.filter((f) => /\.(ts|tsx|js|jsx)$/i.test(f));
+  const pythonFiles = filePaths.filter((f) => /\.py$/i.test(f));
+  let namingIssues = [];
+  if (checkNaming) {
+    const tsJsNamingIssues = tsJsFiles.length > 0 ? await analyzeNamingAST(tsJsFiles) : [];
+    const pythonNamingIssues = pythonFiles.length > 0 ? await analyzePythonNaming(pythonFiles) : [];
+    namingIssues = [...tsJsNamingIssues, ...pythonNamingIssues];
+  }
   const patternIssues = checkPatterns ? await analyzePatterns(filePaths) : [];
   const results = [];
   const fileIssuesMap = /* @__PURE__ */ new Map();
@@ -1400,12 +1529,12 @@ function generateRecommendations(namingIssues, patternIssues) {
 }
 
 // src/analyzers/naming.ts
-var import_core4 = require("@aiready/core");
+var import_core5 = require("@aiready/core");
 async function analyzeNaming(files) {
   const issues = [];
   const { customAbbreviations, customShortWords, disabledChecks } = await loadNamingConfig(files);
   for (const file of files) {
-    const content = await (0, import_core4.readFileContent)(file);
+    const content = await (0, import_core5.readFileContent)(file);
     const fileIssues = analyzeFileNaming(file, content, customAbbreviations, customShortWords, disabledChecks);
     issues.push(...fileIssues);
   }

package/dist/index.mjs

@@ -7,7 +7,7 @@ import {
   detectNamingConventions,
   loadNamingConfig,
   snakeCaseToCamelCase
-} from "./chunk-MM2PLUCH.mjs";
+} from "./chunk-YEHXYHGY.mjs";
 
 // src/analyzers/naming.ts
 import { readFileContent } from "@aiready/core";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aiready/consistency",
-  "version": "0.6.16",
+  "version": "0.7.0",
   "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.7.13"
+    "@aiready/core": "0.8.0"
   },
   "devDependencies": {
     "@types/node": "^24.0.0",

package/src/analyzer.ts

@@ -2,12 +2,12 @@ import { scanFiles } from '@aiready/core';
 import type { AnalysisResult, Issue } from '@aiready/core';
 import type { ConsistencyOptions, ConsistencyReport, ConsistencyIssue } from './types';
 import { analyzeNamingAST } from './analyzers/naming-ast';
+import { analyzePythonNaming } from './analyzers/naming-python';
 import { analyzePatterns } from './analyzers/patterns';
 
 /**
  * Main consistency analyzer that orchestrates all analysis types
- * Note: Currently only supports TypeScript/JavaScript files (.ts, .tsx, .js, .jsx)
- * Python and other language files will be ignored during naming analysis.
+ * Supports: TypeScript, JavaScript, Python
  */
 export async function analyzeConsistency(
   options: ConsistencyOptions
@@ -23,8 +23,18 @@ export async function analyzeConsistency(
   // Scan files
   const filePaths = await scanFiles(scanOptions);
 
-  // Collect issues by category
-  const namingIssues = checkNaming ? await analyzeNamingAST(filePaths) : [];
+  // Separate files by language
+  const tsJsFiles = filePaths.filter(f => /\.(ts|tsx|js|jsx)$/i.test(f));
+  const pythonFiles = filePaths.filter(f => /\.py$/i.test(f));
+
+  // Collect issues by category - now handles multiple languages
+  let namingIssues: any[] = [];
+  if (checkNaming) {
+    const tsJsNamingIssues = tsJsFiles.length > 0 ? await analyzeNamingAST(tsJsFiles) : [];
+    const pythonNamingIssues = pythonFiles.length > 0 ? await analyzePythonNaming(pythonFiles) : [];
+    namingIssues = [...tsJsNamingIssues, ...pythonNamingIssues];
+  }
+  
   const patternIssues = checkPatterns ? await analyzePatterns(filePaths) : [];
 
   // Convert to AnalysisResult format