feat: pass file name to add/addAll progress handler

Since ipfs/js-ipfs-unixfs#87 landed we can now pass the file name to
the progress handler for adding files:

```js
await ipfs.addAll(..., {
  progress: (bytes, fileName) => {
    //...
  }
})
```

This should make showing progress a bit more usable.

Author: achingbrain

docs/core-api/FILES.md

@@ -252,7 +252,7 @@ An optional object which may have the following keys:
 | hashAlg | `String` | `'sha2-256'` | multihash hashing algorithm to use |
 | onlyHash | `boolean` | `false` | If true, will not add blocks to the blockstore |
 | pin | `boolean` | `true` | pin this object when adding |
-| progress | function | `undefined` | a function that will be called with the byte length of chunks as a file is added to ipfs |
+| progress | function | `undefined` | a function that will be called with the number of bytes added as a file is added to ipfs and the name of the file being added |
 | rawLeaves | `boolean` | `false` | if true, DAG leaves will contain raw file data and not be wrapped in a protobuf |
 | shardSplitThreshold | `Number` | `1000` | Directories with more than this number of files will be created as HAMT-sharded directories |
 | trickle | `boolean` | `false` | if true will use the [trickle DAG](https://godoc.org/github.com/ipsn/go-ipfs/gxlibs/github.com/ipfs/go-unixfs/importer/trickle) format for DAG generation |

packages/interface-ipfs-core/src/add-all.js

@@ -170,6 +170,26 @@ module.exports = (common, options) => {
       expect(root.cid.toString()).to.equal(fixtures.directory.cid)
     })
 
+    it('should receive file name from progress event', async () => {
+      const receivedNames = []
+      function handler (p, name) {
+        receivedNames.push(name)
+      }
+
+      await drain(ipfs.add([{
+        content: 'hello',
+        path: 'foo.txt'
+      }, {
+        content: 'world',
+        path: 'bar.txt'
+      }], {
+        progress: handler,
+        wrapWithDirectory: true
+      }))
+
+      expect(receivedNames).to.deep.equal(['foo.txt', 'bar.txt'])
+    })
+
     it('should add files to a directory non sequentially', async function () {
       const content = path => ({
         path: `test-dir/${path}`,

packages/interface-ipfs-core/src/add.js

@@ -130,6 +130,20 @@ module.exports = (common, options) => {
       expect(accumProgress).to.equal(fixtures.emptyFile.data.length)
     })
 
+    it('should receive file name from progress event', async () => {
+      let receivedName
+      function handler (p, name) {
+        receivedName = name
+      }
+
+      await ipfs.add({
+        content: 'hello',
+        path: 'foo.txt'
+      }, { progress: handler })
+
+      expect(receivedName).to.equal('foo.txt')
+    })
+
     it('should add an empty file without progress enabled', async () => {
       const file = await ipfs.add(fixtures.emptyFile.data)
 

packages/ipfs-core/src/components/add-all/index.js

@@ -47,9 +47,9 @@ module.exports = ({ block, gcLock, preload, pin, options: constructorOptions })
       let total = 0
       const prog = opts.progress
 
-      opts.progress = (bytes) => {
+      opts.progress = (bytes, file) => {
         total += bytes
-        prog(total)
+        prog(total, file)
       }
     }
 
@@ -162,8 +162,8 @@ function pinFile (pin, opts) {
  * @property {boolean} [onlyHash=false] - If true, will not add blocks to the
  * blockstore.
  * @property {boolean} [pin=true] - Pin this object when adding.
- * @property {(bytes:number) => void} [progress] - A function that will be
- * called with the byte length of chunks as a file is added to ipfs.
+ * @property {(bytes:number, fileName:string) => void} [progress] - A function that will be
+ * called with the number of bytes added as a file is added to ipfs and the name of the file being added.
  * @property {boolean} [rawLeaves=false] - If true, DAG leaves will contain raw
  * file data and not be wrapped in a protobuf.
  * @property {number} [shardSplitThreshold=1000] - Directories with more than this

packages/ipfs-http-client/src/add-all.js

@@ -38,7 +38,7 @@ module.exports = configure((api) => {
       if (file.hash !== undefined) {
         yield toCoreInterface(file)
       } else if (progressFn) {
-        progressFn(file.bytes || 0)
+        progressFn(file.bytes || 0, file.name)
       }
     }
   }