search

elixir-tools / elixir-tools.vscode

Visual Studio Code extension for Elixir
https://marketplace.visualstudio.com/items?itemName=elixir-tools.elixir-tools
Other
87 stars 9 forks source link

Invalid `Content-Length` property enabling Credo #94

Closed rvonzimr closed 1 week ago

rvonzimr commented 1 week ago

Whenever I attempt to enable Credo on my application, I get spammed with notifications all mentioning an invalid Content-Length property from jsonrpc. The actual messages being logged appear to be warnings when compiling the app, but nothing broken.

Here's a snipped of the extension output:

[Error - 11:16:36 AM] Client Credo: connection to server is erroring.
Header must provide a Content-Length property.
{"{\"jsonrpc\"":"\"2.0\",\"method\":\"window/logMessage\",\"params\":{\"message\":\"warning: redefining module Conversation.Graph.Node.Processor.Upload (current version loaded from /home/[redacted]/git/project/core/.elixir-tools/_build/dev/lib/conversation/ebin/Elixir.Conversation.Graph.Node.Processor.Upload.beam)\\n  lib/conversation/graph/node/processor/upload.ex:1: Conversation.Graph.Node.Processor.Upload (module)\",\"type\":4}}Content-Length: 350"}
Shutting down server.

nextls version:

./nextls --version
0.23.0
rvonzimr commented 1 week ago

Ah! just saw this was disabled by default as next supports credo itself? Maybe I can safely ignore this then :grin:

mhanberg commented 1 week ago

I'm not really sure what this is, but I am going to archive the credo language server soon, as Next LS has credo support