@aiready/context-analyzer 0.22.14 → 0.22.16

package/src/classify/file-classifiers.ts

@@ -1,4 +1,5 @@
 import { DependencyNode } from '../types';
+import type { ExportInfo } from '@aiready/core';
 import {
   BARREL_EXPORT_MIN_EXPORTS,
   BARREL_EXPORT_TOKEN_LIMIT,
@@ -23,7 +24,7 @@ export function isBoilerplateBarrel(node: DependencyNode): boolean {
   if (!exports || exports.length === 0) return false;
 
   // 1. Must be purely re-exports
-  const isPurelyReexports = exports.every((exp: any) => !!exp.source);
+  const isPurelyReexports = exports.every((exp: ExportInfo) => !!exp.source);
   if (!isPurelyReexports) return false;
 
   // 2. Must be low local token cost (no actual logic)
@@ -31,7 +32,7 @@ export function isBoilerplateBarrel(node: DependencyNode): boolean {
 
   // 3. Detect "Architectural Theater"
   // If it re-exports everything from exactly ONE source, it's a pass-through
-  const sources = new Set(exports.map((exp: any) => exp.source));
+  const sources = new Set(exports.map((exp: ExportInfo) => exp.source));
 
   // Pattern: export * from '../actual'
   const isSingleSourcePassThrough = sources.size === 1;
@@ -63,7 +64,7 @@ export function isBarrelExport(node: DependencyNode): boolean {
 
   const isReexportPattern =
     (exports || []).length >= BARREL_EXPORT_MIN_EXPORTS &&
-    (exports || []).every((exp: any) =>
+    (exports || []).every((exp: ExportInfo) =>
       ['const', 'function', 'type', 'interface'].includes(exp.type)
     );
 
@@ -86,7 +87,7 @@ export function isTypeDefinition(node: DependencyNode): boolean {
   const areAllTypes =
     hasExports &&
     nodeExports.every(
-      (exp: any) => exp.type === 'type' || exp.type === 'interface'
+      (exp: ExportInfo) => exp.type === 'type' || exp.type === 'interface'
     );
 
   const isTypePath = /\/(types|interfaces|models)\//i.test(file);
@@ -124,7 +125,7 @@ export function isLambdaHandler(node: DependencyNode): boolean {
   );
   const isHandlerPath = /\/(handlers|lambdas|lambda|functions)\//i.test(file);
   const hasHandlerExport = (exports || []).some(
-    (exp: any) =>
+    (exp: ExportInfo) =>
       ['handler', 'main', 'lambdahandler'].includes(exp.name.toLowerCase()) ||
       exp.name.toLowerCase().endsWith('handler')
   );
@@ -146,11 +147,11 @@ export function isServiceFile(node: DependencyNode): boolean {
     fileName.includes(pattern)
   );
   const isServicePath = file.toLowerCase().includes('/services/');
-  const hasServiceNamedExport = (exports || []).some((exp: any) =>
+  const hasServiceNamedExport = (exports || []).some((exp: ExportInfo) =>
     exp.name.toLowerCase().includes('service')
   );
   const hasClassExport = (exports || []).some(
-    (exp: any) => exp.type === 'class'
+    (exp: ExportInfo) => exp.type === 'class'
   );
   return (
     isServiceName || isServicePath || (hasServiceNamedExport && hasClassExport)
@@ -173,7 +174,7 @@ export function isEmailTemplate(node: DependencyNode): boolean {
   );
   const isEmailPath = /\/(emails|mail|notifications)\//i.test(file);
   const hasTemplateFunction = (exports || []).some(
-    (exp: any) =>
+    (exp: ExportInfo) =>
       exp.type === 'function' &&
       (exp.name.toLowerCase().startsWith('render') ||
         exp.name.toLowerCase().startsWith('generate'))
@@ -197,7 +198,7 @@ export function isParserFile(node: DependencyNode): boolean {
   );
   const isParserPath = /\/(parsers|transformers)\//i.test(file);
   const hasParseFunction = (exports || []).some(
-    (exp: any) =>
+    (exp: ExportInfo) =>
       exp.type === 'function' &&
       (exp.name.toLowerCase().startsWith('parse') ||
         exp.name.toLowerCase().startsWith('transform'))
@@ -220,7 +221,7 @@ export function isSessionFile(node: DependencyNode): boolean {
     fileName.includes(pattern)
   );
   const isSessionPath = /\/(sessions|state)\//i.test(file);
-  const hasSessionExport = (exports || []).some((exp: any) =>
+  const hasSessionExport = (exports || []).some((exp: ExportInfo) =>
     ['session', 'state', 'store'].some((pattern: string) =>
       exp.name.toLowerCase().includes(pattern)
     )
@@ -246,10 +247,10 @@ export function isNextJsPage(node: DependencyNode): boolean {
     return false;
 
   const hasDefaultExport = (exports || []).some(
-    (exp: any) => exp.type === 'default'
+    (exp: ExportInfo) => exp.type === 'default'
   );
 
-  const hasNextJsExport = (exports || []).some((exp: any) =>
+  const hasNextJsExport = (exports || []).some((exp: ExportInfo) =>
     NEXTJS_METADATA_EXPORTS.includes(exp.name.toLowerCase())
   );
 
@@ -272,7 +273,7 @@ export function isConfigFile(node: DependencyNode): boolean {
     fileName.includes(pattern)
   );
   const isConfigPath = /\/(config|settings|schemas)\//i.test(lowerPath);
-  const hasSchemaExport = (exports || []).some((exp: any) =>
+  const hasSchemaExport = (exports || []).some((exp: ExportInfo) =>
     ['schema', 'config', 'setting'].some((pattern: string) =>
       exp.name.toLowerCase().includes(pattern)
     )

package/src/cli-action.ts

@@ -7,6 +7,7 @@ import {
 } from '@aiready/core';
 import { analyzeContext } from './orchestrator';
 import { generateSummary } from './summary';
+import type { ContextAnalyzerOptions } from './types';
 import chalk from 'chalk';
 import { writeFileSync } from 'fs';
 
@@ -28,7 +29,7 @@ export async function contextActionHandler(directory: string, options: any) {
   try {
     // Define defaults
     const defaults = {
-      maxDepth: 5,
+      maxDepth: 10,
       maxContextBudget: 10000,
       minCohesion: 0.6,
       maxFragmentation: 0.5,
@@ -68,7 +69,9 @@ export async function contextActionHandler(directory: string, options: any) {
     }
 
     // Run analysis
-    const results = await analyzeContext(finalOptions as any);
+    const results = await analyzeContext(
+      finalOptions as ContextAnalyzerOptions
+    );
     const summary = generateSummary(results, finalOptions);
 
     const duration = getElapsedTime(startTime);
@@ -104,7 +107,7 @@ export async function contextActionHandler(directory: string, options: any) {
     } else {
       // Default: Console (Dynamic Import)
       const { displayConsoleReport } = await import('./report/console-report');
-      displayConsoleReport(summary, results, (finalOptions as any).maxResults);
+      displayConsoleReport(summary, results, finalOptions.maxResults);
       console.log(chalk.dim(`\n✨ Analysis completed in ${duration}ms\n`));
     }
   } catch (error) {

package/src/graph-builder.ts

@@ -23,6 +23,35 @@ function resolveImport(
   // If it's not a relative import, we treat it as an external dependency for now
   // (unless it's an absolute path that exists in our set)
   if (!source.startsWith('.') && !source.startsWith('/')) {
+    // Ignore standard libraries and external packages we don't control
+    const externalIgnores = [
+      'react',
+      'next',
+      'lucide-react',
+      'framer-motion',
+      '@aws-sdk',
+      'stripe',
+      'clsx',
+      'tailwind-merge',
+      'zod',
+      'commander',
+      'chalk',
+      'fs',
+      'path',
+      'util',
+      'child_process',
+      'os',
+      'crypto',
+    ];
+
+    if (
+      externalIgnores.some(
+        (pkg) => source === pkg || source.startsWith(`${pkg}/`)
+      )
+    ) {
+      return null;
+    }
+
     // Handle monorepo package imports (@aiready/*)
     if (source.startsWith('@aiready/')) {
       const pkgName = source.split('/')[1];
@@ -144,12 +173,15 @@ export async function buildDependencyGraph(
     // 1. Get high-fidelity AST-based imports & exports
     const { imports: astImports } = await parseFileExports(content, file);
 
-    // 2. Resolve imports to absolute paths in the graph
-    const resolvedImports = astImports
+    // 2. Filter out type-only imports (they don't create runtime dependencies)
+    const runtimeImports = astImports.filter((i) => !i.isTypeOnly);
+
+    // 3. Resolve imports to absolute paths in the graph
+    const resolvedImports = runtimeImports
       .map((i) => resolveImport(i.source, file, allFilePaths))
       .filter((path): path is string => path !== null);
 
-    const importSources = astImports.map((i) => i.source);
+    const importSources = runtimeImports.map((i) => i.source);
 
     // 3. Wrap with platform-specific metadata (v0.11+)
     const exports = await extractExportsWithAST(
@@ -233,7 +265,7 @@ export function getTransitiveDependencies(
   file: string,
   graph: DependencyGraph,
   visited = new Set<string>()
-): string[] {
+): Map<string, number> {
   return getTransitiveDependenciesFromEdges(file, graph.edges, visited);
 }
 
@@ -242,7 +274,7 @@ export function getTransitiveDependencies(
  *
  * @param file - File path to calculate budget for.
  * @param graph - The dependency graph.
- * @returns Total token count including recursive dependencies.
+ * @returns Total token count including recursive dependencies (discounted by depth).
  */
 export function calculateContextBudget(
   file: string,
@@ -254,14 +286,17 @@ export function calculateContextBudget(
   let totalTokens = node.tokenCost;
   const deps = getTransitiveDependencies(file, graph);
 
-  for (const dep of deps) {
+  for (const [dep, depth] of deps.entries()) {
     const depNode = graph.nodes.get(dep);
     if (depNode) {
-      totalTokens += depNode.tokenCost;
+      // Discount token cost by depth (20% reduction per level)
+      // This prevents "barrel file" false positives where a facade pulls in the entire project
+      const discountFactor = Math.pow(0.8, depth - 1);
+      totalTokens += depNode.tokenCost * discountFactor;
     }
   }
 
-  return totalTokens;
+  return Math.round(totalTokens);
 }
 
 /**

package/src/issue-analyzer.ts

@@ -17,6 +17,7 @@ export { isBuildArtifact };
 export function analyzeIssues(params: {
   file: string;
   importDepth: number;
+  tokenCost: number;
   contextBudget: number;
   cohesionScore: number;
   fragmentationScore: number;
@@ -34,6 +35,7 @@ export function analyzeIssues(params: {
   const {
     file,
     importDepth,
+    tokenCost,
     contextBudget,
     cohesionScore,
     fragmentationScore,
@@ -74,22 +76,32 @@ export function analyzeIssues(params: {
     potentialSavings += contextBudget * 0.15;
   }
 
-  // Check context budget
-  if (contextBudget > maxContextBudget * 1.5) {
-    severity = Severity.Critical;
+  // Check direct file size
+  const MAX_FILE_TOKENS = 10000;
+  if (tokenCost > MAX_FILE_TOKENS) {
+    if (severity !== Severity.Critical) severity = Severity.Major;
     issues.push(
-      `Context budget ${contextBudget.toLocaleString()} tokens is 50% over limit`
+      `File is excessively large (${tokenCost.toLocaleString()} tokens)`
     );
     recommendations.push(
-      'Split into smaller modules or reduce dependency tree'
+      'Split file into smaller, single-responsibility modules'
+    );
+  }
+
+  // Check transitive context budget
+  if (contextBudget > maxContextBudget * 1.5) {
+    severity = Severity.Critical;
+    issues.push(
+      `Total context budget ${contextBudget.toLocaleString()} tokens is 50% over limit`
     );
+    recommendations.push('Reduce dependency tree width or reduce deep imports');
     potentialSavings += contextBudget * 0.4;
   } else if (contextBudget > maxContextBudget) {
     if (severity !== Severity.Critical) severity = Severity.Major;
     issues.push(
-      `Context budget ${contextBudget.toLocaleString()} exceeds ${maxContextBudget.toLocaleString()}`
+      `Total context budget ${contextBudget.toLocaleString()} exceeds ${maxContextBudget.toLocaleString()}`
     );
-    recommendations.push('Reduce file size or dependencies');
+    recommendations.push('Optimize dependency graph and reduce deep imports');
     potentialSavings += contextBudget * 0.2;
   }
 

package/src/orchestrator.ts

@@ -76,6 +76,7 @@ function mapNodeToResult(
     {
       file,
       importDepth,
+      tokenCost,
       contextBudget,
       cohesionScore,
       fragmentationScore,
@@ -102,8 +103,8 @@ function mapNodeToResult(
     tokenCost,
     linesOfCode: node.linesOfCode,
     importDepth,
-    dependencyCount: transitiveDeps.length,
-    dependencyList: transitiveDeps,
+    dependencyCount: transitiveDeps.size,
+    dependencyList: Array.from(transitiveDeps.keys()),
     circularDeps,
     cohesionScore,
     domains: Array.from(
@@ -166,8 +167,8 @@ export async function analyzeContext(
     ...scanOptions,
     exclude:
       includeNodeModules && scanOptions.exclude
-        ? scanOptions.exclude.filter(
-            (pattern) => pattern !== '**/node_modules/**'
+        ? (scanOptions.exclude as string[]).filter(
+            (pattern: string) => pattern !== '**/node_modules/**'
           )
         : scanOptions.exclude,
   });
@@ -197,6 +198,7 @@ export async function analyzeContext(
         analyzeIssues({
           file: metric.file,
           importDepth: metric.importDepth,
+          tokenCost: metric.contextBudget, // For Python, we use the reported budget as self-cost for now
           contextBudget: metric.contextBudget,
           cohesionScore: metric.cohesion,
           fragmentationScore: 0,

package/src/report/console-report.ts

@@ -99,12 +99,12 @@ export function displayConsoleReport(
   console.log(chalk.cyan(divider));
   console.log(
     chalk.dim(
-      '\n⭐ Like aiready? Star us on GitHub: https://github.com/caopengau/aiready-context-analyzer'
+      '\n⭐ Like aiready? Star us on GitHub: https://github.com/getaiready/aiready-context-analyzer'
     )
   );
   console.log(
     chalk.dim(
-      '🐛 Found a bug? Report it: https://github.com/caopengau/aiready-context-analyzer/issues\n'
+      '🐛 Found a bug? Report it: https://github.com/getaiready/aiready-context-analyzer/issues\n'
     )
   );
 }

package/src/report/html-report.ts

@@ -74,8 +74,8 @@ export function generateHTMLReport(
     {
       title: 'Context Analysis Report',
       packageName: 'context-analyzer',
-      packageUrl: 'https://github.com/caopengau/aiready-context-analyzer',
-      bugUrl: 'https://github.com/caopengau/aiready-context-analyzer/issues',
+      packageUrl: 'https://github.com/getaiready/aiready-context-analyzer',
+      bugUrl: 'https://github.com/getaiready/aiready-context-analyzer/issues',
       emoji: '🧠',
     },
     stats,

package/src/semantic/domain-inference.ts

@@ -171,7 +171,7 @@ export function extractExports(
         domainOptions,
         fileImports
       );
-      exports.push({ name, type, inferredDomain } as any);
+      exports.push({ name, type, inferredDomain });
     }
   });
 

package/src/types.ts

@@ -18,6 +18,8 @@ export interface ContextAnalyzerOptions extends ScanOptions {
   focus?: 'fragmentation' | 'cohesion' | 'depth' | 'all';
   /** Whether to include node_modules in the analysis (default: false) */
   includeNodeModules?: boolean;
+  /** Maximum number of results to display in console output (default: 10) */
+  maxResults?: number;
 }
 
 /**

package/src/utils/dependency-graph-utils.ts

@@ -31,24 +31,33 @@ export function getTransitiveDependenciesFromEdges(
   file: string,
   edges: Map<string, Set<string>>,
   visited = new Set<string>()
-): string[] {
-  if (visited.has(file)) return [];
+): Map<string, number> {
+  const result = new Map<string, number>();
+
+  function dfs(current: string, depth: number) {
+    if (visited.has(current)) {
+      const existingDepth = result.get(current);
+      if (existingDepth !== undefined && depth < existingDepth) {
+        result.set(current, depth);
+      }
+      return;
+    }
+    visited.add(current);
 
-  const nextVisited = new Set(visited);
-  nextVisited.add(file);
+    if (current !== file) {
+      result.set(current, depth);
+    }
 
-  const dependencies = edges.get(file);
-  if (!dependencies || dependencies.size === 0) return [];
+    const dependencies = edges.get(current);
+    if (!dependencies) return;
 
-  const allDeps: string[] = [];
-  for (const dep of dependencies) {
-    allDeps.push(dep);
-    allDeps.push(
-      ...getTransitiveDependenciesFromEdges(dep, edges, nextVisited)
-    );
+    for (const dep of dependencies) {
+      dfs(dep, depth + 1);
+    }
   }
 
-  return [...new Set(allDeps)];
+  dfs(file, 0);
+  return result;
 }
 
 export function detectGraphCycles(edges: Map<string, Set<string>>): string[][] {