@aiready/context-analyzer 0.22.15 → 0.22.17

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/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[][] {

package/tsconfig.json

@@ -2,7 +2,8 @@
   "extends": "../core/tsconfig.json",
   "compilerOptions": {
     "outDir": "./dist",
-    "rootDir": "./src"
+    "rootDir": "./src",
+    "ignoreDeprecations": "6.0"
   },
   "include": ["src/**/*"]
 }