npm - @aiready/context-analyzer - Versions diffs - 0.21.22 → 0.21.24 - Mend

@aiready/context-analyzer 0.21.22 → 0.21.24

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (78) hide show

package/.turbo/turbo-build.log +26 -25
package/.turbo/turbo-lint.log +5 -5
package/.turbo/turbo-test.log +91 -41
package/coverage/clover.xml +2615 -1242
package/coverage/coverage-final.json +30 -13
package/coverage/dist/chunk-64U3PNO3.mjs.html +367 -0
package/coverage/dist/chunk-J3MUOWHC.mjs.html +5326 -0
package/coverage/dist/index.html +146 -0
package/coverage/{classifier.ts.html → dist/index.mjs.html} +537 -912
package/coverage/index.html +84 -189
package/coverage/src/analyzer.ts.html +88 -0
package/coverage/src/analyzers/index.html +116 -0
package/coverage/src/analyzers/python-context.ts.html +910 -0
package/coverage/{ast-utils.ts.html → src/ast-utils.ts.html} +84 -54
package/coverage/src/classifier.ts.html +892 -0
package/coverage/src/classify/classification-patterns.ts.html +307 -0
package/coverage/src/classify/file-classifiers.ts.html +973 -0
package/coverage/src/classify/index.html +131 -0
package/coverage/{cluster-detector.ts.html → src/cluster-detector.ts.html} +154 -91
package/coverage/{defaults.ts.html → src/defaults.ts.html} +74 -65
package/coverage/{graph-builder.ts.html → src/graph-builder.ts.html} +268 -229
package/coverage/src/index.html +341 -0
package/coverage/{index.ts.html → src/index.ts.html} +70 -13
package/coverage/{scoring.ts.html → src/issue-analyzer.ts.html} +201 -261
package/coverage/src/mapper.ts.html +439 -0
package/coverage/{metrics.ts.html → src/metrics.ts.html} +201 -132
package/coverage/src/orchestrator.ts.html +493 -0
package/coverage/{provider.ts.html → src/provider.ts.html} +21 -21
package/coverage/{remediation.ts.html → src/remediation.ts.html} +112 -52
package/coverage/src/report/console-report.ts.html +415 -0
package/coverage/src/report/html-report.ts.html +361 -0
package/coverage/src/report/index.html +146 -0
package/coverage/src/report/interactive-setup.ts.html +373 -0
package/coverage/src/scoring.ts.html +895 -0
package/coverage/src/semantic/co-usage.ts.html +340 -0
package/coverage/src/semantic/consolidation.ts.html +223 -0
package/coverage/src/semantic/domain-inference.ts.html +859 -0
package/coverage/src/semantic/index.html +161 -0
package/coverage/src/semantic/type-graph.ts.html +163 -0
package/coverage/{summary.ts.html → src/summary.ts.html} +155 -275
package/coverage/{types.ts.html → src/types.ts.html} +133 -31
package/coverage/src/utils/dependency-graph-utils.ts.html +463 -0
package/coverage/src/utils/index.html +131 -0
package/coverage/src/utils/string-utils.ts.html +148 -0
package/dist/chunk-AMPK6SWS.mjs +1754 -0
package/dist/chunk-BHCRDEE4.mjs +1745 -0
package/dist/chunk-IKRP7ECY.mjs +1754 -0
package/dist/chunk-J3MUOWHC.mjs +1747 -0
package/dist/chunk-TWWPY7FD.mjs +1754 -0
package/dist/chunk-Z5WY6A4P.mjs +1754 -0
package/dist/cli.js +77 -185
package/dist/cli.mjs +1 -1
package/dist/index.d.mts +1 -1
package/dist/index.d.ts +1 -1
package/dist/index.js +73 -181
package/dist/index.mjs +1 -1
package/dist/python-context-BWDC4E5Z.mjs +162 -0
package/package.json +3 -3
package/src/__tests__/analyzer.test.ts +14 -14
package/src/__tests__/auto-detection.test.ts +16 -16
package/src/__tests__/consolidation.test.ts +247 -0
package/src/__tests__/defaults.test.ts +121 -0
package/src/__tests__/domain-inference.test.ts +420 -0
package/src/__tests__/fragmentation-coupling.test.ts +4 -4
package/src/__tests__/issue-analyzer.test.ts +155 -0
package/src/__tests__/orchestrator.test.ts +143 -0
package/src/__tests__/python-context.test.ts +100 -0
package/src/__tests__/report/console-report.test.ts +292 -0
package/src/__tests__/report/html-report.test.ts +243 -0
package/src/__tests__/scoring.test.ts +17 -11
package/src/analyzers/python-context.ts +2 -2
package/src/ast-utils.ts +3 -3
package/src/graph-builder.ts +4 -4
package/src/mapper.ts +6 -0
package/src/orchestrator.ts +1 -1
package/src/report/html-report.ts +73 -181
package/coverage/analyzer.ts.html +0 -1369
package/coverage/semantic-analysis.ts.html +0 -1201

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

@@ -0,0 +1,243 @@
+import { describe, it, expect, vi } from 'vitest';
+import { generateHTMLReport } from '../../report/html-report';
+import type { ContextSummary, ContextAnalysisResult } from '../../types';
+// Mock @aiready/core
+vi.mock('@aiready/core', () => ({
+  generateReportHead: (title: string) => `<head><title>${title}</title></head>`,
+  generateStatCards: (stats: any[]) =>
+    `<div class="stats">${stats.map((s) => s.label).join(', ')}</div>`,
+  generateTable: (options: { headers: string[]; rows: string[][] }) =>
+    `<table><thead><tr>${options.headers.map((h) => `<th>${h}</th>`).join('')}</thead><tbody>${options.rows.map((r) => `<tr>${r.map((c) => `<td>${c}</td>`).join('')}</tr>`).join('')}</tbody></table>`,
+  generateReportFooter: (options: any) =>
+    `<footer>${options.title} - ${options.packageUrl}</footer>`,
+  generateReportHero: (title: string, subtitle?: string) =>
+    `<div class="hero"><h1>${title}</h1><p>${subtitle}</p></div>`,
+  generateIssueSummary: (
+    critical: number,
+    major: number,
+    minor: number,
+    savings?: number
+  ) =>
+    `<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>`,
+}));
+describe('generateHTMLReport', () => {
+  const createMockSummary = (
+    overrides: Partial<ContextSummary> = {}
+  ): ContextSummary => ({
+    totalFiles: 10,
+    totalTokens: 50000,
+    avgContextBudget: 5000,
+    maxContextBudget: 10000,
+    avgImportDepth: 3,
+    maxImportDepth: 7,
+    deepFiles: [],
+    avgFragmentation: 0.3,
+    fragmentedModules: [],
+    avgCohesion: 0.7,
+    lowCohesionFiles: [],
+    criticalIssues: 2,
+    majorIssues: 5,
+    minorIssues: 3,
+    totalPotentialSavings: 10000,
+    topExpensiveFiles: [],
+    config: {},
+    ...overrides,
+  });
+  const createMockResults = (): ContextAnalysisResult[] => [
+    {
+      file: 'src/services/user-service.ts',
+      tokenCost: 5000,
+      linesOfCode: 500,
+      importDepth: 5,
+      dependencyCount: 10,
+      dependencyList: [],
+      circularDeps: [],
+      cohesionScore: 0.8,
+      domains: ['user'],
+      exportCount: 5,
+      contextBudget: 15000,
+      fragmentationScore: 0.3,
+      relatedFiles: [],
+      fileClassification: 'service-file',
+      severity: 'critical',
+      issues: ['High context budget'],
+      recommendations: ['Split into smaller modules'],
+      potentialSavings: 5000,
+    },
+  ];
+  it('should generate HTML report with all sections', () => {
+    const summary = createMockSummary({
+      criticalIssues: 2,
+      majorIssues: 1,
+      minorIssues: 1,
+    });
+    const results = createMockResults();
+    const html = generateHTMLReport(summary, results);
+    expect(html).toContain('AIReady Context Analysis Report');
+    expect(html).toContain('head');
+    expect(html).toContain('body');
+  });
+  it('should include stats cards', () => {
+    const summary = createMockSummary();
+    const results: ContextAnalysisResult[] = [];
+    const html = generateHTMLReport(summary, results);
+    expect(html).toContain('Files Analyzed');
+    expect(html).toContain('Total Tokens');
+    expect(html).toContain('Avg Context Budget');
+    expect(html).toContain('Total Issues');
+  });
+  it('should include issues summary when issues exist', () => {
+    const summary = createMockSummary({
+      criticalIssues: 2,
+      majorIssues: 3,
+      minorIssues: 1,
+      totalPotentialSavings: 5000,
+    });
+    const results: ContextAnalysisResult[] = [];
+    const html = generateHTMLReport(summary, results);
+    expect(html).toContain('Issues:');
+    expect(html).toContain('Critical:');
+    expect(html).toContain('Major:');
+    expect(html).toContain('Minor:');
+    expect(html).toContain('Savings');
+  });
+  it('should not include issues section when no issues', () => {
+    const summary = createMockSummary({
+      criticalIssues: 0,
+      majorIssues: 0,
+      minorIssues: 0,
+    });
+    const results: ContextAnalysisResult[] = [];
+    const html = generateHTMLReport(summary, results);
+    expect(html).not.toContain('Issues Summary');
+  });
+  it('should include fragmented modules when present', () => {
+    const summary = createMockSummary({
+      fragmentedModules: [
+        {
+          domain: 'src/features',
+          files: ['a.ts', 'b.ts', 'c.ts'],
+          totalTokens: 3000,
+          fragmentationScore: 0.75,
+          avgCohesion: 0.4,
+          suggestedStructure: {
+            targetFiles: 2,
+            consolidationPlan: ['Consolidate into fewer modules'],
+          },
+        },
+      ],
+    });
+    const results: ContextAnalysisResult[] = [];
+    const html = generateHTMLReport(summary, results);
+    expect(html).toContain('Fragmented Modules');
+    expect(html).toContain('src/features');
+    expect(html).toContain('Domain');
+    expect(html).toContain('Files');
+    expect(html).toContain('Fragmentation');
+  });
+  it('should not include fragmented modules when none exist', () => {
+    const summary = createMockSummary({
+      fragmentedModules: [],
+    });
+    const results: ContextAnalysisResult[] = [];
+    const html = generateHTMLReport(summary, results);
+    expect(html).not.toContain('Fragmented Modules');
+  });
+  it('should include expensive files when present', () => {
+    const summary = createMockSummary({
+      topExpensiveFiles: [
+        {
+          file: 'src/services/user-service.ts',
+          contextBudget: 15000,
+          severity: 'critical',
+        },
+        {
+          file: 'src/services/order-service.ts',
+          contextBudget: 12000,
+          severity: 'major',
+        },
+      ],
+    });
+    const results: ContextAnalysisResult[] = [];
+    const html = generateHTMLReport(summary, results);
+    expect(html).toContain('Most Expensive Files');
+    expect(html).toContain('src/services/user-service.ts');
+    expect(html).toContain('Context Budget');
+    expect(html).toContain('Severity');
+  });
+  it('should not include expensive files when none exist', () => {
+    const summary = createMockSummary({
+      topExpensiveFiles: [],
+    });
+    const results: ContextAnalysisResult[] = [];
+    const html = generateHTMLReport(summary, results);
+    expect(html).not.toContain('Most Expensive Files');
+  });
+  it('should include footer with package info', () => {
+    const summary = createMockSummary();
+    const results: ContextAnalysisResult[] = [];
+    const html = generateHTMLReport(summary, results);
+    expect(html).toContain('<footer>');
+    expect(html).toContain('context-analyzer');
+    expect(html).toContain('github.com');
+  });
+  it('should include token values in stats', () => {
+    const summary = createMockSummary({
+      totalTokens: 50000,
+      avgContextBudget: 5000,
+    });
+    const results: ContextAnalysisResult[] = [];
+    const html = generateHTMLReport(summary, results);
+    // Should include stats section
+    expect(html).toContain('Total Tokens');
+  });
+  it('should show issues when present', () => {
+    const summary = createMockSummary({
+      criticalIssues: 1,
+      majorIssues: 0,
+      minorIssues: 0,
+    });
+    const results: ContextAnalysisResult[] = [];
+    const html = generateHTMLReport(summary, results);
+    // Should show issues section
+    expect(html).toContain('Issues:');
+  });
+});

package/src/__tests__/scoring.test.ts CHANGED Viewed

@@ -38,7 +38,7 @@ describe('Context Scoring', () => {
       const result = calculateContextScore(summary);
       expect(result.score).toBeLessThan(70);
-      expect(result.factors.some((f) => f.name === 'Context Budget')).toBe(
+      expect(result.factors.some((f: any) => f.name === 'Context Budget')).toBe(
         true
       );
       expect(result.recommendations.length).toBeGreaterThan(0);
@@ -60,9 +60,13 @@ describe('Context Scoring', () => {
       // With depth=12: depthScore=80, rawScore=100*0.35+80*0.25+100*0.25=80, no bonus (frag=0.2 not <0.2)
       expect(result.score).toBe(80);
-      expect(result.factors.some((f) => f.name === 'Import Depth')).toBe(true);
+      expect(result.factors.some((f: any) => f.name === 'Import Depth')).toBe(
+        true
+      );
       expect(
-        result.recommendations.some((r) => r.action.includes('import chains'))
+        result.recommendations.some((r: any) =>
+          r.action.includes('import chains')
+        )
       ).toBe(true);
     });
@@ -86,7 +90,9 @@ describe('Context Scoring', () => {
       // Actually frag=0.7 >= 0.2, so no bonus
       // rawScore = 80, no penalties = 80
       expect(result.score).toBeLessThan(85); // Adjusted for new calculation
-      expect(result.factors.some((f) => f.name === 'Fragmentation')).toBe(true);
+      expect(result.factors.some((f: any) => f.name === 'Fragmentation')).toBe(
+        true
+      );
     });
     it('should apply critical issue penalties', () => {
@@ -108,9 +114,9 @@ describe('Context Scoring', () => {
       // criticalPenalty = min(20, 5*3) = min(20,15) = 15
       // finalScore = 85 - 15 = 70
       expect(result.score).toBe(70);
-      expect(result.factors.some((f) => f.name === 'Critical Issues')).toBe(
-        true
-      );
+      expect(
+        result.factors.some((f: any) => f.name === 'Critical Issues')
+      ).toBe(true);
     });
     it('should handle extreme max budget penalty', () => {
@@ -128,10 +134,10 @@ describe('Context Scoring', () => {
       const result = calculateContextScore(summary);
       expect(
-        result.factors.some((f) => f.name === 'Extreme File Detected')
+        result.factors.some((f: any) => f.name === 'Extreme File Detected')
       ).toBe(true);
       expect(
-        result.recommendations.some((r) =>
+        result.recommendations.some((r: any) =>
           r.action.includes('Split large file')
         )
       ).toBe(true);
@@ -214,7 +220,7 @@ describe('Context Scoring', () => {
         // finalScore = 90
         expect(result.score).toBe(90);
         expect(
-          result.factors.some((f) => f.name === 'Well-Organized Codebase')
+          result.factors.some((f: any) => f.name === 'Well-Organized Codebase')
         ).toBe(true);
       });
@@ -235,7 +241,7 @@ describe('Context Scoring', () => {
         // No bonus because fragmentation >= 0.2
         expect(result.score).toBe(85); // 100*0.35 + 100*0.25 + 100*0.25 = 85
         expect(
-          result.factors.some((f) => f.name === 'Well-Organized Codebase')
+          result.factors.some((f: any) => f.name === 'Well-Organized Codebase')
         ).toBe(false);
       });

package/src/analyzers/python-context.ts CHANGED Viewed

@@ -51,7 +51,7 @@ export async function analyzePythonContext(
   rootDir: string
 ): Promise<PythonContextMetrics[]> {
   const results: PythonContextMetrics[] = [];
-  const parser = getParser('dummy.py');
+  const parser = await getParser('dummy.py');
   if (!parser) {
     console.warn('Python parser not available');
@@ -135,7 +135,7 @@ async function buildPythonDependencyGraph(
   rootDir: string
 ): Promise<Map<string, Set<string>>> {
   const graph = new Map<string, Set<string>>();
-  const parser = getParser('dummy.py');
+  const parser = await getParser('dummy.py');
   if (!parser) return graph;

package/src/ast-utils.ts CHANGED Viewed

@@ -12,14 +12,14 @@ import { inferDomain, extractExports } from './semantic/domain-inference';
  * @returns Array of high-fidelity export metadata.
  * @lastUpdated 2026-03-18
  */
-export function extractExportsWithAST(
+export async function extractExportsWithAST(
   content: string,
   filePath: string,
   domainOptions?: { domainKeywords?: string[] },
   fileImports?: string[]
-): ExportInfo[] {
+): Promise<ExportInfo[]> {
   try {
-    const { exports: astExports } = parseFileExports(content, filePath);
+    const { exports: astExports } = await parseFileExports(content, filePath);
     if (astExports.length === 0 && !isTestFile(filePath)) {
       // If AST fails to find anything, we still use regex as a last resort

package/src/graph-builder.ts CHANGED Viewed

@@ -128,10 +128,10 @@ export function extractDomainKeywordsFromPaths(files: FileContent[]): string[] {
  * @param options - Optional configuration for domain detection.
  * @returns Complete dependency graph with nodes, edges, and semantic matrices.
  */
-export function buildDependencyGraph(
+export async function buildDependencyGraph(
   files: FileContent[],
   options?: { domainKeywords?: string[] }
-): DependencyGraph {
+): Promise<DependencyGraph> {
   const nodes = new Map<string, DependencyNode>();
   const edges = new Map<string, Set<string>>();
@@ -142,7 +142,7 @@ export function buildDependencyGraph(
   for (const { file, content } of files) {
     // 1. Get high-fidelity AST-based imports & exports
-    const { imports: astImports } = parseFileExports(content, file);
+    const { imports: astImports } = await parseFileExports(content, file);
     // 2. Resolve imports to absolute paths in the graph
     const resolvedImports = astImports
@@ -152,7 +152,7 @@ export function buildDependencyGraph(
     const importSources = astImports.map((i) => i.source);
     // 3. Wrap with platform-specific metadata (v0.11+)
-    const exports = extractExportsWithAST(
+    const exports = await extractExportsWithAST(
       content,
       file,
       { domainKeywords: autoDetectedKeywords },

package/src/mapper.ts CHANGED Viewed

@@ -27,6 +27,12 @@ export interface MappingOptions {
 /**
  * Maps a single dependency node to a comprehensive ContextAnalysisResult.
+ *
+ * @param node - The dependency node to map
+ * @param graph - The full dependency graph
+ * @param clusters - All identified module clusters
+ * @param allCircularDeps - All identified circular dependencies
+ * @param options - Mapping options for detailed analysis
  */
 export function mapNodeToResult(
   node: DependencyNode,

package/src/orchestrator.ts CHANGED Viewed

@@ -69,7 +69,7 @@ export async function analyzeContext(
     }))
   );
-  const graph = buildDependencyGraph(
+  const graph = await buildDependencyGraph(
     fileContents.filter((f) => !f.file.toLowerCase().endsWith('.py'))
   );

package/src/report/html-report.ts CHANGED Viewed

@@ -1,5 +1,14 @@
 import { analyzeContext } from '../analyzer';
 import { generateSummary } from '../summary';
+import {
+  generateReportHead,
+  generateReportHero,
+  generateStatCards,
+  generateIssueSummary,
+  generateTable,
+  generateReportFooter,
+  wrapInCard,
+} from '@aiready/core';
 /**
  * Generate HTML report
@@ -14,195 +23,78 @@ export function generateHTMLReport(
   // 'results' may be used in templates later; reference to avoid lint warnings
   void results;
-  return `<!DOCTYPE html>
-<html lang="en">
-<head>
-  <meta charset="UTF-8">
-  <meta name="viewport" content="width=device-width, initial-scale=1.0">
-  <title>aiready Context Analysis Report</title>
-  <style>
-    body {
-      font-family: -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, 'Helvetica Neue', Arial, sans-serif;
-      line-height: 1.6;
-      color: #333;
-      max-width: 1200px;
-      margin: 0 auto;
-      padding: 20px;
-      background-color: #f5f5f5;
-    }
-    h1, h2, h3 { color: #2c3e50; }
-    .header {
-      background: linear-gradient(135deg, #667eea 0%, #764ba2 100%);
-      color: white;
-      padding: 30px;
-      border-radius: 8px;
-      margin-bottom: 30px;
-    }
-    .summary {
-      display: grid;
-      grid-template-columns: repeat(auto-fit, minmax(250px, 1fr));
-      gap: 20px;
-      margin-bottom: 30px;
-    }
-    .card {
-      background: white;
-      padding: 20px;
-      border-radius: 8px;
-      box-shadow: 0 2px 4px rgba(0,0,0,0.1);
-    }
-    .metric {
-      font-size: 2em;
-      font-weight: bold;
-      color: #667eea;
-    }
-    .label {
-      color: #666;
-      font-size: 0.9em;
-      margin-top: 5px;
-    }
-    .issue-critical { color: #e74c3c; }
-    .issue-major { color: #f39c12; }
-    .issue-minor { color: #3498db; }
-    table {
-      width: 100%;
-      border-collapse: collapse;
-      background: white;
-      border-radius: 8px;
-      overflow: hidden;
-      box-shadow: 0 2px 4px rgba(0,0,0,0.1);
-    }
-    th, td {
-      padding: 12px;
-      text-align: left;
-      border-bottom: 1px solid #eee;
-    }
-    th {
-      background-color: #667eea;
-      color: white;
-      font-weight: 600;
-    }
-    tr:hover { background-color: #f8f9fa; }
-    .footer {
-      text-align: center;
-      margin-top: 40px;
-      padding: 20px;
-      color: #666;
-      font-size: 0.9em;
-    }
-  </style>
-</head>
-<body>
-  <div class="header">
-    <h1>🔍 AIReady Context Analysis Report</h1>
-    <p>Generated on ${new Date().toLocaleString()}</p>
-  </div>
+  const head = generateReportHead('AIReady Context Analysis Report');
+  const stats = generateStatCards([
+    { value: summary.totalFiles, label: 'Files Analyzed' },
+    { value: summary.totalTokens.toLocaleString(), label: 'Total Tokens' },
+    { value: summary.avgContextBudget.toFixed(0), label: 'Avg Context Budget' },
+    {
+      value: totalIssues,
+      label: 'Total Issues',
+      color: totalIssues > 0 ? '#f39c12' : undefined,
+    },
+  ]);
+  const hero = generateReportHero(
+    '🔍 AIReady Context Analysis Report',
+    `Generated on ${new Date().toLocaleString()}`
+  );
-  <div class="summary">
-    <div class="card">
-      <div class="metric">${summary.totalFiles}</div>
-      <div class="label">Files Analyzed</div>
-    </div>
-    <div class="card">
-      <div class="metric">${summary.totalTokens.toLocaleString()}</div>
-      <div class="label">Total Tokens</div>
-    </div>
-    <div class="card">
-      <div class="metric">${summary.avgContextBudget.toFixed(0)}</div>
-      <div class="label">Avg Context Budget</div>
-    </div>
-    <div class="card">
-      <div class="metric ${totalIssues > 0 ? 'issue-major' : ''}">${totalIssues}</div>
-      <div class="label">Total Issues</div>
-    </div>
-  </div>
+  let body = `${hero}
+${stats}`;
-  ${
-    totalIssues > 0
-      ? `
-  <div class="card" style="margin-bottom: 30px;">
-    <h2>⚠️ Issues Summary</h2>
-    <p>
-      <span class="issue-critical">🔴 Critical: ${summary.criticalIssues}</span> &nbsp;
-      <span class="issue-major">🟡 Major: ${summary.majorIssues}</span> &nbsp;
-      <span class="issue-minor">🔵 Minor: ${summary.minorIssues}</span>
-    </p>
-    <p><strong>Potential Savings:</strong> ${summary.totalPotentialSavings.toLocaleString()} tokens</p>
-  </div>
-  `
-      : ''
+  if (totalIssues > 0) {
+    body += generateIssueSummary(
+      summary.criticalIssues,
+      summary.majorIssues,
+      summary.minorIssues,
+      summary.totalPotentialSavings
+    );
   }
-  ${
-    summary.fragmentedModules.length > 0
-      ? `
-  <div class="card" style="margin-bottom: 30px;">
-    <h2>🧩 Fragmented Modules</h2>
-    <table>
-      <thead>
-        <tr>
-          <th>Domain</th>
-          <th>Files</th>
-          <th>Fragmentation</th>
-          <th>Token Cost</th>
-        </tr>
-      </thead>
-      <tbody>
-        ${summary.fragmentedModules
-          .map(
-            (m) => `
-          <tr>
-            <td>${m.domain}</td>
-            <td>${m.files.length}</td>
-            <td>${(m.fragmentationScore * 100).toFixed(0)}%</td>
-            <td>${m.totalTokens.toLocaleString()}</td>
-          </tr>
-        `
-          )
-          .join('')}
-      </tbody>
-    </table>
-  </div>
-  `
-      : ''
+  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({
+        headers: ['Domain', 'Files', 'Fragmentation', 'Token Cost'],
+        rows: fragmentedRows,
+      }),
+      '🧩 Fragmented Modules'
+    );
   }
-  ${
-    summary.topExpensiveFiles.length > 0
-      ? `
-  <div class="card" style="margin-bottom: 30px;">
-    <h2>💸 Most Expensive Files</h2>
-    <table>
-      <thead>
-        <tr>
-          <th>File</th>
-          <th>Context Budget</th>
-          <th>Severity</th>
-        </tr>
-      </thead>
-      <tbody>
-        ${summary.topExpensiveFiles
-          .map(
-            (f) => `
-          <tr>
-            <td>${f.file}</td>
-            <td>${f.contextBudget.toLocaleString()} tokens</td>
-            <td class="issue-${f.severity}">${f.severity.toUpperCase()}</td>
-          </tr>
-        `
-          )
-          .join('')}
-      </tbody>
-    </table>
-  </div>
-  `
-      : ''
+  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({
+        headers: ['File', 'Context Budget', 'Severity'],
+        rows: expensiveRows,
+      }),
+      '💸 Most Expensive Files'
+    );
   }
-  <div class="footer">
-    <p>Generated by <strong>@aiready/context-analyzer</strong></p>
-    <p>Like AIReady? <a href="https://github.com/caopengau/aiready-context-analyzer">Star us on GitHub</a></p>
-    <p>Found a bug? <a href="https://github.com/caopengau/aiready-context-analyzer/issues">Report it here</a></p>
-  </div>
+  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>`;
 }