feat: windots refactor, pester test suite, and CI pipeline

- Convert Windots modules from .psm1 to .ps1 with dot-sourcing, remove Export-ModuleMember blocks
- Add Copy-ConfigFiles helper for DRY config installation across all dotfile installers
- Add Test-IsExcluded helper in Organizer.ps1 to centralize exclusion logic
- Add Assert-AdminOrElevate, Initialize-Logging, Invoke-MenuLoop utilities in Common.ps1
- Add smart install prompts for Oh My Posh and FastFetch (detect missing → install/skip/cancel)
- Add Pester test suite with 5 test files covering core functions, configs, and script syntax
- Add Pester CI job with NUnit XML artifact upload, rename workflow to "CI"
- Migrate Organizer.ps1 from Write-Host to Write-Log throughout
- Use shared Set-RegistryValue in Windots.Customization.ps1 instead of inline Set-ItemProperty
- Switch Steam Millennium install from piped irm|iex to direct .exe download
- Expand README tools grid with DefendNot, RemoveWindowsAI, and Sparkle
- Rewrite Windots README with structured tables, remove emojis
- Fix Expand-StartFolders path resolution using $PSScriptRoot directly

Author: emylfy

.github/workflows/lint.yml

@@ -1,12 +1,12 @@
-name: PSScriptAnalyzer
+name: CI
 
 on:
   push:
     branches: [main]
-    paths: ['**.ps1', '**.psm1', '**.psd1']
+    paths: ['**.ps1', '**.psm1', '**.psd1', 'config/**', 'tests/**']
   pull_request:
     branches: [main]
-    paths: ['**.ps1', '**.psm1', '**.psd1']
+    paths: ['**.ps1', '**.psm1', '**.psd1', 'config/**', 'tests/**']
 
 jobs:
   lint:
@@ -43,3 +43,34 @@ jobs:
           } else {
             Write-Host "No issues found." -ForegroundColor Green
           }
+
+  test:
+    name: Run Pester Tests
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Install Pester
+        shell: pwsh
+        run: |
+          Set-PSRepository PSGallery -InstallationPolicy Trusted
+          Install-Module -Name Pester -Force -Scope CurrentUser -MinimumVersion 5.5.0
+
+      - name: Run Pester Tests
+        shell: pwsh
+        run: |
+          $config = New-PesterConfiguration
+          $config.Run.Path = './tests'
+          $config.Run.Exit = $true
+          $config.Output.Verbosity = 'Detailed'
+          $config.TestResult.Enabled = $true
+          $config.TestResult.OutputFormat = 'NUnitXml'
+          $config.TestResult.OutputPath = './testresults.xml'
+          Invoke-Pester -Configuration $config
+
+      - name: Upload Test Results
+        uses: actions/upload-artifact@v4
+        if: always()
+        with:
+          name: test-results
+          path: ./testresults.xml

CHANGELOG.md

@@ -7,6 +7,29 @@ and this project uses [Calendar Versioning](https://calver.org/) (YY.M format).
 
 ## [26.3] - 2026-03-16
 
+### March 16 — Windots refactor, Pester tests, and CI pipeline
+
+- Added `Copy-ConfigFiles` helper in `Windots.Configs.ps1` — reusable config-copy-with-logging for all dotfile installers
+- Added `Test-IsExcluded` function in `Organizer.ps1` — centralizes file/folder exclusion logic
+- Added `Assert-AdminOrElevate`, `Initialize-Logging`, `Invoke-MenuLoop` utilities in `Common.ps1`
+- Added smart install prompts for Oh My Posh and FastFetch — detect missing tools and offer install/skip/cancel
+- Added Pester test suite (`tests/`) — Common.ps1 functions, config validation, ExternalLauncher params, module exports, script syntax
+- Added Pester CI job in `.github/workflows/lint.yml` with NUnit XML artifact upload
+- Added DefendNot, RemoveWindowsAI, and Sparkle to README tools grid
+- Changed Windots modules from `.psm1` to `.ps1` — dot-sourced instead of `Import-Module`, all `Export-ModuleMember` blocks removed
+- Changed `Organizer.ps1` migrated from `Write-Host` to `Write-Log` throughout
+- Changed `ExternalLauncher.ps1` header from `Write-Header` to `Write-Log`
+- Changed `Windots.Customization.ps1` uses shared `Set-RegistryValue` instead of `Set-ItemProperty` with inline try/catch
+- Changed `Set-OtherVSCConfig` delegates to `Set-VSCodeConfig` instead of duplicating copy logic
+- Changed `Set-VSCodeConfig` validates target path before copying
+- Changed Steam Millennium install from piped `irm | iex` to direct `.exe` download
+- Changed CI workflow renamed from "PSScriptAnalyzer" to "CI"; path triggers expanded to include `config/**` and `tests/**`
+- Changed README tools grid expanded to 8 entries (two rows of 4 at 25% width)
+- Changed Windots README rewritten — emoji-free headings, structured tables for Configs/Apps/Customization
+- Changed main menu box width normalized to 59 characters
+- Changed `Write-Log` catch block now surfaces warning instead of silently swallowing errors
+- Fixed `Expand-StartFolders` path resolution — uses `$PSScriptRoot` directly instead of double parent traversal
+
 ### March 16 — Bundle overhaul, return-to-menu navigation, and documentation rewrite
 
 - Added `Invoke-ReturnToMenu` in `Common.ps1` — reads saved launch directory from temp file and re-launches `simplify11.ps1` in the same window

README.md

@@ -142,27 +142,44 @@ irm https://raw.githubusercontent.com/emylfy/simplify11/main/scripts/install.ps1
 
 <table>
 <tr>
-<td align="center" width="20%">
+<td align="center" width="25%">
 <img src="https://github.com/ChrisTitusTech/winutil/blob/main/docs/assets/favicon.png?raw=true" width="60px" alt="WinUtil Logo"><br/>
 <b><a href="https://github.com/ChrisTitusTech/winutil" title="Visit WinUtil on GitHub">WinUtil</a></b><br/>
 <sub>Install programs, tweaks, fixes and updates</sub>
 </td>
-<td align="center" width="20%">
+<td align="center" width="25%">
 <img src="https://raw.githubusercontent.com/flick9000/winscript/refs/heads/main/app/public/logo.svg" width="60px" alt="WinScript Logo"><br/>
 <b><a href="https://github.com/flick9000/winscript" title="Visit WinScript on GitHub">WinScript</a></b><br/>
 <sub>Build custom setup scripts</sub>
 </td>
-<td align="center" width="20%">
+<td align="center" width="25%">
 <img src="https://github.com/Greedeks/GTweak/blob/main/Assets/GTweak.png" width="60px" alt="GTweak Logo"><br/>
 <b><a href="https://github.com/Greedeks/GTweak" title="Visit GTweak on GitHub">GTweak</a></b><br/>
 <sub>Tweaking tool and debloater</sub>
 </td>
-<td align="center" width="20%">
+<td align="center" width="25%">
+<img src="https://github.com/Parcoil/Sparkle/blob/v2/resources/sparklelogo.png?raw=true" width="60px" alt="Sparkle Logo"><br/>
+<b><a href="https://github.com/Parcoil/Sparkle" title="Visit Sparkle on GitHub">Sparkle</a></b><br/>
+<sub>Windows package manager</sub>
+</td>
+</tr>
+<tr>
+<td align="center" width="25%">
+<img src="https://i.imgur.com/F9gWA92.png" width="60px" alt="DefendNot Logo"><br/>
+<b><a href="https://github.com/es3n1n/defendnot" title="Visit DefendNot on GitHub">DefendNot</a></b><br/>
+<sub>Disable Windows Defender</sub>
+</td>
+<td align="center" width="25%">
+<img src="https://github.com/zoicware/RemoveWindowsAI/assets/118035521/33efb033-c935-416c-977d-777bb69a3737" width="60px" alt="RemoveWindowsAI Logo"><br/>
+<b><a href="https://github.com/zoicware/RemoveWindowsAI" title="Visit RemoveWindowsAI on GitHub">RemoveWindowsAI</a></b><br/>
+<sub>Remove Copilot & Recall</sub>
+</td>
+<td align="center" width="25%">
 <img src="https://raw.githubusercontent.com/undergroundwires/privacy.sexy/refs/heads/master/img/logo.svg" width="60px" alt="Privacy.sexy Logo"><br/>
 <b><a href="https://github.com/undergroundwires/privacy.sexy" title="Visit Privacy.sexy on GitHub">Privacy.sexy</a></b><br/>
 <sub>Security enhancement</sub>
 </td>
-<td align="center" width="20%">
+<td align="center" width="25%">
 <img src="https://raw.githubusercontent.com/marticliment/UniGetUI/refs/heads/main/media/icon.svg" width="60px" alt="UniGetUI Logo"><br/>
 <b><a href="https://github.com/marticliment/UniGetUI" title="Visit UniGetUI on GitHub">UniGetUI</a></b><br/>
 <sub>Discover, install and update packages</sub>

modules/tools/ExternalLauncher.ps1

@@ -24,7 +24,7 @@ if (-not $tool) {
 
 $Host.UI.RawUI.WindowTitle = "$($tool.name) Launcher"
 
-Write-Header -Text "Launching $($tool.name)..."
+Write-Log -Message "Launching $($tool.name)..." -Level INFO
 
 try {
     switch ($tool.type) {

modules/windots/Organizer.ps1

@@ -41,9 +41,23 @@ foreach ($item in $excludeList) {
     $null = $excludeHash.Add($item)
 }
 
+function Test-IsExcluded {
+    param(
+        [string]$BaseName,
+        [string]$Name
+    )
+    if ($excludeHash.Contains($Name) -or $excludeHash.Contains($BaseName) -or $BaseName -like "Uninstall*") {
+        return $true
+    }
+    foreach ($term in $excludeList) {
+        if ($BaseName -like "*$term*") { return $true }
+    }
+    return $false
+}
+
 if (-not ([Security.Principal.WindowsPrincipal][Security.Principal.WindowsIdentity]::GetCurrent()).IsInRole([Security.Principal.WindowsBuiltInRole]::Administrator) -and
     (Test-Path -Path $targetPaths[1])) {
-    Write-Host "`nAdministrator rights required for system directory!`n" -ForegroundColor Red
+    Write-Log -Message "Administrator rights required for system directory!" -Level ERROR
 
     $adminLaunchPath = Join-Path -Path $PSScriptRoot -ChildPath "..\..\scripts\AdminLaunch.ps1"
     . $adminLaunchPath
@@ -55,81 +69,69 @@ if (-not ([Security.Principal.WindowsPrincipal][Security.Principal.WindowsIdenti
 try {
     $dirsToBackup = $targetPaths | Where-Object { Test-Path $_ }
     if ($dirsToBackup) {
-        Write-Host "`nCreating backup to: $backupPath" -ForegroundColor Cyan
+        Write-Log -Message "Creating backup to: $backupPath" -Level INFO
         Compress-Archive -Path $dirsToBackup -DestinationPath $backupPath -CompressionLevel Fastest
 
         if (Test-Path $backupPath) {
-            Write-Host "Backup successful. Size: $('{0:N2} MB' -f ((Get-Item $backupPath).Length/1MB))" -ForegroundColor Green
-            Write-Host "Press Enter to continue" -ForegroundColor DarkCyan
+            Write-Log -Message "Backup successful. Size: $('{0:N2} MB' -f ((Get-Item $backupPath).Length/1MB))" -Level SUCCESS
+            Write-Log -Message "Press Enter to continue" -Level INFO
             $null = Read-Host
         }
         else {
-            Write-Host "Backup failed! Aborting operation." -ForegroundColor Red
+            Write-Log -Message "Backup failed! Aborting operation." -Level ERROR
             exit
         }
     }
 }
 catch {
-    Write-Host "Backup error: $_" -ForegroundColor Red
+    Write-Log -Message "Backup error: $_" -Level ERROR
     exit
 }
 
 foreach ($targetDir in $targetPaths) {
     if (-not (Test-Path -Path $targetDir)) {
-        Write-Host "`nSkipping missing directory: $targetDir" -ForegroundColor Yellow
+        Write-Log -Message "Skipping missing directory: $targetDir" -Level WARNING
         continue
     }
 
-    Write-Host "`nProcessing directory: $targetDir" -ForegroundColor Cyan
+    Write-Log -Message "Processing directory: $targetDir" -Level INFO
 
     $subFolders = Get-ChildItem -Path $targetDir -Directory | Where-Object {
         $_.Name -notmatch $excludeRegex -and
         -not $excludeHash.Contains($_.Name) -and
-        -not (Get-ChildItem -Path $_.FullName -Recurse -ErrorAction SilentlyContinue | 
-              Where-Object { 
-                  $file = $_
-                  $excludeHash.Contains($_.Name) -or 
-                  $excludeHash.Contains($_.BaseName) -or
-                  ($excludeList | Where-Object { $file.BaseName -like "*$_*" }) -or
-                  $_.BaseName -like "Uninstall*"
-              })
+        -not (Get-ChildItem -Path $_.FullName -Recurse -ErrorAction SilentlyContinue |
+              Where-Object { Test-IsExcluded -BaseName $_.BaseName -Name $_.Name })
     }
 
     foreach ($folder in $subFolders) {
         $folderName = $folder.Name
         $folderFullPath = $folder.FullName
 
         $files = Get-ChildItem -Path $folderFullPath -File -Recurse -ErrorAction SilentlyContinue |
-                 Where-Object { 
-                     $file = $_
-                     -not ($excludeList | Where-Object { $file.BaseName -like "*$_*" }) -and
-                     -not $excludeHash.Contains($_.Name) -and 
-                     -not $excludeHash.Contains($_.BaseName) -and
-                     -not ($_.BaseName -like "Uninstall*")
-                 }
+                 Where-Object { -not (Test-IsExcluded -BaseName $_.BaseName -Name $_.Name) }
 
         if (-not $files) {
-            Write-Host "`nNo movable files in: $folderName" -ForegroundColor DarkGray
+            Write-Log -Message "No movable files in: $folderName" -Level SKIP
             continue
         }
 
-        Write-Host "`nFolder: $folderName" -ForegroundColor Cyan
-        Write-Host "Contains these files:" -ForegroundColor White
+        Write-Log -Message "Folder: $folderName" -Level INFO
+        Write-Log -Message "Contains these files:" -Level INFO
         $files | ForEach-Object {
-            Write-Host "  - $($_.Name)" -ForegroundColor Gray
+            Write-Log -Message "  - $($_.Name)" -Level INFO
         }
 
         do {
             $response = Read-Host "`nMove $($files.Count) files from '$folderName'? (Y/N/Q)"
             $response = $response.Trim().ToUpper()
             if ($response -eq 'Q') { 
-                Write-Host "Operation cancelled by user." -ForegroundColor Yellow
+                Write-Log -Message "Operation cancelled by user." -Level WARNING
                 exit 
             }
         } until ($response -match '^[YN]$')
 
         if ($response -eq 'N') {
-            Write-Host "Skipping folder: $folderName" -ForegroundColor Yellow
+            Write-Log -Message "Skipping folder: $folderName" -Level SKIP
             continue
         }
 
@@ -140,7 +142,7 @@ foreach ($targetDir in $targetPaths) {
             try {
                 $destination = Join-Path -Path $targetDir -ChildPath $file.Name
                 if (Test-Path $destination) {
-                    Write-Host "Replacing existing: $($file.Name)" -ForegroundColor DarkYellow
+                    Write-Log -Message "Replacing existing: $($file.Name)" -Level WARNING
                 }
                 Move-Item -Path $file.FullName -Destination $targetDir -Force -ErrorAction Stop
                 $movedCount++
@@ -151,24 +153,24 @@ foreach ($targetDir in $targetPaths) {
         }
 
         if ($errors.Count -gt 0) {
-            Write-Host "`nCompleted with $($errors.Count) errors:" -ForegroundColor Red
-            $errors | ForEach-Object { Write-Host "  $_" -ForegroundColor Red }
+            Write-Log -Message "Completed with $($errors.Count) errors:" -Level ERROR
+            $errors | ForEach-Object { Write-Log -Message "  $_" -Level ERROR }
         }
         else {
-            Write-Host "`nSuccessfully moved $movedCount files" -ForegroundColor Green
+            Write-Log -Message "Successfully moved $movedCount files" -Level SUCCESS
         }
 
         try {
             $remainingItems = Get-ChildItem -Path $folderFullPath -Recurse -Force -ErrorAction SilentlyContinue
             if (-not $remainingItems) {
                 Remove-Item -Path $folderFullPath -Recurse -Force -ErrorAction Stop
-                Write-Host "Cleaned empty folder: $folderName" -ForegroundColor DarkGray
+                Write-Log -Message "Cleaned empty folder: $folderName" -Level INFO
             }
         }
         catch {
-            Write-Host "Error cleaning folder: $_" -ForegroundColor Red
+            Write-Log -Message "Error cleaning folder: $_" -Level ERROR
         }
     }
 }
 
-Write-Host "`nOperation completed! Backup saved to: $backupPath`n" -ForegroundColor Green
+Write-Log -Message "Operation completed! Backup saved to: $backupPath" -Level SUCCESS