[dartdev] refactor the 'dart migrate' package:args command

Change-Id: Id9f2a5289b75bb26258925f7686d32f4f6a0eb51
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/199760
Reviewed-by: Ben Konyi <[email protected]>
Reviewed-by: Ivan Inozemtsev <[email protected]>
Commit-Queue: Devon Carew <[email protected]>

Author: devoncarew

pkg/nnbd_migration/lib/migration_cli.dart

@@ -21,8 +21,8 @@ import 'package:analyzer/src/util/sdk.dart';
 import 'package:analyzer_plugin/protocol/protocol_common.dart'
     hide AnalysisError;
 import 'package:args/args.dart';
-import 'package:args/command_runner.dart';
 import 'package:cli_util/cli_logging.dart';
+import 'package:dartdev/src/core.dart';
 import 'package:meta/meta.dart';
 import 'package:nnbd_migration/src/edit_plan.dart';
 import 'package:nnbd_migration/src/exceptions.dart';
@@ -137,8 +137,7 @@ class CommandLineOptions {
       @required this.webPreview});
 }
 
-// TODO(devoncarew): Refactor so this class extends DartdevCommand.
-class MigrateCommand extends Command<int> {
+class MigrateCommand extends DartdevCommand {
   static const String cmdName = 'migrate';
 
   static const String cmdDescription =
@@ -152,31 +151,16 @@ class MigrateCommand extends Command<int> {
 
   final bool verbose;
 
-  ArgParser _argParser;
-
-  MigrateCommand({this.verbose = false}) {
+  MigrateCommand({this.verbose = false})
+      : super(cmdName, '$cmdDescription\n\n$migrationGuideLink') {
     MigrationCli._defineOptions(argParser, !verbose);
   }
 
-  @override
-  ArgParser get argParser {
-    // We override this in order to configure the help text line wrapping.
-    return _argParser ??= ArgParser(
-      usageLineLength: stdout.hasTerminal ? stdout.terminalColumns : null,
-    );
-  }
-
-  @override
-  String get description => '$cmdDescription\n\n$migrationGuideLink';
-
   @override
   String get invocation {
     return '${super.invocation} [project or directory]';
   }
 
-  @override
-  String get name => cmdName;
-
   @override
   FutureOr<int> run() async {
     var cli = MigrationCli(binaryName: 'dart $name');

pkg/nnbd_migration/pubspec.yaml

@@ -18,6 +18,8 @@ dependencies:
   cli_util: ^0.2.0
   collection: ^1.14.11
   crypto: ^2.0.6
+  dartdev:
+    path: ../dartdev
   meta:
     path: ../meta
   path: ^1.6.2