Fix dynamic requires

Author: aminya

lib/connection.js

@@ -2,13 +2,21 @@
 import { time } from './misc';
 import externalTerminal from './connection/terminal';
 
+// TODO use babel to export ... from ...
+import IPC from './connection/ipc'
+import messages from './connection/messages'
+import client from './connection/client'
+import local from './connection/local'
+import terminal from './connection/terminal'
+
 export default {
-  //  TODO Fix all of these dynamic requires and circular dependencies
-  IPC:      require('./connection/ipc'),
-  messages: require('./connection/messages'),
-  client:   require('./connection/client'),
-  local:    require('./connection/local'),
-  terminal: require('./connection/terminal'),
+  // TODO remove these from the export default and export them directly (prevents expensive copy)
+  // TODO don't use this.message use message directly (prevents expensive copy)
+  IPC:      IPC,
+  messages: messages,
+  client:   client,
+  local:    local,
+  terminal: terminal,
 
   activate() {
     this.messages.activate();