refactor: convert version API to async/await

src/core/components/index.js

@@ -5,6 +5,7 @@ exports.config = require('./config')
 exports.init = require('./init')
 exports.start = require('./start')
 exports.stop = require('./stop')
+exports.version = require('./version')
 
 exports.legacy = { // TODO: these will be removed as the new API is completed
   dag: require('./dag'),

src/core/components/init.js

@@ -301,7 +301,8 @@ function createApi ({
     add,
     config: Commands.config({ repo }),
     init: () => { throw new AlreadyInitializedError() },
-    start
+    start,
+    version: Commands.version({ repo })
   }
 
   return api

src/core/components/start.js

@@ -134,7 +134,8 @@ function createApi ({
     config: Commands.config({ repo }),
     init: () => { throw new AlreadyInitializedError() },
     start: () => apiManager.api,
-    stop
+    stop,
+    version: Commands.version({ repo })
   }
 
   return api

src/core/components/stop.js

@@ -101,7 +101,8 @@ function createApi ({
     config: Commands.config({ repo }),
     init: () => { throw new AlreadyInitializedError() },
     start,
-    stop: () => apiManager.api
+    stop: () => apiManager.api,
+    version: Commands.version({ repo })
   }
 
   return api

src/core/components/version.js

@@ -1,17 +1,16 @@
 'use strict'
 
 const pkg = require('../../../package.json')
-const callbackify = require('callbackify')
 
 // TODO add the commit hash of the current ipfs version to the response.
-module.exports = function version (self) {
-  return callbackify(async () => {
-    const repoVersion = await self.repo.version()
+module.exports = ({ repo }) => {
+  return async function version () {
+    const repoVersion = await repo.version()
 
     return {
       version: pkg.version,
       repo: repoVersion,
       commit: ''
     }
-  })
+  }
 }