[Copilot]

Alias correction can rewrite DSC resource declarations such as Package PackageExample to Get-Package PackageExample, invalidating the configuration.

• Resource detection: Exclude resource-shaped declarations inside configurations from implicit Get- prefix correction, supporting opening braces on either line.
• Command handling: Preserve correction of explicit aliases and executable commands inside functions and DSC script properties.
• Regression coverage: Cover both brace layouts and distinguish resource declarations from ordinary commands.

The following declaration should remain unchanged:

Configuration PackageTest {
    Package PackageExample
    {
        Name = 'TestPackage'
    }
}

• Fixes Auto Correct Aliases incorrectly expands DSC resources name thinking its an alias of a cmdlet #2025

[Copilot]

🟡 Changes recommended

The shared guard suppresses expected Get- corrections for executable commands in configurations and adds quadratic sibling-statement searches even in ordinary scripts.

3 open issues

• Defer DSC body lookup to prevent quadratic scans in ordinary scripts · New
• Avoid correcting commands in directly assigned script blocks · New
• Restrict resource command detection to standalone single-command pipelines · New

What changed in this PR

This PR updates alias correction to distinguish DSC resource declarations from executable commands, addressing resource names such as Package being incorrectly rewritten as Get-Package.

Changes:

• Adds resource detection for opening braces on the same or following line.
• Adds context checks intended to preserve corrections inside executable code.
• Adds regression tests for resource formatting, explicit aliases, and ordinary commands.

File	Description
Tests/​Rules/​AvoidUsingAlias.tests.ps1	Adds formatting and command-analysis regression coverage.
Rules/​AvoidAlias.cs	Adds DSC resource detection before implicit Get- prefix correction.

_{Review effort: Balanced}

[Copilot]

🟡 Changes recommended

DSC resource detection incorrectly suppresses alias diagnostics for executable commands and pipeline elements, while introducing quadratic scanning for ordinary scripts.

7 open issues

• This scans sibling statements before checking whether the command belongs to a configuration. For… · New
• Inside a configuration, executable commands such as $result = Verb Argument {}, `$action = { Verb… · New
• The CommandExpressionAst exception accepts standalone blocks without checking that they belong to… · New
• The resource-body checks also accept commands belonging to a multi-command pipeline. For example,… · New
• Restrict resource command detection to standalone single-command pipelines
• Avoid correcting commands in directly assigned script blocks
• Defer DSC body lookup to prevent quadratic scans in ordinary scripts

_{Review effort: Balanced}

[Copilot]

🔵 Needs a closer look

Human assessment is needed because the following-line AST path can misclassify an executable cmdlet as configuration syntax and suppress an expected Get-Verb diagnostic.

No open issues 🎉

2 resolved since last review

• Handle trailing ampersands after fallback resource blocks
• Reject background pipelines before fallback resource classification

_{Review effort: Balanced}

[Copilot]

🟡 Changes recommended

The fallback AST handling can misclassify an executable command followed by a standalone block as a resource, suppressing a valid Get-Verb diagnostic; it needs the same alias and command-resolution checks as the same-line path, preserving the Node exception, plus a regression test.

1 open issue

• Preserve diagnostics for following-line verb blocks in fallback AST · New

_{Review effort: Balanced}

[Copilot]

🔵 Needs a closer look

Human assessment is needed because separate-runspace command resolution can misclassify script-defined function callbacks as resource bodies, suppressing expected corrections.

No open issues 🎉

1 resolved since last review

• Preserve diagnostics for following-line verb blocks in fallback AST

_{Review effort: Balanced}

[Copilot]

🟡 Changes recommended

The two-element exception in AvoidAlias.cs suppresses the Get-ChildItem correction for gci . followed by a next-line block, making alias handling inconsistent with same-line brace placement.

1 open issue

• Fix gci alias handling when block follows on the next line · New

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

No active review issues are reported for the change preventing alias correction from rewriting DSC resource names.

No open issues 🎉

1 resolved since last review

• Fix gci alias handling when block follows on the next line

_{Review effort: Balanced}