fix: improve perf (#764)

Author: alexander-akait

lint-staged.config.js

@@ -1,4 +1,7 @@
 module.exports = {
-  "*": ["prettier --write --ignore-unknown", "cspell --no-must-find-files"],
+  "*": [
+    "prettier --cache --write --ignore-unknown",
+    "cspell --cache --no-must-find-files",
+  ],
   "*.js": ["eslint --cache --fix"],
 };

package.json

@@ -25,9 +25,9 @@
     "build": "npm-run-all -p \"build:**\"",
     "commitlint": "commitlint --from=master",
     "security": "npm audit --production",
-    "lint:prettier": "prettier --list-different .",
+    "lint:prettier": "prettier --cache --list-different .",
     "lint:js": "eslint --cache .",
-    "lint:spelling": "cspell --cache --quiet \"**/*.*\"",
+    "lint:spelling": "cspell --cache --no-must-find-files --quiet \"**/*.*\"",
     "lint:types": "tsc --pretty --noEmit",
     "lint": "npm-run-all -l -p \"lint:**\"",
     "fix:js": "npm run lint:js -- --fix",

src/index.js

@@ -6,13 +6,7 @@ const { validate } = require("schema-utils");
 const { version } = require("../package.json");
 
 const schema = require("./options.json");
-const {
-  readFile,
-  stat,
-  throttleAll,
-  memoize,
-  asyncMemoize,
-} = require("./utils");
+const { readFile, stat, throttleAll, memoize } = require("./utils");
 
 const template = /\[\\*([\w:]+)\\*\]/i;
 
@@ -36,7 +30,7 @@ const getFastGlob = memoize(() =>
   require("fast-glob"),
 );
 
-const getGlobby = asyncMemoize(async () => {
+const getGlobby = memoize(async () => {
   // @ts-ignore
   const { globby } = await import("globby");
 

src/utils.js

@@ -145,31 +145,4 @@ function memoize(fn) {
   };
 }
 
-/**
- * @template T
- * @param fn {(function(): any) | undefined}
- * @returns {function(): Promise<T>}
- */
-function asyncMemoize(fn) {
-  let cache = false;
-  /** @type {T} */
-  let result;
-
-  return async () => {
-    if (cache) {
-      return result;
-    }
-
-    result = await /** @type {function(): any} */ (fn)();
-    cache = true;
-
-    // Allow to clean up memory for fn
-    // and all dependent resources
-    // eslint-disable-next-line no-undefined, no-param-reassign
-    fn = undefined;
-
-    return result;
-  };
-}
-
-module.exports = { stat, readFile, throttleAll, memoize, asyncMemoize };
+module.exports = { stat, readFile, throttleAll, memoize };

types/utils.d.ts

@@ -39,9 +39,3 @@ export function throttleAll<T>(limit: number, tasks: Task<T>[]): Promise<T[]>;
  * @returns {function(): T}
  */
 export function memoize<T>(fn: (() => any) | undefined): () => T;
-/**
- * @template T
- * @param fn {(function(): any) | undefined}
- * @returns {function(): Promise<T>}
- */
-export function asyncMemoize<T>(fn: (() => any) | undefined): () => Promise<T>;