feat: improve analytics tracking with model/provider info and faster shutdown

Author: shivasurya

extension/secureflow/packages/secureflow-cli/bin/secureflow

@@ -66,9 +66,14 @@ async function main() {
     .option('--defectdojo-test-title <title>', 'Title for the DefectDojo test (defaults to "SecureFlow Scan")')
     .action(async (projectPath, options) => {
     try {
-      // Track scan command usage
+      // Load config to get actual model being used
+      const config = loadConfig();
+      const actualModel = options.model || config.model;
+      
+      // Track scan command usage with actual model
       await analytics.trackEvent('CLI Command: Scan', {
-        ai_model: options.model || 'default',
+        ai_model: actualModel,
+        ai_provider: config.provider,
         output_format: options.format || 'text',
         has_defectdojo: !!options.defectdojo
       });
@@ -111,9 +116,14 @@ async function main() {
     .option('--output <file>', 'Save results to file')
     .action(async (projectPath, options) => {
     try {
-      // Track profile command usage
+      // Load config to get actual model being used
+      const config = loadConfig();
+      const actualModel = options.model || config.model;
+      
+      // Track profile command usage with actual model
       await analytics.trackEvent('CLI Command: Profile', {
-        ai_model: options.model || 'default',
+        ai_model: actualModel,
+        ai_provider: config.provider,
         output_format: options.format || 'text'
       });
 
@@ -198,8 +208,8 @@ async function main() {
   // Parse CLI arguments
   await program.parseAsync(process.argv);
 
-  // Shutdown analytics after command completes
-  await analytics.shutdown();
+  // Quick shutdown for fast exit (fire and forget)
+  analytics.shutdown(true).catch(() => {});
 }
 
 // Run the main function

extension/secureflow/packages/secureflow-cli/lib/services/analytics.d.ts

@@ -66,6 +66,7 @@ export declare class AnalyticsService {
 
   /**
    * Shutdown analytics service
+   * @param quick - If true, don't wait for flush (faster exit)
    */
-  shutdown(): Promise<void>;
+  shutdown(quick?: boolean): Promise<void>;
 }

extension/secureflow/packages/secureflow-cli/lib/services/analytics.js

@@ -17,7 +17,7 @@ const os = require('os');
 const POSTHOG_API_KEY = 'phc_iOS0SOw2gDax8kq44Z9FBEVAs8m6QG7yANvBF8ItV6g';
 const POSTHOG_HOST = 'https://us.i.posthog.com';
 const DISTINCT_ID_KEY = 'secureflow.analytics.distinctId';
-const SHUTDOWN_TIMEOUT_MS = 2000;
+const SHUTDOWN_TIMEOUT_MS = 500; // Reduced from 2000ms to 500ms
 
 /**
  * Storage adapter interface for distinct ID persistence
@@ -248,7 +248,8 @@ class AnalyticsService {
         event: eventName,
         properties: eventProperties
       });
-      await this.posthog.flush();
+      // Fire and forget - don't wait for flush to complete
+      this.posthog.flush().catch(() => {});
     } catch (error) {
       // Silently fail - analytics should not disrupt application
     }
@@ -270,13 +271,23 @@ class AnalyticsService {
 
   /**
    * Shutdown analytics service
+   * @param {boolean} quick - If true, don't wait for flush (faster exit)
    */
-  async shutdown() {
+  async shutdown(quick = false) {
     if (!this.posthog) {
       return;
     }
 
     try {
+      if (quick) {
+        // For quick commands, just shutdown without waiting
+        this.posthog.shutdown().catch(() => {}); // Fire and forget
+        this.posthog = null;
+        this.initialized = false;
+        return;
+      }
+
+      // Normal shutdown with reduced timeout
       const shutdownPromise = this.posthog.shutdown();
       const timeoutPromise = new Promise((_, reject) => 
         setTimeout(() => reject(new Error('Shutdown timeout')), SHUTDOWN_TIMEOUT_MS)