Fix behavior tracking initialization timing

- Move behavior tracking init to after ACE editor is ready
- Use custom events to bridge ACE editor events to tracking module
- Track paste events from ACE editor properly
- Track typing patterns from ACE editor change events
- Add debug logging throughout

The issue was that behavior tracking was initialized before the editor existed.
Now it initializes after the editor is created and hooks into ACE editor events.

Author: Archith

scripts/app.js

@@ -81,13 +81,12 @@
           // Remove the security check message - just proceed
         }
 
-        // Initialize behavior tracking for candidates only
-        if (window.initBehaviorTracking) {
-          console.log('Initializing behavior tracking for candidate:', name);
-          window.initBehaviorTracking(sessionCode, name, 'candidate');
-        } else {
-          console.warn('Behavior tracking module not loaded');
-        }
+        // Store behavior tracking info to initialize after editor loads
+        window.pendingBehaviorTracking = {
+          sessionCode: sessionCode,
+          name: name,
+          type: 'candidate'
+        };
 
         Auth.joinAsCandidate(name);
         window.location.hash = sessionCode;

scripts/behavior-tracking.js

@@ -102,19 +102,20 @@
   function trackPasteEvents() {
     console.log('Setting up paste tracking...');
 
-    // Track paste in CodeMirror
-    if (window.codeMirror) {
-      console.log('CodeMirror found, attaching paste listener');
-      window.codeMirror.on('paste', function(cm, event) {
-        console.log('CodeMirror paste event detected');
-        if (!isTracking) return;
-        handlePasteEvent(event);
-      });
-    } else {
-      console.log('CodeMirror not found, will track global paste only');
-    }
+    // Listen for ACE editor paste events (sent from firepad.js)
+    document.addEventListener('editorPaste', function(e) {
+      console.log('Editor paste event received:', e.detail);
+      if (!isTracking) return;
+      
+      const fakeEvent = {
+        clipboardData: {
+          getData: function() { return e.detail.text; }
+        }
+      };
+      handlePasteEvent(fakeEvent);
+    });
 
-    // Also track global paste
+    // Also track global paste (for paste outside editor)
     document.addEventListener('paste', function(e) {
       console.log('Global paste event detected, tracking:', isTracking);
       if (!isTracking) return;
@@ -198,24 +199,29 @@
 
   // 3. Typing Pattern Analysis
   function trackTypingPatterns() {
-    if (!window.codeMirror) return;
+    console.log('Setting up typing pattern tracking...');
 
     let keystrokeBuffer = [];
     let lastEventTime = Date.now();
 
-    window.codeMirror.on('change', function(cm, change) {
-      if (!isTracking || change.origin === 'setValue') return;
+    // Listen for ACE editor change events (sent from firepad.js)
+    document.addEventListener('editorChange', function(e) {
+      if (!isTracking) return;
+      
+      const change = e.detail;
+      if (change.action === 'setValue') return; // Ignore programmatic changes
 
       const now = Date.now();
       const timeDiff = now - lastEventTime;
       lastEventTime = now;
 
       // Track keystroke timing
-      if (change.origin === 'input' || change.origin === '+input') {
+      if (change.action === 'insert' || change.action === 'insertText') {
         typingMetrics.totalKeystrokes++;
 
         // Calculate WPM from recent keystrokes
-        keystrokeBuffer.push({ time: now, chars: change.text.join('').length });
+        const textLength = change.text ? change.text.length : 0;
+        keystrokeBuffer.push({ time: now, chars: textLength });
 
         // Keep only last 5 seconds of keystrokes
         keystrokeBuffer = keystrokeBuffer.filter(k => now - k.time < 5000);
@@ -240,9 +246,10 @@
       }
 
       // Detect large insertions (potential paste via typing)
-      if (change.text.join('').length > 50 && timeDiff < 100) {
+      const insertedText = change.text || '';
+      if (insertedText.length > 50 && timeDiff < 100) {
         notifyInterviewer('instant_code_appearance', {
-          size: change.text.join('').length,
+          size: insertedText.length,
           timeGap: timeDiff,
           severity: 'high'
         });

scripts/firepad.js

@@ -155,6 +155,39 @@
     editor.on('changeSelection', updateCursorPosition);
 
     console.log('Editor initialized - ReadOnly:', editor.getReadOnly());
+    
+    // Initialize behavior tracking if pending (for candidates)
+    if (window.pendingBehaviorTracking && window.initBehaviorTracking) {
+      const { sessionCode, name, type } = window.pendingBehaviorTracking;
+      console.log('Initializing delayed behavior tracking for:', name, type);
+      
+      // Set up ACE editor paste tracking
+      editor.on('paste', function(e) {
+        console.log('ACE Editor paste event detected');
+        const pasteEvent = new CustomEvent('editorPaste', { 
+          detail: { text: e.text || '' }
+        });
+        document.dispatchEvent(pasteEvent);
+      });
+      
+      // Track editor changes for typing patterns
+      editor.on('change', function(e) {
+        const changeEvent = new CustomEvent('editorChange', { 
+          detail: { 
+            action: e.action,
+            lines: e.lines,
+            text: e.lines ? e.lines.join('\n') : ''
+          }
+        });
+        document.dispatchEvent(changeEvent);
+      });
+      
+      // Initialize behavior tracking
+      window.initBehaviorTracking(sessionCode, name, type);
+      
+      // Clear pending flag
+      window.pendingBehaviorTracking = null;
+    }
   }
 
   // Initialize Firebase and Firepad