Add Process.Start and update StartShellExecute to accept arguments

[Copilot]

• Remove redundant Start_WithoutArguments_Test
• Remove #if guard around using System.Collections.Generic;
• Change Process.Start to return Process
• Add #else argument-string fallback for both Start and StartShellExecute
• Update Start_Test to be async and assert ExitCode == 0
• Fix CS0103: StartShellExecute's #else branch now correctly uses startInfo.Arguments instead of process.StartInfo.Arguments

[Copilot]

Pull request overview

This PR extends PowerKit.Extensions.ProcessExtensions with additional process-starting helpers, adding a new Process.Start(...) extension that accepts an argument list and updating StartShellExecute to accept optional arguments.

Changes:

• Added Process.Start(string, IReadOnlyList<string>?) extension that starts a process and (when available) populates ProcessStartInfo.ArgumentList.
• Updated StartShellExecute to accept an optional argument list and changed its return type to void on newer TFMs (with a net35 fallback).
• Added tests to exercise the new Start API.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File	Description
PowerKit/Extensions/ProcessExtensions.cs	Adds new Start extension and updates StartShellExecute to optionally accept argument lists with TFM guards.
PowerKit.Tests/Extensions/ProcessExtensionsTests.cs	Adds tests for the new Start overload(s).

Comments suppressed due to low confidence (1)

PowerKit.Tests/Extensions/ProcessExtensionsTests.cs:46

• Start_WithoutArguments_Test calls Process.Start("dotnet"), which will bind to the BCL overload Process.Start(string) rather than the new extension member (since the framework member is applicable). As a result this test doesn’t cover the new API. If the intent is to validate the extension method’s no-arguments behavior, call it with an explicit second argument (e.g., Process.Start("dotnet", (IReadOnlyList<string>?)null) or an empty list).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov]

Codecov Report

❌ Patch coverage is 31.25000% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 93.71%. Comparing base (45628b8) to head (0b12ba8).
⚠️ Report is 8 commits behind head on prime.

Files with missing lines	Patch %	Lines
PowerKit/Extensions/ProcessExtensions.cs	0.00%	11 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##            prime      #73      +/-   ##
==========================================
- Coverage   93.85%   93.71%   -0.15%     
==========================================
  Files         128      132       +4     
  Lines        2524     2658     +134     
  Branches      200      212      +12     
==========================================
+ Hits         2369     2491     +122     
- Misses        112      123      +11     
- Partials       43       44       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Tyrrrz]

@copilot formatting issues

[Tyrrrz]

@copilot build is failing