chore(cli): use camelCaseValues transform for cleaner property access

Apply bargs' camelCaseValues transform to all parsers with kebab-case
options. This enables cleaner property access in command handlers:

- values.noColor instead of values['no-color']
- values.outputFile instead of values['output-file']
- values.excludeTag instead of values['exclude-tag']
- etc.

Co-Authored-By: Claude <noreply@anthropic.com>

Author: boneskull

src/cli/builder.ts

@@ -86,13 +86,13 @@ export const createCli = (abortController: AbortController) => {
           cwd: values.cwd,
           engine: values.engine,
           exclude: values.exclude,
-          excludeTags: values['exclude-tag'],
+          excludeTags: values.excludeTag,
           iterations: values.iterations,
           json: values.json,
-          jsonPretty: values['json-pretty'],
-          noColor: values['no-color'],
+          jsonPretty: values.jsonPretty,
+          noColor: values.noColor,
           outputDir: values.output,
-          outputFile: values['output-file'],
+          outputFile: values.outputFile,
           pattern,
           progress: values.progress,
           quiet: values.quiet,
@@ -193,9 +193,9 @@ export const createCli = (abortController: AbortController) => {
               const exitCode = await handleCleanCommand(context, {
                 confirm: values.yes,
                 cwd: values.cwd,
-                maxAge: values['max-age'],
-                maxRuns: values['max-runs'],
-                maxSize: values['max-size'],
+                maxAge: values.maxAge,
+                maxRuns: values.maxRuns,
+                maxSize: values.maxSize,
                 quiet: values.quiet,
                 verbose: values.verbose,
               });
@@ -241,7 +241,7 @@ export const createCli = (abortController: AbortController) => {
                 default: values.default,
                 name,
                 quiet: Boolean(values.quiet),
-                runId: values['run-id'],
+                runId: values.runId,
                 verbose: values.verbose,
               });
               process.exit(exitCode);
@@ -321,7 +321,7 @@ export const createCli = (abortController: AbortController) => {
         const [type] = positionals;
         const context = await createCliContext(values, abortController);
         const exitCode = await initCommand(context, {
-          configType: values['config-type'],
+          configType: values.configType,
           cwd: values.cwd,
           examples: values.examples,
           force: values.force,
@@ -343,13 +343,13 @@ export const createCli = (abortController: AbortController) => {
         const context = {} as CliContext;
 
         const options: AnalyzeOptions = {
-          color: !values['no-color'],
+          color: !values.noColor,
           command,
           cwd: values.cwd || process.cwd(),
-          filterFile: values['filter-file'],
-          groupByFile: values['group-by-file'],
+          filterFile: values.filterFile,
+          groupByFile: values.groupByFile,
           input: values.input,
-          minPercent: values['min-percent'],
+          minPercent: values.minPercent,
           top: values.top,
         };
 
@@ -372,7 +372,7 @@ export const createCli = (abortController: AbortController) => {
           framework,
           iterations: values.iterations,
           json: values.json,
-          noColor: values['no-color'],
+          noColor: values.noColor,
           pattern: files,
           quiet: values.quiet,
           verbose: values.verbose,

src/cli/context.ts

@@ -64,7 +64,7 @@ export const createCliContext = async (
     engine.registerReporter(
       Reporters.HUMAN,
       new HumanReporter({
-        color: !options['no-color'],
+        color: !options.noColor,
         verbose: options.verbose,
       }),
     );
@@ -94,7 +94,7 @@ export const createCliContext = async (
     engine.registerReporter(
       'nyan',
       new NyanReporter({
-        color: !options['no-color'],
+        color: !options.noColor,
       }),
     );
 

src/cli/parsers/analyze.ts

@@ -7,7 +7,7 @@
  * @packageDocumentation
  */
 
-import { map, merge, opt, pos } from '@boneskull/bargs';
+import { camelCaseValues, map, merge, opt, pos } from '@boneskull/bargs';
 
 /**
  * Base parser for `analyze` command options and positionals
@@ -44,17 +44,18 @@ const analyzeParserBase = merge(
 );
 
 /**
- * Parser for `analyze` command with validation
+ * Parser for `analyze` command with validation and camelCase values
  *
- * Validates that either a command or --input is provided.
+ * Validates that either a command or --input is provided. Uses camelCaseValues
+ * transform for cleaner property access.
  */
 export const analyzeParser = map(
-  analyzeParserBase,
-  ({ positionals, values }) => {
+  map(analyzeParserBase, ({ positionals, values }) => {
     const [command] = positionals;
     if (!command && !values.input) {
       throw new Error('Either [command] or --input must be provided');
     }
     return { positionals, values };
-  },
+  }),
+  camelCaseValues,
 );

src/cli/parsers/baseline.ts

@@ -7,33 +7,38 @@
  * @packageDocumentation
  */
 
-import { merge, opt, pos } from '@boneskull/bargs';
+import { camelCaseValues, map, merge, opt, pos } from '@boneskull/bargs';
 
 /**
  * Parser for `baseline set` command
+ *
+ * Uses camelCaseValues transform for cleaner property access.
  */
-export const baselineSetParser = merge(
-  opt.options({
-    branch: opt.string({
-      description: 'Git branch name',
-    }),
-    commit: opt.string({
-      description: 'Git commit SHA (40 characters)',
-    }),
-    default: opt.boolean({
-      description: 'Set as default baseline',
-    }),
-    'run-id': opt.string({
-      description: 'Specific run ID to save (default: most recent)',
-    }),
-  }),
-  pos.positionals(
-    pos.string({
-      description: 'Name for the baseline',
-      name: 'name',
-      required: true,
+export const baselineSetParser = map(
+  merge(
+    opt.options({
+      branch: opt.string({
+        description: 'Git branch name',
+      }),
+      commit: opt.string({
+        description: 'Git commit SHA (40 characters)',
+      }),
+      default: opt.boolean({
+        description: 'Set as default baseline',
+      }),
+      'run-id': opt.string({
+        description: 'Specific run ID to save (default: most recent)',
+      }),
     }),
+    pos.positionals(
+      pos.string({
+        description: 'Name for the baseline',
+        name: 'name',
+        required: true,
+      }),
+    ),
   ),
+  camelCaseValues,
 );
 
 /**

src/cli/parsers/global.ts

@@ -7,33 +7,39 @@
  * @packageDocumentation
  */
 
-import { opt } from '@boneskull/bargs';
+import { camelCaseValues, map, opt } from '@boneskull/bargs';
 
 /**
  * Global options available to all commands
+ *
+ * Uses camelCaseValues transform so handlers can use `values.noColor` instead
+ * of `values['no-color']`.
  */
-export const globalOptions = opt.options({
-  config: opt.string({
-    aliases: ['c'],
-    description: 'Path to configuration file',
-  }),
-  cwd: opt.string({
-    description: 'Working directory',
-  }),
-  json: opt.boolean({
-    description: 'Output results in JSON format',
-  }),
-  'no-color': opt.boolean({
-    description: 'Disable colored output',
+export const globalOptions = map(
+  opt.options({
+    config: opt.string({
+      aliases: ['c'],
+      description: 'Path to configuration file',
+    }),
+    cwd: opt.string({
+      description: 'Working directory',
+    }),
+    json: opt.boolean({
+      description: 'Output results in JSON format',
+    }),
+    'no-color': opt.boolean({
+      description: 'Disable colored output',
+    }),
+    progress: opt.boolean({
+      description: 'Show animated progress bar',
+    }),
+    verbose: opt.boolean({
+      aliases: ['v'],
+      description: 'Enable verbose output',
+    }),
   }),
-  progress: opt.boolean({
-    description: 'Show animated progress bar',
-  }),
-  verbose: opt.boolean({
-    aliases: ['v'],
-    description: 'Enable verbose output',
-  }),
-});
+  camelCaseValues,
+);
 
 /**
  * Additional global options for history and baseline subcommands

src/cli/parsers/history.ts

@@ -7,7 +7,7 @@
  * @packageDocumentation
  */
 
-import { map, merge, opt, pos } from '@boneskull/bargs';
+import { camelCaseValues, map, merge, opt, pos } from '@boneskull/bargs';
 
 /**
  * Parser for `history list` command
@@ -115,32 +115,36 @@ export const historyTrendsParser = merge(
 /**
  * Parser for `history clean` command
  *
- * Includes validation requiring at least one cleanup criterion.
+ * Includes validation requiring at least one cleanup criterion. Uses
+ * camelCaseValues transform for cleaner property access.
  */
 export const historyCleanParser = map(
-  opt.options({
-    'max-age': opt.number({
-      description: 'Remove runs older than this many days',
-    }),
-    'max-runs': opt.number({
-      description: 'Keep only this many most recent runs',
-    }),
-    'max-size': opt.number({
-      description: 'Keep history under this size in bytes',
-    }),
-    yes: opt.boolean({
-      aliases: ['y'],
-      description: 'Confirm cleanup without prompting',
-    }),
-  }),
-  ({ positionals, values }) => {
-    if (!values['max-age'] && !values['max-runs'] && !values['max-size']) {
-      throw new Error(
-        'At least one cleanup criterion must be specified (--max-age, --max-runs, or --max-size)',
-      );
-    }
-    return { positionals, values };
-  },
+  map(
+    opt.options({
+      'max-age': opt.number({
+        description: 'Remove runs older than this many days',
+      }),
+      'max-runs': opt.number({
+        description: 'Keep only this many most recent runs',
+      }),
+      'max-size': opt.number({
+        description: 'Keep history under this size in bytes',
+      }),
+      yes: opt.boolean({
+        aliases: ['y'],
+        description: 'Confirm cleanup without prompting',
+      }),
+    }),
+    ({ positionals, values }) => {
+      if (!values['max-age'] && !values['max-runs'] && !values['max-size']) {
+        throw new Error(
+          'At least one cleanup criterion must be specified (--max-age, --max-runs, or --max-size)',
+        );
+      }
+      return { positionals, values };
+    },
+  ),
+  camelCaseValues,
 );
 
 /**

src/cli/parsers/init.ts

@@ -6,35 +6,40 @@
  * @packageDocumentation
  */
 
-import { merge, opt, pos } from '@boneskull/bargs';
+import { camelCaseValues, map, merge, opt, pos } from '@boneskull/bargs';
 
 /**
  * Parser for `init` command
+ *
+ * Uses camelCaseValues transform for cleaner property access.
  */
-export const initParser = merge(
-  opt.options({
-    'config-type': opt.enum(['json', 'yaml', 'js', 'ts'] as const, {
-      description: 'Configuration file format',
-    }),
-    examples: opt.boolean({
-      description: 'Include example benchmark files',
-    }),
-    force: opt.boolean({
-      description: 'Overwrite existing files',
-    }),
-    quiet: opt.boolean({
-      aliases: ['q'],
-      description: 'Minimal output',
-    }),
-    yes: opt.boolean({
-      aliases: ['y'],
-      description: 'Accept all prompts automatically',
-    }),
-  }),
-  pos.positionals(
-    pos.enum(['basic', 'advanced', 'library'] as const, {
-      description: 'Type of project to initialize',
-      name: 'type',
+export const initParser = map(
+  merge(
+    opt.options({
+      'config-type': opt.enum(['json', 'yaml', 'js', 'ts'] as const, {
+        description: 'Configuration file format',
+      }),
+      examples: opt.boolean({
+        description: 'Include example benchmark files',
+      }),
+      force: opt.boolean({
+        description: 'Overwrite existing files',
+      }),
+      quiet: opt.boolean({
+        aliases: ['q'],
+        description: 'Minimal output',
+      }),
+      yes: opt.boolean({
+        aliases: ['y'],
+        description: 'Accept all prompts automatically',
+      }),
     }),
+    pos.positionals(
+      pos.enum(['basic', 'advanced', 'library'] as const, {
+        description: 'Type of project to initialize',
+        name: 'type',
+      }),
+    ),
   ),
+  camelCaseValues,
 );