Skip to content

fix: add Windows path support for CursorParser and ClineParser - #25

Open
krishnesh1 wants to merge 1 commit into
uber:mainfrom
krishnesh1:fix-windows-path-support
Open

fix: add Windows path support for CursorParser and ClineParser#25
krishnesh1 wants to merge 1 commit into
uber:mainfrom
krishnesh1:fix-windows-path-support

Conversation

@krishnesh1

Copy link
Copy Markdown

What changed?
Added Windows path support for CursorParser and ClineParser.
- Imported the os module in both parser files.
- Used os.getenv("APPDATA") with a fallback to ~/AppData/Roaming to locate the %APPDATA% directory
dynamically on Windows machines.
- Added windows_path configuration strings matching the target application data stores for both Cursor and
Cline.
- Incorporated the windows_path directory check into the main parser __init__ resolution chains.

<!-- Tell your future self why have you made these changes -->
**Why?**
Previously, `CursorParser` and `ClineParser` relied solely on hardcoded macOS and Linux configurations to

discover log paths (~/.config and ~/Library/Application Support/). This caused the tool to silently fail and
report zero telemetry on Windows.

<!-- How have you verified this change? Tested locally? Added a unit test? Checked in a staging env? -->
**How did you test it?**
Tested locally to ensure existing fallback chains correctly identify the local machine's OS path. Existing test

suite checks for missing directories continue to pass as expected.

<!-- Assuming the worst case, what can be broken when deploying this change to production? -->
**Potential risks**
Minimal risk. The path checks (`exists()`) safely fail over through macOS -> Windows -> Linux. This change is

purely additive and does not modify the telemetry parsing logic itself, so it shouldn't disrupt existing telemetry
capabilities on non-Windows environments.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants