adjust the bugs for sending getchatdetail twice

Author: pengyu

frontend/src/components/chat/code-engine/code-engine.tsx

@@ -71,39 +71,27 @@ export function CodeEngine({
     }
   }, [chatId]);
 
-  // Poll for project if needed using chatId
   useEffect(() => {
-    // 如果项目已经完成，跳过轮询
-    if (projectCompleted || isProjectLoadedRef.current) {
-      return;
-    }
-
-    if (!curProject && chatId && !projectLoading) {
-      const loadProjectFromChat = async () => {
-        try {
-          setIsLoading(true);
-          const project = await pollChatProject(chatId);
-          if (project) {
-            setLocalProject(project);
-            // 如果成功加载项目，将状态设置为已完成
-            if (project.projectPath) {
-              setProjectCompleted(true);
-              isProjectLoadedRef.current = true;
-            }
-          }
-        } catch (error) {
-          logger.error('Failed to load project from chat:', error);
-        } finally {
-          setIsLoading(false);
-        }
-      };
+    // 如果全局轮询完毕，projectPath 可用了，就完成 loading bar
+    if (
+      curProject?.id === chatId &&
+      curProject?.projectPath &&
+      !projectCompleted &&
+      !isProjectLoadedRef.current
+    ) {
+      setProgress(100);
+      setTimerActive(false);
+      setIsCompleting(false);
+      setProjectCompleted(true);
+      isProjectLoadedRef.current = true;
 
-      loadProjectFromChat();
-    } else {
-      setIsLoading(projectLoading);
+      try {
+        localStorage.setItem(`project-completed-${chatId}`, 'true');
+      } catch (e) {
+        logger.error('Failed to save project completion status:', e);
+      }
     }
-  }, [chatId, curProject, projectLoading, pollChatProject, projectCompleted]);
-
+  }, [curProject?.projectPath, chatId, projectCompleted]);
   // Use either curProject from context or locally polled project
   const activeProject = curProject || localProject;
 
@@ -148,6 +136,9 @@ export function CodeEngine({
       !isFileStructureLoading;
 
     if (shouldFetchFiles) {
+      setIsLoading(false);
+      setProjectCompleted(true);
+      isProjectLoadedRef.current = true;
       fetchFiles();
     }
   }, [
@@ -451,12 +442,22 @@ export function CodeEngine({
               )}
 
               <div className="w-64 flex flex-col items-center">
-                <p className="text-sm text-muted-foreground mb-2">
-                  {progress === 100
-                    ? 'Project ready!'
-                    : projectLoading
-                      ? 'Loading project...'
-                      : `Initializing project (${progress}%)`}
+                <p className="text-sm text-muted-foreground mb-2 text-center flex flex-col items-center">
+                  {progress === 100 ? (
+                    <span>Project ready!</span>
+                  ) : (
+                    <>
+                      {estimateTime > 0 ? (
+                        <span className="text-xs text-muted-foreground">
+                          Preparing your project (about 5-6 minutes)…
+                        </span>
+                      ) : (
+                        <span className="text-xs text-orange-500">
+                          Still working on it... thanks for your patience 🙏
+                        </span>
+                      )}
+                    </>
+                  )}
                 </p>
 
                 <div className="w-full bg-gray-200 dark:bg-gray-700 rounded-full h-2.5 mb-1">

frontend/src/components/global-toast-listener.tsx

@@ -6,6 +6,28 @@ import { ProjectContext } from './chat/code-engine/project-context';
 import { logger } from '@/app/log/logger';
 import { ProjectReadyToast } from './project-ready-toast';
 
+const COMPLETED_CACHE_KEY = 'completedChatIds';
+
+const getCompletedFromLocalStorage = (): Set<string> => {
+  try {
+    const raw = localStorage.getItem(COMPLETED_CACHE_KEY);
+    if (raw) {
+      return new Set(JSON.parse(raw));
+    }
+  } catch (e) {
+    logger.warn('Failed to read completedChatIds from localStorage');
+  }
+  return new Set();
+};
+
+const saveCompletedToLocalStorage = (set: Set<string>) => {
+  try {
+    localStorage.setItem(COMPLETED_CACHE_KEY, JSON.stringify(Array.from(set)));
+  } catch (e) {
+    logger.warn('Failed to save completedChatIds to localStorage');
+  }
+};
+
 const GlobalToastListener = () => {
   const {
     recentlyCompletedProjectId,
@@ -15,53 +37,52 @@ const GlobalToastListener = () => {
   } = useContext(ProjectContext);
   const router = useRouter();
   const intervalRef = useRef<NodeJS.Timeout | null>(null);
+  const completedIdsRef = useRef<Set<string>>(getCompletedFromLocalStorage());
 
-  // optional: if you use this in your logic
-  const setCurrentChatid = (id: string) => {}; // or import your actual setter
+  const setCurrentChatid = (id: string) => {};
 
   useEffect(() => {
-    if (!recentlyCompletedProjectId) return;
+    const chatId = recentlyCompletedProjectId;
 
-    const checkProjectReady = async () => {
+    if (!chatId || completedIdsRef.current.has(chatId)) return;
+
+    intervalRef.current = setInterval(async () => {
       try {
-        const project = await pollChatProject(recentlyCompletedProjectId);
+        const project = await pollChatProject(chatId);
 
         if (project?.projectPath) {
           toast.custom(
             (t) => (
               <ProjectReadyToast
-                chatId={recentlyCompletedProjectId}
+                chatId={chatId}
                 close={() => toast.dismiss(t)}
                 router={router}
                 setCurrentChatid={setCurrentChatid}
                 setChatId={setChatId}
               />
             ),
-            {
-              duration: 30000,
-            }
+            { duration: 10000 }
           );
 
+          completedIdsRef.current.add(chatId);
+          saveCompletedToLocalStorage(completedIdsRef.current);
+
           setRecentlyCompletedProjectId(null);
 
           if (intervalRef.current) {
             clearInterval(intervalRef.current);
             intervalRef.current = null;
           }
         } else {
-          logger.debug('Project not ready yet, will retry...');
+          logger.debug(`Chat ${chatId} not ready yet...`);
         }
-      } catch (err) {
-        logger.error('Error polling project status:', err);
+      } catch (e) {
+        logger.error('pollChatProject error:', e);
       }
-    };
-
-    intervalRef.current = setInterval(checkProjectReady, 5000);
+    }, 6000);
 
     return () => {
-      if (intervalRef.current) {
-        clearInterval(intervalRef.current);
-      }
+      if (intervalRef.current) clearInterval(intervalRef.current);
     };
   }, [recentlyCompletedProjectId]);
 

frontend/src/components/project-ready-toast.tsx

@@ -21,36 +21,58 @@ export const ProjectReadyToast = ({
 }: ProjectReadyToastProps) => {
   return (
     <motion.div
-      className="w-[360px] p-4 rounded-xl bg-green-50 dark:bg-green-900 text-green-800 dark:text-green-100 shadow-lg flex flex-col gap-2 relative"
-      initial={{ opacity: 0, y: 30, scale: 0.95 }}
+      initial={{ opacity: 0, y: 40, scale: 0.95 }}
       animate={{ opacity: 1, y: 0, scale: 1 }}
-      exit={{ opacity: 0, y: 30, scale: 0.95 }}
-      transition={{ type: 'spring', stiffness: 260, damping: 20 }}
+      exit={{ opacity: 0, y: 40, scale: 0.95 }}
+      transition={{
+        type: 'spring',
+        stiffness: 260,
+        damping: 20,
+        duration: 0.4,
+      }}
+      className="fixed bottom-24 right-6 z-[9999] w-[360px] px-5 py-4 
+                 bg-white dark:bg-zinc-900 text-gray-800 dark:text-gray-100 
+                 border border-gray-200 dark:border-zinc-700 
+                 rounded-lg shadow-2xl flex items-center justify-between gap-4"
     >
-      {/* Close button */}
-      <button
-        onClick={close}
-        className="absolute top-2 right-2 text-green-500 hover:text-green-700 dark:hover:text-green-300"
-      >
-        <X size={18} />
-      </button>
-
-      {/* Title */}
-      <p className="text-base font-semibold text-center mt-2">
-        🎉 Project is ready!
-      </p>
+      {/* Left: Icon + Text */}
+      <div className="flex items-center gap-2">
+        <div className="bg-green-100 dark:bg-green-800 text-green-600 dark:text-green-300 rounded-full p-1">
+          <svg
+            xmlns="http://www.w3.org/2000/svg"
+            className="w-4 h-4"
+            fill="none"
+            viewBox="0 0 24 24"
+            stroke="currentColor"
+          >
+            <path
+              strokeLinecap="round"
+              strokeLinejoin="round"
+              strokeWidth={2}
+              d="M5 13l4 4L19 7"
+            />
+          </svg>
+        </div>
+        <span className="text-sm font-medium">Project is ready!</span>
+      </div>
 
-      {/* Centered Button */}
-      <div className="flex justify-center mt-2">
+      {/* Right: Open Chat + Close */}
+      <div className="flex items-center gap-2">
         <button
           onClick={() => {
             redirectChatPage(chatId, setCurrentChatid, setChatId, router);
             close();
           }}
-          className="bg-green-600 hover:bg-green-700 text-white px-4 py-2 rounded-md text-sm font-medium transition-all"
+          className="text-sm px-3 py-1.5 rounded-md bg-green-600 hover:bg-green-700 text-white transition"
         >
           Open Chat
         </button>
+        <button
+          onClick={close}
+          className="text-gray-400 hover:text-gray-600 dark:hover:text-gray-300"
+        >
+          <X size={16} />
+        </button>
       </div>
     </motion.div>
   );