code-server icon indicating copy to clipboard operation
code-server copied to clipboard

[Bug]: About lib\vscode\extensions\node_modules\.bin

Open ericzhucode opened this issue 3 years ago • 0 comments

Is there an existing issue for this?

  • [ ] I have searched the existing issues

OS/Web Information

  • Web Browser:
  • Local OS:
  • Remote OS:
  • Remote Architecture:
  • code-server --version:

Steps to Reproduce

A package issue

Expected

lib\vscode\extensions\node_modules.bin should not be packaged as the path these soft link files referring to do not exist

Actual

  1. I'm not quite sure if your team have realized it and decided to keep it but just from my view, tsc & tsserver(soft link file) in lib\vscode\extensions\node_modules.bin are meaningless and should not be packaged image As yarn run release:standalone will call postinstall.mjs script and it will delete all files in typescript lib & bin, the soft link files in screenshot refer to a non-existing path.

  2. Sorry to ask more but could you tell me why we need to run yarn again in build-standalone-release.sh. Maybe it's useful in vscode source code but for built-in extension, they are packaged by webpack and all node_modules should have been added to each extension's dist folder. Why do we need to run the following command again to install node_modules? yarn --production --frozen-lockfile --non-interactive

Sorry to disturb you guys in your busy time, looking forward to your reply

Logs

No response

Screenshot/Video

No response

Does this issue happen in VS Code or GitHub Codespaces?

  • [ ] I cannot reproduce this in VS Code.
  • [ ] I cannot reproduce this in GitHub Codespaces.

Are you accessing code-server over HTTPS?

  • [ ] I am using HTTPS.

Notes

No response

ericzhucode avatar Jul 26 '22 02:07 ericzhucode