Skip to content

Conversation

@AloofBuddha
Copy link
Contributor

@AloofBuddha AloofBuddha commented Nov 22, 2025

Summary

This change removes the erroneous “Install Cursor” notification in the desktop environment when Cursor is already installed. The issue is the timeout of 500ms was simply too short for Cursor Desktop to respond in time, so I increased it to 1500ms. This seems to consistently work, but it could easily be increased to 2000ms to be safe.

Issue

Fixes #20289

Testing

  • Verified that when Cursor is installed, the notification no longer appears

QA

Before

Screen.Recording.2025-11-22.155304.mp4

After

Screen.Recording.2025-11-22.155748.mp4

@cdr-bot cdr-bot bot added the community Pull Requests and issues created by the community. label Nov 22, 2025
@github-actions
Copy link

github-actions bot commented Nov 22, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@AloofBuddha AloofBuddha changed the title fix(frontend): remove false "install Cursor" notification when Cursor Desktop is already installed (Fixes #20289) fix(frontend): remove erroneous "install Cursor" notification for Cursor Desktop (Fixes #20289) Nov 22, 2025
@AloofBuddha AloofBuddha changed the title fix(frontend): remove erroneous "install Cursor" notification for Cursor Desktop (Fixes #20289) fix(useAppLink): remove erroneous "install Cursor" notification for Cursor Desktop (Fixes #20289) Nov 22, 2025
@AloofBuddha AloofBuddha changed the title fix(useAppLink): remove erroneous "install Cursor" notification for Cursor Desktop (Fixes #20289) fix(site): remove erroneous "install Cursor" notification for Cursor Desktop Nov 22, 2025
@AloofBuddha
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

cdrci2 added a commit to coder/cla that referenced this pull request Nov 23, 2025
Copy link
Member

@ethanndickson ethanndickson left a comment

Choose a reason for hiding this comment

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

Approving since I've seen this issue reported a few times now, and it makes sense that 500ms is just too fast for a lot of IDEs.

Thanks for the PR! This will be available in the next mainline release (next week!)

@ethanndickson ethanndickson merged commit 9c2f94b into coder:main Nov 26, 2025
22 of 23 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Nov 26, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

community Pull Requests and issues created by the community.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

False notification prompting Cursor installation despite it being installed

2 participants