refactor(scripts): replace GitHub-only CI wrappers with CIHelpers in linting scripts

[WilliamBerryiii]

Description

Refactored all four PowerShell linting scripts under scripts/linting/ to replace GitHub Actions-specific CI wrapper calls with the shared CIHelpers.psm1 module. This enables CI platform portability across GitHub Actions, Azure DevOps, and local environments while preserving the existing external contract (file paths, parameters, exit codes, artifact output paths).

Closes #69

• Imported CIHelpers.psm1 in all four linting scripts alongside existing LintingHelpers.psm1
• Replaced Write-GitHubAnnotation with Write-CIAnnotation, Set-GitHubOutput with Set-CIOutput, Set-GitHubEnv with Set-CIEnv, and Write-GitHubStepSummary with Write-CIStepSummary
• Removed inline ::group:: / ::endgroup:: / ::error:: workflow commands from Invoke-PSScriptAnalyzer.ps1
• Added standard file headers (copyright, SPDX, #Requires -Version 7.0), Set-StrictMode, and main execution guards across all scripts
• Wrapped core logic into testable functions (Invoke-PSScriptAnalyzerCore, Invoke-MarkdownLinkCheckCore, Invoke-LinkLanguageCheckCore, Invoke-LinkLanguageCheck)
• Replaced exit calls with return inside wrapper functions for improved testability
• Removed dead Get-RelativePrefix function from Markdown-Link-Check.ps1
• Removed unused -SoftFail parameter from Invoke-PSScriptAnalyzer.ps1 (callers use workflow-level continue-on-error instead)
• Replaced if ($Verbose) guards with native Write-Verbose cmdlet usage in Link-Lang-Check.ps1

Type of Change

• 🐛 Bug fix (non-breaking change fixing an issue)
• ✨ New feature (non-breaking change adding functionality)
• 💥 Breaking change (fix or feature causing existing functionality to change)
• 📚 Documentation update
• 🏗️ Infrastructure change (Terraform/IaC)
• ♻️ Refactoring (no functional changes)

Component(s) Affected

• deploy/000-prerequisites - Azure subscription setup
• deploy/001-iac - Terraform infrastructure
• deploy/002-setup - OSMO control plane / Helm
• deploy/004-workflow - Training workflows
• src/training - Python training scripts
• docs/ - Documentation

Note: Changes target scripts/linting/ (not listed above). Four scripts modified:
Invoke-PSScriptAnalyzer.ps1, Markdown-Link-Check.ps1, Invoke-LinkLanguageCheck.ps1, Link-Lang-Check.ps1

Testing Performed

• Terraform plan reviewed (no unexpected changes)
• Terraform apply tested in dev environment
• Training scripts tested locally with Isaac Sim
• OSMO workflow submitted successfully
• Smoke tests passed (smoke_test_azure.py)

PSScriptAnalyzer ran against all four modified scripts with project settings (PSScriptAnalyzer.psd1) — zero violations detected. Workflow YAML files verified as unchanged callers with identical file paths, parameters, exit codes, and artifact output paths.

Documentation Impact

• No documentation changes needed
• Documentation updated in this PR
• Documentation issue filed

Checklist

• My code follows the project conventions
• Commit messages follow conventional commit format
• I have performed a self-review
• Documentation impact assessed above
• No new linting warnings introduced

🔧 - Generated by Copilot

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

[Copilot]

Pull request overview

This PR refactors four PowerShell linting scripts under scripts/linting/ to replace GitHub Actions-specific CI wrapper calls with the shared CIHelpers.psm1 module, enabling CI platform portability across GitHub Actions, Azure DevOps, and local environments. The refactoring wraps core logic into testable functions with main execution guards, replaces GitHub-specific functions (Write-GitHubAnnotation, Set-GitHubOutput, Set-GitHubEnv, Write-GitHubStepSummary) with platform-agnostic equivalents (Write-CIAnnotation, Set-CIOutput, Set-CIEnv, Write-CIStepSummary), standardizes file headers, and removes dead code.

Changes:

• Imported CIHelpers.psm1 alongside existing LintingHelpers.psm1 in all four scripts
• Replaced inline GitHub Actions workflow commands and GitHub-specific function calls with CI-agnostic alternatives
• Wrapped core logic into testable functions with standardized main execution guards and error handling
• Removed -SoftFail parameter from Invoke-PSScriptAnalyzer.ps1 and unused Get-RelativePrefix function from Markdown-Link-Check.ps1

Reviewed changes

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

File	Description
scripts/linting/Invoke-PSScriptAnalyzer.ps1	Removed -SoftFail parameter; replaced GitHub-specific calls with CIHelpers; wrapped logic in Invoke-PSScriptAnalyzerCore function; removed inline ::group::/::error:: commands
scripts/linting/Markdown-Link-Check.ps1	Replaced GitHub-specific calls with CIHelpers; wrapped logic in Invoke-MarkdownLinkCheckCore function; removed unused Get-RelativePrefix function
scripts/linting/Link-Lang-Check.ps1	Imported CIHelpers; wrapped logic in Invoke-LinkLanguageCheck function; replaced if ($Verbose) guards with native Write-Verbose cmdlet
scripts/linting/Invoke-LinkLanguageCheck.ps1	Replaced GitHub-specific calls with CIHelpers; wrapped logic in Invoke-LinkLanguageCheckCore function; added comprehensive documentation

[Copilot]

Pull request overview

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