Fix code review issues: remove hardcoded API key, update GH Actions, fix drive scan, graceful module-loader degradation

- C1: Remove hardcoded Bing API key from bing.ps1, require env var
- C2: Replace deprecated set-output with GITHUB_OUTPUT in manual.yml
- C3: Fix push URL to PS-Services org, update checkout to v4
- H1: Fix Snippetscode typo in _common.ps1 verbose output
- H2: Replace recursive C:\ scan with targeted search in github.ps1
- H5: Graceful degradation when powershell-yaml unavailable
- M4: Use Version.TryParse for SemVer pre-release strings
- M6: Remove invalid -Scope from Uninstall-PSResource
- M8: Fix oh-my-posh-windows Desktop branch to use 'powershell'
- D2: Fix wq to wg winget alias in README
- L2: Update all GitHub Actions to latest versions
- L3: Add SnippetsModulesYaml to ReadmeTest.ps9 templates
- Add full project code review document

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

Author: sharpninja

.github/workflows/jekyll-gh-pages.yml

@@ -30,9 +30,9 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
       - name: Setup Pages
-        uses: actions/configure-pages@v3
+        uses: actions/configure-pages@v5
 #       - name: Build with Jekyll
 #         uses: actions/jekyll-build-pages@v1
 #         with:
@@ -43,7 +43,7 @@ jobs:
         with:
           args: docs/docfx.json
       - name: Upload artifact
-        uses: actions/upload-pages-artifact@v1
+        uses: actions/upload-pages-artifact@v4
 
   # Deployment job
   deploy:
@@ -55,4 +55,4 @@ jobs:
     steps:
       - name: Deploy to GitHub Pages
         id: deployment
-        uses: actions/deploy-pages@v2
+        uses: actions/deploy-pages@v4

.github/workflows/manual.yml

@@ -16,7 +16,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: "Checkout source code"
-        uses: "actions/checkout@v1"
+        uses: "actions/checkout@v4"
       - name: Setup latest .NET SDKs
         uses: Elskom/setup-latest-dotnet@v1
         with:
@@ -35,12 +35,12 @@ jobs:
             echo "Error!" 1>&2
             exit 64
           fi
-          echo "::set-output name=version::$version"
+          echo "version=$version" >> "$GITHUB_OUTPUT"
           git config --global user.email "ninja@thesharp.ninja"
           git config --global user.name "GitHub on behalf of Sharp Ninja"
           git stage ./.version
           git commit -m 'Updated `.version`' --allow-empty --status
-          git push --repo='https://sharpninja:${{ secrets.GITHUB_TOKEN }}@github.com/sharpninja/Snippets.git'
+          git push --repo='https://x-access-token:${{ secrets.GITHUB_TOKEN }}@github.com/PS-Services/Snippets.git'
 
       - uses: "marvinpinto/action-automatic-releases@latest"
         with:

Linux-ReadmeTest.ps9

@@ -1,6 +1,8 @@
 # $env:VerboseStartup = 'true'
 $profileScript = Split-Path $PROFILE -Leaf
 
+$env:SnippetsModulesYaml = "$env:HOME/modules.yml"
+
 $exclusions=@()
 # EXCLUSIONS-START
 $exclusions += '*Windows*','repos*','choco*'

README.md

@@ -228,7 +228,7 @@ Write-Verbose 'PowerShell Ready.' -Verbose:$MasterVerbose
 ### Windows
 
 - `repos` Search all OS repos
-- `wq` winget
+- `wg` winget
 - `scp` scoop
 - `ch` chocolatey
 

SnippetsManager.psm1

@@ -1306,7 +1306,7 @@ class PSGalleryManager : PackageManager {
             '^uninstall|remove' {
                 if ($moduleName) {
                     if ($usePSResourceGet) {
-                        Uninstall-PSResource -Name $moduleName -Scope CurrentUser -ErrorAction Stop
+                        Uninstall-PSResource -Name $moduleName -ErrorAction Stop
                     }
                     else {
                         Uninstall-Module -Name $moduleName -Force -ErrorAction Stop

Windows-ReadmeTest.ps9

@@ -1,6 +1,8 @@
 # $env:VerboseStartup = 'true'
 $profileScript = Split-Path $PROFILE -Leaf
 
+$env:SnippetsModulesYaml = "$env:USERPROFILE\modules.yml"
+
 $exclusions = @()
 # EXCLUSIONS-START
 $exclusions += '*Linux*'

_common.ps1

@@ -96,7 +96,7 @@ function Initialize-Snippets {
         }
     }
     finally {
-        Write-Verbose "[$script] [$env:Snippetscode Initialized] -not `$env:SnippetsInitialized: $(-not $env:SnippetsInitialized)" -Verbose:$VerboseSwitch
+        Write-Verbose "[$script] [$env:SnippetsInitialized] -not `$env:SnippetsInitialized: $(-not $env:SnippetsInitialized)" -Verbose:$VerboseSwitch
 
         Pop-Location -Verbose:$VerboseSwitch
         $Verbose = $VerboseSwitch

bing.ps1

@@ -30,9 +30,10 @@ try {
     $env:BingLocation = $env:Snippets
     if (-not $env:BingLocation.EndsWith("Snippets")) { $env:BingLocation = Join-Path $env:BingLocation -Child "Snippets" }
     Write-Verbose "[$script] `$env:BingLocation: $env:BingLocation" -Verbose:$Verbose
-    $ApiKey = '3c7e251544ba414cbeacad9db55bdf6e'
-
-    $env:BingApiKey = $ApiKey
+    if (-not $env:BingApiKey) {
+      Write-Verbose "[$script] `$env:BingApiKey not set. Bing search will not work." -Verbose:$Verbose
+      return "Set `$env:BingApiKey to use Bing search."
+    }
     function Search-Bing {
       $query = [System.String]::Join(' ', $args);
       $csxPath = Join-Path $env:BingLocation -Child 'bing.csx' ;