rework again, do linting in argv parser

features/retry.feature

@@ -12,6 +12,7 @@ Feature: Retry flaky tests
       """
     And it fails
 
+  @spawn
   Scenario: running Cucumber JS with --retryTagFilter in camel case will result in a warning
     Given a file named "features/a.feature" with:
       """

features/support/world.ts

@@ -65,7 +65,6 @@ export class World {
         argv: args,
         cwd,
         stdout,
-        stderr,
       })
       let error: any
       try {

src/cli/argv_parser.ts

@@ -224,6 +224,14 @@ const ArgvParser = {
       args: program.args,
     }
   },
+
+  lint(fullArgv: string[]): void {
+    if (fullArgv.includes('--retryTagFilter')) {
+      console.warn(
+        'the argument --retryTagFilter is deprecated and will be removed in a future release; please use --retry-tag-filter'
+      )
+    }
+  },
 }
 
 export default ArgvParser

src/cli/configuration_builder.ts

@@ -55,6 +55,7 @@ export default class ConfigurationBuilder {
   constructor({ argv, cwd }: INewConfigurationBuilderOptions) {
     this.cwd = cwd
 
+    ArgvParser.lint(argv)
     const parsedArgv = ArgvParser.parse(argv)
     this.args = parsedArgv.args
     this.options = parsedArgv.options

src/cli/index.ts

@@ -22,7 +22,6 @@ import { doesNotHaveValue } from '../value_checker'
 import Gherkin from 'gherkin'
 import { ISupportCodeLibrary } from '../support_code_library_builder/types'
 import { IParsedArgvFormatOptions } from './argv_parser'
-import { Console } from 'console'
 import { WriteStream } from 'fs'
 
 const { incrementing, uuid } = IdGenerator
@@ -50,45 +49,32 @@ export default class Cli {
   private readonly argv: string[]
   private readonly cwd: string
   private readonly stdout: IFormatterStream
-  private readonly console: Console
 
   constructor({
     argv,
     cwd,
     stdout,
-    stderr,
   }: {
     argv: string[]
     cwd: string
     stdout: IFormatterStream
-    stderr: IFormatterStream
   }) {
     this.argv = argv
     this.cwd = cwd
     this.stdout = stdout
-    this.console = new Console({ stdout, stderr })
   }
 
   async getConfiguration(): Promise<IConfiguration> {
     const fullArgv = await getExpandedArgv({
       argv: this.argv,
       cwd: this.cwd,
     })
-    this.lintArgv(fullArgv)
     return ConfigurationBuilder.build({
       argv: fullArgv,
       cwd: this.cwd,
     })
   }
 
-  private lintArgv(fullArgv: string[]): void {
-    if (fullArgv.includes('--retryTagFilter')) {
-      this.console.warn(
-        'the argument --retryTagFilter is deprecated and will be removed in a future release; please use --retry-tag-filter'
-      )
-    }
-  }
-
   async initializeFormatters({
     eventBroadcaster,
     eventDataCollector,
@@ -118,7 +104,7 @@ export default class Cli {
       }
       if (type === 'progress-bar' && !(stream as TtyWriteStream).isTTY) {
         const outputToName = outputTo === '' ? 'stdout' : outputTo
-        this.console.warn(
+        console.warn(
           `Cannot use 'progress-bar' formatter for output to '${outputToName}' as not a TTY. Switching to 'progress' formatter.`
         )
         type = 'progress'

src/cli/run.ts

@@ -12,7 +12,6 @@ export default async function run(): Promise<void> {
     argv: process.argv,
     cwd,
     stdout: process.stdout,
-    stderr: process.stderr,
   })
 
   let result: ICliRunResult