index.js

@@ -1,17 +1,16 @@
 #!/usr/bin/env node
-const _sortObjectKeys = require('sort-object-keys')
+const sortObjectKeys = require('sort-object-keys')
 const detectIndent = require('detect-indent')
 const detectNewline = require('detect-newline').graceful
 const globby = require('globby')
-const sortObjectKeys = comp => x => _sortObjectKeys(x, comp)
 
 const onArray = fn => x => (Array.isArray(x) ? fn(x) : x)
 const uniq = onArray(xs => xs.filter((x, i) => i === xs.indexOf(x)))
 const isPlainObject = x =>
   x && Object.prototype.toString.call(x) === '[object Object]'
 const onObject = fn => x => (isPlainObject(x) ? fn(x) : x)
-const sortObjectBy = comparator => onObject(sortObjectKeys(comparator))
-const sortObject = onObject(sortObjectKeys())
+const sortObjectBy = comparator => onObject(x => sortObjectKeys(x, comparator))
+const sortObject = sortObjectBy()
 const sortURLObject = sortObjectBy(['type', 'url'])
 const sortAuthorObject = sortObjectBy(['name', 'email', 'url'])
 const sortDirectories = sortObjectBy(['lib', 'bin', 'man', 'doc', 'example'])
@@ -160,15 +159,18 @@ function editStringJSON(json, over) {
 }
 
 function sortPackageJson(jsonIsh, options = {}) {
-  return editStringJSON(jsonIsh, json => {
-    const newJson = sortObjectKeys(options.sortOrder || sortOrder)(json)
+  return editStringJSON(
+    jsonIsh,
+    onObject(json => {
+      const newJson = sortObjectKeys(json, options.sortOrder || sortOrder)
 
-    for (const { key, over } of fields) {
-      if (over && newJson[key]) newJson[key] = over(newJson[key])
-    }
+      for (const { key, over } of fields) {
+        if (over && newJson[key]) newJson[key] = over(newJson[key])
+      }
 
-    return newJson
-  })
+      return newJson
+    }),
+  )
 }
 
 module.exports = sortPackageJson

test.js

@@ -134,6 +134,19 @@ fs.readFile('./package.json', 'utf8', (error, contents) => {
     sortPackageJson('{\r\n  "foo": "bar"\n}\n'),
     '{\n  "foo": "bar"\n}\n',
   )
+
+  const array = ['foo', 'bar']
+  const string = JSON.stringify(array)
+  assert.strictEqual(
+    sortPackageJson(array),
+    array,
+    'should not sort object that is not plain object',
+  )
+  assert.strictEqual(
+    sortPackageJson(string),
+    string,
+    'should not sort object that is not plain object',
+  )
 })
 
 // fields tests