Network: ⌘↑ in the share list goes back to the host list, like Backspace

In the SMB share browser (a host's share listing), `Escape` and `Backspace` already returned to the host list, but `⌘↑` fell into the cursor-move arrow handler instead — inconsistent with the file list, where `⌘↑` = parent. `⌘↑` now also goes back, via a shared `handleBackToHostKey` helper (Escape / Backspace / ⌘↑), checked before the arrow handler.

Does NOT cover the related dead-end where parent-nav from inside a *mounted* SMB share's root returns nothing (the file browser, not the share list) — that needs reconstructing the `NetworkHost` from the mounted volume's server and a cross-volume switch into the virtual `network` volume. Separate, riskier change.

Author: vdavid

apps/desktop/src/lib/commands/command-registry.ts

@@ -844,7 +844,9 @@ const commandSources: CommandSource[] = [
     nameKey: 'commands.shareBack.label',
     scope: 'Main window/Share browser',
     showInPalette: true,
-    shortcuts: ['Backspace', 'Escape'],
+    // `⌘↑` mirrors the file list's `⌘↑` = parent; ShareBrowser handles all three
+    // keys (`handleBackToHostKey`). Display-only — `fixedKey` handling is in-component.
+    shortcuts: ['Backspace', 'Escape', '⌘↑'],
     fixedKey: true,
   },
   {

apps/desktop/src/lib/file-explorer/network/ShareBrowser.svelte

@@ -448,6 +448,20 @@
         return e.metaKey && (e.key === 'ArrowLeft' || e.key === 'ArrowRight')
     }
 
+    /**
+     * Escape, Backspace, and `⌘↑` all return to the host list. `⌘↑` mirrors the
+     * file list's `⌘↑` = parent, and must be handled before the arrow handler,
+     * which would otherwise treat it as a cursor move.
+     */
+    function handleBackToHostKey(e: KeyboardEvent): boolean {
+        if (e.key === 'Escape' || e.key === 'Backspace' || (e.key === 'ArrowUp' && e.metaKey)) {
+            e.preventDefault()
+            onBack?.()
+            return true
+        }
+        return false
+    }
+
     export function handleKeyDown(e: KeyboardEvent): boolean {
         if (showLoginForm) {
             // Login form handles its own keyboard events
@@ -460,6 +474,9 @@
 
         if (sortedShares.length === 0) return false
 
+        // Escape / Backspace / ⌘↑ → back to the host list (before the arrow handler).
+        if (handleBackToHostKey(e)) return true
+
         // Try centralized navigation shortcuts first (PageUp, PageDown, Home, End, Option+arrows)
         const visibleItems = Math.max(1, Math.floor(containerHeight / SHARE_ROW_HEIGHT))
         const navResult = handleNavigationShortcut(e, {
@@ -489,12 +506,6 @@
             return true
         }
 
-        if (e.key === 'Escape' || e.key === 'Backspace') {
-            e.preventDefault()
-            onBack?.()
-            return true
-        }
-
         return false
     }
 

apps/desktop/src/lib/file-explorer/network/ShareBrowser.test.ts

@@ -64,6 +64,7 @@ const naspi: ShareInfo = { name: 'naspi', isDisk: true, comment: null }
 /** The exported ShareBrowser API surface the tests drive. */
 interface ShareBrowserApi {
   openCursorItem: () => void
+  handleKeyDown: (e: KeyboardEvent) => boolean
 }
 
 function mountBrowser(
@@ -146,3 +147,28 @@ describe('ShareBrowser credential gate', () => {
     await unmount(component)
   })
 })
+
+describe('ShareBrowser back-navigation', () => {
+  beforeEach(() => {
+    vi.clearAllMocks()
+    document.body.innerHTML = ''
+    h.getSmbCredentials.mockRejectedValue(new Error('not found'))
+  })
+
+  it('⌘↑ goes back to the host list (like Escape / Backspace), not a cursor move', async () => {
+    h.fetchShares.mockResolvedValue({ shares: [naspi], authMode: 'guest_allowed', fromCache: false })
+    const onShareSelect = vi.fn()
+    const onBack = vi.fn()
+    const { target, component, api } = mountBrowser(onShareSelect, onBack)
+    await waitForShareList(target)
+
+    const handled = api.handleKeyDown(new KeyboardEvent('keydown', { key: 'ArrowUp', metaKey: true }))
+
+    expect(handled).toBe(true)
+    expect(onBack).toHaveBeenCalledOnce()
+    // The cursor-move path must NOT have fired (no share got activated).
+    expect(onShareSelect).not.toHaveBeenCalled()
+
+    await unmount(component)
+  })
+})