make lambdas static where possible

Bumps [Spectre.Console.Cli](https://github.com/spectreconsole/spectre.console) from 0.48.0 to 0.49.1.
- [Release notes](https://github.com/spectreconsole/spectre.console/releases)
- [Commits](https://github.com/spectreconsole/spectre.console/compare/0.48.0...0.49.1)

---
updated-dependencies:
- dependency-name: Spectre.Console.Cli
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

.github/workflows/codeql.yml

@@ -25,16 +25,16 @@ jobs:
       uses: actions/checkout@v4
 
     - name: Initialize CodeQL
-      uses: github/codeql-action/init@v2
+      uses: github/codeql-action/init@v3
       with:
         languages: ${{ matrix.language }}
         # Details on CodeQL's query packs refer to : https://docs.github.com/en/code-security/code-scanning/automatically-scanning-your-code-for-vulnerabilities-and-errors/configuring-code-scanning#using-queries-in-ql-packs
         # queries: security-extended,security-and-quality
 
     - name: Autobuild
-      uses: github/codeql-action/autobuild@v2
+      uses: github/codeql-action/autobuild@v3
 
     - name: Perform CodeQL Analysis
-      uses: github/codeql-action/analyze@v2
+      uses: github/codeql-action/analyze@v3
       with:
         category: "/language:${{matrix.language}}"

Directory.Build.props

@@ -0,0 +1,5 @@
+<Project>
+  <PropertyGroup>
+    <ArtifactsPath>$(MSBuildThisFileDirectory)artifacts</ArtifactsPath>
+  </PropertyGroup>
+</Project>

README.md

@@ -33,18 +33,19 @@ _**NOTE**_: The option `-p:PublishTrimmed=true` may produce some *warnings*. If
 
 ```sh
 USAGE:
-    scrl <path> [OPTIONS]
+    scrl [path] [OPTIONS]
 
 ARGUMENTS:
-    <path>    Starting directory (Default: .)
+    [path]    Starting directory (Default: .)
 
 OPTIONS:
-    -h, --help          Prints help information
-    -x, --extensions    Comma separated list of script extensions
-    -d, --depth         Search depth
-    -e, --elevated      Run with elevated privileges
-    -g, --group         Group scripts by folder
-    -b, --brief         Show brief information
+                                     DEFAULT
+    -h, --help                                  Prints help information
+    -x, --extensions <EXTENSIONS>               List of script extensions to search for
+    -d, --depth                      3          Search depth
+    -e, --elevated                              Run with elevated privileges
+    -g, --group                                 Group scripts by folder
+    -b, --brief                                 Show brief information
 ```
 
 [CLI]: https://docs.microsoft.com/en-us/dotnet/core/tools/ ".NET CLI Docs"

src/Commands.cs

@@ -1,5 +1,5 @@
 using System.ComponentModel;
-
+using System.Diagnostics.CodeAnalysis;
 using Spectre.Console;
 using Spectre.Console.Cli;
 
@@ -76,27 +76,30 @@ public sealed class RootCommand : AsyncCommand<RootCommandSettings>
 
 public sealed class RootCommandSettings : CommandSettings
 {
-    [Description("Comma separated list of script extensions")]
-    [CommandOption("-x|--extensions")]
-    public string? Extensions { get; init; }
+    [Description("List of script extensions to search for")]
+    [CommandOption("-x|--extensions <EXTENSIONS>")]
+    [SuppressMessage("Performance", "CA1819:Properties should not return arrays")]
+    public string[] Extensions { get; init; } = [".ps1", ".*sh", ".bat", ".cmd", ".nu"];
 
     [Description("Search depth")]
     [CommandOption("-d|--depth")]
-    public int Depth { get; init; } = 1;
+    [DefaultValue(3)]
+    public int Depth { get; init; }
 
     [Description("Run with elevated privileges")]
     [CommandOption("-e|--elevated")]
-    public bool Elevated { get; init; } = false;
+    public bool Elevated { get; init; }
 
     [Description("Group scripts by folder")]
     [CommandOption("-g|--group")]
-    public bool Group { get; init; } = false;
+    public bool Group { get; init; }
 
     [Description("Show brief information")]
     [CommandOption("-b|--brief")]
-    public bool Brief { get; init; } = false;
+    public bool Brief { get; init; }
 
     [Description("Starting directory (Default: .)")]
-    [CommandArgument(0, "<path>")]
-    public string Directory { get; init; } = ".";
-}
+    [CommandArgument(0, "[path]")]
+    [DefaultValue(".")]
+    public string Directory { get; init; } = string.Empty;
+}

src/Program.cs

@@ -2,5 +2,6 @@ using ScriptLauncher;
 using Spectre.Console.Cli;
 
 var app = new CommandApp<RootCommand>();
-app.Configure(x => x.SetApplicationName("scrl"));
+app.Configure(static x => x.SetApplicationName("scrl"));
+
 return app.Run(args);

src/ScriptExecutor.cs

@@ -26,36 +26,55 @@ 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,
+            },
+            ".nu" => new ProcessStartInfo
+            {
+                FileName = "nu",
+                Arguments = $"--no-config-file ./{file.Name}",
+                Verb = verb,
+                WorkingDirectory = file.DirectoryName,
+            },
+            _ => new ProcessStartInfo
+            {
+                FileName = "sh",
+                Arguments = $"-c ./{file.Name}",
+                Verb = verb,
+                WorkingDirectory = file.DirectoryName,
+            },
+        };
+    }
 }

src/ScriptFinder.cs

@@ -2,23 +2,15 @@ namespace ScriptLauncher;
 
 internal readonly struct ScriptFinder
 {
-    private static readonly string[] DefaultExtensions = new[] { ".ps1", ".*sh", ".bat", ".cmd" };
-    private static readonly char[] DefaultSeparators = new[] { ',', ' ' };
-
-    public string[] Extensions { get; }
+    public IEnumerable<string> Extensions { get; }
     public string RootDirectory { get; }
-    public int Depth { get; }
+    private int Depth { get; }
 
     private readonly EnumerationOptions _options;
 
-    public ScriptFinder(string? extensions, string directory, int depth)
+    public ScriptFinder(IEnumerable<string> extensions, string directory, int depth)
     {
-        Extensions =
-            extensions
-                ?.Split(DefaultSeparators, StringSplitOptions.RemoveEmptyEntries)
-                .ToHashSet()
-                .Select(x => $".{x.TrimStart('.')}")
-                .ToArray() ?? DefaultExtensions;
+        Extensions = extensions.ToHashSet().Select(static x => $".{x.TrimStart('.')}");
 
         Depth = depth;
         RootDirectory = directory;
@@ -36,7 +28,7 @@ internal readonly struct ScriptFinder
         try
         {
             var filenames = Directory.GetFiles(RootDirectory, $"*{extension}", _options);
-            return filenames.Select(x => new FileInfo(x));
+            return filenames.Select(static x => new FileInfo(x));
         }
         catch (UnauthorizedAccessException)
         {
@@ -45,13 +37,13 @@ internal readonly struct ScriptFinder
     }
 
     public FileInfo[] GetScripts() =>
-        Extensions.Select(GetScriptFilesWithExtension).SelectMany(x => x).ToArray();
+        Extensions.Select(GetScriptFilesWithExtension).SelectMany(static x => x).ToArray();
 
     public IDictionary<DirectoryInfo, FileInfo[]> GetScriptsByDirectory() =>
         Extensions
             .Select(GetScriptFilesWithExtension)
-            .SelectMany(x => x)
-            .GroupBy(x => x.DirectoryName!)
-            .OrderBy(x => x.Key)
-            .ToDictionary(x => new DirectoryInfo(x.Key), x => x.ToArray());
+            .SelectMany(static x => x)
+            .GroupBy(static x => x.DirectoryName!)
+            .OrderBy(static x => x.Key)
+            .ToDictionary(static x => new DirectoryInfo(x.Key), static x => x.ToArray());
 }

src/ScriptLauncher.csproj

@@ -1,12 +1,12 @@
-<Project Sdk="Microsoft.NET.Sdk">
+<Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
     <OutputType>Exe</OutputType>
-    <TargetFrameworks>net8.0</TargetFrameworks>
+    <TargetFramework>net8.0</TargetFramework>
     <ImplicitUsings>enable</ImplicitUsings>
     <Nullable>enable</Nullable>
     <AnalysisMode>All</AnalysisMode>
-    <Version>0.1.5</Version>
+    <Version>0.1.7</Version>
     <PackAsTool>true</PackAsTool>
     <Description>Tool to find and exec shell scripts</Description>
     <PackageReadmeFile>README.md</PackageReadmeFile>
@@ -14,8 +14,8 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="Spectre.Console" Version="0.47.0" />
-    <PackageReference Include="Spectre.Console.Cli" Version="0.47.0" />
+    <PackageReference Include="Spectre.Console" Version="0.49.1" />
+    <PackageReference Include="Spectre.Console.Cli" Version="0.49.1" />
   </ItemGroup>
 
   <ItemGroup>