dashcam 1.3.5-beta → 1.3.7-beta

package/bin/dashcam-background.js

@@ -13,9 +13,15 @@ import os from 'os';
 
 // Get process directory for status files
 const PROCESS_DIR = path.join(os.homedir(), '.dashcam-cli');
+const PID_FILE = path.join(PROCESS_DIR, 'recording.pid');
 const STATUS_FILE = path.join(PROCESS_DIR, 'status.json');
 const RESULT_FILE = path.join(PROCESS_DIR, 'upload-result.json');
 
+// Ensure process directory exists
+if (!fs.existsSync(PROCESS_DIR)) {
+  fs.mkdirSync(PROCESS_DIR, { recursive: true });
+}
+
 // Parse options from command line argument
 const optionsJson = process.argv[2];
 if (!optionsJson) {
@@ -33,6 +39,14 @@ logger.info('Background recording process started', {
   options 
 });
 
+// Write PID file immediately
+try {
+  fs.writeFileSync(PID_FILE, process.pid.toString());
+  logger.info('PID file written', { path: PID_FILE, pid: process.pid });
+} catch (error) {
+  logger.error('Failed to write PID file', { error });
+}
+
 // Write status file
 function writeStatus(status) {
   try {

package/bin/dashcam.js

@@ -109,7 +109,7 @@ async function recordingAction(options, command) {
       }, 2000))
     ]);
     if (!hasPermissions) {
-      log('\n⚠️  Cannot start recording without screen recording permission.');
+      log('\nCannot start recording without screen recording permission.');
       process.exit(1);
     }
 
@@ -134,7 +134,7 @@ async function recordingAction(options, command) {
       
       const result = await Promise.race([startRecordingPromise, timeoutPromise]);
 
-      log(`✅ Recording started successfully (PID: ${result.pid})`);
+      log(`Recording started successfully (PID: ${result.pid})`);
       log(`Output: ${result.outputPath}`);
       log('');
       log('Use "dashcam status" to check progress');
@@ -417,7 +417,7 @@ program
         
         // Wait for upload to complete (background process handles this)
         logger.debug('Waiting for background upload to complete...');
-        console.log('⏳ Uploading recording...');
+        console.log('Uploading recording...');
         
         // Wait up to 2 minutes for upload result to appear
         const maxWaitForUpload = 120000; // 2 minutes
@@ -434,7 +434,7 @@ program
         logger.debug('Upload result read attempt', { found: !!uploadResult, shareLink: uploadResult?.shareLink });
         
         if (uploadResult && uploadResult.shareLink) {
-          console.log('📹 Watch your recording:', 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);
@@ -446,7 +446,7 @@ program
                           (!result.snapshotPath || fs.existsSync(result.snapshotPath));
         
         if (!filesExist) {
-          console.log('✅ Recording uploaded by background process');
+          console.log('Recording uploaded by background process');
           logger.info('Files were cleaned up by background process');
           process.exit(0);
         }
@@ -466,7 +466,7 @@ program
             snapshotPath: result.snapshotPath
           });
           
-          console.log('📹 Watch your recording:', uploadResult.shareLink);
+          console.log('Watch your recording:', uploadResult.shareLink);
         } catch (uploadError) {
           console.error('Upload failed:', uploadError.message);
           console.log('Recording saved locally:', result.outputPath);
@@ -681,7 +681,7 @@ program
         project: options.project
       });
       
-      console.log('✅ Upload complete! Share link:', uploadResult.shareLink);
+      console.log('> Upload complete! Share link:', uploadResult.shareLink);
       process.exit(0);
       
     } catch (error) {

package/lib/processManager.js

@@ -111,16 +111,23 @@ class ProcessManager {
   }
 
   isRecordingActive() {
-    const pid = this.readPid();
     const status = this.readStatus();
     
-    if (!pid || !this.isProcessRunning(pid)) {
+    if (!status || !status.pid) {
       // Clean up but preserve upload result in case the background process just finished uploading
       this.cleanup({ preserveResult: true });
       return false;
     }
     
-    return status && status.isRecording;
+    const pid = status.pid;
+    
+    if (!this.isProcessRunning(pid)) {
+      // Clean up but preserve upload result in case the background process just finished uploading
+      this.cleanup({ preserveResult: true });
+      return false;
+    }
+    
+    return status.isRecording;
   }
 
   getActiveStatus() {
@@ -149,13 +156,14 @@ class ProcessManager {
     
     try {
       const status = this.readStatus();
-      const pid = this.readPid();
       
       if (!status || !status.isRecording) {
         logger.warn('No active recording found');
         return false;
       }
       
+      const pid = status.pid;
+      
       if (!pid || !this.isProcessRunning(pid)) {
         logger.warn('Background process not running');
         this.cleanup({ preserveResult: true });

package/package.json

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