better formatting for performance results

Author: bhouston

packages/cli/src/commands/test-profile.ts

@@ -1,4 +1,5 @@
 import { CommandModule } from 'yargs';
+
 import { SharedOptions } from '../options.js';
 
 export const command: CommandModule<object, SharedOptions> = {
@@ -7,8 +8,8 @@ export const command: CommandModule<object, SharedOptions> = {
   handler: async () => {
     console.log('Profile test completed successfully');
     // Profiling report will be automatically displayed by the main function
-    
+
     // Force a delay to simulate some processing
-    await new Promise(resolve => setTimeout(resolve, 100));
+    await new Promise((resolve) => setTimeout(resolve, 100));
   },
-};
+};

packages/cli/src/index.ts

@@ -7,12 +7,12 @@ import { hideBin } from 'yargs/helpers';
 
 import { command as defaultCommand } from './commands/$default.js';
 import { command as configCommand } from './commands/config.js';
-import { command as testSentryCommand } from './commands/test-sentry.js';
 import { command as testProfileCommand } from './commands/test-profile.js';
+import { command as testSentryCommand } from './commands/test-sentry.js';
 import { command as toolsCommand } from './commands/tools.js';
 import { sharedOptions } from './options.js';
-import { getConfig } from './settings/config.js';
 import { initSentry, captureException } from './sentry/index.js';
+import { getConfig } from './settings/config.js';
 import { enableProfiling, mark, reportTimings } from './utils/performance.js';
 
 mark('After imports');
@@ -27,30 +27,35 @@ mark('After sourceMapSupport install');
 
 const main = async () => {
   // Parse argv early to check for profiling flag
-  const parsedArgv = await yargs(hideBin(process.argv)).options(sharedOptions).parse();
-  
+  const parsedArgv = await yargs(hideBin(process.argv))
+    .options(sharedOptions)
+    .parse();
+
   // Get config to check for profile setting
   const config = getConfig();
-  
+
   // Enable profiling if --profile flag is set or if enabled in config
   enableProfiling(Boolean(parsedArgv.profile) || Boolean(config.profile));
-  
+
   mark('Main function start');
-  
+
   dotenv.config();
   mark('After dotenv config');
-  
+
   // Only initialize Sentry if needed
-  if (process.env.NODE_ENV !== 'development' || process.env.ENABLE_SENTRY === 'true') {
+  if (
+    process.env.NODE_ENV !== 'development' ||
+    process.env.ENABLE_SENTRY === 'true'
+  ) {
     initSentry();
     mark('After Sentry init');
   }
-  
+
   mark('Before package.json load');
   const require = createRequire(import.meta.url);
   const packageInfo = require('../package.json') as PackageJson;
   mark('After package.json load');
-  
+
   // Set up yargs with the new CLI interface
   mark('Before yargs setup');
   await yargs(hideBin(process.argv))
@@ -69,17 +74,16 @@ const main = async () => {
     .strict()
     .showHelpOnFail(true)
     .help().argv;
-  mark('After yargs setup');
-  
-  // Report timings if profiling is enabled
-  await reportTimings();
 };
 
-await main().catch(async (error) => {
-  console.error(error);
-  // Report profiling data even if there's an error
-  await reportTimings();
-  // Capture the error with Sentry
-  captureException(error);
-  process.exit(1);
-});
+await main()
+  .catch(async (error) => {
+    console.error(error);
+    // Capture the error with Sentry
+    captureException(error);
+    process.exit(1);
+  })
+  .finally(async () => {
+    // Report timings if profiling is enabled
+    await reportTimings();
+  });

packages/cli/src/utils/performance.ts

@@ -25,22 +25,26 @@ export function mark(label: string): void {
  */
 export async function reportTimings(): Promise<void> {
   if (!isEnabled) return;
-  
-  console.log('\n📊 Performance Profile:');
+
+  console.log('\n📊 Performance Profile Results');
   console.log('=======================');
-  
+  console.log(
+    `${'Label'.padEnd(40, ' ')}${'Time'.padStart(10, ' ')}${'Duration'.padStart(10, ' ')}`,
+  );
+
   // Sort timings by time value
-  const sortedTimings = Object.entries(timings)
-    .sort((a, b) => a[1] - b[1]);
-  
+  const sortedTimings = Object.entries(timings).sort((a, b) => a[1] - b[1]);
+
   // Calculate durations between steps
   let previousTime = 0;
   for (const [label, time] of sortedTimings) {
     const duration = time - previousTime;
-    console.log(`${label}: ${time.toFixed(2)}ms (${duration.toFixed(2)}ms)`);
+    console.log(
+      `${label.padEnd(40, ' ')}${`${time.toFixed(2)}ms`.padStart(10, ' ')}${`${duration.toFixed(2)}ms`.padStart(10, ' ')}`,
+    );
     previousTime = time;
   }
-  
+
   console.log(`Total startup time: ${previousTime.toFixed(2)}ms`);
   console.log('=======================\n');
 
@@ -55,34 +59,37 @@ export async function reportTimings(): Promise<void> {
  */
 async function reportPlatformInfo(): Promise<void> {
   if (!isEnabled) return;
-  
+
   console.log('\n🖥️ Platform Information:');
   console.log('=======================');
   console.log(`Platform: ${process.platform}`);
   console.log(`Architecture: ${process.arch}`);
   console.log(`Node.js version: ${process.version}`);
-  
+
   // Windows-specific information
   if (process.platform === 'win32') {
     console.log('Windows-specific details:');
     console.log(`- Current working directory: ${process.cwd()}`);
     console.log(`- Path length: ${process.cwd().length} characters`);
-    
+
     // Check for antivirus markers by measuring file read time
     try {
       // Using dynamic import to avoid require
       const fs = await import('fs');
       const startTime = performance.now();
       fs.readFileSync(process.execPath);
-      console.log(`- Time to read Node.js executable: ${(performance.now() - startTime).toFixed(2)}ms`);
+      console.log(
+        `- Time to read Node.js executable: ${(performance.now() - startTime).toFixed(2)}ms`,
+      );
     } catch (error: unknown) {
-      const errorMessage = error instanceof Error ? error.message : String(error);
+      const errorMessage =
+        error instanceof Error ? error.message : String(error);
       console.log(`- Error reading Node.js executable: ${errorMessage}`);
     }
   }
-  
+
   console.log('=======================\n');
 }
 
 // Initial mark for module load time
-mark('Module initialization');
+mark('Module initialization');