Skip to content

Conversation

@auscompgeek
Copy link
Member

Add CURSORLESS_VSCODE_COMMAND environment variable to allow using VS Code forks like VSCodium, Cursor, etc. in the install scripts.

  • install-from-pr.sh: Use env var with fallback to 'code'
  • install-local.sh: Use env var with fallback to 'code'
  • uninstall-local.sh: Use env var with fallback to 'code'
  • Updated error messages with helpful installation guidance
  • Maintains backward compatibility

Closes #3093

Add CURSORLESS_VSCODE_COMMAND environment variable to allow using
VS Code forks like VSCodium, Cursor, etc. in the install scripts.

- install-from-pr.sh: Use env var with fallback to 'code'
- install-local.sh: Use env var with fallback to 'code'
- uninstall-local.sh: Use env var with fallback to 'code'
- Updated error messages with helpful installation guidance
- Maintains backward compatibility

Closes #3093

Amp-Thread-ID: https://ampcode.com/threads/T-51f7db4c-7163-40d8-a87e-bfd7a6764249
Co-authored-by: Amp <amp@ampcode.com>
@auscompgeek
Copy link
Member Author

of course, I've actually tested that this works :)

and yes, this is AI generated, although I know enough bash to have made these changes by hand

@rinOfTheStars
Copy link
Contributor

rinOfTheStars commented Oct 31, 2025

Yeah, this looks like it'd work fine as a replacement for my PR. Didn't know about the default variable stuff, hence why I didn't use it >.<

I'll close #3093 since this does the same thing but in a safer way, since removing -u from set pipefail isn't great, even if it should be safe in the version of the scripts presented in that PR; once this is merged I'll make another PR to update CONTRIBUTING.mdx (or someone else can add updates to it to this one ig?)

@pokey pokey added this pull request to the merge queue Nov 1, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 1, 2025
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.

4 participants