Fixed skyux test/watch performance

config/webpack/build.webpack.config.js

@@ -26,7 +26,8 @@ function getWebpackConfig(skyPagesConfig) {
                 // Ignore the "Cannot find module" error that occurs when referencing
                 // an aliased file.  Webpack will still throw an error when a module
                 // cannot be resolved via a file path or alias.
-                ignoreDiagnostics: [2307]
+                ignoreDiagnostics: [2307],
+                transpileOnly: true
               }
             },
             {

config/webpack/serve.webpack.config.js

@@ -13,8 +13,6 @@ const LoaderOptionsPlugin = require('webpack/lib/LoaderOptionsPlugin');
 const skyPagesConfigUtil = require('../sky-pages/sky-pages.config');
 const hostUtils = require('../../utils/host-utils');
 
-const moduleLoader = skyPagesConfigUtil.outPath('loader', 'sky-pages-module');
-
 /**
  * Returns the querystring base for parameters allowed to be passed through.
  * PLEASE NOTE: The method is nearly duplicated in `runtime/params.ts`.
@@ -120,11 +118,6 @@ function getWebpackConfig(argv, skyPagesConfig) {
     watch: true,
     module: {
       rules: [
-        {
-          enforce: 'pre',
-          test: /sky-pages\.module\.ts$/,
-          loader: moduleLoader
-        },
         {
           test: /\.ts$/,
           use: [
@@ -134,15 +127,17 @@ function getWebpackConfig(argv, skyPagesConfig) {
                 // Ignore the "Cannot find module" error that occurs when referencing
                 // an aliased file.  Webpack will still throw an error when a module
                 // cannot be resolved via a file path or alias.
-                ignoreDiagnostics: [2307]
+                ignoreDiagnostics: [2307],
+                transpileOnly: true
               }
             },
             {
               loader: 'angular2-template-loader'
             }
-          ]
+          ],
+          exclude: [/\.e2e\.ts$/]
         }
-      ],
+      ]
     },
     devServer: {
       compress: true,

config/webpack/test.webpack.config.js

@@ -1,31 +1,29 @@
 /*jslint node: true */
 'use strict';
 
-function getWebpackConfig(skyPagesConfig, argv) {
-
-  function spaPath() {
-    return skyPagesConfigUtil.spaPath.apply(skyPagesConfigUtil, arguments);
-  }
-
-  function outPath() {
-    return skyPagesConfigUtil.outPath.apply(skyPagesConfigUtil, arguments);
-  }
-
-  const path = require('path');
+const path = require('path');
+const DefinePlugin = require('webpack/lib/DefinePlugin');
+const LoaderOptionsPlugin = require('webpack/lib/LoaderOptionsPlugin');
+const ContextReplacementPlugin = require('webpack/lib/ContextReplacementPlugin');
+const ProcessExitCode = require('../../plugin/process-exit-code');
+const SkyTsLintCheckerPlugin = require('../../loader/sky-tslint/checker-plugin');
+const skyPagesConfigUtil = require('../sky-pages/sky-pages.config');
+const aliasBuilder = require('./alias-builder');
+
+function spaPath() {
+  return skyPagesConfigUtil.spaPath.apply(skyPagesConfigUtil, arguments);
+}
 
-  const DefinePlugin = require('webpack/lib/DefinePlugin');
-  const LoaderOptionsPlugin = require('webpack/lib/LoaderOptionsPlugin');
-  const ContextReplacementPlugin = require('webpack/lib/ContextReplacementPlugin');
-  const ProcessExitCode = require('../../plugin/process-exit-code');
-  const skyPagesConfigUtil = require('../sky-pages/sky-pages.config');
-  const aliasBuilder = require('./alias-builder');
+function outPath() {
+  return skyPagesConfigUtil.outPath.apply(skyPagesConfigUtil, arguments);
+}
 
+function getWebpackConfig(skyPagesConfig, argv) {
   const runCoverage = (!argv || argv.coverage !== false);
   skyPagesConfig.runtime.includeRouteModule = false;
 
   const ENV = process.env.ENV = process.env.NODE_ENV = 'test';
   const srcPath = path.resolve(process.cwd(), 'src', 'app');
-  const moduleLoader = outPath('loader', 'sky-pages-module');
 
   const resolves = [
     process.cwd(),
@@ -66,17 +64,7 @@ function getWebpackConfig(skyPagesConfig, argv) {
         {
           enforce: 'pre',
           test: /sky-pages\.module\.ts$/,
-          loader: moduleLoader
-        },
-        {
-          enforce: 'pre',
-          test: /\.ts$/,
-          loader: 'tslint-loader',
-          exclude: excludes,
-          options: {
-            emitErrors: true,
-            failOnHint: true
-          }
+          loader: outPath('loader', 'sky-pages-module')
         },
         {
           enforce: 'pre',
@@ -87,7 +75,13 @@ function getWebpackConfig(skyPagesConfig, argv) {
         {
           enforce: 'pre',
           loader: outPath('loader', 'sky-processor', 'preload'),
-          exclude: /node_modules/
+          exclude: excludes
+        },
+        {
+          enforce: 'pre',
+          test: /\.ts$/,
+          loader: outPath('loader', 'sky-tslint'),
+          exclude: excludes
         },
         {
           test: /\.ts$/,
@@ -98,7 +92,9 @@ function getWebpackConfig(skyPagesConfig, argv) {
                 // Ignore the "Cannot find module" error that occurs when referencing
                 // an aliased file.  Webpack will still throw an error when a module
                 // cannot be resolved via a file path or alias.
-                ignoreDiagnostics: [2307]
+                ignoreDiagnostics: [2307],
+                // Linting is handled by the sky-tslint loader.
+                transpileOnly: true
               }
             },
             {
@@ -108,19 +104,12 @@ function getWebpackConfig(skyPagesConfig, argv) {
           exclude: [/\.e2e\.ts$/]
         },
         {
-          test: /\.css$/,
-          loader: 'raw-loader'
+          test: /\.s?css$/,
+          use: ['raw-loader', 'sass-loader']
         },
         {
           test: /\.html$/,
           loader: 'raw-loader'
-        },
-        {
-          test: /\.scss$/,
-          use: [
-            'raw-loader',
-            'sass-loader'
-          ]
         }
       ]
     },
@@ -130,13 +119,7 @@ function getWebpackConfig(skyPagesConfig, argv) {
         debug: true,
         options: {
           context: __dirname,
-          skyPagesConfig: skyPagesConfig,
-          tslint: {
-            emitErrors: false,
-            failOnHint: false,
-            resourcePath: 'src',
-            typeCheck: true
-          }
+          skyPagesConfig: skyPagesConfig
         }
       }),
 
@@ -159,6 +142,9 @@ function getWebpackConfig(skyPagesConfig, argv) {
         {}
       ),
 
+      // Handles watch-mode maintenance for TSLint.
+      new SkyTsLintCheckerPlugin(),
+
       // Webpack 2 behavior does not correctly return non-zero exit code.
       new ProcessExitCode()
     ]

loader/sky-tslint/checker-plugin.js

@@ -0,0 +1,32 @@
+/*jshint node: true*/
+'use strict';
+
+function SkyTsLintCheckerPlugin() {
+  let isWatchMode = false;
+
+  const apply = (compiler) => {
+    compiler.plugin('run', function (params, callback) {
+      isWatchMode = false;
+      callback();
+    });
+
+    compiler.plugin('watch-run', function (params, callback) {
+      isWatchMode = true;
+      callback();
+    });
+
+    compiler.plugin('emit', function (compilation, callback) {
+      if (isWatchMode === true) {
+        require('./program').clearProgram();
+      }
+
+      callback();
+    });
+  };
+
+  return {
+    apply
+  };
+}
+
+module.exports = SkyTsLintCheckerPlugin;

loader/sky-tslint/index.js

@@ -0,0 +1,34 @@
+const tslint = require('tslint');
+const skyPagesConfigUtil = require('../../config/sky-pages/sky-pages.config');
+const skyTsLintProgram = require('./program');
+
+const lint = (instance, input) => {
+  const options = {
+    fix: false,
+    typeCheck: true
+  };
+
+  const program = skyTsLintProgram.getProgram();
+  const linter = new tslint.Linter(options, program);
+  const tslintConfig = skyPagesConfigUtil.spaPath('tslint.json');
+  const configuration = tslint.Configuration.findConfiguration(tslintConfig).results;
+
+  linter.lint(instance.resourcePath, input, configuration);
+  const result = linter.getResult();
+
+  if (result.failures.length) {
+    return new Error(`Compilation failed due to tslint errors. ${result.output}`);
+  }
+
+  return;
+};
+
+module.exports = function (input, map) {
+  const instance = this;
+  const callback = instance.async();
+
+  instance.cacheable && instance.cacheable();
+
+  let error = lint(instance, input);
+  callback(error, input, map);
+};

loader/sky-tslint/program.js

@@ -0,0 +1,25 @@
+/*jshint node: true*/
+'use strict';
+
+const tslint = require('tslint');
+const skyPagesConfigUtil = require('../../config/sky-pages/sky-pages.config');
+
+const tsconfigPath = skyPagesConfigUtil.spaPath('tsconfig.json');
+let _context = {};
+
+const getProgram = () => {
+  if (!_context.tslintProgram) {
+    _context.tslintProgram = tslint.Linter.createProgram(tsconfigPath);
+  }
+
+  return _context.tslintProgram;
+};
+
+const clearProgram = () => {
+  delete _context.tslintProgram;
+};
+
+module.exports = {
+  getProgram,
+  clearProgram
+};

package.json

@@ -29,7 +29,7 @@
   },
   "homepage": "https://github.com/blackbaud/skyux-builder#readme",
   "peerDependencies": {
-    "@blackbaud/skyux": "2.0.0-rc.1"
+    "@blackbaud/skyux": "2.0.0-rc.2"
   },
   "dependencies": {
     "@angular/common": "4.1.3",
@@ -86,7 +86,6 @@
     "ts-helpers": "1.1.2",
     "ts-node": "3.0.4",
     "tslint": "5.2.0",
-    "tslint-loader": "3.5.3",
     "typescript": "2.3.2",
     "webpack": "2.5.1",
     "webpack-dev-server": "2.4.5",