Conversation
…default DevinAdapter::new() only probed XDG (~/.local/share/devin), the XDG env override and macOS' ~/Library/Application Support/devin. Devin Desktop on Windows keeps its sessions.db at %APPDATA%\devin\cli\sessions.db, so the default instance found no library and Devin sessions never showed up — only a manually added custom location worked around it. Add the AppData/Roaming candidate (derived from home_dir(), same convention as cursor_ide so WAKE_HOME redirection keeps working). The candidate is probed unconditionally like app_support, which keeps the XDG form first in the order and makes the resolution testable on every platform.
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
1 participant
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.
Problem
DevinAdapter::new()probes three roots forcli/sessions.db:/devin(env override)~/.local/share/devin(XDG default; also what macOS uses in practice)~/Library/Application Support/devin(macOS desktop Electron profile, historical candidate)None of these exist on Windows. Devin Desktop on Windows keeps its sessions at
%APPDATA%\devin\cli\sessions.db, so the default instance finds no library and Devin sessions never appear — the only workaround is adding a custom location by hand.Fix
Add
%APPDATA%\devin(i.e.home/AppData/Roaming/devin, derived fromhome_dir()) as a fourth candidate in the probe chain, afterapp_support.Design notes:
home_dir()instead ofdirs::config_dir(), same convention ascursor_ide::default_db_path— otherwise theWAKE_HOMEredirection would not apply on Windows (SHGetKnownFolderPath ignores it).app_supportcandidate) rather than behindcfg!(windows): the directory simply doesn't exist on macOS/Linux, and keeping it unconditional lets the contract test below exercise the resolution on every platform.Test
New
crates/wake-core/tests/devin_default_root.rs(own file per the env-isolation convention used by mcp_stdio/remote_sync, sinceWAKE_HOME/XDG_DATA_HOMEare process-global):Verified on a real Windows machine with Devin Desktop installed (
%APPDATA%\devin\cli\sessions.db, ~1.1 GB, 99 sessions): after removing the manually-added custom location, a freshly builtwake-cli refreshwith this patch indexes Devin via the default path — 10 agents instead of 9, Devin sessions listed and searchable.Full suite:
cargo test -p wake-core— 311 passed, 0 failed.cargo fmt --all --checkclean.