build: use biome for formatting and fix tests (#125)

Author: JustinBeckwith

biome.json

@@ -0,0 +1,30 @@
+{
+  "$schema": "https://biomejs.dev/schemas/2.2.6/schema.json",
+  "files": {
+    "includes": [
+      "**/lib/**/*.js",
+      "**/test/**/*.js",
+      "**/example/**/*.js",
+      "!**/node_modules/**/*",
+      "!**/coverage/**/*"
+    ]
+  },
+  "assist": { "actions": { "source": { "organizeImports": "on" } } },
+  "linter": {
+    "enabled": true,
+    "rules": {
+      "recommended": true
+    }
+  },
+  "formatter": {
+    "enabled": true,
+    "indentStyle": "space",
+    "indentWidth": 2,
+    "lineWidth": 80
+  },
+  "javascript": {
+    "formatter": {
+      "quoteStyle": "single"
+    }
+  }
+}

example/app.js

@@ -7,11 +7,11 @@ const app = express();
 // Use the yes-https connect middleware.  Note - this will only work if NODE_ENV is set to production.
 app.use(yes());
 
-app.get('/', (request, response) => {
-	response.end('Thanks for checking it out!');
+app.get('/', (_request, response) => {
+  response.end('Thanks for checking it out!');
 });
 
 const server = app.listen(process.env.PORT || 3000, () => {
-	console.log('App listening on port %s', server.address().port);
-	console.log('Press Ctrl+C to quit.');
+  console.log('App listening on port %s', server.address().port);
+  console.log('Press Ctrl+C to quit.');
 });

lib/index.js

@@ -2,40 +2,43 @@ import process from 'node:process';
 
 // eslint-disable-next-line import/no-anonymous-default-export
 export default function (options) {
-	options ||= {};
-	const maxAge = options.maxAge || 86_400;
-	const includeSubDomains = options.includeSubDomains === undefined ? true : options.includeSubdomains;
+  options ||= {};
+  const maxAge = options.maxAge || 86_400;
+  const includeSubDomains =
+    options.includeSubDomains === undefined ? true : options.includeSubdomains;
 
-	return function (request, response, next) {
-		let ignoreRequest = (process.env.NODE_ENV !== 'production');
-		const secure = request.connection.encrypted || (request.get('X-Forwarded-Proto') === 'https');
+  return (request, response, next) => {
+    let ignoreRequest = process.env.NODE_ENV !== 'production';
+    const secure =
+      request.connection.encrypted ||
+      request.get('X-Forwarded-Proto') === 'https';
 
-		if (options.ignoreFilter) {
-			ignoreRequest ||= options.ignoreFilter(request);
-		}
+    if (options.ignoreFilter) {
+      ignoreRequest ||= options.ignoreFilter(request);
+    }
 
-		if (ignoreRequest) {
-			next();
-			return;
-		}
+    if (ignoreRequest) {
+      next();
+      return;
+    }
 
-		if (secure) {
-			let header = 'max-age=' + maxAge;
-			if (includeSubDomains) {
-				header += '; includeSubDomains';
-			}
+    if (secure) {
+      let header = `max-age=${maxAge}`;
+      if (includeSubDomains) {
+        header += '; includeSubDomains';
+      }
 
-			if (options.preload) {
-				header += '; preload';
-			}
+      if (options.preload) {
+        header += '; preload';
+      }
 
-			response.setHeader('Strict-Transport-Security', header);
-			next();
-		} else {
-			response.writeHead(301, {
-				Location: 'https://' + request.get('host') + request.url,
-			});
-			response.end();
-		}
-	};
+      response.setHeader('Strict-Transport-Security', header);
+      next();
+    } else {
+      response.writeHead(301, {
+        Location: `https://${request.get('host')}${request.url}`,
+      });
+      response.end();
+    }
+  };
 }