dashcam 0.4.0 → 0.4.1

package/index.js

@@ -70,7 +70,7 @@ program
       dashcam.onConnected = () => dashcam.emit("track-cli", logFile);
       fs.appendFileSync(logFile, "");
       const recorder = new Recorder(logFile);
-      recorder.start();
+      await recorder.start();
     } catch (e) {
       console.log("Error: ", e);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dashcam",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "description": "Fix bugs, close pulls, and update your team with desktop instant replay.",
   "main": "index.js",
   "scripts": {
@@ -15,6 +15,7 @@
   "dependencies": {
     "cli-color": "^2.0.3",
     "commander": "^9.4.0",
+    "find-process": "^1.4.7",
     "husky": "^7.0.4",
     "node-ipc": "^10.1.0",
     "node-pty-prebuilt-multiarch": "^0.10.1-pre.5",

package/recorder.js

@@ -1,5 +1,7 @@
 const fs = require("fs");
 const pty = require("node-pty-prebuilt-multiarch");
+const process = require("process");
+const find = require("find-process");
 
 class Recorder {
   #ptyProcess = null;
@@ -24,7 +26,7 @@ class Recorder {
     fs.appendFileSync(this.#logFile, data, "ascii");
   }
 
-  start() {
+  async start() {
     console.log(
       "This session is being recorded by Dashcam and dumped to",
       this.#logFile
@@ -34,7 +36,15 @@ class Recorder {
     // TODO: Find a way to consistently get the current shell this is running from
     // instead of using the default user shell (Maybe use parent processId to find
     // the process filepath)
-    const shell = process.env.SHELL;
+
+    const shell = (
+      process.env.SHELL ||
+      (await find("pid", process.ppid).then((arr) => arr[0].bin)) ||
+      ""
+    ).trim();
+
+    if (!shell) throw new Error("Could not detect the current shell");
+
     const args = [];
     if (!shell.toLowerCase().includes("powershell")) args.push("-l");
     this.#ptyProcess = pty.spawn(shell, args, {