@geekbeer/minion 3.5.31 → 3.5.32

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@geekbeer/minion",
-  "version": "3.5.31",
+  "version": "3.5.32",
   "description": "AI Agent runtime for Minion - manages status and skill deployment on VPS",
   "main": "linux/server.js",
   "bin": {

package/win/minion-cli.ps1

@@ -996,7 +996,7 @@ function Invoke-Uninstall {
     }
     Write-Host ""
 
-    $totalSteps = 6
+    $totalSteps = 7
 
     # Step 1: Stop and remove all NSSM services
     Write-Step 1 $totalSteps "Stopping and removing services..."
@@ -1103,8 +1103,43 @@ function Invoke-Uninstall {
         Write-Detail "Removed legacy service credentials file"
     }
 
-    # Step 6: Remove Cloudflare Tunnel configuration
-    Write-Step 6 $totalSteps "Removing Cloudflare Tunnel configuration..."
+    # Step 6: Remove npm global bin from user PATH (added by postinstall)
+    Write-Step 6 $totalSteps "Cleaning up PATH entries..."
+    $npmBin = $null
+    try { $npmBin = (& npm prefix -g 2>$null) } catch {}
+    if ($npmBin) {
+        $targetUserName = Split-Path $TargetUserProfile -Leaf
+        $targetUserSid = $null
+        try {
+            $targetUserSid = (New-Object System.Security.Principal.NTAccount($targetUserName)).Translate(
+                [System.Security.Principal.SecurityIdentifier]).Value
+        } catch {}
+        if ($targetUserSid) {
+            $regPath = "Registry::HKEY_USERS\$targetUserSid\Environment"
+            if (Test-Path $regPath) {
+                $currentPath = (Get-ItemProperty -Path $regPath -Name PATH -ErrorAction SilentlyContinue).PATH
+                if ($currentPath -and $currentPath -like "*$npmBin*") {
+                    $newPath = ($currentPath -split ';' | Where-Object { $_ -ne $npmBin }) -join ';'
+                    if ($newPath) {
+                        Set-ItemProperty -Path $regPath -Name PATH -Value $newPath
+                    } else {
+                        Remove-ItemProperty -Path $regPath -Name PATH -ErrorAction SilentlyContinue
+                    }
+                    Write-Detail "Removed $npmBin from target user's PATH"
+                }
+            }
+        }
+        # Also clean up the current user's PATH
+        $userPath = [Environment]::GetEnvironmentVariable('PATH', 'User')
+        if ($userPath -and $userPath -like "*$npmBin*") {
+            $newPath = ($userPath -split ';' | Where-Object { $_ -ne $npmBin }) -join ';'
+            [Environment]::SetEnvironmentVariable('PATH', $newPath, 'User')
+            Write-Detail "Removed $npmBin from current user's PATH"
+        }
+    }
+
+    # Step 7: Remove Cloudflare Tunnel configuration
+    Write-Step 7 $totalSteps "Removing Cloudflare Tunnel configuration..."
     $cfConfigDir = Join-Path $TargetUserProfile '.cloudflared'
     if (Test-Path $cfConfigDir) {
         Remove-Item $cfConfigDir -Recurse -Force

package/win/postinstall.ps1

@@ -1,13 +1,52 @@
-# postinstall.ps1 — Re-apply file ACLs after npm install -g replaces the package directory.
+# postinstall.ps1 — Post-install tasks for @geekbeer/minion on Windows.
 # Called automatically from postinstall.js on Windows.
-# Requires Administrator privileges (npm install -g typically runs as admin).
-# Exit silently if not admin or if setup has never been run (no .target-user-profile).
+#
+# Tasks:
+# 1. Ensure npm global bin directory is in the user's PATH (any user)
+# 2. Re-apply file ACLs after npm install -g replaces the package directory (admin only)
 
 $ErrorActionPreference = 'SilentlyContinue'
 
+# --- Ensure npm global bin is in PATH ----------------------------------------
+# npm's global bin directory (where minion-cli-win.cmd is created) may not be in
+# PATH, especially when Node.js was installed via winget or exe installer.
+# Add it to the current user's persistent PATH if missing.
+
+function Ensure-NpmBinInPath {
+    $npmBin = $null
+    try { $npmBin = (& npm prefix -g 2>$null) } catch {}
+    if (-not $npmBin -or -not (Test-Path $npmBin)) { return }
+
+    # Check if already in Machine or User PATH
+    $machinePath = [Environment]::GetEnvironmentVariable('PATH', 'Machine')
+    $userPath = [Environment]::GetEnvironmentVariable('PATH', 'User')
+    $allPath = "$machinePath;$userPath"
+
+    if ($allPath -split ';' | Where-Object { $_ -eq $npmBin }) { return }
+
+    # Append to User PATH
+    if ($userPath) {
+        # Remove trailing semicolons before appending
+        $userPath = $userPath.TrimEnd(';')
+        [Environment]::SetEnvironmentVariable('PATH', "$userPath;$npmBin", 'User')
+    } else {
+        [Environment]::SetEnvironmentVariable('PATH', $npmBin, 'User')
+    }
+
+    # Also update current session so subsequent commands in this process work
+    $env:PATH = "$env:PATH;$npmBin"
+
+    Write-Host "[@geekbeer/minion] Added npm global bin to PATH: $npmBin"
+    Write-Host "[@geekbeer/minion] Please restart your terminal for the PATH change to take effect."
+}
+
+Ensure-NpmBinInPath
+
 # --- Locate setup metadata ---------------------------------------------------
 # During setup, .minion/.target-user-profile is written to the target user's
 # home directory.  We need to find it by scanning user profiles.
+# ACL restoration requires Administrator privileges.
+# Exit silently if not admin or if setup has never been run (no .target-user-profile).
 
 function Find-TargetUserProfile {
     $profiles = Get-CimInstance Win32_UserProfile -ErrorAction SilentlyContinue |