From 637e0b458530ab5224f2368f0bc31bd2265946c0 Mon Sep 17 00:00:00 2001
From: Jamie Pine <ijamespine@me.com>
Date: Tue, 3 Feb 2026 06:07:19 -0800
Subject: [PATCH] Enhance ProviderSettings component for platform-specific
 functionality

- Added platform detection for macOS and Windows to improve user experience.
- Updated UI to conditionally disable options and provide clearer guidance based on installed providers.
- Refactored button states and labels for PyTorch CUDA and CPU to reflect availability and download status accurately.
---
 .../ServerSettings/ProviderSettings.tsx       | 161 ++++++++++--------
 backend/providers/__init__.py                 |  15 +-
 tauri/src-tauri/gen/Assets.car                | Bin 3847048 -> 3847048 bytes
 3 files changed, 94 insertions(+), 82 deletions(-)

diff --git a/app/src/components/ServerSettings/ProviderSettings.tsx b/app/src/components/ServerSettings/ProviderSettings.tsx
index 566e73d2..7cd68270 100644
--- a/app/src/components/ServerSettings/ProviderSettings.tsx
+++ b/app/src/components/ServerSettings/ProviderSettings.tsx
@@ -22,6 +22,12 @@ import { apiClient } from '@/lib/api/client';
 import { useModelDownloadToast } from '@/lib/hooks/useModelDownloadToast';
 
 const isMacOS = () => navigator.platform.toLowerCase().includes('mac');
+const isWindows = () => navigator.platform.toLowerCase().includes('win');
+const getPlatformName = () => {
+  if (isMacOS()) return 'macOS';
+  if (isWindows()) return 'Windows';
+  return 'Linux';
+};
 
 type ProviderType =
   | 'auto'
@@ -200,14 +206,12 @@ export function ProviderSettings() {
             disabled={isStarting}
           >
             {/* PyTorch CUDA */}
-            <div
-              className={`flex items-center justify-between py-2 ${isMacOS() ? 'opacity-50' : ''}`}
-            >
-              <div className="flex items-center space-x-3 flex-1">
-                <RadioGroupItem value="pytorch-cuda" id="cuda" disabled={isMacOS() || isStarting} />
+            <div className="flex items-center justify-between py-2">
+              <div className={`flex items-center space-x-3 flex-1 ${isMacOS() || !installedProviders.includes('pytorch-cuda') ? 'opacity-50' : ''}`}>
+                <RadioGroupItem value="pytorch-cuda" id="cuda" disabled={isMacOS() || isStarting || !installedProviders.includes('pytorch-cuda')} />
                 <Label
                   htmlFor="cuda"
-                  className={`flex-1 ${isMacOS() || isStarting ? 'cursor-not-allowed' : 'cursor-pointer'}`}
+                  className={`flex-1 ${isMacOS() || isStarting || !installedProviders.includes('pytorch-cuda') ? 'cursor-not-allowed' : 'cursor-pointer'}`}
                 >
                   <div className="font-medium">PyTorch CUDA</div>
                   <div className="text-sm text-muted-foreground">
@@ -216,41 +220,45 @@ export function ProviderSettings() {
                 </Label>
               </div>
               <div className="flex items-center gap-2">
-                {!installedProviders.includes('pytorch-cuda') && (
-                  <Button
-                    onClick={() => handleDownload('pytorch-cuda')}
-                    size="sm"
-                    disabled={downloadingProvider === 'pytorch-cuda' || isStarting}
-                  >
-                    {downloadingProvider === 'pytorch-cuda' ? (
-                      <Icon icon="svg-spinners:ring-resize" className="h-4 w-4 animate-spin" />
-                    ) : (
-                      <>
-                        <HugeiconsIcon icon={Download01Icon} size={16} className="h-4 w-4 mr-1" />
-                        Download (2.4GB)
-                      </>
-                    )}
-                  </Button>
+                {isMacOS() && (
+                  <>
+                    <span className="text-xs text-muted-foreground">2.4GB</span>
+                    <Button size="sm" variant="secondary" disabled>
+                      Not Available on macOS
+                    </Button>
+                  </>
                 )}
-                {installedProviders.includes('pytorch-cuda') &&
-                  selectedProvider !== 'pytorch-cuda' && (
+                {!isMacOS() && !installedProviders.includes('pytorch-cuda') && (
+                  <>
+                    <span className="text-xs text-muted-foreground">2.4GB</span>
                     <Button
-                      onClick={() => handleStart('pytorch-cuda')}
+                      onClick={() => handleDownload('pytorch-cuda')}
                       size="sm"
                       variant="outline"
-                      disabled={isStarting}
+                      disabled={downloadingProvider === 'pytorch-cuda' || isStarting}
+                      className="shrink-0"
                     >
-                      Start
+                      {downloadingProvider === 'pytorch-cuda' ? (
+                        <Icon icon="svg-spinners:ring-resize" className="h-4 w-4 animate-spin" />
+                      ) : (
+                        <>
+                          <HugeiconsIcon icon={Download01Icon} size={16} className="h-4 w-4 mr-2" />
+                          Download
+                        </>
+                      )}
                     </Button>
-                  )}
+                  </>
+                )}
                 {installedProviders.includes('pytorch-cuda') && (
                   <Button
                     onClick={() => handleDelete('pytorch-cuda')}
                     size="sm"
-                    variant="ghost"
+                    variant="outline"
                     disabled={isStarting}
+                    className="shrink-0"
                   >
-                    <HugeiconsIcon icon={Delete01Icon} size={16} className="h-4 w-4" />
+                    <HugeiconsIcon icon={Delete01Icon} size={16} className="h-4 w-4 mr-2" />
+                    Uninstall
                   </Button>
                 )}
               </div>
@@ -258,11 +266,11 @@ export function ProviderSettings() {
 
             {/* PyTorch CPU */}
             <div className="flex items-center justify-between py-2">
-              <div className="flex items-center space-x-3 flex-1">
-                <RadioGroupItem value="pytorch-cpu" id="cpu" disabled={isStarting} />
+              <div className={`flex items-center space-x-3 flex-1 ${!installedProviders.includes('pytorch-cpu') ? 'opacity-50' : ''}`}>
+                <RadioGroupItem value="pytorch-cpu" id="cpu" disabled={isStarting || !installedProviders.includes('pytorch-cpu')} />
                 <Label
                   htmlFor="cpu"
-                  className={`flex-1 ${isStarting ? 'cursor-not-allowed' : 'cursor-pointer'}`}
+                  className={`flex-1 ${isStarting || !installedProviders.includes('pytorch-cpu') ? 'cursor-not-allowed' : 'cursor-pointer'}`}
                 >
                   <div className="font-medium">PyTorch CPU</div>
                   <div className="text-sm text-muted-foreground">
@@ -272,64 +280,67 @@ export function ProviderSettings() {
               </div>
               <div className="flex items-center gap-2">
                 {!installedProviders.includes('pytorch-cpu') && (
-                  <Button
-                    onClick={() => handleDownload('pytorch-cpu')}
-                    size="sm"
-                    disabled={downloadingProvider === 'pytorch-cpu' || isStarting}
-                  >
-                    {downloadingProvider === 'pytorch-cpu' ? (
-                      <Icon icon="svg-spinners:ring-resize" className="h-4 w-4 animate-spin" />
-                    ) : (
-                      <>
-                        <HugeiconsIcon icon={Download01Icon} size={16} className="h-4 w-4 mr-1" />
-                        Download (300MB)
-                      </>
-                    )}
-                  </Button>
-                )}
-                {installedProviders.includes('pytorch-cpu') &&
-                  selectedProvider !== 'pytorch-cpu' && (
+                  <>
+                    <span className="text-xs text-muted-foreground">242MB</span>
                     <Button
-                      onClick={() => handleStart('pytorch-cpu')}
+                      onClick={() => handleDownload('pytorch-cpu')}
                       size="sm"
                       variant="outline"
-                      disabled={isStarting}
+                      disabled={downloadingProvider === 'pytorch-cpu' || isStarting}
+                      className="shrink-0"
                     >
-                      Start
+                      {downloadingProvider === 'pytorch-cpu' ? (
+                        <Icon icon="svg-spinners:ring-resize" className="h-4 w-4 animate-spin" />
+                      ) : (
+                        <>
+                          <HugeiconsIcon icon={Download01Icon} size={16} className="h-4 w-4 mr-2" />
+                          Download
+                        </>
+                      )}
                     </Button>
-                  )}
+                  </>
+                )}
                 {installedProviders.includes('pytorch-cpu') && (
                   <Button
                     onClick={() => handleDelete('pytorch-cpu')}
                     size="sm"
-                    variant="ghost"
+                    variant="outline"
                     disabled={isStarting}
+                    className="shrink-0"
                   >
-                    <HugeiconsIcon icon={Delete01Icon} size={16} className="h-4 w-4" />
+                    <HugeiconsIcon icon={Delete01Icon} size={16} className="h-4 w-4 mr-2" />
+                    Uninstall
                   </Button>
                 )}
               </div>
             </div>
 
             {/* MLX bundled (macOS Apple Silicon only) */}
-            <div className="flex items-center space-x-3 py-2">
-              <RadioGroupItem value="apple-mlx" id="mlx" disabled={isStarting || !isMacOS()} />
-              <Label
-                htmlFor="mlx"
-                className={`flex-1 ${isStarting ? 'cursor-not-allowed' : 'cursor-pointer'}`}
-              >
-                <div className="font-medium">Apple MLX</div>
-                <div className="text-sm text-muted-foreground">
-                  {isMacOS()
-                    ? 'Bundled with this version, optimized for Apple Silicon'
-                    : 'Only available on Apple Silicon'}
-                </div>
-              </Label>
+            <div className="flex items-center justify-between py-2">
+              <div className={`flex items-center space-x-3 flex-1 ${!isMacOS() ? 'opacity-50' : ''}`}>
+                <RadioGroupItem value="apple-mlx" id="mlx" disabled={isStarting || !isMacOS()} />
+                <Label
+                  htmlFor="mlx"
+                  className={`flex-1 ${isStarting || !isMacOS() ? 'cursor-not-allowed' : 'cursor-pointer'}`}
+                >
+                  <div className="font-medium">Apple MLX</div>
+                  <div className="text-sm text-muted-foreground">
+                    {isMacOS()
+                      ? 'Bundled with this version, optimized for Apple Silicon'
+                      : 'Only available on Apple Silicon'}
+                  </div>
+                </Label>
+              </div>
+              {!isMacOS() && (
+                <Button size="sm" variant="secondary" disabled>
+                  Not Available on {getPlatformName()}
+                </Button>
+              )}
             </div>
 
             {/* Remote */}
-            <div className="space-y-2 py-2 opacity-50">
-              <div className="flex items-center space-x-3">
+            <div className="flex items-center justify-between py-2">
+              <div className="flex items-center space-x-3 flex-1 opacity-50">
                 <RadioGroupItem value="remote" id="remote" disabled />
                 <Label htmlFor="remote" className="flex-1 cursor-not-allowed">
                   <div className="font-medium">Remote Server</div>
@@ -338,17 +349,23 @@ export function ProviderSettings() {
                   </div>
                 </Label>
               </div>
+              <Button size="sm" variant="secondary" disabled>
+                Coming Soon
+              </Button>
             </div>
 
             {/* OpenAI */}
-            <div className="space-y-2 py-2 opacity-50">
-              <div className="flex items-center space-x-3">
+            <div className="flex items-center justify-between py-2">
+              <div className="flex items-center space-x-3 flex-1 opacity-50">
                 <RadioGroupItem value="openai" id="openai" disabled />
                 <Label htmlFor="openai" className="flex-1 cursor-not-allowed">
                   <div className="font-medium">OpenAI API</div>
                   <div className="text-sm text-muted-foreground">Use OpenAI's TTS API</div>
                 </Label>
               </div>
+              <Button size="sm" variant="secondary" disabled>
+                Coming Soon
+              </Button>
             </div>
           </RadioGroup>
           <p className="text-xs text-muted-foreground mt-5">
diff --git a/backend/providers/__init__.py b/backend/providers/__init__.py
index 57e5583e..8ad9681d 100644
--- a/backend/providers/__init__.py
+++ b/backend/providers/__init__.py
@@ -220,16 +220,11 @@ class ProviderManager:
             installed.append("pytorch-cpu")
         # Linux: no bundled provider - users must download
         
-        # Check for downloaded providers (Phase 2)
-        providers_dir = _get_providers_dir()
-        if providers_dir.exists():
-            for provider_file in providers_dir.glob("tts-provider-*"):
-                if provider_file.is_file() and provider_file.stat().st_size > 0:
-                    name = provider_file.name
-                    if "pytorch-cpu" in name:
-                        installed.append("pytorch-cpu")
-                    elif "pytorch-cuda" in name:
-                        installed.append("pytorch-cuda")
+        # Check for downloaded providers by checking if binary path exists
+        for provider_type in ["pytorch-cpu", "pytorch-cuda"]:
+            binary_path = get_provider_binary_path(provider_type)
+            if binary_path and binary_path.exists() and provider_type not in installed:
+                installed.append(provider_type)
         
         return installed
     
diff --git a/tauri/src-tauri/gen/Assets.car b/tauri/src-tauri/gen/Assets.car
index fbe8f7112d3a477897665837ed31bb74dd8b2763..df9a80fcbcf1ade9897fa1cd35d81324244f4b68 100644
GIT binary patch
delta 842
zcmZwGJ5N+W6bJBK5Lup!@(@-~Q1Lx5k27axXf88*=Z2D!Z(wIEEi6ndmQq6Ov?sAK
zp}3TO1HOR08)Hngq4keIwcg}+Po_Dq%<O;v(cFhGr|T<YjYgvd6EF^Kn1(5sJpR3Y
z+CGpYhr28fzuNbQZ+W@-?f4V_87i2xjVztWF_IwGNu+T8laxxXQfXKVt&Gz`{`(Ro
ztF6mbcjMK|7YA#dcm2@ohb9ccFkFKXxDKOrKeX~lp;e|BoDqg|5G0)vo>4JG>y^?m
zI#z>qObppy)LE|gi`IwXf>hQfR9Qshqo7I}5flZH4JDaKD&zmtFlFCOSBYMCTl38#
zkgt+;PH^%SIx4mBrIsR=Iy+ki?xM$m$tie>SBd&%YiYPpoTSPeM8YV7qKg<US>f2S
zs#3Hz#(_k}hEnB=M2o$>ofeG2I84AKOu;m?VFqr%O_+r_n1=<p1?M@IpaZvI8CKv9
z+=W$GgL`lv9>6+0gh%igp1{+3Z|B+BSy4($k)1^2U4x;r6@haplqK$xVhXt$7)nPO
z{pEH#y`$~tAYcPFL4pDe1_*3@w9WOeqGT+w)KP@2aS=@*;k`0KdU6~pO5Wwcg1AD#
tTx_w^JK1#LA=D?ETeEN0<{Ay2!4nulTt!?!+`c^TA}-cHKW}{;{R@B#;DZ1F

delta 842
zcmZwGxo#6t6b4{BA;ei7$U+hvhyk<T`0h)Z$=yaNDf0$&h?0T=iNY?4EIQ~T(IBN5
zm%Kq90}CMpNGL5oBoT!f>C@4SM*q3@+?oF8FZDirJn3vTYqi=U%s>McU>@dR_V`EV
zq<KiE2H(}2!6!uruXL;a>-fF?IZ=dI$}*Wmm=J-;RC;1dA4sMmDeHCQED)x-Glc%V
zlrUO)mXzLk{_N>td-L_k^+v7^6EF#vU<xk7bTx8~JXK_7sLaX{&21(!^Nc<+5+`CT
z73Y*nKi<cNVC7k#R@E;WZzqc&${4&*MA?J|YlR?THQ~{eDLcl!r{h5`aPBXHUc|;~
zy$DQO<7G)o!x@p1mc%o~h)y&Z%WZO)#n{fBD*vr$lo}h8Mda3t6ezKbYK#z(HhD*a
z#mg#7X0ja%QeLXe&sMbFJJ@f)49vnD%)<gKLKBwY3S5O{Sb<eogKO|7V*@tfI<#O5
zZop00hBn-S+i(Xua2M{ueRu#5tAqVVr>BKaF*{B(i8?4Eh4I8uZi!LRE2#y`(v59O
zW{=68e;2)@y~jXc2f9Fk0S5sRRCTnct8YcNRtl>;@d~q(hEd|32?Wnf&N#nh|8pEc
v*bpxo4!fo|>{@Wuu)Dkbvb|QT=_ULE6R6v$>!^F@_fJq)s;?h*-%bApZ${z~