.all-contributorsrc

@@ -635,6 +635,17 @@
       "contributions": [
         "code"
       ]
+    },
+    {
+      "login": "dwmkerr",
+      "name": "Dave Kerr",
+      "avatar_url": "https://avatars.githubusercontent.com/u/1926984?v=4",
+      "profile": "http://www.dwmkerr.com",
+      "contributions": [
+        "bug",
+        "test",
+        "code"
+      ]
     }
   ],
   "skipCi": true

README.md

@@ -169,6 +169,7 @@ Thanks goes to these wonderful people
       <td align="center" valign="top" width="16.66%"><a href="https://www.lieret.net"><img src="https://avatars.githubusercontent.com/u/13602468?v=4?s=100" width="100px;" alt="Kilian Lieret"/><br /><sub><b>Kilian Lieret</b></sub></a><br /><a href="https://github.com/all-contributors/cli/issues?q=author%3Aklieret" title="Bug reports">🐛</a></td>
       <td align="center" valign="top" width="16.66%"><a href="https://github.com/KnorpelSenf"><img src="https://avatars.githubusercontent.com/u/12952387?v=4?s=100" width="100px;" alt="KnorpelSenf"/><br /><sub><b>KnorpelSenf</b></sub></a><br /><a href="https://github.com/all-contributors/cli/issues?q=author%3AKnorpelSenf" title="Bug reports">🐛</a></td>
       <td align="center" valign="top" width="16.66%"><a href="https://rea9lizer.carrd.co"><img src="https://avatars.githubusercontent.com/u/68494132?v=4?s=100" width="100px;" alt="QriLa &#124; Hyeon Gu"/><br /><sub><b>QriLa &#124; Hyeon Gu</b></sub></a><br /><a href="https://github.com/all-contributors/cli/commits?author=qyurila" title="Code">💻</a></td>
+      <td align="center" valign="top" width="16.66%"><a href="http://www.dwmkerr.com"><img src="https://avatars.githubusercontent.com/u/1926984?v=4?s=100" width="100px;" alt="Dave Kerr"/><br /><sub><b>Dave Kerr</b></sub></a><br /><a href="https://github.com/all-contributors/cli/issues?q=author%3Adwmkerr" title="Bug reports">🐛</a> <a href="https://github.com/all-contributors/cli/commits?author=dwmkerr" title="Tests">⚠️</a> <a href="https://github.com/all-contributors/cli/commits?author=dwmkerr" title="Code">💻</a></td>
     </tr>
   </tbody>
   <tfoot>

package.json

@@ -61,6 +61,9 @@
     "nock": "^12.0.0",
     "semantic-release": "^17.0.8"
   },
+  "optionalDependencies": {
+    "prettier": "^2"
+  },
   "eslintIgnore": [
     "node_modules",
     "coverage",

src/util/__tests__/__stubs__/.prettierrc.json

@@ -0,0 +1,3 @@
+{
+	"useTabs": true
+}

src/util/__tests__/formatting.js

@@ -0,0 +1,35 @@
+import path from 'path'
+import formatting from '../formatting'
+
+const content = {contributors: [{id: 'abc123'}]}
+
+const absentFile = '/!@#*&^DefinitelyDoesNotExistAllContributorsCLI$!@#'
+const absentConfigFileExpected = `{
+  "contributors": [
+    {
+      "id": "abc123"
+    }
+  ]
+}`
+
+const presentFile = path.join(__dirname, '__stubs__', 'file.json')
+const presentConfigFileExpected = `{
+	"contributors": [
+		{
+			"id": "abc123"
+		}
+	]
+}
+`
+
+test('falls back to JSON.stringify when the configPath cannot resolve to a config', () => {
+  expect(formatting.formatConfig(absentFile, content)).toBe(
+    absentConfigFileExpected,
+  )
+})
+
+test('uses Prettier when the configPath can resolve to a config', () => {
+  expect(formatting.formatConfig(presentFile, content)).toBe(
+    presentConfigFileExpected,
+  )
+})

src/util/config-file.js

@@ -2,6 +2,7 @@ const fs = require('fs')
 const pify = require('pify')
 const _ = require('lodash/fp')
 const jf = require('json-fixer')
+const {formatConfig} = require('./formatting')
 
 function readConfig(configPath) {
   try {
@@ -14,7 +15,7 @@ function readConfig(configPath) {
     }
     if (changed) {
       //Updates the file with fixes
-      fs.writeFileSync(configPath, JSON.stringify(config, null, 2))
+      fs.writeFileSync(configPath, formatConfig(config))
     }
     return config
   } catch (error) {
@@ -43,7 +44,7 @@ function writeConfig(configPath, content) {
       `Error! Project files was overridden and is empty in ${configPath}`,
     )
   }
-  return pify(fs.writeFile)(configPath, `${JSON.stringify(content, null, 2)}\n`)
+  return pify(fs.writeFile)(configPath, `${formatConfig(content)}\n`)
 }
 
 function writeContributors(configPath, contributors) {

src/util/formatting.js

@@ -0,0 +1,21 @@
+function formatConfig(configPath, content) {
+  const stringified = JSON.stringify(content, null, 2)
+  try {
+    const prettier = require('prettier')
+    const prettierConfig = prettier.resolveConfig.sync(configPath, {
+      useCache: false,
+    })
+
+    return prettierConfig
+      ? prettier.format(stringified, {...prettierConfig, parser: 'json'})
+      : stringified
+  } catch (error) {
+    // If Prettier can't be required or throws in general,
+    // assume it's not usable and we should fall back to JSON.stringify
+    return stringified
+  }
+}
+
+module.exports = {
+  formatConfig,
+}