Merge pull request #77 from ipfs/feature/config-edit

Add ipfs config edit to cli

Author: daviddias

package.json

@@ -81,6 +81,7 @@
     "lodash.get": "^4.0.0",
     "lodash.set": "^4.0.0",
     "peer-id": "^0.5.0",
-    "ronin": "^0.3.11"
+    "ronin": "^0.3.11",
+    "temp": "^0.8.3"
   }
 }

src/cli/commands/config/edit.js

@@ -1,36 +1,108 @@
 'use strict'
 
 const Command = require('ronin').Command
+const spawn = require('child_process').spawn
+const fs = require('fs')
+const temp = require('temp')
+const async = require('async')
 const IPFS = require('../../../ipfs-core')
 const debug = require('debug')
 const log = debug('cli:version')
 log.error = debug('cli:version:error')
 
 module.exports = Command.extend({
-  desc: 'Shows IPFS version information',
-
-  options: {
-    number: {
-      alias: 'n',
-      type: 'boolean',
-      default: false
-    },
-    commit: {
-      type: 'boolean',
-      default: false
-    },
-    repo: {
-      type: 'boolean',
-      default: false
-    }
-  },
+  desc: 'Opens the config file for editing in $EDITOR',
 
   run: (name) => {
-    var node = new IPFS()
-    node.version((err, version) => {
-      if (err) { return log.error(err) }
+    const node = new IPFS()
+    const editor = process.env.EDITOR
+
+    if (!editor) {
+      throw new Error('ENV variable $EDITOR not set')
+    }
+
+    function getConfig (next) {
+      node.config.show((err, config) => {
+        if (err) {
+          log.error(err)
+          next(new Error('failed to get the config'))
+        }
+
+        next(null, config)
+      })
+    }
+
+    function saveTempConfig (config, next) {
+      temp.open('ipfs-config', (err, info) => {
+        if (err) {
+          log.error(err)
+          next(new Error('failed to open the config'))
+        }
+
+        fs.write(info.fd, JSON.stringify(config, null, 2))
+        fs.close(info.fd, (err) => {
+          if (err) {
+            log.error(err)
+            next(new Error('failed to open the config'))
+          }
+        })
+
+        next(null, info.path)
+      })
+    }
+
+    function openEditor (path, next) {
+      const child = spawn(editor, [path], {
+        stdio: 'inherit'
+      })
+
+      child.on('exit', (err, code) => {
+        if (err) {
+          log.error(err)
+          throw new Error('error on the editor')
+        }
+
+        next(null, path)
+      })
+    }
+
+    function readTempConfig (path, next) {
+      fs.readFile(path, 'utf8', (err, data) => {
+        if (err) {
+          log.error(err)
+          next(new Error('failed to get the updated config'))
+        }
+
+        try {
+          next(null, JSON.parse(data))
+        } catch (err) {
+          log.error(err)
+          next(new Error(`failed to parse the updated config "${err.message}"`))
+        }
+      })
+    }
+
+    function saveConfig (config, next) {
+      node.config.replace(config, (err) => {
+        if (err) {
+          log.error(err)
+          next(new Error('failed to save the config'))
+        }
+
+        next()
+      })
+    }
 
-      console.log(version)
+    async.waterfall([
+      getConfig,
+      saveTempConfig,
+      openEditor,
+      readTempConfig,
+      saveConfig
+    ], (err) => {
+      if (err) {
+        throw err
+      }
     })
   }
 })