chronos migration wip (#225)

* chronos migration wip

- [ ] Need to review `traceAsyncErrors`
- [ ] Handle new `except`s

* Apply suggestions. Reenable suggestapi tests. Remove Exceptions. Adds back traceAsyncErrors

* Progress: Hooks the first call via stdio to the lspclient (still failing)

* Update utils.nim

Co-authored-by: Jacek Sieka <arnetheduck@gmail.com>

* Implements a initial routes and the callAction, now the server can make requests to the client in stdio transport

* Discards notifications when responding

* Progress basic functionality working

* Implements showNotifications. Hook (old) future cancellations

* Implements exits procs

* Small refactor

* progress

* WIP stdio draft implemented

* refactor in preparation to socket support

* WIP getting socket transport to work

* socket transport works for basic functionality (similar to stdio but worst than previous impl). Switch to refc (orc crashes)

* Progress: starts to unify transports

* Progress unifying transports (both now work)

* Progress unifying transports: now both transports uses the same actions

* unify reads as first step to unify the loop

* unifies transports loop

* Completes transport unification

* Refactor: simplifies further the code by making the RPCServer part of the lsp

* Improves stability (for some reason the socket version is more stable than stdio)

* Use chronos cancelation system for canceling requests

* Progress. Improves stability. Switch to chronos recommend way to thread in lsTransport

* wip improves stability and error reporting

* Refactor: cleanup startup

* Autobinds port when not specified

* Fixes an issue where the langserver hanged when failing to parse a file (happens in the prev version too)

* [green] initialize from the client should call initialized on the server

* Refactor: extract lspSocketClient

* specify chronos v4

* update deps cache

* Use `nimble local`

* A better timeout. Uses timeout in the known command as it usually takes too long and hangs

* WIP Tests

* progress

* Test progress: LspClient mostly done now. Second suite almost there

* suite "Suggest api" green

* LSP features green

* tnimlangserver.nim green

* test cleanup

* green: should pick `testproject.nim` as the main file and provide suggestions

* all tests green

* cleanup

* lock deps

* adds back refc

* asyncprocmonitor now compiles in win

* remove comments and extra lines

---------

Co-authored-by: Jacek Sieka <arnetheduck@gmail.com>

Author: jmgomez

.github/workflows/binaries.yml

@@ -106,7 +106,7 @@ jobs:
 
       - name: Restore nimble dependencies from cache
         id: nimble_deps
-        uses: actions/cache@v4
+        uses: actions/cache@v5
         with:
           path: ~/.nimble/
           key: ${{ matrix.target.os }}-${{ env.cache_nonce }}

.github/workflows/ci.yml

@@ -34,7 +34,7 @@ jobs:
       - name: Nimble Install
         shell: bash
         run: |
-          nimble -y install
+          nimble -y install -l
       
       - name: Nimble Test
         shell: bash

asyncprocmonitor.nim

@@ -1,32 +1,29 @@
 # Monitor a client process and shutdown the current process, if the client
 # process is found to be dead
 
-import os, asyncdispatch
+import os, chronos, utils, sugar
 
 when defined(posix):
   import posix_utils
   import posix
 
+type Callback* = proc() {.closure, gcsafe, raises: [].}
+
 when defined(windows):
   import winlean
 
-when defined(windows):
-  proc hookAsyncProcMonitor*(pid: int, cb: Callback) =
-    addProcess(pid, cb)
+  proc hookAsyncProcMonitor*(pid: int, cb: Callback) = discard
+    addProcess2(pid, (arg: pointer) => cb())
 
-when defined(posix):
+elif defined(posix):
   proc hookAsyncProcMonitor*(pid: int, cb: Callback) =
-
     var processExitCallbackCalled = false
 
-    proc checkProcCallback(fd: AsyncFD): bool =
+    proc checkProcCallback(arg: pointer) =
       if not processExitCallbackCalled:
         try:
           sendSignal(Pid(pid), 0)
         except:
           processExitCallbackCalled = true
-          result = cb(fd)
-      else:
-        result = true
 
-    addTimer(1000, false, checkProcCallback)
+    addTimer(1000.int64, checkProcCallback)

config.nims

@@ -4,3 +4,5 @@
 when withDir(thisDir(), system.fileExists("nimble.paths")):
   include "nimble.paths"
 # end Nimble config
+
+--mm:refc