dashcam 1.3.6-beta → 1.3.8-beta

package/bin/dashcam-background.js

@@ -12,10 +12,21 @@ import path from 'path';
 import os from 'os';
 
 // Get process directory for status files
-const PROCESS_DIR = path.join(os.homedir(), '.dashcam-cli');
+// Use a fixed system-wide directory for cross-process communication
+// On Windows: C:\ProgramData\dashcam-cli
+// On Unix: /tmp/dashcam-cli
+const PROCESS_DIR = process.platform === 'win32' 
+  ? path.join('C:', 'ProgramData', 'dashcam-cli')
+  : path.join('/tmp', '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 +44,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/lib/processManager.js

@@ -8,8 +8,12 @@ import { logger } from './logger.js';
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 
-// Use a fixed directory in the user's home directory for cross-process communication
-const PROCESS_DIR = path.join(os.homedir(), '.dashcam-cli');
+// Use a fixed system-wide directory for cross-process communication
+// On Windows: C:\ProgramData\dashcam-cli
+// On Unix: /tmp/dashcam-cli
+const PROCESS_DIR = process.platform === 'win32' 
+  ? path.join('C:', 'ProgramData', 'dashcam-cli')
+  : path.join('/tmp', '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');
@@ -111,16 +115,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 +160,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.6-beta",
+  "version": "1.3.8-beta",
   "description": "Minimal CLI version of Dashcam desktop app",
   "main": "bin/dashcam.js",
   "bin": {