[PR #11] [CLOSED] feat: add `vs-code-insiders` support to installer scripts #96

New issue

Closed

opened 2026-05-06 13:13:19 +02:00 by BreizhHardware · 0 comments

BreizhHardware commented

2026-05-06 13:13:19 +02:00

Owner

📋 Pull Request Information

Original PR: https://github.com/bwya77/vscode-dark-islands/pull/11
Author: @Jabolol
Created: 2/15/2026
Status: ❌ Closed

Base: main ← Head: jabolol/code-insiders-support

📝 Commits (1)

c94efa7 feat: add vs-code-insiders support to installer scripts

📊 Changes

3 files changed (+88 additions, -54 deletions)

View changed files

📝 README.md (+5 -3)
📝 install.ps1 (+49 -28)
📝 install.sh (+34 -23)

📄 Description

This PR adds VS Code Insiders support to the installer scripts. I use VS Code Insiders and didn't want to manually install the theme by copying files to the correct directories, and figured other users might need this too.

The scripts now try code first, then fall back to code-insiders if not found. They automatically use the correct paths for Insiders (~/.vscode-insiders/extensions, ~/Library/Application Support/Code - Insiders/User on macOS, etc.) while maintaining backward compatibility with stable VS Code.

This PR also fixes a bug in the JSONC parser that was stripping // from inside strings, which broke URLs like https://example.com in settings.json. The regex now only removes actual comment lines, not // within JSON values.

The PowerShell script changes haven't been tested on Windows since I don't have a Windows machine. The bash script works correctly on macOS with VS Code Insiders.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/bwya77/vscode-dark-islands/pull/11 **Author:** [@Jabolol](https://github.com/Jabolol) **Created:** 2/15/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `jabolol/code-insiders-support` --- ### 📝 Commits (1) - [`c94efa7`](https://github.com/bwya77/vscode-dark-islands/commit/c94efa7a2476b4dd865e9becd7dca1d71bc1ca3a) feat: add `vs-code-insiders` support to installer scripts ### 📊 Changes **3 files changed** (+88 additions, -54 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+5 -3) 📝 `install.ps1` (+49 -28) 📝 `install.sh` (+34 -23) </details> ### 📄 Description This PR adds VS Code Insiders support to the installer scripts. I use VS Code Insiders and didn't want to manually install the theme by copying files to the correct directories, and figured other users might need this too. The scripts now try `code` first, then fall back to `code-insiders` if not found. They automatically use the correct paths for Insiders (`~/.vscode-insiders/extensions`, `~/Library/Application Support/Code - Insiders/User` on macOS, etc.) while maintaining backward compatibility with stable VS Code. This PR also fixes a bug in the JSONC parser that was stripping `//` from inside strings, which broke URLs like `https://example.com` in settings.json. The regex now only removes actual comment lines, not `//` within JSON values. The PowerShell script changes haven't been tested on Windows since I don't have a Windows machine. The bash script works correctly on macOS with VS Code Insiders. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>