Remove the path arguments and drop BOM stripping

Since the name is always `package.json`, we can simplify the API by only accepting a `cwd` option instead.

Author: sindresorhus

index.js

@@ -1,47 +1,42 @@
 'use strict';
+const fs = require('fs');
 const path = require('path');
-const loadJsonFile = require('load-json-file');
-const pathType = require('path-type');
+const parseJson = require('parse-json');
+const promisify = require('util.promisify');
 
-module.exports = (fp, opts) => {
-	if (typeof fp !== 'string') {
-		opts = fp;
-		fp = '.';
-	}
+const readFileAsync = promisify(fs.readFile);
 
-	opts = opts || {};
+module.exports = options => {
+	options = Object.assign({
+		cwd: process.cwd(),
+		normalize: true
+	}, options);
 
-	return pathType.dir(fp)
-		.then(isDir => {
-			if (isDir) {
-				fp = path.join(fp, 'package.json');
-			}
+	const filePath = path.resolve(options.cwd, 'package.json');
 
-			return loadJsonFile(fp);
-		})
-		.then(x => {
-			if (opts.normalize !== false) {
-				require('normalize-package-data')(x);
-			}
+	return readFileAsync(filePath, 'utf8').then(file => {
+		const json = parseJson(file);
 
-			return x;
-		});
-};
+		if (options.normalize) {
+			require('normalize-package-data')(json);
+		}
 
-module.exports.sync = (fp, opts) => {
-	if (typeof fp !== 'string') {
-		opts = fp;
-		fp = '.';
-	}
+		return json;
+	});
+};
 
-	opts = opts || {};
-	fp = pathType.dirSync(fp) ? path.join(fp, 'package.json') : fp;
+module.exports.sync = options => {
+	options = Object.assign({
+		cwd: process.cwd(),
+		normalize: true
+	}, options);
 
-	const x = loadJsonFile.sync(fp);
+	const filePath = path.resolve(options.cwd, 'package.json');
+	const json = parseJson(fs.readFileSync(filePath, 'utf8'));
 
-	if (opts.normalize !== false) {
-		require('normalize-package-data')(x);
+	if (options.normalize) {
+		require('normalize-package-data')(json);
 	}
 
-	return x;
+	return json;
 };

package.json

@@ -31,9 +31,9 @@
 		"normalize"
 	],
 	"dependencies": {
-		"load-json-file": "^5.0.0",
 		"normalize-package-data": "^2.3.2",
-		"path-type": "^3.0.0"
+		"parse-json": "^4.0.0",
+		"util.promisify": "^1.0.0"
 	},
 	"devDependencies": {
 		"ava": "*",

readme.md

@@ -6,7 +6,6 @@
 ## Why
 
 - [Gracefully handles filesystem issues](https://github.com/isaacs/node-graceful-fs)
-- [Strips UTF-8 BOM](https://github.com/sindresorhus/strip-bom)
 - [Throws more helpful JSON errors](https://github.com/sindresorhus/parse-json)
 - [Normalizes the data](https://github.com/npm/normalize-package-data#what-normalization-currently-entails)
 
@@ -24,30 +23,35 @@ $ npm install read-pkg
 const readPkg = require('read-pkg');
 
 (async () => {
-	console.log( await readPkg());
+	console.log(await readPkg());
 	//=> {name: 'read-pkg', ...}
+
+	console.log(await readPkg({cwd: 'some-other-directory'});
+	//=> {name: 'unicorn', ...}
 })();
 ```
 
 
 ## API
 
-### readPkg([path], [options])
+### readPkg([options])
 
 Returns a `Promise` for the parsed JSON.
 
-### readPkg.sync([path], [options])
+### readPkg.sync([options])
 
 Returns the parsed JSON.
 
-#### path
+#### options
+
+Type: `Object`
+
+##### cwd
 
 Type: `string`<br>
 Default: `process.cwd()`
 
-Path to a `package.json` file or its directory.
-
-#### options
+Current working directory.
 
 ##### normalize
 

test/pkg.json renamed to test/package.json

@@ -3,39 +3,28 @@ import path from 'path';
 import test from 'ava';
 import m from '..';
 
-const pkg = path.join(__dirname, '..');
-const otherName = path.join(__dirname, 'pkg.json');
+process.chdir(__dirname);
+
+const rootCwd = path.join(__dirname, '..');
 
 test('async', async t => {
-	const x = await m(otherName);
+	const x = await m();
 	t.is(x.name, 'unicorn');
 	t.truthy(x._id);
 });
 
-test('async - directory', async t => {
-	const x = await m(pkg);
-	t.is(x.name, 'read-pkg');
-	t.truthy(x._id);
-});
-
-test.serial('async - default filepath', async t => {
-	const x = await m();
+test('async - cwd option', async t => {
+	const x = await m({cwd: rootCwd});
 	t.is(x.name, 'read-pkg');
 });
 
 test('sync', t => {
-	const x = m.sync(otherName);
+	const x = m.sync();
 	t.is(x.name, 'unicorn');
 	t.truthy(x._id);
 });
 
-test('sync - directory', t => {
-	const x = m.sync(pkg);
-	t.is(x.name, 'read-pkg');
-	t.truthy(x._id);
-});
-
-test.serial('sync - default filepath', t => {
-	const x = m.sync();
+test('sync - cwd option', t => {
+	const x = m.sync({cwd: rootCwd});
 	t.is(x.name, 'read-pkg');
 });