build(deps-dev): replace standard with neostandard (#421)

Fdawgs · web-flow · commit 5587737f1fe5 · 2024-11-29T15:27:56.000Z
diff --git a/README.md b/README.md
@@ -2,7 +2,7 @@
 
 ![CI](https://github.com/fastify/fastify-autoload/workflows/CI/badge.svg)
 [![NPM version](https://img.shields.io/npm/v/@fastify/autoload.svg?style=flat)](https://www.npmjs.com/package/@fastify/autoload)
-[![js-standard-style](https://img.shields.io/badge/code%20style-standard-brightgreen.svg?style=flat)](https://standardjs.com/)
+[![neostandard javascript style](https://img.shields.io/badge/code_style-neostandard-brightgreen?style=flat)](https://github.com/neostandard/neostandard)
 
 Convenience plugin for Fastify that loads all plugins found in a directory and automatically configures routes matching the folder structure.
 
diff --git a/eslint.config.js b/eslint.config.js
@@ -0,0 +1,11 @@
+'use strict'
+
+module.exports = require('neostandard')({
+  ignores: [
+    ...require('neostandard').resolveIgnoresFromGitignore(),
+    'test/commonjs/syntax-error/lib/a.js',
+    'test/issues/369/invalid-autohooks',
+    'test/issues/369/non-SyntaxError'
+  ],
+  ts: true
+})
diff --git a/package.json b/package.json
@@ -6,8 +6,8 @@
   "type": "commonjs",
   "types": "types/index.d.ts",
   "scripts": {
-    "lint": "standard | snazzy",
-    "lint:fix": "standard --fix | snazzy",
+    "lint": "eslint",
+    "lint:fix": "eslint --fix",
     "test": "npm run typescript && npm run typescript:jest && npm run typescript:swc-node-register && npm run typescript:tsm && npm run typescript:tsx && npm run typescript:vitest && npm run typescript:esbuild && npm run unit",
     "typescript": "tsd",
     "typescript:jest": "jest",
@@ -56,8 +56,7 @@
     "fastify": "^5.0.0",
     "fastify-plugin": "^5.0.0",
     "jest": "^29.7.0",
-    "snazzy": "^9.0.0",
-    "standard": "^17.1.0",
+    "neostandard": "^0.11.9",
     "tap": "^19.0.2",
     "ts-jest": "^29.1.4",
     "ts-node": "^10.9.2",
@@ -70,11 +69,6 @@
     "vite": "^5.2.12",
     "vitest": "^2.0.3"
   },
-  "standard": {
-    "ignore": [
-      "test/*/*-error/lib/a.js"
-    ]
-  },
   "jest": {
     "preset": "ts-jest",
     "testEnvironment": "node",
diff --git a/test/commonjs/babel-node/routes/foo/index.ts b/test/commonjs/babel-node/routes/foo/index.ts
@@ -1,5 +1,5 @@
 module.exports = async (fastify) => {
-  fastify.get("/", function () {
-    return { foo: "bar" };
+  fastify.get('/', function () {
+    return { foo: 'bar' }
   })
-};
+}
diff --git a/test/commonjs/babel-node/routes/root.ts b/test/commonjs/babel-node/routes/root.ts
@@ -1,5 +1,5 @@
 module.exports = async (fastify) => {
-  fastify.get("/", function () {
-    return { hello: "world" };
+  fastify.get('/', function () {
+    return { hello: 'world' }
   })
-};
+}
diff --git a/test/commonjs/ts-node/routes/bar/index.ts b/test/commonjs/ts-node/routes/bar/index.ts
@@ -1,5 +1,5 @@
 module.exports.default = async (fastify: any) => {
-  fastify.get("/", function () {
-    return { bar: "bar" };
+  fastify.get('/', function () {
+    return { bar: 'bar' }
   })
-};
+}
diff --git a/test/commonjs/ts-node/routes/foo/baz/index.ts b/test/commonjs/ts-node/routes/foo/baz/index.ts
@@ -1,5 +1,5 @@
 module.exports.default = async (fastify: any) => {
-  fastify.get("/customPath", function () {
-    return { baz: "baz" };
+  fastify.get('/customPath', function () {
+    return { baz: 'baz' }
   })
-};
+}
diff --git a/test/commonjs/ts-node/routes/foo/index.ts b/test/commonjs/ts-node/routes/foo/index.ts
@@ -1,5 +1,5 @@
 module.exports.default = async (fastify: any) => {
-  fastify.get("/", function () {
-    return { foo: "foo" };
+  fastify.get('/', function () {
+    return { foo: 'foo' }
   })
-};
+}
diff --git a/test/commonjs/ts-node/routes/root.ts b/test/commonjs/ts-node/routes/root.ts
@@ -1,5 +1,5 @@
 module.exports.default = async (fastify: any) => {
-  fastify.get("/", function () {
-    return { hello: "world" };
+  fastify.get('/', function () {
+    return { hello: 'world' }
   })
-};
+}
diff --git a/test/issues/369/routes/.autohooks.js b/test/issues/369/routes/.autohooks.js
@@ -2,7 +2,7 @@
 
 module.exports = async function (app, opts) {
   app.addHook('onRequest', async (req, reply) => {
-      req.hooked = req.hooked || []
-      req.hooked.push('root')
-    })
+    req.hooked = req.hooked || []
+    req.hooked.push('root')
+  })
 }
diff --git a/test/issues/369/routes/child/.autohooks.js b/test/issues/369/routes/child/.autohooks.js
@@ -1,8 +1,8 @@
 'use strict'
 
 module.exports = async function (app, opts) {
-    app.addHook('onRequest', async (req, reply) => {
-        req.hooked = req.hooked || []
-        req.hooked.push('child')
-      })
+  app.addHook('onRequest', async (req, reply) => {
+    req.hooked = req.hooked || []
+    req.hooked.push('child')
+  })
 }
diff --git a/test/issues/369/routes/promisified/.autohooks.js b/test/issues/369/routes/promisified/.autohooks.js
@@ -1,10 +1,10 @@
 'use strict'
 
-module.exports = new Promise((res) => {
-    res(async function (app, opts) {
-        app.addHook('onRequest', async (req, reply) => {
-            req.hooked = req.hooked || []
-            req.hooked.push('promisified')
-          })
+module.exports = new Promise((resolve) => {
+  resolve(async function (app, opts) {
+    app.addHook('onRequest', async (req, reply) => {
+      req.hooked = req.hooked || []
+      req.hooked.push('promisified')
     })
+  })
 })
diff --git a/test/issues/374/routes/entity/.autohooks.js b/test/issues/374/routes/entity/.autohooks.js
@@ -2,7 +2,7 @@
 
 module.exports = async function (app, opts) {
   app.addHook('onRequest', async (req, reply) => {
-      req.hooked = req.hooked || []
-      req.hooked.push('root')
-    })
+    req.hooked = req.hooked || []
+    req.hooked.push('root')
+  })
 }
diff --git a/test/issues/376/routes/entity/.autohooks.js b/test/issues/376/routes/entity/.autohooks.js
@@ -2,7 +2,7 @@
 
 module.exports = async function (app, opts) {
   app.addHook('onRequest', async (req, reply) => {
-      req.hooked = req.hooked || []
-      req.hooked.push('root')
-    })
+    req.hooked = req.hooked || []
+    req.hooked.push('root')
+  })
 }
diff --git a/test/typescript-jest/babel-node/index.test.ts b/test/typescript-jest/babel-node/index.test.ts
@@ -2,30 +2,30 @@ import fastify from 'fastify'
 import { join } from 'path'
 import AutoLoad from '../../../'
 
-describe("load typescript using babel-node", () => {
+describe('load typescript using babel-node', () => {
   const app = fastify()
 
-  beforeAll( done => {
+  beforeAll(done => {
     app.register(AutoLoad, {
       dir: join(__dirname, '../../commonjs/babel-node/routes')
     })
     app.ready(done)
   })
 
-  it("tests the root route", async function () {
+  it('tests the root route', async function () {
     const response = await app.inject({
       method: 'GET',
       url: '/'
     })
     expect(response.statusCode).toEqual(200)
     expect(JSON.parse(response.payload)).toEqual({ hello: 'world' })
   })
-  it("tests /foo route", async function () {
+  it('tests /foo route', async function () {
     const response = await app.inject({
       method: 'GET',
       url: '/foo'
     })
     expect(response.statusCode).toBe(200)
     expect(JSON.parse(response.payload)).toEqual({ foo: 'bar' })
   })
-})
+})
diff --git a/test/typescript-jest/basic/app.ts b/test/typescript-jest/basic/app.ts
@@ -10,4 +10,4 @@ const app: FastifyPluginCallback = function (fastify, opts, next): void {
   next()
 }
 
-export default app;
+export default app
diff --git a/test/typescript-jest/basic/basic.test.ts b/test/typescript-jest/basic/basic.test.ts
@@ -5,12 +5,12 @@ const app = fastify()
 
 app.register(basicApp)
 
-describe("load typescript plugin in jest environment", () => {
-  beforeAll( done => {
+describe('load typescript plugin in jest environment', () => {
+  beforeAll(done => {
     app.ready(done)
   })
 
-  it("should response code 200 OK", async () => {
+  it('should response code 200 OK', async () => {
     const { payload, statusCode } = await app.inject({
       url: '/typescript'
     })
diff --git a/test/typescript-jest/integration/instance.ts b/test/typescript-jest/integration/instance.ts
@@ -1,13 +1,13 @@
-import fastify from "fastify";
-import basicApp from "../../typescript/basic/app";
+import fastify from 'fastify'
+import basicApp from '../../typescript/basic/app'
 
-const app = fastify();
-app.register(basicApp);
+const app = fastify()
+app.register(basicApp)
 
 app.listen({
   port: Math.floor(Math.random() * 3000 + 3000)
 }, function (err) {
-  if (err) process.stderr.write("failed");
-  process.stdout.write("success");
-  app.close();
-});
+  if (err) process.stderr.write('failed')
+  process.stdout.write('success')
+  app.close()
+})
diff --git a/test/typescript-jest/integration/integration.test.ts b/test/typescript-jest/integration/integration.test.ts
@@ -1,26 +1,26 @@
-import { exec } from "child_process";
+import { exec } from 'child_process'
 
-describe("integration test", function () {
-  test.concurrent.each(["ts-node", "ts-node-dev"])(
-    "integration with %s",
+describe('integration test', function () {
+  test.concurrent.each(['ts-node', 'ts-node-dev'])(
+    'integration with %s',
     async function (instance) {
       await new Promise(function (resolve, reject) {
-        const child = exec(`${instance} "${process.cwd()}/test/typescript-jest/integration/instance.ts"`);
-        let stderr = "";
-        child.stderr?.on("data", function (b) {
-          stderr = stderr + b.toString();
-        });
-        let stdout = "";
-        child.stdout?.on("data", function (b) {
-          stdout = stdout + b.toString();
-        });
-        child.once("close", function () {
-          expect(stderr.includes("failed")).toStrictEqual(false);
-          expect(stdout.includes("success")).toStrictEqual(true);
-          resolve("");
-        });
-      });
+        const child = exec(`${instance} "${process.cwd()}/test/typescript-jest/integration/instance.ts"`)
+        let stderr = ''
+        child.stderr?.on('data', function (b) {
+          stderr = stderr + b.toString()
+        })
+        let stdout = ''
+        child.stdout?.on('data', function (b) {
+          stdout = stdout + b.toString()
+        })
+        child.once('close', function () {
+          expect(stderr.includes('failed')).toStrictEqual(false)
+          expect(stdout.includes('success')).toStrictEqual(true)
+          resolve('')
+        })
+      })
     },
     30000
-  );
-});
+  )
+})
diff --git a/test/typescript/basic/app.ts b/test/typescript/basic/app.ts
@@ -10,4 +10,4 @@ const app: FastifyPluginCallback = function (fastify, opts, next): void {
   next()
 }
 
-export default app;
+export default app
diff --git a/test/vitest/basic.test.ts b/test/vitest/basic.test.ts
@@ -5,13 +5,13 @@ import Fastify from 'fastify'
 import AutoLoad from '../../index'
 import { join } from 'path'
 
-describe.concurrent("Vitest test suite", function () {
+describe.concurrent('Vitest test suite', function () {
   const app = Fastify()
-    app.register(AutoLoad, {
-      dir: join(__dirname, '../commonjs/ts-node/routes')
-    })
+  app.register(AutoLoad, {
+    dir: join(__dirname, '../commonjs/ts-node/routes')
+  })
 
-  test("Test the root route", async function () {
+  test('Test the root route', async function () {
     const response = await app.inject({
       method: 'GET',
       url: '/'
@@ -20,7 +20,7 @@ describe.concurrent("Vitest test suite", function () {
     expect(JSON.parse(response.payload)).toEqual({ hello: 'world' })
   })
 
-  test("Test /foo route", async function () {
+  test('Test /foo route', async function () {
     const response = await app.inject({
       method: 'GET',
       url: '/foo'
diff --git a/test/vitest/filter.test.ts b/test/vitest/filter.test.ts
diff --git a/types/index.d.ts b/types/index.d.ts
diff --git a/types/index.test-d.ts b/types/index.test-d.ts
