Merge branch 'main' into vgv-ai-bot/issue-360

Author: marcossevilla

.github/workflows/site.yaml

@@ -25,7 +25,7 @@ jobs:
         uses: actions/checkout@v7
 
       - name: ⚙️ Setup Node
-        uses: actions/setup-node@v6
+        uses: actions/setup-node@v7
         with:
           node-version: 20.x
           cache: npm

.github/workflows/site_deploy.yaml

@@ -19,7 +19,7 @@ jobs:
         uses: actions/checkout@v7
 
       - name: ⚙️ Setup Node
-        uses: actions/setup-node@v6
+        uses: actions/setup-node@v7
         with:
           node-version: 20.x
           cache: npm

lib/src/cli/git_cli.dart

@@ -25,6 +25,7 @@ class Git {
     try {
       await _Cmd.run('git', [
         'ls-remote',
+        '--get-url',
         '$remote',
         '--exit-code',
       ], logger: logger);

lib/src/commands/dart/commands/dart_test_command.dart

@@ -28,6 +28,7 @@ class DartTestOptions {
     required this.reportOn,
     required this.runSkipped,
     required this.checkIgnore,
+    required this.fileReporter,
   });
 
   /// Parses [ArgResults] into a [DartTestOptions] instance.
@@ -61,6 +62,7 @@ class DartTestOptions {
         .toList();
     final runSkipped = argResults['run-skipped'] as bool;
     final checkIgnore = argResults['check-ignore'] as bool;
+    final fileReporter = argResults['file-reporter'] as String?;
     final rest = argResults.rest;
 
     return DartTestOptions._(
@@ -80,6 +82,7 @@ class DartTestOptions {
       reportOn: reportOn,
       runSkipped: runSkipped,
       checkIgnore: checkIgnore,
+      fileReporter: fileReporter,
       rest: rest,
     );
   }
@@ -133,6 +136,10 @@ class DartTestOptions {
   /// Whether to check for and respect coverage ignore comments.
   final bool checkIgnore;
 
+  /// Additional reporter that writes test results to a file, expressed as
+  /// `<name>:<path>` (e.g. `json:reports/tests.json`).
+  final String? fileReporter;
+
   /// The remaining arguments passed to the `dart test` command.
   final List<String> rest;
 }
@@ -281,6 +288,13 @@ class DartTestCommand extends Command<int> {
         help:
             'Whether to check for and respect coverage ignore comments '
             '(e.g. // coverage:ignore-line).',
+      )
+      ..addOption(
+        'file-reporter',
+        help:
+            'Enable an additional reporter writing test results to a file. '
+            'Should be in the form <name>:<path> (e.g. "json:reports/tests.json").',
+        valueHelp: 'name:path',
       );
   }
 
@@ -347,6 +361,8 @@ This command should be run from the root of your Dart project.''');
             if (options.runSkipped) '--run-skipped',
             if (options.platform != null) ...['--platform', options.platform!],
             if (options.platform == null) ...['-j', options.concurrency],
+            if (options.fileReporter != null)
+              '--file-reporter=${options.fileReporter}',
             ...options.rest,
           ],
           reportOn: options.reportOn.isEmpty ? null : options.reportOn,

lib/src/commands/test/test.dart

@@ -32,6 +32,7 @@ class FlutterTestOptions {
     required this.runSkipped,
     required this.flavor,
     required this.timeout,
+    required this.fileReporter,
     required this.rest,
   });
 
@@ -151,6 +152,7 @@ class FlutterTestOptions {
     final effectiveTimeout = timeoutSeconds != null
         ? Duration(seconds: timeoutSeconds)
         : null;
+    final fileReporter = argResults['file-reporter'] as String?;
     final rest = argResults.rest;
 
     return FlutterTestOptions._(
@@ -174,6 +176,7 @@ class FlutterTestOptions {
       runSkipped: runSkipped,
       flavor: flavor,
       timeout: effectiveTimeout,
+      fileReporter: fileReporter,
       rest: rest,
     );
   }
@@ -240,6 +243,10 @@ class FlutterTestOptions {
   /// Maximum time to let tests run before killing the process.
   final Duration? timeout;
 
+  /// Additional reporter that writes test results to a file, expressed as
+  /// `<name>:<path>` (e.g. `json:reports/tests.json`).
+  final String? fileReporter;
+
   /// The remaining arguments passed to the test command.
   final List<String> rest;
 }
@@ -445,6 +452,13 @@ class TestCommand extends Command<int> {
             'Maximum seconds to let tests run before killing the process. '
             'Useful when tests hang due to an unbounded pumpAndSettle() call.',
         valueHelp: 'seconds',
+      )
+      ..addOption(
+        'file-reporter',
+        help:
+            'Enable an additional reporter writing test results to a file. '
+            'Should be in the form <name>:<path> (e.g. "json:reports/tests.json").',
+        valueHelp: 'name:path',
       );
   }
 
@@ -536,6 +550,8 @@ This command should be run from the root of your Flutter project.''');
             '--no-pub',
             if (options.timeout != null)
               '--timeout=${options.timeout!.inSeconds}s',
+            if (options.fileReporter != null)
+              '--file-reporter=${options.fileReporter}',
             ...options.rest,
           ],
         );

site/docs/commands/test.md

@@ -48,10 +48,25 @@ very_good test [arguments]
     --fail-fast                       Stop running tests after the first failure.
     --timeout=<seconds>               Maximum seconds to let tests run before killing the process.
                                       Useful when tests hang due to an unbounded pumpAndSettle() call.
+    --file-reporter=<name:path>       Enable an additional reporter writing test results to a file.
+                                      Should be in the form <name>:<path> (e.g. "json:reports/tests.json").
 
 Run "very_good help" to see global options.
 ```
 
+### Machine-readable test reports
+
+If you need a machine-readable summary of your test results (for example to feed a CI tool or convert to JUnit XML), use `--file-reporter`. It maps directly to the `--file-reporter` flag from `flutter test` and `dart test`, and writes the report to a file instead of stdout — so it stays compatible with `very_good test`'s optimization and progress rendering.
+
+```sh
+# Emits reports/tests.json alongside the normal test output.
+very_good test --coverage --file-reporter json:reports/tests.json
+```
+
+:::info
+`--reporter` and `--machine` are intentionally not exposed: they replace stdout with a machine-readable stream, which prevents `very_good test` from rendering progress and picking up failures. `--file-reporter` gives you the same machine-readable content without that trade-off.
+:::
+
 :::tip
 For **Dart** projects, use **`very_good dart test`** instead.