Correct the reason given for shim teardown by name

Copilot review flagged the comments as factually wrong, and it was half
right. Set-Item does accept 'function:global:X' - it creates X in the global
scope. The real problem is asymmetry: Remove-Item and Get-Item do not resolve
that same path back to the function, and fail silently instead of erroring,
so the teardown was a no-op.

Verified no 'global:' path form removes the function (function:global:X,
Function:global:X, Function:\global:X all leave it in place), so removing by
name is required. Behaviour is unchanged; only the explanation is corrected.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Author: 2 people

.github/actions/Publish-PSModule/tests/Publish-PSModule.Recovery.Tests.ps1

@@ -29,8 +29,10 @@ AfterAll {
     foreach ($name in $script:environmentVariableNames) {
         [System.Environment]::SetEnvironmentVariable($name, $script:originalEnvironment[$name])
     }
-    # The 'global:' scope qualifier is not a valid provider path, so it must be omitted here.
-    # Otherwise the shims leak into the global scope and shadow the real cmdlets in later test files.
+    # Set-Item accepts 'function:global:X' and creates a function named 'X' in the global scope, but
+    # Remove-Item and Get-Item do not resolve that same path back to it, and fail silently rather than
+    # erroring. Removing by name is what actually deletes the shims; leaving them behind would shadow the
+    # real cmdlets for every test file that runs later in the session.
     Remove-Item -Path 'Function:\Find-PSResource' -ErrorAction SilentlyContinue
     Remove-Item -Path 'Function:\Publish-PSResource' -ErrorAction SilentlyContinue
     Remove-Item -Path 'Function:\Resolve-PSModuleDependency' -ErrorAction SilentlyContinue

.github/actions/Release-PSModule/tests/Release-PSModule.WhatIf.Tests.ps1

@@ -35,8 +35,10 @@ AfterAll {
     foreach ($name in $script:environmentVariableNames) {
         [System.Environment]::SetEnvironmentVariable($name, $script:originalEnvironment[$name])
     }
-    # The 'global:' scope qualifier is not a valid provider path, so it must be omitted here.
-    # Otherwise the shims leak into the global scope and shadow the real commands in later test files.
+    # Set-Item accepts 'function:global:X' and creates a function named 'X' in the global scope, but
+    # Remove-Item and Get-Item do not resolve that same path back to it, and fail silently rather than
+    # erroring. Removing by name is what actually deletes the shims; leaving them behind would shadow the
+    # real commands for every test file that runs later in the session.
     Remove-Item -Path 'Function:\gh' -ErrorAction SilentlyContinue
     Remove-Item -Path 'Function:\git' -ErrorAction SilentlyContinue
 }