Improve some exception messaging and comments (#3691)

Author: srawlins

lib/src/tool_runner.dart

@@ -98,12 +98,18 @@ class ToolRunner {
       var result =
           await Process.run(commandPath, args, environment: environment);
       if (result.exitCode != 0) {
-        toolErrorCallback('Tool "$name" returned non-zero exit code '
-            '(${result.exitCode}) when run as "${commandString()}" from '
-            '${pathContext.current}\n'
-            'Input to $name was:\n'
-            '$content\n'
-            'Stderr output was:\n${result.stderr}\n');
+        var envString =
+            environment.entries.map((e) => '${e.key}: ${e.value}').join(', ');
+        toolErrorCallback(
+          'Tool "$name" returned non-zero exit code '
+          '(${result.exitCode}) when run as "${commandString()}".\n'
+          '  Working directory: "${pathContext.current}"\n'
+          '  Env: $envString\n'
+          '  Input to $name was:\n'
+          '    $content\n'
+          '  Stderr output was:\n'
+          '    ${result.stderr}\n',
+        );
         return '';
       } else {
         return result.stdout as String;

lib/src/warnings.dart

@@ -218,7 +218,7 @@ enum PackageWarning implements Comparable<PackageWarning> {
   ),
   duplicateFile(
     'duplicate-file',
-    'failed to write file at: {0}',
+    'file already written at "{0}"',
     shortHelp:
         'Dartdoc is trying to write to a duplicate filename based on the names '
         'of Dart symbols.',

tool/src/flutter_repo.dart

@@ -55,7 +55,7 @@ class FlutterRepo {
     return FlutterRepo._(flutterPath, env, cacheDart, launcher);
   }
 
-  /// Copies an existing, initialized flutter repo.
+  /// Copies an existing, initialized flutter repo to [flutterPath].
   static Future<FlutterRepo> copyFromExistingFlutterRepo(
       FlutterRepo originalRepo, String flutterPath, Map<String, String> env,
       [String? label]) async {