Fix #11: Add ignore-pattern option, ignore node_modules by default

Author: micschro

README.md

@@ -48,6 +48,7 @@ This is useful if you are sure that the code is already correct.
 | --- | --- | --- | 
 | `-i`  | `--incremental` | Run the migration tool in [incremental mode](#incremental-mode). |
 | `-e`  | `--extensions`  | Which file extensions to transform. Default: `.js` and `.ts`|
+|       | `--ignore-pattern` | Ignore files that match the provided glob expression (default: `**/node_modules/**`).
 | `-l`  | `--singleline`  | Write log messages in a single line instead of printing the source location to a second line. |
 | `-v`  | `--verbose`  | Log verbose jscodeshift messages |
 |       | `--force`       | Overwrite files in the destination directory.

src/cli.js

@@ -46,6 +46,12 @@ const argv = require('yargs')
     choices: ['js', 'ts'],
     default: ['js', 'ts']
   })
+  .option('ignore-pattern', {
+    group: optionalGroup,
+    description: 'Ignore files that match the provided glob expression',
+    type: 'string',
+    default: '**/node_modules/**'
+  })
   .option('singleline', {
     group: optionalGroup,
     alias: 'l',

src/index.js

@@ -16,7 +16,8 @@ module.exports = async function migrate({
   extensions,
   incremental,
   force,
-  verbose
+  verbose,
+  ignorePattern
 }) {
   const srcExists = fs.existsSync(src)
   const destExists = fs.existsSync(dest)
@@ -65,6 +66,8 @@ module.exports = async function migrate({
     '--parser=ts',
     '--transforms',
     transforms.join(','),
+    '--ignore-pattern',
+    ignorePattern,
     dest
   ]
   if (singleline) {

test/test-runner.js

@@ -32,7 +32,8 @@ module.exports = function prepareTests(dir, options = {}) {
       transforms: config.transforms,
       extensions: config.extensions,
       incremental: config.incremental,
-      verbose: config.verbose
+      verbose: config.verbose,
+      ignorePattern: config.ignorePattern
     })
   }
 

transforms/master-transform.ts

@@ -105,7 +105,7 @@ export default function transformer(file, api, options: Options) {
         debug(`Skipping external transform '${name}' due to incremental mode`)
         continue
       }
-      if (!stringSource) {
+      if (typeof stringSource !== 'string') {
         stringSource = sources.toSource()
         sources = null
       }
@@ -138,7 +138,7 @@ export default function transformer(file, api, options: Options) {
     }
   }
 
-  if (!stringSource) {
+  if (typeof stringSource !== 'string') {
     stringSource = sources.toSource()
   }