LspError.data missing #[serde(default)] swallows the real jdtls error

Aberta Para iniciantes
  • Algo errado?
#325 1 comentário 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
1/5
Tempo estimado
Menos de uma hora
Facilidade para iniciantes
88/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
json, rust
Domínio
backend

Direção de pesquisa

A struct LspError está definida em src/lsp.rs, e o enum untagged LspResponse que a envolve é onde o parsing falha. Comece lendo essas duas definições e depois adicione #[serde(default)] ao campo data. Para verificar, envie uma resposta do proxy com um objeto de erro que não tenha a chave data e confirme que o campo message chega ao usuário sem alterações.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

Summary

src/lsp.rs defines

pub struct LspError {
    code: i64,
    message: String,
    data: Value,   // no #[serde(default)]
}

LSP allows error.data to be omitted. When jdtls returns an error without data, the Error variant fails to deserialize, and since LspResponse is an untagged enum, the whole parse fails with:

Failed to parse response from LSP proxy: data did not match any variant of untagged enum LspResponse

Impact

The actual error message is swallowed. In our case the real response was:

{"error":{"code":-32601,"message":"No delegateCommandHandler for vscode.java.resolveMainClass"},"id":"...","jsonrpc":"2.0"}

but Zed surfaced the misleading serde error, which pointed everyone (including maintainers reading our report) at "protocol/proxy problems" instead of the broken debug plugin (see #324). This cost a long debugging session.

Repro

  • Start a debug session while the java-debug bundle is broken/unloaded (e.g. #324)
  • Or directly: curl -X POST http://127.0.0.1:<proxy-port>/ -d '{"method":"workspace/executeCommand","params":{"command":"vscode.java.resolveMainClass","arguments":[]}}' — the response error has no data field

Fix

Add #[serde(default)] to data (one-liner) so the real message (No delegateCommandHandler ...) reaches the user.

Linguagem predominante
Rust
Estrelas
225
Forks
47
Merge médio
3h
PRs com merge (30d)
5

Preparar o ambiente

Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de zed-extensions/java

Todas as issues de zed-extensions/java

Issues semelhantes

Mais issues de Rust

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.