feat: update prettier rules/config

Author: jdalton

.editorconfig

@@ -1,6 +1,10 @@
 root = true
 
-[*.js]
-indent_size = 2
-indent_style = space
+[*]
+charset = utf-8
 end_of_line = lf
+indent_size = 4
+indent_style = space
+insert_final_newline = true
+max_line_length = 80
+trim_trailing_whitespace = true

.gitattributes

@@ -0,0 +1 @@
+* text=auto eol=lf

.gitignore

@@ -1,2 +1,9 @@
 .DS_Store
-node_modules/
+._.DS_Store
+Thumbs.db
+/.env
+/.nvm
+/.vscode
+/npm-debug.log
+/yarn.lock
+**/node_modules

.prettierignore

@@ -1,5 +1,3 @@
-# Ignore artifacts:
+.github/
 *.json
-*.xml
-*.yml
 *.md

.prettierrc

@@ -0,0 +1,7 @@
+{
+  "arrowParens": "avoid",
+  "semi": false,
+  "singleQuote": true,
+  "trailingComma": "none",
+  "proseWrap": "always"
+}

.prettierrc.json

@@ -56,7 +56,7 @@ async function pFilterChunk(chunks, callbackFn, options) {
         } else {
             const chunk = chunks[i]
             const predicateResults = await Promise.all(
-                chunk.map((value) => {
+                chunk.map(value => {
                     if (signal?.aborted) {
                         return Promise.resolve()
                     }
@@ -96,7 +96,7 @@ void (async () => {
         ])
     ]
     const rawLegacyNames = allThePackageNames
-        .filter((n) => !validateNpmPackageName(n).validForNewPackages)
+        .filter(n => !validateNpmPackageName(n).validForNewPackages)
         .sort(alphanumericComparator)
     const seenNames = new Set()
     const invalidNames = new Set()
@@ -105,7 +105,7 @@ void (async () => {
         await pFilter(
             rawLegacyNames,
             3,
-            async (n) => {
+            async n => {
                 if (!seenNames.has(n)) {
                     seenNames.add(n)
                     spinner.text = `Checking package ${n}...`
@@ -126,7 +126,7 @@ void (async () => {
         [
             { json: builtinNames, path: npmBuiltinNamesJsonPath },
             { json: legacyNames, path: npmLegacyNamesJsonPath }
-        ].map((d) =>
+        ].map(d =>
             fs.writeFile(d.path, `${JSON.stringify(d.json, null, 2)}\n`, 'utf8')
         )
     )

scripts/update-data-npm.js

@@ -232,7 +232,7 @@ class PackageURL {
         let rawQualifiers
         const { searchParams } = url
         if (searchParams.size !== 0) {
-            searchParams.forEach((value) =>
+            searchParams.forEach(value =>
                 decodePurlComponent('qualifiers', value)
             )
             // Split the remainder once from right on '?'.

src/package-url.js

@@ -32,10 +32,10 @@ const {
     validateSubpath
 } = require('./validate')
 
-const PurlComponentEncoder = (comp) =>
+const PurlComponentEncoder = comp =>
     isNonEmptyString(comp) ? encodeURIComponent(comp) : ''
 
-const PurlComponentStringNormalizer = (comp) =>
+const PurlComponentStringNormalizer = comp =>
     typeof comp === 'string' ? comp : undefined
 
 const PurlComponentValidator = (_comp, _throws) => true

src/purl-component.js

@@ -14,7 +14,7 @@ const {
 const { validateEmptyByType, validateRequiredByType } = require('./validate')
 const { PurlError } = require('./error')
 
-const PurlTypNormalizer = (purl) => purl
+const PurlTypNormalizer = purl => purl
 const PurlTypeValidator = (_purl, _throws) => true
 
 const getNpmBuiltinNames = (() => {
@@ -48,9 +48,9 @@ function getNpmId(purl) {
     return `${namespace?.length > 0 ? `${namespace}/` : ''}${name}`
 }
 
-const isNpmBuiltinName = (id) => getNpmBuiltinNames().includes(id.toLowerCase())
+const isNpmBuiltinName = id => getNpmBuiltinNames().includes(id.toLowerCase())
 
-const isNpmLegacyName = (id) => getNpmLegacyNames().includes(id)
+const isNpmLegacyName = id => getNpmLegacyNames().includes(id)
 
 module.exports = {
     // PURL types:

src/purl-type.js

@@ -7,7 +7,7 @@ const { PackageURL } = require('../src/package-url')
 describe('PackageURL', () => {
     it('Benchmarking the library', () => {
         const iterations = 10000
-        const data = TEST_FILE.filter((obj) => !obj.is_invalid)
+        const data = TEST_FILE.filter(obj => !obj.is_invalid)
         const { length: dataLength } = data
         const objects = []
         for (let i = 0; i < iterations; i += dataLength) {

test/benchmark.spec.js

@@ -94,7 +94,7 @@ describe('PackageURL', function () {
         }
 
         it('should validate required params', function () {
-            const testValid = (paramName) => {
+            const testValid = paramName => {
                 const paramIndex = paramMap[paramName]
                 const args = createArgs(paramName, paramName)
                 const message = JSON.stringify(args[paramIndex])
@@ -106,7 +106,7 @@ describe('PackageURL', function () {
                 }
             }
 
-            const testInvalid = (paramName) => {
+            const testInvalid = paramName => {
                 const paramIndex = paramMap[paramName]
                 ;[
                     createArgs(paramName, 0),
@@ -117,7 +117,7 @@ describe('PackageURL', function () {
                     createArgs(paramName, null),
                     createArgs(paramName, undefined),
                     createArgs(paramName, '')
-                ].forEach((args) => {
+                ].forEach(args => {
                     const message = JSON.stringify(args[paramIndex])
                     try {
                         new PackageURL(...args)
@@ -128,21 +128,21 @@ describe('PackageURL', function () {
                 })
             }
 
-            ;['type', 'name'].forEach((paramName) => {
+            ;['type', 'name'].forEach(paramName => {
                 testValid(paramName)
                 testInvalid(paramName)
             })
         })
 
         it('should validate string params', function () {
-            const testValid = (paramName) => {
+            const testValid = paramName => {
                 const paramIndex = paramMap[paramName]
                 ;[
                     createArgs(paramName, paramName),
                     createArgs(paramName, null),
                     createArgs(paramName, undefined),
                     createArgs(paramName, '')
-                ].forEach((args) => {
+                ].forEach(args => {
                     const message = JSON.stringify(args[paramIndex])
                     try {
                         new PackageURL(...args)
@@ -153,15 +153,15 @@ describe('PackageURL', function () {
                 })
             }
 
-            const testInvalid = (paramName) => {
+            const testInvalid = paramName => {
                 const paramIndex = paramMap[paramName]
                 ;[
                     createArgs(paramName, 0),
                     createArgs(paramName, false),
                     createArgs(paramName, 1),
                     createArgs(paramName, true),
                     createArgs(paramName, {})
-                ].forEach((args) => {
+                ].forEach(args => {
                     const message = JSON.stringify(args[paramIndex])
                     try {
                         new PackageURL(...args)
@@ -172,7 +172,7 @@ describe('PackageURL', function () {
                 })
             }
 
-            ;['namespace', 'version', 'subpath'].forEach((paramName) => {
+            ;['namespace', 'version', 'subpath'].forEach(paramName => {
                 testValid(paramName)
                 testInvalid(paramName)
             })
@@ -223,7 +223,7 @@ describe('PackageURL', function () {
 
     describe('toString()', function () {
         it('type is validated', function () {
-            ;['ty#pe', 'ty@pe', 'ty/pe', '1type'].forEach((type) => {
+            ;['ty#pe', 'ty@pe', 'ty/pe', '1type'].forEach(type => {
                 assert.throws(
                     () => new PackageURL(type, undefined, 'name'),
                     /contains an illegal character|cannot start with a number/