v0.1.2: fix theme download stdout leak, fix chocolatey+winget release

- Fix PowerShell Invoke-WebRequest progress output leaking to TUI (root cause of black tab bar)
- Add $ProgressPreference = 'SilentlyContinue' to all PowerShell download commands
- Add Stdio::piped() to all subprocess calls to prevent stdout/stderr leaks
- Fix Chocolatey nuspec schema namespace (chocolatey.org -> microsoft.com/2010/07)
- Use WriteAllText for nuspec to ensure clean UTF-8 without BOM
- Fix winget: use wingetcreate new+submit for first-time package, update for subsequent
- Change winget PackageIdentifier to marlocarlo.OmpManager (proper casing)
- Bump version to 0.1.2

Author: Your Name

.github/workflows/release.yml

@@ -142,9 +142,9 @@ jobs:
 
           New-Item -ItemType Directory -Force -Path choco/tools
 
-          @"
+          $nuspec = @"
           <?xml version="1.0" encoding="utf-8"?>
-          <package xmlns="http://schemas.chocolatey.org/packaging/2015/06/nuspec.xsd">
+          <package xmlns="http://schemas.microsoft.com/packaging/2010/07/nuspec.xsd">
             <metadata>
               <id>omp-manager</id>
               <version>$version</version>
@@ -154,13 +154,17 @@ jobs:
               <projectUrl>https://github.com/marlocarlo/omp-manager</projectUrl>
               <licenseUrl>https://github.com/marlocarlo/omp-manager/blob/master/LICENSE</licenseUrl>
               <requireLicenseAcceptance>false</requireLicenseAcceptance>
-              <description>A TUI manager for Oh My Posh - browse themes, install fonts, and configure shells with zero effort.</description>
-              <tags>oh-my-posh prompt theme tui terminal</tags>
+              <description>A TUI manager for Oh My Posh — browse themes, install fonts, and configure shells with zero effort.</description>
+              <summary>TUI manager for Oh My Posh prompt theming</summary>
+              <tags>oh-my-posh prompt theme tui terminal cli</tags>
+              <packageSourceUrl>https://github.com/marlocarlo/omp-manager</packageSourceUrl>
+              <bugTrackerUrl>https://github.com/marlocarlo/omp-manager/issues</bugTrackerUrl>
             </metadata>
           </package>
-          "@ | Out-File -Encoding utf8 choco/omp-manager.nuspec
+          "@
+          [System.IO.File]::WriteAllText("$PWD/choco/omp-manager.nuspec", $nuspec.Trim())
 
-          @"
+          $install = @"
           `$ErrorActionPreference = 'Stop'
           `$toolsDir = Split-Path -Parent `$MyInvocation.MyCommand.Definition
           `$url = '$url'
@@ -171,7 +175,8 @@ jobs:
             -UnzipLocation `$toolsDir ``
             -Checksum64 `$checksum ``
             -ChecksumType64 'sha256'
-          "@ | Out-File -Encoding utf8 choco/tools/chocolateyinstall.ps1
+          "@
+          [System.IO.File]::WriteAllText("$PWD/choco/tools/chocolateyinstall.ps1", $install.Trim())
 
           choco pack choco/omp-manager.nuspec --output-directory choco
 
@@ -210,11 +215,90 @@ jobs:
       - name: Install wingetcreate
         shell: pwsh
         run: |
+          $ProgressPreference = 'SilentlyContinue'
           Invoke-WebRequest -Uri "https://aka.ms/wingetcreate/latest" -OutFile wingetcreate.exe
 
       - name: Submit to winget-pkgs
         shell: pwsh
         run: |
           $version = "${{ steps.version.outputs.VERSION }}"
+          $sha256  = "${{ steps.checksum.outputs.SHA256 }}"
           $url = "https://github.com/marlocarlo/omp-manager/releases/download/${{ github.ref_name }}/omp-manager-x86_64-pc-windows-msvc.zip"
-          ./wingetcreate.exe update marlocarlo.omp-manager --version $version --urls $url --token ${{ secrets.WINGET_GH_PAT }} --submit
+
+          # Try update first (works when the package already exists in winget-pkgs)
+          ./wingetcreate.exe update marlocarlo.OmpManager `
+            --version $version --urls $url `
+            --token ${{ secrets.WINGET_GH_PAT }} --submit 2>&1 | Tee-Object -Variable updateOutput
+
+          if ($LASTEXITCODE -ne 0) {
+            Write-Host "Package not yet in winget-pkgs — creating initial manifest..."
+
+            $manifestDir = "manifests/m/marlocarlo/OmpManager/$version"
+            New-Item -ItemType Directory -Force -Path $manifestDir
+
+            # Version manifest
+            @"
+          # Created using GitHub Actions automated release pipeline
+          # yaml-language-server: `$schema=https://aka.ms/winget-manifest.version.1.6.0.schema.json
+          PackageIdentifier: marlocarlo.OmpManager
+          PackageVersion: $version
+          DefaultLocale: en-US
+          ManifestType: version
+          ManifestVersion: 1.6.0
+          "@ | ForEach-Object { $_.Trim() } | Set-Content -Path "$manifestDir/marlocarlo.OmpManager.yaml" -Encoding utf8NoBOM
+
+            # Installer manifest
+            @"
+          # yaml-language-server: `$schema=https://aka.ms/winget-manifest.installer.1.6.0.schema.json
+          PackageIdentifier: marlocarlo.OmpManager
+          PackageVersion: $version
+          InstallerType: zip
+          NestedInstallerType: portable
+          NestedInstallerFiles:
+            - RelativeFilePath: omp-manager.exe
+              PortableCommandAlias: omp-manager
+          Installers:
+            - Architecture: x64
+              InstallerUrl: $url
+              InstallerSha256: $sha256
+          ManifestType: installer
+          ManifestVersion: 1.6.0
+          "@ | ForEach-Object { $_.Trim() } | Set-Content -Path "$manifestDir/marlocarlo.OmpManager.installer.yaml" -Encoding utf8NoBOM
+
+            # Locale manifest
+            @"
+          # yaml-language-server: `$schema=https://aka.ms/winget-manifest.defaultLocale.1.6.0.schema.json
+          PackageIdentifier: marlocarlo.OmpManager
+          PackageVersion: $version
+          PackageLocale: en-US
+          Publisher: marlocarlo
+          PublisherUrl: https://github.com/marlocarlo
+          PackageName: OMP Manager
+          PackageUrl: https://github.com/marlocarlo/omp-manager
+          License: MIT
+          LicenseUrl: https://github.com/marlocarlo/omp-manager/blob/master/LICENSE
+          ShortDescription: A TUI manager for Oh My Posh — browse themes, install fonts, and configure shells
+          Description: A terminal user interface to browse themes, install Nerd Fonts, and configure shell profiles for Oh My Posh with zero effort.
+          Tags:
+            - cli
+            - oh-my-posh
+            - prompt
+            - terminal
+            - tui
+          ManifestType: defaultLocale
+          ManifestVersion: 1.6.0
+          "@ | ForEach-Object { $_.Trim() } | Set-Content -Path "$manifestDir/marlocarlo.OmpManager.locale.en-US.yaml" -Encoding utf8NoBOM
+
+            Write-Host "Generated manifests in $manifestDir"
+            Get-ChildItem -Recurse $manifestDir | ForEach-Object { Write-Host $_.FullName }
+
+            # Validate and submit
+            ./wingetcreate.exe submit $manifestDir `
+              --prtitle "New package: marlocarlo.OmpManager version $version" `
+              --token ${{ secrets.WINGET_GH_PAT }}
+
+            if ($LASTEXITCODE -ne 0) {
+              Write-Error "wingetcreate submit failed"
+              exit 1
+            }
+          }

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "omp-manager"
-version = "0.1.0"
+version = "0.1.2"
 edition = "2021"
 description = "A TUI manager for Oh My Posh - browse themes, install fonts, and configure shells with zero effort"
 license = "MIT"

Cargo.toml

@@ -162,7 +162,9 @@ fn list_nerd_fonts() -> Vec<String> {
                      Select-Object -ExpandProperty Name
         "#;
         Command::new("powershell")
-            .args(["-NoProfile", "-Command", script])
+            .args(["-NoProfile", "-Command", &format!("$ProgressPreference = 'SilentlyContinue'; {}", script)])
+            .stdout(std::process::Stdio::piped())
+            .stderr(std::process::Stdio::piped())
             .output()
             .ok()
             .and_then(|o| String::from_utf8(o.stdout).ok())

src/detect.rs

@@ -44,8 +44,10 @@ fn install_omp_windows() -> OpResult {
             let result2 = Command::new("powershell")
                 .args([
                     "-NoProfile", "-ExecutionPolicy", "Bypass", "-Command",
-                    "Set-ExecutionPolicy Bypass -Scope Process -Force; Invoke-Expression ((New-Object System.Net.WebClient).DownloadString('https://ohmyposh.dev/install.ps1'))"
+                    "$ProgressPreference = 'SilentlyContinue'; Set-ExecutionPolicy Bypass -Scope Process -Force; Invoke-Expression ((New-Object System.Net.WebClient).DownloadString('https://ohmyposh.dev/install.ps1'))"
                 ])
+                .stdout(std::process::Stdio::piped())
+                .stderr(std::process::Stdio::piped())
                 .output();
             match result2 {
                 Ok(o) if o.status.success() => OpResult {

src/install.rs

@@ -15,13 +15,13 @@ mod theme;
 mod themes;
 mod ui;
 
-use std::io::{self, BufWriter};
+use std::io;
 use std::time::Duration;
 
 use crossterm::{
     event::{self, DisableMouseCapture, EnableMouseCapture, Event, KeyCode, KeyEventKind, KeyModifiers, MouseButton, MouseEventKind},
-    execute, queue,
-    terminal::{disable_raw_mode, enable_raw_mode, BeginSynchronizedUpdate, EndSynchronizedUpdate, EnterAlternateScreen, LeaveAlternateScreen},
+    execute,
+    terminal::{disable_raw_mode, enable_raw_mode, EnterAlternateScreen, LeaveAlternateScreen},
 };
 use ratatui::{backend::CrosstermBackend, Terminal};
 
@@ -42,11 +42,7 @@ async fn main() -> anyhow::Result<()> {
     enable_raw_mode()?;
     let mut stdout = io::stdout();
     execute!(stdout, EnterAlternateScreen, EnableMouseCapture)?;
-    // 1 MiB BufWriter ensures the entire frame (escape-sequences for every
-    // changed cell) fits in a single buffer, so `flush()` emits one large
-    // write() syscall instead of many 8 KiB partial writes that the
-    // terminal would render individually, causing a visible flicker.
-    let backend = CrosstermBackend::new(BufWriter::with_capacity(1_048_576, stdout));
+    let backend = CrosstermBackend::new(stdout);
     let mut terminal = Terminal::new(backend)?;
 
     // Run
@@ -63,22 +59,15 @@ async fn main() -> anyhow::Result<()> {
 // ── Event loop ───────────────────────────────────────────────────────────────
 
 fn run_app(
-    terminal: &mut Terminal<CrosstermBackend<BufWriter<io::Stdout>>>,
+    terminal: &mut Terminal<CrosstermBackend<io::Stdout>>,
     app: &mut App,
 ) -> anyhow::Result<()> {
-    let mut needs_redraw = true;
-
     loop {
-        // Draw only when state has changed (prevents flickering from constant redraws)
-        if needs_redraw {
-            // Synchronized update: tell the terminal to buffer all screen
-            // writes until EndSynchronizedUpdate, then paint in one pass.
-            // Combined with the 1 MiB BufWriter this guarantees flicker-free
-            // atomic frame updates on Windows Terminal, ConPTY, etc.
-            queue!(terminal.backend_mut(), BeginSynchronizedUpdate)?;
-            terminal.draw(|f| ui::draw(f, app))?;
-            execute!(terminal.backend_mut(), EndSynchronizedUpdate)?;
-            needs_redraw = false;
+        // Draw every frame (ratatui diffs internally – unchanged frames are cheap)
+        terminal.draw(|f| ui::draw(f, app))?;
+
+        if !app.running {
+            return Ok(());
         }
 
         // Poll events (~20 fps)
@@ -97,14 +86,9 @@ fn run_app(
                     } else {
                         handle_normal_input(app, key.code, key.modifiers);
                     }
-                    needs_redraw = true;
                 }
                 Event::Mouse(mouse) => {
                     handle_mouse(app, mouse.kind, mouse.column, mouse.row);
-                    needs_redraw = true;
-                }
-                Event::Resize(_, _) => {
-                    needs_redraw = true;
                 }
                 _ => {}
             }
@@ -113,19 +97,10 @@ fn run_app(
         // Tick status message countdown
         if app.status.tick > 0 {
             app.tick_status();
-            needs_redraw = true;
         }
 
         // Apply any completed background theme downloads
-        let prev_downloading = app.theme_downloading.len();
         app.poll_theme_downloads();
-        if app.theme_downloading.len() != prev_downloading {
-            needs_redraw = true;
-        }
-
-        if !app.running {
-            return Ok(());
-        }
     }
 }
 

src/main.rs

@@ -296,13 +296,16 @@ pub fn download_theme(name: &str, dest_dir: &Path) -> Result<PathBuf, String> {
     #[cfg(target_os = "windows")]
     {
         let ps_cmd = format!(
-            "[Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12; \
+            "$ProgressPreference = 'SilentlyContinue'; \
+             [Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12; \
              Invoke-WebRequest -Uri '{}' -OutFile '{}' -UseBasicParsing",
             url,
             dest.to_string_lossy()
         );
         let output = std::process::Command::new("powershell")
             .args(["-NoProfile", "-ExecutionPolicy", "Bypass", "-Command", &ps_cmd])
+            .stdout(std::process::Stdio::piped())
+            .stderr(std::process::Stdio::piped())
             .output()
             .map_err(|e| format!("Failed to run PowerShell: {e}"))?;
         if !output.status.success() {
@@ -315,6 +318,8 @@ pub fn download_theme(name: &str, dest_dir: &Path) -> Result<PathBuf, String> {
     {
         let output = std::process::Command::new("curl")
             .args(["-fsSL", "-o", &dest.to_string_lossy(), &url])
+            .stdout(std::process::Stdio::piped())
+            .stderr(std::process::Stdio::piped())
             .output()
             .map_err(|e| format!("Failed to run curl: {e}"))?;
         if !output.status.success() {