dashcam 1.0.1-beta.32 → 1.0.1-beta.33

package/bin/dashcam.js

@@ -136,10 +136,7 @@ async function recordingAction(options, command) {
       log('Use "dashcam status" to check progress');
       log('Use "dashcam stop" to stop recording and upload');
       
-      // Close stdout/stderr and exit immediately to prevent blocking
-      // when called from automation scripts with piped output
-      process.stdout.end();
-      process.stderr.end();
+      // Process can exit now - background process is detached
       process.exit(0);
       
     } catch (error) {

package/lib/processManager.js

@@ -159,11 +159,19 @@ class ProcessManager {
       const isWindows = process.platform === 'win32';
       
       if (isWindows) {
-        logger.info('Windows detected, using taskkill to stop process');
+        logger.info('Windows detected, attempting graceful shutdown first');
         try {
-          // Use taskkill to forcefully stop the process tree on Windows
+          // First try graceful shutdown without /F (force) flag
+          // This sends CTRL+C which Node.js can handle
           const { execSync } = await import('child_process');
-          execSync(`taskkill /PID ${pid} /F /T`, { stdio: 'ignore' });
+          try {
+            execSync(`taskkill /PID ${pid} /T`, { stdio: 'ignore', timeout: 5000 });
+            logger.info('Sent graceful shutdown signal to process');
+          } catch (error) {
+            // If graceful shutdown fails or times out, force kill
+            logger.warn('Graceful shutdown failed, forcing termination');
+            execSync(`taskkill /PID ${pid} /F /T`, { stdio: 'ignore' });
+          }
         } catch (error) {
           logger.warn('Failed to kill process with taskkill', { error: error.message });
         }
@@ -238,20 +246,11 @@ class ProcessManager {
     const __dirname = path.dirname(__filename);
     const backgroundScript = path.join(__dirname, '..', 'bin', 'dashcam-background.js');
     
-    // On Windows, we need to ensure stdio is completely detached
-    const isWindows = process.platform === 'win32';
-    const spawnOptions = {
+    // Spawn a detached background process
+    const child = spawn(process.execPath, [backgroundScript, JSON.stringify(options)], {
       detached: true,
       stdio: 'ignore'
-    };
-    
-    // On Windows, also create a new process group to fully detach
-    if (isWindows) {
-      spawnOptions.windowsHide = true;
-    }
-    
-    // Spawn a detached background process
-    const child = spawn(process.execPath, [backgroundScript, JSON.stringify(options)], spawnOptions);
+    });
     
     // Unref so the parent process can exit
     child.unref();
@@ -263,8 +262,7 @@ class ProcessManager {
 
     logger.info('Background process spawned successfully', { 
       pid,
-      backgroundScript,
-      platform: process.platform
+      backgroundScript
     });
     
     // Wait a moment for the background process to write its status

package/package.json

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