zed
                                
                                 zed copied to clipboard
                                
                                    zed copied to clipboard
                            
                            
                            
                        Add `zip` extract support for Windows
Release Notes:
- ~~[x] Added extension download zipfile to extract supports on Windows.~~
- [x] Fixed install Node.js runtime and NPM lsp installation on Windows.
- ~~[x] Fixed Copilot on Windows.~~
- [x] Added http_proxyenv for NPM command, if system env has this env.
- [x] Update Node runtime command to execute on Windows with no window popup.
Checklist
- [x] Test Node.js runtime and others on Windows platform.
- ~~[x] Test Copilot changes on macOS.~~
- [x] Test typescript-language-server install on macOS
- [x] Test Node runtime execute with no window on Windows
Ref #9619, #9424
Node.js runtime have successfully installed on Windows now.
Now, Node.js runtime has fixed and Copilot works.
https://github.com/zed-industries/zed/assets/5518/e27cedf9-6aa1-4a94-9734-89e257f6704e
Node runtime have been passed test.
https://github.com/zed-industries/zed/assets/5518/eed3243f-e201-47db-9504-7d70ba61d013
Hi @maxdeviant, @mikayla-maki
I think I have done of all about Windows install zip and Node.js runtime install, we can review and testing now.
I have to do a lot of file change, because there are too many issues on Windows. They are mostly issues that have different path on Windows.
Anything I can do to help with this? Was trying to solve the same issue and found this PR π Those force-pushes are killing me trying to work out what's going on with the current state of the PR so thought it better I just ask π
I have updated:
- Moved archiveintonode_runtimecrate as a mod, I keep the methods not just only the zip, we may use that in the future.
- Reverted other crate changes, just keep the node_runtime.
- The since_v0_0_6.rsstill useunzipcommand to extract zip file, this can't work on Windows, I think we can make author PR to discuss this. For example: https://github.com/zed-industries/zed/blob/main/extensions/csharp/src/csharp.rs#L75
~~I just did a fresh installation of the Node.js runtime on Windows. It seems that we have to modify the root_path part of Copilot (Commit: https://github.com/zed-industries/zed/pull/11156/commits/5131e1a5db1ea941e1f72e253af6c170d545eb19), otherwise it will not start and will directly crash.~~
Reverted this copilot change.
The https://github.com/zed-industries/zed/pull/11591/files#diff-9aed0f58926aee71638164c95da3800a90460d525d736a354c71490e333c8d88R581 is a better way.
@maxdeviant For other parts of the zip decompression, you can follow up to see how to deal with it. Currently, there are still problems using the unzip command in the extension, and you need to follow up.
However, the parts that have been merged so far and the panic problem caused by Copilot's root_path unwrap will be fixed. Then the most Node.js related ones can already be run.
The HTTP Proxy issue has not been resolved yet. I donβt know if it is a problem with my proxy software. I have tested it before, but it was strange today, so I canceled it first. I'll submit another one after it's resolved.
However, the parts that have been merged so far and the panic problem caused by Copilot's root_path unwrap will be fixed. Then the most Node.js related ones can already be run.
this should fixed, currently open immediaterly crashed.