better exit code handling -- fix error highlights to not cover SIGINT or SIGPIPE (#465)

* fix exitcode return to handle signals correctly -- use bash convention of 128 + signum.  don't show red error mask for SIGINT or SIGPIPE

* dont show left border for errors unless selected

Author: sawka

public/themes/default.css

@@ -156,7 +156,6 @@
     --line-text-color: rgb(211, 215, 207);
     --line-svg-fill-color: rgb(150, 152, 150);
     --line-svg-hover-fill-color: #eceeec;
-    --line-selected-border-color: rgb(193, 195, 193);
     --line-separator-color: rgb(126, 126, 126);
     --line-error-color: var(--app-error-color);
     --line-warning-color: var(--app-warning-color);

src/app/line/line.less

@@ -69,7 +69,6 @@
 
         &.error-mask {
             background-color: var(--line-error-bg-color);
-            border-left: 4px solid var(--line-error-border-left-color);
         }
     }
 

src/app/line/linecomps.tsx

@@ -53,8 +53,16 @@ let heightLog = {};
 
 dayjs.extend(localizedFormat);
 
-function cmdHasError(cmd: Cmd): boolean {
-    return cmd.getStatus() == "error" || cmd.getExitCode() != 0;
+function cmdShouldMarkError(cmd: Cmd): boolean {
+    if (cmd.getStatus() == "error") {
+        return true;
+    }
+    let exitCode = cmd.getExitCode();
+    // 0, SIGINT, or SIGPIPE
+    if (exitCode == 0 || exitCode == 130 || exitCode == 141) {
+        return false;
+    }
+    return true;
 }
 
 function getIsHidePrompt(line: LineType): boolean {
@@ -701,7 +709,7 @@ class LineCmd extends React.Component<
             )
             .get();
         const isRunning = cmd.isRunning();
-        const cmdError = cmdHasError(cmd);
+        const cmdError = cmdShouldMarkError(cmd);
         const mainDivCn = cn(
             "line",
             "line-cmd",

waveshell/pkg/packet/combined.go

@@ -1069,6 +1069,23 @@ func copyToCirFile(dest *cirfile.File, src io.Reader) error {
 	}
 }
 
+func GetCmdExitCode(cmd *exec.Cmd, err error) int {
+	if cmd == nil || cmd.ProcessState == nil {
+		return GetExitCode(err)
+	}
+	status, ok := cmd.ProcessState.Sys().(syscall.WaitStatus)
+	if !ok {
+		return cmd.ProcessState.ExitCode()
+	}
+	signaled := status.Signaled()
+	if signaled {
+		signal := status.Signal()
+		return 128 + int(signal)
+	}
+	exitStatus := status.ExitStatus()
+	return exitStatus
+}
+
 func GetExitCode(err error) int {
 	if err == nil {
 		return 0
@@ -1082,7 +1099,6 @@ func GetExitCode(err error) int {
 
 func (c *ShExecType) ProcWait() error {
 	exitErr := c.Cmd.Wait()
-	base.Logf("procwait: %v\n", exitErr)
 	c.Lock.Lock()
 	c.Exited = true
 	c.Lock.Unlock()
@@ -1095,6 +1111,7 @@ func (c *ShExecType) IsExited() bool {
 	return c.Exited
 }
 
+// called in waveshell --single mode (returns the real cmddone packet)
 func (c *ShExecType) WaitForCommand() *packet.CmdDonePacketType {
 	donePacket := packet.MakeCmdDonePacket(c.CK)
 	exitErr := c.ProcWait()
@@ -1116,7 +1133,7 @@ func (c *ShExecType) WaitForCommand() *packet.CmdDonePacketType {
 	endTs := time.Now()
 	cmdDuration := endTs.Sub(c.StartTs)
 	donePacket.Ts = endTs.UnixMilli()
-	donePacket.ExitCode = GetExitCode(exitErr)
+	donePacket.ExitCode = GetCmdExitCode(c.Cmd, exitErr)
 	donePacket.DurationMs = int64(cmdDuration / time.Millisecond)
 	if c.FileNames != nil {
 		os.Remove(c.FileNames.StdinFifo) // best effort (no need to check error)