Add trailing commas

Author: jdalton

biome.json

@@ -33,7 +33,7 @@
       "quoteProperties": "asNeeded",
       "quoteStyle": "single",
       "semicolons": "asNeeded",
-      "trailingCommas": "none"
+      "trailingCommas": "all"
     }
   },
   "json": {

eslint.config.js

@@ -4,7 +4,7 @@ const path = require('node:path')
 
 const {
   convertIgnorePatternToMinimatch,
-  includeIgnoreFile
+  includeIgnoreFile,
 } = require('@eslint/compat')
 const js = require('@eslint/js')
 const importXPlugin = require('eslint-plugin-import-x')
@@ -23,7 +23,7 @@ const gitignorePath = path.join(rootPath, GITIGNORE)
 
 const biomeConfig = require(biomeConfigPath)
 const nodeGlobalsConfig = Object.fromEntries(
-  Object.entries(globals.node).map(([k]) => [k, 'readonly'])
+  Object.entries(globals.node).map(([k]) => [k, 'readonly']),
 )
 
 module.exports = [
@@ -32,7 +32,7 @@ module.exports = [
     name: 'Imported biome.json ignore patterns',
     ignores: biomeConfig.files.includes
       .filter(p => p.startsWith('!'))
-      .map(p => convertIgnorePatternToMinimatch(p.slice(1)))
+      .map(p => convertIgnorePatternToMinimatch(p.slice(1))),
   },
   {
     ...js.configs.recommended,
@@ -48,22 +48,22 @@ module.exports = [
         ...importXPlugin.flatConfigs.recommended.languageOptions?.globals,
         ...nodePlugin.configs['flat/recommended-script'].languageOptions
           ?.globals,
-        ...nodeGlobalsConfig
+        ...nodeGlobalsConfig,
       },
-      sourceType: 'script'
+      sourceType: 'script',
     },
     linterOptions: {
       ...js.configs.recommended.linterOptions,
       ...importXPlugin.flatConfigs.recommended.linterOptions,
       ...nodePlugin.configs['flat/recommended-script'].linterOptions,
-      reportUnusedDisableDirectives: 'off'
+      reportUnusedDisableDirectives: 'off',
     },
     plugins: {
       ...js.configs.recommended.plugins,
       ...importXPlugin.flatConfigs.recommended.plugins,
       ...nodePlugin.configs['flat/recommended-script'].plugins,
       'sort-destructure-keys': sortDestructureKeysPlugin,
-      unicorn: unicornPlugin
+      unicorn: unicornPlugin,
     },
     rules: {
       ...js.configs.recommended.rules,
@@ -76,8 +76,8 @@ module.exports = [
           cjs: 'ignorePackages',
           js: 'ignorePackages',
           json: 'always',
-          mjs: 'ignorePackages'
-        }
+          mjs: 'ignorePackages',
+        },
       ],
       'import-x/no-named-as-default-member': 'off',
       'import-x/no-unresolved': ['error', { commonjs: true }],
@@ -89,20 +89,20 @@ module.exports = [
             'external',
             'internal',
             ['parent', 'sibling', 'index'],
-            'type'
+            'type',
           ],
           pathGroups: [
             {
               pattern: '@socket{registry,security}/**',
-              group: 'internal'
-            }
+              group: 'internal',
+            },
           ],
           pathGroupsExcludedImportTypes: ['type'],
           'newlines-between': 'always',
           alphabetize: {
-            order: 'asc'
-          }
-        }
+            order: 'asc',
+          },
+        },
       ],
       'n/exports-style': ['error', 'module.exports'],
       // The n/no-unpublished-bin rule does does not support non-trivial glob
@@ -116,8 +116,8 @@ module.exports = [
         {
           ignores: ['test', 'test.describe'],
           // Lazily access constants.maintainedNodeVersions.
-          version: constants.maintainedNodeVersions.current
-        }
+          version: constants.maintainedNodeVersions.current,
+        },
       ],
       'n/prefer-node-protocol': 'error',
       'unicorn/consistent-function-scoping': 'error',
@@ -133,20 +133,20 @@ module.exports = [
         {
           argsIgnorePattern: '^_|^this$',
           ignoreRestSiblings: true,
-          varsIgnorePattern: '^_'
-        }
+          varsIgnorePattern: '^_',
+        },
       ],
       'no-var': 'error',
       'no-warning-comments': ['warn', { terms: ['fixme'] }],
       'prefer-const': 'error',
       'sort-destructure-keys/sort-destructure-keys': 'error',
-      'sort-imports': ['error', { ignoreDeclarationSort: true }]
-    }
+      'sort-imports': ['error', { ignoreDeclarationSort: true }],
+    },
   },
   {
     files: ['**/*.mjs'],
     languageOptions: {
-      sourceType: 'module'
-    }
-  }
+      sourceType: 'module',
+    },
+  },
 ]

index.js

@@ -25,12 +25,12 @@ const {
   PackageURL,
   PurlComponent,
   PurlQualifierNames,
-  PurlType
+  PurlType,
 } = require('./src/package-url')
 
 module.exports = {
   PackageURL,
   PurlComponent,
   PurlQualifierNames,
-  PurlType
+  PurlType,
 }

scripts/get-coverage-percentage.js

@@ -24,7 +24,7 @@ async function logCoveragePercentage(argv) {
   const coverageJsonPath = path.join(
     process.cwd(),
     'coverage',
-    'coverage-final.json'
+    'coverage-final.json',
   )
 
   if (!existsSync(coverageJsonPath)) {
@@ -34,7 +34,7 @@ async function logCoveragePercentage(argv) {
     try {
       result = await spawn('pnpm', ['run', 'test:unit:coverage'], {
         stdio: 'ignore',
-        shell: constants.WIN32
+        shell: constants.WIN32,
       })
     } catch (error) {
       spinner.stop()
@@ -146,44 +146,44 @@ async function logCoveragePercentage(argv) {
           statements: {
             percent: stmtPercent,
             covered: coveredStatements,
-            total: totalStatements
+            total: totalStatements,
           },
           branches: {
             percent: branchPercent,
             covered: coveredBranches,
-            total: totalBranches
+            total: totalBranches,
           },
           functions: {
             percent: funcPercent,
             covered: coveredFunctions,
-            total: totalFunctions
+            total: totalFunctions,
           },
           lines: {
             percent: linePercent,
             covered: coveredLines,
-            total: totalLines
+            total: totalLines,
           },
-          overall: overall
+          overall: overall,
         },
         null,
-        2
-      )
+        2,
+      ),
     )
   } else if (argv.simple) {
     logger.log(stmtPercent)
   } else {
     logger.info(`Coverage Summary:`)
     logger.info(
-      `${indent}Statements: ${stmtPercent}% (${coveredStatements}/${totalStatements})`
+      `${indent}Statements: ${stmtPercent}% (${coveredStatements}/${totalStatements})`,
     )
     logger.info(
-      `${indent}Branches:   ${branchPercent}% (${coveredBranches}/${totalBranches})`
+      `${indent}Branches:   ${branchPercent}% (${coveredBranches}/${totalBranches})`,
     )
     logger.info(
-      `${indent}Functions:  ${funcPercent}% (${coveredFunctions}/${totalFunctions})`
+      `${indent}Functions:  ${funcPercent}% (${coveredFunctions}/${totalFunctions})`,
     )
     logger.info(
-      `${indent}Lines:      ${linePercent}% (${coveredLines}/${totalLines})`
+      `${indent}Lines:      ${linePercent}% (${coveredLines}/${totalLines})`,
     )
     logger.info('')
     logger.info(colors.bold(`Current coverage: ${overall}% overall!${emoji}`))
@@ -195,8 +195,8 @@ void (async () => {
     boolean: ['json', 'simple'],
     alias: {
       j: 'json',
-      s: 'simple'
-    }
+      s: 'simple',
+    },
   })
   await logCoveragePercentage(argv)
 })()

scripts/update-data-npm.js

@@ -35,7 +35,7 @@ void (async () => {
   if (
     await confirm({
       message: `Update builtin package names?${isGteNext ? '' : ` (Requires Node >=${next})`}`,
-      default: true
+      default: true,
     })
   ) {
     const nodeVersion = process.version.slice(1)
@@ -54,7 +54,7 @@ void (async () => {
   if (
     !(await confirm({
       message: 'Update npm package names data?',
-      default: false
+      default: false,
     }))
   ) {
     spinner.stop()
@@ -69,8 +69,8 @@ void (async () => {
       // Load the 24.7MB names.json from '[email protected]',
       // the last v1 release, because it has different names resolved by
       // npm's replicate.npmjs.com service.
-      ...require('all-the-package-names-v1.3905.0/names.json')
-    ])
+      ...require('all-the-package-names-v1.3905.0/names.json'),
+    ]),
   ]
   const rawLegacyNames = allThePackageNames
     // Don't simply check validateNpmPackageName(n).validForOldPackages.
@@ -98,7 +98,7 @@ void (async () => {
         }
         return false
       },
-      { concurrency: 3, retries: 4, signal: abortSignal }
+      { concurrency: 3, retries: 4, signal: abortSignal },
     )
   await writeJson(npmLegacyNamesJsonPath, legacyNames, { spaces: 2 })
   spinner.stop()

src/constants.js

@@ -12,5 +12,5 @@ module.exports = {
   LOOP_SENTINEL,
   REUSED_SEARCH_PARAMS,
   REUSED_SEARCH_PARAMS_KEY,
-  REUSED_SEARCH_PARAMS_OFFSET
+  REUSED_SEARCH_PARAMS_OFFSET,
 }

src/decode.js

@@ -14,5 +14,5 @@ function decodePurlComponent(comp, encodedComponent) {
 }
 
 module.exports = {
-  decodePurlComponent
+  decodePurlComponent,
 }

src/encode.js

@@ -3,7 +3,7 @@
 const {
   REUSED_SEARCH_PARAMS,
   REUSED_SEARCH_PARAMS_KEY,
-  REUSED_SEARCH_PARAMS_OFFSET
+  REUSED_SEARCH_PARAMS_OFFSET,
 } = require('./constants')
 const { isObject } = require('./objects')
 const { isNonEmptyString } = require('./strings')
@@ -81,5 +81,5 @@ module.exports = {
   encodeVersion,
   encodeQualifiers,
   encodeQualifierParam,
-  encodeSubpath
+  encodeSubpath,
 }

src/error.js

@@ -30,5 +30,5 @@ class PurlError extends Error {
 
 module.exports = {
   formatPurlErrorMessage,
-  PurlError
+  PurlError,
 }

src/helpers.js

@@ -4,7 +4,7 @@ function createHelpersNamespaceObject(helpers, options_ = {}) {
   const { comparator, ...defaults } = { __proto__: null, ...options_ }
   const helperNames = Object.keys(helpers).sort()
   const propNames = [
-    ...new Set(Object.values(helpers).flatMap(Object.keys))
+    ...new Set(Object.values(helpers).flatMap(Object.keys)),
   ].sort(comparator)
   const nsObject = Object.create(null)
   for (let i = 0, { length } = propNames; i < length; i += 1) {
@@ -23,5 +23,5 @@ function createHelpersNamespaceObject(helpers, options_ = {}) {
 }
 
 module.exports = {
-  createHelpersNamespaceObject
+  createHelpersNamespaceObject,
 }