Merge pull request #1 from CodinGame/fix-unregister-listener-on-dispose

Unregister listener on dispose

Author: CGNonofr

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@codingame/monaco-languageclient-react",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "Monaco Editor React component",
   "scripts": {
     "build": "tsc"

package.json

@@ -23,19 +23,21 @@ function LanguageClient ({
   useEffect(() => {
     console.info(`Starting language server for language ${languageServerConfig.language}`)
     const languageClient = createLanguageClientManager(languageServerUrl, getSecurityToken, languageServerConfig, libraryUrls ?? [])
-    languageClient.onError((error) => {
+    const errorDisposable = languageClient.onError((error) => {
       if (onErrorRef.current != null) {
         onErrorRef.current(error)
       }
     })
-    languageClient.onDidChangeStatus(status => {
+    const statusChangeDisposable = languageClient.onDidChangeStatus(status => {
       if (onDidChangeStatusRef.current != null) {
         onDidChangeStatusRef.current(status)
       }
     })
     const startTimeout = setTimeout(() => languageClient.start())
 
     return () => {
+      errorDisposable.dispose()
+      statusChangeDisposable.dispose()
       console.info('Shutting down language server')
       clearTimeout(startTimeout)
       languageClient.dispose().then(() => {