From b2c16002e49056a154ff6c67ac342d686705f870 Mon Sep 17 00:00:00 2001
From: Sayo <ws11579@gmail.com>
Date: Mon, 3 Nov 2025 03:17:54 +0800
Subject: [PATCH 1/3] fix: fix image viewer return 401 error

---
 src/components/ImageViewer.jsx | 92 +++++++++++++++++++++++++++-------
 1 file changed, 74 insertions(+), 18 deletions(-)

diff --git a/src/components/ImageViewer.jsx b/src/components/ImageViewer.jsx
index b5077ae..15401a0 100644
--- a/src/components/ImageViewer.jsx
+++ b/src/components/ImageViewer.jsx
@@ -1,9 +1,63 @@
-import React from 'react';
+import React, { useEffect, useState } from 'react';
 import { Button } from './ui/button';
 import { X } from 'lucide-react';
 
 function ImageViewer({ file, onClose }) {
   const imagePath = `/api/projects/${file.projectName}/files/content?path=${encodeURIComponent(file.path)}`;
+  const [imageUrl, setImageUrl] = useState(null);
+  const [error, setError] = useState(null);
+  const [loading, setLoading] = useState(true);
+
+  useEffect(() => {
+    let objectUrl;
+    const controller = new AbortController();
+
+    const loadImage = async () => {
+      try {
+        setLoading(true);
+        setError(null);
+        setImageUrl(null);
+
+        const token = localStorage.getItem('auth-token');
+        if (!token) {
+          setError('Missing authentication token');
+          return;
+        }
+
+        const response = await fetch(imagePath, {
+          headers: {
+            'Authorization': `Bearer ${token}`
+          },
+          signal: controller.signal
+        });
+
+        if (!response.ok) {
+          throw new Error(`Request failed with status ${response.status}`);
+        }
+
+        const blob = await response.blob();
+        objectUrl = URL.createObjectURL(blob);
+        setImageUrl(objectUrl);
+      } catch (err) {
+        if (err.name === 'AbortError') {
+          return;
+        }
+        console.error('Error loading image:', err);
+        setError('Unable to load image');
+      } finally {
+        setLoading(false);
+      }
+    };
+
+    loadImage();
+
+    return () => {
+      controller.abort();
+      if (objectUrl) {
+        URL.revokeObjectURL(objectUrl);
+      }
+    };
+  }, [imagePath]);
 
   return (
     <div className="fixed inset-0 bg-black bg-opacity-50 flex items-center justify-center z-50">
@@ -23,22 +77,24 @@ function ImageViewer({ file, onClose }) {
         </div>
 
         <div className="p-4 flex justify-center items-center bg-gray-50 dark:bg-gray-900 min-h-[400px]">
-          <img
-            src={imagePath}
-            alt={file.name}
-            className="max-w-full max-h-[70vh] object-contain rounded-lg shadow-md"
-            onError={(e) => {
-              e.target.style.display = 'none';
-              e.target.nextSibling.style.display = 'block';
-            }}
-          />
-          <div
-            className="text-center text-gray-500 dark:text-gray-400"
-            style={{ display: 'none' }}
-          >
-            <p>Unable to load image</p>
-            <p className="text-sm mt-2">{file.path}</p>
-          </div>
+          {loading && (
+            <div className="text-center text-gray-500 dark:text-gray-400">
+              <p>Loading image…</p>
+            </div>
+          )}
+          {!loading && imageUrl && (
+            <img
+              src={imageUrl}
+              alt={file.name}
+              className="max-w-full max-h-[70vh] object-contain rounded-lg shadow-md"
+            />
+          )}
+          {!loading && !imageUrl && (
+            <div className="text-center text-gray-500 dark:text-gray-400">
+              <p>{error || 'Unable to load image'}</p>
+              <p className="text-sm mt-2 break-all">{file.path}</p>
+            </div>
+          )}
         </div>
 
         <div className="p-4 border-t bg-gray-50 dark:bg-gray-800">
@@ -51,4 +107,4 @@ function ImageViewer({ file, onClose }) {
   );
 }
 
-export default ImageViewer;
\ No newline at end of file
+export default ImageViewer;

From c875907f55b1382061841bacab009658c5ac57f5 Mon Sep 17 00:00:00 2001
From: LeoZheng1738 <leo.zheng@shareworks.cn>
Date: Tue, 4 Nov 2025 11:33:58 +0800
Subject: [PATCH 2/3] fix(Sidebar): The undefined setShowSuggestions method has
 been removed.

---
 src/components/Sidebar.jsx | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/components/Sidebar.jsx b/src/components/Sidebar.jsx
index cda6c5a..f5c3e06 100644
--- a/src/components/Sidebar.jsx
+++ b/src/components/Sidebar.jsx
@@ -484,7 +484,6 @@ function Sidebar({
         
         setShowNewProject(false);
         setNewProjectPath('');
-        setShowSuggestions(false);
         
         // Refresh projects to show the new one
         if (window.refreshProjects) {
@@ -507,7 +506,6 @@ function Sidebar({
   const cancelNewProject = () => {
     setShowNewProject(false);
     setNewProjectPath('');
-    setShowSuggestions(false);
   };
 
   const loadMoreSessions = async (project) => {

From b416e542c7835c9d37f2266044b95a53b6bf5659 Mon Sep 17 00:00:00 2001
From: viper151 <simosmik@gmail.com>
Date: Tue, 4 Nov 2025 09:39:37 +0100
Subject: [PATCH 3/3] Apply suggestion from @coderabbitai[bot]

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
---
 server/routes/projects.js | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/server/routes/projects.js b/server/routes/projects.js
index 665f094..ff79671 100644
--- a/server/routes/projects.js
+++ b/server/routes/projects.js
@@ -101,7 +101,12 @@ router.post('/create-workspace', async (req, res) => {
           await cloneGitHubRepository(githubUrl, absolutePath, githubToken);
         } catch (error) {
           // Clean up created directory on failure
-          await fs.rm(absolutePath, { recursive: true, force: true });
+          try {
+            await fs.rm(absolutePath, { recursive: true, force: true });
+          } catch (cleanupError) {
+            console.error('Failed to clean up directory after clone failure:', cleanupError);
+            // Continue to throw original error
+          }
           throw new Error(`Failed to clone repository: ${error.message}`);
         }
       }