All transformations return AST to allow chaining

lib/transformations/bannerPlugin/bannerPlugin.js

@@ -19,6 +19,5 @@ module.exports = function(j, ast) {
 	return utils.findPluginsRootNodes(j, ast)
 		.forEach(path => {
 			utils.createOrUpdatePluginByName(j, path, 'webpack.BannerPlugin', loaderOptions);
-		})
-		.toSource({ quote: 'single' });
+		});
 };

lib/transformations/defineTest.js

@@ -53,7 +53,7 @@ function runTest(dirName, transformName, options, testFilePrefix) {
 		jscodeshift = jscodeshift.withParser(module.parser);
 	}
 	const ast = jscodeshift(source);
-	const output = transform(jscodeshift, ast);
+	const output = transform(jscodeshift, ast).toSource({ quote: 'single' });
 	expect(output).toMatchSnapshot();
 }
 

lib/transformations/extractTextPlugin/extractTextPlugin.js

@@ -2,14 +2,14 @@ const safeTraverse = require('../utils').safeTraverse;
 const findPluginsByName = require('../utils').findPluginsByName;
 
 module.exports = function(j, ast) {
-    const changeArguments = function(p) {
+	const changeArguments = function(p) {
 		const args = p.value.arguments;
 		// if(args.length === 1) {
 		// 	return p;
-		// } else 
+		// } else
 		const literalArgs = args.filter(p => p.type === 'Literal');
 		if (literalArgs && literalArgs.length > 1) {
-			const newArgs = j.objectExpression(literalArgs.map((p, index) => 
+			const newArgs = j.objectExpression(literalArgs.map((p, index) =>
 				j.property('init',
 					j.identifier(index === 0 ? 'fallback': 'use'),
 					j.literal(p.value)
@@ -18,16 +18,15 @@ module.exports = function(j, ast) {
 			p.value.arguments = [newArgs];
 		}
 		console.log(p.value.arguments);
-        return p;
+		return p;
 	};
 	const name = getName(j, ast, 'extract-text-webpack-plugin');
 	return ast.find(j.CallExpression)
         .filter(p => p.value.callee.object)
         .filter(p => p.value.callee.object.name === name)
-        .forEach(changeArguments)
-		.toSource({ quote: 'single' })
+        .forEach(changeArguments);
 		//.forEach(p => console.log(p.toSource()));
-	
+
 	//return ast.toSource({ quote: 'single' });
 };
 

lib/transformations/loaderOptionsPlugin/loaderOptionsPlugin.js

@@ -21,6 +21,5 @@ module.exports = function(j, ast) {
 		.filter(path => path.parent.value.key.name === 'plugins')
 		.forEach(path => {
 			createOrUpdatePluginByName(j, path, 'webpack.optimize.LoaderOptionsPlugin', loaderOptions);
-		})
-		.toSource({ quote: 'single' });
+		});
 };

lib/transformations/loaders/loaders.js

@@ -169,5 +169,5 @@ module.exports = function(j, ast) {
 	];
 	transforms.forEach(t => t());
 
-	return ast.toSource({ quote: 'single' });
+	return ast;
 };

lib/transformations/removeDeprecatedPlugins/removeDeprecatedPlugins.js

@@ -17,6 +17,5 @@ module.exports = function(j, ast) {
 			} else {
 				j(path).remove();
 			}
-		})
-		.toSource({ quote: 'single' });
+		});
 };

lib/transformations/removeJsonLoader/removeJsonLoader.js

@@ -44,5 +44,5 @@ module.exports = function(j, ast) {
 
 	transforms.forEach(t => t(ast));
 
-	return ast.toSource({ quote: 'single' });
+	return ast;
 };

lib/transformations/resolve/resolve.js

@@ -45,6 +45,5 @@ module.exports = function transformer(j, ast) {
 					.filter(prop => prop.key.name === 'root')
 					.length === 1;
 		})
-		.forEach(createModuleArray)
-		.toSource();
+		.forEach(createModuleArray);
 };

lib/transformations/uglifyJsPlugin/uglifyJsPlugin.js

@@ -21,6 +21,5 @@ module.exports = function(j, ast) {
 				// Plugin is called without arguments
 				args.push(j.objectExpression([createSourceMapsProperty()]));
 			}
-		})
-		.toSource({ quote: 'single' });
+		});
 };