wip/change ~ fix biome linter complaints

- lint/complexity/useLiteralKeys
- lint/correctness/noConstructorReturn
- lint/style/noParameterAssign
- lint/style/useTemplate
- lint/suspicious/noExplicitAny
- lint/suspicious/noRedundantUseStrict

.commitlint.config.js

@@ -6,9 +6,9 @@
 
 /* @prettier */ // note: (dprint) {.dprint.json}.prettier.associations should contain the name of this file
 
-const isNPMTestDist = !!process.env['npm_config_test_dist'];
-const isTestDist = !!process.env['test_dist'];
-const isTestRelease = !!process.env['test_release'];
+const isNPMTestDist = !!process.env.npm_config_test_dist;
+const isTestDist = !!process.env.test_dist;
+const isTestRelease = !!process.env.test_release;
 
 /** Relax linting rules/strictures (for development; *not* when submitting for distribution/release). */
 const relaxedReview = !(isNPMTestDist || isTestDist || isTestRelease);

biome.json

@@ -0,0 +1,28 @@
+{
+  "$schema": "https://biomejs.dev/schemas/1.2.2/schema.json",
+  "formatter":{
+		"lineWidth": 100
+	},
+	"javascript": {
+		"formatter": {
+			"quoteStyle": "single",
+			"jsxQuoteStyle": "single",
+			"trailingComma": "all"
+		}
+	},
+	"json": {
+		"parser": {
+			"allowComments": true,
+			"allowTrailingCommas": true
+		}
+	},
+	"linter": {
+    "enabled": true,
+    "rules": {
+      "recommended": true
+    }
+  },
+	"organizeImports": {
+    "enabled": false
+  }
+}

eg/show-paths.cjs.js

@@ -1,7 +1,7 @@
 // deno-fmt-ignore-file ## prefer customized `prettier` formatting
 // # spell-checker:ignore APPNAME
 /* eslint-env es6, node */
-'use strict';
+// 'use strict';
 
 const path = require('path');
 
@@ -20,27 +20,30 @@ xdgAppPaths.log = function (dirOptions) {
 		return typeof x;
 	}
 
-	if (typeOf(dirOptions) === 'boolean') {
-		dirOptions = { isolated: dirOptions };
-	}
-
-	if (
-		typeOf(dirOptions) !== 'object' ||
-		dirOptions === null ||
-		typeOf(dirOptions.isolated) !== 'boolean'
-	) {
-		dirOptions = { isolated: self.$isolated() };
-	}
-
-	return path.join(self.state(dirOptions), (dirOptions.isolated ? '' : self.$name() + '-') + 'log');
+	const dirOptions_ = (() => {
+		if (typeOf(dirOptions) === 'boolean') {
+			return { isolated: dirOptions };
+		}
+		if (
+			typeOf(dirOptions) !== 'object' ||
+			dirOptions == null ||
+			typeOf(dirOptions.isolated) !== 'boolean'
+		) {
+			return { isolated: self.$isolated() };
+		}
+	})();
+
+	return path.join(
+		self.state(dirOptions_),
+		`${dirOptions_.isolated ? '' : `${self.$name()}-`} + 'log'`
+	);
 };
 
 function showObjectEntries(obj) {
-	var strings = [];
-	Object.keys(obj).forEach((key) => {
+	const strings = Object.keys(obj).map((key) => {
 		const value = obj[key];
 		const val = typeof value === 'function' ? value() : value;
-		strings.push(key + ' = ' + val);
+		return `${key} = ${val}`;
 	});
 	return strings.join('\n');
 }
@@ -53,7 +56,7 @@ console.log('appPaths.log(false):', xdgAppPaths.log(false));
 console.log('appPaths.log(true):', xdgAppPaths.log(true));
 
 // eslint-disable-next-line functional/immutable-data
-delete process.env.XDG_CONFIG_HOME;
+process.env.XDG_CONFIG_HOME = void 0;
 // eslint-disable-next-line functional/no-let
 let p = require(xdgAppPathsModulePath)('dross');
 

eg/show-paths.esm.mjs

@@ -1,7 +1,7 @@
 // deno-fmt-ignore-file ## prefer customized `prettier` formatting
 // # spell-checker:ignore APPNAME
 /* eslint-env es6, node */
-'use strict';
+// 'use strict';
 
 import path from 'path';
 import { inspect } from 'util';
@@ -11,13 +11,10 @@ import { inspect } from 'util';
 import xdgAppPaths from '../dist/cjs/esm-wrapper/mod.esm.js';
 
 function objectEntries(obj) {
-	const map = {};
-	Object.keys(obj).forEach((key) => {
+	return Object.keys(obj).map((key) => {
 		const value = obj[key];
-		const val = typeof value === 'function' ? value() : value;
-		map[key] = val;
+		return typeof value === 'function' ? value() : value;
 	});
-	return map;
 }
 
 // Extend appPaths with a "log" location function
@@ -28,27 +25,30 @@ xdgAppPaths.log = function (dirOptions = null) {
 		return typeof x;
 	}
 
-	if (typeOf(dirOptions) === 'boolean') {
-		dirOptions = { isolated: dirOptions };
-	}
-
-	if (
-		typeOf(dirOptions) !== 'object' ||
-		dirOptions === null ||
-		typeOf(dirOptions.isolated) !== 'boolean'
-	) {
-		dirOptions = { isolated: self.$isolated() };
-	}
-
-	return path.join(self.state(dirOptions), (dirOptions.isolated ? '' : self.$name() + '-') + 'log');
+	const dirOptions_ = (() => {
+		if (typeOf(dirOptions) === 'boolean') {
+			return { isolated: dirOptions };
+		}
+		if (
+			typeOf(dirOptions) !== 'object' ||
+			dirOptions == null ||
+			typeOf(dirOptions.isolated) !== 'boolean'
+		) {
+			return { isolated: self.$isolated() };
+		}
+	})();
+
+	return path.join(
+		self.state(dirOptions_),
+		`${dirOptions_.isolated ? '' : `${self.$name()}-`} + 'log'`
+	);
 };
 
 function showObjectEntries(obj) {
-	var strings = [];
-	Object.keys(obj).forEach((key) => {
+	const strings = Object.keys(obj).map((key) => {
 		const value = obj[key];
 		const val = typeof value === 'function' ? value() : value;
-		strings.push(key + ' = ' + val);
+		return `${key} = ${val}`;
 	});
 	return strings.join('\n');
 }
@@ -60,7 +60,7 @@ console.log('appPaths.log():', xdgAppPaths.log());
 console.log('appPaths.log(false):', xdgAppPaths.log(false));
 console.log('appPaths.log(true):', xdgAppPaths.log(true));
 
-delete process.env.XDG_CONFIG_HOME;
+process.env.XDG_CONFIG_HOME = void 0;
 // eslint-disable-next-line functional/no-let
 let p = xdgAppPaths('dross');
 

eg/show-paths.local.deno.ts

@@ -14,7 +14,7 @@ const deno = Deno;
 
 const inspect = deno.inspect;
 
-/* eslint-disable @typescript-eslint/no-explicit-any , functional/immutable-data , no-console , security-node/detect-crlf , security/detect-object-injection */
+/* eslint-disable functional/immutable-data , no-console , security-node/detect-crlf , security/detect-object-injection */
 
 // eslint-disable-next-line @typescript-eslint/ban-ts-comment
 // @ts-ignore // deno-type import
@@ -23,14 +23,12 @@ import xdgAppPaths from '../src/mod.deno.ts';
 // @ts-ignore // deno-type import
 import type { DirOptions, XDGAppPaths } from '../src/mod.deno.ts';
 
-function objectEntries(obj: any) {
-	const map: any = {};
-	Object.keys(obj).forEach((key) => {
-		const value = obj[key];
-		const val = typeof value === 'function' ? value() : value;
-		map[key] = val;
+function objectEntries(obj: unknown) {
+	const obj_ = obj as { readonly [key: string]: unknown };
+	return Object.keys(obj_).map((key) => {
+		const value = obj_[key];
+		return typeof value === 'function' ? value() : value;
 	});
-	return map;
 }
 
 // eslint-disable-next-line functional/prefer-readonly-type
@@ -39,9 +37,9 @@ type XDGAppPathsWithLog = XDGAppPaths & { log: (dirOptions?: DirOptions | boolea
 // Extend appPaths with a "log" location
 (xdgAppPaths as XDGAppPathsWithLog).log = function log(dirOptions?: DirOptions | boolean) {
 	const self = xdgAppPaths;
-	dirOptions = dirOptions ?? { isolated: self.$isolated() };
-	const isolated = typeof dirOptions === 'boolean' ? dirOptions : dirOptions.isolated || true;
-	return path.join(self.state(isolated), (isolated ? '' : self.$name() + '-') + 'log');
+	const dirOptions_ = dirOptions ?? { isolated: self.$isolated() };
+	const isolated = typeof dirOptions_ === 'boolean' ? dirOptions_ : dirOptions_.isolated || true;
+	return path.join(self.state(isolated), `${isolated ? '' : `${self.$name()}-`} + 'log'`);
 };
 
 console.log('appPaths:', inspect(xdgAppPaths));
@@ -79,4 +77,4 @@ p = xdgAppPaths({ name: 'fluffy', isolated: false });
 console.log('p:', inspect(p));
 console.log(objectEntries(p));
 
-/* eslint-enable @typescript-eslint/no-explicit-any , functional/immutable-data , no-console , security-node/detect-crlf , security/detect-object-injection */
+/* eslint-enable functional/immutable-data , no-console , security-node/detect-crlf , security/detect-object-injection */

eg/show-paths.remote(CDN).deno.ts

@@ -4,7 +4,7 @@
 // --allow-env (transitive from 'xdg-app-paths')
 // --allow-read
 
-/* eslint-disable @typescript-eslint/ban-ts-comment , @typescript-eslint/no-explicit-any , functional/immutable-data , import/order , no-console , security-node/detect-crlf , security/detect-object-injection */
+/* eslint-disable @typescript-eslint/ban-ts-comment , functional/immutable-data , import/order , no-console , security-node/detect-crlf , security/detect-object-injection */
 
 // @ts-ignore // deno-type URL import
 import * as path from 'https://deno.land/std@0.150.0/path/mod.ts';
@@ -21,14 +21,12 @@ import xdgAppPaths from 'https://cdn.jsdelivr.net/gh/rivy/js.xdg-app-paths@v8.1.
 // @ts-ignore // deno-type import
 import type { DirOptions, XDGAppPaths } from '../src/mod.deno.ts';
 
-function objectEntries(obj: any) {
-	const map: any = {};
-	Object.keys(obj).forEach((key) => {
-		const value = obj[key];
-		const val = typeof value === 'function' ? value() : value;
-		map[key] = val;
+function objectEntries(obj: unknown) {
+	const obj_ = obj as { readonly [key: string]: unknown };
+	return Object.keys(obj_).map((key) => {
+		const value = obj_[key];
+		return typeof value === 'function' ? value() : value;
 	});
-	return map;
 }
 
 // eslint-disable-next-line functional/prefer-readonly-type
@@ -37,9 +35,9 @@ type XDGAppPathsWithLog = XDGAppPaths & { log: (dirOptions?: DirOptions | boolea
 // Extend appPaths with a "log" location
 (xdgAppPaths as XDGAppPathsWithLog).log = function log(dirOptions?: DirOptions | boolean) {
 	const self = xdgAppPaths;
-	dirOptions = dirOptions ?? { isolated: self.$isolated() };
-	const isolated = typeof dirOptions === 'boolean' ? dirOptions : dirOptions.isolated || true;
-	return path.join(self.state(isolated), (isolated ? '' : self.$name() + '-') + 'log');
+	const dirOptions_ = dirOptions ?? { isolated: self.$isolated() };
+	const isolated = typeof dirOptions_ === 'boolean' ? dirOptions_ : dirOptions_.isolated || true;
+	return path.join(self.state(isolated), `${isolated ? '' : `${self.$name()}-`}log`);
 };
 
 console.log('appPaths:', inspect(xdgAppPaths));
@@ -77,4 +75,4 @@ p = xdgAppPaths({ name: 'fluffy', isolated: false });
 console.log('p:', inspect(p));
 console.log(objectEntries(p));
 
-/* eslint-enable @typescript-eslint/ban-ts-comment , @typescript-eslint/no-explicit-any , functional/immutable-data , import/order , no-console , security-node/detect-crlf , security/detect-object-injection */
+/* eslint-enable @typescript-eslint/ban-ts-comment , functional/immutable-data , import/order , no-console , security-node/detect-crlf , security/detect-object-injection */

eg/show-paths.remote.deno.ts

@@ -4,7 +4,7 @@
 // --allow-env (transitive from 'xdg-app-paths')
 // --allow-read
 
-/* eslint-disable @typescript-eslint/ban-ts-comment , @typescript-eslint/no-explicit-any , functional/immutable-data , import/order , no-console , security-node/detect-crlf , security/detect-object-injection */
+/* eslint-disable @typescript-eslint/ban-ts-comment , functional/immutable-data , import/order , no-console , security-node/detect-crlf , security/detect-object-injection */
 
 // @ts-ignore // deno-type URL import
 import * as path from 'https://deno.land/std@0.150.0/path/mod.ts';
@@ -21,14 +21,12 @@ import xdgAppPaths from 'https://deno.land/x/xdg_app_paths@v8.1.0/src/mod.deno.t
 // @ts-ignore // deno-type import
 import type { DirOptions, XDGAppPaths } from '../src/mod.deno.ts';
 
-function objectEntries(obj: any) {
-	const map: any = {};
-	Object.keys(obj).forEach((key) => {
-		const value = obj[key];
-		const val = typeof value === 'function' ? value() : value;
-		map[key] = val;
+function objectEntries(obj: unknown) {
+	const obj_ = obj as { readonly [key: string]: unknown };
+	return Object.keys(obj_).map((key) => {
+		const value = obj_[key];
+		return typeof value === 'function' ? value() : value;
 	});
-	return map;
 }
 
 // eslint-disable-next-line functional/prefer-readonly-type
@@ -37,9 +35,9 @@ type XDGAppPathsWithLog = XDGAppPaths & { log: (dirOptions?: DirOptions | boolea
 // Extend appPaths with a "log" location
 (xdgAppPaths as XDGAppPathsWithLog).log = function log(dirOptions?: DirOptions | boolean) {
 	const self = xdgAppPaths;
-	dirOptions = dirOptions ?? { isolated: self.$isolated() };
-	const isolated = typeof dirOptions === 'boolean' ? dirOptions : dirOptions.isolated || true;
-	return path.join(self.state(isolated), (isolated ? '' : self.$name() + '-') + 'log');
+	const dirOptions_ = dirOptions ?? { isolated: self.$isolated() };
+	const isolated = typeof dirOptions_ === 'boolean' ? dirOptions_ : dirOptions_.isolated || true;
+	return path.join(self.state(isolated), `${isolated ? '' : `${self.$name()}-`}log`);
 };
 
 console.log('appPaths:', inspect(xdgAppPaths));
@@ -77,4 +75,4 @@ p = xdgAppPaths({ name: 'fluffy', isolated: false });
 console.log('p:', inspect(p));
 console.log(objectEntries(p));
 
-/* eslint-enable @typescript-eslint/ban-ts-comment , @typescript-eslint/no-explicit-any , functional/immutable-data , import/order , no-console , security-node/detect-crlf , security/detect-object-injection */
+/* eslint-enable @typescript-eslint/ban-ts-comment , functional/immutable-data , import/order , no-console , security-node/detect-crlf , security/detect-object-injection */

eg/show-paths.ts

@@ -3,19 +3,17 @@
 import path from 'path';
 import { inspect } from 'util';
 
-/* eslint-disable @typescript-eslint/no-explicit-any , functional/immutable-data , no-console , security-node/detect-crlf , security/detect-object-injection */
+/* eslint-disable functional/immutable-data , no-console , security-node/detect-crlf , security/detect-object-injection */
 
 import xdgAppPaths from '../dist/cjs/mod.cjs';
 import type { DirOptions, XDGAppPaths } from '../src/mod.esm';
 
-function objectEntries(obj: any) {
-	const map: any = {};
-	Object.keys(obj).forEach((key) => {
-		const value = obj[key];
-		const val = typeof value === 'function' ? value() : value;
-		map[key] = val;
+function objectEntries(obj: unknown) {
+	const obj_ = obj as { readonly [key: string]: unknown };
+	return Object.keys(obj_).map((key) => {
+		const value = obj_[key];
+		return typeof value === 'function' ? value() : value;
 	});
-	return map;
 }
 
 // eslint-disable-next-line functional/prefer-readonly-type
@@ -24,9 +22,9 @@ type XDGAppPathsWithLog = XDGAppPaths & { log: (dirOptions?: DirOptions | boolea
 // Extend appPaths with a "log" location
 (xdgAppPaths as XDGAppPathsWithLog).log = function log(dirOptions?: DirOptions | boolean) {
 	const self = xdgAppPaths;
-	dirOptions = dirOptions ?? { isolated: self.$isolated() };
-	const isolated = typeof dirOptions === 'boolean' ? dirOptions : dirOptions.isolated || true;
-	return path.join(self.state(isolated), (isolated ? '' : self.$name() + '-') + 'log');
+	const dirOptions_ = dirOptions ?? { isolated: self.$isolated() };
+	const isolated = typeof dirOptions_ === 'boolean' ? dirOptions_ : dirOptions_.isolated || true;
+	return path.join(self.state(isolated), `${isolated ? '' : `${self.$name()}-`} + 'log'`);
 };
 
 console.log('appPaths:', inspect(xdgAppPaths));
@@ -36,7 +34,7 @@ console.log(objectEntries(xdgAppPaths));
 console.log('appPaths.log(false):', (xdgAppPaths as XDGAppPathsWithLog).log(false));
 console.log('appPaths.log(true):', (xdgAppPaths as XDGAppPathsWithLog).log(true));
 
-delete process.env.XDG_CONFIG_HOME;
+process.env.XDG_CONFIG_HOME = void 0;
 // eslint-disable-next-line functional/no-let
 let p = xdgAppPaths('dross');
 
@@ -58,4 +56,4 @@ p = xdgAppPaths({ name: 'fluffy', isolated: false });
 console.log('p:', inspect(p));
 console.log(objectEntries(p));
 
-/* eslint-enable @typescript-eslint/no-explicit-any , functional/immutable-data , no-console , security-node/detect-crlf , security/detect-object-injection */
+/* eslint-enable functional/immutable-data , no-console , security-node/detect-crlf , security/detect-object-injection */