From e1e6a384c01ef8a39aeedc619aa6fcbd7da9ec86 Mon Sep 17 00:00:00 2001
From: Marcello Lamonaca <marcello@lamonaca.eu>
Date: Fri, 25 Oct 2024 00:15:35 +0200
Subject: [PATCH] use appropriate elevation verb bases on OS

---
 src/ScriptExecutor.cs | 73 +++++++++++++++++++++++++------------------
 1 file changed, 43 insertions(+), 30 deletions(-)

diff --git a/src/ScriptExecutor.cs b/src/ScriptExecutor.cs
index 77e1fe0..8c1cb4b 100644
--- a/src/ScriptExecutor.cs
+++ b/src/ScriptExecutor.cs
@@ -26,36 +26,49 @@ internal static class ScriptExecutor
         ).ConfigureAwait(ConfigureAwaitOptions.None);
     }
 
-    private static ProcessStartInfo? GetExecutableProcessInfo(FileInfo file, bool elevated) =>
-        file.Extension switch
+    private static string GetElevationVerb(bool elevated)
+    {
+        if (!elevated)
         {
-            ".bat"
-            or ".cmd"
-                => new ProcessStartInfo
-                {
-                    FileName = "cmd",
-                    Arguments = $"/Q /C .\\{file.Name}",
-                    Verb = elevated ? "runas /user:Administrator" : string.Empty,
-                    WorkingDirectory = file.DirectoryName
-                },
-            ".ps1"
-                => new ProcessStartInfo
-                {
-                    FileName = "powershell.exe",
-                    Arguments = $"-NoProfile -ExecutionPolicy Bypass -File .\\{file.Name}",
-                    Verb = elevated ? "runas /user:Administrator" : string.Empty,
-                    WorkingDirectory = file.DirectoryName
-                },
-            ".sh"
-            or ".zsh"
-            or ".fish"
-                => new ProcessStartInfo
-                {
-                    FileName = "sh",
-                    Arguments = $"-c ./{file.Name}",
-                    Verb = elevated ? "sudo" : string.Empty,
-                    WorkingDirectory = file.DirectoryName
-                },
-            _ => null
+            return string.Empty;
+        }
+
+        var platform = Environment.OSVersion.Platform;
+        return platform switch
+        {
+            PlatformID.Win32NT => "runas /user:Administrator",
+            PlatformID.Unix or PlatformID.MacOSX => "sudo",
+            _ => string.Empty,
         };
+    }
+
+    private static ProcessStartInfo? GetExecutableProcessInfo(FileInfo file, bool elevated)
+    {
+        var verb = GetElevationVerb(elevated);
+        return file.Extension switch
+        {
+            ".bat" or ".cmd" => new ProcessStartInfo
+            {
+                FileName = "cmd",
+                Arguments = $"/Q /C {file.Name}",
+                Verb = verb,
+                WorkingDirectory = file.DirectoryName,
+            },
+            ".ps1" => new ProcessStartInfo
+            {
+                FileName = "powershell.exe",
+                Arguments = $"-NoProfile -ExecutionPolicy Bypass -File {file.Name}",
+                Verb = verb,
+                WorkingDirectory = file.DirectoryName,
+            },
+            ".sh" or ".zsh" or ".fish" => new ProcessStartInfo
+            {
+                FileName = "sh",
+                Arguments = $"-c {file.Name}",
+                Verb = verb,
+                WorkingDirectory = file.DirectoryName,
+            },
+            _ => null,
+        };
+    }
 }