fix: environment variable loss for running processes during refresh

src/process.cpp

@@ -154,6 +154,8 @@
     _app_prep_it = _app_prep_begin;
 
     // Add Stream-specific environment variables
+    // These variables are dynamically set for each streaming session and will be passed
+    // to the launched application. They should be preserved during refresh() if an app is running.
     _env["SUNSHINE_APP_ID"] = std::to_string(_app_id);
     _env["SUNSHINE_APP_NAME"] = _app.name;
     _env["SUNSHINE_CLIENT_WIDTH"] = std::to_string(launch_session->width);
@@ -751,8 +753,30 @@
   void refresh(const std::string &file_name) {
     auto proc_opt = proc::parse(file_name);
 
-    if (proc_opt) {
-      proc = std::move(*proc_opt);
+    if (!proc_opt) {
+      return;
+    }
+    // If it is running, the environment variable (SUNSHINE_*) should be kept.
+    // These variables are dynamically added in execute() and should not be 
+    // overridden by environment variables in the configuration file
+    if (proc.running()) {
+      const boost::process::v1::environment &current_env = proc.get_env();
+      boost::process::v1::environment new_env = proc_opt->get_env();
+
+      // Copy the 'SUNSHINE_*' from the current environment into the new environment.
+      for (const auto &entry : current_env) {
+        const std::string &var_name = entry.get_name();
+        if (var_name.find("SUNSHINE_") == 0) {
+          new_env[var_name] = entry.to_string();
+        }
+      }
+
+      proc.set_env(std::move(new_env));
+    }
+    else {
+      // Not a running state, which can safely replace environment variables
+      proc.set_env(proc_opt->get_env());
     }
+    proc.set_apps(proc_opt->get_apps());
   }
 }  // namespace proc