cli(run): avoid duplicate runtime error message

Propagate negative exit code from script runner as 'already handled' to suppress generic error output, while preserving real exit status.

Author: GaspardKirira

include/vix/cli/commands/run/RunDetail.hpp

@@ -76,6 +76,12 @@ namespace vix::commands::RunCommand::detail
     std::string badDoubleDashArg;
   };
 
+  struct ScriptRunResult
+  {
+    int code = 0;
+    bool handled = false;
+  };
+
   // Process / IO
   int run_cmd_live_filtered(
       const std::string &cmd,
@@ -91,6 +97,9 @@ namespace vix::commands::RunCommand::detail
     if (code < 0)
       return 1;
 
+    if (code <= 255)
+      return code;
+
     if (WIFEXITED(code))
       return WEXITSTATUS(code);
 

src/commands/RunCommand.cpp

@@ -301,7 +301,12 @@ namespace vix::commands::RunCommand
 
     if (opt.singleCpp)
     {
-      return detail::run_single_cpp(opt);
+      int rc = detail::run_single_cpp(opt);
+
+      if (rc < 0)
+        return -rc;
+
+      return rc;
     }
 
     // 2) Mode projet (apps)

src/commands/run/RunProcess.cpp

@@ -559,7 +559,9 @@ namespace vix::commands::RunCommand::detail
 
     if (::pipe(outPipe) != 0)
     {
-      result.exitCode = std::system(cmd.c_str());
+      const int st = std::system(cmd.c_str());
+      result.rawStatus = st;
+      result.exitCode = normalize_exit_code(st);
       return result;
     }
 
@@ -569,7 +571,9 @@ namespace vix::commands::RunCommand::detail
       close_safe(outPipe[0]);
       close_safe(outPipe[1]);
 
-      result.exitCode = std::system(cmd.c_str());
+      const int st = std::system(cmd.c_str());
+      result.rawStatus = st;
+      result.exitCode = normalize_exit_code(st);
       return result;
     }
 

src/commands/run/RunScript.cpp

@@ -507,7 +507,13 @@ namespace vix::commands::RunCommand::detail
         }
       }
 
-      handle_runtime_exit_code(runCode, "Script execution failed", /*alreadyHandled=*/handled);
+      const bool already = handled || rr.printed_live;
+
+      handle_runtime_exit_code(runCode, "Script execution failed", /*alreadyHandled=*/already);
+
+      if (already && runCode > 0 && runCode != 130)
+        return -runCode;
+
       return runCode;
     }