chore(deps): bump yargs-parser and ava (#451)

* chore(deps): bump yargs-parser and ava

Bumps [yargs-parser](https://github.com/yargs/yargs-parser) to 21.1.1 and updates ancestor dependency [ava](https://github.com/avajs/ava). These dependencies need to be updated together.


Updates `yargs-parser` from 10.1.0 to 21.1.1
- [Release notes](https://github.com/yargs/yargs-parser/releases)
- [Changelog](https://github.com/yargs/yargs-parser/blob/main/CHANGELOG.md)
- [Commits](yargs/yargs-parser@v10.1.0...yargs-parser-v21.1.1)

Updates `ava` from 2.4.0 to 5.2.0
- [Release notes](https://github.com/avajs/ava/releases)
- [Commits](avajs/ava@v2.4.0...v5.2.0)

---
updated-dependencies:
- dependency-name: yargs-parser
  dependency-type: indirect
- dependency-name: ava
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>

* chore: improve eslint setup

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Daniel Tschinder <[email protected]>

Author: dependabot[bot]

.eslintrc.json

@@ -1,5 +1,4 @@
 {
-  "plugins": ["node", "prettier"],
   "parserOptions": {
     "sourceType": "module"
   },
@@ -8,11 +7,11 @@
     "es6": true
   },
   "extends": [
+    "eslint:recommended",
     "plugin:import/errors",
     "plugin:import/warnings",
-    "eslint:recommended",
-    "plugin:node/recommended",
-    "prettier"
+    "plugin:n/recommended",
+    "plugin:prettier/recommended"
   ],
   "rules": {
     "import/order": [2, { "newlines-between": "always" }]

.prettierrc.json

@@ -1,5 +1,4 @@
 {
   "semi": false,
-  "singleQuote": true,
-  "printWidth": 120
+  "singleQuote": true
 }

package-lock.json

@@ -6,9 +6,7 @@
   "ava": {
     "files": [
       "src/**/*.test.js"
-    ],
-    "compileEnhancements": false,
-    "babel": false
+    ]
   },
   "bugs": {
     "url": "https://github.com/netlify/open-api/issues"
@@ -17,18 +15,18 @@
   "devDependencies": {
     "analytics": "^0.2.0",
     "analytics-plugin-ga": "^0.1.5",
-    "ava": "^2.4.0",
+    "ava": "^5.2.0",
     "cp-file": "^7.0.0",
-    "eslint": "^6.3.0",
-    "eslint-config-prettier": "^6.2.0",
-    "eslint-plugin-import": "^2.18.2",
-    "eslint-plugin-node": "^10.0.0",
-    "eslint-plugin-prettier": "^3.1.0",
+    "eslint": "^8.38.0",
+    "eslint-config-prettier": "^8.8.0",
+    "eslint-plugin-import": "^2.27.5",
+    "eslint-plugin-n": "^15.7.0",
+    "eslint-plugin-prettier": "^4.2.1",
     "execa": "^2.0.4",
     "is-plain-obj": "^2.1.0",
     "make-dir": "^3.0.0",
     "npm-run-all": "^4.1.5",
-    "prettier": "^1.18.2",
+    "prettier": "^2.8.7",
     "react": "^16.9.0",
     "react-dom": "^16.9.0",
     "redoc-cli": "^0.13.7",

package.json

@@ -10,7 +10,7 @@ const SWAGGER_PATH = `${__dirname}/../swagger.yml`
 
 // Modify the `info.version` field inside `swagger.yml` so it matches the
 // `package.json` `version`
-const bumpSwagger = async function() {
+const bumpSwagger = async function () {
   const swaggerFile = await pReadFile(SWAGGER_PATH, 'utf8')
   const newSwaggerFile = swaggerFile.replace(VERSION_REGEXP, `$1${version}`)
 

src/bump-swagger.js

@@ -12,10 +12,10 @@ const JSON_OUTPUT = `${OUTPUT_DIR}/swagger.json`
 
 // Validate `swagger.yml`, dereference the JSON references then serialize to
 // `swagger.json`
-const convertOpenApi = async function() {
+const convertOpenApi = async function () {
   const [openapiDef] = await Promise.all([
     SwaggerParser.validate(YAML_INPUT, { dereference: { circular: false } }),
-    makeDir(OUTPUT_DIR)
+    makeDir(OUTPUT_DIR),
   ])
   const openapiJson = JSON.stringify(openapiDef, null, 2)
   await pWriteFile(JSON_OUTPUT, openapiJson)

src/convert.js

@@ -2,11 +2,10 @@ const test = require('ava')
 const isPlainObj = require('is-plain-obj')
 
 const { version } = require('../package.json')
-
-// eslint-disable-next-line node/no-missing-require
+// eslint-disable-next-line n/no-missing-require
 const openApiDef = require('..')
 
-test('OpenAPI definition normalization', async t => {
+test('OpenAPI definition normalization', async (t) => {
   // Ensure the OpenAPI definition general shape looks normal
   t.true(isPlainObj(openApiDef))
   t.true(isPlainObj(openApiDef.definitions))

src/convert.test.js

@@ -7,15 +7,15 @@ const ASSETS = [
   `${__dirname}/script.js`,
   `${__dirname}/styles.css`,
   `${__dirname}/../../node_modules/analytics/dist/analytics.min.js`,
-  `${__dirname}/../../node_modules/analytics-plugin-ga/dist/analytics-plugin-ga.min.js`
+  `${__dirname}/../../node_modules/analytics-plugin-ga/dist/analytics-plugin-ga.min.js`,
 ]
 
 // Copy files for static site to use (logo, JavaScript libraries)
-const copyAssets = async function(outputDir) {
-  await Promise.all(ASSETS.map(path => copyAsset(path, outputDir)))
+const copyAssets = async function (outputDir) {
+  await Promise.all(ASSETS.map((path) => copyAsset(path, outputDir)))
 }
 
-const copyAsset = async function(path, outputDir) {
+const copyAsset = async function (path, outputDir) {
   await cpFile(path, `${outputDir}/${basename(path)}`)
 }
 

src/docs/assets.js

@@ -10,11 +10,11 @@ const OUTPUT_DIR = `${__dirname}/../../dist`
 const OUTPUT_PATH = `${OUTPUT_DIR}/index.html`
 
 // Build API documentation single self-contained HTML file using `redoc-cli`
-const buildDocs = async function() {
+const buildDocs = async function () {
   await Promise.all([redocCli(), copyAssets(OUTPUT_DIR)])
 }
 
-const redocCli = async function() {
+const redocCli = async function () {
   await execa(
     'redoc-cli',
     [
@@ -27,7 +27,7 @@ const redocCli = async function() {
       `--options.theme.logo.gutter=${LOGO_PADDING}`,
       `--output=${normalize(OUTPUT_PATH)}`,
       'build',
-      SWAGGER_PATH
+      SWAGGER_PATH,
     ],
     { stdio: 'inherit' }
   )

src/docs/build.js

@@ -8,11 +8,11 @@ const pReadFile = promisify(readFile)
 const pWriteFile = promisify(writeFile)
 
 // Inject HTML content after Redoc has built the documentation
-const injectContent = async function(outputPath) {
+const injectContent = async function (outputPath) {
   const [siteContent, head, footer] = await Promise.all([
     pReadFile(outputPath, 'utf8'),
     pReadFile(HEAD_PATH, 'utf8'),
-    pReadFile(FOOTER_PATH, 'utf8')
+    pReadFile(FOOTER_PATH, 'utf8'),
   ])
 
   const updatedContent = siteContent