Optimize fileExists callback path (#1266)

* Optimize fileExists callback path
Instead of looking up filePathKey twice, cache in in the local scope. This isn't a hugely expensive lookup path, but is a hot path on large project recompiles (especially after caches are blown away). Profiling shows that even the Map lookup shows non-trivial cache lookup aggregate time

* Update Changelog & package.json

Author: berickson1

CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## v8.0.18
+* [Perf: Optimize fileExists callback path](https://github.com/TypeStrong/ts-loader/issues/1266) - thanks @berickson1
+
 ## v8.0.17
 * [Included correct webpack source location in emitted errors](https://github.com/TypeStrong/ts-loader/issues/1199) - thanks @lorenzodallavecchia
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ts-loader",
-  "version": "8.0.17",
+  "version": "8.0.18",
   "description": "TypeScript loader for webpack",
   "main": "index.js",
   "types": "dist",

src/servicesHost.ts

@@ -683,10 +683,14 @@ export function makeSolutionBuilderHost(
     scriptRegex,
     loader,
     instance,
-    fileName =>
-      !!instance.files.has(filePathKeyMapper(fileName)) ||
-      !!instance.otherFiles.get(filePathKeyMapper(fileName)) ||
-      compiler.sys.fileExists(fileName),
+    fileName => {
+      const filePathKey = filePathKeyMapper(fileName);
+      return (
+        instance.files.has(filePathKey) ||
+        instance.otherFiles.has(filePathKey) ||
+        compiler.sys.fileExists(fileName)
+      );
+    },
     /*enableFileCaching*/ true
   );