fix big files compilation

Author: wclr

README.md

@@ -29,15 +29,13 @@ ts-node-dev --respawn server.ts
 
 Also there is additional options specific to `ts-node-dev`:
 
-- `--compile-timeout` (default: 10000 ms) - for how long to wait before report the error that something went wrong with compilation of a file.
 - `--prefer-ts` (default: false) - for each `.js` file (that is not in `node_modules`) will try to check if corresponding `.ts` version exists and require it.
 - `--ignore-watch` (default: []) - files/folders to be [ignored by `node-dev`](https://github.com/fgnass/node-dev#ignore-paths). 
 
-NB! `--ignore-watch` will NOT affect files ignored by TS compilation. Use `--ignore` option (or `TS_NODE_IGNORE`) to pass **regExp strings** (string values for `new RegExp(..)`) for filtering files that should not be compiled, by default `/node_modules/` are ignored by compiler.
-
 **But also this behaviour enhanced:** it will also make up `new RegExp` of passed ignore string and check absolute paths of required files for match. 
 So, to ignore everthing in `node_modules`, just pass `--ignore-watch node_modules`
 
+NB! `--ignore-watch` will NOT affect files ignored by TS compilation. Use `--ignore` option (or `TS_NODE_IGNORE`) to pass **RegExp strings** for filtering files that should not be compiled, by default `/node_modules/` are ignored.
 
 By defalut to keep things clean it puts cached files to system temp directory, you may change this with `--cache-directory` option.
 

lib/child-require-hook.js

@@ -7,22 +7,22 @@ var allowJs = false
 var compiledDir
 var preferTs = false
 var ignore = [/node_modules/]
+var readyFile
 
 var compile = (code, fileName) => {
   var compiledPath = getCompiledPath(code, fileName, compiledDir)
   process.send({
     compile: fileName,
-    code: code,
     compiledPath: compiledPath
   })
   var compiled
   var start = new Date().getTime()
-  var timeout = false
-  while (compiled === undefined || timeout) {    
+  var passed
+  while (compiled === undefined) {
     if (fs.existsSync(compiledPath + '.done')) {
       compiled = fs.readFileSync(compiledPath, 'utf-8')
     }
-    var passed = (new Date().getTime() - start)
+    passed = (new Date().getTime() - start)
     if (passed > timeThreshold) {
       throw new Error(
         'Could not require ' + fileName + ', compiled path:' + compiledPath
@@ -62,7 +62,9 @@ function registerJsExtension() {
         }
       }
       var _compile = m._compile
-      var isIgnored = ignore && ignore.reduce((res, ignore) => res || ignore.test(fileName), false)
+      var isIgnored = ignore && ignore.reduce(function (res, ignore) {
+        return res || ignore.test(fileName)
+      }, false)
       if (tsCode !== undefined || (allowJs && !isIgnored)) {
         m._compile = function (code, fileName) {
           if (tsCode !== undefined) {
@@ -80,4 +82,13 @@ function registerJsExtension() {
 registerExtensions(['.ts', '.tsx'])
 registerJsExtension()
 
+if (readyFile) {
+  var time = new Date().getTime()
+  while (!fs.existsSync(readyFile)) {
+    if (new Date().getTime() - time > 5000) {
+      throw new Error('Waiting ts-node-dev ready file failed')
+    }
+  }
+}
+
 module.exports.registerExtensions = registerExtensions

lib/compiler.js

@@ -23,10 +23,17 @@ var compiler = {
   getCompiledDir: function () {
     return path.join(tmpDir, 'compiled').replace(/\\/g, '/')
   },
+  getCompilerReadyFilePath: function () {
+    return path.join(os.tmpdir(), 'ts-node-dev-ready-' + comilationInstanceStampt)
+      .replace(/\\/g, '/')
+  },  
   getChildHookPath: function () {
     return path.join(os.tmpdir(), 'ts-node-dev-hook-' + comilationInstanceStampt + '.js')
       .replace(/\\/g, '/')
   },
+  writeReadyFile: function () {
+    var fileData = fs.writeFileSync(compiler.getCompilerReadyFilePath(), '')
+  },
   writeChildHookFile: function (options) {
     var fileData = fs.readFileSync(path.join(__dirname, 'child-require-hook.js'), 'utf-8')
     var compileTimeout = parseInt(options['compile-timeout'])
@@ -44,11 +51,13 @@ var compiler = {
       var ignoreVal = !ignore || ignore === 'false'
         ? 'false'
         : '[' + (Array.isArray(ignore) ? ignore : ignore.split(/, /))
-          .map(ignore => 'new RegExp("' + ignore + '")').join(', ') + ']'      
+          .map(ignore => 'new RegExp("' + ignore + '")').join(', ') + ']'
       fileData = fileData.replace('var ignore = [/node_modules/]', 'var ignore = ' + ignoreVal)
     }
     fileData = fileData.replace('var compiledDir', 'var compiledDir = "' + compiler.getCompiledDir() + '"')
     fileData = fileData.replace('./get-compiled-path', path.join(__dirname, 'get-compiled-path').replace(/\\/g, '/'))
+    fileData = fileData.replace('var readyFile',
+      'var readyFile = "' + compiler.getCompilerReadyFilePath() + '"')
     fs.writeFileSync(compiler.getChildHookPath(), fileData)
   },
   init: function (options) {
@@ -73,7 +82,7 @@ var compiler = {
       ignoreWarnings: options['ignoreWarnings'],
       disableWarnings: options['disableWarnings'],
       compilerOptions: options['compilerOptions']
-    }
+    }    
     try {
       register(tsNodeOptions)
     } catch (e) {
@@ -110,8 +119,8 @@ var compiler = {
   },
   compile: function (params) {
     var fileName = params.compile
-    var code = params.code
-    var compiledPath = params.compiledPath
+    var code = fs.readFileSync(fileName, 'utf-8')
+    var compiledPath = params.compiledPath    
     function writeCompiled(code, filename) {
       fs.writeFileSync(compiledPath, code)
       fs.writeFileSync(compiledPath + '.done', '')

lib/index.js

@@ -70,20 +70,22 @@ module.exports = function (script, scriptArgs, nodeArgs, opts) {
       cwd: process.cwd(),
       env: process.env
     });
-    if (!compiler.tsConfigPath) {
-      throw new Error('Check existance of tsconfig.json file.')
-    }
-    watcher.add(compiler.tsConfigPath);
 
-    if (cfg.respawn) {
-      child.respawn = true;
-    }
     child.on('message', compiler.compile);
     child.on('exit', function (code) {
       if (!child.respawn) process.exit(code);
       child = undefined;
     });
-    
+
+    if (cfg.respawn) {
+      child.respawn = true;
+    }
+
+    if (!compiler.tsConfigPath) {
+      throw new Error('Check existance of tsconfig.json file.')
+    }
+    watcher.add(compiler.tsConfigPath);
+
     // Listen for `required` messages and watch the required file.        
     ipc.on(child, 'required', function (m) {
       var isIgnored = cfg.ignore.some(isPrefixOf(m.required)) || cfg.ignore.some(isRegExpMatch(m.required));
@@ -98,6 +100,7 @@ module.exports = function (script, scriptArgs, nodeArgs, opts) {
       notify(m.error, m.message, 'error');
       stop(m.willTerminate);
     });
+    compiler.writeReadyFile()
   }
 
   function stop(willTerminate) {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ts-node-dev",
-  "version": "1.0.0-pre.10",
+  "version": "1.0.0-pre.15",
   "description": "Compiles your TS app and restarts when files are modified.",
   "keywords": [
     "restart",
@@ -33,7 +33,8 @@
   },
   "scripts": {
     "test-node-dev": "tap test/*.js",
-    "test": "node ./bin/ts-node-dev -r ./test/ts/add-require.js --cache --respawn --ignore-watch 'lib' --ignore-watch bin --ignore '/some/' --ignore '/other/' --prefer-ts --cache-directory .ts-node test/ts/test-script test-arg --fd"
+    "test": "node ./bin/ts-node-dev -r ./test/ts/add-require.js --cache --respawn --ignore-watch 'lib' --ignore-watch bin --prefer-ts --cache-directory .ts-node test/ts/test-script test-arg --fd",
+    "test-docker": "docker run --rm -v ${PWD}:/app mhart/alpine-node:8.7.0 sh -c 'cd app && node ./bin/ts-node-dev --cache-directory .ts-node test/ts/big'"
   },
   "dependencies": {
     "ts-node": "*",