review notes

pkgs/puppy/README.md

@@ -5,7 +5,7 @@ cd <puppy pkg path>
 dart pub global activate --source=path .
 ```
 
-### Tools
+### Commands
 
-- `for_all_package_dirs`: runs a command in every directory containing
+- `run`: runs a command in every directory containing
   `pubspec.yaml`.

pkgs/puppy/bin/puppy.dart

@@ -0,0 +1,10 @@
+import 'package:args/command_runner.dart';
+import 'package:puppy/src/map_command.dart';
+
+Future<void> main(List<String> args) async {
+  var runner = CommandRunner<void>(
+      'dgit', 'A dart implementation of distributed version control.')
+    ..addCommand(MapCommand());
+
+  await runner.run(args);
+}

pkgs/puppy/lib/src/map_command.dart

@@ -0,0 +1,86 @@
+// Copyright (c) 2025, the Dart project authors. Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+
+import 'dart:async';
+import 'dart:io';
+
+import 'package:args/command_runner.dart';
+import 'package:build_cli_annotations/build_cli_annotations.dart';
+import 'package:io/ansi.dart';
+
+part 'map_command.g.dart';
+
+class MapCommand extends _$MapArgsCommand<void> {
+  @override
+  String get description =>
+      'Run the provided command in each subdirectory containing '
+      '`pubspec.yaml`.';
+
+  @override
+  String get name => 'map';
+
+  @override
+  Future<void>? run() async {
+    await _doMap(_options);
+  }
+}
+
+@CliOptions(createCommand: true)
+class MapArgs {
+  @CliOption(abbr: 'd', help: 'Keep looking for "nested" pubspec files.')
+  final bool deep;
+
+  final List<String> rest;
+
+  MapArgs({
+    this.deep = false,
+    required this.rest,
+  }) {
+    if (rest.isEmpty) {
+      throw UsageException(
+        'Missing command to invoke!',
+        'puppy map [--deep] <command to invoke>',
+      );
+    }
+  }
+}
+
+Future<void> _doMap(MapArgs args) async {
+  final exe = args.rest.first;
+  final extraArgs = args.rest.skip(1).toList();
+
+  final exits = <String, int>{};
+
+  Future<void> inspectDirectory(Directory dir, {required bool deep}) async {
+    final pubspecs = dir
+        .listSync()
+        .whereType<File>()
+        .where((element) => element.uri.pathSegments.last == 'pubspec.yaml')
+        .toList();
+
+    final pubspecHere = pubspecs.isNotEmpty;
+    if (pubspecHere) {
+      print(green.wrap(dir.path));
+      final proc = await Process.start(
+        exe,
+        extraArgs,
+        mode: ProcessStartMode.inheritStdio,
+        workingDirectory: dir.path,
+      );
+
+      // TODO(kevmoo): display a summary of results on completion
+      exits[dir.path] = await proc.exitCode;
+    }
+
+    if (!pubspecHere || deep) {
+      for (var subDir in dir.listSync().whereType<Directory>().where(
+          (element) => !element.uri.pathSegments
+              .any((element) => element.startsWith('.')))) {
+        await inspectDirectory(subDir, deep: deep);
+      }
+    }
+  }
+
+  await inspectDirectory(Directory.current, deep: args.deep);
+}

pkgs/puppy/pubspec.yaml

@@ -5,10 +5,14 @@ environment:
   sdk: ^3.6.0
 
 dependencies:
+  args: ^2.6.0
+  build_cli_annotations: ^2.1.0
   io: ^1.0.5
 
 dev_dependencies:
+  build_cli: ^2.2.4
+  build_runner: ^2.4.14
   dart_flutter_team_lints: ^3.0.0
 
 executables:
-  for_all_package_dirs:
+  puppy: