Fix PSReviewUnusedParameter code scanning alerts

[aholstrup1]

Addresses open PSScriptAnalyzer code scanning alerts

Approach

The alerts fall into distinct categories, each handled differently:

Production code - true fixes:

• Build-AppsInWorkspace: Removed MajorMinorVersion, BuildNumber, and RevisionNumber parameters. These were accepted but never used - version stamping is handled by Update-AppJsonProperties which is called separately before compilation. Also removed the corresponding entries from the \ splatting in Compile.ps1.
• e2eTestHelper.psm1: The \ parameter was accepted by CreateNewAppInFolder but never written into the generated app.json. Added runtime to the output - this is a bug fix.
• CompileFromWorkspace.psm1: Replaced an empty catch block (for unparseable .NET runtime versions) with an OutputDebug call that logs the version string and error.

Production code - suppressed:

• Compile.ps1: \ is passed by the action YAML workflow and cannot be removed without updating the action definition. It may also be needed for future incremental build support (see existing TODO). Added a targeted SuppressMessageAttribute.

Test files - fixed:

• AL-Go-Helper.Test.ps1: \ was assigned in BeforeEach but PSScriptAnalyzer flagged it as unused since usage was in separate It blocks. Scoped it to \ so the cross-block usage is explicit.

Test files - suppressed:

• CompileFromWorkspace.Test.ps1 and DownloadProjectDependencies.Test.ps1: All 57 alerts are Pester mock scriptblock parameters that must match the mocked function's signature for positional binding. Added file-level SuppressMessageAttribute with justification.

[Copilot]

Pull request overview

This PR reduces PowerShell code scanning noise by fixing or intentionally suppressing PSReviewUnusedParameter findings across production scripts, helper modules, and Pester tests—keeping the action/module APIs aligned with real usage.

Changes:

• Removed unused version-stamping parameters from Build-AppsInWorkspace and stopped splatting them from the compile action.
• Fixed CreateNewAppInFolder to actually persist the provided runtime into generated app.json.
• Added targeted SuppressMessageAttribute suppressions for parameters that must exist (workflow-provided) and for Pester mock/callback signatures.

Reviewed changes

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

Show a summary per file

File	Description
Tests/DownloadProjectDependencies.Test.ps1	Adds file-level suppression for unused mock parameters (Pester signature parity).
Tests/CompileFromWorkspace.Test.ps1	Adds file-level suppression for unused mock parameters while preserving the UTF-8 BOM.
e2eTests/e2eTestHelper.psm1	Writes runtime into generated app.json to match the function’s parameter contract.
Actions/CompileApps/Compile.ps1	Suppresses the workflow-provided-but-currently-unused buildMode parameter and removes unused build splat keys.
Actions/.Modules/CompileFromWorkspace.psm1	Removes unused Build-AppsInWorkspace parameters and updates comment-based help accordingly.