From a872ad9d8bd337c78375a2a0d9076e461d95e6ff Mon Sep 17 00:00:00 2001
From: Dick Choi <fluke8259@gmail.com>
Date: Tue, 15 Dec 2015 13:43:28 +0900
Subject: [PATCH] change IPC module(Raw buffer -> Socket) & set window
 behaviour for Windows

---
 atom-lib/finder-window.js |  19 ++++---
 browser/finder/index.html |   3 +-
 browser/finder/index.js   |   4 +-
 browser/main/index.html   |   5 +-
 finder.js                 | 117 ++++++++++++++++++++++++++++----------
 lib/dataStore.js          |  14 ++++-
 main.js                   | 105 ++++++++++++++++++----------------
 package.json              |   6 +-
 8 files changed, 179 insertions(+), 94 deletions(-)

diff --git a/atom-lib/finder-window.js b/atom-lib/finder-window.js
index 466c40b5..58e25e56 100644
--- a/atom-lib/finder-window.js
+++ b/atom-lib/finder-window.js
@@ -2,29 +2,32 @@ const electron = require('electron')
 const BrowserWindow = electron.BrowserWindow
 const path = require('path')
 
-var finderWindow = new BrowserWindow({
+var config = {
   width: 640,
   height: 400,
   show: false,
   frame: false,
   resizable: false,
   'zoom-factor': 1.0,
-  'always-on-top': true,
   'web-preferences': {
     'overlay-scrollbars': true,
     'skip-taskbar': true
   },
   'standard-window': false
-})
+}
+
+if (process.platform === 'darwin') {
+  config['always-on-top'] = true
+}
+
+var finderWindow = new BrowserWindow(config)
 
 var url = path.resolve(__dirname, '../browser/finder/index.html')
 
 finderWindow.loadURL('file://' + url)
 
-finderWindow.on('blur', function () {
-  finderWindow.hide()
-})
-
-finderWindow.setVisibleOnAllWorkspaces(true)
+if (process.platform === 'darwin') {
+  finderWindow.setVisibleOnAllWorkspaces(true)
+}
 
 module.exports = finderWindow
diff --git a/browser/finder/index.html b/browser/finder/index.html
index 5c64764f..1d9dfd6b 100644
--- a/browser/finder/index.html
+++ b/browser/finder/index.html
@@ -30,7 +30,8 @@
   <script>
     const electron = require('electron')
     electron.webFrame.setZoomLevelLimits(1, 1)
-    var scriptUrl = process.env.BOOST_ENV === 'development'
+    const _ = require('lodash')
+    var scriptUrl = _.find(electron.remote.process.argv, a => a === '--hot')
       ? 'http://localhost:8080/assets/finder.js'
       : '../../compiled/finder.js'
     var scriptEl=document.createElement('script')
diff --git a/browser/finder/index.js b/browser/finder/index.js
index 36360693..596ccd31 100644
--- a/browser/finder/index.js
+++ b/browser/finder/index.js
@@ -13,7 +13,9 @@ import dataStore from 'boost/dataStore'
 const electron = require('electron')
 const { remote, clipboard, ipcRenderer } = electron
 
-var hideFinder = remote.getGlobal('hideFinder')
+function hideFinder () {
+  ipcRenderer.send('hide-finder')
+}
 
 function notify (...args) {
   return new window.Notification(...args)
diff --git a/browser/main/index.html b/browser/main/index.html
index 5db97120..667fcc80 100644
--- a/browser/main/index.html
+++ b/browser/main/index.html
@@ -58,10 +58,11 @@
     electron.webFrame.setZoomLevelLimits(1, 1)
     var version = electron.remote.app.getVersion()
     document.title = 'Boost' + ((version == null || version.length === 0) ? ' DEV' : '')
-    var scriptUrl = process.env.BOOST_ENV === 'development'
+    const _ = require('lodash')
+    var scriptUrl = _.find(electron.remote.process.argv, a => a === '--hot')
       ? 'http://localhost:8080/assets/main.js'
       : '../../compiled/main.js'
-    var scriptEl=document.createElement('script')
+    var scriptEl = document.createElement('script')
     scriptEl.setAttribute("type","text/javascript")
     scriptEl.setAttribute("src", scriptUrl)
     document.getElementsByTagName("head")[0].appendChild(scriptEl)
diff --git a/finder.js b/finder.js
index 627a3cbd..ca5c0b63 100755
--- a/finder.js
+++ b/finder.js
@@ -4,26 +4,84 @@ const Tray = electron.Tray
 const Menu = electron.Menu
 const MenuItem = electron.MenuItem
 const ipcMain = electron.ipcMain
+const path = require('path')
 
-process.stdin.setEncoding('utf8')
+const nodeIpc = require('node-ipc')
 
-console.log = function () {
-  process.stdout.write(JSON.stringify({
-    type: 'log',
-    data: JSON.stringify(Array.prototype.slice.call(arguments).join(' '))
-  }), 'utf-8')
+var finderWindow = null
+var isFinderLoaded = false
+
+function hideFinder () {
+  if (!isFinderLoaded) return false
+
+  if (process.platform === 'darwin') {
+    Menu.sendActionToFirstResponder('hide:')
+  }
+  if (process.platform === 'win32') {
+    finderWindow.minimize()
+  }
+  finderWindow.hide()
 }
 
+function showFinder () {
+  if (!isFinderLoaded) return false
+
+  if (!finderWindow.isVisible()) {
+    finderWindow.show()
+  }
+  if (process.platform === 'win32') {
+    finderWindow.minimize()
+    finderWindow.restore()
+  }
+}
+
+nodeIpc.config.id = 'finder'
+nodeIpc.config.retry = 1500
+nodeIpc.config.silent = true
+
+nodeIpc.connectTo(
+  'main',
+  path.join(app.getPath('userData'), 'boost.service'),
+  function () {
+      nodeIpc.of.main.on(
+        'connect',
+        function () {
+          nodeIpc.log('<< ## connected to world ##'.rainbow, nodeIpc.config.delay)
+        }
+      )
+      nodeIpc.of.main.on(
+        'disconnect',
+        function(){
+          nodeIpc.log('<< disconnected from main'.notice)
+        }
+      )
+      nodeIpc.of.main.on(
+        'message',
+        function (payload) {
+          switch (payload.type) {
+            case 'open-finder':
+              showFinder()
+              break
+          }
+        }
+      )
+  }
+)
+
 function emit (type, data) {
-  process.stdout.write(JSON.stringify({
+  var payload = {
     type: type,
-    data: JSON.stringify(data)
-  }), 'utf-8')
+    data: data
+  }
+  nodeIpc.of.main.emit('message', payload)
 }
 
-var finderWindow
+var appQuit = false
 app.on('ready', function () {
-  app.dock.hide()
+  if (process.platform === 'darwin') {
+    app.dock.hide()
+  }
+
   var appIcon = new Tray(__dirname + '/resources/tray-icon.png')
   appIcon.setToolTip('Boost')
 
@@ -43,7 +101,7 @@ app.on('ready', function () {
     trayMenu.append(new MenuItem({
       label: 'Open Finder window',
       click: function () {
-        finderWindow.show()
+        showFinder()
       }
     }))
     trayMenu.append(new MenuItem({
@@ -53,29 +111,30 @@ app.on('ready', function () {
       }
     }))
     appIcon.setContextMenu(trayMenu)
-
-    process.stdin.on('data', function (payload) {
-      try {
-        payload = JSON.parse(payload)
-      } catch (e) {
-        console.log('Not parsable payload : ', payload)
-        return
-      }
-      console.log('from main >> ', payload.type)
-      switch (payload.type) {
-        case 'open-finder':
-          finderWindow.show()
-          break
-      }
+    appIcon.on('click', function (e) {
+      e.preventDefault()
+      appIcon.popUpContextMenu(trayMenu)
     })
 
     ipcMain.on('copy-finder', function () {
       emit('copy-finder')
     })
+
+    ipcMain.on('hide-finder', function () {
+      hideFinder()
+    })
+
+    isFinderLoaded = true
   })
 
-  global.hideFinder = function () {
-    Menu.sendActionToFirstResponder('hide:')
-  }
+  finderWindow.on('blur', function () {
+    hideFinder()
+  })
+
+  finderWindow.on('close', function (e) {
+    if (appQuit) return true
+    e.preventDefault()
+    hideFinder()
+  })
 })
 
diff --git a/lib/dataStore.js b/lib/dataStore.js
index e8b62af4..15017d01 100644
--- a/lib/dataStore.js
+++ b/lib/dataStore.js
@@ -13,14 +13,22 @@ function getLocalPath () {
 }
 
 function forgeInitialRepositories () {
-  return [{
+  let defaultRepo = {
     key: keygen(),
     name: 'local',
     type: 'userData',
     user: {
-      name: remote.getGlobal('process').env.USER
+      name: 'New user'
     }
-  }]
+  }
+
+  if (process.platform === 'darwin') {
+    defaultRepo.user.name = remote.process.env.USER
+  } else if (process.platform === 'win32') {
+    defaultRepo.user.name = remote.process.env.USERNAME
+  }
+
+  return [defaultRepo]
 }
 
 function getRepositories () {
diff --git a/main.js b/main.js
index 205f60a5..b543c217 100644
--- a/main.js
+++ b/main.js
@@ -7,7 +7,8 @@ const autoUpdater = electron.autoUpdater
 const jetpack = require('fs-jetpack')
 const path = require('path')
 const ChildProcess = require('child_process')
-electron.crashReporter.start()
+const _ = require('lodash')
+// electron.crashReporter.start()
 
 var mainWindow = null
 var finderProcess
@@ -61,6 +62,44 @@ autoUpdater
     }
   })
 
+
+const nodeIpc = require('node-ipc')
+var isNodeIpcReady = false
+nodeIpc.config.id   = 'node'
+nodeIpc.config.retry= 1500
+nodeIpc.config.silent = true
+
+nodeIpc.serve(
+  path.join(app.getPath('userData'), 'boost.service'),
+  function(){
+    isNodeIpcReady = true
+    nodeIpc.server.on(
+      'message',
+      function (data, socket){
+        console.log('>>', data)
+        format(data)
+      }
+    )
+  }
+)
+
+
+function format (payload) {
+  switch (payload.type) {
+    case 'show-main-window':
+      mainWindow.minimize()
+      mainWindow.restore()
+      break
+    case 'copy-finder':
+      mainWindow.webContents.send('copy-finder')
+      break
+    case 'quit-app':
+      appQuit = true
+      app.quit()
+      break
+  }
+}
+
 app.on('ready', function () {
   app.on('before-quit', function () {
     if (finderProcess) finderProcess.kill()
@@ -93,14 +132,26 @@ app.on('ready', function () {
     e.preventDefault()
     mainWindow.hide()
   })
+
+  function emitToFinder (type, data) {
+    if (!isNodeIpcReady) {
+      console.log('server is not ready')
+    }
+    var payload = {
+      type: type,
+      data: data
+    }
+    nodeIpc.server.broadcast('message', payload)
+  }
+
   mainWindow.webContents.on('did-finish-load', function () {
     if (finderProcess == null) {
+      var finderArgv = [path.resolve(__dirname, 'finder.js'), '--finder']
+      if (_.find(process.argv, a => a === '--hot')) finderArgv.push('--hot')
       finderProcess = ChildProcess
-        .execFile(process.execPath, [path.resolve(__dirname, 'finder.js'), '--finder'])
-      finderProcess.stdout.setEncoding('utf8')
-      finderProcess.stderr.setEncoding('utf8')
-      finderProcess.stdout.on('data', format)
-      finderProcess.stderr.on('data', errorFormat)
+        .execFile(process.execPath, finderArgv)
+
+      nodeIpc.server.start()
     }
 
     if (update != null) {
@@ -115,48 +166,6 @@ app.on('ready', function () {
     mainWindow.show()
   })
 
-  function format (payload) {
-    // console.log('from finder >> ', payload)
-    try {
-      payload = JSON.parse(payload)
-    } catch (e) {
-      console.log('Not parsable payload : ', payload)
-      return
-    }
-    switch (payload.type) {
-      case 'log':
-        console.log('FINDER(stdout): ' + payload.data)
-        break
-      case 'show-main-window':
-        mainWindow.show()
-        break
-      case 'copy-finder':
-        mainWindow.webContents.send('copy-finder')
-        break
-      case 'request-data':
-        mainWindow.webContents.send('request-data')
-        break
-      case 'quit-app':
-        appQuit = true
-        app.quit()
-        break
-    }
-  }
-  function errorFormat (output) {
-    console.error('FINDER(stderr):' + output)
-  }
-
-  function emitToFinder (type, data) {
-    if (!finderProcess) {
-      console.log('finder process is not ready')
-      return
-    }
-    var payload = {
-      type: type,
-      data: data
-    }
-    finderProcess.stdin.write(JSON.stringify(payload), 'utf-8')
-  }
 
   var userDataPath = app.getPath('userData')
   if (!jetpack.cwd(userDataPath).exists('keymap.json')) {
diff --git a/package.json b/package.json
index fb0d247a..9fc50387 100644
--- a/package.json
+++ b/package.json
@@ -4,9 +4,10 @@
   "description": "Boost App",
   "main": "index.js",
   "scripts": {
-    "start": "set BOOST_ENV=development&& electron ./main.js",
+    "start": "electron ./index.js",
+    "hot": "electron ./index.js --hot",
     "webpack": "webpack-dev-server --hot --inline --config webpack.config.js",
-    "compile": "NODE_ENV=production webpack --config webpack.config.production.js",
+    "compile": "webpack --config webpack.config.production.js",
     "pack:osx": "electron-packager ./ Boost --app-version=$npm_package_version $npm_package_config_platform  $npm_package_config_version $npm_package_config_ignore --overwrite --out=\"dist\"",
     "codesign": "codesign --verbose --deep --force --sign \"MAISIN solutions Inc.\" dist/Boost-darwin-x64/Boost.app",
     "build:osx": "electron-builder \"dist/Boost-darwin-x64/Boost.app\" --platform=osx --out=\"dist\" --config=\"./builder-config.json\"",
@@ -48,6 +49,7 @@
     "markdown-it-emoji": "^1.1.0",
     "md5": "^2.0.0",
     "moment": "^2.10.3",
+    "node-ipc": "^5.0.0",
     "socket.io-client": "^1.3.6",
     "superagent": "^1.2.0",
     "superagent-promise": "^1.0.3"