Fix SolutionBuilder watches

src/after-compile.ts

@@ -11,6 +11,7 @@ import {
   WebpackError,
   WebpackModule
 } from './interfaces';
+import { getSolutionErrors } from './servicesHost';
 import {
   collectAllDependants,
   ensureProgram,
@@ -20,7 +21,8 @@ import {
 
 export function makeAfterCompile(
   instance: TSInstance,
-  configFilePath: string | undefined
+  configFilePath: string | undefined,
+  loaderContext: webpack.loader.LoaderContext
 ) {
   let getCompilerOptionDiagnostics = true;
   let checkAllFilesForErrors = true;
@@ -65,9 +67,14 @@ export function makeAfterCompile(
     provideDeclarationFilesToWebpack(
       filesToCheckForErrors,
       instance,
-      compilation
+      compilation,
+      loaderContext
     );
 
+    // append errors
+    compilation.errors.push(
+      ...getSolutionErrors(instance, compilation.compiler.context)
+    );
     instance.filesWithErrors = filesWithErrors;
     instance.modifiedFiles = null;
     instance.projectsMissingSourceMaps = new Set();
@@ -195,7 +202,6 @@ function provideErrorsToWebpack(
     }
 
     const sourceFile = program && program.getSourceFile(filePath);
-
     // If the source file is undefined, that probably means it’s actually part of an unbuilt project reference,
     // which will have already produced a more useful error than the one we would get by proceeding here.
     // If it’s undefined and we’re not using project references at all, I guess carry on so the user will
@@ -262,14 +268,15 @@ function provideErrorsToWebpack(
 function provideDeclarationFilesToWebpack(
   filesToCheckForErrors: TSFiles,
   instance: TSInstance,
-  compilation: webpack.compilation.Compilation
+  compilation: webpack.compilation.Compilation,
+  loaderContext: webpack.loader.LoaderContext
 ) {
   for (const filePath of filesToCheckForErrors.keys()) {
     if (filePath.match(constants.tsTsxRegex) === null) {
       continue;
     }
 
-    const outputFiles = getEmitOutput(instance, filePath);
+    const outputFiles = getEmitOutput(instance, filePath, loaderContext);
     const declarationFiles = outputFiles.filter(outputFile =>
       outputFile.name.match(constants.dtsDtsxOrDtsDtsxMapRegex)
     );

src/index.ts

@@ -4,7 +4,11 @@ import * as typescript from 'typescript';
 import * as webpack from 'webpack';
 
 import * as constants from './constants';
-import { getEmitOutput, getTypeScriptInstance } from './instances';
+import {
+  getEmitOutput,
+  getTypeScriptInstance,
+  isReferencedFile
+} from './instances';
 import {
   LoaderOptions,
   LoaderOptionsCache,
@@ -343,19 +347,33 @@ function updateFileInCache(
   if (file === undefined) {
     file = instance.otherFiles.get(filePath);
     if (file !== undefined) {
-      instance.otherFiles.delete(filePath);
-      instance.files.set(filePath, file);
+      if (!isReferencedFile(instance, filePath)) {
+        instance.otherFiles.delete(filePath);
+        instance.files.set(filePath, file);
+        instance.changedFilesList = true;
+      }
     } else {
-      if (instance.watchHost !== undefined) {
+      if (
+        instance.watchHost !== undefined ||
+        instance.solutionBuilderHost !== undefined
+      ) {
         fileWatcherEventKind = instance.compiler.FileWatcherEventKind.Created;
       }
       file = { version: 0 };
-      instance.files.set(filePath, file);
+      if (!isReferencedFile(instance, filePath)) {
+        instance.files.set(filePath, file);
+        instance.changedFilesList = true;
+      } else {
+        instance.otherFiles.set(filePath, file);
+      }
     }
-    instance.changedFilesList = true;
   }
 
-  if (instance.watchHost !== undefined && contents === undefined) {
+  if (
+    (instance.watchHost !== undefined ||
+      instance.solutionBuilderHost !== undefined) &&
+    contents === undefined
+  ) {
     fileWatcherEventKind = instance.compiler.FileWatcherEventKind.Deleted;
   }
 
@@ -366,6 +384,7 @@ function updateFileInCache(
   //
   // See https://github.com/TypeStrong/ts-loader/issues/943
   if (
+    !isReferencedFile(instance, filePath) &&
     !instance.rootFileNames.has(filePath) &&
     // however, be careful not to add files from node_modules unless
     // it is allowed by the options.
@@ -380,7 +399,8 @@ function updateFileInCache(
     file.text = contents;
     instance.version!++;
     if (
-      instance.watchHost !== undefined &&
+      (instance.watchHost !== undefined ||
+        instance.solutionBuilderHost !== undefined) &&
       fileWatcherEventKind === undefined
     ) {
       fileWatcherEventKind = instance.compiler.FileWatcherEventKind.Changed;
@@ -393,6 +413,20 @@ function updateFileInCache(
     instance.watchHost.invokeDirectoryWatcher(path.dirname(filePath), filePath);
   }
 
+  if (
+    instance.solutionBuilderHost !== undefined &&
+    fileWatcherEventKind !== undefined
+  ) {
+    instance.solutionBuilderHost.invokeFileWatcher(
+      filePath,
+      fileWatcherEventKind
+    );
+    instance.solutionBuilderHost.invokeDirectoryWatcher(
+      path.dirname(filePath),
+      filePath
+    );
+  }
+
   // push this file to modified files hash.
   if (instance.modifiedFiles === null || instance.modifiedFiles === undefined) {
     instance.modifiedFiles = new Map<string, TSFile>();
@@ -408,52 +442,54 @@ function getEmit(
   instance: TSInstance,
   loaderContext: webpack.loader.LoaderContext
 ) {
-  const outputFiles = getEmitOutput(instance, filePath);
+  const outputFiles = getEmitOutput(instance, filePath, loaderContext);
 
-  loaderContext.clearDependencies();
-  loaderContext.addDependency(rawFilePath);
+  if (!isReferencedFile(instance, filePath)) {
+    loaderContext.clearDependencies();
+    loaderContext.addDependency(rawFilePath);
 
-  const allDefinitionFiles = [...instance.files.keys()].filter(defFilePath =>
-    defFilePath.match(constants.dtsDtsxOrDtsDtsxMapRegex)
-  );
+    const allDefinitionFiles = [...instance.files.keys()].filter(defFilePath =>
+      defFilePath.match(constants.dtsDtsxOrDtsDtsxMapRegex)
+    );
 
-  // Make this file dependent on *all* definition files in the program
-  const addDependency = loaderContext.addDependency.bind(loaderContext);
-  allDefinitionFiles.forEach(addDependency);
-
-  // Additionally make this file dependent on all imported files
-  const fileDependencies = instance.dependencyGraph[filePath];
-  const additionalDependencies =
-    fileDependencies === undefined
-      ? []
-      : fileDependencies.map(({ resolvedFileName, originalFileName }) => {
-          const projectReference = getAndCacheProjectReference(
-            resolvedFileName,
-            instance
-          );
-          // In the case of dependencies that are part of a project reference,
-          // the real dependency that webpack should watch is the JS output file.
-          return projectReference !== undefined
-            ? getAndCacheOutputJSFileName(
-                resolvedFileName,
-                projectReference,
-                instance
-              )
-            : originalFileName;
-        });
-
-  if (additionalDependencies.length > 0) {
-    additionalDependencies.forEach(addDependency);
-  }
+    // Make this file dependent on *all* definition files in the program
+    const addDependency = loaderContext.addDependency.bind(loaderContext);
+    allDefinitionFiles.forEach(addDependency);
+
+    // Additionally make this file dependent on all imported files
+    const fileDependencies = instance.dependencyGraph[filePath];
+    const additionalDependencies =
+      fileDependencies === undefined
+        ? []
+        : fileDependencies.map(({ resolvedFileName, originalFileName }) => {
+            const projectReference = getAndCacheProjectReference(
+              resolvedFileName,
+              instance
+            );
+            // In the case of dependencies that are part of a project reference,
+            // the real dependency that webpack should watch is the JS output file.
+            return projectReference !== undefined
+              ? getAndCacheOutputJSFileName(
+                  resolvedFileName,
+                  projectReference,
+                  instance
+                )
+              : originalFileName;
+          });
+
+    if (additionalDependencies.length > 0) {
+      additionalDependencies.forEach(addDependency);
+    }
 
-  loaderContext._module.buildMeta.tsLoaderDefinitionFileVersions = allDefinitionFiles
-    .concat(additionalDependencies)
-    .map(
-      defFilePath =>
-        defFilePath +
-        '@' +
-        (instance.files.get(defFilePath) || { version: '?' }).version
-    );
+    loaderContext._module.buildMeta.tsLoaderDefinitionFileVersions = allDefinitionFiles
+      .concat(additionalDependencies)
+      .map(
+        defFilePath =>
+          defFilePath +
+          '@' +
+          (instance.files.get(defFilePath) || { version: '?' }).version
+      );
+  }
 
   const outputFile = outputFiles
     .filter(file => file.name.match(constants.jsJsx))
@@ -490,7 +526,10 @@ function getTranspilationEmit(
   });
 
   // _module.errors is not available inside happypack - see https://github.com/TypeStrong/ts-loader/issues/336
-  if (!instance.loaderOptions.happyPackMode) {
+  if (
+    !instance.loaderOptions.happyPackMode &&
+    !isReferencedFile(instance, fileName)
+  ) {
     const errors = formatErrors(
       diagnostics,
       instance.loaderOptions,