fix: remove unnecessary optional peer dependencies

[ylemkimon]

What kind of change does this PR introduce?
Bugfix (this doesn't fix any bugs, but in terms of semver I think it should be a fix)

Did you add tests for your changes?
N/A

If relevant, did you update the documentation?
N/A

Summary
In #1816, default commands packages were added to peer dependencies of @webpack-cli/package-utils, as they were required by packageExists in order to fix #1815.

However, in #1822, packageExists was moved directly into webpack-cli not @webpack-cli/utils, so @webpack-cli/utils doesn't require them.

Does this PR introduce a breaking change?
No

Other information
Note #1815 has been fixed by #1822.

[alexander-akait]

No, they are needed

[ylemkimon]

@evilebottnawi This is every require/import calls in @webpack-cli/utils and they don't seem to require @webpack-cli/info, @webpack-cli/init, or @webpack-cli/serve:

webpack-cli/packages/utils/src/ast-utils.ts

Lines 1 to 2 in 258219a

	import { JSCodeshift, Node, valueType } from './types/NodePath';
	import { isIdentifierStart, isIdentifierChar, isKeyword } from './validate-identifier';

webpack-cli/packages/utils/src/copy-utils.ts

Line 1 in 258219a

import path from 'path';

webpack-cli/packages/utils/src/global-packages-path.ts

Lines 1 to 3 in 258219a

	import spawn from 'cross-spawn';
	import path from 'path';
	import { getPackageManager } from 'webpack-cli/lib/utils/get-package-manager';

webpack-cli/packages/utils/src/global-packages-path.ts

Line 24 in 258219a

return require('global-modules');

webpack-cli/packages/utils/src/index.ts

Lines 1 to 12 in 258219a

	export * from './ast-utils';
	export * from './copy-utils';
	export * from './modify-config-helper';
	export * from './npm-exists';
	export * from './npm-packages-exists';
	export * from './recursive-parser';
	export * from './resolve-packages';
	export * from './run-prettier';
	export * from './scaffold';
	export * from './validate-identifier';
	export * from './prop-types';
	export * from './global-packages-path';

webpack-cli/packages/utils/src/isWebpack5.ts

Line 1 in 258219a

import { version } from 'webpack';

webpack-cli/packages/utils/src/modify-config-helper.ts

Lines 1 to 8 in 258219a

	import { green } from 'colorette';
	import fs from 'fs';
	import logger from 'webpack-cli/lib/utils/logger';
	import path from 'path';
	import yeoman from 'yeoman-environment';
	import Generator from 'yeoman-generator';
	import { runTransform } from './scaffold';
	import { getPackageManager } from 'webpack-cli/lib/utils/get-package-manager';

webpack-cli/packages/utils/src/modify-config-helper.ts

Line 67 in 258219a

return this.composeWith(require.resolve(pkgPath), {});

webpack-cli/packages/utils/src/modify-config-helper.ts

Line 82 in 258219a

const packageData = require(packagePath);

webpack-cli/packages/utils/src/modify-config-helper.ts

Line 107 in 258219a

configModule = require(confPath);

webpack-cli/packages/utils/src/npm-exists.ts

Line 1 in 258219a

import got from 'got';

webpack-cli/packages/utils/src/npm-packages-exists.ts

Lines 1 to 7 in 258219a

	import { resolve } from 'path';
	import { existsSync } from 'fs';
	import { red, bold } from 'colorette';
	import { npmExists } from './npm-exists';
	import { isLocalPath } from './path-utils';
	import { resolvePackages } from './resolve-packages';
	import { getPathToGlobalPackages } from './global-packages-path';

webpack-cli/packages/utils/src/path-utils.ts

Lines 1 to 3 in 258219a

	import findup from 'findup-sync';
	import fs from 'fs';
	import path from 'path';

webpack-cli/packages/utils/src/prop-types.ts

Lines 1 to 2 in 258219a

	import { isWebpack5 } from './isWebpack5';
	import { config } from 'webpack';

webpack-cli/packages/utils/src/recursive-parser.ts

Lines 1 to 4 in 258219a

	import { parseTopScope, findRootNodesByName, addProperty, removeProperty, parseMerge, safeTraverse } from './ast-utils';
	import { JSCodeshift, Node, valueType } from './types/NodePath';
	import logger from 'webpack-cli/lib/utils/logger';

webpack-cli/packages/utils/src/resolve-packages.ts

Lines 1 to 8 in 258219a

	import { bold } from 'colorette';
	import logger from 'webpack-cli/lib/utils/logger';
	import path from 'path';
	import { modifyHelperUtil } from './modify-config-helper';
	import { getPathToGlobalPackages } from './global-packages-path';
	import { spawnChild } from './spawn-child';
	import { isLocalPath } from './path-utils';
	import { ExecaSyncReturnValue } from 'execa';

webpack-cli/packages/utils/src/resolve-packages.ts

Line 59 in 258219a

require.resolve(absolutePath);

webpack-cli/packages/utils/src/run-prettier.ts

Lines 1 to 2 in 258219a

	import fs from 'fs';
	import logger from 'webpack-cli/lib/utils/logger';

webpack-cli/packages/utils/src/run-prettier.ts

Line 19 in 258219a

prettier = require('prettier');

webpack-cli/packages/utils/src/scaffold.ts

Lines 1 to 13 in 258219a

	import { green } from 'colorette';
	import j from 'jscodeshift';
	import logger from 'webpack-cli/lib/utils/logger';
	import pEachSeries = require('p-each-series');
	import path from 'path';
	import { getPackageManager } from 'webpack-cli/lib/utils/get-package-manager';
	import { findProjectRoot } from './path-utils';
	import { Error } from './types';
	import { Config, TransformConfig } from './types';
	import { PROP_TYPES } from './prop-types';
	import { recursiveTransform } from './recursive-parser';
	import { runPrettier } from './run-prettier';
	import { Node } from './types/NodePath';

webpack-cli/packages/utils/src/spawn-child.ts

Lines 1 to 5 in 258219a

	import path from 'path';
	import fs from 'fs';
	import { ExecaSyncReturnValue, sync } from 'execa';
	import { getPathToGlobalPackages } from './global-packages-path';
	import { getPackageManager } from 'webpack-cli/lib/utils/get-package-manager';

[alexander-akait]

They are in require when you use external comamnds, without it yarn pnp doesn't work, do not touch this

[alexander-akait]

oh, my mistake, yes, no need to have it, I think it is webpack-cli package

[snitin315]

Thanks!