Repository navigation

Add ProcessStartOptions class with platform-aware path resolution - #124271

Merged

adamsitnik merged 36 commits into

mainfrom

copilot/add-processstartoptions-class

Feb 18, 2026

Commits on Feb 11, 2026

Initial plan
Copilot
committed
4f3f950
Add ProcessStartOptions class with path resolution and tests

Copilot
and
adamsitnik
committed
92de299
Fix build errors and test failures in ProcessStartOptions

Copilot
and
adamsitnik
committed
0c9d3e2
Improve code comments based on review feedback

Copilot
and
adamsitnik
committed
794a8a2
Fix exception documentation and add caching comment

Copilot
and
adamsitnik
committed
a1b3426
Address PR feedback: use Process.ResolvePath, fix tests, add null checks

Copilot
and
adamsitnik
committed
10ff95a
Refactor: ProcessStartOptions.ResolvePath as main impl, Process calls it

Copilot
and
adamsitnik
committed
fc0fd30
address my own feedback
adamsitnik
committed
f687eb8
Simplify Windows tests: remove ConditionalFact, fix nested try-finally

Copilot
and
adamsitnik
committed
5459ec9

Commits on Feb 12, 2026

Refactor tests: partial classes, remove Test prefix, use new() syntax, Theory for utils

Copilot
and
adamsitnik
committed
c3517ee
Fix test conditions: remove redundant IsWindows check, use GetFullPath for sh

Copilot
and
adamsitnik
committed
bbea91e
Fix test failures: use ResolveTarget helper for symlinks, fix lambda syntax

Copilot
and
adamsitnik
committed
d9d381f
Fix Unix test failures: use EndsWith instead of exact path comparison

Copilot
and
adamsitnik
committed
b4d6e64
Address code review: add file check, fix .exe logic, update comments, remove dead code

Copilot
and
adamsitnik
committed
98d4201
Cache Windows directory value for performance

Copilot
and
adamsitnik
committed
5b024d4
Remove DictionaryWrapper and fix Unix path test for macOS symlinks

Copilot
and
adamsitnik
committed
add4af6

Commits on Feb 13, 2026

Commits on Feb 15, 2026

address code review feedback:
adamsitnik
committed
bf02bd9

Commits on Feb 16, 2026

Remove executable directory probing per code review feedback

Copilot
and
adamsitnik
committed
e778275
Simplify path resolution: remove CWD and Windows dir probing, require explicit ./

Copilot
and
adamsitnik
committed
168ddf6
Restore Process.Unix.cs ResolvePath for backward compatibility

Copilot
and
adamsitnik
committed
200d9f4
address my own feedback, fix the tests
adamsitnik
committed
ec3f354
update comments, minor cleanup
adamsitnik
committed
bdd6301
changes after reading everything again
adamsitnik
committed
02905f0
apply improved suggestion
adamsitnik
committed
bf6e551

Commits on Feb 17, 2026

address code review feedback:
adamsitnik
committed
4415d7e
update the comment
adamsitnik
committed
a012a1e
address code review feedback: move the common logic to a helper class that is used by both old and new APIs
adamsitnik
committed
bc6543b
Refactor FindProgramInPath to ProcessUtils per jkotas feedback

Copilot
and
adamsitnik
committed
c056741
Make IsExecutable private in ProcessUtils.Windows.cs

Copilot
and
adamsitnik
committed
c6b1f56
Remove WINDOWS compilation constant per jkotas feedback

Copilot
and
adamsitnik
committed
b61103d
Deduplicate FindProgramInPath: move to ProcessUtils.cs and restore comment

Copilot
and
adamsitnik
committed
2dd4a3b