fix: remove non existent commands (#925)

src/cli/bin.js

@@ -39,33 +39,25 @@ const args = process.argv.slice(2)
 // Need to skip to avoid locking as these commands
 // don't require a daemon
 if (args[0] === 'daemon' || args[0] === 'init') {
-  return cli
+  cli
     .help()
     .strict(false)
     .completion()
     .parse(args)
-}
-
-utils.getIPFS((err, ipfs, cleanup) => {
-  if (err) {
-    throw err
-  }
-
-  // finalize cli setup
-  cli // eslint-disable-line
-    .help()
-    .strict(false)
-    .completion()
-    .parse(args, {
-      ipfs: ipfs
-    }, (err, argv, output) => {
-      if (output) {
-        console.log(output)
-      }
-      cleanup(() => {
-        if (err) {
-          throw err
-        }
+} else {
+  utils.getIPFS((err, ipfs, cleanup) => {
+    if (err) { throw err }
+
+    cli
+      .help()
+      .strict(false)
+      .completion()
+      .parse(args, { ipfs: ipfs }, (err, argv, output) => {
+        if (output) { console.log(output) }
+
+        cleanup(() => {
+          if (err) { throw err }
+        })
       })
-    })
-})
+  })
+}

src/cli/commands/bitswap.js

@@ -3,13 +3,13 @@
 module.exports = {
   command: 'bitswap',
 
-  description: 'A set of commands to manipulate the bitswap agent.',
+  description: 'Interact with the bitswap agent.',
 
   builder (yargs) {
-    return yargs
-      .commandDir('bitswap')
+    return yargs.commandDir('bitswap')
   },
 
   handler (argv) {
+    console.log('Type `jsipfs bitswap --help` for more information about this command')
   }
 }

src/cli/commands/files.js

@@ -3,12 +3,14 @@
 module.exports = {
   command: 'files',
 
-  description: 'Unixfs commands',
+  description: 'Operations over files (add, cat, get, ls, etc)',
 
   builder (yargs) {
     return yargs
       .commandDir('files')
   },
 
-  handler (argv) {}
+  handler (argv) {
+    console.log('Type `jsipfs bitswap --help` for more instructions')
+  }
 }

src/cli/commands/files/add.js

@@ -36,6 +36,52 @@ function checkPath (inPath, recursive) {
   return inPath
 }
 
+function addPipeline (index, addStream, list, wrapWithDirectory) {
+  pull(
+    zip(
+      pull.values(list),
+      pull(
+        pull.values(list),
+        paramap(fs.stat.bind(fs), 50)
+      )
+    ),
+    pull.map((pair) => ({
+      path: pair[0],
+      isDirectory: pair[1].isDirectory()
+    })),
+    pull.filter((file) => !file.isDirectory),
+    pull.map((file) => ({
+      path: file.path.substring(index, file.path.length),
+      originalPath: file.path
+    })),
+    pull.map((file) => ({
+      path: wrapWithDirectory ? path.join(WRAPPER, file.path) : file.path,
+      content: fs.createReadStream(file.originalPath)
+    })),
+    addStream,
+    pull.map((file) => ({
+      hash: file.hash,
+      path: wrapWithDirectory ? file.path.substring(WRAPPER.length) : file.path
+    })),
+    pull.collect((err, added) => {
+      if (err) {
+        throw err
+      }
+
+      sortBy(added, 'path')
+        .reverse()
+        .map((file) => {
+          const log = [ 'added', file.hash ]
+
+          if (file.path.length > 0) log.push(file.path)
+
+          return log.join(' ')
+        })
+        .forEach((msg) => console.log(msg))
+    })
+  )
+}
+
 module.exports = {
   command: 'add <file>',
 
@@ -112,49 +158,3 @@ module.exports = {
     })
   }
 }
-
-function addPipeline (index, addStream, list, wrapWithDirectory) {
-  pull(
-    zip(
-      pull.values(list),
-      pull(
-        pull.values(list),
-        paramap(fs.stat.bind(fs), 50)
-      )
-    ),
-    pull.map((pair) => ({
-      path: pair[0],
-      isDirectory: pair[1].isDirectory()
-    })),
-    pull.filter((file) => !file.isDirectory),
-    pull.map((file) => ({
-      path: file.path.substring(index, file.path.length),
-      originalPath: file.path
-    })),
-    pull.map((file) => ({
-      path: wrapWithDirectory ? path.join(WRAPPER, file.path) : file.path,
-      content: fs.createReadStream(file.originalPath)
-    })),
-    addStream,
-    pull.map((file) => ({
-      hash: file.hash,
-      path: wrapWithDirectory ? file.path.substring(WRAPPER.length) : file.path
-    })),
-    pull.collect((err, added) => {
-      if (err) {
-        throw err
-      }
-
-      sortBy(added, 'path')
-        .reverse()
-        .map((file) => {
-          const log = [ 'added', file.hash ]
-
-          if (file.path.length > 0) log.push(file.path)
-
-          return log.join(' ')
-        })
-        .forEach((msg) => console.log(msg))
-    })
-  )
-}

src/cli/commands/files/cat.js

@@ -3,7 +3,7 @@
 module.exports = {
   command: 'cat <ipfs-path>',
 
-  describe: 'Download IPFS objects',
+  describe: 'Fetch and cat an IPFS path referencing a file',
 
   builder: {},
 

src/cli/commands/files/get.js

@@ -50,7 +50,7 @@ function fileHandler (dir) {
 module.exports = {
   command: 'get <ipfs-path>',
 
-  describe: 'Download IPFS objects',
+  describe: 'Fetch a file or directory with files references from an IPFS Path',
 
   builder: {
     output: {

test/cli/commands.js

@@ -4,7 +4,7 @@
 const expect = require('chai').expect
 const runOnAndOff = require('../utils/on-and-off')
 
-const commandCount = 63
+const commandCount = 56
 
 describe('commands', () => runOnAndOff((thing) => {
   let ipfs