Repository navigation
fix(discover): sanitize drive-letter colon so Windows discover finds sessions (adopt upstream #2952) - #38
Merged
Conversation
…sessions encode_project_path mirrors Claude Code's cwd -> ~/.claude/projects slug, but never replaced the drive-letter ':' with '-'. Every Windows cwd carries one, so the encoded slug (C:-Users-me) never matched Claude's real folder (C--Users-me) and the default `rtk discover` scanned 0 sessions on Windows. Add ':' to SANITIZED_CHARS; the shared encoder also fixes `rtk learn`. Refs rtk-ai#2919
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.
Adopts rtk-ai/rtk#2952 by @zbl1998-sdjn — cherry-picked preserving authorship, no amendments needed.
Closes #25
What
ClaudeProvider::encode_project_pathwas missing:inSANITIZED_CHARS, so every Windows cwd encoded toC:-Users-...while Claude's real project folder isC--Users-.... The slug never matched, and the defaultrtk discover(andrtk learn, same encoder) scanned 0 sessions on Windows. One-char fix + regression tests.Adoption checklist (CONTEXT.md)
C:\Users\kyleh\projects\rtk):f0c7d5d, author zbl1998-sdjn), fixed behavior confirmed with the rebuilt binary:~/.claude/projects/C--Users-kyleh-projects-rtk(colon →-).cargo fmt --all(no changes) &&cargo clippy --all-targets(clean) &&cargo test --all(2470 passed, 0 failed)🤖 Generated with Claude Code