[puppy] actually rename command to run, sort directories, better lo…

…gs (#337)
dart-lang · Jan 14, 2025 · 94b4cea · 94b4cea
1 parent f2d3178
commit 94b4cea
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 2 deletions.
diff --git a/pkgs/puppy/lib/src/run_command.dart b/pkgs/puppy/lib/src/run_command.dart
@@ -8,6 +8,7 @@ import 'dart:io';
 import 'package:args/command_runner.dart';
 import 'package:build_cli_annotations/build_cli_annotations.dart';
 import 'package:io/ansi.dart';
+import 'package:path/path.dart' as p;
 
 import 'constants.dart';
 import 'utils.dart';
@@ -21,7 +22,7 @@ class RunCommand extends _$RunArgsCommand<void> {
       '`pubspec.yaml`.';
 
   @override
-  String get name => 'map';
+  String get name => 'run';
 
   @override
   Future<void>? run() async {
@@ -33,8 +34,13 @@ class RunCommand extends _$RunArgsCommand<void> {
     final packages = findPackages(Directory.current, deep: args.deep);
     final exits = <String, int>{};
 
+    var count = 0;
     for (final packageDir in packages) {
-      print(green.wrap(packageDir.path));
+      final relative = p.relative(packageDir.path);
+
+      print(
+        green.wrap('$relative (${++count} of ${packages.length})'),
+      );
       final proc = await Process.start(
         exe,
         extraArgs,

diff --git a/pkgs/puppy/lib/src/utils.dart b/pkgs/puppy/lib/src/utils.dart
@@ -29,5 +29,7 @@ List<Directory> findPackages(Directory root, {bool deep = false}) {
 
   traverse(Directory.current, deep: deep);
 
+  results.sort((a, b) => a.path.compareTo(b.path));
+
   return results;
 }
diff --git a/pkgs/puppy/pubspec.yaml b/pkgs/puppy/pubspec.yaml
@@ -9,6 +9,7 @@ dependencies:
   args: ^2.6.0
   build_cli_annotations: ^2.1.0
   io: ^1.0.5
+  path: ^1.9.1
 
 dev_dependencies:
   build_cli: ^2.2.4