Polish: LSP and Tools package (#833)

* update ci, readme and contributing.md

* fix

* add Cli.resi and rescript.json

* fixes

* restore some changes

Author: aspeddro

README.md

@@ -176,16 +176,6 @@ A screenshot of the result:
 
 ![Shows the end result in VSCode, with ReScript related files nested under eachother appropriately.](https://user-images.githubusercontent.com/1457626/168123647-400e2f09-31e3-45a2-b74b-190c7c207446.png)
 
-## ⌨️ Use with Other Editors
-
-This repo also contains a language server that can power other editors. **However, the language server in this project is a pure implementation detail. We don't guarantee its stability for other editors' consumption** apart from Vim and Sublime Text.
-
-Still, if you'd like to use this language-server with other editors:
-
-- Get the release binaries from the Github Releases page.
-- Unzip the `.vsix` and get the `server` folder. That's the only folder you need.
-- The language server will be at `server/out/server.js`. Call it through node, and optionally pass `--stdio` if your editor doesn't support the default JSONRPC.
-
 ## 📰 Changelog
 
 See [CHANGELOG](CHANGELOG.md)

tools/package.json

@@ -17,6 +17,7 @@
 		"src/*.res",
 		"src/*.resi",
 		"analysis_binaries/",
+		"rescript.json",
 		"README.md"
 	],
 	"engines": {

tools/rescript.json

@@ -3,8 +3,7 @@
   "version": "0.1.0",
   "sources": [
     {
-      "dir": "src",
-      "public": ["RescriptTools"]
+      "dir": "src"
     }
   ],
   "suffix": ".bs.js",

tools/src/Cli.resi

@@ -0,0 +1 @@
+

tools/src/Tools_Docgen.res

@@ -3,20 +3,20 @@ type field = {
   docstrings: array<string>,
   signature: string,
   optional: bool,
-  deprecated: option<string>,
+  deprecated?: string,
 }
 
 type constructor = {
   name: string,
   docstrings: array<string>,
   signature: string,
-  deprecated: option<string>,
+  deprecated?: string,
 }
 
 @tag("kind")
 type detail =
-  | @as("record") Record(array<field>)
-  | @as("variant") Variant(array<constructor>)
+  | @as("record") Record({items: array<field>})
+  | @as("variant") Variant({items: array<constructor>})
 
 @tag("kind")
 type rec item =
@@ -26,17 +26,17 @@ type rec item =
       docstrings: array<string>,
       signature: string,
       name: string,
-      deprecated: option<string>,
+      deprecated?: string,
     })
   | @as("type")
   Type({
       id: string,
       docstrings: array<string>,
       signature: string,
       name: string,
-      deprecated: option<string>,
+      deprecated?: string,
       /** Additional documentation for constructors and record fields, if available. */
-      detail: option<detail>,
+      detail?: detail,
     })
   | @as("module") Module(docsForModule)
   | @as("moduleAlias")
@@ -49,7 +49,7 @@ type rec item =
 and docsForModule = {
   id: string,
   docstrings: array<string>,
-  deprecated: option<string>,
+  deprecated?: string,
   name: string,
   items: array<item>,
 }
@@ -86,7 +86,7 @@ let decodeDepreacted = item => {
 
 let decodeRecordFields = fields => {
   open Js.Json
-  let fields = fields->Js.Array2.map(field => {
+  let items = fields->Js.Array2.map(field => {
     switch field {
     | Object(doc) => {
         let name = doc->decodeStringByField("name")
@@ -98,32 +98,32 @@ let decodeRecordFields = fields => {
         | _ => assert(false)
         }
 
-        {name, docstrings, signature, optional, deprecated}
+        {name, docstrings, signature, optional, ?deprecated}
       }
 
     | _ => assert(false)
     }
   })
-  Record(fields)
+  Record({items: items})
 }
 
 let decodeConstructorFields = fields => {
   open Js.Json
-  let fields = fields->Js.Array2.map(field => {
+  let items = fields->Js.Array2.map(field => {
     switch field {
     | Object(doc) => {
         let name = doc->decodeStringByField("name")
         let docstrings = doc->decodeDocstrings
         let signature = doc->decodeStringByField("signature")
         let deprecated = doc->decodeDepreacted
 
-        {name, docstrings, signature, deprecated}
+        {name, docstrings, signature, ?deprecated}
       }
 
     | _ => assert(false)
     }
   })
-  Variant(fields)
+  Variant({items: items})
 }
 
 let decodeDetail = detail => {
@@ -151,7 +151,7 @@ let rec decodeValue = item => {
   let name = item->decodeStringByField("name")
   let deprecated = item->decodeDepreacted
   let docstrings = item->decodeDocstrings
-  Value({id, docstrings, signature, name, deprecated})
+  Value({id, docstrings, signature, name, ?deprecated})
 }
 and decodeType = item => {
   let id = item->decodeStringByField("id")
@@ -163,7 +163,7 @@ and decodeType = item => {
   | Some(field) => decodeDetail(field)->Some
   | None => None
   }
-  Type({id, docstrings, signature, name, deprecated, detail})
+  Type({id, docstrings, signature, name, ?deprecated, ?detail})
 }
 and decodeModuleAlias = item => {
   open Js.Json
@@ -186,7 +186,7 @@ and decodeModule = item => {
   | Some(Array(items)) => items->Js.Array2.map(item => decodeItem(item))
   | _ => assert(false)
   }
-  Module({id, name, docstrings, deprecated, items})
+  Module({id, name, docstrings, ?deprecated, items})
 }
 and decodeItem = item => {
   open Js.Json

tools/src/Tools_Docgen.resi

@@ -3,37 +3,37 @@ type field = {
   docstrings: array<string>,
   signature: string,
   optional: bool,
-  deprecated: option<string>,
+  deprecated?: string,
 }
 type constructor = {
   name: string,
   docstrings: array<string>,
   signature: string,
-  deprecated: option<string>,
+  deprecated?: string,
 }
-type detail = Record(array<field>) | Variant(array<constructor>)
+type detail = Record({items: array<field>}) | Variant({items: array<constructor>})
 type rec item =
   | Value({
       id: string,
       docstrings: array<string>,
       signature: string,
       name: string,
-      deprecated: option<string>,
+      deprecated?: string,
     })
   | Type({
       id: string,
       docstrings: array<string>,
       signature: string,
       name: string,
-      deprecated: option<string>,
-      detail: option<detail>,
+      deprecated?: string,
+      detail?: detail,
     })
   | Module(docsForModule)
   | ModuleAlias({id: string, docstrings: array<string>, name: string, items: array<item>})
 and docsForModule = {
   id: string,
   docstrings: array<string>,
-  deprecated: option<string>,
+  deprecated?: string,
   name: string,
   items: array<item>,
 }