[rodrigogs]

Root cause

FileDropZone.svelte's handleDrop awaited Promise.all(getAsFileSystemHandle()) before reading e.dataTransfer.files. Chromium invalidates DataTransfer access as soon as the drop handler's first await resolves (drag_controller.cc sets the access policy to kNumb on return), so .files returns an empty list past that point — drops silently did nothing in Chrome/Edge/Brave/Electron while the file picker (no intervening await) kept working. Introduced in #66 when the handler became async; the existing comment only protected .items, not .files.

Fix

Snapshot dt.files and start the handle promises synchronously, before any await, in a new pure helper snapshotDroppedFiles (src/lib/helpers/drop-files.ts). handleDrop is now a thin caller. onFilesSelected's prop type moved from FileList to readonly File[]; updated the three +page.svelte callers and dropped the new DataTransfer() round-trips that only existed to keep producing a FileList.

Second commit: getElectronFilePath relied on File#path, removed in Electron 32 (this project ships electron ^39.8.6), so dropped/picked files in the desktop app never got a path for persistence. Now prefers window.electronAPI.getPathForFile (backed by webUtils.getPathForFile, exposed via contextBridge in the preload script), falling back to the old 'path' in file check. Not runnable on the CI/dev host that produced this PR (no display server) — verified via lint/check/build/unit tests only.

Mutation evidence (RED → GREEN) for tests/drop-zone/snapshot-dropped-files.test.ts

RED — temporarily changed snapshotDroppedFiles to await Promise.resolve() before reading dt.files, reproducing the bug the helper exists to prevent:

 FAIL  tests/drop-zone/snapshot-dropped-files.test.ts > snapshotDroppedFiles > captures files BEFORE they become invalid across async boundaries
AssertionError: expected +0 to equal 1
 ❯ tests/drop-zone/snapshot-dropped-files.test.ts:86:24
     85|   assert.equal(snapshot.files.length, 1);

 FAIL  tests/drop-zone/snapshot-dropped-files.test.ts > snapshotDroppedFiles > returns an array of files (not a FileList) for immutability
AssertionError: The expression evaluated to a falsy value: (Array.isArray(snapshot.files))
 ❯ tests/drop-zone/snapshot-dropped-files.test.ts:106:3

 FAIL  tests/drop-zone/snapshot-dropped-files.test.ts > snapshotDroppedFiles > handles multiple dropped files
TypeError: Cannot read properties of undefined (reading 'length')
 ❯ tests/drop-zone/snapshot-dropped-files.test.ts:117:31

 FAIL  tests/drop-zone/snapshot-dropped-files.test.ts > snapshotDroppedFiles > handles empty drop
TypeError: Cannot read properties of undefined (reading 'length')
 ❯ tests/drop-zone/snapshot-dropped-files.test.ts:127:31

 Test Files  1 failed (1)
      Tests  4 failed (4)

(The mutation makes snapshotDroppedFiles return a Promise instead of the snapshot object, since the fake DataTransfer in the test isn't awaited by the test's synchronous calls — which is itself proof the mutation broke the "no await before reading files" contract the test enforces.)

GREEN — reverted the mutation:

 Test Files  1 passed (1)
      Tests  4 passed (4)
   Duration  157ms

Gates run locally

• npm ci — clean install
• npm run lint — Checked 134 files, no fixes applied
• npm run check — svelte-check found 0 errors and 0 warnings
• npm test — Test Files 5 passed (5), Tests 61 passed (61)
• npm run build — succeeds

npm run test:global-search needs Playwright browsers, which cannot launch on this host (no display) — left to CI to prove that leg.

Closes #91