@geminilight/mindos 0.6.54 → 0.6.56

package/_standalone/__tests__/settings/update-tab-desktop.test.tsx

@@ -126,4 +126,44 @@ describe('Desktop UpdateTab error handling', () => {
 
     expect(host.textContent).toContain("You're up to date");
   });
+
+  it('resets state to idle on mount to clear stale state after app restart', async () => {
+    // This test verifies the fix for: restart → 'state' still 'downloading' → button disabled
+    mockBridge.checkUpdate.mockResolvedValue({ available: true, version: '0.2.0' });
+
+    const { UpdateTab } = await import('@/components/settings/UpdateTab');
+
+    // First render (before restart)
+    await act(async () => { root.render(<UpdateTab />); });
+
+    const updateBtn = Array.from(host.querySelectorAll('button'))
+      .find(b => b.textContent?.includes('Update to v0.2.0'));
+    expect(updateBtn).toBeTruthy();
+    expect((updateBtn as HTMLButtonElement).disabled).toBe(false);
+
+    // User clicks update, state becomes 'downloading'
+    await act(async () => { updateBtn!.click(); });
+    await act(async () => { await Promise.resolve(); });
+
+    // In real scenario, app restarts here and component remounts
+    // Unmount and create new root to simulate fresh load
+    await act(async () => { root.unmount(); });
+    host.remove();
+
+    const newHost = document.createElement('div');
+    document.body.appendChild(newHost);
+    const newRoot = createRoot(newHost);
+
+    await act(async () => { newRoot.render(<UpdateTab />); });
+
+    // CRITICAL: After remount, the "Check for Updates" button should be ENABLED
+    // (not disabled like before the fix)
+    const checkBtn = Array.from(newHost.querySelectorAll('button'))
+      .find(b => b.textContent?.includes('Check for Updates'));
+    expect(checkBtn).toBeTruthy();
+    expect((checkBtn as HTMLButtonElement).disabled).toBe(false); // Must be enabled!
+
+    await act(async () => { newRoot.unmount(); });
+    newHost.remove();
+  });
 });

package/_standalone/components/Breadcrumb.tsx

@@ -13,23 +13,27 @@ function FileTypeIcon({ name }: { name: string }) {
 export default function Breadcrumb({ filePath }: { filePath: string }) {
   const parts = filePath.split('/');
   return (
-    <nav className="flex items-center gap-1 text-xs text-muted-foreground flex-wrap">
-      <Link href="/" className="hover:text-foreground transition-colors">
+    <nav className="flex items-center gap-0.5 text-xs text-muted-foreground flex-wrap">
+      <Link
+        href="/"
+        className="p-1.5 rounded-md hover:bg-muted/50 hover:text-foreground transition-colors"
+        title="Home"
+      >
         <Home size={14} />
       </Link>
       {parts.map((part, i) => {
         const isLast = i === parts.length - 1;
         const href = '/view/' + parts.slice(0, i + 1).map(encodeURIComponent).join('/');
         return (
-          <span key={i} className="flex items-center gap-1">
-            <ChevronRight size={12} className="text-muted-foreground/50" />
+          <span key={i} className="flex items-center gap-0.5">
+            <ChevronRight size={12} className="text-muted-foreground/50 shrink-0" />
             {isLast ? (
-              <span className="flex items-center gap-1.5 text-foreground font-medium">
+              <span className="flex items-center gap-1.5 px-2 py-1 text-foreground font-medium">
                 <FileTypeIcon name={part} />
                 <span suppressHydrationWarning>{part}</span>
               </span>
             ) : (
-              <Link href={href} className="px-2 py-0.5 rounded-md hover:bg-muted/50 transition-colors truncate max-w-[200px]" title={part}>
+              <Link href={href} className="px-2 py-1 rounded-md hover:bg-muted/50 hover:text-foreground transition-colors truncate max-w-[200px]" title={part}>
                 <span suppressHydrationWarning>{part}</span>
               </Link>
             )}

package/_standalone/components/settings/UpdateTab.tsx

@@ -108,7 +108,15 @@ function DesktopCoreCard() {
       // Get current Core version via a check call (also probes for updates)
       try {
         const info = await bridge.checkCoreUpdate?.();
-        if (info) setCurrentVersion(info.currentVersion);
+        if (info) {
+          setCurrentVersion(info.currentVersion);
+          // CRITICAL FIX: If no update available, explicitly set state to 'idle'
+          // This ensures the UI refreshes to show the new (current) version
+          if (!info.available) {
+            setState('idle');
+            return;
+          }
+        }
       } catch { /* ignore */ }
 
       // Check for pending download (from previous session)
@@ -196,7 +204,11 @@ function DesktopCoreCard() {
     setState('applying');
     try {
       const result = await bridge.applyCoreUpdate();
-      if (result?.version) setCurrentVersion(result.version);
+      // CRITICAL FIX: Update currentVersion to the version returned by the main process
+      // This ensures the UI immediately shows the new version after the update is applied
+      if (result?.version) {
+        setCurrentVersion(result.version);
+      }
       setState('idle');
       // Page will be reloaded by main process
     } catch (err) {
@@ -346,6 +358,10 @@ function DesktopShellCard() {
   const [errorMsg, setErrorMsg] = useState('');
 
   useEffect(() => {
+    // CRITICAL: Reset state on mount to clear stale state after app restart
+    // (without this, 'state' can be 'downloading' from before the restart)
+    setState('idle');
+    
     bridge.getAppInfo?.().then((info) => {
       if (info?.version) setAppVersion(info.version);
     }).catch((err) => { console.warn("[UpdateTab] getAppInfo failed:", err); });