fix: #1561

Author: DrSkillIssue

Rules/UseConsistentWhitespace.cs

@@ -556,11 +556,32 @@ private IEnumerable<DiagnosticRecord> FindParameterViolations(Ast ast)
                 {
                     IScriptExtent leftExtent = commandParameterAstElements[i].Extent;
                     IScriptExtent rightExtent = commandParameterAstElements[i + 1].Extent;
+
+                    // Skip if elements are on different lines
                     if (leftExtent.EndLineNumber != rightExtent.StartLineNumber)
                     {
                         continue;
                     }
 
+                    // # 1561 - Skip if the whitespace is inside a string literal
+                    // Check if any string in the command contains this whitespace region
+                    var stringAsts = commandAst.FindAll(a => a is StringConstantExpressionAst || a is ExpandableStringExpressionAst, true);
+                    bool isInsideString = false;
+                    foreach (var stringAst in stringAsts)
+                    {
+                        if (stringAst.Extent.StartOffset < leftExtent.EndOffset && 
+                            stringAst.Extent.EndOffset > rightExtent.StartOffset)
+                        {
+                            isInsideString = true;
+                            break;
+                        }
+                    }
+                    
+                    if (isInsideString)
+                    {
+                        continue;
+                    }
+
                     var expectedStartColumnNumberOfRightExtent = leftExtent.EndColumnNumber + 1;
                     if (rightExtent.StartColumnNumber > expectedStartColumnNumberOfRightExtent)
                     {

Tests/Rules/UseConsistentWhitespace.tests.ps1

@@ -681,6 +681,27 @@ bar -h i `
             Invoke-Formatter -ScriptDefinition $def -Settings $settings |
                 Should -Be $expected
         }
+
+        # Tests for #1561
+        It "Should not remove whitespace inside string literals" {
+            $def = @'
+        $InputList | ForEach-Object {
+            $_.Name
+        } | Select-Object -First 2 | Join-String -sep ", " -OutputPrefix 'Results: '
+'@
+            $expected = @'
+        $InputList | ForEach-Object {
+            $_.Name
+        } | Select-Object -First 2 | Join-String -sep ", " -OutputPrefix 'Results: '
+'@
+            Invoke-Formatter -ScriptDefinition $def -Settings $settings | Should -BeExactly $expected
+        }
+
+        It "Should not remove whitespace from string parameters with multiple arguments" {
+            $def = 'Get-Process | Out-String -Stream | Select-String -Pattern "chrome", "firefox" -SimpleMatch'
+            $expected = 'Get-Process | Out-String -Stream | Select-String -Pattern "chrome", "firefox" -SimpleMatch'
+            Invoke-Formatter -ScriptDefinition $def -Settings $settings | Should -BeExactly $expected
+        }
     }
 
     Context "When keywords follow closing braces" {