Merge pull request #89 from vinomanick/ignore-options-support

Author: rwjblue

src/bin-support.js

@@ -10,7 +10,7 @@ async function runJsTransform(root, transformName, args, extensions = DEFAULT_JS
   const { parseTransformArgs } = require('./options-support');
   const { getTransformPath } = require('./transform-support');
 
-  let { paths, options } = parseTransformArgs(args);
+  let { paths, options, transformerOptions } = parseTransformArgs(args);
 
   try {
     let foundPaths = await globby(paths, {
@@ -22,7 +22,14 @@ async function runJsTransform(root, transformName, args, extensions = DEFAULT_JS
     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,
+      ...transformerOptions,
+      ...foundPaths,
+    ];
 
     return execa(binPath, binOptions, {
       stdio: 'inherit',

src/options-support.js

@@ -1,15 +1,31 @@
 const yargs = require('yargs');
 
-function parseTransformArgs(args) {
+const jsCodeShiftOptions = ['ignore-config', 'ignore-pattern'];
+
+function parseTransformArgs(args, codeShiftOptions = jsCodeShiftOptions) {
   let parsedArgs = yargs.parse(args);
   let paths = parsedArgs._;
-  let options = Object.keys(parsedArgs).reduce((acc, key) => {
-    if (!['_', '$0', 'help', 'version'].includes(key)) {
-      acc[key] = parsedArgs[key];
+  let { options, transformerOptions } = Object.keys(parsedArgs).reduce(
+    (acc, key) => {
+      if (!['_', '$0', 'help', 'version'].includes(key)) {
+        let KeyInkebabCase = key.replace(
+          /([a-z])([A-Z])/g,
+          (_, $1, $2) => `${$1}-${$2.toLowerCase()}`
+        );
+        if (codeShiftOptions.includes(key)) {
+          acc.transformerOptions.push(`--${key}`, parsedArgs[key]);
+        } else if (!codeShiftOptions.includes(KeyInkebabCase)) {
+          acc.options[key] = parsedArgs[key];
+        }
+      }
+      return acc;
+    },
+    {
+      options: {},
+      transformerOptions: [],
     }
-    return acc;
-  }, {});
-  return { paths, options };
+  );
+  return { paths, options, transformerOptions };
 }
 
 function getOptions() {

tests/cli-test.js

@@ -485,6 +485,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: {