refactor install time

Author: rf

packages/docs/src/components/DependencyStats.astro

@@ -91,7 +91,11 @@ function formatBytesToMB(bytes: number): string {
               <th scope="col">Min Install</th>
               <th scope="col">Max Install</th>
               <th scope="col">Avg Cold Build</th>
+              <th scope="col">Min Cold Build</th>
+              <th scope="col">Max Cold Build</th>
               <th scope="col">Avg Warm Build</th>
+              <th scope="col">Min Warm Build</th>
+              <th scope="col">Max Warm Build</th>
               <th scope="col">Build Output</th>
             </tr>
           </thead>
@@ -100,11 +104,15 @@ function formatBytesToMB(bytes: number): string {
               starterStats.map((framework) => (
                 <tr>
                   <td class="framework-name">{framework.name}</td>
-                  <td>{(framework.avgInstallTimeMs / 1000).toFixed(2)}s</td>
-                  <td>{(framework.minInstallTimeMs / 1000).toFixed(2)}s</td>
-                  <td>{(framework.maxInstallTimeMs / 1000).toFixed(2)}s</td>
+                  <td>{(framework.installTime.avgMs / 1000).toFixed(2)}s</td>
+                  <td>{(framework.installTime.minMs / 1000).toFixed(2)}s</td>
+                  <td>{(framework.installTime.maxMs / 1000).toFixed(2)}s</td>
                   <td>{(framework.coldBuildTime.avgMs / 1000).toFixed(2)}s</td>
+                  <td>{(framework.coldBuildTime.minMs / 1000).toFixed(2)}s</td>
+                  <td>{(framework.coldBuildTime.maxMs / 1000).toFixed(2)}s</td>
                   <td>{(framework.warmBuildTime.avgMs / 1000).toFixed(2)}s</td>
+                  <td>{(framework.warmBuildTime.minMs / 1000).toFixed(2)}s</td>
+                  <td>{(framework.warmBuildTime.maxMs / 1000).toFixed(2)}s</td>
                   <td>{formatBytesToMB(framework.buildOutputSize)}</td>
                 </tr>
               ))

packages/docs/src/content/config.ts

@@ -1,5 +1,11 @@
 import { defineCollection, z } from 'astro:content'
 
+const timeSchema = z.object({
+  avgMs: z.number(),
+  minMs: z.number(),
+  maxMs: z.number(),
+})
+
 const devtimeCollection = defineCollection({
   type: 'data',
   schema: z.object({
@@ -8,19 +14,9 @@ const devtimeCollection = defineCollection({
     package: z.string(),
     prodDependencies: z.number(),
     devDependencies: z.number(),
-    avgInstallTimeMs: z.number(),
-    minInstallTimeMs: z.number(),
-    maxInstallTimeMs: z.number(),
-    coldBuildTime: z.object({
-      avgMs: z.number(),
-      minMs: z.number(),
-      maxMs: z.number(),
-    }),
-    warmBuildTime: z.object({
-      avgMs: z.number(),
-      minMs: z.number(),
-      maxMs: z.number(),
-    }),
+    installTime: timeSchema,
+    coldBuildTime: timeSchema,
+    warmBuildTime: timeSchema,
     buildOutputSize: z.number(),
     nodeModulesSize: z.number(),
     nodeModulesSizeProdOnly: z.number(),

packages/stats-generator/src/run-install-benchmark.ts

@@ -130,9 +130,11 @@ async function main() {
 
     const stats: InstallStats = {
       frameworkVersion,
-      avgInstallTimeMs,
-      minInstallTimeMs,
-      maxInstallTimeMs,
+      installTime: {
+        avgMs: avgInstallTimeMs,
+        minMs: minInstallTimeMs,
+        maxMs: maxInstallTimeMs
+      },
       nodeModulesSize,
       nodeModulesSizeProdOnly,
     }
@@ -141,9 +143,9 @@ async function main() {
     writeJsonFile(outputPath, stats)
 
     console.info(`\n✓ Saved install stats to ${outputPath}`)
-    console.info(`  Average: ${stats.avgInstallTimeMs}ms`)
-    console.info(`  Min: ${stats.minInstallTimeMs}ms`)
-    console.info(`  Max: ${stats.maxInstallTimeMs}ms`)
+    console.info(`  Average: ${stats.installTime.avgMs}ms`)
+    console.info(`  Min: ${stats.installTime.minMs}ms`)
+    console.info(`  Max: ${stats.installTime.maxMs}ms`)
   } finally {
     rmSync(tempDir, { recursive: true, force: true })
   }

packages/stats-generator/src/save-ci-stats.ts

@@ -46,9 +46,7 @@ async function main() {
         stats = {
           ...stats,
           frameworkVersion: installStats.frameworkVersion,
-          avgInstallTimeMs: installStats.avgInstallTimeMs,
-          minInstallTimeMs: installStats.minInstallTimeMs,
-          maxInstallTimeMs: installStats.maxInstallTimeMs,
+          installTime: installStats.installTime,
           nodeModulesSize: installStats.nodeModulesSize,
           nodeModulesSizeProdOnly: installStats.nodeModulesSizeProdOnly,
         }

packages/stats-generator/src/types.ts

@@ -24,14 +24,12 @@ export interface CIStats {
   runner?: string
   frameworkVersion?: string
   // Install stats
-  avgInstallTimeMs?: number
-  minInstallTimeMs?: number
-  maxInstallTimeMs?: number
+  installTime?: TimeStat
   nodeModulesSize?: number
   nodeModulesSizeProdOnly?: number
   // Build stats
-  coldBuildTime?: BuildTimeStat
-  warmBuildTime?: BuildTimeStat
+  coldBuildTime?: TimeStat
+  warmBuildTime?: TimeStat
   buildOutputSize?: number
   testTimeMs?: number
   // SSR stats
@@ -44,20 +42,18 @@ export interface CIStats {
 
 export interface InstallStats {
   frameworkVersion: string
-  avgInstallTimeMs: number
-  minInstallTimeMs: number
-  maxInstallTimeMs: number
+  installTime: TimeStat
   nodeModulesSize: number
   nodeModulesSizeProdOnly: number
 }
 
 export interface BuildStats {
-  coldBuildTime: BuildTimeStat
-  warmBuildTime: BuildTimeStat
+  coldBuildTime: TimeStat
+  warmBuildTime: TimeStat
   buildOutputSize: number
 }
 
-export interface BuildTimeStat {
+export interface TimeStat {
   avgMs: number
   minMs: number
   maxMs: number