fix(core): rework where tmp tsconfigs are generated (#2805)

nrwl · Apr 16, 2020 · 2fc5ce2 · 2fc5ce2
1 parent 1f59ce4
commit 2fc5ce2
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 13 deletions.
diff --git a/packages/node/src/builders/package/package.impl.spec.ts b/packages/node/src/builders/package/package.impl.spec.ts
@@ -284,7 +284,12 @@ describe('NodeCompileBuilder', () => {
     });
 
     it('should call the tsc compiler with the modified tsconfig.json', done => {
-      let tmpTsConfigPath = join('libs/nodelib', 'tsconfig.nx-tmp');
+      let tmpTsConfigPath = join(
+        '/root',
+        'tmp',
+        'libs/nodelib',
+        'tsconfig.generated.json'
+      );
 
       runNodePackageBuilder(testOptions, context).subscribe({
         complete: () => {

diff --git a/packages/web/src/builders/package/package.impl.ts b/packages/web/src/builders/package/package.impl.ts
@@ -32,7 +32,7 @@ import {
   calculateProjectDependencies,
   checkDependentProjectsHaveBeenBuilt,
   DependentBuildableProjectNode,
-  readTsConfigWithRemappedPaths,
+  computeCompilerOptionsPaths,
   updateBuildableProjectPackageJsonDependencies
 } from '@nrwl/workspace/src/utils/buildable-libs-utils';
 
@@ -147,7 +147,7 @@ function createRollupOptions(
   context: BuilderContext,
   packageJson: any
 ): rollup.InputOptions {
-  const parsedTSConfig = readTsConfigWithRemappedPaths(
+  const compilerOptionPaths = computeCompilerOptionsPaths(
     options.tsConfig,
     dependencies
   );
@@ -162,7 +162,7 @@ function createRollupOptions(
           rootDir: options.entryRoot,
           allowJs: false,
           declaration: true,
-          paths: parsedTSConfig.compilerOptions.paths
+          paths: compilerOptionPaths
         }
       }
     }),

diff --git a/packages/workspace/src/utils/buildable-libs-utils.ts b/packages/workspace/src/utils/buildable-libs-utils.ts
@@ -4,7 +4,7 @@ import {
   ProjectType
 } from '../core/project-graph';
 import { BuilderContext } from '@angular-devkit/architect';
-import { join, resolve, dirname } from 'path';
+import { join, resolve, dirname, relative } from 'path';
 import {
   fileExists,
   readJsonFile,
@@ -81,15 +81,27 @@ function recursivelyCollectDependencies(
   return acc;
 }
 
-export function readTsConfigWithRemappedPaths(
+function readTsConfigWithRemappedPaths(
   tsConfig: string,
+  generatedTsConfigPath: string,
   dependencies: DependentBuildableProjectNode[]
 ) {
-  const parsedTSConfig = ts.readConfigFile(tsConfig, ts.sys.readFile).config;
-  parsedTSConfig.compilerOptions = parsedTSConfig.compilerOptions || {};
-  parsedTSConfig.compilerOptions.paths = readPaths(tsConfig) || {};
-  updatePaths(dependencies, parsedTSConfig.compilerOptions.paths);
-  return parsedTSConfig;
+  const generatedTsConfig: any = { compilerOptions: {} };
+  generatedTsConfig.extends = relative(
+    dirname(generatedTsConfigPath),
+    tsConfig
+  );
+  generatedTsConfig.compilerOptions.paths = computeCompilerOptionsPaths(
+    tsConfig,
+    dependencies
+  );
+  return generatedTsConfig;
+}
+
+export function computeCompilerOptionsPaths(tsConfig, dependencies) {
+  const paths = readPaths(tsConfig) || {};
+  updatePaths(dependencies, paths);
+  return paths;
 }
 
 function readPaths(tsConfig: string) {
@@ -116,11 +128,17 @@ export function createTmpTsConfig(
   projectRoot: string,
   dependencies: DependentBuildableProjectNode[]
 ) {
+  const tmpTsConfigPath = join(
+    workspaceRoot,
+    'tmp',
+    projectRoot,
+    'tsconfig.generated.json'
+  );
   const parsedTSConfig = readTsConfigWithRemappedPaths(
     tsconfigPath,
+    tmpTsConfigPath,
     dependencies
   );
-  const tmpTsConfigPath = join(workspaceRoot, projectRoot, 'tsconfig.nx-tmp');
   process.on('exit', () => {
     cleanupTmpTsConfigFile(tmpTsConfigPath);
   });
@@ -133,7 +151,7 @@ export function createTmpTsConfig(
     process.exit(0);
   });
   writeJsonFile(tmpTsConfigPath, parsedTSConfig);
-  return join(projectRoot, 'tsconfig.nx-tmp');
+  return join(tmpTsConfigPath);
 }
 
 function cleanupTmpTsConfigFile(tmpTsConfigPath) {