Skip to content

Conversation

@hillaoui
Copy link

This is a fix of the issue faced by a lot concerning the path of the file products.json when choosing the option "Bypass Cursor version check".

Copy link

@Akshat-prog-mis Akshat-prog-mis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I noticed that in the Windows section, you’ve replaced the dynamic LOCALAPPDATA lookup with a hardcoded "C:/Program Files/cursor" path. That breaks compatibility for non-admin or per-user installs, since many systems (especially Windows 10/11 user setups) install Cursor under LOCALAPPDATA\Programs.

The original code handled multiple environments and config overrides — the hardcoded path removes that flexibility and can cause file-not-found errors on most user setups.

If your goal was to fix a specific issue, could you explain the scenario or make the path configurable instead of hardcoding it? Otherwise, this change introduces a regression in portability.

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