chore(cli): remove findup-sync from root move to utils package && format ts files

bin/utils/convert-argv.js

@@ -7,6 +7,7 @@ const webpackConfigurationSchema = require("../config/webpackConfigurationSchema
 const validateSchema = require("webpack").validateSchema;
 const WebpackOptionsValidationError = require("webpack").WebpackOptionsValidationError;
 const findup = require("findup-sync");
+// const { webpackConfigPath } = require("@webpack-cli/utils/path-utils");
 
 module.exports = function(...args) {
 	const argv = args[1] || args[0];
@@ -72,6 +73,7 @@ module.exports = function(...args) {
 		const defaultConfigFileNames = ["webpack.config", "webpackfile"].join("|");
 		const webpackConfigFileRegExp = `(${defaultConfigFileNames})(${extensions.join("|")})`;
 		const pathToWebpackConfig = findup(webpackConfigFileRegExp);
+		// const pathToWebpackConfig = webpackConfigPath(extensions);
 
 		if (pathToWebpackConfig) {
 			const resolvedPath = path.resolve(pathToWebpackConfig);

packages/utils/__tests__/is-local-path.test.ts

@@ -1,7 +1,7 @@
 "use strict";
 
 import * as path from "path";
-import isLocalPath from "../is-local-path";
+import { isLocalPath } from "../path-utils";
 
 describe("is-local-path", () => {
 	it("returns true for paths beginning in the current directory", () => {

packages/utils/npm-packages-exists.ts

@@ -1,7 +1,7 @@
 import chalk from "chalk";
 
-import isLocalPath from "./is-local-path";
 import npmExists from "./npm-exists";
+import { isLocalPath } from "./path-utils";
 import { resolvePackages } from "./resolve-packages";
 
 const WEBPACK_SCAFFOLD_PREFIX = "webpack-scaffold";

packages/utils/path-utils.ts

@@ -0,0 +1,43 @@
+import * as findup from "findup-sync";
+import * as fs from "fs";
+import * as path from "path";
+
+/**
+ * Attempts to detect whether the string is a local path regardless of its
+ * existence by checking its format. The point is to distinguish between
+ * paths and modules on the npm registry. This will fail for non-existent
+ * local Windows paths that begin with a drive letter, e.g. C:..\generator.js,
+ * but will succeed for any existing files and any absolute paths.
+ *
+ * @param {String} str - string to check
+ * @returns {Boolean} whether the string could be a path to a local file or directory
+ */
+
+export function isLocalPath(str: string): boolean {
+	return path.isAbsolute(str) || /^\./.test(str) || fs.existsSync(str);
+}
+
+/**
+ * Get absolute path of a webpack config in a project.
+ *
+ * @param {String[]} str - array of extensions to look for.
+ * @returns {String} Absolute path of the config.
+ */
+
+export function webpackConfigPath(extensions: string[]): string {
+	const defaultConfigFileNames = ["webpack.config", "webpackfile"].join("|");
+	const webpackConfigFileRegExp = `(${defaultConfigFileNames})(${extensions.join("|")})`;
+	return findup(webpackConfigFileRegExp);
+}
+
+/**
+ * Find the root directory path of a project.
+ *
+ * @returns {String} Absolute path of the project root.
+ */
+
+export function findProjectRoot(): string {
+	const rootFilePath = findup(`package.json`);
+	const projectRoot = path.dirname(rootFilePath);
+	return projectRoot;
+}

packages/utils/resolve-packages.ts

@@ -1,10 +1,10 @@
 import chalk from "chalk";
 import * as path from "path";
 
-import isLocalPath from "./is-local-path";
 import modifyConfigHelper from "./modify-config-helper";
 import { getPathToGlobalPackages } from "./package-manager";
 import { spawnChild } from "./package-manager";
+import { isLocalPath } from "./path-utils";
 
 interface IChildProcess {
 	status: number;

packages/utils/scaffold.ts

@@ -2,7 +2,7 @@ import chalk from "chalk";
 import * as j from "jscodeshift";
 import pEachSeries = require("p-each-series");
 import * as path from "path";
-import { findProjectRoot } from "./find-root";
+import { findProjectRoot } from "./path-utils";
 
 import { IError } from "../init/types";
 import { IConfig, ITransformConfig } from "./modify-config-helper";