Sanitize trace with fs caching traces as they are not deterministic

Author: sheetalkamat

internal/testutil/harnessutil/harnessutil.go

@@ -223,7 +223,7 @@ func CompileFilesEx(
 		Errors:     errors,
 	}, harnessOptions)
 	result.Symlinks = symlinks
-	result.Trace = host.tracer.String()
+	result.Trace = host.tracer.string()
 	result.Repeat = func(testConfig TestConfiguration) *CompilationResult {
 		newHarnessOptions := *harnessOptions
 		newCompilerOptions := compilerOptions.Clone()
@@ -474,7 +474,7 @@ func getOptionValue(t *testing.T, option *tsoptions.CommandLineOption, value str
 
 type cachedCompilerHost struct {
 	compiler.CompilerHost
-	tracer *strings.Builder
+	tracer *tracer
 }
 
 var sourceFileCache collections.SyncMap[SourceFileCacheKey, *ast.SourceFile]
@@ -515,13 +515,74 @@ func (h *cachedCompilerHost) GetSourceFile(opts ast.SourceFileParseOptions) *ast
 	return result
 }
 
-func createCompilerHost(fs vfs.FS, defaultLibraryPath string, currentDirectory string) *cachedCompilerHost {
-	var tracer strings.Builder
-	trace := func(msg string) {
-		fmt.Fprintln(&tracer, strings.Replace(msg, "'"+core.Version()+"'", "'"+FakeTSVersion+"'", 1))
+type tracer struct {
+	fs               vfs.FS
+	currentDirectory string
+	packageJsonCache map[tspath.Path]bool
+	builder          strings.Builder
+}
+
+func (t *tracer) trace(msg string) {
+	fmt.Fprintln(&t.builder, t.sanitizeTrace(msg))
+}
+
+func (t *tracer) sanitizeTrace(msg string) string {
+	// Version
+	if str := strings.Replace(msg, "'"+core.Version()+"'", "'"+FakeTSVersion+"'", 1); str != msg {
+		return str
+	}
+	// caching of fs in trace to be replaces with non caching version
+	if str := strings.TrimSuffix(msg, "' does not exist according to earlier cached lookups."); str != msg {
+		file := strings.TrimPrefix(str, "File '")
+		filePath := tspath.ToPath(file, t.currentDirectory, t.fs.UseCaseSensitiveFileNames())
+		if _, has := t.packageJsonCache[filePath]; has {
+			return msg
+		} else {
+			t.packageJsonCache[filePath] = false
+			return fmt.Sprintf("File '%s' does not exist.", file)
+		}
 	}
+	if str := strings.TrimSuffix(msg, "' does not exist."); str != msg {
+		file := strings.TrimPrefix(str, "File '")
+		filePath := tspath.ToPath(file, t.currentDirectory, t.fs.UseCaseSensitiveFileNames())
+		if _, has := t.packageJsonCache[filePath]; !has {
+			t.packageJsonCache[filePath] = false
+			return msg
+		} else {
+			return fmt.Sprintf("File '%s' does not exist according to earlier cached lookups.", file)
+		}
+	}
+	if str := strings.TrimSuffix(msg, "' exists according to earlier cached lookups."); str != msg {
+		file := strings.TrimPrefix(str, "File '")
+		filePath := tspath.ToPath(file, t.currentDirectory, t.fs.UseCaseSensitiveFileNames())
+		if _, has := t.packageJsonCache[filePath]; has {
+			return msg
+		} else {
+			t.packageJsonCache[filePath] = true
+			return fmt.Sprintf("Found 'package.json' at '%s'.", file)
+		}
+	}
+	if str := strings.TrimPrefix(msg, "Found 'package.json' at '"); str != msg {
+		file := strings.TrimSuffix(str, "'.")
+		filePath := tspath.ToPath(file, t.currentDirectory, t.fs.UseCaseSensitiveFileNames())
+		if _, has := t.packageJsonCache[filePath]; !has {
+			t.packageJsonCache[filePath] = true
+			return msg
+		} else {
+			return fmt.Sprintf("File '%s' exists according to earlier cached lookups.", file)
+		}
+	}
+	return msg
+}
+
+func (t *tracer) string() string {
+	return t.builder.String()
+}
+
+func createCompilerHost(fs vfs.FS, defaultLibraryPath string, currentDirectory string) *cachedCompilerHost {
+	tracer := tracer{fs: fs, currentDirectory: currentDirectory, packageJsonCache: make(map[tspath.Path]bool)}
 	return &cachedCompilerHost{
-		CompilerHost: compiler.NewCompilerHost(currentDirectory, fs, defaultLibraryPath, nil, trace),
+		CompilerHost: compiler.NewCompilerHost(currentDirectory, fs, defaultLibraryPath, nil, tracer.trace),
 		tracer:       &tracer,
 	}
 }