fix handing require extensions

Author: wclr

CHANGELOG.md

@@ -1,5 +1,9 @@
 ### Yalc changelog
 
+## 1.0.0-pre.59 (2020-08-20)
+
+- fix handing require extensions (#185, #196)
+
 ## 1.0.0-pre.58 (2020-08-18)
 
 - show versions only on first start

lib/compiler.js

@@ -21,7 +21,7 @@ var compilationInstanceStamp = Math.random().toString().slice(2)
 
 var originalJsHandler = require.extensions['.js']
 
-var extHandlers = {}
+// var extHandlers = {}
 
 function hasOwnProperty (object, property) {
   return Object.prototype.hasOwnProperty.call(object, property)
@@ -186,7 +186,9 @@ var compiler = {
     /* clean up compiled on each new init*/
     rimraf.sync(compiler.getCompiledDir())
     compiler.createCompiledDir()
-    /* check if `allowJs` compiler option enable */
+    
+    // check if `allowJs` compiler option enable
+    // (if it was changed after ts-node registration)
     var allowJsEnabled = require.extensions['.js'] !== originalJsHandler
     if (allowJsEnabled) {
       compiler.allowJs = true
@@ -198,27 +200,12 @@ var compiler = {
   },
   registerTsNode: function () {
     var options = compiler.options
+    // revert back original handler extensions
+    // in case of re-registration after an error
     extensions.forEach(function (ext) {
       require.extensions[ext] = originalJsHandler
     })
 
-    // var tsNodeOptions = {
-    //   //dir: should add
-    //   emit: options['emit'],
-    //   files: options['files'],
-    //   pretty: options['pretty'],
-    //   transpileOnly: options['transpile-only'],
-    //   ignore: [].concat(options['ignore']),
-    //   preferTsExts: options['prefer-ts-exts'] || options['prefer-ts'],
-    //   logError: options['log-error'],
-    //   project: options['project'],
-    //   skipProject: options['skip-project'],
-    //   skipIgnore: options['skip-ignore'],
-    //   compiler: options['compiler'],
-    //   ignoreDiagnostics: options['ignore-diagnostics'],
-    //   disableWarnings: options['disableWarnings'],
-    //   compilerOptions: options['compiler-options'],
-    // }
     var compilerOptionsArg =
       options['compilerOptions'] || options['compiler-options']
     var compilerOptions
@@ -271,10 +258,6 @@ var compiler = {
       console.log(e)
       return
     }
-    extensions.forEach(function (ext) {
-      extHandlers[ext] = require.extensions[ext]
-      require.extensions[ext] = originalJsHandler
-    })
   },
   compileChanged: function (fileName) {
     var ext = path.extname(fileName)
@@ -311,7 +294,7 @@ var compiler = {
     }
     const _compile = () => {
       var ext = path.extname(fileName)
-      var extHandler = extHandlers[ext] || require.extensions[ext]
+      var extHandler = require.extensions[ext]
       extHandler(m, fileName)
       m._compile(code, fileName)
       compiler.log.debug(
@@ -331,7 +314,7 @@ var compiler = {
 
       // reinitialize ts-node compilation to clean up state after error
       // without timeout in causes cases error not be printed out
-      setTimeout(() => {
+      setTimeout(() => {        
         compiler.registerTsNode()
       })
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ts-node-dev",
-  "version": "1.0.0-pre.58",
+  "version": "1.0.0-pre.59",
   "description": "Compiles your TS app and restarts when files are modified.",
   "keywords": [
     "restart",
@@ -36,7 +36,7 @@
     "ts-node-dev": "node ./bin/ts-node-dev",
     "tsnd": "yarn ts-node-dev",
     "test": "ts-node -T test/index.ts",
-    "test-dev": "ts-node-dev -T --respawn test/index.ts --output",
+    "test-dev": "ts-node-dev -T --respawn --deps --watch lib test/index.ts --output",
     "test-docker": "docker-compose up",
     "ci": "yarn test",
     "ci-local": "docker run --name travis-debug -dit quay.io/travisci/ci-nodejs",

test/index.ts

@@ -49,7 +49,7 @@ test('It should restart on file change', async (t) => {
 
 test('It allow watch arbitrary folder/file', async (t) => {
   const ps = spawnTsNodeDev('--respawn --watch folder,folder2 simple.ts')
-  await ps.waitForLine(/Using/)
+  await ps.waitForLine(/v1/)  
   setTimeout(() => touch(join(scriptsDir, 'folder/some-file')), 250)
   await ps.waitForLine(/Restarting.*some-file/)
   t.pass('works')