winutil icon indicating copy to clipboard operation
winutil copied to clipboard

Added Cursor AI code editor

Open SHR1SHAK opened this issue 1 year ago • 6 comments

Cursor is an AI-first code Editor that transcends traditional Integrated Development Environments (IDEs) by integrating artificial intelligence to enhance the coding experience significantly.

SHR1SHAK avatar Jul 24 '24 10:07 SHR1SHAK

Hey @SHR1SHAK

As seen in #2234, another package for cursor did not work.

Have you tested this package?

MyDrift-user avatar Jul 24 '24 11:07 MyDrift-user

I have tested using this argument winget.exe install --id Anysphere.Cursor --exact --source winget --accept-source-agreements --version "0.37.1" --silent --disable-interactivity --accept-package-agreements do i also need to inside the whole thing to work ?

SHR1SHAK avatar Jul 24 '24 11:07 SHR1SHAK

I have tested using this argument winget.exe install --id Anysphere.Cursor --exact --source winget --accept-source-agreements --version "0.37.1" --silent --disable-interactivity --accept-package-agreements do i also need to inside the whole thing to work ?

@SHR1SHAK The --version "0.37.1" part isn't necessary, as it's the latest version for Anysphere.Cursor WinGet Package. Also as far as I know, the --disable-interactivity flag isn't used by WinGet Package Install/Uninstall Function, so is this flag required to make this package install/uninstall successfully? Thanks in advance.

EDIT: Found the line for it, and indeed it doesn't use the flag I've referenced earlier. image

og-mrk avatar Jul 24 '24 16:07 og-mrk

For this PR I have just added the ID for Winget "Anysphere.Cursor" but it error out. To install successfully using winget Anysphere.Cursor --exact --source winget --accept-source-agreements --silent --disable-interactivity --accept-package-agreements this whole thing should be used. My question is can winutil use the whole argument in json file? @og-mrk

SHR1SHAK avatar Jul 24 '24 17:07 SHR1SHAK

For this PR I have just added the ID for Winget "Anysphere.Cursor" but it error out. To install successfully using winget Anysphere.Cursor --exact --source winget --accept-source-agreements --silent --disable-interactivity --accept-package-agreements this whole thing should be used. My question is can winutil use the whole argument in json file? @og-mrk

If you see the screenshot posted in this comment you'll find what arguments are passed to WinGet whenever you install any program that has a WinGet ID (almost all apps in applications.json File has a WinGet ID), and for providing arguments inside the json file, although it's a nice idea.. I don't recommend doing this, as it'll introduce more bugs, and increase maintenance time on Install/Uninstall Logic, plus it'll bloat the applications.json File even more (which FYI.. it's copy-pasted into the compiled winutil.ps1 file when you run Compile.ps1 Script, which's something to consider long-term with File Size).

All things considered.. I would (as a suggestion to you, or "If I was in your shoes" kind of thing) make this PR as Draft, and make another PR changing the Install-WinUtilProgramWinget Private Function to handle such case (maybe try adding that --disable-interactivity flag?).

og-mrk avatar Jul 24 '24 17:07 og-mrk

Yeah, I think I'm setting this PR to draft for now. But with all my testing the argument is required for successful installation.

SHR1SHAK avatar Jul 24 '24 17:07 SHR1SHAK