From 28818bed777d1eebe82643687da55abf8b4b3236 Mon Sep 17 00:00:00 2001
From: zhom <2717306+zhom@users.noreply.github.com>
Date: Thu, 7 Aug 2025 04:52:28 +0400
Subject: [PATCH] refactor: simplify app update toast

---
 src/components/app-update-toast.tsx | 22 +------
 src/components/custom-toast.tsx     | 91 +----------------------------
 src/lib/toast-utils.ts              | 37 +-----------
 3 files changed, 4 insertions(+), 146 deletions(-)

diff --git a/src/components/app-update-toast.tsx b/src/components/app-update-toast.tsx
index 21b14a3..1606a07 100644
--- a/src/components/app-update-toast.tsx
+++ b/src/components/app-update-toast.tsx
@@ -144,11 +144,7 @@ export function AppUpdateToast({
 
         {/* Other stage progress (with visual indicators) */}
         {showOtherStageProgress && (
-          <div className="mt-2 space-y-2">
-            <p className="text-xs text-muted-foreground">
-              {updateProgress.message}
-            </p>
-
+          <div className="mt-2 space-y-1">
             {/* Progress indicator for non-downloading stages */}
             <div className="w-full bg-gray-200 dark:bg-gray-700 rounded-full h-1.5">
               <div
@@ -159,22 +155,6 @@ export function AppUpdateToast({
                 }`}
               />
             </div>
-
-            {updateProgress.stage === "extracting" && (
-              <p className="text-xs text-muted-foreground">
-                Preparing update files...
-              </p>
-            )}
-            {updateProgress.stage === "installing" && (
-              <p className="text-xs text-muted-foreground">
-                Installing new version...
-              </p>
-            )}
-            {updateProgress.stage === "completed" && (
-              <p className="text-xs text-green-600 dark:text-green-400">
-                Update completed! Restarting application...
-              </p>
-            )}
           </div>
         )}
 
diff --git a/src/components/custom-toast.tsx b/src/components/custom-toast.tsx
index 94bcf7c..cf45972 100644
--- a/src/components/custom-toast.tsx
+++ b/src/components/custom-toast.tsx
@@ -111,16 +111,6 @@ interface TwilightUpdateToastProps extends BaseToastProps {
   hasUpdate?: boolean;
 }
 
-interface AppUpdateToastProps extends BaseToastProps {
-  type: "app-update";
-  stage?: "downloading" | "extracting" | "installing" | "completed";
-  progress?: {
-    percentage: number;
-    speed?: string;
-    eta?: string;
-  };
-}
-
 type ToastProps =
   | LoadingToastProps
   | SuccessToastProps
@@ -128,8 +118,7 @@ type ToastProps =
   | DownloadToastProps
   | VersionUpdateToastProps
   | FetchingToastProps
-  | TwilightUpdateToastProps
-  | AppUpdateToastProps;
+  | TwilightUpdateToastProps;
 
 function getToastIcon(type: ToastProps["type"], stage?: string) {
   switch (type) {
@@ -144,21 +133,7 @@ function getToastIcon(type: ToastProps["type"], stage?: string) {
         );
       }
       return <LuDownload className="flex-shrink-0 w-4 h-4 text-blue-500" />;
-    case "app-update":
-      if (stage === "completed") {
-        return (
-          <LuCheckCheck className="flex-shrink-0 w-4 h-4 text-green-500" />
-        );
-      } else if (stage === "downloading") {
-        return <LuDownload className="flex-shrink-0 w-4 h-4 text-blue-500" />;
-      } else if (stage === "installing") {
-        return (
-          <LuRefreshCw className="flex-shrink-0 w-4 h-4 text-blue-500 animate-spin" />
-        );
-      }
-      return (
-        <LuRefreshCw className="flex-shrink-0 w-4 h-4 text-blue-500 animate-spin" />
-      );
+
     case "version-update":
       return (
         <LuRefreshCw className="flex-shrink-0 w-4 h-4 text-blue-500 animate-spin" />
@@ -239,47 +214,6 @@ export function UnifiedToast(props: ToastProps) {
             </div>
           )}
 
-        {/* App update progress */}
-        {type === "app-update" && (
-          <div className="mt-2 space-y-1">
-            {/* Download progress with percentage */}
-            {progress &&
-              "percentage" in progress &&
-              stage === "downloading" && (
-                <>
-                  <div className="flex justify-between items-center">
-                    <p className="flex-1 min-w-0 text-xs text-gray-600 dark:text-gray-300">
-                      {progress.percentage.toFixed(1)}%
-                      {progress.speed && ` • ${progress.speed} MB/s`}
-                      {progress.eta && ` • ${progress.eta} remaining`}
-                    </p>
-                  </div>
-                  <div className="w-full bg-gray-200 dark:bg-gray-700 rounded-full h-1.5">
-                    <div
-                      className="bg-blue-500 h-1.5 rounded-full transition-all duration-300"
-                      style={{ width: `${progress.percentage}%` }}
-                    />
-                  </div>
-                </>
-              )}
-
-            {/* Progress indicator for other stages */}
-            {(stage === "extracting" ||
-              stage === "installing" ||
-              stage === "completed") && (
-              <div className="w-full bg-gray-200 dark:bg-gray-700 rounded-full h-1.5">
-                <div
-                  className={`h-1.5 rounded-full transition-all duration-500 ${
-                    stage === "completed"
-                      ? "bg-green-500 w-full"
-                      : "bg-blue-500 w-full animate-pulse"
-                  }`}
-                />
-              </div>
-            )}
-          </div>
-        )}
-
         {/* Version update progress */}
         {type === "version-update" &&
           progress &&
@@ -355,27 +289,6 @@ export function UnifiedToast(props: ToastProps) {
             )}
           </>
         )}
-
-        {/* Stage-specific descriptions for app updates */}
-        {type === "app-update" && !description && (
-          <>
-            {stage === "extracting" && (
-              <p className="mt-1 text-xs text-gray-600 dark:text-gray-300">
-                Preparing update files...
-              </p>
-            )}
-            {stage === "installing" && (
-              <p className="mt-1 text-xs text-gray-600 dark:text-gray-300">
-                Installing new version...
-              </p>
-            )}
-            {stage === "completed" && (
-              <p className="mt-1 text-xs text-green-600 dark:text-green-400">
-                Update completed! Restarting application...
-              </p>
-            )}
-          </>
-        )}
       </div>
     </div>
   );
diff --git a/src/lib/toast-utils.ts b/src/lib/toast-utils.ts
index 1285d63..66f5c8a 100644
--- a/src/lib/toast-utils.ts
+++ b/src/lib/toast-utils.ts
@@ -46,23 +46,12 @@ interface VersionUpdateToastProps extends BaseToastProps {
   };
 }
 
-interface AppUpdateToastProps extends BaseToastProps {
-  type: "app-update";
-  stage?: "downloading" | "extracting" | "installing" | "completed";
-  progress?: {
-    percentage: number;
-    speed?: string;
-    eta?: string;
-  };
-}
-
 type ToastProps =
   | SuccessToastProps
   | ErrorToastProps
   | DownloadToastProps
   | LoadingToastProps
-  | VersionUpdateToastProps
-  | AppUpdateToastProps;
+  | VersionUpdateToastProps;
 
 export function showToast(props: ToastProps & { id?: string }) {
   const toastId = props.id ?? `toast-${props.type}-${Date.now()}`;
@@ -257,27 +246,3 @@ export function showUnifiedVersionUpdateToast(
     ...options,
   });
 }
-
-export function showAppUpdateToast(
-  title: string,
-  stage: "downloading" | "extracting" | "installing" | "completed",
-  options?: {
-    id?: string;
-    description?: string;
-    progress?: {
-      percentage: number;
-      speed?: string;
-      eta?: string;
-    };
-    duration?: number;
-  },
-) {
-  return showToast({
-    type: "app-update",
-    title,
-    stage,
-    id: options?.id ?? "app-update-progress",
-    duration: stage === "downloading" ? Number.POSITIVE_INFINITY : 5000,
-    ...options,
-  });
-}