Fix project references bug

src/typescript-reporter/reporter/ControlledTypeScriptSystem.ts

@@ -150,11 +150,18 @@ function createControlledTypeScriptSystem(
   }
 
   function getReadFileSystem(path: string) {
-    if (
-      !isInitialRun &&
-      (mode === 'readonly' || mode === 'write-tsbuildinfo') &&
-      isArtifact(path)
-    ) {
+    if ((mode === 'readonly' || mode === 'write-tsbuildinfo') && isArtifact(path)) {
+      if (isInitialRun && !memFileSystem.exists(path) && passiveFileSystem.exists(path)) {
+        // copy file to memory on initial run
+        const stats = passiveFileSystem.readStats(path);
+        if (stats?.isFile()) {
+          const content = passiveFileSystem.readFile(path);
+          if (content) {
+            memFileSystem.writeFile(path, content);
+            memFileSystem.updateTimes(path, stats.atime, stats.mtime);
+          }
+        }
+      }
       return memFileSystem;
     }
 
@@ -175,6 +182,9 @@ function createControlledTypeScriptSystem(
   const controlledSystem: ControlledTypeScriptSystem = {
     ...typescript.sys,
     useCaseSensitiveFileNames: caseSensitive,
+    realpath(path: string): string {
+      return getReadFileSystem(path).realPath(path);
+    },
     fileExists(path: string): boolean {
       const stats = getReadFileSystem(path).readStats(path);
 

src/typescript-reporter/reporter/TypeScriptReporter.ts

@@ -266,16 +266,15 @@ function createTypeScriptReporter(configuration: TypeScriptReporterConfiguration
         diagnosticsPerProject.clear();
         configurationChanged = true;
       } else {
-        const previousParsedConfiguration = parsedConfiguration;
+        const previousDependencies = dependencies;
         [parsedConfiguration, parseConfigurationDiagnostics] = parseConfiguration();
+        dependencies = getDependencies();
 
         if (
-          previousParsedConfiguration &&
-          JSON.stringify(previousParsedConfiguration.fileNames) !==
-            JSON.stringify(parsedConfiguration.fileNames)
+          previousDependencies &&
+          JSON.stringify(previousDependencies) !== JSON.stringify(dependencies)
         ) {
-          // root files changed - we need to recompute dependencies and artifacts
-          dependencies = getDependencies();
+          // dependencies changed - we need to recompute artifacts
           artifacts = getArtifacts();
           shouldUpdateRootFiles = true;
         }

test/e2e/TypeScriptGenerateTrace.spec.ts

@@ -73,7 +73,7 @@ describe('TypeScript Generate Trace', () => {
 
     // update sandbox to generate trace
     await sandbox.patch(
-      'tsconfig.json',
+      'tsconfig.base.json',
       '    "rootDir": "./packages"',
       ['    "rootDir": "./packages",', '    "generateTrace": "./traces"'].join('\n')
     );

test/e2e/TypeScriptSolutionBuilderApi.spec.ts

@@ -105,6 +105,10 @@ describe('TypeScript SolutionBuilder API', () => {
     await driver.waitForNoErrors();
 
     await sandbox.write('packages/client/src/nested/additional.ts', 'export const x = 10;');
+
+    // this compilation should be successful
+    await driver.waitForNoErrors();
+
     await sandbox.patch(
       'packages/client/src/index.ts',
       'import { intersect, subtract } from "@project-references-fixture/shared";',
@@ -125,31 +129,31 @@ describe('TypeScript SolutionBuilder API', () => {
         break;
 
       case 'write-tsbuildinfo':
-        expect(await sandbox.exists('packages/shared/tsconfig.tsbuildinfo')).toEqual(true);
-        expect(await sandbox.exists('packages/client/tsconfig.tsbuildinfo')).toEqual(true);
-        expect(await sandbox.exists('packages/shared/lib')).toEqual(false);
-        expect(await sandbox.exists('packages/client/lib')).toEqual(false);
+        expect(await sandbox.exists('packages/shared/lib/tsconfig.tsbuildinfo')).toEqual(true);
+        expect(await sandbox.exists('packages/client/lib/tsconfig.tsbuildinfo')).toEqual(true);
+        expect(await sandbox.exists('packages/shared/lib')).toEqual(true);
+        expect(await sandbox.exists('packages/client/lib')).toEqual(true);
+        expect(await sandbox.exists('packages/shared/lib/index.js')).toEqual(false);
+        expect(await sandbox.exists('packages/client/lib/index.js')).toEqual(false);
 
-        expect(await sandbox.read('packages/shared/tsconfig.tsbuildinfo')).not.toEqual('');
-        expect(await sandbox.read('packages/client/tsconfig.tsbuildinfo')).not.toEqual('');
+        expect(await sandbox.read('packages/shared/lib/tsconfig.tsbuildinfo')).not.toEqual('');
+        expect(await sandbox.read('packages/client/lib/tsconfig.tsbuildinfo')).not.toEqual('');
 
-        await sandbox.remove('packages/shared/tsconfig.tsbuildinfo');
-        await sandbox.remove('packages/client/tsconfig.tsbuildinfo');
+        await sandbox.remove('packages/shared/lib');
+        await sandbox.remove('packages/client/lib');
         break;
 
       case 'write-references':
-        expect(await sandbox.exists('packages/shared/tsconfig.tsbuildinfo')).toEqual(true);
-        expect(await sandbox.exists('packages/client/tsconfig.tsbuildinfo')).toEqual(true);
+        expect(await sandbox.exists('packages/shared/lib/tsconfig.tsbuildinfo')).toEqual(true);
+        expect(await sandbox.exists('packages/client/lib/tsconfig.tsbuildinfo')).toEqual(true);
         expect(await sandbox.exists('packages/shared/lib')).toEqual(true);
         expect(await sandbox.exists('packages/client/lib')).toEqual(true);
         expect(await sandbox.exists('packages/shared/lib/index.js')).toEqual(true);
         expect(await sandbox.exists('packages/client/lib/index.js')).toEqual(true);
 
-        expect(await sandbox.read('packages/shared/tsconfig.tsbuildinfo')).not.toEqual('');
-        expect(await sandbox.read('packages/client/tsconfig.tsbuildinfo')).not.toEqual('');
+        expect(await sandbox.read('packages/shared/lib/tsconfig.tsbuildinfo')).not.toEqual('');
+        expect(await sandbox.read('packages/client/lib/tsconfig.tsbuildinfo')).not.toEqual('');
 
-        await sandbox.remove('packages/shared/tsconfig.tsbuildinfo');
-        await sandbox.remove('packages/client/tsconfig.tsbuildinfo');
         await sandbox.remove('packages/shared/lib');
         await sandbox.remove('packages/client/lib');
         break;

test/e2e/fixtures/environment/typescript-monorepo.fixture

@@ -23,7 +23,7 @@
   }
 }
 
-/// tsconfig.json
+/// tsconfig.base.json
 {
   "compilerOptions": {
     "target": "es5",
@@ -42,7 +42,12 @@
     "declarationMap": true,
     "sourceMap": true,
     "rootDir": "./packages"
-  },
+  }
+}
+
+/// tsconfig.json
+{
+  "extends": "./tsconfig.base.json",
   "files": [],
   "references": [
     { "path": "./packages/shared" },

test/e2e/fixtures/implementation/typescript-monorepo.fixture

@@ -12,15 +12,15 @@
 
 /// packages/shared/tsconfig.json
 {
-  "extends": "../../tsconfig",
+  "extends": "../../tsconfig.base.json",
   "compilerOptions": {
     "rootDir": "./src",
     "outDir": "./lib",
+    "tsBuildInfoFile": "lib/tsconfig.tsbuildinfo",
     "sourceRoot": "./src",
     "baseUrl": "./src"
   },
-  "include": ["src"],
-  "exclude": ["node_modules", "lib"]
+  "include": ["src"]
 }
 
 /// packages/shared/src/intersect.ts
@@ -63,16 +63,16 @@ export {
 
 /// packages/client/tsconfig.json
 {
-  "extends": "../../tsconfig",
+  "extends": "../../tsconfig.base.json",
   "compilerOptions": {
     "rootDir": "./src",
     "outDir": "./lib",
+    "tsBuildInfoFile": "lib/tsconfig.tsbuildinfo",
     "sourceRoot": "./src",
     "baseUrl": "./src"
   },
   "references": [{ "path": "../shared" }],
-  "include": ["src"],
-  "exclude": ["node_modules", "lib"]
+  "include": ["src"]
 }