Go to path: Dialog, handler, and wiring

- Extract the shared "jump into a pane" primitives into `lib/file-explorer/navigation/navigate-and-select.ts` (`navigateToDirInPane` / `navigateToFileInPane`, preserving the `navigateToPath` sync-error-string handling verbatim) and repoint `lib/downloads/go-to-latest.ts` at them — behavior-preserving, `go-to-latest.test.ts` still green.
- Add the `lib/go-to-path/` feature: `goToPath` handler (switches on the typed `GoToPathResolution`, navigates, records the resolved target into recents, fires a snapshotted-`nav.back`-shortcut ancestor toast), the pure helpers `digitToRecentIndex` + `shouldPrefillClipboard`, the `recent-paths-state` `$state` mirror, `GoToPathDialog.svelte` (clipboard prefill, digit-jump recents, live ancestor warning, `[x]` remove), and `GoToPathAncestorToastContent.svelte`.
- Wire `nav.goToPath` (⌘G) into `command-registry.ts` + `command-dispatch.ts` + `+page.svelte` (mount, close, `isModalDialogOpen`, and the `showGoToPathDialog` idempotency guard for the menu double-dispatch). Register the `go-to-path` dialog id.
- Vitest: handler per-`kind` + dynamic-shortcut proof, `digitToRecentIndex` / `shouldPrefillClipboard` tables, recents mirror add/dedupe/move-to-top/cap/remove, navigate-and-select primitives, dialog behavior, and toast + dialog a11y.
- Docs: `lib/go-to-path/CLAUDE.md`, the `navigation/` row, and the frontend `go-to-path/` row in `architecture.md`.

Author: vdavid

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

@@ -68,6 +68,18 @@ export const commands: Command[] = [
     shortcuts: ['⌘F', '⌥F7'],
   },
 
+  // ============================================================================
+  // Main window - Navigation (Go to path)
+  // ============================================================================
+  {
+    id: 'nav.goToPath',
+    name: 'Go to path…',
+    scope: 'Main window',
+    showInPalette: true,
+    shortcuts: ['⌘G'],
+    description: 'Jump the focused pane to a typed, pasted, or recent path.',
+  },
+
   // ============================================================================
   // Main window - Downloads
   // ============================================================================

apps/desktop/src/lib/downloads/go-to-latest.ts

@@ -12,6 +12,7 @@
 import { commands } from '$lib/ipc/bindings'
 import { addToast } from '$lib/ui/toast'
 import { getAppLogger } from '$lib/logging/logger'
+import { navigateToFileInPane } from '$lib/file-explorer/navigation/navigate-and-select'
 import type { ExplorerAPI } from '../../routes/(main)/explorer-api'
 
 import LatestDownloadEmptyToastContent from './LatestDownloadEmptyToastContent.svelte'
@@ -43,7 +44,7 @@ export async function goToLatestDownload(explorer: ExplorerAPI | undefined): Pro
 
   const result = await commands.goToLatestDownload()
   if (result.status === 'ok') {
-    await navigateToDownloadFile(explorer, result.data.parentDir, result.data.fileName)
+    await navigateToFileInPane(explorer, explorer.getFocusedPane(), result.data.parentDir, result.data.fileName)
     return
   }
 
@@ -82,27 +83,7 @@ export async function goToDownload(
     log.debug('goToDownload: no explorer; skipping (HMR or pre-mount)')
     return
   }
-  await navigateToDownloadFile(explorer, parentDir, fileName)
-}
-
-async function navigateToDownloadFile(explorer: ExplorerAPI, parentDir: string, fileName: string): Promise<void> {
-  const pane = explorer.getFocusedPane()
-  // `navigateToPath` returns a sync error string when navigation can't even
-  // start (snapshot pane on a missing volume, etc.); otherwise it returns a
-  // Promise that settles when the listing completes. We await the Promise
-  // but report-and-bail on the sync-error string — without the listing
-  // settled, `moveCursor` would race against an empty cache.
-  const navResult = explorer.navigateToPath(pane, parentDir)
-  if (typeof navResult === 'string') {
-    log.warn('goToDownload: navigateToPath refused {pane} {parentDir}: {reason}', {
-      pane,
-      parentDir,
-      reason: navResult,
-    })
-    return
-  }
-  await navResult
-  await explorer.moveCursor(pane, fileName)
+  await navigateToFileInPane(explorer, explorer.getFocusedPane(), parentDir, fileName)
 }
 
 async function showEmptyToast(explorer: ExplorerAPI): Promise<void> {

apps/desktop/src/lib/file-explorer/navigation/CLAUDE.md

@@ -4,22 +4,23 @@ Browser-style back/forward history, path resolution, paged keyboard shortcuts, a
 
 ## Key files
 
-| File                                   | Purpose                                                                                                                                                            |
-| -------------------------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------ |
-| `navigation-history.ts`                | Purely functional immutable history stack                                                                                                                          |
-| `path-navigation.ts`                   | Picks initial path when switching volumes                                                                                                                          |
-| `path-resolution.ts`                   | Walk-up `resolveValidPath` (split out to break cycle)                                                                                                              |
-| `path-segments.ts`                     | Splits the breadcrumb display path into segments and flags any inside a `.git/...` portal (consumer: `FilePane.svelte` paints them with `--color-git-portal-text`) |
-| `keyboard-shortcuts.ts`                | Home/End/PageUp/PageDown handling for file lists                                                                                                                   |
-| `VolumeBreadcrumb.svelte`              | Clickable volume label + grouped dropdown                                                                                                                          |
-| `volume-grouping.ts`                   | Pure logic: group volumes by category, get volume icons                                                                                                            |
-| `volume-space-manager.svelte.ts`       | Reactive state machine for disk space fetch/retry/timeout                                                                                                          |
-| `volume-breadcrumb-handlers.svelte.ts` | Submenu/breadcrumb-popup controllers, keyboard-mode tracker, and pure key-dispatch helpers for `VolumeBreadcrumb.svelte`                                           |
-| `eject-predicate.ts`                   | Pure `isVolumeEjectable(volume)` used by the eject button gate. Returns true when NSURL says ejectable OR the volume has any SMB connection state                  |
-| `navigation-history.test.ts`           | Full unit test coverage of history functions                                                                                                                       |
-| `path-navigation.test.ts`              | Unit tests for path resolution and timeouts                                                                                                                        |
-| `keyboard-shortcuts.test.ts`           | Unit tests for shortcut calculations                                                                                                                               |
-| `path-segments.test.ts`                | Unit tests for git-portal segment detection                                                                                                                        |
+| File                                   | Purpose                                                                                                                                                                          |
+| -------------------------------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
+| `navigation-history.ts`                | Purely functional immutable history stack                                                                                                                                        |
+| `path-navigation.ts`                   | Picks initial path when switching volumes                                                                                                                                        |
+| `navigate-and-select.ts`               | Shared "jump into a pane" primitives (`navigateToDirInPane` / `navigateToFileInPane`) used by Go-to-latest-download and Go-to-path; handles `navigateToPath`'s sync-error string |
+| `path-resolution.ts`                   | Walk-up `resolveValidPath` (split out to break cycle)                                                                                                                            |
+| `path-segments.ts`                     | Splits the breadcrumb display path into segments and flags any inside a `.git/...` portal (consumer: `FilePane.svelte` paints them with `--color-git-portal-text`)               |
+| `keyboard-shortcuts.ts`                | Home/End/PageUp/PageDown handling for file lists                                                                                                                                 |
+| `VolumeBreadcrumb.svelte`              | Clickable volume label + grouped dropdown                                                                                                                                        |
+| `volume-grouping.ts`                   | Pure logic: group volumes by category, get volume icons                                                                                                                          |
+| `volume-space-manager.svelte.ts`       | Reactive state machine for disk space fetch/retry/timeout                                                                                                                        |
+| `volume-breadcrumb-handlers.svelte.ts` | Submenu/breadcrumb-popup controllers, keyboard-mode tracker, and pure key-dispatch helpers for `VolumeBreadcrumb.svelte`                                                         |
+| `eject-predicate.ts`                   | Pure `isVolumeEjectable(volume)` used by the eject button gate. Returns true when NSURL says ejectable OR the volume has any SMB connection state                                |
+| `navigation-history.test.ts`           | Full unit test coverage of history functions                                                                                                                                     |
+| `path-navigation.test.ts`              | Unit tests for path resolution and timeouts                                                                                                                                      |
+| `keyboard-shortcuts.test.ts`           | Unit tests for shortcut calculations                                                                                                                                             |
+| `path-segments.test.ts`                | Unit tests for git-portal segment detection                                                                                                                                      |
 
 ## `navigation-history.ts`
 

apps/desktop/src/lib/file-explorer/navigation/navigate-and-select.test.ts

@@ -0,0 +1,66 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest'
+import { navigateToDirInPane, navigateToFileInPane } from './navigate-and-select'
+import type { ExplorerAPI } from '../../../routes/(main)/explorer-api'
+
+function makeExplorer(navResult: string | Promise<void>) {
+  const navigateToPath = vi.fn(() => navResult)
+  const moveCursor = vi.fn(() => Promise.resolve())
+  const explorer = { navigateToPath, moveCursor } as unknown as ExplorerAPI
+  return { explorer, navigateToPath, moveCursor }
+}
+
+describe('navigateToDirInPane', () => {
+  it('navigates to the dir and never moves the cursor', async () => {
+    const { explorer, navigateToPath, moveCursor } = makeExplorer(Promise.resolve())
+    await navigateToDirInPane(explorer, 'left', '/tmp/here')
+    expect(navigateToPath).toHaveBeenCalledWith('left', '/tmp/here')
+    expect(moveCursor).not.toHaveBeenCalled()
+  })
+
+  it('bails on the sync-error string without throwing', async () => {
+    const { explorer, moveCursor } = makeExplorer('snapshot pane on a missing volume')
+    await expect(navigateToDirInPane(explorer, 'right', '/tmp')).resolves.toBeUndefined()
+    expect(moveCursor).not.toHaveBeenCalled()
+  })
+})
+
+describe('navigateToFileInPane', () => {
+  let moveCursorCalls: unknown[][]
+
+  beforeEach(() => {
+    moveCursorCalls = []
+  })
+
+  it('navigates to the parent, then moves the cursor onto the file', async () => {
+    const { explorer, navigateToPath, moveCursor } = makeExplorer(Promise.resolve())
+    await navigateToFileInPane(explorer, 'left', '/tmp', 'a.txt')
+    expect(navigateToPath).toHaveBeenCalledWith('left', '/tmp')
+    expect(moveCursor).toHaveBeenCalledWith('left', 'a.txt')
+  })
+
+  it('awaits the listing before moving the cursor', async () => {
+    let resolveListing!: () => void
+    const listing = new Promise<void>((resolve) => {
+      resolveListing = resolve
+    })
+    const navigateToPath = vi.fn(() => listing)
+    const moveCursor = vi.fn(() => {
+      moveCursorCalls.push(['called'])
+      return Promise.resolve()
+    })
+    const explorer = { navigateToPath, moveCursor } as unknown as ExplorerAPI
+
+    const promise = navigateToFileInPane(explorer, 'left', '/tmp', 'a.txt')
+    // Cursor must NOT move before the listing settles.
+    expect(moveCursorCalls).toHaveLength(0)
+    resolveListing()
+    await promise
+    expect(moveCursorCalls).toHaveLength(1)
+  })
+
+  it('bails on the sync-error string and never moves the cursor', async () => {
+    const { explorer, moveCursor } = makeExplorer('cannot start')
+    await navigateToFileInPane(explorer, 'left', '/tmp', 'a.txt')
+    expect(moveCursor).not.toHaveBeenCalled()
+  })
+})

apps/desktop/src/lib/file-explorer/navigation/navigate-and-select.ts

@@ -0,0 +1,60 @@
+/**
+ * Shared navigation primitives for the "jump somewhere in the focused pane"
+ * features: "Go to latest download" (⌘J) and "Go to path" (⌘G). Both want to
+ * point a pane at a directory and (for files) land the cursor on a specific
+ * entry, with the same careful handling of `navigateToPath`'s
+ * `string | Promise<void>` return.
+ *
+ * `navigateToPath` returns a sync error string when navigation can't even
+ * start (a snapshot pane on a missing volume, etc.); otherwise it returns a
+ * Promise that settles when the listing completes. We await the Promise but
+ * report-and-bail on the sync-error string — without the listing settled,
+ * `moveCursor` would race against an empty cache.
+ */
+
+import { getAppLogger } from '$lib/logging/logger'
+import type { ExplorerAPI } from '../../../routes/(main)/explorer-api'
+
+const log = getAppLogger('navigation')
+
+type Pane = 'left' | 'right'
+
+/**
+ * Navigate `pane` to a directory. No cursor move — the directory's own normal
+ * navigation lands the cursor on the 0th row (`..`).
+ */
+export async function navigateToDirInPane(explorer: ExplorerAPI, pane: Pane, dir: string): Promise<void> {
+  const navResult = explorer.navigateToPath(pane, dir)
+  if (typeof navResult === 'string') {
+    log.warn('navigateToDirInPane: navigateToPath refused {pane} {dir}: {reason}', {
+      pane,
+      dir,
+      reason: navResult,
+    })
+    return
+  }
+  await navResult
+}
+
+/**
+ * Navigate `pane` to `parentDir`, then move the cursor onto `fileName` so the
+ * file is revealed/selected (we do NOT open it).
+ */
+export async function navigateToFileInPane(
+  explorer: ExplorerAPI,
+  pane: Pane,
+  parentDir: string,
+  fileName: string,
+): Promise<void> {
+  const navResult = explorer.navigateToPath(pane, parentDir)
+  if (typeof navResult === 'string') {
+    log.warn('navigateToFileInPane: navigateToPath refused {pane} {parentDir}: {reason}', {
+      pane,
+      parentDir,
+      reason: navResult,
+    })
+    return
+  }
+  await navResult
+  await explorer.moveCursor(pane, fileName)
+}