Feature:Support using Ctrl+C/X and Ctrl+V to invoke FastCopy by b1xcy · Pull Request #64 · HO-COOH/FastCopy

b1xcy · 2026-08-11T05:25:40Z

feature：通过 Ctrl+C/X 和 Ctrl+V 调用 FastCopy

Ctrl+C 和 Ctrl+X 仍由资源管理器原生处理。这样可以保留标准 Shell IDataObject、文件系统路径，以及复制或移动操作标志。

新增一个随 MSIX 打包的后台辅助进程，用于接管粘贴操作：

通过 MSIX 启动任务以及主程序启动流程运行。
结合 RegisterHotKey 和 WH_KEYBOARD_LL 监听 Ctrl+V。
阻止资源管理器继续执行原生粘贴命令。
对全局热键和键盘钩子产生的事件进行去重，避免重复启动任务。
只接管资源管理器中的文件粘贴操作。
通过活动 Shell View 获取目标目录。
针对带标签页的新版资源管理器，使用 LocationURL 作为目录解析后备方案。
优先通过 IShellItemArray 解析剪贴板文件和文件夹。
在 Shell Item 解析失败时，通过标准 CF_HDROP 格式读取路径。
通过 CFSTR_PREFERREDDROPEFFECT 区分复制和移动操作。
将源文件路径写入 FastCopy 现有的任务记录文件格式。
通过现有的 fastcopy:// 协议启动 FastCopy。
移动操作成功交给 FastCopy 后清空剪贴板。
如果目标目录解析、剪贴板读取或 FastCopy 启动失败，则恢复执行系统原生 Ctrl+V。

同时新增了本地化设置开关，允许用户启用或关闭资源管理器快捷键集成功能。

Note

Medium Risk
Global low-level keyboard hooking and clipboard interception affect core Explorer paste behavior; mistakes could block paste or mis-route copy/move, though fallback replay and a user toggle limit blast radius.

Overview
Adds Explorer keyboard integration so Ctrl+V in File Explorer can route file paste through RoboCopyEx instead of the shell copy engine, while Ctrl+C/X stay on Explorer’s native clipboard behavior.

A new FastCopyKeyboardHook full-trust helper (MSIX startup task + launch from the main app) uses RegisterHotKey and a WH_KEYBOARD_LL hook to catch Ctrl+V, suppress Explorer’s paste when appropriate, resolve the destination folder (shell view / tabbed Explorer fallbacks), read file paths from the clipboard, write the existing record-file format, and invoke fastcopy://. Failed handoffs replay native Ctrl+V; successful move pastes clear the clipboard.

The main app gains KeyboardHookController (start/stop the helper, registry-backed enablement), a settings toggle with localized strings, and shared Public helpers (KeyboardHookSettings, ModulePath, moved ShellWindows/WebBrowser2).

^{Reviewed by Cursor Bugbot for commit 247b2c1. Bugbot is set up for automated code reviews on this repo. Configure here.}

HO-COOH · 2026-08-13T07:40:29Z

IShellWindow 不行！我之前已经踩过坑了，就是有用户装了那个tab工具，让多个explorer作为tab显示在一个窗口里，这时候我拿不到正确的那个，你可以翻一下issue

HO-COOH · 2026-08-13T09:12:22Z

 	ResumeThread(hThread);
 }

-void RobocopyProcess::Suspend() const


怎么把我上一次commit的改动给还原了？

sry revert错commit了我重新弄一下

HO-COOH · 2026-08-13T10:10:33Z

看了下，有大量重复代码，包括注册表那块以及打开程序那块，应该全部提出来放Public里。
能用wil管理资源的尽量用wil
有错误不要吞掉，直接抛出来，在最外面catch住，显示个MessageBox，不然会跟现在ShellExetension一样很难debug
对现有ShellExtension的改动单独开个PR

先感谢大佬帮忙

b1xcy · 2026-08-13T10:34:29Z

那我先把这个pr改成单hook，卸载的问题我再开一个
对于IShellWindow的坑，我看这个issue应该是拿错实例了，具体路径提取我走QueryActiveShellView → IFolderView → GetCurFolder，等我写完试试吧

Keep Ctrl+C and Ctrl+X in Explorer so the Shell produces its standard IDataObject, including the preferred copy or move drop effect. Start a packaged background helper at sign-in and app launch. It combines RegisterHotKey with WH_KEYBOARD_LL to observe Ctrl+V, suppress Explorer's native accelerator, and deduplicate the two input paths. Resolve the foreground Explorer folder through IShellWindows, using the active ShellView first and LocationURL as a tab-compatible fallback. Read filesystem items through IShellItemArray with CF_HDROP fallback, serialize them in FastCopy's existing task-file format, and launch the fastcopy protocol. Clear the clipboard after a move; on any failure, temporarily release the hotkey and replay native Ctrl+V. Add a localized settings toggle backed by HKCU and package the helper as an enabled startup task. Verified: user-tested file and folder copy/move with Ctrl+C/Ctrl+X then Ctrl+V using the signed 1.2.1.8 MSIX; FastCopyKeyboardHook Package|x64 builds with 0 warnings and 0 errors; git diff --cached --check passes.

The desktop window (Progman/WorkerW) is not an Explorer frame, so the keyboard hook let Ctrl+V fall through to the native paste. Treat the desktop as a valid target and resolve its folder through FOLDERID_Desktop.

HO-COOH · 2026-08-13T16:08:14Z

+    {
+        DWORD value = 0;
+        DWORD valueSize = sizeof(value);
+        auto const result = RegGetValueW(


use wil::try_get_value_dword

HO-COOH · 2026-08-13T16:11:37Z

Please wrap all the COM types you used into a class. Throw if any of them failed, then catch them outside

HO-COOH · 2026-08-13T16:13:58Z

+
+struct ClipboardFileTransfer
+{
+    std::vector<std::wstring> paths;


Why copying from cotaskmem to a std::wstring? You can simply use its value later. Wrap it in wil::unique_cotaskmem_string

HO-COOH · 2026-08-13T16:15:44Z

+        &startupInfo,
+        &processInfo))
+    {
+        CloseHandle(processInfo.hThread);


Use wil::unique_process_information

HO-COOH · 2026-08-13T16:16:53Z

+        return;
+    }
+
+    std::wstring modulePath(32768, L'\0');


Not necessary. Use std::array<wchar_t, MAX_PATH> is enough. Actually already have one in currentDllPath(). Move that into public and reuse it.

HO-COOH · 2026-08-13T16:17:18Z

+    auto const hookPath = std::filesystem::path{ modulePath }.parent_path() / L"FastCopyKeyboardHook.exe";
+    if (!std::filesystem::exists(hookPath))
+    {
+        return;


don't silence this error.

HO-COOH · 2026-08-13T16:22:05Z

+        return 0;
+    }
+
+    auto const singleton = CreateMutexW(nullptr, FALSE, KeyboardHookSettings::SingletonName);


Use wil::unique_mutex

HO-COOH · 2026-08-13T16:23:34Z

+        return 0;
+    }
+
+    auto const comResult = OleInitialize(nullptr);


Use wil::scope_exit to uninitialize ole

HO-COOH · 2026-08-13T16:24:32Z

Please refactor out this file. Definitely put the message window into a class. Private constant goes into class member.

HO-COOH · 2026-08-13T16:25:19Z

+    {
+        std::array<wchar_t, 256> buffer{};
+        auto const length = GetClassNameW(window, buffer.data(), static_cast<int>(buffer.size()));
+        return length > 0 ? std::wstring{ buffer.data(), static_cast<size_t>(length) } : std::wstring{};


why not just return the array?

…ard hook - KeyboardHookSettings: use wil::reg::try_get_value_dword / create_unique_key - ClipboardFileTransfer: hold GetDisplayName results in wil::unique_cotaskmem_string - ExplorerWindow: wrap the COM types in an ExplorerFolderResolver class and throw (wil::ResultException) on COM failures; GetWindowClass returns the array instead of copying to a std::wstring - main: refactor into a KeyboardHookApp class (message window and private constants become class members), use wil::unique_mutex for the singleton and wil::scope_exit for OleUninitialize; failures are caught at the message loop and wWinMain and shown in a message box - add the WIL NuGet dependency to FastCopyKeyboardHook

…p errors - Move the module path helper out of DllIconFormatter into Public/ModulePath and reuse it in KeyboardHookController instead of the 32K scratch buffer - KeyboardHookController: use wil::unique_process_information and throw when the hook executable is missing instead of silently returning - SettingsViewModel: catch hook startup errors and show them

ClipboardFileTransfer::paths now holds wil::unique_cotaskmem_string instead of copying every display name into a std::wstring; the values are consumed (normalized and serialized) only when the record file is written.

…derResolver

HO-COOH · 2026-08-14T12:15:34Z

+            }
+        }
+        auto const isPaste = event->vkCode == L'V';
+        if (isReplayInput || !isPaste)


refactor this if chain

HO-COOH · 2026-08-14T12:20:27Z

+    }
+}
+
+class KeyboardHookApp


Please separate this class into 4 classes, a Window class, a keyboard hook class, a HotKey class (for that paste hot key using RAII to unregister automatically) and an App class. Also, please make private class member start s with lower case letter

HO-COOH · 2026-08-14T12:24:38Z

+    static KeyboardHookApp* s_instance;
+
+    HWND m_messageWindow{};
+    HHOOK m_keyboardHook{};


Use wil::unique_hhook

HO-COOH · 2026-08-14T12:26:42Z

+    auto const className = GetWindowClass(expectedForegroundWindow);
+    if (std::wcscmp(className.data(), L"Progman") == 0 || std::wcscmp(className.data(), L"WorkerW") == 0)
+    {
+        PWSTR desktopPath{};


Use wil::unique_cotaskmem_string

HO-COOH · 2026-08-14T12:27:20Z

+    // boundary; failures that are expected for individual entries are
+    // skipped, and a window whose active view does not support the older
+    // IFolderView interfaces falls back to the browser location URL.
+    class ExplorerFolderResolver


Move into its separate .h and .cpp file

HO-COOH · 2026-08-14T12:34:33Z

+    public:
+        ExplorerFolderResolver()
+        {
+            THROW_IF_FAILED(CoCreateInstance(CLSID_ShellWindows, nullptr, CLSCTX_ALL, IID_PPV_ARGS(&m_shellWindows)));


I already have a ShellWindows class, a IShellWindow wrapper in FastCopyShellExtension. Please use that.

…lasses - Move the ShellWindows/WebBrowser2 wrappers from FastCopyShellExtension into Public/ (drop the ATL dependency) and reuse them from the keyboard hook's ExplorerFolderResolver instead of a private implementation - ExplorerFolderResolver moves into its own .h/.cpp; ExplorerWindow keeps only the window checks, and the desktop branch now uses wil::unique_cotaskmem_string - Split the hook app into four classes: PasteWindow (message window), PasteHotKey (RAII register/unregister), KeyboardHook (RAII hook via wil::unique_hhook) and KeyboardHookApp (composition and paste logic); private members start with a lower-case letter, and the keyboard event handler is restructured to early-return instead of nested ifs

cursor

Cursor Bugbot has reviewed your changes using default effort and found 4 potential issues.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 247b2c1. Configure here.}

cursor · 2026-08-18T17:13:19Z

+    {
+        PostMessageW(window, WM_CLOSE, 0, 0);
+    }
+}


Stop cannot find hook window

High Severity

Stop uses FindWindowW to locate the hook HWND, but PasteWindow is created as an HWND_MESSAGE window. FindWindowW never searches message-only windows, so disable always fails to post WM_CLOSE. The registry flag flips off while the helper keeps intercepting Ctrl+V until reboot. The existing SettingsChangeListener already uses FindWindowEx(HWND_MESSAGE, ...) for this case.

Additional Locations (1)

FastCopyKeyboardHook/PasteWindow.cpp#L15-L28

^{Reviewed by Cursor Bugbot for commit 247b2c1. Configure here.}

cursor · 2026-08-18T17:13:19Z

+        }
+    }
+
+    return locationFallback;


Wrong tab folder fallback

High Severity

locationFallback keeps the first ShellWindows entry that shares the frame HWND, and focus is only tested against the shell view window, not the shell browser. With Win11 tabs, focus in the navigation pane or a failed IFolderView path can return another tab's LocationURL, so FastCopy pastes into the wrong directory.

Additional Locations (1)

FastCopyKeyboardHook/ExplorerFolderResolver.cpp#L52-L58

^{Reviewed by Cursor Bugbot for commit 247b2c1. Configure here.}

cursor · 2026-08-18T17:13:19Z

+        {
+            ShowError(e.what());
+        }
+        return 0;


Paste not restored on errors

High Severity

The low-level hook already eats Ctrl+V before HandlePaste runs. Soft failures call ReplayPaste, but COM/THROW_IF_FAILED errors are caught in OnWindowMessage, show a message box, and never replay. Folder resolution helpers throw on routine COM failures, so Explorer paste can disappear entirely instead of falling back to the native paste.

Additional Locations (2)

FastCopyKeyboardHook/KeyboardHookApp.cpp#L179-L193

FastCopyKeyboardHook/ExplorerFolderResolver.cpp#L64-L88

^{Reviewed by Cursor Bugbot for commit 247b2c1. Configure here.}

cursor · 2026-08-18T17:13:19Z

+    if (!singleton || GetLastError() == ERROR_ALREADY_EXISTS)
+    {
+        return 0;
+    }


Hook before singleton check

Medium Severity

KeyboardHook is installed during KeyboardHookApp construction, while the singleton mutex is only taken later in Run. Every FastCopy launch calls Start and can spawn a short-lived second process that briefly installs another WH_KEYBOARD_LL hook before exiting, creating a race where two helpers can both handle the same paste.

Additional Locations (2)

FastCopy/KeyboardHookController.cpp#L25-L48

FastCopy/App.xaml.cpp#L64-L66

^{Reviewed by Cursor Bugbot for commit 247b2c1. Configure here.}

HO-COOH requested changes Aug 13, 2026

View reviewed changes

b1xcy force-pushed the master branch from 8d94b01 to 6918159 Compare August 13, 2026 09:31

b1xcy changed the title ~~Fix:Bug of slow uninstallation & Feature:Support using Ctrl+C/X and Ctrl+V to invoke FastCopy~~ Feature:Support using Ctrl+C/X and Ctrl+V to invoke FastCopy Aug 13, 2026

b1xcy added 2 commits August 13, 2026 18:59

Fix malformed MSBuild project closing tag

5f0c82a

b1xcy force-pushed the master branch from 6918159 to 5f0c82a Compare August 13, 2026 11:05

Support the desktop as a paste target for Ctrl+V

ddf7c0b

The desktop window (Progman/WorkerW) is not an Explorer frame, so the keyboard hook let Ctrl+V fall through to the native paste. Treat the desktop as a valid target and resolve its folder through FOLDERID_Desktop.

HO-COOH requested changes Aug 13, 2026

View reviewed changes

b1xcy added 4 commits August 14, 2026 00:44

Address review: store clipboard paths as wil::unique_cotaskmem_string

76f4878

ClipboardFileTransfer::paths now holds wil::unique_cotaskmem_string instead of copying every display name into a std::wstring; the values are consumed (normalized and serialized) only when the record file is written.

Address review: keep per-entry skip/fallback semantics in ExplorerFol…

45408ad

…derResolver

HO-COOH requested changes Aug 14, 2026

View reviewed changes

b1xcy and others added 2 commits August 14, 2026 21:08

Fix PasteWindow incorrect creation

247b2c1

cursor Bot reviewed Aug 18, 2026

View reviewed changes

Conversation

b1xcy commented Aug 11, 2026 • edited by cursor Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

feature：通过 Ctrl+C/X 和 Ctrl+V 调用 FastCopy

Uh oh!

HO-COOH commented Aug 13, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

b1xcy Aug 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

HO-COOH commented Aug 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

b1xcy commented Aug 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

cursor Bot Aug 18, 2026

Choose a reason for hiding this comment

Stop cannot find hook window

Uh oh!

cursor Bot Aug 18, 2026

Choose a reason for hiding this comment

Wrong tab folder fallback

Uh oh!

cursor Bot Aug 18, 2026

Choose a reason for hiding this comment

Paste not restored on errors

Uh oh!

cursor Bot Aug 18, 2026

Choose a reason for hiding this comment

Hook before singleton check

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

b1xcy commented Aug 11, 2026 •

edited by cursor Bot

Loading

b1xcy Aug 13, 2026 •

edited

Loading

HO-COOH commented Aug 13, 2026 •

edited

Loading

b1xcy commented Aug 13, 2026 •

edited

Loading