Skip to content

Remove redundant @vscode/vsce uninstall/reinstall from postinstall#723

Merged
gfs merged 3 commits intomainfrom
copilot/fix-pack-ext-build-steps
Feb 6, 2026
Merged

Remove redundant @vscode/vsce uninstall/reinstall from postinstall#723
gfs merged 3 commits intomainfrom
copilot/fix-pack-ext-build-steps

Conversation

Copy link
Contributor

Copilot AI commented Feb 6, 2026

The postinstall script had a workaround that uninstalled and reinstalled @vscode/vsce after every npm install to fix a missing out/ directory issue in older versions. This was resolved by the upgrade to @vscode/vsce@^3.7.1 in v1.0.71, making the workaround dead weight.

  • Simplified postinstall to only install client dependencies:
-"postinstall": "cd client && npm install && cd .. && npm uninstall @vscode/vsce && npm install --save-dev @vscode/vsce"
+"postinstall": "cd client && npm install"
Original prompt

The package.json command for the vs code extension currently uses an npm install command for vsce that was to try to work around a strange issue where even after installing the dependencies bits of vsce were missing. This is however kind of messy and it would be better to remove this strangeness of having an uninstall and reinstll in the build script. Please investigte and try to fix the pack-ext build steps to remove the explicit install and uninstall steps that should alredy have been covered by the original npm install.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…script

Co-authored-by: gfs <98900+gfs@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix pack-ext build steps to remove unnecessary install commands Remove redundant @vscode/vsce uninstall/reinstall from postinstall Feb 6, 2026
Copilot AI requested a review from gfs February 6, 2026 01:00
@gfs gfs marked this pull request as ready for review February 6, 2026 01:02
danfiedler-msft
danfiedler-msft previously approved these changes Feb 6, 2026
@gfs gfs merged commit 941862b into main Feb 6, 2026
6 checks passed
@gfs gfs deleted the copilot/fix-pack-ext-build-steps branch February 6, 2026 17:18
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.

3 participants