dashcam 1.0.2-beta.1 → 1.0.2-beta.2

package/lib/processManager.js

@@ -226,8 +226,15 @@ class ProcessManager {
     const stdoutLog = path.join(logDir, 'background-stdout.log');
     const stderrLog = path.join(logDir, 'background-stderr.log');
     
-    const stdoutFd = fs.openSync(stdoutLog, 'a');
-    const stderrFd = fs.openSync(stderrLog, 'a');
+    // On Windows, we need to use 'ignore' for all stdio to allow the parent to exit
+    // On Unix, we can redirect to log files
+    const isWindows = process.platform === 'win32';
+    let stdoutFd, stderrFd;
+    
+    if (!isWindows) {
+      stdoutFd = fs.openSync(stdoutLog, 'a');
+      stderrFd = fs.openSync(stderrLog, 'a');
+    }
     
     // Spawn a detached process that will handle the recording
     const backgroundProcess = spawn(process.execPath, [
@@ -235,20 +242,27 @@ class ProcessManager {
       JSON.stringify(options)
     ], {
       detached: true,
-      stdio: ['ignore', stdoutFd, stderrFd], // Log stdout and stderr
+      stdio: isWindows ? 'ignore' : ['ignore', stdoutFd, stderrFd],
+      windowsHide: true, // Hide the console window on Windows
       env: {
         ...process.env,
         DASHCAM_BACKGROUND: 'true'
       }
     });
     
-    // Close the file descriptors in the parent process
-    fs.closeSync(stdoutFd);
-    fs.closeSync(stderrFd);
+    // Close the file descriptors in the parent process (Unix only)
+    if (!isWindows) {
+      fs.closeSync(stdoutFd);
+      fs.closeSync(stderrFd);
+    }
 
     // Get the background process PID before unreffing
     const backgroundPid = backgroundProcess.pid;
     
+    // Write PID file immediately so other commands can find the background process
+    // Use the spawned process PID rather than waiting for status file
+    this.writePid(backgroundPid);
+    
     // Allow the parent process to exit independently
     backgroundProcess.unref();
 
@@ -259,12 +273,11 @@ class ProcessManager {
     const status = this.readStatus();
     
     if (!status || !status.isRecording) {
+      // Clean up PID file if recording failed to start
+      this.cleanup();
       throw new Error('Background process failed to start recording');
     }
 
-    // Write PID file so other commands can find the background process
-    this.writePid(status.pid);
-
     logger.info('Background recording process started', { 
       pid: status.pid,
       outputPath: status.outputPath

package/package.json

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