@aiready/context-analyzer 0.21.26 → 0.21.27

package/dist/index.mjs

@@ -1,3 +1,8 @@
+import {
+  displayConsoleReport,
+  generateHTMLReport,
+  runInteractiveSetup
+} from "./chunk-J3SZQZNU.mjs";
 import {
   BARREL_EXPORT_MIN_EXPORTS,
   BARREL_EXPORT_TOKEN_LIMIT,
@@ -17,22 +22,14 @@ import {
   buildTypeGraph,
   calculateCohesion,
   calculateContextBudget,
-  calculateDirectoryDistance,
   calculateDomainConfidence,
-  calculateEnhancedCohesion,
-  calculateFragmentation,
   calculateImportDepth,
-  calculatePathEntropy,
-  calculateStructuralCohesionFromCoUsage,
   classifyFile,
   detectCircularDependencies,
   detectModuleClusters,
-  displayConsoleReport,
   extractDomainKeywordsFromPaths,
   extractExports,
   findSemanticClusters,
-  generateHTMLReport,
-  generateSummary,
   getClassificationRecommendations,
   getCoUsageData,
   getGeneralRecommendations,
@@ -50,10 +47,19 @@ import {
   isServiceFile,
   isSessionFile,
   isTypeDefinition,
-  isUtilityModule,
-  runInteractiveSetup
-} from "./chunk-VLV6MXPL.mjs";
+  isUtilityModule
+} from "./chunk-BCEZGRXI.mjs";
 import "./chunk-64U3PNO3.mjs";
+import {
+  generateSummary
+} from "./chunk-BQCISA2F.mjs";
+import {
+  calculateDirectoryDistance,
+  calculateEnhancedCohesion,
+  calculateFragmentation,
+  calculatePathEntropy,
+  calculateStructuralCohesionFromCoUsage
+} from "./chunk-EMYD7NS6.mjs";
 
 // src/index.ts
 import { ToolRegistry } from "@aiready/core";
@@ -259,8 +265,10 @@ var ContextAnalyzerProvider = {
   id: ToolName2.ContextAnalyzer,
   alias: ["context", "fragmentation", "budget"],
   async analyze(options) {
-    const results = await analyzeContext(options);
-    const summary = generateSummary(results, options);
+    const { analyzeContext: analyzeContext2 } = await import("./orchestrator-3L3NAZYP.mjs");
+    const { generateSummary: generateSummary2 } = await import("./summary-7PZVW72O.mjs");
+    const results = await analyzeContext2(options);
+    const summary = generateSummary2(results, options);
     const normalizedResults = results.map(
       (r) => ({
         fileName: r.file,
@@ -274,7 +282,6 @@ var ContextAnalyzerProvider = {
         metrics: {
           tokenCost: r.tokenCost,
           complexityScore: r.importDepth
-          // Map other context-specific metrics if needed
         }
       })
     );

package/dist/orchestrator-3L3NAZYP.mjs

@@ -0,0 +1,10 @@
+import {
+  analyzeContext,
+  calculateCohesion
+} from "./chunk-BCEZGRXI.mjs";
+import "./chunk-64U3PNO3.mjs";
+import "./chunk-EMYD7NS6.mjs";
+export {
+  analyzeContext,
+  calculateCohesion
+};

package/dist/orchestrator-MONOZHVW.mjs

@@ -0,0 +1,10 @@
+import {
+  analyzeContext,
+  calculateCohesion
+} from "./chunk-22ZO4EKZ.mjs";
+import "./chunk-64U3PNO3.mjs";
+import "./chunk-EMYD7NS6.mjs";
+export {
+  analyzeContext,
+  calculateCohesion
+};

package/dist/orchestrator-ZR7JSKWI.mjs

@@ -0,0 +1,10 @@
+import {
+  analyzeContext,
+  calculateCohesion
+} from "./chunk-OZE3FVZT.mjs";
+import "./chunk-64U3PNO3.mjs";
+import "./chunk-4U4LDWGF.mjs";
+export {
+  analyzeContext,
+  calculateCohesion
+};

package/dist/summary-7PZVW72O.mjs

@@ -0,0 +1,7 @@
+import {
+  generateSummary
+} from "./chunk-BQCISA2F.mjs";
+import "./chunk-EMYD7NS6.mjs";
+export {
+  generateSummary
+};

package/dist/summary-LKUCJAIS.mjs

@@ -0,0 +1,7 @@
+import {
+  generateSummary
+} from "./chunk-WHB7QI7N.mjs";
+import "./chunk-4U4LDWGF.mjs";
+export {
+  generateSummary
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aiready/context-analyzer",
-  "version": "0.21.26",
+  "version": "0.21.27",
   "description": "AI context window cost analysis - detect fragmented code, deep import chains, and expensive context budgets",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",
@@ -49,7 +49,7 @@
     "commander": "^14.0.0",
     "chalk": "^5.3.0",
     "prompts": "^2.4.2",
-    "@aiready/core": "0.23.23"
+    "@aiready/core": "0.23.24"
   },
   "devDependencies": {
     "@types/node": "^24.0.0",

package/src/__tests__/analyzer.test.ts

@@ -207,7 +207,7 @@ describe('calculateCohesion', () => {
     // Mock file - should return 1 despite mixed domains
     const cohesionMockFile = calculateCohesion(
       exports,
-      'src/test-utils/mocks.ts'
+      'src/__tests__/mocks.ts'
     );
     expect(cohesionMockFile).toBe(1);
 

package/src/__tests__/enhanced-cohesion.test.ts

@@ -132,7 +132,10 @@ describe('Enhanced Cohesion Calculation', () => {
       },
     ];
 
-    const cohesion = calculateCohesion(exports, 'src/utils/test-helpers.ts');
+    const cohesion = calculateCohesion(
+      exports,
+      'src/__tests__/test-helpers.ts'
+    );
     expect(cohesion).toBe(1);
   });
 });

package/src/__tests__/orchestrator.test.ts

@@ -12,6 +12,20 @@ vi.mock('@aiready/core', async () => {
   };
 });
 
+// Mock fs to prevent ENOENT errors when python-context tries to read files
+vi.mock('fs', () => ({
+  default: {
+    promises: {
+      readFile: vi.fn().mockResolvedValue('# Python file\nprint("hello")'),
+    },
+    existsSync: vi.fn().mockReturnValue(true),
+  },
+  promises: {
+    readFile: vi.fn().mockResolvedValue('# Python file\nprint("hello")'),
+  },
+  existsSync: vi.fn().mockReturnValue(true),
+}));
+
 describe('analyzeContext', () => {
   beforeEach(() => {
     vi.clearAllMocks();
@@ -60,7 +74,7 @@ describe('analyzeContext', () => {
     expect(results).toEqual([]);
   });
 
-  it('should filter Python files when not present', async () => {
+  it('should process both TypeScript and Python files', async () => {
     const mockFiles = ['src/file1.ts', 'src/file2.py'];
     vi.mocked(core.scanFiles).mockResolvedValue(mockFiles);
     vi.mocked(core.readFileContent).mockResolvedValue('export const a = 1;');
@@ -69,9 +83,10 @@ describe('analyzeContext', () => {
       rootDir: '.',
     });
 
-    // Should only process TypeScript files
-    expect(results.length).toBe(1);
-    expect(results[0].file).toBe('src/file1.ts');
+    // Orchestrator processes both TypeScript and Python files
+    expect(results.length).toBe(2);
+    expect(results.some((r) => r.file === 'src/file1.ts')).toBe(true);
+    expect(results.some((r) => r.file === 'src/file2.py')).toBe(true);
   });
 
   it('should use default options when not provided', async () => {

package/src/__tests__/python-context.test.ts

@@ -27,6 +27,12 @@ vi.mock('@aiready/core', () => ({
 
 // Mock fs
 vi.mock('fs', () => ({
+  default: {
+    promises: {
+      readFile: vi.fn(),
+    },
+    existsSync: vi.fn(),
+  },
   promises: {
     readFile: vi.fn(),
   },

package/src/__tests__/report/html-report.test.ts

@@ -22,6 +22,13 @@ vi.mock('@aiready/core', () => ({
     `<div class="issue-summary">Issues: Critical:${critical}, Major:${major}, Minor:${minor}. Savings: ${savings}</div>`,
   wrapInCard: (content: string, title?: string) =>
     `<div class="card"><h2>${title}</h2>${content}</div>`,
+  generateStandardHtmlReport: (config: any, stats: any[], sections: any[]) => {
+    const sectionHtml = sections
+      .map((s: any) => `<h2>${s.title}</h2>${s.content}`)
+      .join('');
+    const footerHtml = `<footer>${config.packageName} - ${config.packageUrl}</footer>`;
+    return `<!DOCTYPE html><html>${config.title}<body>${stats.map((s: any) => s.label).join(', ')}${sectionHtml}${footerHtml}</body></html>`;
+  },
 }));
 
 describe('generateHTMLReport', () => {
@@ -81,8 +88,7 @@ describe('generateHTMLReport', () => {
 
     const html = generateHTMLReport(summary, results);
 
-    expect(html).toContain('AIReady Context Analysis Report');
-    expect(html).toContain('head');
+    expect(html).toContain('Context Analysis Report');
     expect(html).toContain('body');
   });
 

package/src/ast-utils.ts

@@ -1,4 +1,4 @@
-import { parseFileExports } from '@aiready/core';
+import { parseFileExports, isTestFile } from '@aiready/core';
 import type { ExportInfo } from './types';
 import { inferDomain, extractExports } from './semantic/domain-inference';
 
@@ -46,31 +46,6 @@ export async function extractExportsWithAST(
   }
 }
 
-/**
- * Check if a file is a test, mock, or fixture file.
- *
- * @param filePath - The path to the file to check.
- * @returns True if the file matches test/mock patterns.
- */
-export function isTestFile(filePath: string): boolean {
-  const lower = filePath.toLowerCase();
-  return (
-    lower.includes('.test.') ||
-    lower.includes('.spec.') ||
-    lower.includes('/__tests__/') ||
-    lower.includes('/tests/') ||
-    lower.includes('/test/') ||
-    lower.includes('test-') ||
-    lower.includes('-test') ||
-    lower.includes('/__mocks__/') ||
-    lower.includes('/mocks/') ||
-    lower.includes('/fixtures/') ||
-    lower.includes('.mock.') ||
-    lower.includes('.fixture.') ||
-    lower.includes('/test-utils/')
-  );
-}
-
 /**
  * Heuristic to check if all exports share a common entity noun
  */

package/src/cli-definition.ts

@@ -1,5 +1,4 @@
 import { Command } from 'commander';
-import { contextActionHandler } from './cli-action';
 
 /**
  * Define the context analysis command structure.
@@ -57,5 +56,8 @@ export function defineContextCommand(program: Command): void {
       '--interactive',
       'Run interactive setup to suggest excludes and focus areas'
     )
-    .action(contextActionHandler);
+    .action(async (directory, options) => {
+      const { contextActionHandler } = await import('./cli-action');
+      await contextActionHandler(directory, options);
+    });
 }

package/src/issue-analyzer.ts

@@ -1,4 +1,7 @@
-import { Severity } from '@aiready/core';
+import { Severity, isBuildArtifact } from '@aiready/core';
+
+// Re-export for testing
+export { isBuildArtifact };
 
 /**
  * Internal issue analysis logic
@@ -136,21 +139,3 @@ export function analyzeIssues(params: {
     potentialSavings: Math.floor(potentialSavings),
   };
 }
-
-/**
- * Determines if a file path belongs to a build artifact or dependency folder.
- * Helps exclude generated files from analysis to prevent false positives.
- *
- * @param filePath - The path to check
- * @returns True if the file is a build artifact, false otherwise
- */
-export function isBuildArtifact(filePath: string): boolean {
-  const lower = filePath.toLowerCase();
-  return (
-    lower.includes('/node_modules/') ||
-    lower.includes('/dist/') ||
-    lower.includes('/build/') ||
-    lower.includes('/out/') ||
-    lower.includes('/.next/')
-  );
-}

package/src/metrics.ts

@@ -1,6 +1,5 @@
-import { calculateImportSimilarity } from '@aiready/core';
+import { calculateImportSimilarity, isTestFile } from '@aiready/core';
 import type { ExportInfo } from './types';
-import { isTestFile } from './ast-utils';
 
 /**
  * Calculates a cohesion score (0-1) for a module based on its exports,

package/src/provider.ts

@@ -9,10 +9,8 @@ import {
   IssueType,
   SpokeOutputSchema,
 } from '@aiready/core';
-import { analyzeContext } from './orchestrator';
-import { generateSummary } from './summary';
 import { calculateContextScore } from './scoring';
-import { ContextAnalyzerOptions, ContextAnalysisResult } from './types';
+import type { ContextAnalyzerOptions, ContextAnalysisResult } from './types';
 
 /**
  * Context Analyzer Tool Provider
@@ -22,6 +20,9 @@ export const ContextAnalyzerProvider: ToolProvider = {
   alias: ['context', 'fragmentation', 'budget'],
 
   async analyze(options: ScanOptions): Promise<SpokeOutput> {
+    const { analyzeContext } = await import('./orchestrator');
+    const { generateSummary } = await import('./summary');
+
     const results = await analyzeContext(options as ContextAnalyzerOptions);
     const summary = generateSummary(results, options);
 
@@ -39,7 +40,6 @@ export const ContextAnalyzerProvider: ToolProvider = {
         metrics: {
           tokenCost: r.tokenCost,
           complexityScore: r.importDepth,
-          // Map other context-specific metrics if needed
         },
       })
     );

package/src/report/html-report.ts

@@ -1,13 +1,9 @@
 import { analyzeContext } from '../orchestrator';
 import { generateSummary } from '../summary';
 import {
-  generateReportHead,
-  generateReportHero,
-  generateStatCards,
   generateIssueSummary,
   generateTable,
-  generateReportFooter,
-  wrapInCard,
+  generateStandardHtmlReport,
 } from '@aiready/core';
 
 /**
@@ -19,13 +15,9 @@ export function generateHTMLReport(
 ): string {
   const totalIssues =
     summary.criticalIssues + summary.majorIssues + summary.minorIssues;
-
-  // 'results' may be used in templates later; reference to avoid lint warnings
   void results;
 
-  const head = generateReportHead('AIReady Context Analysis Report');
-
-  const stats = generateStatCards([
+  const stats = [
     { value: summary.totalFiles, label: 'Files Analyzed' },
     { value: summary.totalTokens.toLocaleString(), label: 'Total Tokens' },
     { value: summary.avgContextBudget.toFixed(0), label: 'Avg Context Budget' },
@@ -34,67 +26,59 @@ export function generateHTMLReport(
       label: 'Total Issues',
       color: totalIssues > 0 ? '#f39c12' : undefined,
     },
-  ]);
-
-  const hero = generateReportHero(
-    '🔍 AIReady Context Analysis Report',
-    `Generated on ${new Date().toLocaleString()}`
-  );
-
-  let body = `${hero}
-${stats}`;
+  ];
 
+  const sections: any[] = [];
   if (totalIssues > 0) {
-    body += generateIssueSummary(
-      summary.criticalIssues,
-      summary.majorIssues,
-      summary.minorIssues,
-      summary.totalPotentialSavings
-    );
+    sections.push({
+      title: '⚠️ Issues Summary',
+      content: generateIssueSummary(
+        summary.criticalIssues,
+        summary.majorIssues,
+        summary.minorIssues,
+        summary.totalPotentialSavings
+      ),
+    });
   }
 
   if (summary.fragmentedModules.length > 0) {
-    const fragmentedRows = summary.fragmentedModules.map((m) => [
-      m.domain,
-      String(m.files.length),
-      `${(m.fragmentationScore * 100).toFixed(0)}%`,
-      m.totalTokens.toLocaleString(),
-    ]);
-    body += wrapInCard(
-      generateTable({
+    sections.push({
+      title: '🧩 Fragmented Modules',
+      content: generateTable({
         headers: ['Domain', 'Files', 'Fragmentation', 'Token Cost'],
-        rows: fragmentedRows,
+        rows: summary.fragmentedModules.map((m) => [
+          m.domain,
+          String(m.files.length),
+          `${(m.fragmentationScore * 100).toFixed(0)}%`,
+          m.totalTokens.toLocaleString(),
+        ]),
       }),
-      '🧩 Fragmented Modules'
-    );
+    });
   }
 
   if (summary.topExpensiveFiles.length > 0) {
-    const expensiveRows = summary.topExpensiveFiles.map((f) => [
-      f.file,
-      `${f.contextBudget.toLocaleString()} tokens`,
-      `<span class="issue-${f.severity}">${f.severity.toUpperCase()}</span>`,
-    ]);
-    body += wrapInCard(
-      generateTable({
+    sections.push({
+      title: '💸 Most Expensive Files',
+      content: generateTable({
         headers: ['File', 'Context Budget', 'Severity'],
-        rows: expensiveRows,
+        rows: summary.topExpensiveFiles.map((f) => [
+          f.file,
+          `${f.contextBudget.toLocaleString()} tokens`,
+          `<span class="issue-${f.severity}">${f.severity.toUpperCase()}</span>`,
+        ]),
       }),
-      '💸 Most Expensive Files'
-    );
+    });
   }
 
-  const footer = generateReportFooter({
-    title: 'Context Analysis Report',
-    packageName: 'context-analyzer',
-    packageUrl: 'https://github.com/caopengau/aiready-context-analyzer',
-    bugUrl: 'https://github.com/caopengau/aiready-context-analyzer/issues',
-  });
-
-  return `${head}
-<body>
-  ${body}
-  ${footer}
-</body>
-</html>`;
+  return generateStandardHtmlReport(
+    {
+      title: 'Context Analysis Report',
+      packageName: 'context-analyzer',
+      packageUrl: 'https://github.com/caopengau/aiready-context-analyzer',
+      bugUrl: 'https://github.com/caopengau/aiready-context-analyzer/issues',
+      emoji: '🧠',
+    },
+    stats,
+    sections
+  );
 }