fix: remove scroll and cursor tracking from analytics configuration

Author: estruyf

apps/vscode-extension/src/services/DecoratorService.ts

@@ -80,6 +80,9 @@ export class DecoratorService {
     // Create blur decorator with step-specific or global blur/opacity values
     const blur = DecoratorService.getBlurValue(stepBlur);
     const opacity = DecoratorService.getOpacityValue(stepOpacity);
+    if (DecoratorService.blurDecorator) {
+      DecoratorService.blurDecorator.dispose();
+    }
     DecoratorService.blurDecorator = DecoratorService.createBlurDecorator(blur, opacity);
 
     // Reset the decorators

apps/vscode-extension/src/services/analytics/AnalyticsService.ts

@@ -50,8 +50,6 @@ export class AnalyticsService {
     AnalyticsService.config = {
       enabled: enabled ?? DEFAULT_ANALYTICS_CONFIG.enabled,
       narrativeThreshold: DEFAULT_ANALYTICS_CONFIG.narrativeThreshold,
-      trackCursorMovements: DEFAULT_ANALYTICS_CONFIG.trackCursorMovements,
-      trackScrollEvents: DEFAULT_ANALYTICS_CONFIG.trackScrollEvents,
       trackTerminalCommands: DEFAULT_ANALYTICS_CONFIG.trackTerminalCommands,
       autoSaveInterval: DEFAULT_ANALYTICS_CONFIG.autoSaveInterval,
     };
@@ -425,56 +423,6 @@ export class AnalyticsService {
     AnalyticsService.lastEventTime = Date.now();
   }
 
-  /**
-   * Records a scroll event.
-   */
-  public static recordScroll(
-    filePath: string,
-    direction: 'up' | 'down',
-    linesScrolled: number,
-    visibleStart?: number,
-    visibleEnd?: number,
-  ): void {
-    if (!AnalyticsService.currentSession || !AnalyticsService.config.trackScrollEvents) {
-      return;
-    }
-
-    const now = new Date().toISOString();
-    const normalizedPath = AnalyticsService.normalizePath(filePath);
-
-    const activity = AnalyticsService.fileActivityMap.get(normalizedPath);
-    if (activity) {
-      // Calculate dwell time from last event
-      const dwellTime = Date.now() - AnalyticsService.lastEventTime;
-
-      activity.scrollEvents.push({
-        timestamp: now,
-        direction,
-        linesScrolled,
-        dwellTime,
-        visibleRange:
-          visibleStart !== undefined && visibleEnd !== undefined
-            ? { start: visibleStart, end: visibleEnd }
-            : undefined,
-      });
-
-      // Record navigation event
-      AnalyticsService.recordNavigationEvent({
-        timestamp: now,
-        type: 'scroll',
-        toFile: normalizedPath,
-        details: {
-          lineRange:
-            visibleStart !== undefined && visibleEnd !== undefined
-              ? { start: visibleStart, end: visibleEnd }
-              : undefined,
-        },
-      });
-    }
-
-    AnalyticsService.lastEventTime = Date.now();
-  }
-
   /**
    * Records a terminal command execution.
    */

packages/common/src/models/Analytics.ts

@@ -411,10 +411,6 @@ export interface AnalyticsConfig {
   enabled: boolean;
   /** Minimum narrative duration to detect (milliseconds) */
   narrativeThreshold: number;
-  /** Whether to track cursor movements */
-  trackCursorMovements: boolean;
-  /** Whether to track scroll events */
-  trackScrollEvents: boolean;
   /** Whether to track terminal commands */
   trackTerminalCommands: boolean;
   /** Auto-save interval in milliseconds (0 = only save at end) */
@@ -427,8 +423,6 @@ export interface AnalyticsConfig {
 export const DEFAULT_ANALYTICS_CONFIG: AnalyticsConfig = {
   enabled: true,
   narrativeThreshold: 30000, // 30 seconds
-  trackCursorMovements: false, // Can be verbose
-  trackScrollEvents: true,
   trackTerminalCommands: true,
   autoSaveInterval: 30000, // 30 seconds
 };