LS job fails on windows vim if &shell is powershell
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 72/100
- Tipo de issue
- Bug
- Clareza
- Claramente especificada
- Status de atividade
- Pouca atividade
- Stack de tecnologia
- node.js, powershell, vim
- Domínio
- operating-systems, tooling
Direção de pesquisa
Comece em autoload/copilot/client.vim, especificamente em copilot#client#New(), e inspecione como o comando é preparado quando o Vim é executado no Windows com &shell definido como PowerShell ou pwsh. Reproduza a inicialização do language server nesse ambiente e verifique se ele também continua funcionando com a configuração de shell existente; considera-se concluído quando o servidor inicia com sucesso sem quebrar outras plataformas.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
The language server runs in node.js and is launched using:
> npx @github/copilot-language-server@^1.408.0 --stdio
the problem is that the plugin (see copilot.vim/autoload/copilot/client.vim function copilot#client#New())
uses:
job_start(['npx', '@github/copilot-language-server@^1.408.0', '--stdio'], ...)
which fails (is a win32 CreateProcess(...) issue) and should instead use:
job_start('npx "@github/copilot-language-server@^1.408.0" --stdio', ...)
The following patch works for me:
diff --git a/autoload/copilot/client.vim b/autoload/copilot/client.vim
index c3ec862..c052bc8 100644
--- a/autoload/copilot/client.vim
+++ b/autoload/copilot/client.vim
@@ -736,7 +736,7 @@ function! copilot#client#New() abort
let instance.workspaceFolders[folder.uri] = v:true
endfor
call copilot#logger#Debug('Spawning ' . join(command, ' '))
- let is_win_shell = has('win32') && &shellcmdflag !~# '^-'
+ let is_win_shell = has('win32') && ( &shell =~? 'powershell\|pwsh' || &shellcmdflag !~# '^-')
if is_win_shell && command[0] !~# '[\/]'
let exepath = exepath(command[0])
if exepath !~? '\.exe$\|^$'
- Linguagem predominante
- Vim Script
- Estrelas
- 11.7k
- Forks
- 821
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de github/copilot.vim
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
github/copilot.vim#305 ·
-
No link to DeepWikiAberta
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 82/100
github/copilot.vim#303 ·
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 15/100
github/copilot.vim#308 ·
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 35/100
github/copilot.vim#306 ·
-
NeomarleyAberta
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 1/100
github/copilot.vim#304 · 1 reação ·
Todas as issues de github/copilot.vim
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 66/100
siderolabs/pkgs#1710 ·
Mantenedores costumam responder em até 1 dia
-
good first issue kernel
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
JackFurton/who-would-build-a-kernel-in-java#81 ·
Mantenedores costumam responder em até 1 dia
-
[BUG] Windows: per-process values that can't be read (handles, I/O, network) still show as 0, not N/ATalvez já em andamento Um pull request vinculado a esta issue está aberto ou já foi mesclado. Abertaagent:Windows bug MEDIUM windows
Dificuldade 2/5 Meio dia Facilidade para iniciantes 70/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
Mantenedores costumam responder em até 1 dia
-
app bug sandbox windows-os
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
openai/codex#50915 · 1 reação ·
Mantenedores costumam responder em até 1 dia