Handle closed channels and upgrade dependencies

Author: piotr-oles

CHANGELOG.md

@@ -0,0 +1,12 @@
+## v0.1.3
+ * Fix Cannot read property 'mtime' of undefined on OSX
+
+## v0.1.2
+ * Workers mode works correctly (fixed typo)
+
+## v0.1.1
+ * Support memory limit in multi-process mode
+ * Handle already closed channel case on sending ipc message
+
+## v0.1.0
+ * Initial release - not production ready.

README.md

@@ -94,6 +94,7 @@ This plugin provides some custom webpack hooks (all are sync):
 |`fork-ts-checker-cancel`| Cancellation has been requested | `cancellationToken` |
 |`fork-ts-checker-waiting`| Waiting for results | `hasTsLint` |
 |`fork-ts-checker-service-start`| Service will be started | `tsconfigPath`, `tslintPath`, `watchPaths`, `workersNumber`, `memoryLimit` |
+|`fork-ts-checker-service-start-error` | Cannot start service | `error` |
 |`fork-ts-checker-service-out-of-memory`| Service is out of memory | - |
 |`fork-ts-checker-receive`| Plugin receives diagnostics and lints from service | `diagnostics`, `lints` | 
 |`fork-ts-checker-emit`| Service will add errors and warnings to webpack compilation (`blockEmit: true`) | `diagnostics`, `lints`, `elapsed` |

lib/cluster.js

@@ -29,7 +29,12 @@ var result = new WorkResult(pids);
 process.on('message', function (message) {
   // broadcast message to all workers
   workers.forEach(function (worker) {
-    worker.send(message);
+    try {
+      worker.send(message);
+    } catch (e) {
+      // channel closed - something went wrong - close cluster...
+      process.exit();
+    }
   });
 
   // clear previous result set

lib/index.js

@@ -156,7 +156,16 @@ ForkTsCheckerWebpackPlugin.prototype.pluginCompile = function () {
     if (!this.service || !this.service.connected) {
       this.spawnService();
     }
-    this.service.send(this.cancellationToken);
+
+    try {
+      this.service.send(this.cancellationToken);
+    } catch (error) {
+      if (!this.options.silent && this.logger) {
+        this.logger.error(this.colors.red('Cannot start checker service: ' + (error ? error.toString() : 'Unknown error')));
+      }
+
+      this.compiler.applyPlugins('fork-ts-checker-service-start-error', error);
+    }
   }.bind(this));
 };
 

package.json

@@ -34,11 +34,11 @@
     "chai": "^3.5.0",
     "eslint": "^3.19.0",
     "istanbul": "^0.4.5",
-    "mocha": "^3.2.0",
-    "mock-fs": "^4.2.0",
+    "mocha": "^3.4.1",
+    "mock-fs": "^4.3.0",
     "mock-require": "^2.0.2",
     "rimraf": "^2.5.4",
-    "sinon": "^2.1.0",
+    "sinon": "^2.2.0",
     "typescript": "^2.1.0"
   },
   "peerDependencies": {
@@ -47,12 +47,9 @@
   },
   "dependencies": {
     "chalk": "^1.1.3",
-    "chokidar": "^1.6.1",
+    "chokidar": "^1.7.0",
     "lodash.endswith": "^4.2.1",
     "lodash.isstring": "^4.0.1",
     "lodash.startswith": "^4.2.1"
-  },
-  "optionalDependencies": {
-    "tslint": "^5.0.0"
   }
 }

yarn.lock

@@ -249,9 +249,9 @@ chalk@^1.0.0, chalk@^1.1.0, chalk@^1.1.1, chalk@^1.1.3:
     strip-ansi "^3.0.0"
     supports-color "^2.0.0"
 
-chokidar@^1.6.1:
-  version "1.6.1"
-  resolved "https://registry.yarnpkg.com/chokidar/-/chokidar-1.6.1.tgz#2f4447ab5e96e50fb3d789fd90d4c72e0e4c70c2"
+chokidar@^1.7.0:
+  version "1.7.0"
+  resolved "https://registry.yarnpkg.com/chokidar/-/chokidar-1.7.0.tgz#798e689778151c8076b4b360e5edd28cda2bb468"
   dependencies:
     anymatch "^1.3.0"
     async-each "^1.0.0"
@@ -1232,9 +1232,9 @@ [email protected], [email protected], "mkdirp@>=0.5 0", mkdirp@^0.5.0, mkdirp@^0.5.1:
   dependencies:
     minimist "0.0.8"
 
-mocha@^3.2.0:
-  version "3.3.0"
-  resolved "https://registry.yarnpkg.com/mocha/-/mocha-3.3.0.tgz#d29b7428d3f52c82e2e65df1ecb7064e1aabbfb5"
+mocha@^3.4.1:
+  version "3.4.1"
+  resolved "https://registry.yarnpkg.com/mocha/-/mocha-3.4.1.tgz#a3802b4aa381934cacb38de70cf771621da8f9af"
   dependencies:
     browser-stdout "1.3.0"
     commander "2.9.0"
@@ -1248,9 +1248,9 @@ mocha@^3.2.0:
     mkdirp "0.5.1"
     supports-color "3.1.2"
 
-mock-fs@^4.2.0:
-  version "4.2.0"
-  resolved "https://registry.yarnpkg.com/mock-fs/-/mock-fs-4.2.0.tgz#ef53ae17b77e64f67816dd0467f29208a3b26e19"
+mock-fs@^4.3.0:
+  version "4.3.0"
+  resolved "https://registry.yarnpkg.com/mock-fs/-/mock-fs-4.3.0.tgz#c2fab8d784283287e9b6ae7538f2dc56c1a05ed7"
 
 mock-require@^2.0.2:
   version "2.0.2"
@@ -1638,9 +1638,9 @@ signal-exit@^3.0.0:
   version "3.0.2"
   resolved "https://registry.yarnpkg.com/signal-exit/-/signal-exit-3.0.2.tgz#b5fdc08f1287ea1178628e415e25132b73646c6d"
 
-sinon@^2.1.0:
-  version "2.1.0"
-  resolved "https://registry.yarnpkg.com/sinon/-/sinon-2.1.0.tgz#e057a9d2bf1b32f5d6dd62628ca9ee3961b0cafb"
+sinon@^2.2.0:
+  version "2.2.0"
+  resolved "https://registry.yarnpkg.com/sinon/-/sinon-2.2.0.tgz#3b1b42ff5defcbf51a52a62aca6d61171b9fd262"
   dependencies:
     diff "^3.1.0"
     formatio "1.2.0"