linting

Author: jkyberneees

.github/workflows/tests.yaml

@@ -15,7 +15,7 @@ jobs:
         run: npm install
 
       - name: Linting
-        run: npx standard
+        run: npm run lint
 
       - name: Run tests
         run: npm run test

common.d.ts

@@ -1,29 +1,26 @@
-import { type Server as HTTPServer, IncomingMessage, ServerResponse } from "http";
-import { type Server as HTTPSServer } from "https";
-import { Pattern, Methods } from "trouter";
+import { type Server as HTTPServer, IncomingMessage, ServerResponse } from 'http'
+import { type Server as HTTPSServer } from 'https'
+import { Pattern, Methods } from 'trouter'
 
-import {
-  Http2SecureServer,
-  Http2ServerRequest,
-  Http2ServerResponse
-} from 'http2'
+import { Http2SecureServer, Http2ServerRequest, Http2ServerResponse } from 'http2'
 
 export enum Protocol {
   HTTP = 'http',
   HTTPS = 'https',
   HTTP2 = 'http2'
 }
 
-export type Server<P extends Protocol> = P extends Protocol.HTTP2 ? Http2SecureServer : P extends Protocol.HTTPS ? HTTPSServer : HTTPServer
+export type Server<P extends Protocol> = P extends Protocol.HTTP2
+  ? Http2SecureServer
+  : P extends Protocol.HTTPS
+  ? HTTPSServer
+  : HTTPServer
 
 export type StepFunction = (error?: unknown) => void
 
-export type Request<P extends Protocol> = P extends Protocol.HTTP2
-  ? Http2ServerRequest
-  : IncomingMessage
+export type Request<P extends Protocol> = P extends Protocol.HTTP2 ? Http2ServerRequest : IncomingMessage
 
-export type Response<P extends Protocol> = P extends Protocol.HTTP2
-  ? Http2ServerResponse : ServerResponse
+export type Response<P extends Protocol> = P extends Protocol.HTTP2 ? Http2ServerResponse : ServerResponse
 
 export type RequestHandler<P extends Protocol> = (
   req: Request<P>,
@@ -34,22 +31,21 @@ export type RequestHandler<P extends Protocol> = (
 export interface IRouter<P extends Protocol> {
   lookup: RequestHandler<P>
 
-  use(...handlers: RequestHandler<P>[]): this;
-  use(router: IRouter<P>): this;
-  use(pattern: Pattern, ...handlers: RequestHandler<P>[]): this;
-  use(prefix: Pattern, router: IRouter<P>): this;
-
-	all(pattern: Pattern, ...handlers: RequestHandler<P>[]): this;
-	get(pattern: Pattern, ...handlers: RequestHandler<P>[]): this;
-	head(pattern: Pattern, ...handlers: RequestHandler<P>[]): this;
-	patch(pattern: Pattern, ...handlers: RequestHandler<P>[]): this;
-	options(pattern: Pattern, ...handlers: RequestHandler<P>[]): this;
-	connect(pattern: Pattern, ...handlers: RequestHandler<P>[]): this;
-	delete(pattern: Pattern, ...handlers: RequestHandler<P>[]): this;
-	trace(pattern: Pattern, ...handlers: RequestHandler<P>[]): this;
-	post(pattern: Pattern, ...handlers: RequestHandler<P>[]): this;
-	put(pattern: Pattern, ...handlers: RequestHandler<P>[]): this;
-
-  on(method: Methods, pattern: Pattern, ...middlewares: RequestHandler<P>[]): this;
+  use(...handlers: RequestHandler<P>[]): this
+  use(router: IRouter<P>): this
+  use(pattern: Pattern, ...handlers: RequestHandler<P>[]): this
+  use(prefix: Pattern, router: IRouter<P>): this
+
+  all(pattern: Pattern, ...handlers: RequestHandler<P>[]): this
+  get(pattern: Pattern, ...handlers: RequestHandler<P>[]): this
+  head(pattern: Pattern, ...handlers: RequestHandler<P>[]): this
+  patch(pattern: Pattern, ...handlers: RequestHandler<P>[]): this
+  options(pattern: Pattern, ...handlers: RequestHandler<P>[]): this
+  connect(pattern: Pattern, ...handlers: RequestHandler<P>[]): this
+  delete(pattern: Pattern, ...handlers: RequestHandler<P>[]): this
+  trace(pattern: Pattern, ...handlers: RequestHandler<P>[]): this
+  post(pattern: Pattern, ...handlers: RequestHandler<P>[]): this
+  put(pattern: Pattern, ...handlers: RequestHandler<P>[]): this
+
+  on(method: Methods, pattern: Pattern, ...middlewares: RequestHandler<P>[]): this
 }
-

demos/basic-low-sequential.js

@@ -11,4 +11,4 @@ router.get('/hi', (req, res) => {
   res.end('Hello World!')
 })
 
-server.listen(3000, socket => {})
+server.listen(3000, (socket) => {})

demos/middleware.js

@@ -12,11 +12,15 @@ router.use('/', (req, res, next) => {
   res.write('World')
   return next()
 })
-router.get('/hi', (req, res, next) => {
-  res.write('!')
-  return next()
-}, (req, res) => {
-  res.end()
-})
+router.get(
+  '/hi',
+  (req, res, next) => {
+    res.write('!')
+    return next()
+  },
+  (req, res) => {
+    res.end()
+  }
+)
 
 server.listen(3000)

demos/uwebsockets.js

@@ -1,11 +1,13 @@
-
 const uWS = require('uWebSockets.js')
 
-uWS.App().get('/hi', (res, req) => {
-  res.onAborted(() => {
-    res.aborted = true
+uWS
+  .App()
+  .get('/hi', (res, req) => {
+    res.onAborted(() => {
+      res.aborted = true
+    })
+    setTimeout(() => {
+      res.end('Hello World!')
+    }, 100)
   })
-  setTimeout(() => {
-    res.end('Hello World!')
-  }, 100)
-}).listen(3000, () => {})
+  .listen(3000, () => {})

index.d.ts

@@ -1,13 +1,12 @@
 import { Protocol, IRouter, Server } from './common'
 
 interface IBuildServerAndRouterConfig<P extends Protocol, S extends Server<P>, R extends IRouter<P>> {
-  router?: R;
-  server?: S;
-  prioRequestsProcessing?: boolean;
+  router?: R
+  server?: S
+  prioRequestsProcessing?: boolean
 }
 
 export default function zero<P extends Protocol>(config?: IBuildServerAndRouterConfig<P, Server<P>, IRouter<P>>): {
-  server: Server<P>,
+  server: Server<P>
   router: IRouter<P>
-};
-
+}

index.js

@@ -6,7 +6,8 @@ module.exports = (config = { prioRequestsProcessing: true }) => {
   const router = config.router || require('./lib/router/sequential')()
   const server = config.server || http.createServer()
 
-  server.prioRequestsProcessing = config.prioRequestsProcessing && (server instanceof httpServer || server instanceof httpsServer)
+  server.prioRequestsProcessing =
+    config.prioRequestsProcessing && (server instanceof httpServer || server instanceof httpsServer)
 
   if (server.prioRequestsProcessing) {
     server.on('request', (req, res) => {

lib/next.js

@@ -1,4 +1,4 @@
-function next (middlewares, req, res, index, routers = {}, defaultRoute, errorHandler) {
+function next(middlewares, req, res, index, routers = {}, defaultRoute, errorHandler) {
   const middleware = middlewares[index]
   if (!middleware) {
     if (!res.finished) {
@@ -8,7 +8,7 @@ function next (middlewares, req, res, index, routers = {}, defaultRoute, errorHa
     return
   }
 
-  function step (err) {
+  function step(err) {
     if (err) {
       return errorHandler(err, req, res)
     } else {

lib/router/sequential.d.ts

@@ -1,3 +1,3 @@
 import { Protocol, IRouter, RequestHandler } from './../../common'
 
-export default function createSequentialRouter<P extends Protocol>(config?: object): IRouter<P>
+export default function createSequentialRouter<P extends Protocol>(config?: object): IRouter<P>

lib/utils/queryparams.js

@@ -5,9 +5,7 @@ module.exports = (req, url) => {
   const query = {}
   for (const [name, value] of searchParams.entries()) {
     if (query[name]) {
-      Array.isArray(query[name])
-        ? query[name].push(value)
-        : (query[name] = [query[name], value])
+      Array.isArray(query[name]) ? query[name].push(value) : (query[name] = [query[name], value])
     } else {
       query[name] = value
     }