Pass non-empty directory information to pendingDuplicates() and sendToSurge()

Greptile Summary

This PR fixes a silent bug where the Firefox extension always sent an empty directory string to Surge, causing every download to land in Surge's default directory instead of the user-selected location.

The fix is minimal and correct: handleDownloadIntercept already called extractPathInfo to parse downloadItem.filename into { filename, directory }, but the extracted directory was discarded in both the duplicate-confirmation path (directory: '' hardcoded in pendingDuplicates.set) and the normal download path (only filename was destructured before calling sendToSurge). Both call-sites now properly forward the extracted directory.

• extractPathInfo correctly handles Windows (C:/…) and POSIX (/…) absolute paths and is unchanged.
• sendToSurge already guards the path field with if (absolutePath), so downloads with no directory information still fall back to Surge's default — no regression risk.
• The duplicate-confirmation handler (confirmDuplicate message case) also reads pending.directory when calling sendToSurge, so it too now benefits from the correct stored value.

Confidence Score: 5/5

Safe to merge — targeted two-line fix with no regression risk

The change is minimal and precisely correct: it stops discarding the already-extracted directory in two call-sites. The guard in sendToSurge (if absolutePath) preserves fallback behaviour when no directory is present. No new logic is introduced and no existing behaviour is broken.

No files require special attention

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant Browser
    participant handleDownloadIntercept
    participant extractPathInfo
    participant Target as sendToSurge / pendingDuplicates

    Browser->>handleDownloadIntercept: onCreated(downloadItem)
    alt duplicate detected
        handleDownloadIntercept->>extractPathInfo: extractPathInfo(downloadItem)
        extractPathInfo-->>handleDownloadIntercept: { filename, directory }
        handleDownloadIntercept->>Target: pendingDuplicates.set(id, { filename, directory, … })
        Note over Target: Previously directory was hardcoded ''
    else surge running
        handleDownloadIntercept->>extractPathInfo: extractPathInfo(downloadItem)
        extractPathInfo-->>handleDownloadIntercept: { filename, directory }
        handleDownloadIntercept->>Target: sendToSurge(url, filename, directory)
        Note over Target: Previously directory was hardcoded ''
        Target-->>handleDownloadIntercept: { success }
    end

_{Greploops — Automatically fix all review issues by running /greploops in Claude Code. It iterates: fix, push, re-review, repeat until 5/5 confidence.
Use the Greptile plugin for Claude Code to query reviews, search comments, and manage custom context directly from your terminal.}

_{Reviews (2): Last reviewed commit: "shorthand for directory path" | Re-trigger Greptile}