dashcam 1.0.1-beta.21 → 1.0.1-beta.23

package/bin/dashcam.js

@@ -417,30 +417,29 @@ program
 
         console.log('Recording stopped successfully');
         
+        // Wait a moment for upload to complete (background process handles this)
+        logger.debug('Waiting for background upload to complete...');
+        await new Promise(resolve => setTimeout(resolve, 5000));
+        
+        // Try to read the upload result from the background process
+        const uploadResult = processManager.readUploadResult();
+        logger.debug('Upload result read attempt', { found: !!uploadResult, shareLink: uploadResult?.shareLink });
+        
+        if (uploadResult && uploadResult.shareLink) {
+          console.log('📹 Watch your recording:', uploadResult.shareLink);
+          // Clean up the result file now that we've read it
+          processManager.cleanup();
+          process.exit(0);
+        }
+        
         // Check if files still exist - if not, background process already uploaded
         const filesExist = fs.existsSync(result.outputPath) && 
                           (!result.gifPath || fs.existsSync(result.gifPath)) && 
                           (!result.snapshotPath || fs.existsSync(result.snapshotPath));
         
         if (!filesExist) {
-          // Files were deleted, meaning background process uploaded
-          // Wait for the upload result to be written
-          logger.debug('Waiting for upload result from background process');
-          await new Promise(resolve => setTimeout(resolve, 2000));
-          
-          // Try to read the upload result from the background process
-          const uploadResult = processManager.readUploadResult();
-          logger.debug('Upload result read attempt', { found: !!uploadResult, shareLink: uploadResult?.shareLink });
-          
-          if (uploadResult && uploadResult.shareLink) {
-            console.log('📹 Watch your recording:', uploadResult.shareLink);
-            // Clean up the result file now that we've read it
-            processManager.cleanup();
-          } else {
-            console.log('✅ Recording uploaded (share link not available)');
-            logger.warn('Upload result not available from background process');
-          }
-          
+          console.log('✅ Recording uploaded by background process');
+          logger.info('Files were cleaned up by background process');
           process.exit(0);
         }
         

package/lib/ffmpeg.js

@@ -18,7 +18,7 @@ export async function createSnapshot(inputVideoPath, outputSnapshotPath, snapsho
     '-i', inputVideoPath,
     '-frames:v', '1',
     '-vf', 'scale=640:-1:force_original_aspect_ratio=decrease:eval=frame',
-    '-compression_level', '6', // Use default compression (was 100, which is extremely slow)
+    '-compression_level', '0', // Fast compression for speed (0 = fastest, 9 = slowest)
     outputSnapshotPath,
     '-y',
     '-hide_banner'
@@ -94,62 +94,32 @@ export async function createGif(inputVideoPath, outputGifPath) {
       stdout
     });
     
-    // Fallback: Just sample the video at a fixed rate (e.g., 1 frame every 3 seconds)
-    const framesPath = path.join(os.tmpdir(), `frames_${id}_%04d.png`);
+    // Fallback: Single-pass GIF creation at fixed rate
+    const filters = `fps=1/3,scale=640:-1:flags=lanczos,split[s0][s1];[s0]palettegen=max_colors=128[p];[s1][p]paletteuse=dither=bayer:bayer_scale=5`;
+    
     await execa(ffmpegPath, [
       '-i', inputVideoPath,
-      '-vf', `fps=1/3`, // Sample 1 frame every 3 seconds
-      framesPath
-    ]);
-
-    // Create GIF from frames
-    await execa(ffmpegPath, [
-      '-framerate', `${gifFps}`,
-      '-i', framesPath,
+      '-vf', filters,
       '-loop', '0',
       outputGifPath,
       '-y',
       '-hide_banner'
     ]);
-
-    // Clean up temporary frame files
-    const framesToDelete = fs.readdirSync(os.tmpdir())
-      .filter(file => file.startsWith(`frames_${id}_`) && file.endsWith('.png'))
-      .map(file => path.join(os.tmpdir(), file));
-    
-    for (const frame of framesToDelete) {
-      fs.unlinkSync(frame);
-    }
     
     return;
   }
   
   const extractedFramesInterval = videoDuration / gifFrames;
 
-  // Extract frames
-  const framesPath = path.join(os.tmpdir(), `frames_${id}_%04d.png`);
+  // Single-pass GIF creation with palette for better quality and performance
+  const filters = `fps=1/${extractedFramesInterval},scale=640:-1:flags=lanczos,split[s0][s1];[s0]palettegen=max_colors=128[p];[s1][p]paletteuse=dither=bayer:bayer_scale=5`;
+  
   await execa(ffmpegPath, [
     '-i', inputVideoPath,
-    '-vf', `fps=1/${extractedFramesInterval}`,
-    framesPath
-  ]);
-
-  // Create GIF from frames
-  await execa(ffmpegPath, [
-    '-framerate', `${gifFps}`,
-    '-i', framesPath,
+    '-vf', filters,
     '-loop', '0',
     outputGifPath,
     '-y',
     '-hide_banner'
   ]);
-
-  // Clean up temporary frame files
-  const framesToDelete = fs.readdirSync(os.tmpdir())
-    .filter(file => file.startsWith(`frames_${id}_`) && file.endsWith('.png'))
-    .map(file => path.join(os.tmpdir(), file));
-  
-  for (const frame of framesToDelete) {
-    fs.unlinkSync(frame);
-  }
 }

package/lib/processManager.js

@@ -174,41 +174,27 @@ class ProcessManager {
         await new Promise(resolve => setTimeout(resolve, 1000));
       }
       
-      // Call the actual recorder's stopRecording function to get complete results
+      // The background process handles stopRecording() internally via SIGINT
+      // We just need to return the basic result from the status file
       if (status) {
-        try {
-          const { stopRecording } = await import('./recorder.js');
-          const result = await stopRecording();
-          
-          logger.info('Recording stopped successfully via recorder', { 
-            outputPath: result.outputPath,
-            duration: result.duration,
-            hasLogs: result.logs?.length > 0,
-            hasApps: result.apps?.length > 0
-          });
-          
-          // Cleanup process files but preserve upload result for stop command
-          this.cleanup({ preserveResult: true });
-          
-          return result;
-        } catch (recorderError) {
-          logger.warn('Failed to stop via recorder, falling back to basic result', { error: recorderError.message });
-          
-          // Fallback to basic result if recorder fails
-          const basePath = status.outputPath.substring(0, status.outputPath.lastIndexOf('.'));
-          const result = {
-            outputPath: status.outputPath,
-            gifPath: `${basePath}.gif`,
-            snapshotPath: `${basePath}.png`,
-            duration: Date.now() - status.startTime,
-            clientStartDate: status.startTime,
-            apps: [],
-            logs: []
-          };
-          
-          this.cleanup({ preserveResult: true });
-          return result;
-        }
+        logger.info('Background recording stopped, returning status', { 
+          outputPath: status.outputPath,
+          duration: Date.now() - status.startTime
+        });
+        
+        const basePath = status.outputPath.substring(0, status.outputPath.lastIndexOf('.'));
+        const result = {
+          outputPath: status.outputPath,
+          gifPath: `${basePath}.gif`,
+          snapshotPath: `${basePath}.png`,
+          duration: Date.now() - status.startTime,
+          clientStartDate: status.startTime,
+          apps: [],
+          logs: []
+        };
+        
+        this.cleanup({ preserveResult: true });
+        return result;
       } else {
         throw new Error('No status information available for active recording');
       }

package/lib/recorder.js

@@ -322,9 +322,9 @@ export async function startRecording({
   const platformArgs = await getPlatformArgs({ fps, includeAudio });
   const outputArgs = [
     '-c:v', 'libvpx-vp9',  // Use VP9 codec for better quality and compression
-    '-quality', 'good',     // Use 'good' quality preset (better than realtime, not as slow as best)
-    '-cpu-used', '4',       // Faster encoding (0-8, higher = faster but lower quality)
-    '-deadline', 'good',    // Good quality encoding mode
+    '-quality', 'realtime', // Use realtime quality preset for faster encoding
+    '-cpu-used', '8',       // Maximum speed (0-8, higher = faster but lower quality)
+    '-deadline', 'realtime',// Realtime encoding mode for lowest latency
     '-b:v', '2M',           // Target bitrate
     '-r', fps.toString(),   // Ensure output framerate matches input
     '-g', fps.toString(),   // Keyframe interval = 1 second (every fps frames) - ensures frequent keyframes
@@ -528,24 +528,23 @@ export async function stopRecording() {
       currentRecording.stdin.end(); // Close stdin to signal end
     }
 
-    // Wait longer for FFmpeg to finish gracefully - critical for VP9 encoding
-    // VP9 encoding needs time to flush buffers and finalize the container
+    // Wait for FFmpeg to finish gracefully with realtime encoding
     const gracefulTimeout = setTimeout(() => {
       if (currentRecording && !currentRecording.killed) {
-        logger.warn('FFmpeg did not exit gracefully after 8s, sending SIGTERM...');
+        logger.warn('FFmpeg did not exit gracefully after 10s, sending SIGTERM...');
         // If still running, try SIGTERM
         process.kill(currentRecording.pid, 'SIGTERM');
       }
-    }, 8000); // Increased to 8 seconds for VP9 finalization
+    }, 10000); // Wait longer for graceful shutdown
 
-    // Wait up to 15 seconds for SIGTERM to work
+    // Wait up to 20 seconds for SIGTERM to work
     const hardKillTimeout = setTimeout(() => {
       if (currentRecording && !currentRecording.killed) {
         logger.error('FFmpeg still running after SIGTERM, using SIGKILL...');
         // If still not dead, use SIGKILL as last resort
         process.kill(currentRecording.pid, 'SIGKILL');
       }
-    }, 15000); // Increased to 15 seconds
+    }, 20000); // Longer timeout to avoid killing prematurely
 
     // Wait for the process to fully exit
     if (currentRecording) {
@@ -556,10 +555,30 @@ export async function stopRecording() {
     clearTimeout(gracefulTimeout);
     clearTimeout(hardKillTimeout);
 
-    // Additional wait to ensure filesystem is synced and encoder buffers are flushed
-    // This is especially important for VP9 which has larger encoding buffers
-    logger.debug('Waiting for filesystem sync and VP9 encoder finalization...');
-    await new Promise(resolve => setTimeout(resolve, 3000)); // Keep at 3 seconds after process exit
+    // Wait for filesystem to sync and file to be written
+    // Poll for the file to exist with content
+    logger.debug('Waiting for temp file to be written...');
+    const maxWaitTime = 10000; // Wait up to 10 seconds
+    const startWait = Date.now();
+    let tempFileReady = false;
+    
+    while (!tempFileReady && (Date.now() - startWait) < maxWaitTime) {
+      const tempFile = currentTempFile;
+      if (tempFile && fs.existsSync(tempFile)) {
+        const stats = fs.statSync(tempFile);
+        if (stats.size > 0) {
+          logger.debug('Temp file is ready', { size: stats.size });
+          tempFileReady = true;
+          break;
+        }
+      }
+      // Wait a bit before checking again
+      await new Promise(resolve => setTimeout(resolve, 500));
+    }
+    
+    if (!tempFileReady) {
+      logger.warn('Temp file not ready after waiting, proceeding anyway');
+    }
 
     // Read temp file path from disk (for cross-process access)
     let tempFile = currentTempFile; // Try in-memory first
@@ -607,8 +626,7 @@ export async function stopRecording() {
       path: tempFile
     });
 
-    // Since WebM is already a valid streaming format, re-mux it to ensure
-    // proper container metadata (duration, seekability, etc.)
+    // Re-mux to ensure proper container metadata (duration, seekability, etc.)
     logger.debug('Re-muxing temp file to fix container metadata...');
     
     try {

package/lib/tracking/FileTracker.js

@@ -47,7 +47,7 @@ export class FileTracker {
         fs.writeFileSync(this.trackedFile, '', 'utf8');
       }
       
-      this.tail = new Tail(this.trackedFile, { encoding: 'ascii' });
+      this.tail = new Tail(this.trackedFile, { encoding: 'utf8' });
       this.tail.on('line', (line) => {
         const time = Date.now();
         this.eventTimes.push(time);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dashcam",
-  "version": "1.0.1-beta.21",
+  "version": "1.0.1-beta.23",
   "description": "Minimal CLI version of Dashcam desktop app",
   "main": "bin/index.js",
   "bin": {

package/test_workflow.sh

@@ -46,7 +46,7 @@ RECORDING_START=$(date +%s)
 echo "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
 echo "🔴 EVENT 1: Recording START at $(date '+%H:%M:%S')"
 echo "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
-echo "[EVENT 1] Recording started at $(date '+%H:%M:%S') - TIMESTAMP: $RECORDING_START" >> "$TEMP_FILE"
+echo "[EVENT 1] 🔴 Recording started with emoji at $(date '+%H:%M:%S') - TIMESTAMP: $RECORDING_START" >> "$TEMP_FILE"
 
 # Verify recording is actually running
 if ps -p $RECORD_PID > /dev/null; then
@@ -71,25 +71,25 @@ sleep 3
 # Event 2 - after 3 seconds
 echo ""
 echo "🟡 EVENT 2: 3 seconds mark at $(date '+%H:%M:%S')"
-echo "[EVENT 2] 3 seconds elapsed at $(date '+%H:%M:%S')" >> "$TEMP_FILE"
+echo "[EVENT 2] 🟡 3 seconds elapsed with emoji at $(date '+%H:%M:%S')" >> "$TEMP_FILE"
 sleep 3
 
 # Event 3 - after 6 seconds
 echo ""
 echo "🟢 EVENT 3: 6 seconds mark at $(date '+%H:%M:%S')"
-echo "[EVENT 3] 6 seconds elapsed at $(date '+%H:%M:%S')" >> "$TEMP_FILE"
+echo "[EVENT 3] 🟢 6 seconds elapsed with emoji at $(date '+%H:%M:%S')" >> "$TEMP_FILE"
 sleep 3
 
 # Event 4 - after 9 seconds
 echo ""
 echo "🔵 EVENT 4: 9 seconds mark at $(date '+%H:%M:%S')"
-echo "[EVENT 4] 9 seconds elapsed at $(date '+%H:%M:%S')" >> "$TEMP_FILE"
+echo "[EVENT 4] 🔵 9 seconds elapsed with emoji at $(date '+%H:%M:%S')" >> "$TEMP_FILE"
 sleep 3
 
 # Event 5 - after 12 seconds
 echo ""
 echo "🟣 EVENT 5: 12 seconds mark at $(date '+%H:%M:%S')"
-echo "[EVENT 5] 12 seconds elapsed at $(date '+%H:%M:%S')" >> "$TEMP_FILE"
+echo "[EVENT 5] 🟣 12 seconds elapsed with emoji at $(date '+%H:%M:%S')" >> "$TEMP_FILE"
 sleep 3
 
 # Event 6 - before ending
@@ -98,7 +98,7 @@ echo "━━━━━━━━━━━━━━━━━━━━━━━━
 echo "⚫ EVENT 6: Recording END at $(date '+%H:%M:%S')"
 echo "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
 RECORDING_END=$(date +%s)
-echo "[EVENT 6] Recording ending at $(date '+%H:%M:%S') - TIMESTAMP: $RECORDING_END" >> "$TEMP_FILE"
+echo "[EVENT 6] ⚫ Recording ending with emoji at $(date '+%H:%M:%S') - TIMESTAMP: $RECORDING_END" >> "$TEMP_FILE"
 
 DURATION=$((RECORDING_END - RECORDING_START))
 echo ""
@@ -112,8 +112,13 @@ sleep 2
 # 6. Stop recording and upload (this will kill the background recording process)
 echo ""
 echo "6. Stopping recording and uploading..."
-./bin/dashcam.js stop
-echo "✅ Recording stopped and uploaded"
+# Check if recording is still active
+if ./bin/dashcam.js status | grep -q "Recording in progress"; then
+  ./bin/dashcam.js stop
+  echo "✅ Recording stopped and uploaded"
+else
+  echo "⚠️  Recording already completed (this is expected with background mode)"
+fi
 
 echo ""
 echo "🧹 Cleaning up..."
@@ -140,12 +145,12 @@ echo "3. Verify these events appear at the correct times:"
 echo ""
 echo "   Time   | Terminal Display          | Log Entry"
 echo "   -------|---------------------------|---------------------------"
-echo "   0:00   | 🔴 EVENT 1               | [EVENT 1] Recording started"
-echo "   0:03   | 🟡 EVENT 2               | [EVENT 2] 3 seconds elapsed"
-echo "   0:06   | 🟢 EVENT 3               | [EVENT 3] 6 seconds elapsed"
-echo "   0:09   | 🔵 EVENT 4               | [EVENT 4] 9 seconds elapsed"
-echo "   0:12   | 🟣 EVENT 5               | [EVENT 5] 12 seconds elapsed"
-echo "   0:15   | ⚫ EVENT 6               | [EVENT 6] Recording ending"
+echo "   0:00   | 🔴 EVENT 1               | [EVENT 1] 🔴 Recording started"
+echo "   0:03   | 🟡 EVENT 2               | [EVENT 2] 🟡 3 seconds elapsed"
+echo "   0:06   | 🟢 EVENT 3               | [EVENT 3] 🟢 6 seconds elapsed"
+echo "   0:09   | 🔵 EVENT 4               | [EVENT 4] 🔵 9 seconds elapsed"
+echo "   0:12   | 🟣 EVENT 5               | [EVENT 5] 🟣 12 seconds elapsed"
+echo "   0:15   | ⚫ EVENT 6               | [EVENT 6] ⚫ Recording ending"
 echo ""
 echo "4. The log timestamps should match the video timeline exactly"
 echo "5. Each colored event marker should appear in the video"