Merge pull request #15 from avixiii-dev/ci/github-actions-update

Ci/GitHub actions update

Author: xanaawakens

.github/workflows/ci.yml

@@ -33,14 +33,15 @@ jobs:
     - name: Run tests with coverage
       run: npm run test:coverage
 
-    - name: Upload coverage reports
+    - name: Upload coverage to Codecov
       uses: codecov/codecov-action@v3
       with:
         token: ${{ secrets.CODECOV_TOKEN }}
         files: ./coverage/lcov.info
         flags: unittests
         name: codecov-umbrella
-        fail_ci_if_error: true
+        fail_ci_if_error: false
+        verbose: true
 
   build:
     needs: test

package.json

@@ -14,6 +14,7 @@
   "scripts": {
     "build": "tsc",
     "test": "jest --passWithNoTests",
+    "test:coverage": "jest --coverage",
     "lint": "eslint \"src/**/*.{js,ts}\" --fix",
     "prepublishOnly": "npm run build",
     "prepare": "npm run build"

src/__tests__/history.test.ts

@@ -1,8 +1,8 @@
 import { BundleSizeHistory } from '../plugins/history';
-import { BundleStats } from '../types/history';
 import { mkdir, writeFile, rm } from 'fs/promises';
 import { join } from 'path';
 import { tmpdir } from 'os';
+import { ExportData } from '../types/history';
 
 describe('BundleSizeHistory', () => {
   let history: BundleSizeHistory;
@@ -264,7 +264,8 @@ describe('BundleSizeHistory', () => {
     });
 
     it('should handle invalid import data', async () => {
-      const result = await history.importHistory({} as any);
+      const invalidData = {} as Partial<ExportData>;
+      const result = await history.importHistory(invalidData as ExportData);
       expect(result.success).toBe(false);
       expect(result.message).toContain('Invalid export data format');
     });

src/ai/analyzer.ts

@@ -7,6 +7,27 @@ export interface AIAnalyzerOptions {
   temperature?: number;
 }
 
+interface CodeSplittingResult {
+  splitPoints: string[];
+  impact: Array<{
+    path: string;
+    sizeReduction: number;
+  }>;
+}
+
+interface TreeShakingResult {
+  unusedExports: Array<{
+    module: string;
+    exports: string[];
+  }>;
+  potentialSavings: number;
+}
+
+interface ImpactEstimation {
+  sizeReduction: number;
+  performanceImprovement: number;
+}
+
 export class AIBundleAnalyzer {
   private openai: OpenAI;
   private model: string;
@@ -98,7 +119,7 @@ export class AIBundleAnalyzer {
       .map(line => line.trim());
   }
 
-  private parseCodeSplitting(suggestions: string): any {
+  private parseCodeSplitting(suggestions: string): CodeSplittingResult {
     const lines = suggestions.split('\n').filter(line => line.trim());
     const splitPoints = lines
       .filter(line => line.includes('/'))
@@ -116,28 +137,25 @@ export class AIBundleAnalyzer {
     };
   }
 
-  private parseTreeShaking(suggestions: string): any {
+  private parseTreeShaking(suggestions: string): TreeShakingResult {
     const lines = suggestions.split('\n').filter(line => line.trim());
     const unusedExports = lines
       .filter(line => line.includes('export'))
       .map(line => {
         const [module, ...exports] = line.split(':').map(s => s.trim());
         return {
           module,
-          exports: exports[0].split(',').map(e => e.trim())
+          exports: exports.join(':').split(',').map(e => e.trim())
         };
       });
 
     return {
       unusedExports,
-      potentialSavings: unusedExports.length * 1000 // Example estimation
+      potentialSavings: unusedExports.length * 1024 // Example estimation
     };
   }
 
-  private estimateImpact(suggestions: string, bundleInfo: BundleInfo): {
-    sizeReduction: number;
-    performanceImprovement: number;
-  } {
+  private estimateImpact(suggestions: string, bundleInfo: BundleInfo): ImpactEstimation {
     const totalSize = bundleInfo.size.raw;
     const codeSplitting = this.parseCodeSplitting(suggestions);
     const treeShaking = this.parseTreeShaking(suggestions);