Claude/windows slowness pool error ty jcd#13
Merged
Conversation
On Windows, watchdog's Observer uses WindowsApiObserver which dispatches events through a thread pool. The old code ran file I/O and pyperclip clipboard access synchronously inside the watchdog event handler, blocking pool workers under load and causing pool-exhaustion errors and UI slowness that didn't reproduce on Linux. - Add _in_queue (SimpleQueue) and a dedicated clipsync-in thread that drains the queue and calls _on_file_changed. The watchdog handler now just puts a path on the queue and returns immediately, keeping the thread pool free. - Cache resolved clipboard file paths in _ClipboardFileHandler so _matches() doesn't call Path.resolve() three times per event. - Add fast name-based pre-filter before the (more expensive) resolve() call, since Syncthing generates many temp-file events for other files. https://claude.ai/code/session_0128MyVVEu1Dt3jTeaZ1edyL
…ming views _DevicesContent._refresh() and IncomingWindow._refresh() both called connected_devices() / get_pending_devices() directly on the main thread. Each call has a 10-second timeout and connected_devices() makes 3 sequential requests, so a slow or unresponsive Syncthing froze the entire UI for up to 30 seconds on every auto-refresh cycle (every 3s). - Split each _refresh() into _do_refresh() (background thread) + _apply_refresh() (main thread via after(0, ...)), matching the pattern already used by _nearby_worker. - Catch requests.RequestException separately and show "Syncthing is not responding" instead of the raw HTTPConnectionPool traceback string. - Show a "Loading…" placeholder while the fetch is in flight. https://claude.ai/code/session_0128MyVVEu1Dt3jTeaZ1edyL
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.