WIP: Fix: PSUseConsistentWhitespace formatting for do-while, unary operators, and native commands

[Dr. Skill Issue (DrSkillIssue)]

PR Summary

Fixes multiple whitespace formatting issues in PSUseConsistentWhitespace rule

Issues Fixed:

• Whitespace Between Parameters Is Trimming Required Quotes #1561 - CheckParameter removing whitespace inside string literals
• PSUseConsistentWhitespace CheckInnerBrace - false positive on hashtable closing brace #1742 - False positive on hashtable closing brace with CheckInnerBrace
• Invalid reformatting when docker command executed directly from PS #2094 - Incorrect comma formatting in native command arguments (e.g., docker --secret id=x,env=y)
• Formatting misses certain keywords (unary operators, 'until') #2095 - Missing space between } and keywords in do-while/do-until loops

Changes Made:

Core Fixes:

• Added FindKeywordAfterBraceViolations to handle keywords following closing braces
• Enhanced unary operator detection and spacing logic in FindOperatorViolations
• Fixed FindParameterViolations to skip whitespace inside string literals
• Added comma pattern detection in FindSeparatorViolations for native command arguments
• Added the additional keywords to openParenKeywordAllowList: Until, Do, Else, Catch, Finally

Tests Added:

• Added test-cases for the above-mentioned changes.

PR Checklist

• PR has a meaningful title
  • Use the present tense and imperative mood when describing your changes
• Summarized changes
• Change is not breaking
• Make sure all .cs, .ps1 and .psm1 files have the correct copyright header
• Make sure you've added a new test if existing tests do not effectively test the code changed and/or updated documentation
• This PR is ready to merge and is not Work in Progress.
  • If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

[Dr. Skill Issue (DrSkillIssue)]

These PRs may just need closed as I couldn't reproduce the issues

• Formatting .where and .foreach methods is incorrect #1168 - Issue may have been fixed as the behavior is corrected when PSUseConsistentWhitespace is enabled.
• Invoke-Formatter: Adding whitespace after opening parenthesis #1288 - Couldn't reproduce this issue
• PSUseConsistentWhitespace for Pester tests and 'Should Throw' #1298 - Couldn't reproduce this issue
• PSUseConsistentWhitespace - False validation in switch statements #1319 - Couldn't reproduce this issue

PRs that were fixed with this PR

• Whitespace Between Parameters Is Trimming Required Quotes #1561 - Fixed with this PR
• PSUseConsistentWhitespace CheckInnerBrace - false positive on hashtable closing brace #1742 - Fixed with this PR
• Invalid reformatting when docker command executed directly from PS #2094 - Fixed with this PR
• Formatting misses certain keywords (unary operators, 'until') #2095 - Fixed with this PR

PRs that can be fixed in another PR(s) once this one is merged

Opting for multiple PRs as this one is large already.

• Enhance PSUseConsistentWhiteSpace to exclude more special cases when using unary operators #950
• Lines unaffected from sytax errors get formatted wrong on PowerShell 5 #1813

[Christoph Bergmeister (bergmeister)]

Dr. Skill Issue (@DrSkillIssue) Can you please resolve merge conflict and re-test please. This is due to PR #2140 , which tweaked the logic of the same area of the rule you modify