Add robust pre-commit PowerShell test integration and CI optimization

.githooks/pre-commit.ps1

@@ -0,0 +1,46 @@
+#!/usr/bin/env pwsh
+# Pre-commit hook: Run Pester tests for system-maintenance.ps1 and save results
+$ErrorActionPreference = 'Stop'
+$testScript = 'PowerShell/system-administration/maintenance/system-maintenance.ps1'
+$testPath = 'tests/unit/PowerShell/system-maintenance.Tests.ps1'
+$resultsPath = 'tests/results/system-maintenance.xml'
+
+if (Test-Path $testPath) {
+    Write-Information "Running Pester tests for $testScript..."
+
+    # Set SCRIPTS_ROOT environment variable for tests
+    $env:SCRIPTS_ROOT = (Get-Location).Path
+
+    # Ensure the results directory exists before running Pester
+    New-Item -ItemType Directory -Path (Split-Path $resultsPath -Parent) -Force -ErrorAction SilentlyContinue | Out-Null
+
+    # Use Pester v5 configuration syntax
+    $config = New-PesterConfiguration
+    $config.Run.Path = $testPath
+    $config.Run.PassThru = $true
+    $config.TestResult.Enabled = $true
+    $config.TestResult.OutputPath = $resultsPath
+    $config.TestResult.OutputFormat = 'JUnitXml'
+
+    # Ensure the results directory exists before running Pester
+    New-Item -ItemType Directory -Path (Split-Path $resultsPath -Parent) -Force -ErrorAction SilentlyContinue
+    $result = Invoke-Pester -Configuration $config
+
+    if ($result.FailedCount -gt 0) {
+        Write-Error "Tests failed. Aborting commit."
+        exit 1
+    }
+
+    if (Test-Path $resultsPath) {
+        Write-Information "Test results saved to: $resultsPath"
+    }
+    else {
+        Write-Information "Test results not found, aborting commit."
+        exit 1
+    }
+}
+else {
+    Write-Output "Test file not found: $testPath"
+    Write-Host "Cannot verify script quality. Aborting commit."
+    exit 1
+}

.github/copilot-instructions.md

@@ -264,3 +264,10 @@ This repository prefers:
 - Create pull requests via GitHub web UI
 - Avoid using `gh` CLI in automation
 - Refer to `.github/PR_PREFERENCES.md` for detailed workflow guidance
+
+## Copilot PR Review Policy
+
+**IMPORTANT**: Do NOT automatically review pull requests when they are marked as "ready for review".
+- Only perform PR reviews when explicitly requested by tagging @copilot in a comment
+- Premium review requests should be used wisely and deliberately
+- Wait for manual request before analyzing or reviewing code changes

.gitignore

@@ -2,3 +2,6 @@
 
 #Ignore vscode AI rules
 .github\instructions\codacy.instructions.md
+
+# Test results
+tests/results/

.pre-commit-config.yaml

@@ -35,3 +35,10 @@ repos:
         language: system
         types: [powershell]
         pass_filenames: false
+    -   id: powershell-pester-tests
+        name: powershell-pester-tests
+        entry: pwsh .githooks/pre-commit.ps1
+        language: system
+        types: [powershell]
+        files: ^PowerShell/system-administration/maintenance/system-maintenance\.ps1$
+        pass_filenames: false