jaredly / jaredly/reason-language-server

bs-platform 5.1.0 incompatibility breaks reason-vscode or lsp

Open
#320 36 comments 0 reactions 0 assignees View on GitHub
Dominant language
OCaml
Stars
649
Forks
85
PR merge metrics
No merged PRs in 30d

Description

When upgrading to bs-platform 5.1.0 on a ReasonReact project for example when opening this project using vscode:

https://github.com/sync/reason-graphql-demo

I get the follow error:

```
Request textDocument/documentSymbol failed.
Message: Failed to load /Users/anthonymittaz/Projects/Reason/hackerz/node_modules/.lsp/App-Hackerz.cmt as a cmt w/ ocaml version 402, error: /Users/anthonymittaz/Projects/Reason/hackerz/node_modules/.lsp/App-Hackerz.cmt
is not a compiled interface
Code: -32603
```

See attached debug log
[debug.log](https://github.com/jaredly/reason-language-server/files/3511687/debug.log)

Contributor guide

No contributing guide indexed for this repository

Research direction

Reproduce the failure in the linked reason-graphql-demo project with bs-platform 5.1.0, then read the attached debug.log and trace the documentSymbol request that loads the .cmt file. Done means opening the project in VS Code no longer reports that the compiled interface is invalid.

Written by the indexing model from the issue text.

Assessment

Tech stack
ocaml, vscode
Domain
devtools
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.