[flutter_plugin_tools] ignore flutter_plugin_tools when publishing

script/tool/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.3.0+1
+
+- `publish-plugins` ignores `flutter_plugin_tools`, see: https://github.com/flutter/flutter/issues/85430
+
 ## 0.3.0
 
 - Add a --build-id flag to `firebase-test-lab` instead of hard-coding the use of

script/tool/lib/src/publish_plugin_command.dart

@@ -290,6 +290,14 @@ Safe to ignore if the package is deleted in this commit.
     }
 
     final Pubspec pubspec = Pubspec.parse(pubspecFile.readAsStringSync());
+
+    if (pubspec.name == 'flutter_plugin_tools') {
+      // Ignore flutter_plugin_tools package when running publishing through flutter_plugin_tools.
+      // TODO(cyanglaz): Make the tool also auto publish flutter_plugin_tools package.
+      // https://github.com/flutter/flutter/issues/85430
+      return _CheckNeedsReleaseResult.noRelease;
+    }
+
     if (pubspec.publishTo == 'none') {
       return _CheckNeedsReleaseResult.noRelease;
     }

script/tool/pubspec.yaml

@@ -1,7 +1,7 @@
 name: flutter_plugin_tools
 description: Productivity utils for flutter/plugins and flutter/packages
 repository: https://github.com/flutter/plugins/tree/master/script/tool
-version: 0.3.0
+version: 0.3.0+1
 
 dependencies:
   args: ^2.1.0

script/tool/test/publish_plugin_command_test.dart

@@ -995,6 +995,57 @@ void main() {
           ]));
       expect(processRunner.pushTagsArgs, isEmpty);
     });
+
+    test('Do not release flutter_plugin_tools', () async {
+      const Map<String, dynamic> httpResponsePlugin1 = <String, dynamic>{
+        'name': 'flutter_plugin_tools',
+        'versions': <String>[],
+      };
+
+      final MockClient mockClient = MockClient((http.Request request) async {
+        if (request.url.pathSegments.last == 'flutter_plugin_tools.json') {
+          return http.Response(json.encode(httpResponsePlugin1), 200);
+        }
+        return http.Response('', 500);
+      });
+      final PublishPluginCommand command = PublishPluginCommand(packagesDir,
+          processRunner: processRunner,
+          print: (Object? message) => printedMessages.add(message.toString()),
+          stdinput: mockStdin,
+          httpClient: mockClient,
+          gitDir: gitDir);
+
+      commandRunner = CommandRunner<void>(
+        'publish_check_command',
+        'Test for publish-check command.',
+      );
+      commandRunner.addCommand(command);
+
+      final Directory flutterPluginTools =
+          createFakePlugin('flutter_plugin_tools', packagesDir);
+      await gitDir.runCommand(<String>['add', '-A']);
+      await gitDir.runCommand(<String>['commit', '-m', 'Add plugins']);
+      // Immediately return 0 when running `pub publish`.
+      processRunner.mockPublishCompleteCode = 0;
+      mockStdin.readLineOutput = 'y';
+      await commandRunner
+          .run(<String>['publish-plugin', '--all-changed', '--base-sha=HEAD~']);
+      expect(
+          printedMessages,
+          containsAllInOrder(<String>[
+            'Checking local repo...',
+            'Local repo is ready!',
+            'Done!'
+          ]));
+      expect(
+          printedMessages.contains(
+            'Running `pub publish ` in ${flutterPluginTools.path}...\n',
+          ),
+          isFalse);
+      expect(processRunner.pushTagsArgs, isEmpty);
+      processRunner.pushTagsArgs.clear();
+      printedMessages.clear();
+    });
   });
 }