artempyanykh/marksman

Marksman runs fine on Helix but getting error logs

Koba0117 opened this issue · 3 comments

Koba0117 commented

I talked about this issue in #270 but this is probably unrelated to #270 and my explanation got very long so I opened a new one.

I use Marksman on Helix and it runs fine but every time I open a markdown file, I get an error in helix.log.
I also get errors when I make references on markdown files.

I checked other LSPs on my computer but I got this issue only with Marksman.

Opening a markdown file

  • Before opening a markdown file
    image

  • After opening a markdown file
    image
    2024-01-11T21:06:09.526 helix_lsp::transport [ERROR] marksman err <- "[21:06:09 INF] <LSP Entry> Starting Marksman LSP server: {}\n"

Restarting LSP

Considering the error message, restarting marksman may make a same error so I did it too.

  • Before restarting
    image

  • After restarting
    image
    2024-01-11T21:09:13.114 helix_lsp::transport [ERROR] marksman err <- "[21:09:13 INF] <LSP Entry> Starting Marksman LSP server: {}\n"

Reference

  • Before
    image

  • After
    image
    2024-01-11T21:14:24.637 helix_lsp::transport [ERROR] marksman err <- "[21:14:24 WRN] <Doc> Unexpected document version : {\"newVersion\": 9, \"currentVersion\": 4, \"uri\": \"file:///home/kyosuke/Documents/marksman-test/test.md\"}\n"

  • Before
    image

  • After
    image
    2024-01-11T21:19:20.925 helix_lsp::transport [ERROR] marksman err <- "[21:19:20 WRN] <Doc> Unexpected document version : {\"newVersion\": 52, \"currentVersion\": 47, \"uri\": \"file:///home/kyosuke/Documents/marksman-test/test.md\"}\n"

  • Before
    image

  • After
    image
    2024-01-11T21:28:27.674 helix_lsp::transport [ERROR] marksman err <- "[21:28:27 WRN] <Doc> Unexpected document version : {\"newVersion\": 117, \"currentVersion\": 112, \"uri\": \"file:///home/kyosuke/Documents/marksman-test/test.md\"}\n"

  • Even though I got error logs, referencing works without problems.
    image

Comment

I have no idea if this is happening because of my environment, Marksman or Helix.
If there is any missing information, I'm happy to offer it.

artempyanykh commented

Thanks @Koba0117 for a comprehensive report!

2024-01-11T21:09:13.114 helix_lsp::transport [ERROR] marksman err <- "[21:09:13 INF] <LSP Entry> Starting Marksman LSP  server: {}\n"

This is not an error, this is just Marksman's logging output. I'm not sure why helix marks it with an ERROR tag (perhaps, because it comes from stderr?).

"[21:28:27 WRN] <Doc> Unexpected document version : {\"newVersion\": 117, \"currentVersion\": 112, \"uri\": \"file:///home/kyosuke/Documents/marksman-test/test.md\"}\n"

This is also not an error. I used to have this consistency check but since then realised that none of the editors follow sane document versioning practices. This check (and hence the warning) should no longer be present in the most recent release.

Koba0117 commented

@artempyanykh
Thank you so much for your help!
I understand they are not error logs and there is no problem.

This check (and hence the warning) should no longer be present in the most recent release.

I'm likely using 2023-12-07 version because distro's official repo is still using it. (I flagged it as outdated.)

I was just wondering if those logs caused any problems. So I'm glad to hear that is not the case.
Thank you again for creating this wonderful software!

  • ❤️1
artempyanykh commented

These logs shouldn't cause any problems. And thanks for the kind words @Koba0117!