Added support for jscodeshift ignore options

Author: vinomanick

src/bin-support.js

@@ -7,9 +7,10 @@ async function runTransform(binRoot, transformName, args, extensions = DEFAULT_E
   const execa = require('execa');
   const chalk = require('chalk');
   const path = require('path');
-  const { parseTransformArgs } = require('./options-support');
+  const { parseTransformArgs, extractJSCodeShiftOptions } = require('./options-support');
 
   let { paths, options } = parseTransformArgs(args);
+  let { cliOptions, jsCodeShiftOptions } = extractJSCodeShiftOptions(options);
 
   try {
     let foundPaths = await globby(paths, {
@@ -21,12 +22,12 @@ async function runTransform(binRoot, transformName, args, extensions = DEFAULT_E
     let jscodeshiftPath = path.dirname(require.resolve('jscodeshift/package'));
     let binPath = path.join(jscodeshiftPath, jscodeshiftPkg.bin.jscodeshift);
 
-    let binOptions = ['-t', transformPath, '--extensions', extensions, ...foundPaths];
+    let binOptions = ['-t', transformPath, '--extensions', extensions, ...jsCodeShiftOptions, ...foundPaths];
 
     return execa(binPath, binOptions, {
       stdio: 'inherit',
       env: {
-        CODEMOD_CLI_ARGS: JSON.stringify(options),
+        CODEMOD_CLI_ARGS: JSON.stringify(cliOptions),
       },
     });
   } catch (error) {

src/options-support.js

@@ -1,5 +1,7 @@
 const yargs = require('yargs');
 
+const jsCodeShiftOptions = ['ignore-config', 'ignore-pattern'];
+
 function parseTransformArgs(args) {
   let parsedArgs = yargs.parse(args);
   let paths = parsedArgs._;
@@ -20,4 +22,18 @@ function getOptions() {
   }
 }
 
-module.exports = { parseTransformArgs, getOptions };
+function extractJSCodeShiftOptions(options, codeShiftOptions = jsCodeShiftOptions) {
+  let cliOptions = Object.assign({}, options);
+  let jsCodeShiftOptions = [];
+  codeShiftOptions.forEach(option => {
+    if(cliOptions.hasOwnProperty(option)) {
+      let camelCaseOption = option.replace(/-([a-z])/g, (_, up) => up.toUpperCase());
+      jsCodeShiftOptions.push(`--${option}`, options[option]);
+      delete cliOptions[option];
+      delete cliOptions[camelCaseOption];
+    }
+  });
+  return { cliOptions, jsCodeShiftOptions };
+}
+
+module.exports = { parseTransformArgs, getOptions, extractJSCodeShiftOptions };

tests/cli-test.js

@@ -445,6 +445,70 @@ QUnit.module('codemod-cli', function(hooks) {
         });
       });
 
+      QUnit.test('should ignore patterns from configuration option', async function(assert) {
+        codemodProject.write({
+          transforms: {
+            main: {
+              'index.js': `
+                  module.exports = function transformer(file, api) {
+                    return file.source.toUpperCase();
+                  }
+              `,
+            },
+          },
+        });
+
+        userProject.write({
+          foo: { 'something.hbs': `<Foo />` },
+          bar: { 'something.hbs': `<Foo />` },
+        });
+
+        await CodemodCLI.runTransform(codemodProject.path('bin'), 'main', [
+          'foo/**',
+          'bar/**',
+          '--ignore-pattern',
+          'foo/'
+        ], 'hbs');
+
+        assert.deepEqual(userProject.read(), {
+          foo: { 'something.hbs': `<Foo />` },
+          bar: { 'something.hbs': `<FOO />` },
+        });
+      });
+
+      QUnit.test('should ignore patterns from configuration file', async function(assert) {
+        codemodProject.write({
+          transforms: {
+            main: {
+              'index.js': `
+                  module.exports = function transformer(file, api) {
+                    return file.source.toUpperCase();
+                  }
+              `,
+            },
+          },
+        });
+
+        userProject.write({
+          config: { '.gitignore': `foo/` },
+          foo: { 'something.hbs': `<Foo />` },
+          bar: { 'something.hbs': `<Foo />` },
+        });
+
+        await CodemodCLI.runTransform(codemodProject.path('bin'), 'main', [
+          'foo/**',
+          'bar/**',
+          '--ignore-config',
+          'config/.gitignore'
+        ], 'hbs');
+
+        assert.deepEqual(userProject.read(), {
+          config: { '.gitignore': `foo/` },
+          foo: { 'something.hbs': `<Foo />` },
+          bar: { 'something.hbs': `<FOO />` },
+        });
+      });
+
       QUnit.test('runs transform against class syntax', async function(assert) {
         userProject.write({
           foo: {