fix(cli): Remove redundant style variable and filter empty strings in coerceNumberArray

Author: cursoragent

src/cli/__tests__/schema-to-yargs.test.ts

@@ -37,6 +37,6 @@ describe('schemaToYargsOptions', () => {
     expect(coerce?.('23,18,14')).toEqual([23, 18, 14]);
     expect(coerce?.('23, 18, 14')).toEqual([23, 18, 14]);
     expect(coerce?.(['23', '18,14'])).toEqual([23, 18, 14]);
-    expect(coerce?.('23,')).toEqual([23, Number.NaN]);
+    expect(coerce?.('23,')).toEqual([23]);
   });
 });

src/cli/register-tool-commands.ts

@@ -298,7 +298,6 @@ function registerToolSubcommand(
       const outputStyle: OutputStyle = argv.style === 'minimal' ? 'minimal' : 'normal';
       const socketPath = argv.socket as string;
       const logLevel = argv['log-level'] as string | undefined;
-      const style = argv.style as string | undefined;
       const filePathRenderStyle = argv.filePathRenderStyle as FilePathRenderStyle | undefined;
       const verboseOutput = argv.verbose === true;
 
@@ -409,7 +408,7 @@ function registerToolSubcommand(
           runtime: 'cli',
           outputStyle,
           filePathRenderStyle,
-          includeNextSteps: style !== 'minimal',
+          includeNextSteps: outputStyle !== 'minimal',
         });
         const writeJsonlFragment =
           outputFormat === 'jsonl'

src/cli/schema-to-yargs.ts

@@ -13,7 +13,8 @@ function coerceNumberArray(value: unknown): number[] {
     String(entry)
       .split(',')
       .map((item) => item.trim())
-      .map((item) => (item === '' ? Number.NaN : Number(item))),
+      .filter((item) => item !== '')
+      .map((item) => Number(item)),
   );
 }