fix: performance regression introduced by 290a1fd (PR #79)

There was an unintentional change to use the file path instead of tsconfig path for memoization, which changes for every file (whereas the tsconfig path is often the same for the whole project).

Fixes #90.

Author: simonhaenisch

lib/get-compiler-options.js

@@ -4,13 +4,11 @@ const { findTsconfig } = require('./find-tsconfig');
 const { memoize } = require('./memoize');
 
 /**
- * Get the compiler options from a path to a file in the project.
+ * Get the compiler options from the path to a tsconfig.
  *
- * @param {string} path path to a file in the project
+ * @param {string | undefined} tsconfig path to tsconfig
  */
-function getCompilerOptions(path) {
-	const tsconfig = findTsconfig(path);
-
+function getCompilerOptions(tsconfig) {
 	const compilerOptions = tsconfig
 		? ts.parseJsonConfigFileContent(ts.readConfigFile(tsconfig, ts.sys.readFile).config, ts.sys, dirname(tsconfig))
 				.options
@@ -24,15 +22,13 @@ function getCompilerOptions(path) {
 module.exports.getCompilerOptions = memoize(getCompilerOptions);
 
 /**
- * Get the Vue compiler options from a path to a file in the project.
+ * Get the Vue compiler options from the path to a tsconfig.
  *
  * Uses a dynamic require instead of top-level because this is only needed for Vue.
  *
- * @param {string} path path to a file in the project
+ * @param {string | undefined} tsconfig path to tsconfig
  */
-function getVueCompilerOptions(path) {
-	const tsconfig = findTsconfig(path);
-
+function getVueCompilerOptions(tsconfig) {
 	return tsconfig
 		? require('@volar/vue-language-core').createParsedCommandLine(
 				// @ts-ignore

lib/service-host.js

@@ -14,7 +14,7 @@ const { getCompilerOptions, getVueCompilerOptions } = require('./get-compiler-op
  */
 function getTypeScriptLanguageServiceHost(path, content) {
 	const tsconfig = findTsconfig(path);
-	const compilerOptions = getCompilerOptions(path);
+	const compilerOptions = getCompilerOptions(tsconfig);
 
 	return {
 		directoryExists: ts.sys.directoryExists,
@@ -47,7 +47,8 @@ function getTypeScriptLanguageServiceHost(path, content) {
  * @returns {VueLanguageServiceHost}
  */
 function getVueLanguageServiceHost(path, content) {
-	const vueCompilerOptions = getVueCompilerOptions(path);
+	const tsconfig = findTsconfig(path);
+	const vueCompilerOptions = getVueCompilerOptions(tsconfig);
 
 	return {
 		...getTypeScriptLanguageServiceHost(path, content),