Pane state: skip FilePane MCP sync on Network volume

`FilePane.syncPaneStateToMcp` was pushing pane state even when the pane is on the virtual Network volume, where `NetworkMountView` / `NetworkBrowser` is the actual content owner. The push carried stale data: `files: []`, the old fixture `path`, and the leftover `totalFiles`/`loadedRange` from before the volume switch. Because `NetworkBrowser`'s own pane-state push (with the 14 host entries as `files`) raced FilePane's, and FilePane's typically pushed last, `cmdr://state` ended up frozen with empty `files`.

That's why three SMB tests (`guest/auth host shows share count`, `50-share host shows correct share count`) consistently hit their 30s `pollUntil` deadline on Linux: the state YAML never contained the host entries NetworkBrowser was correctly putting there. The tests had no `expect(result).toBe(true)` so the silent timeout looked like a pass — three dead assertions hiding a real bug.

Fix:
- Short-circuit `syncPaneStateToMcp` when `isNetworkView`. `NetworkBrowser` owns the push for that view (path `smb://`, files = hosts).
- Add the missing `expect(result).toBe(true)` to the three SMB tests so a real silent failure here fails the suite instead of disguising itself as a 30s pass.

After the fix the three tests drop from ~30.4s each (deadline hit) to ~400-800ms (actual pass). MTP is unaffected (its file list comes from a normal `list_directory`, FilePane's sync is the right source of truth there).

Author: vdavid

apps/desktop/src/lib/file-explorer/pane/FilePane.svelte

@@ -870,8 +870,24 @@
     /**
      * Sync pane state to Rust for MCP context tools.
      * Called when files load, cursor position changes, or view mode changes.
+     *
+     * Skipped entirely on the Network virtual volume: `NetworkBrowser`
+     * (mounted inside `NetworkMountView`) owns the pane-state push for that
+     * view and writes the host list as `files`. Without this guard, FilePane's
+     * own sync races NetworkBrowser's and overwrites it with stale local-pane
+     * data (empty `files`, the old fixture `path`, and a leftover
+     * `totalFiles`/`loadedRange`). That clobber is why three SMB tests
+     * (`guest host shows share count`, `auth host shows share count`,
+     * `50-share host shows correct share count`) used to time out at the 30s
+     * pollUntil deadline — `cmdr://state` never contained the host entries
+     * NetworkBrowser had just pushed.
+     *
+     * MTP volumes are not affected: their file list comes from a normal
+     * `list_directory` against the volume, so FilePane's sync is the right
+     * source of truth there.
      */
     async function syncPaneStateToMcp() {
+        if (isNetworkView) return
         try {
             const files = await buildMcpFileList()
             const effectiveTotal = hasParent ? totalCount + 1 : totalCount

apps/desktop/test/e2e-playwright/smb.spec.ts

@@ -204,14 +204,15 @@ describeSmb('SMB host discovery', () => {
     await mcpSelectVolume('left', 'Network')
 
     // Wait for the guest host to appear and its shares to be prefetched
-    await pollUntil(
+    const result = await pollUntil(
       tauriPage,
       async () => {
         const state = await mcpReadResource('cmdr://state')
         return state.includes('SMB Test (Guest)') && state.includes('shares=1')
       },
       30000,
     )
+    expect(result).toBe(true)
   })
 })
 
@@ -370,14 +371,15 @@ describeSmb('SMB authentication', () => {
     await mcpSelectVolume('left', 'Network')
 
     // Wait for the auth host to appear and its shares to be prefetched
-    await pollUntil(
+    const result = await pollUntil(
       tauriPage,
       async () => {
         const state = await mcpReadResource('cmdr://state')
         return state.includes('SMB Test (Auth)') && state.includes('shares=1')
       },
       30000,
     )
+    expect(result).toBe(true)
   })
 
   test('listing shares with valid credentials returns private share', async ({ tauriPage }) => {
@@ -437,14 +439,15 @@ describeSmb('SMB 50-share server', () => {
     await mcpSelectVolume('left', 'Network')
 
     // Wait for the 50-shares host to appear and prefetch shares
-    await pollUntil(
+    const result = await pollUntil(
       tauriPage,
       async () => {
         const state = await mcpReadResource('cmdr://state')
         return state.includes('SMB Test (50 Shares)') && state.includes('shares=50')
       },
       30000,
     )
+    expect(result).toBe(true)
   })
 })