Fix crash on workspace delete and hash restore priority

- Add null check in workspace sort comparison to prevent crash when workspace is deleted
- Fix hash restore: now takes priority over localStorage by running once on mount
- Add guard to prevent duplicate 'Updating workspace' log messages

Author: ammar-agent

src/App.tsx

@@ -224,12 +224,15 @@ function AppInner() {
   }, [selectedWorkspace]);
 
   // Restore workspace from URL on mount (if valid)
+  // This effect runs once on mount to restore from hash, which takes priority over localStorage
+  const [hasRestoredFromHash, setHasRestoredFromHash] = useState(false);
+  
   useEffect(() => {
+    // Only run once
+    if (hasRestoredFromHash) return;
+    
     // Wait for metadata to load before attempting restore
     if (workspaceMetadata.size === 0) return;
-    
-    // Only restore once - if workspace is already selected, skip
-    if (selectedWorkspace) return;
 
     const hash = window.location.hash;
     if (hash.startsWith("#workspace=")) {
@@ -239,7 +242,7 @@ function AppInner() {
       const metadata = workspaceMetadata.get(workspaceId);
 
       if (metadata) {
-        // Find project for this workspace (metadata now includes projectPath)
+        // Restore from hash (overrides localStorage)
         setSelectedWorkspace({
           workspaceId: metadata.id,
           projectPath: metadata.projectPath,
@@ -248,7 +251,9 @@ function AppInner() {
         });
       }
     }
-  }, [workspaceMetadata, selectedWorkspace, setSelectedWorkspace]);
+    
+    setHasRestoredFromHash(true);
+  }, [workspaceMetadata, hasRestoredFromHash, setSelectedWorkspace]);
 
   // Validate selected workspace exists and has all required fields
   useEffect(() => {
@@ -264,8 +269,8 @@ function AppInner() {
         if (window.location.hash) {
           window.history.replaceState(null, "", window.location.pathname);
         }
-      } else if (!selectedWorkspace.namedWorkspacePath) {
-        // Old localStorage entry missing namedWorkspacePath - update it
+      } else if (!selectedWorkspace.namedWorkspacePath && metadata.namedWorkspacePath) {
+        // Old localStorage entry missing namedWorkspacePath - update it once
         console.log(
           `Updating workspace ${selectedWorkspace.workspaceId} with missing fields`
         );
@@ -377,9 +382,11 @@ function AppInner() {
         !compareMaps(prev, next, (a, b) => {
           if (a.length !== b.length) return false;
           // Check both ID and name to detect renames
-          return a.every((metadata, i) => 
-            metadata.id === b[i].id && metadata.name === b[i].name
-          );
+          return a.every((metadata, i) => {
+            const bMeta = b[i];
+            if (!bMeta) return false;
+            return metadata.id === bMeta.id && metadata.name === bMeta.name;
+          });
         })
       ) {
         return false;