feat(utils): add typescript support

package.json

@@ -115,6 +115,7 @@
     "@commitlint/prompt-cli": "^7.0.0",
     "@commitlint/travis-cli": "^7.0.0",
     "@types/node": "^10.5.1",
+    "@types/jest": "^23.1.1",
     "bundlesize": "^0.17.0",
     "codecov": "^3.0.2",
     "commitizen": "^2.10.1",

packages/generators/add-generator.ts

@@ -5,9 +5,9 @@ import * as autoComplete from "inquirer-autocomplete-prompt";
 import * as path from "path";
 import webpackDevServerSchema from "webpack-dev-server/lib/optionsSchema.json";
 
-import * as npmExists from "@webpack-cli/utils/npm-exists";
+import npmExists from "@webpack-cli/utils/npm-exists";
 import { getPackageManager } from "@webpack-cli/utils/package-manager";
-import * as PROP_TYPES from "@webpack-cli/utils/prop-types";
+import PROP_TYPES from "@webpack-cli/utils/prop-types";
 import {
 	AutoComplete,
 	Confirm,

packages/generators/remove-generator.ts

@@ -2,7 +2,7 @@ import * as fs from "fs";
 import * as path from "path";
 import Generator = require("yeoman-generator");
 
-import * as PROP_TYPES from "@webpack-cli/utils/prop-types";
+import PROP_TYPES from "@webpack-cli/utils/prop-types";
 import { List } from "@webpack-cli/webpack-scaffold";
 import { IWebpackOptions } from "./types";
 

packages/init/index.ts

@@ -1,6 +1,6 @@
-import * as defaultGenerator from "@webpack-cli/generators/init-generator";
-import * as modifyConfigHelper from "@webpack-cli/utils/modify-config-helper";
-import * as npmPackagesExists from "@webpack-cli/utils/npm-packages-exists";
+import defaultGenerator from "@webpack-cli/generators/init-generator";
+import modifyConfigHelper from "@webpack-cli/utils/modify-config-helper";
+import npmPackagesExists from "@webpack-cli/utils/npm-packages-exists";
 
 /**
  *
@@ -13,7 +13,7 @@ import * as npmPackagesExists from "@webpack-cli/utils/npm-packages-exists";
  * followed up with a yeoman instance of that if there's packages. If not, it creates a defaultGenerator
  */
 
-export default function initializeInquirer(...args: string[]): Function {
+export default function initializeInquirer(...args: string[]): Function | void {
 	const packages: string[] = args.slice(3);
 
 	if (packages.length === 0) {

packages/init/init.ts

@@ -3,10 +3,11 @@ import * as j from "jscodeshift";
 import pEachSeries = require("p-each-series");
 import * as path from "path";
 
-import * as propTypes from "@webpack-cli/utils/prop-types";
-import * as astTransform from "@webpack-cli/utils/recursive-parser";
-import * as runPrettier from "@webpack-cli/utils/run-prettier";
+import propTypes from "@webpack-cli/utils/prop-types";
+import astTransform from "@webpack-cli/utils/recursive-parser";
+import runPrettier from "@webpack-cli/utils/run-prettier";
 
+import { INode } from "@webpack-cli/utils/types/NodePath";
 import { IError } from "./types";
 import { IConfiguration, IWebpackProperties } from "./types/Transform";
 
@@ -53,7 +54,7 @@ export default function runTransform(webpackProperties: IWebpackProperties, acti
 		);
 		const transformAction: string | null = action || null;
 
-		return pEachSeries(transformations, (f: string): Promise<string[]> => {
+		return pEachSeries(transformations, (f: string): boolean | INode => {
 			return astTransform(j, ast, config.webpackOptions[f], transformAction, f);
 		})
 			.then((_?: any) => {

packages/migrate/bannerPlugin/bannerPlugin.test.js

@@ -1,6 +1,6 @@
 "use strict";
 
-const defineTest = require("@webpack-cli/utils/defineTest");
+const defineTest = require("@webpack-cli/utils/defineTest").default;
 
 defineTest(__dirname, "bannerPlugin", "bannerPlugin-0");
 defineTest(__dirname, "bannerPlugin", "bannerPlugin-1");

packages/migrate/extractTextPlugin/extractTextPlugin.test.js

@@ -1,5 +1,5 @@
 "use strict";
 
-const defineTest = require("@webpack-cli/utils/defineTest");
+const defineTest = require("@webpack-cli/utils/defineTest").default;
 
 defineTest(__dirname, "extractTextPlugin");

packages/migrate/index.ts

@@ -7,7 +7,7 @@ import pLazy = require("p-lazy");
 import * as path from "path";
 import { validate, WebpackOptionsValidationError } from "webpack";
 
-import * as runPrettier from "@webpack-cli/utils/run-prettier";
+import runPrettier from "@webpack-cli/utils/run-prettier";
 
 import { transformations } from "./migrate";
 import { IJSCodeshift, INode } from "./types/NodePath";

packages/migrate/loaderOptionsPlugin/loaderOptionsPlugin.test.js

@@ -1,6 +1,6 @@
 "use strict";
 
-const defineTest = require("@webpack-cli/utils/defineTest");
+const defineTest = require("@webpack-cli/utils/defineTest").default;
 
 defineTest(__dirname, "loaderOptionsPlugin", "loaderOptionsPlugin-0");
 defineTest(__dirname, "loaderOptionsPlugin", "loaderOptionsPlugin-1");

packages/migrate/loaderOptionsPlugin/loaderOptionsPlugin.ts

@@ -23,7 +23,7 @@ interface ILoaderOptions {
  *
  */
 
-export default function(j: IJSCodeshift, ast: INode): void {
+export default function(j: IJSCodeshift, ast: INode): INode {
 	const loaderOptions: ILoaderOptions = {};
 
 	// If there is debug: true, set debug: true in the plugin

packages/migrate/loaders/loaders.test.js

@@ -1,6 +1,6 @@
 "use strict";
 
-const defineTest = require("@webpack-cli/utils/defineTest");
+const defineTest = require("@webpack-cli/utils/defineTest").default;
 
 defineTest(__dirname, "loaders", "loaders-0");
 defineTest(__dirname, "loaders", "loaders-1");

packages/migrate/loaders/loaders.ts

@@ -209,7 +209,7 @@ export default function(j: IJSCodeshift, ast: INode): INode {
 	 * @returns {Node} ast - jscodeshift ast
 	 */
 
-	const prepostLoaders = (_?: void): void =>
+	const prepostLoaders = (_?: void): INode =>
 		ast
 			.find(j.ObjectExpression)
 			.filter((p: INode): boolean => utils.findObjWithOneOfKeys(p, ["preLoaders", "postLoaders"]))
@@ -221,7 +221,7 @@ export default function(j: IJSCodeshift, ast: INode): INode {
 	 * @returns {Node} ast - jscodeshift ast
 	 */
 
-	const loadersToRules = (_?: void): void =>
+	const loadersToRules = (_?: void): INode =>
 		ast
 			.find(j.Identifier)
 			.filter(checkForLoader)
@@ -308,7 +308,7 @@ export default function(j: IJSCodeshift, ast: INode): INode {
 	 * @returns {Node} ast - jscodeshift ast
 	 */
 
-	const addLoaderSuffix = (_?: void): void =>
+	const addLoaderSuffix = (_?: void): INode =>
 		ast.find(j.ObjectExpression).forEach((path: INode): void => {
 			path.value.properties.forEach((prop: INode): void => {
 				if (
@@ -360,7 +360,7 @@ export default function(j: IJSCodeshift, ast: INode): INode {
 	 * @returns {Node} ast - jscodeshift ast
 	 */
 
-	const moveOptionsToUse = (_?: void): void =>
+	const moveOptionsToUse = (_?: void): INode =>
 		ast
 			.find(j.ObjectExpression)
 			.filter((p: INode): boolean => utils.findObjWithOneOfKeys(p, ["use"]))

packages/migrate/moduleConcatenationPlugin/moduleConcatenationPlugin.test.js

@@ -1,6 +1,6 @@
 "use strict";
 
-const defineTest = require("@webpack-cli/utils/defineTest");
+const defineTest = require("@webpack-cli/utils/defineTest").default;
 
 defineTest(
 	__dirname,

packages/migrate/namedModulesPlugin/namedModulesPlugin.test.js

@@ -1,6 +1,6 @@
 "use strict";
 
-const defineTest = require("@webpack-cli/utils/defineTest");
+const defineTest = require("@webpack-cli/utils/defineTest").default;
 
 defineTest(__dirname, "namedModulesPlugin", "namedModulesPlugin-0");
 defineTest(__dirname, "namedModulesPlugin", "namedModulesPlugin-1");

packages/migrate/noEmitOnErrorsPlugin/noEmitOnErrorsPlugin.test.js

@@ -1,6 +1,6 @@
 "use strict";
 
-const defineTest = require("@webpack-cli/utils/defineTest");
+const defineTest = require("@webpack-cli/utils/defineTest").default;
 
 defineTest(__dirname, "noEmitOnErrorsPlugin", "noEmitOnErrorsPlugin-0");
 defineTest(__dirname, "noEmitOnErrorsPlugin", "noEmitOnErrorsPlugin-1");

packages/migrate/outputPath/outputPath.test.js

@@ -1,6 +1,6 @@
 "use strict";
 
-const defineTest = require("@webpack-cli/utils/defineTest");
+const defineTest = require("@webpack-cli/utils/defineTest").default;
 
 defineTest(__dirname, "outputPath", "outputPath-0");
 defineTest(__dirname, "outputPath", "outputPath-1");

packages/migrate/outputPath/outputPath.ts

@@ -57,7 +57,7 @@ export default function(j: IJSCodeshift, ast: INode): INode | void {
 			.replaceWith((p: INode): INode => replaceWithPath(j, p, finalPathName));
 
 		if (!isPathPresent) {
-			const pathRequire: INode[] = utils.getRequire(j, "path", "path");
+			const pathRequire: INode = utils.getRequire(j, "path", "path");
 			return ast
 				.find(j.Program)
 				.replaceWith((p: INode): INode =>

packages/migrate/removeDeprecatedPlugins/removeDeprecatedPlugins.test.js

@@ -1,6 +1,6 @@
 "use strict";
 
-const defineTest = require("@webpack-cli/utils/defineTest");
+const defineTest = require("@webpack-cli/utils/defineTest").default;
 
 defineTest(__dirname, "removeDeprecatedPlugins", "removeDeprecatedPlugins-0");
 defineTest(__dirname, "removeDeprecatedPlugins", "removeDeprecatedPlugins-1");

packages/migrate/removeJsonLoader/removeJsonLoader.test.js

@@ -1,6 +1,6 @@
 "use strict";
 
-const defineTest = require("@webpack-cli/utils/defineTest");
+const defineTest = require("@webpack-cli/utils/defineTest").default;
 
 defineTest(__dirname, "removeJsonLoader", "removeJsonLoader-0");
 defineTest(__dirname, "removeJsonLoader", "removeJsonLoader-1");

packages/migrate/resolve/resolve.test.js

@@ -1,5 +1,5 @@
 "use strict";
 
-const defineTest = require("@webpack-cli/utils/defineTest");
+const defineTest = require("@webpack-cli/utils/defineTest").default;
 
 defineTest(__dirname, "resolve");

packages/migrate/resolve/resolve.ts

@@ -9,7 +9,7 @@ import { IJSCodeshift, INode } from "../types/NodePath";
  * @returns {Node} ast - jscodeshift ast
  */
 
-export default function transformer(j: IJSCodeshift, ast: INode): INode[] | void {
+export default function transformer(j: IJSCodeshift, ast: INode): INode {
 
 	const getRootVal = (p: INode): INode => {
 		return p.node.value.properties.filter((prop: INode): boolean => prop.key.name === "root")[0];