Visitar URL original
Preserve verbatim stop-parsing argument passing by huong8373tt-beep · Pull Request #28162 · PowerShell/PowerShell · GitHub
Skip to content

Preserve verbatim stop-parsing argument passing #28162

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Slai.life (huong8373tt-beep) wants to merge 2 commits into
base: master
Choose a base branch
from
+52 −0
Open

Preserve verbatim stop-parsing argument passing #28162

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b4de1ee
Preserve verbatim stop-parsing argument passing
huong8373tt-beep Oct 7, 2026
a8d7868
Apply stop-parsing legacy binding before formatting
huong8373tt-beep Oct 8, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions src/System.Management.Automation/engine/NativeCommandParameterBinder.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,8 +67,17 @@ internal override object GetDefaultParameterValue(string name)
return null;
}

private bool HasVerbatimArgumentMarker { get; set; }

internal void BindParameters(Collection<CommandParameterInternal> parameters)
{
// The stop-parsing marker requires legacy string construction on Windows.
// Detect it before binding so arguments that precede the marker use the
// same construction rules as arguments that follow it.
HasVerbatimArgumentMarker = Platform.IsWindows && parameters.Any(
parameter => parameter.ArgumentSpecified
&& string.Equals("--%", parameter.ArgumentValue as string, StringComparison.OrdinalIgnoreCase));

bool sawVerbatimArgumentMarker = false;
bool first = true;
foreach (CommandParameterInternal parameter in parameters)
Expand Down Expand Up @@ -193,6 +202,11 @@ internal NativeArgumentPassingStyle ArgumentPassingStyle
{
get
{
if (HasVerbatimArgumentMarker)
{
return NativeArgumentPassingStyle.Legacy;
}

try
{
var preference = LanguagePrimitives.ConvertTo<NativeArgumentPassingStyle>(
Expand Down
38 changes: 38 additions & 0 deletions test/powershell/Language/Scripting/NativeExecution/NativeCommandArguments.Tests.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -181,6 +181,44 @@ Describe "find.exe uses legacy behavior on Windows" -Tag 'CI' {
}
}

Describe "Stop-parsing uses legacy native argument passing" -Tag 'CI' {
BeforeAll {
$currentSetting = $PSNativeCommandArgumentPassing
}

AfterAll {
$PSNativeCommandArgumentPassing = $currentSetting
}

It "Preserves escaped quotes after '--%' with <argumentListValue> argument passing" -Skip:(-not $IsWindows) -TestCases @(
@{ argumentListValue = 'Standard' }
@{ argumentListValue = 'Windows' }
) {
param($argumentListValue)

$PSNativeCommandArgumentPassing = $argumentListValue
$lines = @(Invoke-Expression 'testexe -echoargs --% /p:Category=\"CI,Nightly\"')

$lines | Should -BeExactly 'Arg 0 is </p:Category="CI,Nightly">'
}

It "Uses legacy construction for arguments before '--%' with <argumentListValue> argument passing" -Skip:(-not $IsWindows) -TestCases @(
@{ argumentListValue = 'Standard' }
@{ argumentListValue = 'Windows' }
) {
param($argumentListValue)

$PSNativeCommandArgumentPassing = $argumentListValue
$lines = @(Invoke-Expression 'testexe -echoargs 1, 2 --% tail')

$lines | Should -BeExactly @(
'Arg 0 is <1,>'
'Arg 1 is <2>'
'Arg 2 is <tail>'
)
}
}

foreach ( $argumentListValue in "Standard","Legacy","Windows" ) {
$PSNativeCommandArgumentPassing = $argumentListValue
Describe "Native Command Arguments (${PSNativeCommandArgumentPassing})" -tags "CI" {
Expand Down