Fix $mask variable mutation in GetDependencies inner loop

[Copilot]

❔What, Why & How

GetDependencies mutates the outer foreach($mask in $masks) loop variable inside the inner foreach($dependency in $probingPathsJson) loop when prepending buildMode. This causes the mask to accumulate prefixes across dependency iterations:

Iteration 1: App_xyz has buildMode "ONPREM" → $mask becomes "ONPREMApps"
Iteration 2: App_abc has no buildMode       → $mask becomes "ONPREMONPREMApps"

Introduced a local $currentMask variable initialized from $mask at the top of the inner loop. All inner-loop references now use $currentMask, leaving the outer loop variable untouched.

foreach($mask in $masks) {
    foreach($dependency in $probingPathsJson) {
        $currentMask = $mask                      # local copy
        if($buildMode -ne "Default") {
            $currentMask = "$buildMode$currentMask" # mutate copy, not outer var
        }
    }
}

Related to issue: #2236

✅ Checklist

• Add tests (E2E, unit tests)
• Update RELEASENOTES.md
• Update documentation (e.g. for new settings or scenarios)
• Add telemetry

[mazhelez]

@copilot awesome! Please include release notes.

[Copilot]

@copilot awesome! Please include release notes.

Added in a4cb1a2.

[mazhelez]

@copilot for the release notes, can you please follow the pattern and include the issue number?

[Copilot]

@copilot for the release notes, can you please follow the pattern and include the issue number?

Updated to follow the pattern with Issue 2236 in dffa6a0.

[Copilot]

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Fixes a bug in GetDependencies where the artifact mask string was being mutated across inner-loop iterations, leading to incorrect artifact name masks when multiple dependency probing paths use different buildMode values.

Changes:

• Introduces a per-iteration $currentMask to prevent mutation of the outer $mask variable.
• Updates all inner-loop usages to reference $currentMask instead of $mask.
• Adds a release note entry documenting the issue and fix.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
RELEASENOTES.md	Documents the fixed GetDependencies mask prefix leak (Issue 2236).
Actions/Github-Helper.psm1	Stops mutating the outer $mask by switching inner-loop logic to a local $currentMask.