From 4d43130788d7373a2cc3fda47299586941d86b21 Mon Sep 17 00:00:00 2001
From: Addison Higham <addisonj@gmail.com>
Date: Sun, 28 Apr 2024 06:14:22 -0600
Subject: [PATCH] Fix bug with overriding options object (#19)

Recently, with the addition of supporting running LSP as a standalone
process, the `connect` function was changed.

This changed the promise such that the options object was now the result
of the `connection.onInitialized` function rather than the result of
the callback from `connection.onInitialize`, which seems like an error
in understanding the API of the LSP library

This reverts that change, but still returns the value options out of the
promise, though it is likely an uneeded change.

Automated testing for this would be ideal, but as there doesn't seem to
be any infrastructure for that currently, that is out of scope of this
commit
---
 server/server.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/server/server.ts b/server/server.ts
index d3dd032..3cd297f 100644
--- a/server/server.ts
+++ b/server/server.ts
@@ -29,7 +29,7 @@ export function connect(
   });
 
   return new Promise((resolve) => {
-    const options = connection.onInitialized(() => resolve(options));
+    connection.onInitialized(() => resolve(options));
     connection.listen();
     return options
   });