lint

.github/workflows/ci.yml

@@ -14,7 +14,7 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest, windows-latest, macOS-latest]
-        node: ["15", "14", "12", engines]
+        node: ["14", "12", engines]
         exclude:
           # On Windows, run tests with only the LTS environments.
           - os: windows-latest

.gitignore

@@ -63,4 +63,3 @@ typings/
 .env
 
 index.d.ts
-from.d.ts

CHANGELOG.md

@@ -1,25 +1,6 @@
 Changelog
 =========
 
-## next
-- Fixed a bug where in BlobDataItem when the file was empty (#86)
-
-## v2.1.2
-- Fixed a bug where `start` in BlobDataItem was undefined (#85)
-
-## v2.1.1
-- Add nullish values checking in Symbol.hasInstance (#82)
-- Add generated typings for from.js file (#80)
-- Updated dev dependencies
-
-## v2.1.0
-- Fix: .slice has an implementation bug (#54).
-- Added blob backed up by filesystem (#55)
-
-## v2.0.1
-
-- Fix: remove upper bound for node engine semver (#49).
-
 ## v2.0.0
 
 > Note: This release was previously published as `1.0.7`, but as it contains breaking changes, we renamed it to `2.0.0`.

README.md

@@ -27,24 +27,6 @@ fetch('https://httpbin.org/post', {
     .then(json => console.log(json));
 ```
 
-### Blob part backed up by filesystem
-To use, install [domexception](https://github.com/jsdom/domexception).
-
-```sh
-npm install fetch-blob domexception
-```
-
-```js
-const blobFrom = require('fetch-blob/from.js');
-const blob1 = blobFrom('./2-GiB-file.bin');
-const blob2 = blobFrom('./2-GiB-file.bin');
-
-// Not a 4 GiB memory snapshot, just holds 3 references
-// points to where data is located on the disk
-const blob = new Blob([blob1, blob2]);
-console.log(blob.size) // 4 GiB
-```
-
 See the [MDN documentation](https://developer.mozilla.org/en-US/docs/Web/API/Blob) and [tests](https://github.com/node-fetch/fetch-blob/blob/master/test.js) for more details.
 
 [npm-image]: https://flat.badgen.net/npm/v/fetch-blob

from.js

@@ -1,61 +0,0 @@
-const {statSync, createReadStream} = require('fs');
-const Blob = require('./index.js');
-const DOMException = require('domexception');
-
-/**
- * @param {string} path filepath on the disk
- * @returns {Blob}
- */
-function blobFrom(path) {
-	const {size, mtime} = statSync(path);
-	const blob = new BlobDataItem({path, size, mtime});
-
-	return new Blob([blob]);
-}
-
-/**
- * This is a blob backed up by a file on the disk
- * with minium requirement
- *
- * @private
- */
-class BlobDataItem {
-	constructor(options) {
-		this.size = options.size;
-		this.path = options.path;
-		this.start = options.start || 0;
-		this.mtime = options.mtime;
-	}
-
-	// Slicing arguments is first validated and formated
-	// to not be out of range by Blob.prototype.slice
-	slice(start, end) {
-		return new BlobDataItem({
-			path: this.path,
-			start,
-			mtime: this.mtime,
-			size: end - start
-		});
-	}
-
-	stream() {
-		if (statSync(this.path).mtime > this.mtime) {
-			throw new DOMException('The requested file could not be read, typically due to permission problems that have occurred after a reference to a file was acquired.', 'NotReadableError');
-		}
-
-		if (!this.size) {
-			return new Blob().stream();
-		}
-
-		return createReadStream(this.path, {
-			start: this.start,
-			end: this.start + this.size - 1
-		});
-	}
-
-	get [Symbol.toStringTag]() {
-		return 'Blob';
-	}
-}
-
-module.exports = blobFrom;

index.js

@@ -15,6 +15,20 @@ async function * read(parts) {
 	}
 }
 
+/**
+ * @template T
+ * @param {T} object
+ * @returns {T is Blob}
+ */
+const isBlob = object => {
+	return (
+		typeof object === 'object' &&
+		typeof object.stream === 'function' &&
+		typeof object.constructor === 'function' &&
+		/^(Blob|File)$/.test(object[Symbol.toStringTag])
+	);
+};
+
 class Blob {
 	/**
 	 * The Blob() constructor returns a new Blob object. The content
@@ -24,24 +38,23 @@ class Blob {
 	 * @param {(ArrayBufferLike | ArrayBufferView | Blob | Buffer | string)[]} blobParts
 	 * @param {{ type?: string }} [options]
 	 */
-	constructor(blobParts = [], options = {}) {
+	constructor(blobParts = [], options = {type: ''}) {
 		let size = 0;
 
 		const parts = blobParts.map(element => {
 			let buffer;
-			if (element instanceof Buffer) {
+			if (Buffer.isBuffer(element)) {
 				buffer = element;
 			} else if (ArrayBuffer.isView(element)) {
 				buffer = Buffer.from(element.buffer, element.byteOffset, element.byteLength);
 			} else if (element instanceof ArrayBuffer) {
 				buffer = Buffer.from(element);
-			} else if (element instanceof Blob) {
+			} else if (isBlob(element)) {
 				buffer = element;
 			} else {
 				buffer = Buffer.from(typeof element === 'string' ? element : String(element));
 			}
 
-			// eslint-disable-next-line unicorn/explicit-length-check
 			size += buffer.length || buffer.size || 0;
 			return buffer;
 		});
@@ -139,7 +152,7 @@ class Blob {
 			} else {
 				const chunk = part.slice(relativeStart, Math.min(size, relativeEnd));
 				blobParts.push(chunk);
-				added += ArrayBuffer.isView(chunk) ? chunk.byteLength : chunk.size;
+				added += size;
 				relativeStart = 0; // All next sequental parts should start at 0
 
 				// don't add the overflow to new blobParts
@@ -149,26 +162,11 @@ class Blob {
 			}
 		}
 
-		const blob = new Blob([], {type: String(type).toLowerCase()});
+		const blob = new Blob([], {type});
 		Object.assign(wm.get(blob), {size: span, parts: blobParts});
 
 		return blob;
 	}
-
-	get [Symbol.toStringTag]() {
-		return 'Blob';
-	}
-
-	static [Symbol.hasInstance](object) {
-		return (
-			object &&
-			typeof object === 'object' &&
-			typeof object.stream === 'function' &&
-			object.stream.length === 0 &&
-			typeof object.constructor === 'function' &&
-			/^(Blob|File)$/.test(object[Symbol.toStringTag])
-		);
-	}
 }
 
 Object.defineProperties(Blob.prototype, {
@@ -177,4 +175,11 @@ Object.defineProperties(Blob.prototype, {
 	slice: {enumerable: true}
 });
 
+Object.defineProperty(Blob.prototype, Symbol.toStringTag, {
+	value: 'Blob',
+	writable: false,
+	enumerable: false,
+	configurable: true
+});
+
 module.exports = Blob;

package.json

@@ -1,20 +1,18 @@
 {
     "name": "fetch-blob",
-    "version": "2.1.2",
+    "version": "2.0.0",
     "description": "A Blob implementation in Node.js, originally from node-fetch.",
     "main": "index.js",
     "files": [
-        "from.js",
         "index.js",
-        "index.d.ts",
+        "index.d.ts"
-        "from.d.ts"
     ],
     "scripts": {
         "lint": "xo",
         "test": "xo && ava",
         "report": "c8 --reporter json --reporter text ava",
         "coverage": "c8 --reporter json --reporter text ava && codecov -f coverage/coverage-final.json",
-        "prepublishOnly": "tsc --declaration --emitDeclarationOnly --allowJs index.js from.js"
+        "prepublishOnly": "tsc --declaration --emitDeclarationOnly --allowJs index.js"
     },
     "repository": "https://github.com/node-fetch/fetch-blob.git",
     "keywords": [
@@ -22,7 +20,7 @@
         "node-fetch"
     ],
     "engines": {
-        "node": "^10.17.0 || >=12.3.0"
+        "node": "^10.17.0"
     },
     "author": "David Frank",
     "license": "MIT",
@@ -30,17 +28,16 @@
         "url": "https://github.com/node-fetch/fetch-blob/issues"
     },
     "homepage": "https://github.com/node-fetch/fetch-blob#readme",
+    "devDependencies": {
+        "ava": "^3.8.2",
+        "c8": "^7.2.0",
+        "codecov": "^3.7.0",
+        "get-stream": "^5.1.0",
+        "node-fetch": "^2.6.0",
+        "typescript": "^3.9.5",
+        "xo": "^0.32.0"
+    },
     "xo": {
-        "rules": {
-            "unicorn/import-index": "off",
-            "import/extensions": [
-                "error",
-                "always",
-                {
-                    "ignorePackages": true
-                }
-            ]
-        },
         "overrides": [
             {
                 "files": "test.js",
@@ -51,19 +48,5 @@
             }
         ]
     },
-    "peerDependenciesMeta": {
+    "dependencies": {}
-        "domexception": {
-            "optional": true
-        }
-    },
-    "devDependencies": {
-        "ava": "^3.15.0",
-        "c8": "^7.7.1",
-        "codecov": "^3.8.1",
-        "domexception": "^2.0.1",
-        "get-stream": "^6.0.1",
-        "node-fetch": "^2.6.1",
-        "typescript": "^4.2.4",
-        "xo": "^0.38.2"
-    }
 }

test.js

@@ -1,10 +1,8 @@
-const fs = require('fs');
 const test = require('ava');
+const Blob = require('.');
 const getStream = require('get-stream');
 const {Response} = require('node-fetch');
 const {TextDecoder} = require('util');
-const Blob = require('./index.js');
-const blobFrom = require('./from.js');
 
 test('new Blob()', t => {
 	const blob = new Blob(); // eslint-disable-line no-unused-vars
@@ -116,12 +114,6 @@ test('Blob slice(0, -1)', async t => {
 	t.is(await blob.text(), 'abcdefg');
 });
 
-test('Blob(["hello ", "world"]).slice(5)', async t => {
-	const parts = ['hello ', 'world'];
-	const blob = new Blob(parts);
-	t.is(await blob.slice(5).text(), ' world');
-});
-
 test('throw away unwanted parts', async t => {
 	const blob = new Blob(['a', 'b', 'c']).slice(1, 2);
 	t.is(await blob.text(), 'b');
@@ -144,52 +136,3 @@ test('Blob works with node-fetch Response.text()', async t => {
 	const text = await response.text();
 	t.is(text, data);
 });
-
-test('blob part backed up by filesystem', async t => {
-	const blob = blobFrom('./LICENSE');
-	t.is(await blob.slice(0, 3).text(), 'MIT');
-	t.is(await blob.slice(4, 11).text(), 'License');
-});
-
-test('Reading after modified should fail', async t => {
-	const blob = blobFrom('./LICENSE');
-	await new Promise(resolve => {
-		setTimeout(resolve, 100);
-	});
-	const now = new Date();
-	// Change modified time
-	fs.utimesSync('./LICENSE', now, now);
-	const error = await blob.text().catch(error => error);
-	t.is(error.name, 'NotReadableError');
-});
-
-test('Reading from the stream created by blobFrom', async t => {
-	const blob = blobFrom('./LICENSE');
-	const expected = await fs.promises.readFile('./LICENSE', 'utf-8');
-
-	const actual = await getStream(blob.stream());
-
-	t.is(actual, expected);
-});
-
-test('Reading empty blobs', async t => {
-	const blob = blobFrom('./LICENSE').slice(0, 0);
-	const actual = await blob.text();
-	t.is(actual, '');
-});
-
-test('Blob-ish class is an instance of Blob', t => {
-	class File {
-		stream() {}
-
-		get [Symbol.toStringTag]() {
-			return 'File';
-		}
-	}
-
-	t.true(new File() instanceof Blob);
-});
-
-test('Instanceof check returns false for nullish values', t => {
-	t.false(null instanceof Blob);
-});