discstation 0.1.20 → 0.1.21

package/arduino/c6/DiscStation_C6.ino

@@ -510,7 +510,13 @@ void parseMessage(String msg) {
     line1 = msg.substring(7);
     line2 = "";
     line3 = "";
-    progressPercent = -1;
+    // Don't reset progressPercent here - a STATUS: label change (e.g.
+    // "Copying files..." -> "Converting...") is often immediately
+    // followed by a fresh PROGRESS: for the same ongoing operation.
+    // Wiping it every time flipped the bar to the indeterminate dots
+    // and back on every single label update, flickering between the
+    // two. DONE:/CANCELLED:/ERROR:/NO_DISC: still reset it below - those
+    // really are the end of an operation.
     drawStatus();
 
   } else if (msg.startsWith("PROGRESS:")) {

package/arduino/v1/DiscStation.ino

@@ -535,7 +535,10 @@ void parseMessage(String msg) {
     line1 = msg.substring(7);
     line2 = "";
     line3 = "";
-    progressPercent = -1;
+    // Don't reset progressPercent here - see arduino/c6/DiscStation_C6.ino's
+    // matching comment: STATUS: label changes mid-operation are usually
+    // immediately followed by a fresh PROGRESS:, so wiping it every time
+    // flickered the bar to the indeterminate dots and back.
     drawStatus();
 
   } else if (msg.startsWith("PROGRESS:")) {

package/install-windows.ps1

@@ -64,13 +64,32 @@ if ($winget -and -not $IsWin7) {
         catch { Write-Host "  yt-dlp fetch skipped" }
     }
     if (-not (Have "ffmpeg")) { Get-Zip "https://www.gyan.dev/ffmpeg/builds/ffmpeg-release-essentials.zip" $tools | Out-Null }
-    if (-not (Have "mpv")) { Get-Zip "https://sourceforge.net/projects/mpv-player-windows/files/latest/download" $tools | Out-Null }
+    if (-not (Have "mpv")) {
+        # The SourceForge "latest" mpv build is a .7z (Expand-Archive can't
+        # open it); mpv's own first-party CI release ships plain .zip builds.
+        try {
+            $mpvRelease = Invoke-RestMethod "https://api.github.com/repos/mpv-player/mpv/releases/tags/git-release" -UseBasicParsing -TimeoutSec 25
+            $mpvAsset = $mpvRelease.assets | Where-Object { $_.name -match "x86_64-w64-mingw32\.zip$" } | Select-Object -First 1
+            if ($mpvAsset) { Get-Zip $mpvAsset.browser_download_url (Join-Path $tools "mpv") | Out-Null }
+        } catch { Write-Host "  mpv fetch skipped" }
+    }
+    if (-not (Have "HandBrakeCLI")) {
+        try {
+            $hbRelease = Invoke-RestMethod "https://api.github.com/repos/HandBrake/HandBrake/releases/latest" -UseBasicParsing -TimeoutSec 25
+            $hbAsset = $hbRelease.assets | Where-Object { $_.name -match "^HandBrakeCLI-.*-win-x86_64\.zip$" } | Select-Object -First 1
+            if ($hbAsset) { Get-Zip $hbAsset.browser_download_url (Join-Path $tools "handbrake") | Out-Null }
+        } catch { Write-Host "  HandBrakeCLI fetch skipped" }
+    }
+    # dvdauthor + spumux (DVD-Video authoring/subtitles) have no winget package;
+    # this VideoHelp-hosted plain .zip (no rar/unrar needed) is the only
+    # reliable direct-download source found.
+    if (-not (Have "dvdauthor")) { Get-Zip "https://download.videohelp.com/gfd/edcounter.php?file=download/dvdauthor_winbin.zip" (Join-Path $tools "dvdauthor") | Out-Null }
     if (Test-Path $tools) {
         $env:Path = $env:Path + ";" + $tools + ";" +
             ((Get-ChildItem $tools -Recurse -Filter "ffmpeg.exe" -ErrorAction SilentlyContinue | Select-Object -First 1).DirectoryName)
     }
     Write-Host "xorriso and HandBrakeCLI have no simple direct-download URL - install manually if you need"
-    Write-Host "video-DVD authoring / DVD ripping (see the plan's Windows setup notes)."
+    Write-Host "video-DVD ISO packaging / DVD ripping (see the plan's Windows setup notes)."
 } else {
     Write-Host "winget unavailable (Windows 7). ISO + data + audio burn work via IMAPI2."
     Write-Host "For rip/play/video-DVD install manually: xorriso, HandBrakeCLI 1.5.1, ffmpeg, mpv, yt-dlp."
@@ -126,14 +145,18 @@ try {
         $action    = New-ScheduledTaskAction -Execute $pyw -Argument "`"$target`"" -WorkingDirectory $App
         $trigger   = New-ScheduledTaskTrigger -AtLogOn -User $env:USERNAME
         $set       = New-ScheduledTaskSettingsSet -AllowStartIfOnBatteries -DontStopIfGoingOnBatteries -RestartCount 3 -RestartInterval (New-TimeSpan -Minutes 1)
-        # S4U: runs headlessly for this user without needing an active interactive
-        # desktop session (Interactive logon type only fires while one exists, so
-        # e.g. Start-ScheduledTask over SSH/no console session would silently no-op).
+        # Interactive: attaches to the logged-in desktop session, same
+        # lifecycle as `systemctl --user`/launchd LaunchAgents on the other
+        # two platforms - required so PLAY's mpv window actually renders
+        # somewhere visible (S4U runs headlessly with no session to render
+        # into, which silently made every mpv window invisible). Trade-off,
+        # same one Linux/macOS already accept: won't start until someone is
+        # logged into the desktop.
         # RunLevel Limited (standard, non-elevated): nothing here needs admin -
         # IMAPI2 burning, WMI reads, and binding ports >1024 all work as a normal
         # user, and elevation is what put "Administrator" on the flashing console
         # windows this used to spawn.
-        $principal = New-ScheduledTaskPrincipal -UserId $env:USERNAME -LogonType S4U -RunLevel Limited
+        $principal = New-ScheduledTaskPrincipal -UserId $env:USERNAME -LogonType Interactive -RunLevel Limited
         Register-ScheduledTask -TaskName "DiscStation" -Action $action -Trigger $trigger -Settings $set -Principal $principal -Force | Out-Null
         Start-ScheduledTask -TaskName "DiscStation"
     } else {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "discstation",
-  "version": "0.1.20",
+  "version": "0.1.21",
   "description": "DiscStation optical-media appliance host — one cross-OS installer",
   "bin": {
     "discstation-setup": "scripts/setup.mjs",