From fedf12250518bed7d87175452a9b216f8a22f5d1 Mon Sep 17 00:00:00 2001
From: Nate Meyer <672246+notnmeyer@users.noreply.github.com>
Date: Tue, 31 Oct 2023 12:46:21 -0700
Subject: [PATCH] chore(cli): take a pass on run's help output (#2476)

---
 projects/optic/src/commands/run.ts | 27 ++++++++++++++-------------
 1 file changed, 14 insertions(+), 13 deletions(-)

diff --git a/projects/optic/src/commands/run.ts b/projects/optic/src/commands/run.ts
index 15ca232053..4b5bb52a27 100644
--- a/projects/optic/src/commands/run.ts
+++ b/projects/optic/src/commands/run.ts
@@ -118,31 +118,32 @@ export function registerRunCommand(cli: Command, config: OpticCliConfig) {
   cli
     .command('run')
     .description(
-      `Optic's CI workflow command.
-Tests that every OpenAPI specification in your repo is accurate, has no breaking changes and follows the standards you defined in the optic.yml file;
-then posts a comment with a report to your PR/MR and exits with code 1 when issues are found.`
+      'CI workflow command that tests each OpenAPI specification in your repo and summarizes the results as a pull (or merge) request comment'
     )
     .configureHelp({ commandUsage: usage })
     .option(
-      '--ignore <ignore-glob>',
-      'OpenAPI specification files to ignore, comma separated globs.'
+      '-i, --ignore <glob_pattern,...>',
+      'Glob patterns matching specifications to ignore'
     )
-    .option(
-      '--include-git-ignored',
-      'Set to true to also match Git ignored files.',
-      false
+    .addOption(
+      new Option(
+        '-I, --include-git-ignored <bool>',
+        'Include specifications matched in .gitignore'
+      )
+        .choices(['true', 'false'])
+        .default('false')
     )
     .addOption(
       new Option(
-        '--severity <severity>',
-        'Set to "none" to prevent Optic from exiting 1 when issues are found.'
+        '-s, --severity <severity>',
+        'Control the exit code when there are issues: error=1, none=0'
       )
         .choices(severities)
         .default('error')
     )
     .argument(
-      '[file_paths]',
-      'OpenAPI specification files to handle, comma separated globs. Leave empty to match all non-ignored OpenAPI files in your repository.'
+      'file_paths',
+      'Comma-seperated glob patterns matching specifications to process. When omitted, matches all non-ignored specifications.'
     )
     .action(errorHandler(getRunAction(config), { command: 'run' }));
 }