Update publish_command_test.dart

Update publish_command_test.dart

2

2

Update publish_command_test.dart

lint

Update publish_command.dart

Update release_from_batch_release.yml

Update release.yml

clean up code

Update publish_command.dart

1

Author: hannah-hyj

.github/workflows/release.yml

@@ -1,8 +1,14 @@
-name: release
+name: Reusable Release
+
 on:
-  push:
-    branches:
-      - main
+  workflow_call:
+    inputs:
+      publish-args:
+        required: true
+        type: string
+      workflow-name:
+        required: true
+        type: string
 
 # Declare default permissions as read only.
 permissions: read-all
@@ -82,5 +88,5 @@ jobs:
       run: |
         git config --global user.name ${{ secrets.USER_NAME }}
         git config --global user.email ${{ secrets.USER_EMAIL }}
-        dart ./script/tool/lib/src/main.dart publish --all-changed --base-sha=HEAD~ --skip-confirmation --remote=origin
+        dart ./script/tool/lib/src/main.dart publish ${{ inputs.publish-args }}
       env: {PUB_CREDENTIALS: "${{ secrets.PUB_CREDENTIALS }}"}

.github/workflows/release_from_go_router.yml

@@ -0,0 +1,12 @@
+name: Batch Release
+on:
+  push:
+    branches:
+      - release-go-router
+jobs:
+  release:
+    uses: ./.github/workflows/release.yml
+    with:
+      publish-args: '--all-changed --batch-release --base-sha=HEAD~ --skip-confirmation --remote=origin'
+      workflow-name: 'Batch Release'
+    secrets: inherit

.github/workflows/release_from_main.yml

@@ -0,0 +1,12 @@
+name: Main Release
+on:
+  push:
+    branches:
+      - main
+jobs:
+  release:
+    uses: ./.github/workflows/release.yml
+    with:
+      publish-args: '--all-changed --base-sha=HEAD~ --skip-confirmation --remote=origin'
+      workflow-name: 'Main Release'
+    secrets: inherit

script/tool/lib/src/publish_command.dart

@@ -83,6 +83,9 @@ class PublishCommand extends PackageLoopingCommand {
           'Release all packages that contains pubspec changes at the current commit compares to the base-sha.\n'
           'The --packages option is ignored if this is on.',
     );
+    argParser.addFlag(_batchReleaseFlag,
+        help:
+            'only release the packages that opt-in for batch release option.');
     argParser.addFlag(
       _dryRunFlag,
       help:
@@ -109,6 +112,7 @@ class PublishCommand extends PackageLoopingCommand {
   static const String _pubFlagsOption = 'pub-publish-flags';
   static const String _remoteOption = 'remote';
   static const String _allChangedFlag = 'all-changed';
+  static const String _batchReleaseFlag = 'batch-release';
   static const String _dryRunFlag = 'dry-run';
   static const String _skipConfirmationFlag = 'skip-confirmation';
   static const String _tagForAutoPublishFlag = 'tag-for-auto-publish';
@@ -196,6 +200,37 @@ class PublishCommand extends PackageLoopingCommand {
           .toList();
 
       for (final pubspecPath in changedPubspecs) {
+        // Read the ci_config.yaml file if it exists
+
+        final String packageName = p.basename(p.dirname(pubspecPath));
+        bool isBatchReleasePackage;
+        try {
+          final File ciConfigFile = packagesDir.fileSystem
+              .file(pubspecPath)
+              .parent
+              .childFile('ci_config.yaml');
+          if (!ciConfigFile.existsSync()) {
+            isBatchReleasePackage = false;
+          } else {
+            final YamlMap? ciConfig =
+                loadYaml(ciConfigFile.readAsStringSync()) as YamlMap?;
+            final dynamic batchValue = ciConfig?['release']?['batch'];
+            if (batchValue is! bool) {
+              printError(
+                  '`release.batch` key is missing or not a boolean in ci_config.yaml for $packageName.');
+              continue;
+            }
+            isBatchReleasePackage = batchValue;
+          }
+        } catch (e) {
+          printError('Could not parse ci_config.yaml for $packageName: $e');
+          continue;
+        }
+        // Skip the package if batch release flag is not set to match the ci_config.yaml
+        if (getBoolArg(_batchReleaseFlag) != isBatchReleasePackage) {
+          continue;
+        }
+
         // git outputs a relativa, Posix-style path.
         final File pubspecFile = childFileWithSubcomponents(
           packagesDir.fileSystem.directory((await gitDir).path),

script/tool/test/publish_command_test.dart

@@ -1284,6 +1284,56 @@ void main() {
       );
     });
 
+    group('--batch-release flag', () {
+      test('filters packages based on ci_config.yaml', () async {
+        // Mock packages.
+        final RepositoryPackage plugin1 = createFakePlugin(
+          'plugin1',
+          packagesDir,
+          version: '1.0.0',
+          batchRelease: true,
+        );
+
+        final RepositoryPackage plugin2 = createFakePlugin(
+          'plugin2',
+          packagesDir,
+          version: '1.0.0',
+          batchRelease: false,
+        );
+
+        // Mock git diff to show both packages have changed.
+        processRunner.mockProcessesForExecutable['git-diff'] = <FakeProcessInfo>[
+          FakeProcessInfo(MockProcess(
+              stdout:
+                  '${plugin1.pubspecFile.path}\n${plugin2.pubspecFile.path}'))
+        ];
+
+        // Mock pub.dev responses.
+        mockHttpResponses['plugin1'] = <String, dynamic>{
+          'name': 'plugin1',
+          'versions': <String>['0.0.1'],
+        };
+        mockHttpResponses['plugin2'] = <String, dynamic>{
+          'name': 'plugin2',
+          'versions': <String>['0.0.1'],
+        };
+
+        final List<String> output = await runCapturingPrint(
+            commandRunner, <String>[
+          'publish',
+          '--all-changed',
+          '--base-sha=HEAD~',
+          '--batch-release',
+          '--skip-confirmation'
+        ]);
+
+        expect(output, contains('Publishing plugin1 version 1.0.0'));
+        expect(output, isNot(contains('Publishing plugin2 version 1.0.0')));
+      });
+
+
+    });
+
     test('Do not release flutter_plugin_tools', () async {
       mockHttpResponses['plugin1'] = <String, dynamic>{
         'name': 'flutter_plugin_tools',

script/tool/test/util.dart

@@ -96,6 +96,7 @@ RepositoryPackage createFakePlugin(
   String? version = '0.0.1',
   String flutterConstraint = _defaultFlutterConstraint,
   String dartConstraint = _defaultDartConstraint,
+  bool? batchRelease,
 }) {
   final RepositoryPackage package = createFakePackage(
     name,
@@ -117,6 +118,9 @@ RepositoryPackage createFakePlugin(
     flutterConstraint: flutterConstraint,
     dartConstraint: dartConstraint,
   );
+  if(batchRelease != null){
+    createFakeCiConfig(batchRelease, package);
+  }
 
   return package;
 }
@@ -287,6 +291,21 @@ $pluginSection
   package.pubspecFile.writeAsStringSync(yaml);
 }
 
+/// Creates a `ci_config.yaml` file for [package].
+void createFakeCiConfig(
+  bool batchRelease,
+  RepositoryPackage package,
+  ){
+    final String yaml = '''
+release:
+  batch: $batchRelease
+''';
+
+  package.ciConfigFile.createSync();
+  package.ciConfigFile.writeAsStringSync(yaml);
+
+}
+
 String _pluginPlatformSection(
   String platform,
   PlatformDetails support,