fix(core): migrate to yargs 17.x

docs/generated/cli/affected-apps.md

@@ -69,8 +69,6 @@ Show help
 
 ### ~~only-failed~~
 
-Default: `false`
-
 **Deprecated:** The command to rerun failed projects will appear if projects fail. This now does nothing and will be removed in v15.
 
 Isolate projects which previously failed
@@ -85,8 +83,6 @@ This is the name of the tasks runner configured in nx.json
 
 ### skip-nx-cache
 
-Default: `false`
-
 Rerun the tasks even when the results are available in the cache
 
 ### uncommitted

docs/generated/cli/affected-graph.md

@@ -103,8 +103,6 @@ Bind the project graph server to a specific ip address.
 
 ### ~~only-failed~~
 
-Default: `false`
-
 **Deprecated:** The command to rerun failed projects will appear if projects fail. This now does nothing and will be removed in v15.
 
 Isolate projects which previously failed
@@ -125,8 +123,6 @@ This is the name of the tasks runner configured in nx.json
 
 ### skip-nx-cache
 
-Default: `false`
-
 Rerun the tasks even when the results are available in the cache
 
 ### uncommitted
@@ -147,6 +143,4 @@ Show version number
 
 ### watch
 
-Default: `false`
-
 Watch for changes to project graph and update in-browser

docs/generated/cli/affected-libs.md

@@ -69,8 +69,6 @@ Show help
 
 ### ~~only-failed~~
 
-Default: `false`
-
 **Deprecated:** The command to rerun failed projects will appear if projects fail. This now does nothing and will be removed in v15.
 
 Isolate projects which previously failed
@@ -85,8 +83,6 @@ This is the name of the tasks runner configured in nx.json
 
 ### skip-nx-cache
 
-Default: `false`
-
 Rerun the tasks even when the results are available in the cache
 
 ### uncommitted

docs/generated/cli/affected.md

@@ -87,8 +87,6 @@ Show help
 
 ### ~~only-failed~~
 
-Default: `false`
-
 **Deprecated:** The command to rerun failed projects will appear if projects fail. This now does nothing and will be removed in v15.
 
 Isolate projects which previously failed
@@ -103,8 +101,6 @@ This is the name of the tasks runner configured in nx.json
 
 ### skip-nx-cache
 
-Default: `false`
-
 Rerun the tasks even when the results are available in the cache
 
 ### target

docs/generated/cli/format-check.md

@@ -53,8 +53,6 @@ Format only libraries and applications files.
 
 ### ~~only-failed~~
 
-Default: `false`
-
 **Deprecated:** The command to rerun failed projects will appear if projects fail. This now does nothing and will be removed in v15.
 
 Isolate projects which previously failed
@@ -69,8 +67,6 @@ This is the name of the tasks runner configured in nx.json
 
 ### skip-nx-cache
 
-Default: `false`
-
 Rerun the tasks even when the results are available in the cache
 
 ### uncommitted

docs/generated/cli/format-write.md

@@ -53,8 +53,6 @@ Format only libraries and applications files.
 
 ### ~~only-failed~~
 
-Default: `false`
-
 **Deprecated:** The command to rerun failed projects will appear if projects fail. This now does nothing and will be removed in v15.
 
 Isolate projects which previously failed
@@ -69,8 +67,6 @@ This is the name of the tasks runner configured in nx.json
 
 ### skip-nx-cache
 
-Default: `false`
-
 Rerun the tasks even when the results are available in the cache
 
 ### uncommitted

docs/generated/cli/graph.md

@@ -107,6 +107,4 @@ Show version number
 
 ### watch
 
-Default: `false`
-
 Watch for changes to project graph and update in-browser

docs/generated/cli/print-affected.md

@@ -81,8 +81,6 @@ Show help
 
 ### ~~only-failed~~
 
-Default: `false`
-
 **Deprecated:** The command to rerun failed projects will appear if projects fail. This now does nothing and will be removed in v15.
 
 Isolate projects which previously failed
@@ -97,8 +95,6 @@ Select the subset of the returned json document (e.g., --selected=projects)
 
 ### skip-nx-cache
 
-Default: `false`
-
 Rerun the tasks even when the results are available in the cache
 
 ### uncommitted

docs/generated/cli/run-many.md

@@ -57,8 +57,6 @@ Show help
 
 ### ~~only-failed~~
 
-Default: `false`
-
 **Deprecated:** The command to rerun failed projects will appear if projects fail. This now does nothing and will be removed in v15.
 
 Only run the target on projects which previously failed
@@ -77,8 +75,6 @@ Override the tasks runner in `nx.json`
 
 ### skip-nx-cache
 
-Default: `false`
-
 Rerun the tasks even when the results are available in the cache
 
 ### target
@@ -95,8 +91,6 @@ Show version number
 
 ### ~~with-deps~~
 
-Default: `false`
-
 **Deprecated:** Configure target dependencies instead. It will be removed in v14.
 
 Include dependencies of specified projects when computing what to run

packages/create-nx-workspace/bin/create-nx-workspace.ts

@@ -122,17 +122,15 @@ export const commandsObject: yargs.Argv<Arguments> = yargs
     'strip-dashed': true,
     'dot-notation': false,
   })
-  .strict()
   .command(
     // this is the default and only command
     '$0 [name] [options]',
     'Create a new Nx workspace',
     (yargs) =>
       yargs
-        .positional('name', {
+        .option('name', {
           describe: `Workspace name (e.g. org name)`,
           type: 'string',
-          demandOption: true,
         })
         .option('preset', {
           describe: `Customizes the initial content of your workspace. To build your own see https://nx.dev/nx-plugin/overview#preset`,
@@ -179,7 +177,7 @@ export const commandsObject: yargs.Argv<Arguments> = yargs
           describe: `Default base to use for new projects`,
           type: 'string',
         }),
-    async (argv) => {
+    async (argv: yargs.ArgumentsCamelCase<Arguments>) => {
       await main(argv).catch((error) => {
         const { version } = require('../package.json');
         output.error({
@@ -192,7 +190,7 @@ export const commandsObject: yargs.Argv<Arguments> = yargs
   )
   .help('help')
   .updateLocale(yargsDecorator)
-  .version(nxVersion);
+  .version(nxVersion) as yargs.Argv<Arguments>;
 
 async function main(parsedArgs: yargs.Arguments<Arguments>) {
   const {
@@ -242,7 +240,9 @@ async function main(parsedArgs: yargs.Arguments<Arguments>) {
   }
 }
 
-async function getConfiguration(argv: yargs.Arguments<Arguments>) {
+async function getConfiguration(
+  argv: yargs.Arguments<Arguments>
+): Promise<void> {
   try {
     let style, appName;
 
@@ -260,7 +260,7 @@ async function getConfiguration(argv: yargs.Arguments<Arguments>) {
     const defaultBase = await determineDefaultBase(argv);
     const nxCloud = await determineNxCloud(argv);
 
-    return {
+    Object.assign(argv, {
       name,
       preset,
       appName,
@@ -269,7 +269,7 @@ async function getConfiguration(argv: yargs.Arguments<Arguments>) {
       nxCloud,
       packageManager,
       defaultBase,
-    };
+    });
   } catch (e) {
     console.error(e);
     process.exit(1);
@@ -751,7 +751,7 @@ function execAndWait(command: string, cwd: string) {
   });
 }
 
-async function determineNxCloud(parsedArgs: any) {
+async function determineNxCloud(parsedArgs: yargs.Arguments<Arguments>) {
   if (parsedArgs.nxCloud === undefined) {
     return enquirer
       .prompt([

packages/create-nx-workspace/bin/decorator.ts

@@ -1,6 +1,6 @@
 import * as chalk from 'chalk';
 
 export const yargsDecorator = {
-  'Options:': `${chalk.green('Options')}:`,
-  'Positionals:': `${chalk.green('Positionals')}:`,
+  'Options:': `${chalk.bold('Options')}:`,
+  'Positionals:': `${chalk.bold('Positionals')}:`,
 };

scripts/documentation/documentation.ts

@@ -1,5 +1,7 @@
 import * as chalk from 'chalk';
 import { execSync } from 'child_process';
+import { removeSync } from 'fs-extra';
+import { join } from 'path';
 import { generateCLIDocumentation } from './generate-cli-data';
 import { generateCNWocumentation } from './generate-cnw-documentation';
 import { generateDevkitDocumentation } from './generate-devkit-documentation';
@@ -10,8 +12,16 @@ async function generate() {
     console.log(`${chalk.blue('i')} Generating Documentation`);
     generatePackageSchemas();
     generateDevkitDocumentation();
-    await generateCNWocumentation();
-    await generateCLIDocumentation();
+
+    const commandsOutputDirectory = join(
+      __dirname,
+      '../../docs/',
+      'generated',
+      'cli'
+    );
+    removeSync(commandsOutputDirectory);
+    await generateCNWocumentation(commandsOutputDirectory);
+    await generateCLIDocumentation(commandsOutputDirectory);
 
     console.log(`\n${chalk.green('✓')} Generated Documentation\n`);
   } catch (e) {

scripts/documentation/generate-cli-data.ts

@@ -1,16 +1,18 @@
 import * as chalk from 'chalk';
 import { readFileSync } from 'fs';
-import { readJsonSync, removeSync } from 'fs-extra';
+import { readJsonSync } from 'fs-extra';
 import { join } from 'path';
-import { dedent } from 'tslint/lib/utils';
 import {
   formatDeprecated,
   generateMarkdownFile,
+  getCommands,
+  parseCommand,
+  ParsedCommand,
   sortAlphabeticallyFunction,
 } from './utils';
 import { register as registerTsConfigPaths } from 'tsconfig-paths';
-
 import { examples } from '../../packages/nx/src/command-line/examples';
+import { dedent } from 'tslint/lib/utils';
 
 const importFresh = require('import-fresh');
 
@@ -24,21 +26,9 @@ const sharedCommands = [
   'test',
 ];
 
-interface ParsedCommandOption {
-  name: string;
-  description: string;
-  default: string;
-  deprecated: boolean | string;
-}
-
-interface ParsedCommand {
-  name: string;
-  commandString: string;
-  description: string;
-  options?: Array<ParsedCommandOption>;
-}
-
-export async function generateCLIDocumentation() {
+export async function generateCLIDocumentation(
+  commandsOutputDirectory: string
+) {
   /**
    * For certain commands, they will output dynamic data at runtime in a real workspace,
    * so we leverage an envrionment variable to inform the logic of the context that we
@@ -57,65 +47,6 @@ export async function generateCLIDocumentation() {
     '../../packages/nx/src/command-line/nx-commands'
   );
 
-  const commandsOutputDirectory = join(
-    __dirname,
-    '../../docs/',
-    'generated',
-    'cli'
-  );
-  removeSync(commandsOutputDirectory);
-
-  function getCommands(command) {
-    return command
-      .getInternalMethods()
-      .getCommandInstance()
-      .getCommandHandlers();
-  }
-  async function parseCommandInstance(
-    name: string,
-    command: any
-  ): Promise<ParsedCommand> {
-    // It is not a function return a strip down version of the command
-    if (
-      !(
-        command.builder &&
-        command.builder.constructor &&
-        command.builder.call &&
-        command.builder.apply
-      )
-    ) {
-      return {
-        name,
-        commandString: command.original,
-        description: command.description,
-      };
-    }
-    // Show all the options we can get from yargs
-    const builder = await command.builder(
-      importFresh('yargs')().getInternalMethods().reset()
-    );
-    const builderDescriptions = builder
-      .getInternalMethods()
-      .getUsageInstance()
-      .getDescriptions();
-    const builderDefaultOptions = builder.getOptions().default;
-    const builderDeprecatedOptions = builder.getDeprecatedOptions();
-    return {
-      name,
-      description: command.description,
-      commandString: command.original,
-      options:
-        Object.keys(builderDescriptions).map((key) => ({
-          name: key,
-          description: builderDescriptions[key]
-            ? builderDescriptions[key].replace('__yargsString__:', '')
-            : '',
-          default: builderDefaultOptions[key],
-          deprecated: builderDeprecatedOptions[key],
-        })) || null,
-    };
-  }
-
   function generateMarkdown(command: ParsedCommand) {
     let template = dedent`
 ---
@@ -181,7 +112,7 @@ nx ${command.commandString}
     Object.keys(nxCommands)
       .filter((name) => !sharedCommands.includes(name))
       .filter((name) => nxCommands[name].description)
-      .map((name) => parseCommandInstance(name, nxCommands[name]))
+      .map((name) => parseCommand(name, nxCommands[name]))
       .map(async (command) => generateMarkdown(await command))
       .map(async (templateObject) =>
         generateMarkdownFile(commandsOutputDirectory, await templateObject)