feat(overmind): use single WS server for devtools

Author: eirikhm

packages/node_modules/overmind-devtools-client/DevtoolBackend.js

@@ -1,24 +1,38 @@
 const WebSocket = require('ws')
 
+
+
 class DevtoolBackend {
   constructor(options) {
     this.options = options
-
     this.onError = this.onError.bind(this)
+
     this.onClientConnection = this.onClientConnection.bind(this)
     this.onDevtoolConnection = this.onDevtoolConnection.bind(this)
+    this.onConnection = this.onConnection.bind(this)
     this.onClientMessage = this.onClientMessage.bind(this)
     this.onDevtoolMessage = this.onDevtoolMessage.bind(this)
-
+    
     this.devtoolServer = new WebSocket.Server(
       {
         port: options.port,
       },
       () => {}
     )
 
-    this.devtoolServer.on('connection', this.onDevtoolConnection)
+    this.devtoolServer.on('connection', this.onConnection)
+    this.devtoolServer.on('error', this.onError)
+  }
+
+  onConnection(ws, req) {
+    // devtools connects with ?devtools=1 in url
+    if (req.url.indexOf("devtools") !== -1) {
+      this.onDevtoolConnection(ws, req)
+    } else {
+      this.onClientConnection(ws, req)
+    }
   }
+
   onDevtoolConnection(ws) {
     this.devtoolSocket = ws
     this.devtoolSocket.on('message', this.onDevtoolMessage)
@@ -46,14 +60,16 @@ class DevtoolBackend {
         this.options.onRelaunch()
         break
       case 'connect':
-        this.createClientServer(parsedMessage.data)
+          console.log('TODO: connect'); // TODO: Let devtools know that they are connected to server?
+        // this.createClientServer(parsedMessage.data)
         break
       default:
         this.clientSocket.send(JSON.stringify(parsedMessage.data))
     }
   }
   onError() {
-    this.devtoolServer.emit('port:exists')
+    console.log('TODO: onError');
+    // this.devtoolServer.emit('port:exists')
   }
   onClientConnection(ws) {
     this.clientSocket = ws
@@ -82,23 +98,6 @@ class DevtoolBackend {
       )
     }
   }
-  createClientServer(port) {
-    if (this.clientPort === Number(port)) {
-      return
-    }
-
-    if (this.clientServer) {
-      this.clientServer.close()
-    }
-
-    this.clientPort = Number(port)
-    this.clientServer = new WebSocket.Server(
-      { port: this.clientPort },
-      () => {}
-    )
-    this.clientServer.on('connection', this.onClientConnection)
-    this.clientServer.on('error', this.onError)
-  }
 }
 
 DevtoolBackend.create = (options) => new DevtoolBackend(options)