From 5f01aa89fc0fce402d37f7911a71e01d6641748b Mon Sep 17 00:00:00 2001
From: Hitonabi <Hitonabi@dreivontanke.ddnsfree.com>
Date: Sun, 26 Jul 2026 15:55:05 +0200
Subject: [PATCH] fix(deinstaller): er brauchte Adminrechte und hatte keine

Commander: "Ich kann den worker uebrigens immernoch nicht deinstallieren."

Der Screenshot zeigt ZWEI Dinge. Das erste ist erklaerbar: Es lief noch der ALTE
uninstall.ps1 von der letzten Installation - Zeile 2 mit dem kaputten
`if (-not $Force) { if ([System.Windows.Forms.MessageBox]...` . Meine Reparatur
schreibt die Datei erst beim naechsten Installer-Lauf neu.

Das zweite ist MEIN Fehler, und er haette auch die neue Fassung erwischt:

  Remove-Item : Das Element C:\Program Files\Rippy Worker\doc\LICENSE
  kann nicht entfernt werden: Der Zugriff auf den Pfad wurde verweigert.

Der Standard-Zielordner liegt unter "C:\Program Files", und der gehoert nicht dem
Benutzer. Die Installer-.exe ist mit -requireAdmin gebaut und fragt beim Start per
UAC - der Deinstaller hatte nichts Vergleichbares. Er war damit im STANDARDFALL
nutzlos, und mein `rd /s /q` waere genauso aufgelaufen.

Jetzt erhoeht er sich selbst: Schreibprobe im Ordner, Admin-Abfrage, und nur wenn
beides fehlt, ein Neustart per RunAs mit -Force (damit die Bestaetigung nicht
zweimal kommt). GEMESSEN statt angenommen - wer den Worker ins eigene Profil
installiert hat, bekommt gar keine UAC-Frage. An seinem echten Ordner
gegengeprueft: schreibbar=False, istAdmin=False -> Entscheidung "erhoehen".

Der erzeugte Deinstaller ist diesmal komplett nachgestellt worden: parst, und die
Reihenfolge stimmt (Add-Type Zeile 7, MessageBox Zeile 10, Schreibprobe,
Erhoehung, erst dann Loeschen).

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
---
 deploy/worker-windows/RippyWorkerSetup.exe | Bin 73728 -> 75776 bytes
 deploy/worker-windows/install-gui.ps1      |  39 +++++++++++++++++++++
 deploy/worker-windows/install.ps1          |  27 ++++++++++++++
 3 files changed, 66 insertions(+)

diff --git a/deploy/worker-windows/RippyWorkerSetup.exe b/deploy/worker-windows/RippyWorkerSetup.exe
index 8014d3a002d3994be676432c2318d81185471537..3d1d74cd489a1d4a65db21b23c2100835ddb6ca9 100644
GIT binary patch
delta 2090
zcmaJ?Uu+ab9NsMzO0lGX!M`cLUK`rl?xr>bNDWD^*8>hqIeThjg`|78b9+PgW-qg|
zlorS}CPpI$EN9Z_lZih1)>wQXQPT(Gn=wB5fCOKB!$%W&@SEK`<WIsRcQ^N&neX?_
z_xm%eKeVs@(*Eg3r#APL&l}Bg$7nNmnY#?5!!(SZ#Apv-PMA;N*O*A=ZHCc~e|ofj
zjn%bZ?cKi}G>tiJ#_qpfUhCc0zF->9uK#G7hZ^si1D%I<G|@U5ADewUHRdbW-PzbQ
z5A4KN1IWg~#%(iouv<IEIEhc2@r-fncf)vL+Ay?L8piff%&%V4(64Uf+wKgveR%Dw
zF}qa1`^A^{zPnE8snfZc*=*+ITUnc?n52SogzHGxn&-?9q$^<aIr9`nLSdfDjGxWR
zz>`juj&q-dsS&KWU7da8V%cMrtxc&&@KQy=Ql&{IBDI30yQ`nkdFHZ^W?8i!TwqpC
zVI9u6ps-Y79QcZ=Tu};4G~#^ZNnS3K6P1A_C|`D5T8gC0=mL{VjC(kdqQOj1t%5d_
z)+8%~4UHYPM~~QJqhp6h$PpUqB-SUTvk3QYP&zv?oy%lr3R%sa-Uvh(NCgLQ<S`%L
zObkH}&L}5B<@i3JZFZ3n4`YU1osHXJZ@tKPRVxPCHQey+bmsL~g(=2c$Ee~c#WaIh
zl91kE((@VQDLN8lwErbXLI!>D1TGFh8D2?C=LeyU^BUqr_H?#T$j%TrDMfWhcue5#
zOpH@Dq$Mt0f@><2Dw6RcGzz0q1wpAg2!~B`AgOhn3*=&=FnHnwP#7VqW$sgxIoGiO
z_Yo>e2Ox?Tfy^i?5#UNBJTR}jRAh_ruOM8hq8TQeAcS>XIvxY@g?_|Vv%!7o7%lYU
zVj%=ka+xl5brQxxe@zBOhSgI67gpX;71)}|m$KcKnpV6ml^iNC)oo=0B&AT)JkxkT
z8ft7r`?e>DjqV!<>Ma+Ku~0Gsi7Qlc6coBdSGqbiSHw5%Rw3ar8ho$7N|EGh+0G+H
zOT6az_8b>(uoONpoO{Zq9HdrtoIf`{euAk?Bqa=kLkXiHI~V4}Y~Zt}Ihu^Puaa#*
ziK&62bW|XL)78n#G-!!HL9d4{Xd&4tG1nhlX{aT49~F>(6f(bvY-VvlQL!x2X{F1+
zChK@0OKgNtVm=`>ae)ScRw9puitC6cWr(6quG2>$st8%K^>vkzi^_=;fa)v2s6x49
z+e1;;k&kP~LgtlN7*Z`*LdI5@@5i-fp;U@A8;NvCR$6)y@?1^p3Y-{B&@_Y|DQhN3
z(wCtfrL62lL{9NQ<byYTURu`1$>o|8qFX(FQa3TI4-TXTXc#i)3(am4r{OTcKBO^Y
zMN$yG@ptrm<7#lEJ|9&nD^!_Dp#?%_&?5no3}wSeY;7v;(V%XRXC7{-DH?u^OwwWp
za24HOt4w>=4MNMA7G+&gP2HWMS7$z}2xTvb1@R<Q+DA7Pj^Y>CIOVml1ubmcUS~D2
za1qKZ(m>h|AEixCf;mxG-nxY<76Yvup(v9O3(>?1i;~xLDj|t<FDOghLG35Rp5?XL
za<ltD^U;yU_t$&vTqzJp?m0&aF1#VSOlKlx%|yPR$b-_<L1q`ZLd9;3tnKaJnE3Xa
zOn2dvpSSN?`C$6<qlbTLv;W<{asQvQ9eeNNaczyB_*q`--M<a*B;MKe8>V@AoqEjG
z_1$gepRY}s#*6z~n+@aWylFi5|6gM?jrUo<wat;AznR)*{_)aI!gCGV=#iUyddz#~
I+USn|021_eJ^%m!

delta 362
zcmZp;z|wGlWkQEn4}V$~0~9=9U}EHE<OcE>85rasg6rggYy}`&0Z21AK-f$S3@kt%
z2*7BDgo&5*_#%L+gF(VT)i0zcDhjhWFf#B=cjRO=nH<Te&b8$~NDic6au=gACr}lG
z43NV#c^jiVCr~K^&}0U=$@dtQ)Fyx|1_BQtW@2DvaJbCCAmIac8bc8S1DgSm&D#z#
zT5EFv(*tcL&i2JdddZtN%-~UBWZ1lW&st}JkNW@Ae_VUv(z#q^!5!A4@mnhxV>pBw
zpeBP%p9l8dL`5-XpncPsIT^dB^KmggHUt_WDF#vi0Txz_3?eWNkTy^R`VZt*C<w6H
cuEfoFUy4%z=v|=L2K()TYK-q0CmOH=018Syg#Z8m

diff --git a/deploy/worker-windows/install-gui.ps1 b/deploy/worker-windows/install-gui.ps1
index 808c07a..c8ab523 100644
--- a/deploy/worker-windows/install-gui.ps1
+++ b/deploy/worker-windows/install-gui.ps1
@@ -525,6 +525,45 @@ if (-not `$Force) {
 
 `$ordner = `$PSScriptRoot
 
+# ⚠️ ADMINRECHTE. Der Standard-Zielordner liegt unter "C:\Program Files", und
+# der gehört nicht dem Benutzer — jedes Remove-Item darin scheitert mit "Der
+# Zugriff auf den Pfad wurde verweigert" (Commander-Befund 26.07.2026, an
+# HandBrakes doc\LICENSE und Konsorten aufgelaufen). Die Installer-.exe ist mit
+# -requireAdmin gebaut und fragt beim Start per UAC; der Deinstaller hatte
+# nichts Vergleichbares und war damit im Standardfall nutzlos.
+#
+# GEMESSEN statt angenommen: Es wird ein Schreibversuch gemacht. Wer den Worker
+# ins eigene Profil installiert hat, braucht keine Erhöhung und bekommt auch
+# keine UAC-Frage.
+`$schreibbar = `$false
+try {
+    `$probe = Join-Path `$ordner ".deinstall-probe"
+    Set-Content -Path `$probe -Value "x" -ErrorAction Stop
+    Remove-Item -Force `$probe -ErrorAction SilentlyContinue
+    `$schreibbar = `$true
+} catch { }
+
+`$istAdmin = ([Security.Principal.WindowsPrincipal][Security.Principal.WindowsIdentity]::GetCurrent()
+    ).IsInRole([Security.Principal.WindowsBuiltInRole]::Administrator)
+
+if (-not `$schreibbar -and -not `$istAdmin) {
+    # Sich selbst erhöht neu starten. -Force, weil oben schon zugestimmt wurde —
+    # sonst käme die Frage zweimal.
+    try {
+        Start-Process powershell.exe -Verb RunAs -ArgumentList @(
+            "-NoProfile", "-ExecutionPolicy", "Bypass",
+            "-File", ('"' + `$PSCommandPath + '"'), "-Force")
+    } catch {
+        [System.Windows.Forms.MessageBox]::Show(
+            "Zum Entfernen sind Administratorrechte nötig (der Ordner liegt " +
+            "unter 'Programme'), und die Anfrage wurde abgelehnt.`n`n" +
+            "Alternative: PowerShell mit Rechtsklick 'Als Administrator " +
+            "ausführen' starten und dieses Skript erneut aufrufen.",
+            "Rippy Worker", 0, [System.Windows.Forms.MessageBoxIcon]::Warning) | Out-Null
+    }
+    exit
+}
+
 # 1. Alles beenden, was aus diesem Ordner läuft (Tray, celery, HandBrake).
 Get-CimInstance Win32_Process | Where-Object {
     `$_.ExecutablePath -like "`$ordner*"
diff --git a/deploy/worker-windows/install.ps1 b/deploy/worker-windows/install.ps1
index 570ebf9..c5ffa31 100644
--- a/deploy/worker-windows/install.ps1
+++ b/deploy/worker-windows/install.ps1
@@ -215,6 +215,33 @@ if (-not `$Force) {
     `$antwort = Read-Host "Rippy-Worker '$WorkerName' wirklich deinstallieren? (j/n)"
     if (`$antwort -ne "j") { Write-Host "Abgebrochen."; exit }
 }
+
+# ⚠️ ADMINRECHTE. Der Standard-Zielordner liegt unter "C:\Program Files", und der
+# gehört nicht dem Benutzer — jedes Remove-Item darin scheitert mit "Der Zugriff
+# auf den Pfad wurde verweigert" (Commander-Befund 26.07.2026). Geprüft wird per
+# Schreibversuch, nicht per Annahme: Wer ins eigene Profil installiert hat,
+# bekommt keine UAC-Frage.
+`$schreibbar = `$false
+try {
+    `$probe = Join-Path `$PSScriptRoot ".deinstall-probe"
+    Set-Content -Path `$probe -Value "x" -ErrorAction Stop
+    Remove-Item -Force `$probe -ErrorAction SilentlyContinue
+    `$schreibbar = `$true
+} catch { }
+`$istAdmin = ([Security.Principal.WindowsPrincipal][Security.Principal.WindowsIdentity]::GetCurrent()
+    ).IsInRole([Security.Principal.WindowsBuiltInRole]::Administrator)
+if (-not `$schreibbar -and -not `$istAdmin) {
+    Write-Host "Der Ordner liegt unter 'Programme' — starte mich mit Administratorrechten neu ..." -ForegroundColor Yellow
+    try {
+        Start-Process powershell.exe -Verb RunAs -ArgumentList @(
+            "-NoProfile", "-ExecutionPolicy", "Bypass",
+            "-File", ('"' + `$PSCommandPath + '"'), "-Force")
+    } catch {
+        Write-Host "FEHLER: Erhöhung abgelehnt. PowerShell mit Rechtsklick 'Als Administrator ausführen' starten und erneut aufrufen." -ForegroundColor Red
+    }
+    exit
+}
+
 Get-CimInstance Win32_Process | Where-Object {
     `$_.ExecutablePath -like "`$PSScriptRoot*"
 } | ForEach-Object { Stop-Process -Id `$_.ProcessId -Force -ErrorAction SilentlyContinue }